[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH] virtio: Use ioeventfd for virtqueue notify
From: |
Stefan Hajnoczi |
Subject: |
[Qemu-devel] Re: [PATCH] virtio: Use ioeventfd for virtqueue notify |
Date: |
Tue, 19 Oct 2010 14:44:35 +0100 |
On Tue, Oct 19, 2010 at 2:35 PM, Michael S. Tsirkin <address@hidden> wrote:
> On Tue, Oct 19, 2010 at 08:12:42AM -0500, Anthony Liguori wrote:
>> On 10/19/2010 08:07 AM, Stefan Hajnoczi wrote:
>> >Is there anything stopping this patch from being merged?
>>
>> Michael, any objections? If not, I'll merge it.
>
> I don't really understand what's going on there. The extra state in
> notifiers especially scares me. If you do and are comfortable with the
> code, go ahead :)
I'm happy to address your comments. The state machine was a bit icky
but I don't see a way around it. Will follow up to your review email.
Stefan
- Re: [Qemu-devel] Re: [PATCH] virtio: Use ioeventfd for virtqueue notify, (continued)
[Qemu-devel] Re: [PATCH] virtio: Use ioeventfd for virtqueue notify, Stefan Hajnoczi, 2010/10/04
[Qemu-devel] Re: [PATCH] virtio: Use ioeventfd for virtqueue notify, rukhsana ansari, 2010/10/05
[Qemu-devel] Re: [PATCH] virtio: Use ioeventfd for virtqueue notify, Stefan Hajnoczi, 2010/10/19
[Qemu-devel] Re: [PATCH] virtio: Use ioeventfd for virtqueue notify, Michael S. Tsirkin, 2010/10/19