Re: [Qemu-devel] [PATCH 0/6] virtio: refactor host notifiers

2016-03-30 Thread tu bo
Hi Christian: I got the same crash with qemu master + assertion patch + "[PATCH 0/6] virtio: refactor host notifiers" + Paolo's fix, (gdb) bt #0 blk_aio_read_entry (opaque=0x0) at block/block-backend.c:916 #1 0x02aa2e8e88fe in coroutine_trampoline (i0=, i1=-1677703696) at

Re: [Qemu-devel] [PATCH 0/6] virtio: refactor host notifiers

2016-03-30 Thread tu bo
Hi Christian: On 03/29/2016 07:54 PM, Christian Borntraeger wrote: On 03/29/2016 11:14 AM, tu bo wrote: Hi Paolo: On 03/29/2016 02:11 AM, Paolo Bonzini wrote: On 28/03/2016 05:55, TU BO wrote: Hi Cornelia: I got two crash with qemu master + "[PATCH 0/6] virtio: refactor host notifiers",

Re: [Qemu-devel] [PATCH 0/6] virtio: refactor host notifiers

2016-03-30 Thread tu bo
Hi Christian: On 03/30/2016 12:27 AM, Christian Borntraeger wrote: On 03/29/2016 03:50 PM, Paolo Bonzini wrote: On 29/03/2016 13:45, Cornelia Huck wrote: Hi Tu Bo, please always include the assertion patch at https://lists.gnu.org/archive/html/qemu-block/2016-03/msg00546.html in your

Re: [Qemu-devel] [PATCH 0/6] virtio: refactor host notifiers

2016-03-29 Thread Christian Borntraeger
On 03/29/2016 03:50 PM, Paolo Bonzini wrote: > > > On 29/03/2016 13:45, Cornelia Huck wrote: Hi Tu Bo, please always include the assertion patch at https://lists.gnu.org/archive/html/qemu-block/2016-03/msg00546.html in your tests. Can you include the backtrace from all

Re: [Qemu-devel] [PATCH 0/6] virtio: refactor host notifiers

2016-03-29 Thread Paolo Bonzini
On 29/03/2016 13:45, Cornelia Huck wrote: > > > Hi Tu Bo, > > > > > > please always include the assertion patch at > > > https://lists.gnu.org/archive/html/qemu-block/2016-03/msg00546.html in > > > your tests. Can you include the backtrace from all threads with that > > > patch? > > > > >

Re: [Qemu-devel] [PATCH 0/6] virtio: refactor host notifiers

2016-03-29 Thread Michael S. Tsirkin
On Tue, Mar 29, 2016 at 03:23:57PM +0200, Christian Borntraeger wrote: > On 03/24/2016 05:15 PM, Cornelia Huck wrote: > > Here's the next version of my refactoring of the virtio host notifiers. > > This one actually survives a bit of testing for me (reboot loop). > > > > As this patchset fixes a

Re: [Qemu-devel] [PATCH 0/6] virtio: refactor host notifiers

2016-03-29 Thread Christian Borntraeger
On 03/24/2016 05:15 PM, Cornelia Huck wrote: > Here's the next version of my refactoring of the virtio host notifiers. > This one actually survives a bit of testing for me (reboot loop). > > As this patchset fixes a latent bug exposed by the recent dataplane > changes (we have a deassigned

Re: [Qemu-devel] [PATCH 0/6] virtio: refactor host notifiers

2016-03-29 Thread Christian Borntraeger
On 03/29/2016 11:14 AM, tu bo wrote: > Hi Paolo: > > On 03/29/2016 02:11 AM, Paolo Bonzini wrote: >> On 28/03/2016 05:55, TU BO wrote: >>> Hi Cornelia: >>> >>> I got two crash with qemu master + "[PATCH 0/6] virtio: refactor host >>> notifiers", >> >> Hi Tu Bo, >> >> please always include the

Re: [Qemu-devel] [PATCH 0/6] virtio: refactor host notifiers

