[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 08/12] qga: drop guest_file_init helper and replace
From: |
Michael Roth |
Subject: |
[Qemu-devel] [PATCH 08/12] qga: drop guest_file_init helper and replace it with static initializers |
Date: |
Thu, 15 Oct 2015 11:05:14 -0500 |
From: "Denis V. Lunev" <address@hidden>
This just makes code shorter and better.
Signed-off-by: Denis V. Lunev <address@hidden>
Signed-off-by: Yuri Pudgorodskiy <address@hidden>
Reviewed-by: Michael Roth <address@hidden>
Signed-off-by: Denis V. Lunev <address@hidden>
Signed-off-by: Michael Roth <address@hidden>
---
qga/commands-posix.c | 10 +++-------
qga/commands-win32.c | 10 +++-------
2 files changed, 6 insertions(+), 14 deletions(-)
diff --git a/qga/commands-posix.c b/qga/commands-posix.c
index a932809..03b0f27 100644
--- a/qga/commands-posix.c
+++ b/qga/commands-posix.c
@@ -223,7 +223,9 @@ typedef struct GuestFileHandle {
static struct {
QTAILQ_HEAD(, GuestFileHandle) filehandles;
-} guest_file_state;
+} guest_file_state = {
+ .filehandles = QTAILQ_HEAD_INITIALIZER(guest_file_state.filehandles),
+};
static int64_t guest_file_handle_add(FILE *fh, Error **errp)
{
@@ -586,11 +588,6 @@ void qmp_guest_file_flush(int64_t handle, Error **errp)
}
}
-static void guest_file_init(void)
-{
- QTAILQ_INIT(&guest_file_state.filehandles);
-}
-
/* linux-specific implementations. avoid this if at all possible. */
#if defined(__linux__)
@@ -2486,5 +2483,4 @@ void ga_command_state_init(GAState *s, GACommandState *cs)
#if defined(CONFIG_FSFREEZE)
ga_command_state_add(cs, NULL, guest_fsfreeze_cleanup);
#endif
- ga_command_state_add(cs, guest_file_init, NULL);
}
diff --git a/qga/commands-win32.c b/qga/commands-win32.c
index 0481646..d9de23b 100644
--- a/qga/commands-win32.c
+++ b/qga/commands-win32.c
@@ -55,7 +55,9 @@ typedef struct GuestFileHandle {
static struct {
QTAILQ_HEAD(, GuestFileHandle) filehandles;
-} guest_file_state;
+} guest_file_state = {
+ .filehandles = QTAILQ_HEAD_INITIALIZER(guest_file_state.filehandles),
+};
typedef struct OpenFlags {
@@ -390,11 +392,6 @@ void qmp_guest_file_flush(int64_t handle, Error **errp)
}
}
-static void guest_file_init(void)
-{
- QTAILQ_INIT(&guest_file_state.filehandles);
-}
-
#ifdef CONFIG_QGA_NTDDSCSI
static STORAGE_BUS_TYPE win2qemu[] = {
@@ -1330,5 +1327,4 @@ void ga_command_state_init(GAState *s, GACommandState *cs)
if (!vss_initialized()) {
ga_command_state_add(cs, NULL, guest_fsfreeze_cleanup);
}
- ga_command_state_add(cs, guest_file_init, NULL);
}
--
1.9.1
- [Qemu-devel] [PATCH 11/12] qga: handle G_IO_STATUS_AGAIN in ga_channel_write_all(), (continued)
- [Qemu-devel] [PATCH 11/12] qga: handle G_IO_STATUS_AGAIN in ga_channel_write_all(), Michael Roth, 2015/10/15
- [Qemu-devel] [PATCH 04/12] qga: do not override configuration verbosity, Michael Roth, 2015/10/15
- [Qemu-devel] [PATCH 05/12] qtest: add a few fd-level qmp helpers, Michael Roth, 2015/10/15
- [Qemu-devel] [PATCH 12/12] qga: guest-exec simple stdin/stdout/stderr redirection, Michael Roth, 2015/10/15
- [Qemu-devel] [PATCH 06/12] glib-compat: add 2.38/2.40/2.46 asserts, Michael Roth, 2015/10/15
- [Qemu-devel] [PATCH 07/12] tests: add a local test for guest agent, Michael Roth, 2015/10/15
- [Qemu-devel] [PATCH 10/12] qga: handle possible SIGPIPE in guest-file-write, Michael Roth, 2015/10/15
- [Qemu-devel] [PATCH 09/12] qga: guest exec functionality, Michael Roth, 2015/10/15
- [Qemu-devel] [PATCH 02/12] qga: Use g_new() & friends where that makes obvious sense, Michael Roth, 2015/10/15
- [Qemu-devel] [PATCH 01/12] build: qemu-ga: add 'qemu-ga' build target for w32, Michael Roth, 2015/10/15
- [Qemu-devel] [PATCH 08/12] qga: drop guest_file_init helper and replace it with static initializers,
Michael Roth <=
- Re: [Qemu-devel] [PULL v2 00/12] qemu-ga patch queue, Peter Maydell, 2015/10/16