From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f50.google.com (mail-wm1-f50.google.com [209.85.128.50]) (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 0D5AE3563F6 for ; Sun, 6 Sep 2026 20:55:28 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.50 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788728130; cv=none; b=tRqhecXjlZBAj5tBarP1ZHJImRbS10LzeRHrgJrwRmMzWGIAA7Q9bHN4ZsXbqqNvU7DaYeBpsuCvWZjUuyKtHGKU3ZhSTseSDC7u55mzy2W21oUKEaHazqhMYX56Mh5tSoMeWu3aSgfmKui6EqkkXUCmT0mj/5TCZh7IGblW8zw= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788728130; c=relaxed/simple; bh=UmTnoTVu0uHk5ty8IRYct/UGRDry/oe29M2HASDGF6g=; h=Date:From:To:Cc:Subject:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=foj95nKiF1nna6AJH+/XfOmQtSV/JfzvGvqtj/kiAnlyDWqelq6WwIgINQP2TOn6aJS4PJ6f70mqwfbBbWeUjx+O/cf4rd9FJq0vvbPu7MQsJ218dB1HiN+U+UR7b8BiyPI0MkhMvra34ISshehBYWpk0F/ccJQJPGZAtmGphwc= 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=c0F1sp7d; arc=none smtp.client-ip=209.85.128.50 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="c0F1sp7d" Received: by mail-wm1-f50.google.com with SMTP id 5b1f17b1804b1-49b8687630fso23822975e9.3 for ; Sun, 06 Sep 2026 13:55:28 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1788728127; x=1789332927; darn=vger.kernel.org; h=content-transfer-encoding:content-type:mime-version:references :in-reply-to:message-id:subject:cc:to:from:date:from:to:cc:subject :date:message-id:reply-to:content-type; bh=KApqmYJmHvPUcT9v/KGmQBynW1X598SQBtzRKCAwt84=; b=c0F1sp7dnLqwtjm5XAFNbkHQ+bxKOx2LGyeHnWyw6qKh0mfiQ1jiiKgBpl3hRkXu+0 TOlSIP3XBmzokKZyBGMXk0sV/Nd8UWKLaGyNafmAlmI+IS+pSmTfaKdbHMu7Mg4gvtG9 Kc1E0+Sws+C22iWE4pBdu/Lkd1gdX4Yrw9KW3RP2kicXxSdccc9MeFEuJ6h84PZwgCJm MULyZiTiG9cjtFNAxleZILfmwElVrO7e1GL+vHaEgvde7Qta9xx1Rtjk9hnjy26h1goD +3s+pgok/TEAjsthzfzbFwTmswajL34c4iJ9aWYFRry49aHPRtvsbh/n3DqsPcZ29RG5 P1nw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1788728127; x=1789332927; h=content-transfer-encoding:content-type:mime-version:references :in-reply-to:message-id:subject:cc:to:from:date:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=KApqmYJmHvPUcT9v/KGmQBynW1X598SQBtzRKCAwt84=; b=sJ5hnjIc1TYMFKKdmCCnGPYC1xEJpsU/X/R1zGvgiHjaIe7zoJIEBQnR7iGqpbstAb rhbq1nG/QOgwl/PuBvr7WDugyr4jXiz8AmwF8Vs1/RxVRIBG1YsI2bXe7BXVcqHyEaLo +5RobGhVrnMkXp6dHqyDRONRiAXV5zoqsD4LjgMnZSDMOehvzAi7yVwkNX+ghWqLQ6lY rlblU4SlNP3TD/0PrZLhKA/L7BGRzF9ywO4sNc9nZtdIn9FvMMwqPCeonC1y6BEXTAAx amd97VYCUHr6XpTaJAotBUiy/6Fh+/aI9rCWgyQyEo+P9T7z+9cGKRSB9oa06sLSZFBt /p0A== X-Forwarded-Encrypted: i=1; AKwUvBzS6JJNU7D0SEGxhT4bSRstorQdvpaoxqm0H7lPoowSC/CgVZWlymyQAbBoMFeALM6Ien7xxQQrfNiUZhU=@vger.kernel.org X-Gm-Message-State: AFuF++mtCTqOr6zNltvAWzHA+6KcDuDF9stmz1bovCVMVvQ7Fkzh1SYN 1f79UMvIUEUjQVJs9HAMx+Pre36Ow/c+KfTxpT3Kg2gHndt2xiz61EVK5Pfttu/p X-Gm-Gg: AYBFou2aPBcdZ3lIkKqQ9/aPo8ijiruZXGVkpaBxC/ZZav10y3hBzqoxC4ygMA8aQWj XM2KuCIglrqsIXLZhlZopMrpDNON6+zCm4T1Pls5E0/TpcP78/SMn0oTlEjs1ZbooDcbxsWSPNU mYjEYdMWmXJU987wJ0IDRhgl3kgCZwq9fp3PvATJjnCBuPIT1/nahynijrEK3C9UErHEcZWq6wJ l02fvAylLmFquUajphy51gfQXHZYcWmqABkRflpL6BEXjby11pdIFD4SVroi25Za+rId38iWJYU bAM8Yms/YAvLUtlCNXYUEXAfgqBfLcOeIg6kQYNT1FqbJ436dIL4NyuIdFsHf5rRdq4NajiX8cY iKfSG8e+4WYIFZY8+D50p1558yT/qbMTcDyq0AhW32KogcURZ7HQ4yIKE9jXynmOgS6pbgxTItn Zu7BZ0tqNZ+nimv9qRmfiMf960g3R3C9ePFbYYpo9i4MoB+DEI8ko5Y27XoMtyaGOJ2jkhZe9c7 1KqU/3u01iJrMMsZlPNSNmaA3DOgTKq7MSshHKJ67S1BOMKm5sFh+7T4/fwlpDu2ea1lvznAtdB sjdSXefq2Mc5YInIslBb7QmjFfgpdguiY5DKonYXlCeVeUK4ldrz0wqEkkv7tSWrUds9s9mZQFO /xOsbjfXf5yTIUkRuLPHo X-Received: by 2002:a05:600c:1986:b0:49c:fa21:e744 with SMTP id 5b1f17b1804b1-49cfa21e906mr154347705e9.26.1788728127106; Sun, 06 Sep 2026 13:55:27 -0700 (PDT) Received: from systembl0wer ([2a02:8308:4092:11f0::f9f]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-49d0af538d0sm97914485e9.8.2026.09.06.13.55.26 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 06 Sep 2026 13:55:27 -0700 (PDT) Date: Sun, 6 Sep 2026 22:55:25 +0200 From: Joshua Crofts To: Ryusuke Konishi Cc: Viacheslav Dubeyko , linux-nilfs@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] nilfs2: remove '#if 0' code blocks Message-ID: <20260906225525.6d8b509e@systembl0wer> In-Reply-To: References: <20260905-nilfs2-dead-code-cleanup-v1-1-348ed97b2443@gmail.com> X-Mailer: Claws Mail 4.4.0 (GTK 3.24.52; x86_64-redhat-linux-gnu) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable On Mon, 7 Sep 2026 01:47:35 +0900 Ryusuke Konishi wrote: > On Sat, Sep 5, 2026 at 1:49=E2=80=AFPM Joshua Crofts wrote: > > > > There are two instances of '#if 0' code blocks that are dead code. > > Remove these code blocks. > > > > Additionally, remove unnecessary brackets around a single-statement > > if per checkpatch rules. > > > > Signed-off-by: Joshua Crofts > > --- > > Stumbled upon these blocks when grepping the kernel for #if 0. Not sure > > if they are actually used for debugging; if so, I have no issues with > > the patch being dropped. > > > > Thanks! =20 >=20 > Joshua Crofts, thank you for proposing this patch. >=20 > I agree that it is appropriate to remove the dead code in both locations. >=20 > These sections are not for debugging purposes; rather, they both > relate to features that were never implemented. >=20 > Regarding the ACL-related dead code, it can be properly implemented > when the feature is actually added; therefore, removing it is fine > (and indeed preferable, as it reduces the tree-wide maintenance burden > in the interim). >=20 > The dead code related to mount counts stems from a legacy feature > designed for periodic full checks upon mounting. > However, mandating a heavy, full-filesystem check simply because a > certain number of mounts has been reached is outdated; in an era where > large-capacity storage is the norm, such a process compromises > availability. > While fsck itself remains necessary, periodic health checks should be > handled via alternative online methods. >=20 > I have just one point regarding the ACL dead code removal: in > nilfs_new_inode() within "inode.c", there are lines that are merely > commented out rather than being enclosed in "#if 0" (shown below). >=20 > /* ii->i_file_acl =3D 0; */ > /* ii->i_dir_acl =3D 0; */ >=20 > Could you please revise your patch to remove these lines as well? >=20 > Once you have made these changes, I will accept the patch. >=20 Hi Ryusuke, Thanks for the explanation!=20 I'll implement the change you proposed in v2. --=20 Kind regards, Joshua Crofts