From: Mike Christie <mchristi@redhat.com> To: jsmart2021@gmail.com, 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/12] target: add sysfs support Date: Mon, 20 Apr 2020 19:14:14 +0000 [thread overview] Message-ID: <20200420191426.17055-1-mchristi@redhat.com> (raw) The following patches made over Linus's current tree allow lio to export info about structs that the kernel initiates creation of via events like initiator login where there is no user interaction like a mkdir. These patches specificially focus on the I_T_nexus/session but could be used for other objects if we want. Why sysfs when we have configfs? I started with configfs and hit bugs like: commit cc57c07343bd071cdf1915a91a24ab7d40c9b590 Author: Mike Christie <mchristi@redhat.com> Date: Sun Jul 15 18:16:17 2018 -0500 configfs: fix registered group removal but it turns out that bug was not really a bug and was just how configfs was meant to work. It seems it was not meant to be used where the kernel initiates creation of dirs/files as a result of some internal action. It's more geared to the user initiating the creation, and my patch just lead to other bugs and was reverted: commit f19e4ed1e1edbfa3c9ccb9fed17759b7d6db24c6 Author: Al Viro <viro@zeniv.linux.org.uk> Date: Thu Aug 29 23:13:30 2019 -0400 configfs_register_group() shouldn't be (and isn't) called in rmdirable parts So to export the session info we have debugfs, sysfs, ioctl, netlink, etc. sysfs just seemed like a decent fit since one of the primary users is rtslib and it already has lots of file/dir handling code. V2: - rename top level dir to scsi_target - Fix extra newline - Copy data that's exported to sysfs so we do not have to worry about configfs and sysfs refcounts. - Export session info needed for tracking sessions in userspace and handling commands like PGRs there (still needs a way to notify userspace when sessions are added/deleted, but that will be a different set since the focus is different).
WARNING: multiple messages have this Message-ID (diff)
From: Mike Christie <mchristi@redhat.com> To: jsmart2021@gmail.com, 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/12] target: add sysfs support Date: Mon, 20 Apr 2020 14:14:14 -0500 [thread overview] Message-ID: <20200420191426.17055-1-mchristi@redhat.com> (raw) The following patches made over Linus's current tree allow lio to export info about structs that the kernel initiates creation of via events like initiator login where there is no user interaction like a mkdir. These patches specificially focus on the I_T_nexus/session but could be used for other objects if we want. Why sysfs when we have configfs? I started with configfs and hit bugs like: commit cc57c07343bd071cdf1915a91a24ab7d40c9b590 Author: Mike Christie <mchristi@redhat.com> Date: Sun Jul 15 18:16:17 2018 -0500 configfs: fix registered group removal but it turns out that bug was not really a bug and was just how configfs was meant to work. It seems it was not meant to be used where the kernel initiates creation of dirs/files as a result of some internal action. It's more geared to the user initiating the creation, and my patch just lead to other bugs and was reverted: commit f19e4ed1e1edbfa3c9ccb9fed17759b7d6db24c6 Author: Al Viro <viro@zeniv.linux.org.uk> Date: Thu Aug 29 23:13:30 2019 -0400 configfs_register_group() shouldn't be (and isn't) called in rmdirable parts So to export the session info we have debugfs, sysfs, ioctl, netlink, etc. sysfs just seemed like a decent fit since one of the primary users is rtslib and it already has lots of file/dir handling code. V2: - rename top level dir to scsi_target - Fix extra newline - Copy data that's exported to sysfs so we do not have to worry about configfs and sysfs refcounts. - Export session info needed for tracking sessions in userspace and handling commands like PGRs there (still needs a way to notify userspace when sessions are added/deleted, but that will be a different set since the focus is different).
next reply other threads:[~2020-04-20 19:14 UTC|newest] Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-04-20 19:14 Mike Christie [this message] 2020-04-20 19:14 ` RFC PATCH 00/12] target: add sysfs support Mike Christie 2020-04-20 19:14 ` [RFC PATCH 01/12] target: check enforce_pr_isids during registration Mike Christie 2020-04-20 19:14 ` Mike Christie 2020-04-20 19:14 ` [RFC PATCH 02/12] target: separate acl name from port ids Mike Christie 2020-04-20 19:14 ` Mike Christie 2020-04-21 14:49 ` Bodo Stroesser 2020-04-21 14:49 ` Bodo Stroesser 2020-04-21 17:09 ` Mike Christie 2020-04-21 17:09 ` Mike Christie 2020-04-20 19:14 ` [RFC PATCH 03/12] iscsi target: setup transport_id Mike Christie 2020-04-20 19:14 ` Mike Christie 2020-04-20 19:14 ` [RFC PATCH 04/12] target: use tpid in target_stat_iport_port_ident_show Mike Christie 2020-04-20 19:14 ` Mike Christie 2020-04-21 14:58 ` Bodo Stroesser 2020-04-21 14:58 ` Bodo Stroesser 2020-04-20 19:14 ` [RFC PATCH 05/12] target: drop sess_get_initiator_sid from PR code Mike Christie 2020-04-20 19:14 ` Mike Christie 2020-04-20 19:14 ` [RFC PATCH 06/12] target: drop sess_get_initiator_sid Mike Christie 2020-04-20 19:14 ` Mike Christie 2020-04-20 19:14 ` [RFC PATCH 07/12] target: add sysfs support Mike Christie 2020-04-20 19:14 ` Mike Christie 2020-04-21 15:48 ` Bodo Stroesser 2020-04-21 15:48 ` Bodo Stroesser 2020-04-20 19:14 ` [RFC PATCH 08/12] target: add sysfs session helper functions Mike Christie 2020-04-20 19:14 ` Mike Christie 2020-04-21 16:07 ` Bodo Stroesser 2020-04-21 16:07 ` Bodo Stroesser 2020-04-20 19:14 ` [RFC PATCH 09/12] target: move session setup cb to fabric ops Mike Christie 2020-04-20 19:14 ` Mike Christie 2020-04-20 19:14 ` [RFC PATCH 10/12] target: add target_setup_session sysfs support Mike Christie 2020-04-20 19:14 ` Mike Christie 2020-04-20 19:14 ` [RFC PATCH 11/12] iscsi target: use session sysfs helpers Mike Christie 2020-04-20 19:14 ` Mike Christie 2020-04-21 16:26 ` Bodo Stroesser 2020-04-21 16:26 ` Bodo Stroesser 2020-04-20 19:14 ` [RFC PATCH 12/12] target: drop sess_get_index Mike Christie 2020-04-20 19:14 ` Mike 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=20200420191426.17055-1-mchristi@redhat.com \ --to=mchristi@redhat.com \ --cc=bstroesser@ts.fujitsu.com \ --cc=bvanassche@acm.org \ --cc=jsmart2021@gmail.com \ --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: linkBe 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.