xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: Stefano Stabellini <sstabellini@kernel.org>
To: xen-devel@lists.xen.org
Cc: jgross@suse.com, Stefano Stabellini <stefano@aporeto.com>,
	boris.ostrovsky@oracle.com, sstabellini@kernel.org,
	linux-kernel@vger.kernel.org
Subject: [PATCH v1 02/13] xen/pvcalls: connect to the backend
Date: Fri, 21 Jul 2017 17:11:52 -0700	[thread overview]
Message-ID: <1500682323-25940-2-git-send-email-sstabellini@kernel.org> (raw)
In-Reply-To: <1500682323-25940-1-git-send-email-sstabellini@kernel.org>

Implement the probe function for the pvcalls frontend. Read the
supported versions, max-page-order and function-calls nodes from
xenstore.

Introduce a data structure named pvcalls_bedata. It contains pointers to
the command ring, the event channel, a list of active sockets and a list
of passive sockets. Lists accesses are protected by a spin_lock.

Introduce a waitqueue to allow waiting for a response on commands sent
to the backend.

Introduce an array of struct xen_pvcalls_response to store commands
responses.

Only one frontend<->backend connection is supported at any given time
for a guest. Store the active frontend device to a static pointer.

Introduce a stub functions for the event handler.

Signed-off-by: Stefano Stabellini <stefano@aporeto.com>
CC: boris.ostrovsky@oracle.com
CC: jgross@suse.com
---
 drivers/xen/pvcalls-front.c | 153 ++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 153 insertions(+)

diff --git a/drivers/xen/pvcalls-front.c b/drivers/xen/pvcalls-front.c
index 173e204..fb08ebf 100644
--- a/drivers/xen/pvcalls-front.c
+++ b/drivers/xen/pvcalls-front.c
@@ -20,6 +20,29 @@
 #include <xen/xenbus.h>
 #include <xen/interface/io/pvcalls.h>
 
