Re: [Qemu-devel] [PATCH] monitor: move json init from OPEN event to init

2012-08-23 Thread Max Filippov
On Fri, Aug 24, 2012 at 2:06 AM, Max Filippov wrote: > On Thu, Aug 23, 2012 at 5:22 PM, Anthony Liguori wrote: >> At some point in the past, the OPEN event was changed to be issued from a >> bottom half. This creates a small window whereas a data callback registered >> in >> init may be invoked

Re: [Qemu-devel] [PATCH] monitor: move json init from OPEN event to init

2012-08-23 Thread Max Filippov
On Thu, Aug 23, 2012 at 5:22 PM, Anthony Liguori wrote: > At some point in the past, the OPEN event was changed to be issued from a > bottom half. This creates a small window whereas a data callback registered > in > init may be invoked before the OPEN event has been issued. > > This is reproduc

Re: [Qemu-devel] [PATCH] monitor: move json init from OPEN event to init

2012-08-23 Thread Daniel P. Berrange
On Thu, Aug 23, 2012 at 08:22:35AM -0500, Anthony Liguori wrote: > At some point in the past, the OPEN event was changed to be issued from a > bottom half. This creates a small window whereas a data callback registered > in > init may be invoked before the OPEN event has been issued. > > This is

[Qemu-devel] [PATCH] monitor: move json init from OPEN event to init

2012-08-23 Thread Anthony Liguori
At some point in the past, the OPEN event was changed to be issued from a bottom half. This creates a small window whereas a data callback registered in init may be invoked before the OPEN event has been issued. This is reproducible with: echo "{'execute': 'qmp_capabilities'}" | qemu-system-x86