From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pg1-f181.google.com (mail-pg1-f181.google.com [209.85.215.181]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 3AD2241760 for ; Sat, 21 Mar 2026 14:15:28 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.215.181 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774102529; cv=none; b=VaQNhQegJYlcb1kEmvAsyEqlgmavc29XGa9/P1+x6sohnZOGXbrzeZmkDtc9GAwScVJdPhDjdjc2JKscA9fJPoAldWVCHnUn64dyUOp175nZgO6zoyCF/a8EVKTLumSGfOdTYqPNk4L6UvgwOXFzyTsDzZ5yQBlc71P9hTcGgFg= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774102529; c=relaxed/simple; bh=3+K/ApuxwjyEMBrtQSSDiQv9ECGp4Y+q1xsZnKf+JrU=; h=From:To:Cc:Subject:Date:Message-Id:MIME-Version; b=idDbjzfeOsKPBBrUXeOrBJg1TMZODVWLvuMOTL3Dt1cdJABBSi0V02fG1eL1OWsPn++qvpHZThcXlvM7tv1mMLiqWbdewOU+ajfs49oP3Ko7OoF8rayqZYqvCtLsmwM4i7+JVHw7tQmHAPdGYx2Rozdrj7sEKwDTFBfhhQgbRyo= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=RTfAMkmw; arc=none smtp.client-ip=209.85.215.181 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="RTfAMkmw" Received: by mail-pg1-f181.google.com with SMTP id 41be03b00d2f7-c742882d2a4so628410a12.0 for ; Sat, 21 Mar 2026 07:15:28 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1774102527; x=1774707327; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=UuapLcFBj9/2tM0hd60ExcnAcuLnRL+SJ5nrXPeI/3k=; b=RTfAMkmwC2+H3DbdKJDDZ2ynYzlTPr/6B4yrsKU834f02+kxycSySyQfZ4+q4poT5J CqbJ/3vLE8+rwNoUiDgFyIaGt+bku+ViMxdTWjaLlZxDOlOZtYqQGvKZVHcL/BwtMUFE PwcZjIwtAj+Ljh/JEb1wh9SUeA4/cs9+xUW/utrhV/82Nn6PDbBC0PR/Ljv9nd/qLCIC ew6nLhPCJzpul74+rDufHlg05BtD1JHdYmR3YrKxSnuPDL/C//fCBBbOp+NFAs5CYapI 4McTQku56GKI2Mcugqji+oAVkGV+ZNr52+FLwhqxcJvaYe8esLFBWZtxcplvZ93cRxee /mBQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1774102527; x=1774707327; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=UuapLcFBj9/2tM0hd60ExcnAcuLnRL+SJ5nrXPeI/3k=; b=lCFPj8DtBxA0N6mGgZxLjtVl7RM/CiurtqKh4/OjOl8eUv6yBqav0EOXmeldOpur4u 3QWfkZec7vyvRJGBw4nzwNbD4L4gNEPAxtdAAecikI8rzSnCTbZTb2KSW8D3vqatgmZS ihKLySw2zk0Rfckluyn5Tf5W3A2nuNwj64wiYfTzP78PO2cUIflQmZsmHH3Hp2wXpCtx TCad7wgzQ3cEY0/fdEfGzuNxzVzLrJrpJSPilWCkQYCF+/DAPSHcMbKYdfE6h4JCw2JE aQSQNQNOPk3yc8cGgdFt7xATr7W2PKIOV4ROznchdxmO8rXBMPqOAV62ptQ5mKlVvv/z BvSg== X-Forwarded-Encrypted: i=1; AJvYcCXZ/SUcSnpsbVo301LMFoRQfvzGyZl7Rix+bUH8LNRQm9zusxwyOkh3RMWPl7CEOanJYOxidOiFHvCauSI=@vger.kernel.org X-Gm-Message-State: AOJu0YxxxA2imOggOiW2xQHOpvjAr5/vIBcyhLY7ErdgHduZMXgG/x7g wfaUgKw/i/zrzinoPDLcZYKuyH2KLZ61dmB14A0fipTlyHkl6j3zilVt X-Gm-Gg: ATEYQzwSL1MIEqb9BD8WKkKx428INVzzEK/GngFsS4MEPXGazoyWxIy06Ll7Iqr+YPz 7BXB69VQjQDAY/nk98XRkx7KRdUEoi8ckvWE4ZremEipAw5L8Iho69V12CVsrl2sLvlzwqS+BER XisV6c9OWYWZH0JccSXzqilf2kClKZj3UUH2yWiw2MiYCW021oMHvh7nnVZgiEschfTBz5c21ri d757SYik2MgIKvlWvCI5ouryduu8KQcqIcjlqsfR4yOWreqZ00qcfIhgkIAcbbF3i6xV/U3axpE a5jnRHxVB4P2U1KwuLBW6DrZTJRSXpWCH4d5DxEwB8m6Tkn5NJFsTM3QzfQicgIBUJYEGK5sWN4 pRq7v7RDkKE06+/VdkXxAwjtoAOlIYbnH8EEYhTVuJoLOo3+y7zQDExPl45SgG6wWeK7okjojmv 4PHZZQpvCOsZt+rLpAHf4l7ijovIRi1jETsaHJ+mfIcyfPWGqhxIjTOhi/u2RD+1UTXb1+GOOA7 hd4RvKXjg== X-Received: by 2002:a17:902:e54a:b0:2ae:825b:49a5 with SMTP id d9443c01a7336-2b0825bf592mr63385375ad.0.1774102527511; Sat, 21 Mar 2026 07:15:27 -0700 (PDT) Received: from sean-All-Series.. (1-160-226-215.dynamic-ip.hinet.net. [1.160.226.215]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-2b08353e94asm52680715ad.25.2026.03.21.07.15.24 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 21 Mar 2026 07:15:27 -0700 (PDT) From: Sean Chang To: Andrew Lunn , Chuck Lever , David Laight , Anna Schumaker , Andy Shevchenko Cc: netdev@vger.kernel.org, linux-nfs@vger.kernel.org, linux-kernel@vger.kernel.org, Sean Chang Subject: [PATCH v5 0/5] sunrpc/nfs: cleanup redundant debug checks and refactor macros Date: Sat, 21 Mar 2026 22:15:05 +0800 Message-Id: <20260321141510.68214-1-seanwascoding@gmail.com> X-Mailer: git-send-email 2.34.1 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit This series cleans up redundant IS_ENABLED(CONFIG_SUNRPC_DEBUG) guards across sunrpc, nfsd, and lockd, as these checks are already handled within the dprintk macros. Additionally, it refactors the nfs_errorf() macros into a safer do-while(0) pattern and removes unused nfs_warnf() macros to improve code maintainability. v5: - Reformat the cleanup of __maybe_unused into a formal 'Revert' patch as requested. - Update the macro refactoring commit message to include historical context (commit ce8866f0913f) and use backticks for `git grep`. v4: - Add a missing patch to include/linux/sunrpc/debug.h to ensure dprintk() properly handles variable referencing via no_printk(). - Remove obsolete __maybe_unused from fs/nfsd/export.c (revert ebae102897e7) as suggested by Andy Shevchenko. - Add Reviewed-by and Tested-by tags from Andy Shevchenko. v3: - Added nfs_errorf refactoring and removed unused nfs_warnf macros. - Split sunrpc and nfsd changes for better clarity. v2: - Follow reversed xmas tree order for variables in svc_rdma_transport.c as requested by Andy Shevchenko. - Polish commit message: use dprintk() and remove redundant file list. - Correct the technical claim about dprintk() type checking. Sean Chang (5): sunrpc: Fix dprintk type mismatch using do-while(0) nfsd/lockd: Remove redundant debug checks svcrdma: Remove redundant IS_ENABLED(CONFIG_SUNRPC_DEBUG) guards nfs: Refactor nfs_errorf macros and remove unused ones Revert "nfsd: Mark variable __maybe_unused to avoid W=1 build break" fs/lockd/svclock.c | 7 ------ fs/nfs/internal.h | 28 +++++++++++------------- fs/nfsd/export.c | 2 +- fs/nfsd/nfsfh.c | 8 +++---- include/linux/sunrpc/debug.h | 8 ++----- net/sunrpc/xprtrdma/svc_rdma_transport.c | 25 ++++++++++----------- 6 files changed, 30 insertions(+), 48 deletions(-) -- 2.34.1