From: Stefano Stabellini <sstabellini@kernel.org>
To: xen-devel@lists.xen.org
Cc: linux-kernel@vger.kernel.org, sstabellini@kernel.org,
jgross@suse.com, boris.ostrovsky@oracle.com,
Stefano Stabellini <stefano@aporeto.com>
Subject: [PATCH 15/18] xen/pvcalls: introduce the ioworker
Date: Mon, 15 May 2017 13:36:07 -0700 [thread overview]
Message-ID: <1494880570-14209-15-git-send-email-sstabellini@kernel.org> (raw)
In-Reply-To: <1494880570-14209-1-git-send-email-sstabellini@kernel.org>
We have one ioworker per cpu core. Each ioworker gets assigned active
sockets randomly. Once a socket is assigned to an ioworker, it remains
tied to it until is released.
Each ioworker goes through the list of outstanding read/write requests
by walking a list of struct sock_mapping. Once a request has been dealt
with, the struct sock_mapping is removed from the list.
We use one atomic counter per socket for "read" operations and one
for "write" operations to keep track of the reads/writes to do.
We also use one atomic counter ("io") per ioworker to keep track of how
many outstanding requests we have in total assigned to the ioworker. The
ioworker finishes when there are none.
Signed-off-by: Stefano Stabellini <stefano@aporeto.com>
CC: boris.ostrovsky@oracle.com
CC: jgross@suse.com
---
drivers/xen/pvcalls-back.c | 44 ++++++++++++++++++++++++++++++++++++++++++++
1 file changed, 44 insertions(+)
diff --git a/drivers/xen/pvcalls-back.c b/drivers/xen/pvcalls-back.c
index 0daa90a..db3e02c 100644
--- a/drivers/xen/pvcalls-back.c
+++ b/drivers/xen/pvcalls-back.c
@@ -99,8 +99,52 @@ struct sockpass_mapping {
static int pvcalls_back_release_active(struct xenbus_device *dev,
struct pvcalls_back_priv *priv,
struct sock_mapping *map);
+
+static void pvcalls_conn_back_read(unsigned long opaque)
+{
+}
+
+static int pvcalls_conn_back_write(struct sock_mapping *map)
+{
+ return 0;
+}
+
static void pvcalls_back_ioworker(struct work_struct *work)
{
+ struct pvcalls_ioworker *ioworker = container_of(work,
+ struct pvcalls_ioworker, register_work);
+ int num = ioworker->num;
+ struct sock_mapping *map, *n;
+ unsigned long flags;
+
+ while (atomic_read(&ioworker->io) > 0) {
+ spin_lock_irqsave(&ioworker->lock, flags);
+ list_for_each_entry_safe(map, n, &ioworker->wqs, queue) {
+ if (map->data_worker != num)
+ continue;
+
+ if (atomic_read(&map->release) > 0) {
+ list_del_init(&map->queue);
+ atomic_set(&map->release, 0);
+ continue;
+ }
+
+ spin_unlock_irqrestore(&ioworker->lock, flags);
+ if (atomic_read(&map->read) > 0)
+ pvcalls_conn_back_read((unsigned long)map);
+ if (atomic_read(&map->write) > 0)
+ pvcalls_conn_back_write(map);
+ spin_lock_irqsave(&ioworker->lock, flags);
+
+ if (atomic_read(&map->read) == 0 &&
+ atomic_read(&map->write) == 0) {
+ list_del_init(&map->queue);
+ atomic_set(&map->release, 0);
+ }
+ }
+ atomic_dec(&ioworker->io);
+ spin_unlock_irqrestore(&ioworker->lock, flags);
+ }
}
static int pvcalls_back_socket(struct xenbus_device *dev,
--
1.9.1
next prev parent reply other threads:[~2017-05-15 20:36 UTC|newest]
Thread overview: 41+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-05-15 20:35 [PATCH 00/18] introduce the Xen PV Calls backend Stefano Stabellini
2017-05-15 20:35 ` [PATCH 01/18] xen: introduce the pvcalls interface header Stefano Stabellini
2017-05-15 20:35 ` [PATCH 02/18] xen/pvcalls: introduce the pvcalls xenbus backend Stefano Stabellini
2017-05-15 20:35 ` [PATCH 03/18] xen/pvcalls: initialize the module and register the " Stefano Stabellini
2017-05-16 1:28 ` Boris Ostrovsky
2017-05-16 20:05 ` Stefano Stabellini
2017-05-16 20:22 ` Stefano Stabellini
2017-05-16 6:40 ` Juergen Gross
2017-05-16 19:58 ` Stefano Stabellini
2017-05-17 5:21 ` Juergen Gross
2017-05-18 21:18 ` Stefano Stabellini
2017-05-19 22:33 ` Stefano Stabellini
2017-05-15 20:35 ` [PATCH 04/18] xen/pvcalls: xenbus state handling Stefano Stabellini
2017-05-16 1:34 ` Boris Ostrovsky
2017-05-16 20:11 ` Stefano Stabellini
2017-05-15 20:35 ` [PATCH 05/18] xen/pvcalls: connect to a frontend Stefano Stabellini
2017-05-16 1:52 ` Boris Ostrovsky
2017-05-16 20:23 ` Stefano Stabellini
2017-05-16 20:38 ` Stefano Stabellini
2017-05-15 20:35 ` [PATCH 06/18] xen/pvcalls: handle commands from the frontend Stefano Stabellini
2017-05-16 2:06 ` Boris Ostrovsky
2017-05-16 20:57 ` Stefano Stabellini
2017-05-15 20:35 ` [PATCH 07/18] xen/pvcalls: implement socket command Stefano Stabellini
2017-05-16 2:12 ` Boris Ostrovsky
2017-05-16 20:45 ` Stefano Stabellini
2017-05-15 20:36 ` [PATCH 08/18] xen/pvcalls: implement connect command Stefano Stabellini
2017-05-16 2:36 ` Boris Ostrovsky
2017-05-16 21:02 ` Stefano Stabellini
2017-05-16 21:56 ` Boris Ostrovsky
2017-05-18 19:10 ` Stefano Stabellini
2017-05-18 20:19 ` Boris Ostrovsky
2017-05-15 20:36 ` [PATCH 09/18] xen/pvcalls: implement bind command Stefano Stabellini
2017-05-15 20:36 ` [PATCH 10/18] xen/pvcalls: implement listen command Stefano Stabellini
2017-05-15 20:36 ` [PATCH 11/18] xen/pvcalls: implement accept command Stefano Stabellini
2017-05-15 20:36 ` [PATCH 12/18] xen/pvcalls: implement poll command Stefano Stabellini
2017-05-15 20:36 ` [PATCH 13/18] xen/pvcalls: implement release command Stefano Stabellini
2017-05-15 20:36 ` [PATCH 14/18] xen/pvcalls: disconnect and module_exit Stefano Stabellini
2017-05-15 20:36 ` Stefano Stabellini [this message]
2017-05-15 20:36 ` [PATCH 16/18] xen/pvcalls: implement read Stefano Stabellini
2017-05-15 20:36 ` [PATCH 17/18] xen/pvcalls: implement write Stefano Stabellini
2017-05-15 20:36 ` [PATCH 18/18] xen: introduce a Kconfig option to enable the pvcalls backend 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=1494880570-14209-15-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
Powered by JetHome