> On Thu, 2011-11-17 at 15:24 +0200, Pekka Enberg wrote:
>> On Thu, Nov 17, 2011 at 3:04 PM, Sasha Levin <levinsasha928@xxxxxxxxx> wrote:
>> > Support the event index feature in the virtio spec.
>> >
>> > The results are less notifications between the guest and host, and in
>> > result faster operation of the virt queues.
>> >
>> > Signed-off-by: Sasha Levin <levinsasha928@xxxxxxxxx>
>>
>> What's this patch doing? There's no mention of VIRTIO_RING_F_EVENT_IDX
>> in the diff...
On Thu, Nov 17, 2011 at 3:36 PM, Sasha Levin <levinsasha928@xxxxxxxxx> wrote:
> It allows enabling the VIRTIO_RING_F_EVENT_IDX feature in various
> virtio-* devices by updating the avail event idx and providing a
> function to allow the devices to query the vq and decide if they need to
> signal the VQ or not.
Right. The changelog is extremely confusing as is the ordering in the
patch series. You usually write something like "in preparation for
enabling XYZ support, add helper functions" and include the
conversions immediately after the helpers.
I'll fix the issues myself later today when I have the change to merge
patches. That is, unless you beat me to it and send a v2 of the
series.
Pekka
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
