Re: [Qemu-devel] [PATCH v2 8/9] xen/9pfs: send responses back to the frontend

[Greg Kurz]

On Mon, 13 Mar 2017 16:55:59 -0700
Stefano Stabellini <address@hidden> wrote:

> Once a request is completed, xen_9pfs_push_and_notify gets called. In
> xen_9pfs_push_and_notify, update the indexes (data has already been
> copied to the sg by the common code) and send a notification to the
> frontend.
> 
> Schedule the bottom-half to check if we already have any other requests
> pending.
> 
> Signed-off-by: Stefano Stabellini <address@hidden>
> CC: address@hidden
> CC: address@hidden
> CC: Aneesh Kumar K.V <address@hidden>
> CC: Greg Kurz <address@hidden>
> ---
>  hw/9pfs/xen-9p-backend.c | 16 ++++++++++++++++
>  1 file changed, 16 insertions(+)
> 
> diff --git a/hw/9pfs/xen-9p-backend.c b/hw/9pfs/xen-9p-backend.c
> index d72a749..fed369f 100644
> --- a/hw/9pfs/xen-9p-backend.c
> +++ b/hw/9pfs/xen-9p-backend.c
> @@ -166,6 +166,22 @@ static void xen_9pfs_init_in_iov_from_pdu(V9fsPDU *pdu,
>  
>  static void xen_9pfs_push_and_notify(V9fsPDU *pdu)
>  {
> +    RING_IDX prod;
> +    struct Xen9pfsDev *priv = container_of(pdu->s, struct Xen9pfsDev, state);
> +    struct Xen9pfsRing *ring = &priv->rings[pdu->tag % priv->num_rings];
> +

Coding style for structured types.

> +    ring->intf->out_cons = ring->out_cons;
> +    xen_wmb();
> +
> +    prod = ring->intf->in_prod;
> +    xen_rmb();
> +    ring->intf->in_prod = prod + pdu->size;
> +    xen_wmb();
> +
> +    ring->inprogress = false;
> +    xenevtchn_notify(ring->evtchndev, ring->local_port);
> +
> +    qemu_bh_schedule(ring->bh);
>  }
>  
>  static const struct V9fsTransport xen_9p_transport = {

pgpLjOW1ADSf6.pgp
Description: OpenPGP digital signature