[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [virtio guest] vring_need_event() from virtqueue_kick_p
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [virtio guest] vring_need_event() from virtqueue_kick_prepare() |
Date: |
Thu, 9 Jul 2015 11:43:22 +0100 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On Tue, Jul 07, 2015 at 02:43:06PM +0300, Catalin Vasile wrote:
> My vhost module respects the format vhost-net uses:
>
> /* <code> summary */
> mutex_lock(&vq->mutex);
> vhost_disable_notify();
> for (;;) {
> head = vhost_get_vq_desc();
> if (head == vq->num) {
> if (unlikely(vhost_enable_notify())) {
> vhost_disable_notify();
> continue;
> }
> break;
> }
> vhost_add_used_and_signal();
> }
> mutex_unlock(&vq->mutex);
> /* </code> */
>
> I have made a lot of printk() calls and the first job gets processed
> completely, and gets through all those calls:
> 1. it goes into a first loop and processes the first job (get
> descriptor, work with the descriptor, add used and signal).
> 2. On the second loop it hits head == vq->num, and goes back to
> listening to notifications (successfully, it does not get into the
> fallback).
>
> Now in the guest:
> 1. sends first job and the paramers used to call vring_need_event() are:
> vring_avail_event=0, new=1, old=0 (which makes the function evaluate to "0 <
> 1")
> 2. the queue is kicked and vhost does its job.
> 3. the guest driver reaches the end of the first job, and lets the
> following job take its course, only this time vring_need_event()
> receives the following parameters:
> vring_avail_event=0, new=2, old=1 (which makes the function evaluate to "1 <
> 1")
This means you need to look at the vhost code because vring_avail_event
shouldn't be 0.
Why wasn't vhost_update_avail_event() called?
Maybe the feature bit negotiation for your device is broken and vhost
thinks VIRTIO_RING_F_EVENT_IDX is not set.
Or does the used ring have the VRING_USED_F_NO_NOTIFY flag?
pgpBXbadBgDo3.pgp
Description: PGP signature
- [Qemu-devel] [virtio guest] vring_need_event() from virtqueue_kick_prepare(), Catalin Vasile, 2015/07/06
- Re: [Qemu-devel] [virtio guest] vring_need_event() from virtqueue_kick_prepare(), Stefan Hajnoczi, 2015/07/07
- Re: [Qemu-devel] [virtio guest] vring_need_event() from virtqueue_kick_prepare(), Catalin Vasile, 2015/07/07
- Re: [Qemu-devel] [virtio guest] vring_need_event() from virtqueue_kick_prepare(),
Stefan Hajnoczi <=
- Re: [Qemu-devel] [virtio guest] vring_need_event() from virtqueue_kick_prepare(), Catalin Vasile, 2015/07/09
- Re: [Qemu-devel] [virtio guest] vring_need_event() from virtqueue_kick_prepare(), Stefan Hajnoczi, 2015/07/16
- Re: [Qemu-devel] [virtio guest] vring_need_event() from virtqueue_kick_prepare(), Catalin Vasile, 2015/07/16
- Re: [Qemu-devel] [virtio guest] vring_need_event() from virtqueue_kick_prepare(), Stefan Hajnoczi, 2015/07/16
- Re: [Qemu-devel] [virtio guest] vring_need_event() from virtqueue_kick_prepare(), Catalin Vasile, 2015/07/17
- Re: [Qemu-devel] [virtio guest] vring_need_event() from virtqueue_kick_prepare(), Stefan Hajnoczi, 2015/07/17