From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751741AbdCCJ57 (ORCPT ); Fri, 3 Mar 2017 04:57:59 -0500 Received: from mail-dm3nam03on0081.outbound.protection.outlook.com ([104.47.41.81]:33216 "EHLO NAM03-DM3-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751474AbdCCJ55 (ORCPT ); Fri, 3 Mar 2017 04:57:57 -0500 From: "Rangankar, Manish" To: Arnd Bergmann , Dept-Eng QLogic Storage Upstream , "James E.J. Bottomley" , "Martin K. Petersen" CC: Johannes Thumshirn , Adheer Chandravanshi , "Dupuis, Chad" , "Easi, Arun" , "Kashyap, Saurav" , Hannes Reinecke , Wei Yongjun , "linux-scsi@vger.kernel.org" , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH] [v2] scsi: qedi: fix build error without DEBUG_FS Thread-Topic: [PATCH] [v2] scsi: qedi: fix build error without DEBUG_FS Thread-Index: AQHSk2WQllQn4qcu7UmdvXEeDYnHI6GDEeyA Date: Fri, 3 Mar 2017 07:17:51 +0000 Message-ID: References: <20170302145845.1096307-1-arnd@arndb.de> In-Reply-To: <20170302145845.1096307-1-arnd@arndb.de> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: arndb.de; dkim=none (message not signed) header.d=none;arndb.de; dmarc=none action=none header.from=cavium.com; x-ms-exchange-messagesentrepresentingtype: 1 x-originating-ip: [114.143.185.87] x-ms-office365-filtering-correlation-id: cfd67d91-5a23-43d7-6e41-08d462056766 x-microsoft-antispam: UriScan:;BCL:0;PCL:0;RULEID:(22001);SRVR:SN1PR0701MB1888; x-microsoft-exchange-diagnostics: 1;SN1PR0701MB1888;7:qE9JKY/nbTWBznjcdv6st0oWeofjqpD2fwihSyr+vQEm0YWIOpi8M9d2ke/YkiIqQZcRubCYYPCULbWTtdw+9lNo6lYKbd5fRK4Q+Dges4DiKb/cWGP4LFlusLG21iiQ8ppAUyqLzIpSBTbA37l6P2pu+qBms8XqbeSUHcLg0vZ2EOzXcELdqZmimOnkIZzi5rBoLrRFB4DalL370UN/7aO1nvQiFzEdcF1YyckHqwcCKOsSsTiItDFdg8TuLHcb5eHzLdraJ4IZ+VN7EH618qmc4F0jfstpXe/WT1YNGhNhWQ68mpG2n2uGbbugAGpB2m5PfJFex6/mONDU0iDigw== x-microsoft-antispam-prvs: x-exchange-antispam-report-test: UriScan:; x-exchange-antispam-report-cfa-test: BCL:0;PCL:0;RULEID:(6040375)(601004)(2401047)(8121501046)(5005006)(3002001)(10201501046)(6041248)(20161123562025)(20161123555025)(20161123564025)(20161123560025)(20161123558025)(6072148);SRVR:SN1PR0701MB1888;BCL:0;PCL:0;RULEID:;SRVR:SN1PR0701MB1888; x-forefront-prvs: 0235CBE7D0 x-forefront-antispam-report: SFV:NSPM;SFS:(10009020)(6009001)(7916002)(39450400003)(24454002)(377454003)(92566002)(189998001)(53546006)(38730400002)(76176999)(54356999)(106116001)(229853002)(2900100001)(305945005)(7736002)(2906002)(3660700001)(6246003)(66066001)(122556002)(6116002)(53936002)(2950100002)(3280700002)(99286003)(50986999)(25786008)(3846002)(86362001)(102836003)(81166006)(54906002)(575784001)(6486002)(8676002)(5660300001)(77096006)(36756003)(6436002)(6506006)(6512007)(8936002)(4326008);DIR:OUT;SFP:1101;SCL:1;SRVR:SN1PR0701MB1888;H:BN3PR07MB2483.namprd07.prod.outlook.com;FPR:;SPF:None;MLV:sfv;LANG:en; spamdiagnosticoutput: 1:99 spamdiagnosticmetadata: NSPM Content-Type: text/plain; charset="us-ascii" Content-ID: MIME-Version: 1.0 X-OriginatorOrg: cavium.com X-MS-Exchange-CrossTenant-originalarrivaltime: 03 Mar 2017 07:17:51.3800 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 711e4ccf-2e9b-4bcf-a551-4094005b6194 X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN1PR0701MB1888 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from quoted-printable to 8bit by mail.home.local id v239wwEk030591 On 02/03/17 8:28 PM, "Arnd Bergmann" wrote: >Without CONFIG_DEBUG_FS, we run into a link error: > >drivers/scsi/qedi/qedi_iscsi.o: In function `qedi_ep_poll': >qedi_iscsi.c:(.text.qedi_ep_poll+0x134): undefined reference to >`do_not_recover' >drivers/scsi/qedi/qedi_iscsi.o: In function `qedi_ep_disconnect': >qedi_iscsi.c:(.text.qedi_ep_disconnect+0x36c): undefined reference to >`do_not_recover' >drivers/scsi/qedi/qedi_iscsi.o: In function `qedi_ep_connect': >qedi_iscsi.c:(.text.qedi_ep_connect+0x350): undefined reference to >`do_not_recover' >drivers/scsi/qedi/qedi_fw.o: In function `qedi_tmf_work': >qedi_fw.c:(.text.qedi_tmf_work+0x3b4): undefined reference to >`do_not_recover' > >This defines the symbol as a constant in this case, as there is no way to >set it to anything other than zero without DEBUG_FS. In addition, I'm >renaming >it to qedi_do_not_recover in order to put it into a driver specific >namespace, >as "do_not_recover" is a really bad name for a kernel-wide global >identifier >when it is used only in one driver. > >Fixes: ace7f46ba5fd ("scsi: qedi: Add QLogic FastLinQ offload iSCSI >driver framework.") >Reviewed-by: Johannes Thumshirn >Signed-off-by: Arnd Bergmann >--- >v2: don't rename references to do_not_recover in string literals >--- > drivers/scsi/qedi/qedi_debugfs.c | 16 ++++++++-------- > drivers/scsi/qedi/qedi_fw.c | 4 ++-- > drivers/scsi/qedi/qedi_gbl.h | 8 +++++++- > drivers/scsi/qedi/qedi_iscsi.c | 8 ++++---- > 4 files changed, 21 insertions(+), 15 deletions(-) > >diff --git a/drivers/scsi/qedi/qedi_debugfs.c >b/drivers/scsi/qedi/qedi_debugfs.c >index 955936274241..59417199bf36 100644 >--- a/drivers/scsi/qedi/qedi_debugfs.c >+++ b/drivers/scsi/qedi/qedi_debugfs.c >@@ -14,7 +14,7 @@ > #include > #include > >-int do_not_recover; >+int qedi_do_not_recover; > static struct dentry *qedi_dbg_root; > > void >@@ -74,22 +74,22 @@ qedi_dbg_exit(void) > static ssize_t > qedi_dbg_do_not_recover_enable(struct qedi_dbg_ctx *qedi_dbg) > { >- if (!do_not_recover) >- do_not_recover = 1; >+ if (!qedi_do_not_recover) >+ qedi_do_not_recover = 1; > > QEDI_INFO(qedi_dbg, QEDI_LOG_DEBUGFS, "do_not_recover=%d\n", >- do_not_recover); >+ qedi_do_not_recover); > return 0; > } > > static ssize_t > qedi_dbg_do_not_recover_disable(struct qedi_dbg_ctx *qedi_dbg) > { >- if (do_not_recover) >- do_not_recover = 0; >+ if (qedi_do_not_recover) >+ qedi_do_not_recover = 0; > > QEDI_INFO(qedi_dbg, QEDI_LOG_DEBUGFS, "do_not_recover=%d\n", >- do_not_recover); >+ qedi_do_not_recover); > return 0; > } > >@@ -141,7 +141,7 @@ qedi_dbg_do_not_recover_cmd_read(struct file *filp, >char __user *buffer, > if (*ppos) > return 0; > >- cnt = sprintf(buffer, "do_not_recover=%d\n", do_not_recover); >+ cnt = sprintf(buffer, "do_not_recover=%d\n", qedi_do_not_recover); > cnt = min_t(int, count, cnt - *ppos); > *ppos += cnt; > return cnt; >diff --git a/drivers/scsi/qedi/qedi_fw.c b/drivers/scsi/qedi/qedi_fw.c >index c9f0ef4e11b3..2bce3efc66a4 100644 >--- a/drivers/scsi/qedi/qedi_fw.c >+++ b/drivers/scsi/qedi/qedi_fw.c >@@ -1461,9 +1461,9 @@ static void qedi_tmf_work(struct work_struct *work) > get_itt(tmf_hdr->rtt), get_itt(ctask->itt), cmd->task_id, > qedi_conn->iscsi_conn_id); > >- if (do_not_recover) { >+ if (qedi_do_not_recover) { > QEDI_ERR(&qedi->dbg_ctx, "DONT SEND CLEANUP/ABORT %d\n", >- do_not_recover); >+ qedi_do_not_recover); > goto abort_ret; > } > >diff --git a/drivers/scsi/qedi/qedi_gbl.h b/drivers/scsi/qedi/qedi_gbl.h >index 8e488de88ece..63d793f46064 100644 >--- a/drivers/scsi/qedi/qedi_gbl.h >+++ b/drivers/scsi/qedi/qedi_gbl.h >@@ -12,8 +12,14 @@ > > #include "qedi_iscsi.h" > >+#ifdef CONFIG_DEBUG_FS >+extern int qedi_do_not_recover; >+#else >+#define qedi_do_not_recover (0) >+#endif >+ > extern uint qedi_io_tracing; >-extern int do_not_recover; >+ > extern struct scsi_host_template qedi_host_template; > extern struct iscsi_transport qedi_iscsi_transport; > extern const struct qed_iscsi_ops *qedi_ops; >diff --git a/drivers/scsi/qedi/qedi_iscsi.c >b/drivers/scsi/qedi/qedi_iscsi.c >index b9f79d36142d..4cc474364c50 100644 >--- a/drivers/scsi/qedi/qedi_iscsi.c >+++ b/drivers/scsi/qedi/qedi_iscsi.c >@@ -833,7 +833,7 @@ qedi_ep_connect(struct Scsi_Host *shost, struct >sockaddr *dst_addr, > return ERR_PTR(ret); > } > >- if (do_not_recover) { >+ if (qedi_do_not_recover) { > ret = -ENOMEM; > return ERR_PTR(ret); > } >@@ -957,7 +957,7 @@ static int qedi_ep_poll(struct iscsi_endpoint *ep, >int timeout_ms) > struct qedi_endpoint *qedi_ep; > int ret = 0; > >- if (do_not_recover) >+ if (qedi_do_not_recover) > return 1; > > qedi_ep = ep->dd_data; >@@ -1025,7 +1025,7 @@ static void qedi_ep_disconnect(struct >iscsi_endpoint *ep) > } > > if (test_bit(QEDI_IN_RECOVERY, &qedi->flags)) { >- if (do_not_recover) { >+ if (qedi_do_not_recover) { > QEDI_INFO(&qedi->dbg_ctx, QEDI_LOG_INFO, > "Do not recover cid=0x%x\n", > qedi_ep->iscsi_cid); >@@ -1039,7 +1039,7 @@ static void qedi_ep_disconnect(struct >iscsi_endpoint *ep) > } > } > >- if (do_not_recover) >+ if (qedi_do_not_recover) > goto ep_exit_recover; > > switch (qedi_ep->state) { >-- >2.9.0 Thanks Arnd. Acked-by: Manish Rangankar