From nobody Sun Feb 8 05:59:09 2026 Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.15]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 74A6F3B8BB4; Wed, 4 Feb 2026 09:45:17 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=198.175.65.15 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770198317; cv=none; b=ScMjGbRSstuhEtkE0TRdNVMqMW1FHIuBS5LW6uHFyG0MEWjO1OIn3gWXoeTaaBjXJXT4gFAtWCIUMMvtNp3qw6yLxmv/dlIZjuvqlZhh3XtD2oeDgQKE99RecTsRJHBQQzVxgRlIwp71w5SzafKyEZWv9Mo+HXWeTaqF3E/rOeY= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770198317; c=relaxed/simple; bh=Qj9IYpqXA2nmuwZJsCBesC4DxgJFjbmtatJ+ozIipWE=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=QqpPP+WS5tBrZvgMw8hY7YH+D1rQLQ6BCLyEl6qVrV+Dd5+0VhOnusBEYcsnarsYCz6NPIPpVW7fiLLv89CB2rAsnWWwA1hhmNCPz2KFHbD8jF+qc7TOSK8ON2qy0K3/g9v8qH+pjD90W6CW4rqifF4xUYA5IzgYV1ByQLh0PeA= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.intel.com; spf=pass smtp.mailfrom=linux.intel.com; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b=bDjXvFsY; arc=none smtp.client-ip=198.175.65.15 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.intel.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linux.intel.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b="bDjXvFsY" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1770198318; x=1801734318; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=Qj9IYpqXA2nmuwZJsCBesC4DxgJFjbmtatJ+ozIipWE=; b=bDjXvFsY49RPxWX+ArHgbdzzSNCZ0iKNPUWvC7oSho+xL1yjCPMUKIt4 +aJ6+TwKSr6/ZqBbJIwEyTw6hBOK5Cu1SfZntAXyWuAJ5LxVqDpX+MGHA 8e3jOcGDitJjG1hHL9NEAjs1YOWVambRzklKcUuKFpO8XLRqxm5XEq3lO jdVRq+OAxa41VQE5hsf5LBP/EUqYgjRIlDTP01BFegBeXdKgLSQFRuNuD dN9Jtfg5Id5TeTS0HLfs71MwhsrvaxNBeAP3QHLs/qflleY59rMYner8M xb8YH0Ktdw75fYv9TXy9+m7svdWSDndhn3+G31w3hI4hum8BLusEYBSik g==; X-CSE-ConnectionGUID: UhZQPcU8R5q6m7Q+VUOcAA== X-CSE-MsgGUID: mBbdwxxbQ0CtVDT4DZZGYg== X-IronPort-AV: E=McAfee;i="6800,10657,11691"; a="75001284" X-IronPort-AV: E=Sophos;i="6.21,272,1763452800"; d="scan'208";a="75001284" Received: from fmviesa004.fm.intel.com ([10.60.135.144]) by orvoesa107.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 04 Feb 2026 01:45:17 -0800 X-CSE-ConnectionGUID: VXPMAlcoSCi4g5uJLVZAbQ== X-CSE-MsgGUID: FeqAuVWYTRCKA5M/Bitm7w== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.21,272,1763452800"; d="scan'208";a="214810887" Received: from black.igk.intel.com ([10.91.253.5]) by fmviesa004.fm.intel.com with ESMTP; 04 Feb 2026 01:45:12 -0800 Received: by black.igk.intel.com (Postfix, from userid 1003) id 17A6C99; Wed, 04 Feb 2026 10:45:11 +0100 (CET) From: Andy Shevchenko To: Chuck Lever , NeilBrown , Jeff Layton , linux-nfs@vger.kernel.org, linux-kernel@vger.kernel.org, netdev@vger.kernel.org, llvm@lists.linux.dev Cc: Trond Myklebust , Anna Schumaker , Olga Kornievskaia , Dai Ngo , Tom Talpey , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Simon Horman , Nathan Chancellor , Nick Desaulniers , Bill Wendling , Justin Stitt , Andy Shevchenko Subject: [PATCH v2 2/3] sunrpc: Kill RPC_IFDEBUG() Date: Wed, 4 Feb 2026 10:41:22 +0100 Message-ID: <20260204094500.2443455-3-andriy.shevchenko@linux.intel.com> X-Mailer: git-send-email 2.50.1 In-Reply-To: <20260204094500.2443455-1-andriy.shevchenko@linux.intel.com> References: <20260204094500.2443455-1-andriy.shevchenko@linux.intel.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" RPC_IFDEBUG() is used in only two places. In one the user of the definition is guarded by ifdeffery, in the second one it's implied due to dprintk() usage. Kill the macro and move the ifdeffery to the regular condition with the variable defined inside, while in the second case add the same conditional and move the respective code there. Signed-off-by: Andy Shevchenko --- fs/nfsd/nfsfh.c | 9 +++++--- include/linux/sunrpc/debug.h | 2 -- net/sunrpc/xprtrdma/svc_rdma_transport.c | 27 ++++++++++++------------ 3 files changed, 20 insertions(+), 18 deletions(-) diff --git a/fs/nfsd/nfsfh.c b/fs/nfsd/nfsfh.c index ed85dd43da18..68b629fbaaeb 100644 --- a/fs/nfsd/nfsfh.c +++ b/fs/nfsd/nfsfh.c @@ -105,9 +105,12 @@ static __be32 nfsd_setuser_and_check_port(struct svc_r= qst *rqstp, { /* Check if the request originated from a secure port. */ if (rqstp && !nfsd_originating_port_ok(rqstp, cred, exp)) { - RPC_IFDEBUG(char buf[RPC_MAX_ADDRBUFLEN]); - dprintk("nfsd: request from insecure port %s!\n", - svc_print_addr(rqstp, buf, sizeof(buf))); + if (IS_ENABLED(CONFIG_SUNRPC_DEBUG)) { + char buf[RPC_MAX_ADDRBUFLEN]; + + dprintk("nfsd: request from insecure port %s!\n", + svc_print_addr(rqstp, buf, sizeof(buf))); + } return nfserr_perm; } =20 diff --git a/include/linux/sunrpc/debug.h b/include/linux/sunrpc/debug.h index 891f6173c951..e947d668f7b7 100644 --- a/include/linux/sunrpc/debug.h +++ b/include/linux/sunrpc/debug.h @@ -51,12 +51,10 @@ do { \ } \ } while (0) =20 -# define RPC_IFDEBUG(x) x #else # define ifdebug(fac) if (0) # define dfprintk(fac, fmt, ...) do {} while (0) # define dfprintk_rcu(fac, fmt, ...) do {} while (0) -# define RPC_IFDEBUG(x) #endif =20 /* diff --git a/net/sunrpc/xprtrdma/svc_rdma_transport.c b/net/sunrpc/xprtrdma= /svc_rdma_transport.c index 9b623849723e..f2d72181a6fe 100644 --- a/net/sunrpc/xprtrdma/svc_rdma_transport.c +++ b/net/sunrpc/xprtrdma/svc_rdma_transport.c @@ -414,7 +414,6 @@ static struct svc_xprt *svc_rdma_accept(struct svc_xprt= *xprt) struct ib_qp_init_attr qp_attr; struct ib_device *dev; int ret =3D 0; - RPC_IFDEBUG(struct sockaddr *sap); =20 listen_rdma =3D container_of(xprt, struct svcxprt_rdma, sc_xprt); clear_bit(XPT_CONN, &xprt->xpt_flags); @@ -560,18 +559,20 @@ static struct svc_xprt *svc_rdma_accept(struct svc_xp= rt *xprt) goto errout; } =20 -#if IS_ENABLED(CONFIG_SUNRPC_DEBUG) - dprintk("svcrdma: new connection accepted on device %s:\n", dev->name); - sap =3D (struct sockaddr *)&newxprt->sc_cm_id->route.addr.src_addr; - dprintk(" local address : %pIS:%u\n", sap, rpc_get_port(sap)); - sap =3D (struct sockaddr *)&newxprt->sc_cm_id->route.addr.dst_addr; - dprintk(" remote address : %pIS:%u\n", sap, rpc_get_port(sap)); - dprintk(" max_sge : %d\n", newxprt->sc_max_send_sges); - dprintk(" sq_depth : %d\n", newxprt->sc_sq_depth); - dprintk(" rdma_rw_ctxs : %d\n", ctxts); - dprintk(" max_requests : %d\n", newxprt->sc_max_requests); - dprintk(" ord : %d\n", conn_param.initiator_depth); -#endif + if (IS_ENABLED(CONFIG_SUNRPC_DEBUG)) { + struct sockaddr *sap; + + dprintk("svcrdma: new connection accepted on device %s:\n", dev->name); + sap =3D (struct sockaddr *)&newxprt->sc_cm_id->route.addr.src_addr; + dprintk(" local address : %pIS:%u\n", sap, rpc_get_port(sap)); + sap =3D (struct sockaddr *)&newxprt->sc_cm_id->route.addr.dst_addr; + dprintk(" remote address : %pIS:%u\n", sap, rpc_get_port(sap)); + dprintk(" max_sge : %d\n", newxprt->sc_max_send_sges); + dprintk(" sq_depth : %d\n", newxprt->sc_sq_depth); + dprintk(" rdma_rw_ctxs : %d\n", ctxts); + dprintk(" max_requests : %d\n", newxprt->sc_max_requests); + dprintk(" ord : %d\n", conn_param.initiator_depth); + } =20 return &newxprt->sc_xprt; =20 --=20 2.50.1