All of lore.kernel.org
 help / color / mirror / Atom feed
From: Marc MERLIN <marc@merlins.org>
To: Chris Murphy <lists@colorremedies.com>
Cc: Btrfs BTRFS <linux-btrfs@vger.kernel.org>,
	Hugo Mills <hugo@carfax.org.uk>,
	Eric Sandeen <sandeen@sandeen.net>
Subject: Re: coredump in btrfsck
Date: Sat, 4 Jan 2014 22:13:53 -0800	[thread overview]
Message-ID: <20140105061353.GD11749@merlins.org> (raw)
In-Reply-To: <DC2CCC7E-46BF-428E-BACE-5A7D5931BEC1@colorremedies.com>

On Fri, Jan 03, 2014 at 05:14:56PM -0700, Chris Murphy wrote:
> 
> On Jan 3, 2014, at 5:33 AM, Marc MERLIN <marc@merlins.org> wrote:
> > 
> > Would it be possible for whoever maintains btrfs-tools to change both
> > the man page and the help included in the tool to clearly state that
> > running the fsck tool is unlikely to be the right course of action
> > and talk about btrfs-zero-log as well as mount -o recovery?
> 
> The problem FAQ doesn't even mention btrfsck so I think people are just getting around that page or making assumptions.
> https://btrfs.wiki.kernel.org/index.php/Problem_FAQ

It's easy to find btrfsck without the wiki, whether it's with dpkg -l,
rpm -ql, or command line completion.
My point is that as you said, it's most often not the command to use, it
can even do more damage than good, but neither its command line help,
nor the man page warn of anything dangerous or bad in using it.

Telling people they should have read a wiki instead of the canonical man
page isn't the right way to go longer term, nor how things are done on
linux usually.
 
> Should btrfs check (btrfsck without --repair) work similar to xfs_repair when the file system is not cleanly unmounted? If an XFS volume is not cleanly unmounted, running xfs_repair will instruct the user to first mount the volume so that the journal is replayed, then umount the volume, then run xfs_repair.

I don't know about what the actual tool does when it works, I've never
had it do anything useful for me, so I can't comment, except about the
fact that it should warn users about "I'm not the fsck you're used to or
are likely looking for"

> A possible variant of this for btrfs check: inform the user the first step in repairing a problem Btrfs volume is to use -o recovery, for more information see Btrfs FAQ <url> for additional problem solving recommendations.

Yes, along with tweaking the man page to say the same.

Thanks,
Marc
-- 
"A mouse is a device used to point at the xterm you want to type in" - A.S.R.
Microsoft is to operating systems ....
                                      .... what McDonalds is to gourmet cooking
Home page: http://marc.merlins.org/                         | PGP 1024R/763BE901

      reply	other threads:[~2014-01-05  8:35 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-01-01 21:27 coredump in btrfsck Oliver Mangold
2014-01-01 21:58 ` Chris Murphy
2014-01-01 22:35   ` Oliver Mangold
2014-01-02 17:37     ` Chris Murphy
2014-01-03 12:33       ` Marc MERLIN
2014-01-04  0:14         ` Chris Murphy
2014-01-05  6:13           ` Marc MERLIN [this message]

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=20140105061353.GD11749@merlins.org \
    --to=marc@merlins.org \
    --cc=hugo@carfax.org.uk \
    --cc=linux-btrfs@vger.kernel.org \
    --cc=lists@colorremedies.com \
    --cc=sandeen@sandeen.net \
    /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.