On Thu, 2014-07-03 at 18:51 +0200, David Sterba wrote: > On Thu, Jul 03, 2014 at 10:06:33AM +0800, Gui Hecheng wrote: > > Clear the seeding flag may cause the original filesystem to be writable, > > which is dangerous. > > Can you please describe the dangerous scenario a bit more? This would > also go to the documentation so it's not only to satisfy my curiosity.
Yes, I'll include a certain scenario in the changelog of a v2 patch. > Dropping the seeding flag could be dangerous if the filesystem starts in > seeding mode, a new device is added, some writes are done, then > filesystem is unmounted. > > Now it's a 2 device filesystem, where the orignal holds some data and > without the seeding flag it would accept new writes. Still ok for me, > though this is probably the time where some user assumptions may break. > > > In this case, add user confirmation check when clearing seeding flag. > > Also warn the user that the fs is in a dangerous condition when > > the seeding flag is cleared if it it forced to. > > The -y option is tied only to the seeding option, but it should IMO be > more general and called --force. I agree. > > Signed-off-by: Gui Hecheng <[email protected]> > > --- > > btrfstune.c | 24 +++++++++++++++++++++++- > > 1 file changed, 23 insertions(+), 1 deletion(-) > > > > diff --git a/btrfstune.c b/btrfstune.c > > index 3f2f0cd..0e18088 100644 > > --- a/btrfstune.c > > +++ b/btrfstune.c > > @@ -103,6 +104,7 @@ static void print_usage(void) > > fprintf(stderr, "\t-S value\tpositive value will enable seeding, zero > > to disable, negative is not allowed\n"); > > fprintf(stderr, "\t-r \t\tenable extended inode refs\n"); > > fprintf(stderr, "\t-x \t\tenable skinny metadata extent refs\n"); > > + fprintf(stderr, "\t-y \t\tsay yes to clear the seeding flag, make sure > > that you are aware of the danger\n"); > > The help text could say someting like > > "--force\tallow dangerous changes\n" > > btrfstune only allows setting the bit for extref and skinny-metadata, > unsetting would be dangerous as well. On my part, I don't find any scenarioes for these two, could you please remind me more? -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
