David Sterba posted on Mon, 11 Jan 2016 16:51:08 +0100 as excerpted: > Hi > > On Tue, Jan 05, 2016 at 10:01:08AM +0800, Qu Wenruo wrote: >> Would you please consider merge this patch? >> >> As current output is already quite confusing for a lot of users who are >> not familiar with btrfs. > > I understand it's confusing but as there's some history behind this > option I need to think about the consequences. I'd rather keep it in the > output, removing stuff is likely to cause breakage in scripts etc. OTOH, > the 'fi df' was supposed to be a debugging helper but not everybody is > used to 'fi usage' yet. Not only is not everybody used to btrfs fi usage yet, but because it's still broken in various cases as well as relatively new, btrfs fi df along with btrfs fi sh remains the recommended way of getting that information. Of course we can't go back in time to introduce btrfs fi usage earlier, but the sooner it works on all btrfs, including mixed-mode, which I know it doesn't properly support yet, and raid56, which IIRC it didn't support but I can't directly test, without producing weird EiB readings, the sooner we'll be able to generally recommend it except for people using what will eventually be very old tools, where the first recommendation is to upgrade in any case. But for now, because fi usage doesn't work in a significant number of cases, recommending that people use and post fi df and fi sh remains the easiest way to dependably get the needed information, without going off into paragraphs of exception explanation. -- Duncan - List replies preferred. No HTML msgs. "Every nonfree program has a lord, a master -- and if you use the program, he is your master." Richard Stallman -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
