All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stefano Stabellini <sstabellini@kernel.org>
To: qemu-devel@nongnu.org
Cc: xen-devel@lists.xenproject.org, sstabellini@kernel.org,
	jgross@suse.com, groug@kaod.org,
	Stefano Stabellini <stefano@aporeto.com>,
	anthony.perard@citrix.com,
	"Aneesh Kumar K.V" <aneesh.kumar@linux.vnet.ibm.com>
Subject: [Qemu-devel] [PATCH v3 6/9] xen/9pfs: receive requests from the frontend
Date: Thu, 16 Mar 2017 13:01:55 -0700	[thread overview]
Message-ID: <1489694518-16978-6-git-send-email-sstabellini@kernel.org> (raw)
In-Reply-To: <1489694518-16978-1-git-send-email-sstabellini@kernel.org>

Upon receiving an event channel notification from the frontend, schedule
the bottom half. From the bottom half, read one request from the ring,
create a pdu and call pdu_submit to handle it.

For now, only handle one request per ring at a time.

Signed-off-by: Stefano Stabellini <stefano@aporeto.com>
CC: anthony.perard@citrix.com
CC: jgross@suse.com
CC: Aneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com>
CC: Greg Kurz <groug@kaod.org>
---
 hw/9pfs/xen-9p-backend.c | 48 ++++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 48 insertions(+)

diff --git a/hw/9pfs/xen-9p-backend.c b/hw/9pfs/xen-9p-backend.c
index 3fd20ff..f757591 100644
--- a/hw/9pfs/xen-9p-backend.c
+++ b/hw/9pfs/xen-9p-backend.c
@@ -104,12 +104,60 @@ static int xen_9pfs_init(struct XenDevice *xendev)
     return 0;
 }
 
+static int xen_9pfs_receive(Xen9pfsRing *ring)
+{
+    P9MsgHeader h;
+    RING_IDX cons, prod, masked_prod, masked_cons;
+    V9fsPDU *pdu;
+
+    if (ring->inprogress) {
+        return 0;
+    }
+
+    cons = ring->intf->out_cons;
+    prod = ring->intf->out_prod;
+    xen_rmb();
+
+    if (xen_9pfs_queued(prod, cons, XEN_9PFS_RING_SIZE) < sizeof(h)) {
+        return 0;
+    }
+    ring->inprogress = true;
+
+    masked_prod = xen_9pfs_mask(prod, XEN_9PFS_RING_SIZE);
+    masked_cons = xen_9pfs_mask(cons, XEN_9PFS_RING_SIZE);
+
+    xen_9pfs_read_packet(ring->ring.out, masked_prod, &masked_cons,
+                         XEN_9PFS_RING_SIZE, (uint8_t *) &h, sizeof(h));
+
+    /* cannot fail, because we only handle one request per ring at a time */
+    pdu = pdu_alloc(&ring->priv->state);
+    pdu->size = le32_to_cpu(h.size_le);
+    pdu->id = h.id;
+    pdu->tag = le32_to_cpu(h.tag_le);
+    ring->out_size = le32_to_cpu(h.size_le);
+    ring->out_cons = cons + le32_to_cpu(h.size_le);
+
+    qemu_co_queue_init(&pdu->complete);
+    pdu_submit(pdu);
+
+    return 0;
+}
+
 static void xen_9pfs_bh(void *opaque)
 {
+    Xen9pfsRing *ring = opaque;
+    xen_9pfs_receive(ring);
 }
 
 static void xen_9pfs_evtchn_event(void *opaque)
 {
+    Xen9pfsRing *ring = opaque;
+    evtchn_port_t port;
+
+    port = xenevtchn_pending(ring->evtchndev);
+    xenevtchn_unmask(ring->evtchndev, port);
+
+    qemu_bh_schedule(ring->bh);
 }
 
 static int xen_9pfs_free(struct XenDevice *xendev)
-- 
1.9.1