+#define PVCALLS_INVALID_ID (UINT_MAX)
+#define RING_ORDER XENBUS_MAX_RING_GRANT_ORDER
+#define PVCALLS_NR_REQ_PER_RING __CONST_RING_SIZE(xen_pvcalls, XEN_PAGE_SIZE)
+
+struct pvcalls_bedata {
+	struct xen_pvcalls_front_ring ring;
+	grant_ref_t ref;
+	int irq;
+
+	struct list_head socket_mappings;
+	struct list_head socketpass_mappings;
+	spinlock_t pvcallss_lock;
+
+	wait_queue_head_t inflight_req;
+	struct xen_pvcalls_response rsp[PVCALLS_NR_REQ_PER_RING];
+};
+struct xenbus_device *pvcalls_front_dev;
+
+static irqreturn_t pvcalls_front_event_handler(int irq, void *dev_id)
+{
+	return IRQ_HANDLED;
+}
+
 static const struct xenbus_device_id pvcalls_front_ids[] = {
 	{ "pvcalls" },
 	{ "" }
@@ -33,7 +56,114 @@ static int pvcalls_front_remove(struct xenbus_device *dev)
 static int pvcalls_front_probe(struct xenbus_device *dev,
 			  const struct xenbus_device_id *id)
 {
+	int ret = -EFAULT, evtchn, ref = -1, i;
+	unsigned int max_page_order, function_calls, len;
+	char *versions;
+	grant_ref_t gref_head = 0;
+	struct xenbus_transaction xbt;
+	struct pvcalls_bedata *bedata = NULL;
+	struct xen_pvcalls_sring *sring;
+
+	if (pvcalls_front_dev != NULL) {
+		dev_err(&dev->dev, "only one PV Calls connection supported\n");
+		return -EINVAL;
+	}
+
+	versions = xenbus_read(XBT_NIL, dev->otherend, "versions", &len);
+	if (!len)
+		return -EINVAL;
+	if (strcmp(versions, "1")) {
+		kfree(versions);
+		return -EINVAL;
+	}
+	kfree(versions);
+	ret = xenbus_scanf(XBT_NIL, dev->otherend,
+			   "max-page-order", "%u", &max_page_order);
+	if (ret <= 0)
+		return -ENODEV;
+	if (max_page_order < RING_ORDER)
+		return -ENODEV;
+	ret = xenbus_scanf(XBT_NIL, dev->otherend,
+			   "function-calls", "%u", &function_calls);
+	if (ret <= 0 || function_calls != 1)
+		return -ENODEV;
+	pr_info("%s max-page-order is %u\n", __func__, max_page_order);
+
+	bedata = kzalloc(sizeof(struct pvcalls_bedata), GFP_KERNEL);
+	if (!bedata)
+		return -ENOMEM;
+
+	init_waitqueue_head(&bedata->inflight_req);
+	for (i = 0; i < PVCALLS_NR_REQ_PER_RING; i++)
+		bedata->rsp[i].req_id = PVCALLS_INVALID_ID;
+
+	sring = (struct xen_pvcalls_sring *) __get_free_page(GFP_KERNEL |
+							     __GFP_ZERO);
+	if (!sring)
+		goto error;
+	SHARED_RING_INIT(sring);
+	FRONT_RING_INIT(&bedata->ring, sring, XEN_PAGE_SIZE);
+
+	ret = xenbus_alloc_evtchn(dev, &evtchn);
+	if (ret)
+		goto error;
+
+	bedata->irq = bind_evtchn_to_irqhandler(evtchn,
+						pvcalls_front_event_handler,
+						0, "pvcalls-frontend", dev);
+	if (bedata->irq < 0) {
+		ret = bedata->irq;
+		goto error;
+	}
+
+	ret = gnttab_alloc_grant_references(1, &gref_head);
+	if (ret < 0)
+		goto error;
+	bedata->ref = ref = gnttab_claim_grant_reference(&gref_head);
+	if (ref < 0)
+		goto error;
+	gnttab_grant_foreign_access_ref(ref, dev->otherend_id,
+					virt_to_gfn((void *)sring), 0);
+
+ again:
+	ret = xenbus_transaction_start(&xbt);
+	if (ret) {
+		xenbus_dev_fatal(dev, ret, "starting transaction");
+		goto error;
+	}
+	ret = xenbus_printf(xbt, dev->nodename, "version", "%u", 1);
+	if (ret)
+		goto error_xenbus;
+	ret = xenbus_printf(xbt, dev->nodename, "ring-ref", "%d", ref);
+	if (ret)
+		goto error_xenbus;
+	ret = xenbus_printf(xbt, dev->nodename, "port", "%u",
+			    evtchn);
+	if (ret)
+		goto error_xenbus;
+	ret = xenbus_transaction_end(xbt, 0);
+	if (ret) {
+		if (ret == -EAGAIN)
+			goto again;
+		xenbus_dev_fatal(dev, ret, "completing transaction");
+		goto error;
+	}
+
+	INIT_LIST_HEAD(&bedata->socket_mappings);
+	INIT_LIST_HEAD(&bedata->socketpass_mappings);
+	spin_lock_init(&bedata->pvcallss_lock);
+	dev_set_drvdata(&dev->dev, bedata);
+	pvcalls_front_dev = dev;
+	xenbus_switch_state(dev, XenbusStateInitialised);
+
 	return 0;
+
+ error_xenbus:
+	xenbus_transaction_end(xbt, 1);
+	xenbus_dev_fatal(dev, ret, "writing xenstore");
+ error:
+	pvcalls_front_remove(dev);
+	return ret;
 }
 
 static int pvcalls_front_resume(struct xenbus_device *dev)
@@ -45,6 +175,29 @@ static int pvcalls_front_resume(struct xenbus_device *dev)
 static void pvcalls_front_changed(struct xenbus_device *dev,
 			    enum xenbus_state backend_state)
 {
+	switch (backend_state) {
+	case XenbusStateReconfiguring:
+	case XenbusStateReconfigured:
+	case XenbusStateInitialising:
+	case XenbusStateInitialised:
+	case XenbusStateUnknown:
+		break;
+
+	case XenbusStateInitWait:
+		break;
+
+	case XenbusStateConnected:
+		xenbus_switch_state(dev, XenbusStateConnected);
+		break;
+
+	case XenbusStateClosed:
+		if (dev->state == XenbusStateClosed)
+			break;
+		/* Missed the backend's CLOSING state -- fallthrough */
+	case XenbusStateClosing:
+		xenbus_frontend_closed(dev);
+		break;
+	}
 }
 
 static struct xenbus_driver pvcalls_front_driver = {
-- 
1.9.1


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

  parent reply	other threads:[~2017-07-22  0:11 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <alpine.DEB.2.10.1707201553090.22381@sstabellini-ThinkPad-X260>
2017-07-22  0:11 ` [PATCH v1 01/13] xen/pvcalls: introduce the pvcalls xenbus frontend Stefano Stabellini
     [not found] ` <1500682323-25940-1-git-send-email-sstabellini@kernel.org>
2017-07-22  0:11   ` Stefano Stabellini [this message]
2017-07-24 19:23     ` [PATCH v1 02/13] xen/pvcalls: connect to the backend Juergen Gross
     [not found]     ` <a8d4bf5c-e509-c673-8892-fe6a6bc83aad@suse.com>
2017-07-25 20:10       ` Stefano Stabellini
2017-07-22  0:11   ` [PATCH v1 03/13] xen/pvcalls: implement socket command and handle events Stefano Stabellini
2017-07-24 19:29     ` Juergen Gross
     [not found]     ` <ca977dd3-e3f6-c2bd-14b4-45262e5bc585@suse.com>
2017-07-25 20:43       ` Stefano Stabellini
2017-07-22  0:11   ` [PATCH v1 04/13] xen/pvcalls: implement connect command Stefano Stabellini
2017-07-24 19:40     ` Juergen Gross
     [not found]     ` <7da9743d-61fd-9f1d-a8c4-ea65b953f878@suse.com>
2017-07-24 22:45       ` Stefano Stabellini
2017-07-22  0:11   ` [PATCH v1 05/13] xen/pvcalls: implement bind command Stefano Stabellini
2017-07-22  0:11   ` [PATCH v1 06/13] xen/pvcalls: implement listen command Stefano Stabellini
2017-07-22  0:11   ` [PATCH v1 07/13] xen/pvcalls: implement accept command Stefano Stabellini
2017-07-24 19:47     ` Juergen Gross
     [not found]     ` <fbb5c673-203f-c262-186a-dba9785a3074@suse.com>
2017-07-24 22:53       ` Stefano Stabellini
2017-07-22  0:11   ` [PATCH v1 08/13] xen/pvcalls: implement sendmsg Stefano Stabellini
2017-07-24 19:51     ` Juergen Gross
     [not found]     ` <f66b2a87-2e02-6210-6ce4-a0ca63177887@suse.com>
2017-07-24 22:38       ` Stefano Stabellini
2017-07-22  0:11   ` [PATCH v1 09/13] xen/pvcalls: implement recvmsg Stefano Stabellini
2017-07-22  0:12   ` [PATCH v1 10/13] xen/pvcalls: implement poll command Stefano Stabellini
2017-07-22  0:12   ` [PATCH v1 11/13] xen/pvcalls: implement release command Stefano Stabellini
2017-07-22  0:12   ` [PATCH v1 12/13] xen/pvcalls: implement frontend disconnect Stefano Stabellini
2017-07-22  0:12   ` [PATCH v1 13/13] xen: introduce a Kconfig option to enable the pvcalls frontend Stefano Stabellini
2017-07-24 19:06   ` [PATCH v1 01/13] xen/pvcalls: introduce the pvcalls xenbus frontend Juergen Gross
     [not found]   ` <1500682323-25940-5-git-send-email-sstabellini@kernel.org>
2017-07-24 19:43     ` [PATCH v1 05/13] xen/pvcalls: implement bind command Juergen Gross
     [not found]     ` <c6a3d016-9a06-3daf-ac00-41ccbef47a32@suse.com>
2017-07-24 22:51       ` Stefano Stabellini
     [not found]   ` <1500682323-25940-6-git-send-email-sstabellini@kernel.org>
2017-07-24 19:44     ` [PATCH v1 06/13] xen/pvcalls: implement listen command Juergen Gross
     [not found]     ` <43f1eb35-ae03-a21c-87a8-9a28bc0a7262@suse.com>
2017-07-24 22:51       ` Stefano Stabellini
     [not found]   ` <1500682323-25940-9-git-send-email-sstabellini@kernel.org>
2017-07-24 19:56     ` [PATCH v1 09/13] xen/pvcalls: implement recvmsg Juergen Gross
     [not found]     ` <aaeaa565-19fb-0b2b-8a85-729a0e69d81b@suse.com>
2017-07-24 22:37       ` Stefano Stabellini
     [not found]   ` <1500682323-25940-10-git-send-email-sstabellini@kernel.org>
2017-07-24 20:08     ` [PATCH v1 10/13] xen/pvcalls: implement poll command Juergen Gross
2017-07-25 20:59       ` Stefano Stabellini
     [not found]   ` <1500682323-25940-11-git-send-email-sstabellini@kernel.org>
2017-07-24 20:14     ` [PATCH v1 11/13] xen/pvcalls: implement release command Juergen Gross
     [not found]     ` <2a019876-416b-785d-02ee-244d15240a34@suse.com>
2017-07-25 21:07       ` Stefano Stabellini
     [not found]   ` <1500682323-25940-12-git-send-email-sstabellini@kernel.org>
2017-07-24 20:16     ` [PATCH v1 12/13] xen/pvcalls: implement frontend disconnect Juergen Gross
     [not found]   ` <1500682323-25940-13-git-send-email-sstabellini@kernel.org>
2017-07-24 20:17     ` [PATCH v1 13/13] xen: introduce a Kconfig option to enable the pvcalls frontend Juergen Gross
     [not found]     ` <06a28b65-19ff-6c92-500c-0b20d8ee5c19@suse.com>
2017-07-24 22:36       ` Stefano Stabellini
     [not found]   ` <7905bebe-39a0-380f-139d-66ee48c7e853@suse.com>
2017-07-24 22:32     ` [PATCH v1 01/13] xen/pvcalls: introduce the pvcalls xenbus frontend 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=1500682323-25940-2-git-send-email-sstabellini@kernel.org \
    --to=sstabellini@kernel.org \
    --cc=boris.ostrovsky@oracle.com \
    --cc=jgross@suse.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=stefano@aporeto.com \
    --cc=xen-devel@lists.xen.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).