From 19d3a1778cb6014f776d26b37738610c90c893fd Mon Sep 17 00:00:00 2001
From: Kent Overstreet <kent.overstreet@linux.dev>
Date: Wed, 26 Mar 2025 10:15:24 -0400
Subject: [PATCH] fsck: Kill -R, --reconstruct_alloc
We don't want users hitting this accidentally, and if needed for some
strange reason (it shouldn't be), it can still be specified via -o.
Signed-off-by: Kent Overstreet <kent.overstreet@linux.dev>
---
c_src/cmd_fsck.c | 7 +------
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/c_src/cmd_fsck.c b/c_src/cmd_fsck.c
index 859ec731..6166328c 100644
--- a/c_src/cmd_fsck.c
+++ b/c_src/cmd_fsck.c
@@ -21,7 +21,6 @@ static void fsck_usage(void)
" -y Assume \"yes\" to all questions\n"
" -f Force checking even if filesystem is marked clean\n"
" -r, --ratelimit_errors Don't display more than 10 errors of a given type\n"
- " -R, --reconstruct_alloc Reconstruct the alloc btree\n"
" -k, --kernel Use the in-kernel fsck implementation\n"
" -v Be verbose\n"
" -h, --help Display this help and exit\n"
@@ -205,7 +204,6 @@ int cmd_fsck(int argc, char *argv[])
{
static const struct option longopts[] = {
{ "ratelimit_errors", no_argument, NULL, 'r' },
- { "reconstruct_alloc", no_argument, NULL, 'R' },
{ "kernel", no_argument, NULL, 'k' },
{ "no-kernel", no_argument, NULL, 'K' },
{ "help", no_argument, NULL, 'h' },
@@ -224,7 +222,7 @@ int cmd_fsck(int argc, char *argv[])
append_opt(&opts_str, "read_only");
while ((opt = getopt_long(argc, argv,
- "apynfo:rRkKvh",
+ "apynfo:rkKvh",
longopts, NULL)) != -1)
switch (opt) {
case 'a':
@@ -248,9 +246,6 @@ int cmd_fsck(int argc, char *argv[])
case 'r':
append_opt(&opts_str, "ratelimit_errors");
break;
- case 'R':
- append_opt(&opts_str, "reconstruct_alloc");
- break;
case 'k':
kernel = true;
break;