Re: [Qemu-devel] [PATCH] Introduce QEMU_NEW()

Paolo Bonzini
2011-07-25T15:21:32Z

On 07/25/2011 04:56 PM, Blue Swirl wrote:
> > qemu_malloc uses g_malloc => you keep tracepoints, you just do not trace
> > memory allocated by glib
>
> Unless the plan is to replace all qemu_malloc() calls with calls to g_malloc().

We can worry when the day comes... there is already another task
blocking that replacement (balancing qemu_malloc with qemu_free).

> > The former is much less code, however it requires qemu_malloc to be always
> > balanced with qemu_free (patches ready and on my github tree, won't be sent
> > before KVM Forum though...).
>
> Freeing qemu_malloc() memory with plain free() is a bug.

We have many bugs, then. :)

Paolo