From: Eric Sandeen <sandeen@redhat.com>
To: Dan McGrath <danmcgrath.ca@gmail.com>
Cc: Jan Alexander Steffens <jan.steffens@gmail.com>,
linux-btrfs@vger.kernel.org
Subject: Re: [PATCH] btrfs-progs: No-op when called as fsck.btrfsck
Date: Mon, 15 Apr 2013 12:01:01 -0500 [thread overview]
Message-ID: <516C324D.30200@redhat.com> (raw)
In-Reply-To: <CAK82gMGSGaNywR=RkEQcNbQiMW5-z2O3ZqkMwKtamEdRH+A7aQ@mail.gmail.com>
On 4/15/13 11:45 AM, Dan McGrath wrote:
> Jan,
>
> I got a chance to sit down and dig a little bit deeper into
> `fsck.xfs`. Here is what I discovered.
>
> The "(a|A|y|p)" options in the XFS script appear to be nothing more
> than the expected `fsck` options that imply automated checks (as is
> clearly implied by the use of AUTO). While I have yet to specifically
> test the capitalized "A", my guess is that it matches the "-A" options
> from fsck(8) for when the system is going through the fstab. The
> syntax itself appears to assume that the dev name is the last param
> (as indicated by the argc/$#, which gets eval'd into the DEV
> variable).
>
> After doing some tests with a hacked up version of the `fsck.xfs`
> script, it would appear that the generic `fsck` script calls each
> script in order and passes it some parameters to test, since if I pass
> `fsck` some random/btrfsck switches:
>
> # fsck --repair /dev/storage/lv_btrfs
>
> I get an error back from `fsck.ext4`:
>
> fsck from util-linux 2.20.1
> fsck.ext4: invalid option -- 'e'
2 things; from the fsck manpage:
fsck [-sAVRTMNP] [-C [fd]] [-t fstype] [filesys...] [--] [fs-specific-options]
so I think you need:
fsck -- --repair /dev/storage/lv_btrfs
But the other issues seems to be that fsck & blkid are autodetecting
the device as ext4, not btrfs; a separate issue.
-Eric
next prev parent reply other threads:[~2013-04-15 17:01 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-04-15 13:47 [PATCH] btrfs-progs: No-op when called as fsck.btrfsck Dan McGrath
2013-04-15 13:47 ` Dan McGrath
2013-04-15 14:03 ` Jan Alexander Steffens
2013-04-15 14:10 ` Dan McGrath
2013-04-15 16:38 ` Eric Sandeen
2013-04-15 16:45 ` Dan McGrath
2013-04-15 17:01 ` Eric Sandeen [this message]
2013-04-15 17:23 ` Dan McGrath
2013-04-15 18:59 ` Zach Brown
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=516C324D.30200@redhat.com \
--to=sandeen@redhat.com \
--cc=danmcgrath.ca@gmail.com \
--cc=jan.steffens@gmail.com \
--cc=linux-btrfs@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.