WARNING: multiple messages have this Message-ID (diff)
From: Stefano Stabellini <sstabellini@kernel.org>
To: qemu-devel@nongnu.org
Cc: jgross@suse.com, sstabellini@kernel.org, groug@kaod.org,
	Stefano Stabellini <stefano@aporeto.com>,
	"Aneesh Kumar K.V" <aneesh.kumar@linux.vnet.ibm.com>,
	anthony.perard@citrix.com, xen-devel@lists.xenproject.org
Subject: [PATCH v3 6/9] xen/9pfs: receive requests from the frontend
Date: Thu, 16 Mar 2017 13:01:55 -0700	[thread overview]
Message-ID: <1489694518-16978-6-git-send-email-sstabellini@kernel.org> (raw)
In-Reply-To: <1489694518-16978-1-git-send-email-sstabellini@kernel.org>

Upon receiving an event channel notification from the frontend, schedule
the bottom half. From the bottom half, read one request from the ring,
create a pdu and call pdu_submit to handle it.

For now, only handle one request per ring at a time.

Signed-off-by: Stefano Stabellini <stefano@aporeto.com>
CC: anthony.perard@citrix.com
CC: jgross@suse.com
CC: Aneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com>
CC: Greg Kurz <groug@kaod.org>
---
 hw/9pfs/xen-9p-backend.c | 48 ++++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 48 insertions(+)

diff --git a/hw/9pfs/xen-9p-backend.c b/hw/9pfs/xen-9p-backend.c
index 3fd20ff..f757591 100644
--- a/hw/9pfs/xen-9p-backend.c
+++ b/hw/9pfs/xen-9p-backend.c
@@ -104,12 +104,60 @@ static int xen_9pfs_init(struct XenDevice *xendev)
     return 0;
 }
 
+static int xen_9pfs_receive(Xen9pfsRing *ring)
+{
+    P9MsgHeader h;
+    RING_IDX cons, prod, masked_prod, masked_cons;
+    V9fsPDU *pdu;
+
+    if (ring->inprogress) {
+        return 0;
+    }
+
+    cons = ring->intf->out_cons;
+    prod = ring->intf->out_prod;
+    xen_rmb();
+
+    if (xen_9pfs_queued(prod, cons, XEN_9PFS_RING_SIZE) < sizeof(h)) {
+        return 0;
+    }
+    ring->inprogress = true;
+
+    masked_prod = xen_9pfs_mask(prod, XEN_9PFS_RING_SIZE);
+    masked_cons = xen_9pfs_mask(cons, XEN_9PFS_RING_SIZE);
+
+    xen_9pfs_read_packet(ring->ring.out, masked_prod, &masked_cons,
+                         XEN_9PFS_RING_SIZE, (uint8_t *) &h, sizeof(h));
+
+    /* cannot fail, because we only handle one request per ring at a time */
+    pdu = pdu_alloc(&ring->priv->state);
+    pdu->size = le32_to_cpu(h.size_le);
+    pdu->id = h.id;
+    pdu->tag = le32_to_cpu(h.tag_le);
+    ring->out_size = le32_to_cpu(h.size_le);
+    ring->out_cons = cons + le32_to_cpu(h.size_le);
+
+    qemu_co_queue_init(&pdu->complete);
+    pdu_submit(pdu);
+
+    return 0;
+}
+
 static void xen_9pfs_bh(void *opaque)
 {
+    Xen9pfsRing *ring = opaque;
+    xen_9pfs_receive(ring);
 }
 
 static void xen_9pfs_evtchn_event(void *opaque)
 {
+    Xen9pfsRing *ring = opaque;
+    evtchn_port_t port;
+
+    port = xenevtchn_pending(ring->evtchndev);
+    xenevtchn_unmask(ring->evtchndev, port);
+
+    qemu_bh_schedule(ring->bh);
 }
 
 static int xen_9pfs_free(struct XenDevice *xendev)
-- 
1.9.1


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

  parent reply	other threads:[~2017-03-16 20:02 UTC|newest]

