Re: [Qemu-devel] [PATCH] Fix hanging user monitor when using balloon command

2010-03-08 Thread Anthony Liguori
On 02/22/2010 10:51 AM, Adam Litke wrote: This patch application failed. My patch adds a cb() call in do_balloon(), but the change in git has added the cb() call to do_info_balloon(). That is causing qemu segfaults. Applying the following should correct the damage. Thanks. Fix for commit: 5c3

Re: [Qemu-devel] [PATCH] Fix hanging user monitor when using balloon command

2010-02-26 Thread Adam Litke
On Fri, 2010-02-26 at 17:26 -0300, Luiz Capitulino wrote: > This patch fixes both. One question, though: > > > @@ -2332,6 +2331,7 @@ static int do_balloon(Monitor *mon, const QDict > > *params, > > return -1; > > } > > > > +cb(opaque, NULL); > > return 0; > > } > > Ca

Re: [Qemu-devel] [PATCH] Fix hanging user monitor when using balloon command

2010-02-26 Thread Luiz Capitulino
On Mon, 22 Feb 2010 10:51:20 -0600 Adam Litke wrote: > On Fri, 2010-02-19 at 15:47 -0600, Anthony Liguori wrote: > > On 02/12/2010 02:55 PM, Adam Litke wrote: > > > Arghh... Adding missing S-O-B > > > > > > Hi Anthony. I wonder if there was a problem when importing my async > > > command handler

Re: [Qemu-devel] [PATCH] Fix hanging user monitor when using balloon command

2010-02-22 Thread Adam Litke
On Fri, 2010-02-19 at 15:47 -0600, Anthony Liguori wrote: > On 02/12/2010 02:55 PM, Adam Litke wrote: > > Arghh... Adding missing S-O-B > > > > Hi Anthony. I wonder if there was a problem when importing my async > > command handler patchset. Since the 'balloon' command completes > > immediately,

Re: [Qemu-devel] [PATCH] Fix hanging user monitor when using balloon command

2010-02-19 Thread Anthony Liguori
On 02/12/2010 02:55 PM, Adam Litke wrote: Arghh... Adding missing S-O-B Hi Anthony. I wonder if there was a problem when importing my async command handler patchset. Since the 'balloon' command completes immediately, it must call the completion callback before returning. That call was missing

Re: [Qemu-devel] [PATCH] Fix hanging user monitor when using balloon command

2010-02-16 Thread Daniel P. Berrange
On Fri, Feb 12, 2010 at 02:55:56PM -0600, Adam Litke wrote: > Arghh... Adding missing S-O-B > > Hi Anthony. I wonder if there was a problem when importing my async > command handler patchset. Since the 'balloon' command completes > immediately, it must call the completion callback before returni

[Qemu-devel] [PATCH] Fix hanging user monitor when using balloon command

2010-02-12 Thread Adam Litke
Arghh... Adding missing S-O-B Hi Anthony. I wonder if there was a problem when importing my async command handler patchset. Since the 'balloon' command completes immediately, it must call the completion callback before returning. That call was missing but is added by the patch below. Signed-off

[Qemu-devel] [PATCH] Fix hanging user monitor when using balloon command

2010-02-12 Thread Adam Litke
Hi Anthony. I wonder if there was a problem when importing my async command handler patchset. Since the 'balloon' command completes immediately, it must call the completion callback before returning. That call was missing but is added by the patch below. diff --git a/monitor.c b/monitor.c index