Re: [Qemu-devel] [PATCH 1/1] virtio: fix ioeventfd assignment race

2016-03-31 Thread tu bo
Hi Cornelia: On 03/31/2016 10:43 AM, tu bo wrote: Hi Cornelia: I saw qemu crash for this patch on qemu master yesterday. However, scsi disks of my lpar is not available because of s38 firmware update. I'll double check the test when it's ready. thx I still can see crash like below, (gdb) b

Re: [Qemu-devel] [PATCH 1/1] virtio: fix ioeventfd assignment race

2016-03-30 Thread tu bo
Hi Cornelia: I saw qemu crash for this patch on qemu master yesterday. However, scsi disks of my lpar is not available because of s38 firmware update. I'll double check the test when it's ready. thx On 03/29/2016 10:17 PM, Cornelia Huck wrote: The ->set_host_notifier() callback is invoked w

Re: [Qemu-devel] [PATCH 1/1] virtio: fix ioeventfd assignment race

2016-03-29 Thread Christian Borntraeger
On 03/29/2016 04:17 PM, Cornelia Huck wrote: > The ->set_host_notifier() callback is invoked whenever we want to > switch from or to the generic ioeventfd handler. Currently, all > transports deregister the ioeventfd backing and then re-register > it. This opens a race window where we are without i

Re: [Qemu-devel] [PATCH 1/1] virtio: fix ioeventfd assignment race

2016-03-29 Thread Paolo Bonzini
On 29/03/2016 16:17, Cornelia Huck wrote: > The ->set_host_notifier() callback is invoked whenever we want to > switch from or to the generic ioeventfd handler. Currently, all > transports deregister the ioeventfd backing and then re-register > it. This opens a race window where we are without io

[Qemu-devel] [PATCH 1/1] virtio: fix ioeventfd assignment race

2016-03-29 Thread Cornelia Huck
The ->set_host_notifier() callback is invoked whenever we want to switch from or to the generic ioeventfd handler. Currently, all transports deregister the ioeventfd backing and then re-register it. This opens a race window where we are without ioeventfd backing for a time period: In the virtio-blk