[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-trivial] [PATCH v3 0/8] Add a member in MachineState to store

From: Michael Tokarev
Subject: Re: [Qemu-trivial] [PATCH v3 0/8] Add a member in MachineState to store irq array
Date: Wed, 17 Jun 2015 10:30:01 +0300
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Icedove/31.7.0

17.06.2015 05:59, Shannon Zhao wrote:
> From: Shannon Zhao <address@hidden>
> These are relevant to memory leak in machine init function. Here we add
> a member in MachineState to store irq array returned from
> qemu_allocate_irqs.
> PS. These patches are split from my previous patchset [1] since they are
> relevant to MachineState.
> Thanks,
> Shannon
> [1] [PATCH 00/29] Fix memory leak relevant to calling qemu_allocate_irqs
> changes since v2:
>   * Rebased on QEMU upstream
>   * Add a member in MachineState first, then fix the memory leak [mjt]

Um.  This is not what I asked.  It was a small suggestion, to add
infrastructure first use it in subsequent patches.

But the main question was what this infrastructure, what this patchset
actually brings us?  You just store the irq pointer in machine state,
that's okay, but WHAT FOR?  If your intention is to free resources,
this intention isn't met.  The end result of this whole change is a no-op,
there's nothing changed, the memory which has been allocated isn't being

So the actual question is: what's the point of this patchset?



reply via email to

[Prev in Thread] Current Thread [Next in Thread]