[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/2] virtio: revert virtio_queue_set_notificatio
Re: [Qemu-devel] [PATCH 0/2] virtio: revert virtio_queue_set_notification() nesting
Fri, 13 Jan 2017 09:15:49 -0600
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.12; rv:45.0) Gecko/20100101 Thunderbird/45.6.0
On 1/13/17 6:02 AM, Stefan Hajnoczi wrote:
> On Thu, Jan 12, 2017 at 10:57:53AM -0600, Doug Goldstein wrote:
>> On 1/12/17 5:46 AM, Stefan Hajnoczi wrote:
>>> The virtio_queue_set_notification() nesting introduced for AioContext
>>> raised an assertion with virtio-net (even in non-polling mode). Converting
>>> virtio-net and virtio-crypto to use virtio_queue_set_notification() in a
>>> nesting fashion would be invasive and isn't worth it.
>>> Patch 1 contains the revert to resolve the bug that Doug noticed.
>>> Patch 2 is a less efficient but safe alternative.
>>> Stefan Hajnoczi (2):
>>> Revert "virtio: turn vq->notification into a nested counter"
>>> virtio: disable notifications again after poll succeeded
>>> hw/virtio/virtio.c | 21 +++++++++------------
>>> 1 file changed, 9 insertions(+), 12 deletions(-)
>> So I just gave this series a whirl and it fixes the assert but causes
>> another issue for me. While iPXE is getting a DHCP address the screen
>> immediately flashes over to the UEFI shell. Its like a timeout is
>> getting hit and just dropping me to the shell.
> Sounds like an separate problem.
Is there any debug output that I can provide to help troubleshoot it?
I've built 23425cc and UEFI via OVMF is able to get an IP address via
DHCP inside of iPXE. I've also taken master and only applied the first
patch in this series (the revert) and it too works. Its only when I add
the 2nd patch into the mix or don't revert out the "virtio: turn
vq->notification into a nested counter" patch that it fails.
Description: OpenPGP digital signature
Re: [Qemu-devel] [PATCH 0/2] virtio: revert virtio_queue_set_notification() nesting, Richard Henderson, 2017/01/13
Re: [Qemu-devel] [PATCH 0/2] virtio: revert virtio_queue_set_notification() nesting, Laszlo Ersek, 2017/01/16