linux-btrfs.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Mike <ipso@snappymail.ca>
To: Chris Mason <chris.mason@oracle.com>,
	Jeff Putney <jeffrey.putney@gmail.com>,
	linux-btrfs <linux-btrfs@vger.kernel.org>
Subject: Re: Honest timeline for btrfsck
Date: Fri, 07 Oct 2011 08:39:36 -0700	[thread overview]
Message-ID: <4E8F1D38.7030202@snappymail.ca> (raw)
In-Reply-To: <20111007025050.GA4767@shiny.Netlink.Wireless>

On 11-10-06 07:50 PM, Chris Mason wrote:
> That's how software goes sometimes, and I'll take the criticism because
> it hasn't gone as well as it should have.  But, I can't stress enough how
> much I appreciate everyone's contributions and interest in btrfs.

With all due respect Chris, your actions and your words seem to 
contradict each other. It would appear that people are wanting to help 
contribute, but without showing them the code, you're preventing any 
contributions from happening. As you know, contributing is more than 
just code, just as important is proper testing, especially with a fsck tool.

I also don't think you are giving people enough credit. e2fsck will 
cause corruption pretty much everytime its run on a mounted file system, 
but a nice big nasty warning message seems to handle that quite well and 
anyone who ignores it, well thats their own fault, not the developers:

e2fsck 1.41.14 (22-Dec-2010)
/dev/sdb1 is mounted.

WARNING!!!  The filesystem is mounted.   If you continue you ***WILL***
cause ***SEVERE*** filesystem damage.

Do you really want to continue (y/n)? cancelled!

You could easily place the same warning in btrfs fsck even for normal 
use and recommend/require that it be run on a loopback image rather than 
the actual data itself or something. Heck, even have it run in "make no 
changes" mode by default and require recompiling to enable "fix my 
filesystem" mode.

In fact, when its first released that would probably be a good idea to 
do this anyways. The reality is, it doesn't matter how long you work on 
the fsck tool, its pretty much guaranteed to be a few bugs that corrupt 
some peoples data even more than it was before, thats the price you pay 
for being on the bleeding edge.

Don't get me wrong, I definitely appreciate all your work, I just wish I 
could appreciate it even more with a fsck tool. ;)

-- Mike

  parent reply	other threads:[~2011-10-07 15:39 UTC|newest]

Thread overview: 76+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-08-03  6:57 Honest timeline for btrfsck Erik Jensen
2011-08-03  9:09 ` Jan Schmidt
2011-08-03 20:53 ` Chris Mason
2011-08-15 14:22   ` Francesco Riosa
2011-08-17 15:19   ` Dave
2011-08-18  1:09   ` Yalonda Gishtaka
2011-08-18 20:50     ` Chris Mason
2011-08-18 21:22       ` Hugo Mills
2011-08-26  0:39         ` Yalonda Gishtaka
2011-08-21 13:58       ` Maciej Marcin Piechotka
2011-08-25 15:06       ` Michael Cronenworth
2011-09-01 19:14         ` Michael Cronenworth
2011-09-01 20:20           ` Hugo Mills
2011-09-01 20:24             ` Michael Cronenworth
2011-09-01 20:34               ` Hugo Mills
2011-09-10 10:09                 ` Martin Steigerwald
2011-09-13 18:01                   ` Jeff Putney
2011-10-05  6:16                     ` Chris Mason
2011-10-05 13:59                       ` Jeff Putney
2011-10-05 14:58                         ` Chris Mason
2011-10-06 15:31                           ` Jeff Putney
2011-10-06 20:30                             ` Andi Kleen
2011-10-06 20:33                               ` Jeff Mahoney
2011-10-06 20:56                               ` Francesco Riosa
2011-10-07 14:50                                 ` Josef Bacik
2011-10-07 15:22                                   ` Dave
2011-10-11 21:21                                   ` Francesco Riosa
2011-10-12 13:53                                     ` Josef Bacik
2011-10-13 12:57                                       ` Francesco Riosa
2011-10-13 13:02                                         ` Josef Bacik
2011-10-06 20:52                             ` Randy Barlow
2011-10-06 23:20                             ` Yalonda Gishtaka
2011-10-06 23:29                               ` Chris Samuel
2011-10-07  4:30                               ` Roman Mamedov
2011-10-07  2:25                             ` Chester
2011-10-07 19:10                               ` Asdo
2011-10-07 19:29                                 ` cwillu
2011-10-07 20:19                                 ` Diego Calleja
2011-10-08 21:13                                   ` Asdo
2011-10-09  1:19                                     ` Fajar A. Nugraha
2011-10-07 20:50                                 ` Helmut Hullen
2011-10-10 12:59                                 ` Chris Mason
2011-10-07  2:50                             ` Chris Mason
2011-10-07  4:45                               ` Jeff Mahoney
2011-10-07 13:40                               ` Jeff Putney
2011-10-07 14:48                                 ` Josef Bacik
2011-10-07 15:58                                   ` Jeff Putney
2011-10-07 16:08                                     ` Josef Bacik
2011-10-07 17:07                                       ` Jeff Putney
2011-10-07 18:23                                         ` cwillu
2011-10-07 21:16                                           ` Jeff Putney
2011-10-10 12:55                                     ` Chris Mason
2011-10-13 11:28                                   ` Chris Samuel
2011-10-13 11:37                                     ` Hugo Mills
2011-10-07 15:39                               ` Mike [this message]
2011-10-07 17:27                                 ` Gour-Gadadhara Dasa
2011-10-12 14:41                                   ` Martin Steigerwald
2011-10-12 18:57                                     ` Jeff Putney
2011-10-12 19:53                                       ` Martin Steigerwald
2011-10-12 22:47                                         ` Jeff Putney
2011-10-13  5:56                                           ` Jeff Mahoney
2011-10-13 15:51                                             ` Jeff Putney
2011-10-17 10:49                                               ` Chris Samuel
2011-10-31 10:53                       ` David Summers
2011-11-30 10:19                         ` Clemens Eisserer
2011-12-02 20:05                           ` Jeff Putney
2012-01-06 23:03                       ` Danny Piccirillo
2011-09-09 23:01           ` Yalonda Gishtaka
2011-09-23 13:51       ` Erik Jensen
2011-09-27 14:42       ` Jeff Putney
2011-09-27 18:00         ` Clemens Eisserer
2011-10-04 21:20           ` Jeff Putney
2012-01-17 15:07       ` David Summers
2012-01-18  1:13         ` Chris Mason
2012-03-28  6:15           ` Danny Piccirillo
2012-03-28  9:36             ` Duncan

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=4E8F1D38.7030202@snappymail.ca \
    --to=ipso@snappymail.ca \
    --cc=chris.mason@oracle.com \
    --cc=jeffrey.putney@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).