Thread overview: 40+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-03-16 20:00 [Qemu-devel] [PATCH v3 0/9] xen/9pfs: introduce the Xen 9pfs backend Stefano Stabellini
2017-03-16 20:00 ` Stefano Stabellini
2017-03-16 20:01 ` [Qemu-devel] [PATCH v3 1/9] xen: do not build backends for targets that do not support xen Stefano Stabellini
2017-03-16 20:01   ` Stefano Stabellini
2017-03-16 20:01   ` [Qemu-devel] [PATCH v3 2/9] xen: import ring.h from xen Stefano Stabellini
2017-03-16 20:01     ` Stefano Stabellini
2017-03-17 11:15     ` [Qemu-devel] " Greg Kurz
2017-03-17 11:15       ` Greg Kurz
2017-03-16 20:01   ` [Qemu-devel] [PATCH v3 3/9] 9p: introduce a type for the 9p header Stefano Stabellini
2017-03-16 20:01     ` Stefano Stabellini
2017-03-17 11:16     ` [Qemu-devel] " Greg Kurz
2017-03-17 11:16       ` Greg Kurz
2017-03-17 21:06       ` [Qemu-devel] " Philippe Mathieu-Daudé
2017-03-17 21:06         ` Philippe Mathieu-Daudé
2017-03-16 20:01   ` [Qemu-devel] [PATCH v3 4/9] xen/9pfs: introduce Xen 9pfs backend Stefano Stabellini
2017-03-16 20:01     ` Stefano Stabellini
2017-03-16 20:01   ` [Qemu-devel] [PATCH v3 5/9] xen/9pfs: connect to the frontend Stefano Stabellini
2017-03-16 20:01     ` Stefano Stabellini
2017-03-17 13:22     ` [Qemu-devel] " Greg Kurz
2017-03-17 13:22       ` Greg Kurz
2017-03-17 17:40       ` [Qemu-devel] " Stefano Stabellini
2017-03-17 17:40         ` Stefano Stabellini
2017-03-16 20:01   ` Stefano Stabellini [this message]
2017-03-16 20:01     ` [PATCH v3 6/9] xen/9pfs: receive requests from " Stefano Stabellini
2017-03-16 20:01   ` [Qemu-devel] [PATCH v3 7/9] xen/9pfs: implement in/out_iov_from_pdu and vmarshal/vunmarshal Stefano Stabellini
2017-03-16 20:01     ` Stefano Stabellini
2017-03-17 13:27     ` [Qemu-devel] " Greg Kurz
2017-03-17 13:27       ` Greg Kurz
2017-03-17 18:19       ` [Qemu-devel] " Stefano Stabellini
2017-03-17 18:19         ` Stefano Stabellini
2017-03-16 20:01   ` [Qemu-devel] [PATCH v3 8/9] xen/9pfs: send responses back to the frontend Stefano Stabellini
2017-03-16 20:01     ` Stefano Stabellini
2017-03-16 20:01   ` [Qemu-devel] [PATCH v3 9/9] xen/9pfs: build and register Xen 9pfs backend Stefano Stabellini
2017-03-16 20:01     ` Stefano Stabellini
2017-03-17 11:08   ` [Qemu-devel] [PATCH v3 1/9] xen: do not build backends for targets that do not support xen Greg Kurz
2017-03-17 11:08     ` Greg Kurz
2017-03-17 11:10   ` [Qemu-devel] " Paolo Bonzini
2017-03-17 11:10     ` Paolo Bonzini
2017-03-17 17:34     ` [Qemu-devel] " Stefano Stabellini
2017-03-17 17:34       ` Stefano Stabellini

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1489694518-16978-6-git-send-email-sstabellini@kernel.org \
    --to=sstabellini@kernel.org \
    --cc=aneesh.kumar@linux.vnet.ibm.com \
    --cc=anthony.perard@citrix.com \
    --cc=groug@kaod.org \
    --cc=jgross@suse.com \
    --cc=qemu-devel@nongnu.org \
    --cc=stefano@aporeto.com \
    --cc=xen-devel@lists.xenproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.