Re: [Qemu-devel] Weird iscsi/fd-event issue since recent merge of event system changes

2012-05-24 Thread ronnie sahlberg
On Wed, May 23, 2012 at 2:31 AM, Michael Tokarev wrote: > On 22.05.2012 14:03, ronnie sahlberg wrote: > [] Doh, now I remember.  Whenever you change the aio handlers you need to call qemu_notify_event() afterwards, if the handler may fire right away. >>> >>> Thanks.  I just confirmed tha

Re: [Qemu-devel] Weird iscsi/fd-event issue since recent merge of event system changes

2012-05-22 Thread Michael Tokarev
On 22.05.2012 14:03, ronnie sahlberg wrote: [] >>> Doh, now I remember. Whenever you change the aio handlers you need to >>> call qemu_notify_event() afterwards, if the handler may fire right away. >> >> Thanks. I just confirmed that qemu_notify_event() fixes the issue. >> Ill send a patch that u

Re: [Qemu-devel] Weird iscsi/fd-event issue since recent merge of event system changes

2012-05-22 Thread ronnie sahlberg
None of the other drivers in block/*.c call qemu_notify_event() Do you you think those should be audited and have this call added to where required ? regards ronnie sahlberg On Tue, May 22, 2012 at 7:48 PM, ronnie sahlberg wrote: > On Tue, May 22, 2012 at 7:29 PM, Paolo Bonzini wrote: >> Il 2

Re: [Qemu-devel] Weird iscsi/fd-event issue since recent merge of event system changes

2012-05-22 Thread ronnie sahlberg
On Tue, May 22, 2012 at 7:29 PM, Paolo Bonzini wrote: > Il 22/05/2012 11:15, ronnie sahlberg ha scritto: >> Hi, >> >> Now that I see what happens, I can easily workaround this in block/iscsi.c >> by the patch below, but I dont know if this is the right thing to do. >> >> It does appear that here,

Re: [Qemu-devel] Weird iscsi/fd-event issue since recent merge of event system changes

2012-05-22 Thread Paolo Bonzini
Il 22/05/2012 11:15, ronnie sahlberg ha scritto: > Hi, > > Now that I see what happens, I can easily workaround this in block/iscsi.c > by the patch below, but I dont know if this is the right thing to do. > > It does appear that here, when I use qemu_set_fd_handler() and add a > handler for "wr

Re: [Qemu-devel] Weird iscsi/fd-event issue since recent merge of event system changes

2012-05-22 Thread ronnie sahlberg
Hi, Now that I see what happens, I can easily workaround this in block/iscsi.c by the patch below, but I dont know if this is the right thing to do. It does appear that here, when I use qemu_set_fd_handler() and add a handler for "writeble" it takes 55ms before the event system notices this and

Re: [Qemu-devel] Weird iscsi/fd-event issue since recent merge of event system changes

2012-05-22 Thread Paolo Bonzini
Il 22/05/2012 11:07, ronnie sahlberg ha scritto: > Below iscsi_aio_readv is printed when we enter this function. > This function prepares a CDB and sets it up for queueing, but it does > not actually write it to the socket. > At the end of this function, we set up the events by calling > iscs

Re: [Qemu-devel] Weird iscsi/fd-event issue since recent merge of event system changes

2012-05-22 Thread ronnie sahlberg
Hi Stefan On Mon, May 21, 2012 at 11:20 PM, Stefan Hajnoczi wrote: > On Mon, May 21, 2012 at 12:00 PM, ronnie sahlberg > wrote: >> Yes, I use IDE since I boot from this LUN. >> >> >> I just managed to track it down to the IDE changes. >> >> It looks like basically this change triggered it : >> >

Re: [Qemu-devel] Weird iscsi/fd-event issue since recent merge of event system changes

2012-05-21 Thread Stefan Hajnoczi
On Mon, May 21, 2012 at 12:00 PM, ronnie sahlberg wrote: > Yes, I use IDE since I boot from this LUN. > > > I just managed to track it down to the IDE changes. > > It looks like basically this change triggered it : > > commit bef0fd5958120542f126f2dedbfce65d8839a94d > Author: Stefan Hajnoczi > Da

Re: [Qemu-devel] Weird iscsi/fd-event issue since recent merge of event system changes

2012-05-21 Thread ronnie sahlberg
Yes, I use IDE since I boot from this LUN. I just managed to track it down to the IDE changes. It looks like basically this change triggered it : commit bef0fd5958120542f126f2dedbfce65d8839a94d Author: Stefan Hajnoczi Date: Thu Mar 29 10:31:30 2012 +0100 ide: convert ide_sector_read() to

Re: [Qemu-devel] Weird iscsi/fd-event issue since recent merge of event system changes

2012-05-21 Thread Paolo Bonzini
Il 21/05/2012 12:45, Kevin Wolf ha scritto: >> I have not yet found the exact part of this big patch that cause this >> slowdown, but will continue investigating. >> >> I am posting this here in case someone has any idea or if 55ms rings any >> bells. 55ms is the frequency of the PIT while the B

Re: [Qemu-devel] Weird iscsi/fd-event issue since recent merge of event system changes

2012-05-21 Thread Kevin Wolf
Am 21.05.2012 08:07, schrieb ronnie sahlberg: > List, Kevin, > > Since this merge : > commit 1f8bcac09af61e58c5121aa0a932190700ad554d > Merge: cb4c254 1042ec9 > Author: Anthony Liguori > Date: Mon Apr 23 14:27:04 2012 -0500 > > Merge remote-tracking branch 'kwolf/for-anthony' into staging

[Qemu-devel] Weird iscsi/fd-event issue since recent merge of event system changes

2012-05-20 Thread ronnie sahlberg
List, Kevin, Since this merge : commit 1f8bcac09af61e58c5121aa0a932190700ad554d Merge: cb4c254 1042ec9 Author: Anthony Liguori Date: Mon Apr 23 14:27:04 2012 -0500 Merge remote-tracking branch 'kwolf/for-anthony' into staging * kwolf/for-anthony: (38 commits) qemu-iotests: Fix t