On Mon, Nov 28, 2016 at 07:51:53PM +0100, Adam Borowski wrote: > People who don't frequent IRC nor the mailing list tend to believe RAID 5/6 > are stable; this leads to data loss. Thus, let's do warn them. > > At this point, I think fiery letters that won't be missed are warranted. > > Kernel 4.9 and its -progs will be a part of LTS of multiple distributions, > so leaving experimental features without a warning is inappropriate.
I'm ok with adding the warning about raid56 feature, but I have some comments to how it's implemented. Special case warning for the raid56 is ok, as it corresponds to the 'mkfs_features' table where the missing value for 'safe' should lead to a similar warning. This is planned to be more generic, so I just want to make sure we can adjust it later without problems. The warning should go last, after the final summary (and respect verbosity level). If the message were not colored, I'd completely miss the warning. This also means the warning should not be printed from a helper function and not during the option parsing phase. The colors seem a bit too much to me, red text or just emphasize 'warning' would IMHO suffice. -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html