[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 05/32] qtest: Tidy up temporary files properly
From: |
Blue Swirl |
Subject: |
Re: [Qemu-devel] [PATCH 05/32] qtest: Tidy up temporary files properly |
Date: |
Sat, 7 Jul 2012 07:39:31 +0000 |
On Fri, Jul 6, 2012 at 6:57 AM, Markus Armbruster <address@hidden> wrote:
> Each test litters /tmp with several files: a pid file and two
> sockets. Tidy up.
>
> Signed-off-by: Markus Armbruster <address@hidden>
While a nice patch, I don't think it should belong to this series.
> ---
> tests/libqtest.c | 29 ++++++++++++++++-------------
> 1 files changed, 16 insertions(+), 13 deletions(-)
>
> diff --git a/tests/libqtest.c b/tests/libqtest.c
> index 071b6be..02d0392 100644
> --- a/tests/libqtest.c
> +++ b/tests/libqtest.c
> @@ -40,6 +40,7 @@ struct QTestState
> bool irq_level[MAX_IRQ];
> GString *rx;
> gchar *pid_file;
> + char *socket_path, *qmp_socket_path;
> };
>
> #define g_assert_no_errno(ret) do { \
> @@ -88,8 +89,6 @@ QTestState *qtest_init(const char *extra_args)
> {
> QTestState *s;
> int sock, qmpsock, ret, i;
> - gchar *socket_path;
> - gchar *qmp_socket_path;
> gchar *pid_file;
> gchar *command;
> const char *qemu_binary;
> @@ -98,14 +97,14 @@ QTestState *qtest_init(const char *extra_args)
> qemu_binary = getenv("QTEST_QEMU_BINARY");
> g_assert(qemu_binary != NULL);
>
> - socket_path = g_strdup_printf("/tmp/qtest-%d.sock", getpid());
> - qmp_socket_path = g_strdup_printf("/tmp/qtest-%d.qmp", getpid());
> - pid_file = g_strdup_printf("/tmp/qtest-%d.pid", getpid());
> -
> s = g_malloc(sizeof(*s));
>
> - sock = init_socket(socket_path);
> - qmpsock = init_socket(qmp_socket_path);
> + s->socket_path = g_strdup_printf("/tmp/qtest-%d.sock", getpid());
> + s->qmp_socket_path = g_strdup_printf("/tmp/qtest-%d.qmp", getpid());
> + pid_file = g_strdup_printf("/tmp/qtest-%d.pid", getpid());
> +
> + sock = init_socket(s->socket_path);
> + qmpsock = init_socket(s->qmp_socket_path);
>
> pid = fork();
> if (pid == 0) {
> @@ -115,8 +114,8 @@ QTestState *qtest_init(const char *extra_args)
> "-qmp unix:%s,nowait "
> "-pidfile %s "
> "-machine accel=qtest "
> - "%s", qemu_binary, socket_path,
> - qmp_socket_path, pid_file,
> + "%s", qemu_binary, s->socket_path,
> + s->qmp_socket_path, pid_file,
> extra_args ?: "");
>
> ret = system(command);
> @@ -133,9 +132,6 @@ QTestState *qtest_init(const char *extra_args)
> s->irq_level[i] = false;
> }
>
> - g_free(socket_path);
> - g_free(qmp_socket_path);
> -
> /* Read the QMP greeting and then do the handshake */
> qtest_qmp(s, "");
> qtest_qmp(s, "{ 'execute': 'qmp_capabilities' }");
> @@ -160,6 +156,13 @@ void qtest_quit(QTestState *s)
>
> fclose(f);
> }
> +
> + unlink(s->pid_file);
> + unlink(s->socket_path);
> + unlink(s->qmp_socket_path);
> + g_free(s->pid_file);
> + g_free(s->socket_path);
> + g_free(s->qmp_socket_path);
> }
>
> static void socket_sendf(int fd, const char *fmt, va_list ap)
> --
> 1.7.6.5
>
- [Qemu-devel] [PATCH 00/32] Disk geometry cleanup, Markus Armbruster, 2012/07/06
- [Qemu-devel] [PATCH 16/32] blockdev: Save geometry in DriveInfo, Markus Armbruster, 2012/07/06
- [Qemu-devel] [PATCH 10/32] hd-geometry: Unnest conditional in hd_geometry_guess(), Markus Armbruster, 2012/07/06
- [Qemu-devel] [PATCH 14/32] hd-geometry: Cut out block layer translation middleman, Markus Armbruster, 2012/07/06
- [Qemu-devel] [PATCH 19/32] scsi-hd: qdev properties for disk geometry, Markus Armbruster, 2012/07/06
- [Qemu-devel] [PATCH 13/32] hd-geometry: Clean up confusing use of prior translation hint, Markus Armbruster, 2012/07/06
- [Qemu-devel] [PATCH 04/32] vvfat: Do not clobber the user's geometry, Markus Armbruster, 2012/07/06
- [Qemu-devel] [PATCH 05/32] qtest: Tidy up temporary files properly, Markus Armbruster, 2012/07/06
- Re: [Qemu-devel] [PATCH 05/32] qtest: Tidy up temporary files properly,
Blue Swirl <=
- [Qemu-devel] [PATCH 06/32] qtest: Add hard disk geometry test, Markus Armbruster, 2012/07/06
- [Qemu-devel] [PATCH 11/32] hd-geometry: Factor out guess_chs_for_size(), Markus Armbruster, 2012/07/06
- [Qemu-devel] [PATCH 22/32] qtest: Cover qdev properties for disk geometry, Markus Armbruster, 2012/07/06
- [Qemu-devel] [PATCH 03/32] vvfat: Fix partition table, Markus Armbruster, 2012/07/06
- [Qemu-devel] [PATCH 01/32] fdc: Drop broken code for user-defined floppy geometry, Markus Armbruster, 2012/07/06
- [Qemu-devel] [PATCH 12/32] hd-geometry: Clean up gratuitous goto in hd_geometry_guess(), Markus Armbruster, 2012/07/06
- [Qemu-devel] [PATCH 17/32] qdev: Introduce block geometry properties, Markus Armbruster, 2012/07/06
- [Qemu-devel] [PATCH 31/32] blockdev: Drop redundant CHS validation for if=ide, Markus Armbruster, 2012/07/06