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: Mon, 08 Feb 2016 21:49:02 -0800 Message-ID: <1454996942.10001.170.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> <1454820035.10001.17.camel@haakon3.risingtidesystems.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: Received: from mail.linux-iscsi.org ([67.23.28.174]:44037 "EHLO linux-iscsi.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751054AbcBIFtF (ORCPT ); Tue, 9 Feb 2016 00:49:05 -0500 In-Reply-To: Sender: linux-scsi-owner@vger.kernel.org List-Id: linux-scsi@vger.kernel.org To: Himanshu Madhani Cc: "target-devel@vger.kernel.org" , Giridhar Malavali , linux-scsi , Quinn Tran On Mon, 2016-02-08 at 17:43 +0000, Himanshu Madhani wrote: > > > >So looking at this patch beyond the debugfs part, it does change where > >->check_initiator_node_acl() gets call during qlt_create_sess(). > > > >I assume this is related to new debugfs attribute, and these changes > >(plus others in qlt_del_sess_work_fn) are not bug-fixes on their own, > >correct..? > > > >Aside from that, I don't have an objection to merge as v4.6 for-next > >code if QLogic finds it useful for debugging. > > Yes. The changes in this patch are strictly for making information > available via debugFS and not bug-fixes on their own. Thanks for clarifying. > This information has been requested by few customer who > finds this information useful for debugging and in addition they use this > information in their configuration scripts as well. So I've got no particular objection to people adding debugfs informational attributes for whatever to aid debugging of their drivers. That said, the same information can too be exposed as a tcm_qla2xxx struct se_node_acl->acl_group configfs attribute, and for demo-mode sessions via a tcm_qla2xxx provided dynamic_sessions configfs attribute handler. I'd be happy to add expose this under tcm_qla2xxx configfs for v4.6 code if you'd like, otherwise I'm OK with the patch in queue-next as-is if you find it useful.