From: Boris Ostrovsky <boris.ostrovsky@oracle.com>
To: Stefano Stabellini <sstabellini@kernel.org>, xen-devel@lists.xen.org
Cc: jgross@suse.com, Stefano Stabellini <stefano@aporeto.com>,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v5 11/13] xen/pvcalls: implement poll command
Date: Tue, 17 Oct 2017 18:15:32 -0400 [thread overview]
Message-ID: <aaa92ecf-b067-2e09-08de-ea3eb5821598@oracle.com> (raw)
In-Reply-To: <1507336227-20477-11-git-send-email-sstabellini@kernel.org>
>
> +static unsigned int pvcalls_front_poll_passive(struct file *file,
> + struct pvcalls_bedata *bedata,
> + struct sock_mapping *map,
> + poll_table *wait)
> +{
> + int notify, req_id, ret;
> + struct xen_pvcalls_request *req;
> +
> + if (test_bit(PVCALLS_FLAG_ACCEPT_INFLIGHT,
> + (void *)&map->passive.flags)) {
> + uint32_t req_id = READ_ONCE(map->passive.inflight_req_id);
> +
> + if (req_id != PVCALLS_INVALID_ID &&
> + READ_ONCE(bedata->rsp[req_id].req_id) == req_id)
> + return POLLIN | POLLRDNORM;
Same READ_ONCE() question as for an earlier patch.
-boris
> +
> + poll_wait(file, &map->passive.inflight_accept_req, wait);
> + return 0;
> + }
> +
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
next prev parent reply other threads:[~2017-10-17 22:15 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <alpine.DEB.2.10.1710061708540.3073@sstabellini-ThinkPad-X260>
2017-10-07 0:30 ` [PATCH v5 01/13] xen/pvcalls: introduce the pvcalls xenbus frontend Stefano Stabellini
2017-10-07 0:30 ` [PATCH v5 02/13] xen/pvcalls: implement frontend disconnect Stefano Stabellini
2017-10-07 0:30 ` [PATCH v5 03/13] xen/pvcalls: connect to the backend Stefano Stabellini
2017-10-07 0:30 ` [PATCH v5 04/13] xen/pvcalls: implement socket command and handle events Stefano Stabellini
2017-10-17 16:59 ` Boris Ostrovsky
[not found] ` <fa810065-01b8-9448-61d8-f1e3df44036e@oracle.com>
2017-10-20 1:26 ` Stefano Stabellini
[not found] ` <alpine.DEB.2.10.1710191821370.27209@sstabellini-ThinkPad-X260>
2017-10-20 14:24 ` Boris Ostrovsky
2017-10-07 0:30 ` [PATCH v5 05/13] xen/pvcalls: implement connect command Stefano Stabellini
2017-10-07 0:30 ` [PATCH v5 06/13] xen/pvcalls: implement bind command Stefano Stabellini
2017-10-17 17:39 ` Boris Ostrovsky
[not found] ` <f1d7663a-8e18-a90f-7212-3f22a363356d@oracle.com>
2017-10-20 1:31 ` Stefano Stabellini
[not found] ` <alpine.DEB.2.10.1710191828130.27209@sstabellini-ThinkPad-X260>
2017-10-20 14:40 ` Boris Ostrovsky
2017-10-07 0:30 ` [PATCH v5 07/13] xen/pvcalls: implement listen command Stefano Stabellini
2017-10-07 0:30 ` [PATCH v5 08/13] xen/pvcalls: implement accept command Stefano Stabellini
2017-10-07 0:30 ` [PATCH v5 09/13] xen/pvcalls: implement sendmsg Stefano Stabellini
2017-10-17 21:06 ` Boris Ostrovsky
[not found] ` <c5d2a8e3-cf30-92ae-9bab-14a9b05d5a3e@oracle.com>
2017-10-20 1:41 ` Stefano Stabellini
[not found] ` <alpine.DEB.2.10.1710191840040.27209@sstabellini-ThinkPad-X260>
2017-10-20 14:44 ` Boris Ostrovsky
2017-10-07 0:30 ` [PATCH v5 10/13] xen/pvcalls: implement recvmsg Stefano Stabellini
2017-10-07 0:30 ` [PATCH v5 11/13] xen/pvcalls: implement poll command Stefano Stabellini
2017-10-17 22:15 ` Boris Ostrovsky [this message]
2017-10-23 23:06 ` Stefano Stabellini
[not found] ` <alpine.DEB.2.10.1710231604480.574@sstabellini-ThinkPad-X260>
2017-10-24 14:02 ` Boris Ostrovsky
2017-10-07 0:30 ` [PATCH v5 12/13] xen/pvcalls: implement release command Stefano Stabellini
2017-10-24 14:17 ` Boris Ostrovsky
[not found] ` <ec8420a7-5f3d-6ba3-f6a0-7742350c9aab@oracle.com>
2017-10-24 17:17 ` Stefano Stabellini
2017-10-07 0:30 ` [PATCH v5 13/13] xen: introduce a Kconfig option to enable the pvcalls frontend Stefano Stabellini
[not found] ` <1507336227-20477-2-git-send-email-sstabellini@kernel.org>
2017-10-17 16:01 ` [PATCH v5 02/13] xen/pvcalls: implement frontend disconnect Boris Ostrovsky
[not found] ` <8d730196-8c65-21ec-21a2-3b92a463f935@oracle.com>
2017-10-23 22:44 ` Stefano Stabellini
[not found] ` <1507336227-20477-8-git-send-email-sstabellini@kernel.org>
2017-10-17 18:34 ` [PATCH v5 08/13] xen/pvcalls: implement accept command Boris Ostrovsky
[not found] ` <eed60ea1-1dcc-c942-ccd6-7c9d46415bd6@oracle.com>
2017-10-23 23:03 ` Stefano Stabellini
[not found] ` <alpine.DEB.2.10.1710231546290.574@sstabellini-ThinkPad-X260>
2017-10-24 13:52 ` Boris Ostrovsky
[not found] ` <bb43294d-1f9a-4ffa-6987-fcd10ab4d9a9@oracle.com>
2017-10-24 16:42 ` Stefano Stabellini
[not found] ` <1507336227-20477-10-git-send-email-sstabellini@kernel.org>
2017-10-17 21:35 ` [PATCH v5 10/13] xen/pvcalls: implement recvmsg Boris Ostrovsky
[not found] ` <318acfff-0d58-fa4b-29eb-29a5d6f0bddf@oracle.com>
2017-10-20 1:38 ` Stefano Stabellini
[not found] ` <alpine.DEB.2.10.1710191834250.27209@sstabellini-ThinkPad-X260>
2017-10-20 14:43 ` Boris Ostrovsky
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=aaa92ecf-b067-2e09-08de-ea3eb5821598@oracle.com \
--to=boris.ostrovsky@oracle.com \
--cc=jgross@suse.com \
--cc=linux-kernel@vger.kernel.org \
--cc=sstabellini@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).