[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 3/6] qga: move w32 service handling out of run_agent
From: |
Michael Roth |
Subject: |
[Qemu-devel] [PATCH 3/6] qga: move w32 service handling out of run_agent() |
Date: |
Thu, 26 Oct 2017 18:30:51 -0500 |
Eventually we want a w32 service to be able to restart the qga main
loop from within service_main(). To allow for this we move service
handling out of run_agent() such that service_main() calls
run_agent() instead of the reverse.
Signed-off-by: Michael Roth <address@hidden>
---
qga/main.c | 25 ++++++++++++++-----------
1 file changed, 14 insertions(+), 11 deletions(-)
diff --git a/qga/main.c b/qga/main.c
index 6826cd5f45..2422ccf2f8 100644
--- a/qga/main.c
+++ b/qga/main.c
@@ -133,6 +133,7 @@ DWORD WINAPI service_ctrl_handler(DWORD ctrl, DWORD type,
LPVOID data,
LPVOID ctx);
VOID WINAPI service_main(DWORD argc, TCHAR *argv[]);
#endif
+static int run_agent(GAState *s);
static void
init_dfl_pathnames(void)
@@ -768,7 +769,7 @@ VOID WINAPI service_main(DWORD argc, TCHAR *argv[])
service->status.dwWaitHint = 0;
SetServiceStatus(service->status_handle, &service->status);
- g_main_loop_run(ga_state->main_loop);
+ run_agent(ga_state);
service->status.dwCurrentState = SERVICE_STOPPED;
SetServiceStatus(service->status_handle, &service->status);
@@ -1377,17 +1378,8 @@ static int run_agent(GAState *s)
g_critical("failed to initialize guest agent channel");
return EXIT_FAILURE;
}
-#ifndef _WIN32
+
g_main_loop_run(ga_state->main_loop);
-#else
- if (config->daemonize) {
- SERVICE_TABLE_ENTRY service_table[] = {
- { (char *)QGA_SERVICE_NAME, service_main }, { NULL, NULL } };
- StartServiceCtrlDispatcher(service_table);
- } else {
- g_main_loop_run(ga_state->main_loop);
- }
-#endif
return EXIT_SUCCESS;
}
@@ -1473,7 +1465,18 @@ int main(int argc, char **argv)
g_critical("error initializing guest agent");
goto end;
}
+
+#ifdef _WIN32
+ if (config->daemonize) {
+ SERVICE_TABLE_ENTRY service_table[] = {
+ { (char *)QGA_SERVICE_NAME, service_main }, { NULL, NULL } };
+ StartServiceCtrlDispatcher(service_table);
+ } else {
+ ret = run_agent(s);
+ }
+#endif
ret = run_agent(s);
+
cleanup_agent(s);
end:
--
2.11.0
- [Qemu-devel] [PATCH 0/6] qga: add support for re-opening channel on error, Michael Roth, 2017/10/26
- [Qemu-devel] [PATCH 3/6] qga: move w32 service handling out of run_agent(),
Michael Roth <=
- [Qemu-devel] [PATCH 5/6] qga-win: install service with --retry-path set by default, Michael Roth, 2017/10/26
- [Qemu-devel] [PATCH 4/6] qga: add --retry-path option for re-initializing channel on failure, Michael Roth, 2017/10/26
- [Qemu-devel] [PATCH 1/6] qga: group agent init/cleanup init separate routines, Michael Roth, 2017/10/26
- [Qemu-devel] [PATCH 6/6] qga-win: report specific error when failing to open channel, Michael Roth, 2017/10/26
- [Qemu-devel] [PATCH 2/6] qga: hang GAConfig/socket_activation off of GAState global, Michael Roth, 2017/10/26
- Re: [Qemu-devel] [PATCH 0/6] qga: add support for re-opening channel on error, no-reply, 2017/10/26