All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mike Christie <michael.christie@oracle.com>
To: hare@suse.de, bvanassche@acm.org, bstroesser@ts.fujitsu.com,
	martin.petersen@oracle.com, linux-scsi@vger.kernel.org,
	target-devel@vger.kernel.org
Subject: [RFC PATCH 00/10] target: add configfs interface
Date: Sat, 27 Jun 2020 04:34:59 +0000	[thread overview]
Message-ID: <1593232509-13720-1-git-send-email-michael.christie@oracle.com> (raw)

The following patches made over linus's current tree (also apply
over Martin's 5.9 scsi queue branch) add a configfs interface to
export LIO's sessions.

With Hannes not liking the refcounting/tricks in the sysfs approach
I took another stab at configfs. This approach works similar to the
loop/vhost/usb/xen nexus interface where there is a special file
that allows userspace to add/remove sessions.

Because the kernel is making sessions for most drivers, we do not
have mkdir/rmdir support for the session dir, but using the special
file approach we can still remove children session in the correct
order and avoid the issues I hit before. The interface is a little
odd configfs wise because we use the special file, but it has the
benefit everything is in the one interface so it's easy to update
the tools.

If this approach is preferred over the sysfs one then I can repost
with some other fixes, the transport id patches and the iscsi
conversion one.

WARNING: multiple messages have this Message-ID (diff)
From: Mike Christie <michael.christie@oracle.com>
To: hare@suse.de, bvanassche@acm.org, bstroesser@ts.fujitsu.com,
	martin.petersen@oracle.com, linux-scsi@vger.kernel.org,
	target-devel@vger.kernel.org
Subject: [RFC PATCH 00/10] target: add configfs interface
Date: Fri, 26 Jun 2020 23:34:59 -0500	[thread overview]
Message-ID: <1593232509-13720-1-git-send-email-michael.christie@oracle.com> (raw)

The following patches made over linus's current tree (also apply
over Martin's 5.9 scsi queue branch) add a configfs interface to
export LIO's sessions.

With Hannes not liking the refcounting/tricks in the sysfs approach
I took another stab at configfs. This approach works similar to the
loop/vhost/usb/xen nexus interface where there is a special file
that allows userspace to add/remove sessions.

Because the kernel is making sessions for most drivers, we do not
have mkdir/rmdir support for the session dir, but using the special
file approach we can still remove children session in the correct
order and avoid the issues I hit before. The interface is a little
odd configfs wise because we use the special file, but it has the
benefit everything is in the one interface so it's easy to update
the tools.

If this approach is preferred over the sysfs one then I can repost
with some other fixes, the transport id patches and the iscsi
conversion one.



             reply	other threads:[~2020-06-27  4:34 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-06-27  4:34 Mike Christie [this message]
2020-06-27  4:34 ` [RFC PATCH 00/10] target: add configfs interface Mike Christie
2020-06-27  4:35 ` [RFC PATCH 01/10] target: add common session id Mike Christie
2020-06-27  4:35   ` Mike Christie
2020-06-27  4:35 ` [RFC PATCH 02/10] iscsi target: replace module sids with lio's sid Mike Christie
2020-06-27  4:35   ` Mike Christie
2020-06-27  4:35 ` [RFC PATCH 03/10] target: drop sess_get_index Mike Christie
2020-06-27  4:35   ` Mike Christie
2020-06-27  4:35 ` [RFC PATCH 04/10] target: fix xcopy sess release leak Mike Christie
2020-06-27  4:35   ` Mike Christie
2020-06-27  4:35 ` [RFC PATCH 05/10] target: add free_session callout and use cfgfs refcounts Mike Christie
2020-06-27  4:35   ` Mike Christie
2020-06-27  4:35 ` [RFC PATCH 06/10] tcm_loop: fix nexus races Mike Christie
2020-06-27  4:35   ` Mike Christie
2020-06-27  4:35 ` [RFC PATCH 07/10] target: add return value to close_session Mike Christie
2020-06-27  4:35   ` Mike Christie
2020-06-27  4:35 ` [RFC PATCH 08/10] ibm,loop,vhost,xenscsi: add close_session callouts Mike Christie
2020-06-27  4:35   ` Mike Christie
2020-06-27  4:35 ` [RFC PATCH 09/10] target: add helper to close session synchronously Mike Christie
2020-06-27  4:35   ` Mike Christie
2020-06-27  4:45   ` Mike Christie
2020-06-27  4:45     ` Mike Christie
2020-06-27  4:35 ` [RFC PATCH 10/10] target: export sessions via configfs Mike Christie
2020-06-27  4:35   ` Mike Christie
2020-07-03 12:51   ` Bodo Stroesser
2020-07-03 12:51     ` Bodo Stroesser
2020-07-03 16:27     ` Mike Christie
2020-07-03 16:27       ` Mike Christie
2020-07-03 16:57       ` Bodo Stroesser
2020-07-03 16:57         ` Bodo Stroesser
2020-07-03 21:19         ` Michael Christie
2020-07-03 21:19           ` Michael Christie
2020-07-06 12:32           ` Bodo Stroesser
2020-07-06 12:32             ` Bodo Stroesser
2020-07-03 17:37 ` [RFC PATCH 00/10] target: add configfs interface Bart Van Assche
2020-07-03 17:37   ` Bart Van Assche
2020-07-03 22:12   ` Michael Christie
2020-07-03 22:12     ` Michael Christie

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=1593232509-13720-1-git-send-email-michael.christie@oracle.com \
    --to=michael.christie@oracle.com \
    --cc=bstroesser@ts.fujitsu.com \
    --cc=bvanassche@acm.org \
    --cc=hare@suse.de \
    --cc=linux-scsi@vger.kernel.org \
    --cc=martin.petersen@oracle.com \
    --cc=target-devel@vger.kernel.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.