[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 07/30] migration: make writes blocking
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 07/30] migration: make writes blocking |
Date: |
Mon, 12 Nov 2012 12:52:36 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:16.0) Gecko/20121016 Thunderbird/16.0.1 |
Il 18/10/2012 09:30, Juan Quintela ha scritto:
> @@ -247,10 +247,6 @@ static int migrate_fd_cleanup(MigrationState *s)
> {
> int ret = 0;
>
> - if (s->fd != -1) {
> - qemu_set_fd_handler2(s->fd, NULL, NULL, NULL, NULL);
> - }
> -
> if (s->file) {
> DPRINTF("closing file\n");
> ret = qemu_fclose(s->file);
While reviewing this patch, I noticed that migrate_fd_cleanup calls
migrate_fd_close twice. The first via qemu_fclose's own call to
buffered_close, the second directly.
Perhaps you can include something like
diff --git a/migration.c b/migration.c
--- a/migration.c
+++ b/migration.c
@@ -274,5 +274,5 @@ static int migrate_fd_cleanup(MigrationState *s)
}
- migrate_fd_close(s);
+ assert(s->fd == -1);
return ret;
}
somewhere in the series?
Paolo
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [PATCH 07/30] migration: make writes blocking,
Paolo Bonzini <=