From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Nicholas A. Bellinger" Subject: Re: [PATCH 3/4] qla2xxx: Add DebugFS node for target sess list. Date: Sat, 06 Feb 2016 20:44:40 -0800 Message-ID: <1454820280.10001.20.camel@haakon3.risingtidesystems.com> References: <1454604319-27947-1-git-send-email-himanshu.madhani@qlogic.com> <1454604319-27947-4-git-send-email-himanshu.madhani@qlogic.com> <56B39597.2020405@sandisk.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <56B39597.2020405@sandisk.com> Sender: target-devel-owner@vger.kernel.org To: Bart Van Assche Cc: Himanshu Madhani , target-devel@vger.kernel.org, giridhar.malavali@qlogic.com, linux-scsi@vger.kernel.org, Quinn Tran List-Id: linux-scsi@vger.kernel.org On Thu, 2016-02-04 at 10:16 -0800, Bart Van Assche wrote: > On 02/04/2016 08:45 AM, Himanshu Madhani wrote: > > From: Quinn Tran > > > > #cat /sys/kernel/debug/qla2xxx/qla2xxx_31/tgt_sess > > qla2xxx_31 > > Port ID Port Name Handle > > ff:fc:01 21:fd:00:05:33:c7:ec:16 0 > > 01:0e:00 21:00:00:24:ff:7b:8a:e4 1 > > 01:0f:00 21:00:00:24:ff:7b:8a:e5 2 > > .... > > Hello Quinn and Himanshu, > > The above information is not only useful to people who are debugging the > QLogic target driver but also to end users who want to check which > initiator ports have already logged in to a target port. Hence my > proposal to move this information from debugfs to another location (e.g. > configfs or sysfs). Users of other target drivers are probably also > interested in seeing which sessions are active. We already have that. For explicit NodeACLS, see: /sys/kernel/config/target/$FABRIC/$T_WWPN/$TPGT/acls/$I_WWPN/info and for demo-mode dynamically generated se_node_acl, see: /sys/kernel/config/target/$FABRIC/$T_WWPN/$TPGT/dynamic_sessions