2016-03-29 Thread Cornelia Huck
On Tue, 29 Mar 2016 17:14:21 +0800 tu bo wrote: > Hi Paolo: > > On 03/29/2016 02:11 AM, Paolo Bonzini wrote: > > On 28/03/2016 05:55, TU BO wrote: > >> Hi Cornelia: > >> > >> I got two crash with qemu master + "[PATCH 0/6] virtio: refactor host > >> notifiers", > > > >

Re: [Qemu-devel] [PATCH 0/6] virtio: refactor host notifiers

2016-03-29 Thread Paolo Bonzini
On 29/03/2016 10:18, Cornelia Huck wrote: >> > >> > Tested-by: Paolo Bonzini >> > >> > Resisted 6 minutes versus 10 seconds. At about 2.5 seconds per reboot, >> > that means the failure happened at the fourth reboot before, and >> > resisted about 150 reboots with your

Re: [Qemu-devel] [PATCH 0/6] virtio: refactor host notifiers

2016-03-29 Thread tu bo
Hi Paolo: On 03/29/2016 02:11 AM, Paolo Bonzini wrote: On 28/03/2016 05:55, TU BO wrote: Hi Cornelia: I got two crash with qemu master + "[PATCH 0/6] virtio: refactor host notifiers", Hi Tu Bo, please always include the assertion patch at

Re: [Qemu-devel] [PATCH 0/6] virtio: refactor host notifiers

2016-03-29 Thread Cornelia Huck
On Thu, 24 Mar 2016 18:06:21 +0100 Paolo Bonzini wrote: > On 24/03/2016 17:15, Cornelia Huck wrote: > > Here's the next version of my refactoring of the virtio host notifiers. > > This one actually survives a bit of testing for me (reboot loop). > > > > As this patchset

Re: [Qemu-devel] [PATCH 0/6] virtio: refactor host notifiers

2016-03-28 Thread Paolo Bonzini
On 28/03/2016 05:55, TU BO wrote: > Hi Cornelia: > > I got two crash with qemu master + "[PATCH 0/6] virtio: refactor host > notifiers", Hi Tu Bo, please always include the assertion patch at https://lists.gnu.org/archive/html/qemu-block/2016-03/msg00546.html in your tests. Can you include the

Re: [Qemu-devel] [PATCH 0/6] virtio: refactor host notifiers

2016-03-27 Thread TU BO
Hi Cornelia: I got two crash with qemu master + "[PATCH 0/6] virtio: refactor host notifiers", I can get first crash very often. (gdb) bt #0 blk_aio_read_entry (opaque=0x0) at block/block-backend.c:922 #1 0x02aa17a65f0e in coroutine_trampoline (i0=, i1=-1677713216) at

Re: [Qemu-devel] [PATCH 0/6] virtio: refactor host notifiers

2016-03-25 Thread Fam Zheng
On Thu, 03/24 17:15, Cornelia Huck wrote: > Here's the next version of my refactoring of the virtio host notifiers. > This one actually survives a bit of testing for me (reboot loop). > > As this patchset fixes a latent bug exposed by the recent dataplane > changes (we have a deassigned ioeventfd

Re: [Qemu-devel] [PATCH 0/6] virtio: refactor host notifiers

2016-03-24 Thread Paolo Bonzini
On 24/03/2016 17:15, Cornelia Huck wrote: > Here's the next version of my refactoring of the virtio host notifiers. > This one actually survives a bit of testing for me (reboot loop). > > As this patchset fixes a latent bug exposed by the recent dataplane > changes (we have a deassigned

[Qemu-devel] [PATCH 0/6] virtio: refactor host notifiers

2016-03-24 Thread Cornelia Huck
Here's the next version of my refactoring of the virtio host notifiers. This one actually survives a bit of testing for me (reboot loop). As this patchset fixes a latent bug exposed by the recent dataplane changes (we have a deassigned ioeventfd for a short period of time during dataplane start,