Re: [PATCH][BTRFS-PROGS][V1] btrfs filesystem df

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Oct 9, 2012 at 1:38 PM, Goffredo Baroncelli <kreijack@xxxxxxxxx> wrote:
> Hi Bart,
>
> I replayed in another email to Hugo about that. Basically I am not contrary
> to the change, only it is unrelated to my patches. In may patches I use the
> function pretty_sizes() which adds the suffix KB, MB, and this function was
> here from 2008....

What we could do is change all callers to use pretty_sizes_r(), which
most already do. We could then add a mode flag for SI units, say
DF_HUMAN_SI_UNIT. I would then propose that we enable it by default,
like the human-readable mode itself, and create an option to disable
it.

I like options a lot, but I like sensible defaults as well. We're
hoping btrfs will be the next major filesystem for linux. If we
succeed, it will have a long lifespan and be used on petabyte and
maybe exabyte storage systems. If we keep showing binary bytes, the
discrepancy between what the drive's box says and what 'df' says will
become ever larger. This is probably the last chance we'll get to make
btrfs set a good example by getting rid of this outdated method of
display.

I can make the changes to btrfs-progs if you like.

Best regards,

Bart
--
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


[Index of Archives]     [Linux Filesystem Development]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux