All of lore.kernel.org
 help / color / mirror / Atom feed
From: Asias He <asias@redhat.com>
To: Nicholas Bellinger <nab@linux-iscsi.org>
Cc: Paolo Bonzini <pbonzini@redhat.com>,
	Stefan Hajnoczi <stefanha@redhat.com>,
	"Michael S. Tsirkin" <mst@redhat.com>,
	Rusty Russell <rusty@rustcorp.com.au>,
	kvm@vger.kernel.org, virtualization@lists.linux-foundation.org,
	target-devel@vger.kernel.org, Asias He <asias@redhat.com>
Subject: [PATCH v7 2/3] tcm_vhost: Add helper to check if endpoint is setup
Date: Thu, 18 Apr 2013 09:05:53 +0800	[thread overview]
Message-ID: <1366247154-3136-3-git-send-email-asias@redhat.com> (raw)
In-Reply-To: <1366247154-3136-1-git-send-email-asias@redhat.com>
In-Reply-To: <20130417131726.GA26157@redhat.com>

Signed-off-by: Asias He <asias@redhat.com>
---
 drivers/vhost/tcm_vhost.c | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)

diff --git a/drivers/vhost/tcm_vhost.c b/drivers/vhost/tcm_vhost.c
index 88ebb79..8f05528 100644
--- a/drivers/vhost/tcm_vhost.c
+++ b/drivers/vhost/tcm_vhost.c
@@ -111,6 +111,24 @@ static bool tcm_vhost_check_feature(struct vhost_scsi *vs, int feature)
 	return ret;
 }
 
+static bool tcm_vhost_check_endpoint(struct vhost_virtqueue *vq)
+{
+	bool ret = false;
+
+	/*
+	* We can handle the vq only after the endpoint is setup by calling the
+	* VHOST_SCSI_SET_ENDPOINT ioctl.
+	*
+	* TODO: Check that we are running from vhost_worker which acts
+	* as read-side critical section for vhost kind of RCU.
+	* See the comments in struct vhost_virtqueue in drivers/vhost/vhost.h
+	*/
+	if (rcu_dereference_check(vq->private_data, 1))
+		ret = true;
+
+	return ret;
+}
+
 static int tcm_vhost_check_true(struct se_portal_group *se_tpg)
 {
 	return 1;
-- 
1.8.1.4

  parent reply	other threads:[~2013-04-18  1:05 UTC|newest]

Thread overview: 45+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-04-09  8:51 [PATCH v5 0/3] tcm_vhost hotplug Asias He
2013-04-09  8:51 ` [PATCH v5 1/3] tcm_vhost: Introduce tcm_vhost_check_feature() Asias He
2013-04-09  8:51 ` [PATCH v5 2/3] tcm_vhost: Add helper to check if endpoint is setup Asias He
2013-04-09  8:51 ` [PATCH v5 3/3] tcm_vhost: Add hotplug/hotunplug support Asias He
2013-04-12 23:10 ` [PATCH v5 0/3] tcm_vhost hotplug Nicholas A. Bellinger
2013-04-13  3:20   ` [PATCH v6 " Asias He
2013-04-13  3:20   ` [PATCH v6 1/3] tcm_vhost: Introduce tcm_vhost_check_feature() Asias He
2013-04-13  3:20   ` [PATCH v6 2/3] tcm_vhost: Add helper to check if endpoint is setup Asias He
2013-04-13  3:20   ` [PATCH v6 3/3] tcm_vhost: Add hotplug/hotunplug support Asias He
2013-04-13  3:20   ` Asias He
2013-04-14  8:53   ` [PATCH v5 0/3] tcm_vhost hotplug Michael S. Tsirkin
2013-04-17 13:17   ` Michael S. Tsirkin
2013-04-18  1:05     ` [PATCH v7 " Asias He
2013-04-18  1:05     ` Asias He
2013-04-18  7:36       ` Michael S. Tsirkin
2013-04-18  1:05     ` [PATCH v7 1/3] tcm_vhost: Introduce tcm_vhost_check_feature() Asias He
2013-04-18  7:06       ` Michael S. Tsirkin
2013-04-18  7:06       ` Michael S. Tsirkin
2013-04-18  8:25         ` Asias He
2013-04-18  7:44           ` Michael S. Tsirkin
2013-04-18  1:05     ` Asias He [this message]
2013-04-18  7:09       ` [PATCH v7 2/3] tcm_vhost: Add helper to check if endpoint is setup Michael S. Tsirkin
2013-04-18  8:32         ` Asias He
2013-04-18  7:38           ` Michael S. Tsirkin
2013-04-22  8:53             ` Asias He
2013-04-22  8:53             ` Asias He
2013-04-22 13:28               ` Michael S. Tsirkin
2013-04-22 15:00                 ` Asias He
2013-04-22 16:21                   ` Michael S. Tsirkin
2013-04-22 16:21                   ` Michael S. Tsirkin
2013-04-18  1:05     ` Asias He
2013-04-18  1:05     ` [PATCH v7 3/3] tcm_vhost: Add hotplug/hotunplug support Asias He
2013-04-18  1:05     ` Asias He
2013-04-18  7:34       ` Michael S. Tsirkin
2013-04-18  8:59         ` Asias He
2013-04-18  8:21           ` Michael S. Tsirkin
2013-04-19  2:34             ` Asias He
2013-04-20 19:01               ` Michael S. Tsirkin
2013-04-22  9:20                 ` Asias He
2013-04-22 13:17                   ` Michael S. Tsirkin
2013-04-22 15:22                     ` Asias He
2013-04-23  4:18                       ` Rusty Russell
2013-04-23  4:45                         ` Asias He
2013-04-18  1:09     ` [PATCH v5 0/3] tcm_vhost hotplug Asias He
2013-04-12 23:10 ` Nicholas A. Bellinger

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=1366247154-3136-3-git-send-email-asias@redhat.com \
    --to=asias@redhat.com \
    --cc=kvm@vger.kernel.org \
    --cc=mst@redhat.com \
    --cc=nab@linux-iscsi.org \
    --cc=pbonzini@redhat.com \
    --cc=rusty@rustcorp.com.au \
    --cc=stefanha@redhat.com \
    --cc=target-devel@vger.kernel.org \
    --cc=virtualization@lists.linux-foundation.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.