[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/7] balloon: cleanups, fix segfault
From: |
Amit Shah |
Subject: |
Re: [Qemu-devel] [PATCH 0/7] balloon: cleanups, fix segfault |
Date: |
Mon, 25 Jul 2011 21:19:38 +0530 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On (Mon) 25 Jul 2011 [16:13:33], Markus Armbruster wrote:
> Amit Shah <address@hidden> writes:
>
> > 'balloon 0' in the monitor causes a segfault. This happens because
> > the function that handles change in balloon values is also tasked with
> > handling stats retrieval from the guest, and it does the retrieval
> > when the balloon target is '0'.
> >
> > Silly API, replace it. Also do a few cleanups along the way.
>
> Silly indeed.
>
> Consider explaining the crash in 5/7's commit message.
Thanks for the detailed comment; will add that to the changelog.
> Reviewed-by: Markus Armbruster <address@hidden>
Thanks!
Amit
- [Qemu-devel] [PATCH 2/7] balloon: Add braces around if statements, (continued)
- [Qemu-devel] [PATCH 6/7] balloon: Fix header comment; add Copyright, Amit Shah, 2011/07/20
- [Qemu-devel] [PATCH 7/7] virtio-balloon: Fix header comment; add Copyright, Amit Shah, 2011/07/20
- Re: [Qemu-devel] [PATCH 0/7] balloon: cleanups, fix segfault, Markus Armbruster, 2011/07/25
- Re: [Qemu-devel] [PATCH 0/7] balloon: cleanups, fix segfault,
Amit Shah <=