qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH] migration: use qemu_free() instead of free().
@ 2010-06-09  5:44 Yoshiaki Tamura
  2010-06-14 20:58 ` Anthony Liguori
  0 siblings, 1 reply; 2+ messages in thread
From: Yoshiaki Tamura @ 2010-06-09  5:44 UTC (permalink / raw)
  To: qemu-devel; +Cc: aliguori, Yoshiaki Tamura

Although there is no difference, other migration related code use
qemu_free(), and it should be better to be consistent.

Signed-off-by: Yoshiaki Tamura <tamura.yoshiaki@lab.ntt.co.jp>
---
 migration.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/migration.c b/migration.c
index fbf2339..64ed36e 100644
--- a/migration.c
+++ b/migration.c
@@ -396,7 +396,7 @@ void migrate_fd_release(MigrationState *mig_state)
         s->state = MIG_STATE_CANCELLED;
         migrate_fd_cleanup(s);
     }
-    free(s);
+    qemu_free(s);
 }
 
 void migrate_fd_wait_for_unfreeze(void *opaque)
-- 
1.7.0.31.g1df487

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [Qemu-devel] [PATCH] migration: use qemu_free() instead of free().
  2010-06-09  5:44 [Qemu-devel] [PATCH] migration: use qemu_free() instead of free() Yoshiaki Tamura
@ 2010-06-14 20:58 ` Anthony Liguori
  0 siblings, 0 replies; 2+ messages in thread
From: Anthony Liguori @ 2010-06-14 20:58 UTC (permalink / raw)
  To: Yoshiaki Tamura; +Cc: aliguori, qemu-devel

On 06/09/2010 12:44 AM, Yoshiaki Tamura wrote:
> Although there is no difference, other migration related code use
> qemu_free(), and it should be better to be consistent.
>
> Signed-off-by: Yoshiaki Tamura<tamura.yoshiaki@lab.ntt.co.jp>
>    

Applied.  Thanks.

Regards,

Anthony Liguori
> ---
>   migration.c |    2 +-
>   1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/migration.c b/migration.c
> index fbf2339..64ed36e 100644
> --- a/migration.c
> +++ b/migration.c
> @@ -396,7 +396,7 @@ void migrate_fd_release(MigrationState *mig_state)
>           s->state = MIG_STATE_CANCELLED;
>           migrate_fd_cleanup(s);
>       }
> -    free(s);
> +    qemu_free(s);
>   }
>
>   void migrate_fd_wait_for_unfreeze(void *opaque)
>    

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2010-06-14 20:59 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-06-09  5:44 [Qemu-devel] [PATCH] migration: use qemu_free() instead of free() Yoshiaki Tamura
2010-06-14 20:58 ` Anthony Liguori

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).