zpl: Fix "df -i" to have better free inodes value
authorAndreas Dilger <adilger@whamcloud.com>
Fri, 16 Sep 2011 09:22:00 +0000 (03:22 -0600)
committerBrian Behlendorf <behlendorf1@llnl.gov>
Wed, 28 Sep 2011 18:27:10 +0000 (11:27 -0700)
commitbaab0630167f7539483af0f277aa6eeff39490d6
tree302b98efe26bc10719c594ff4a1ebce9757145a4
parentc70602f1eada6ae9d2c3442e3d469c7227d019ab
zpl: Fix "df -i" to have better free inodes value

Due to the confusion in Linux statfs between f_frsize and f_bsize
the blocks counts were changed to be in units of z_max_blksize
instead of SPA_MINBLOCKSIZE as it is on other platforms.

However, the free files calculation in zfs_statvfs() is limited by
the free blocks count, since each dnode consumes one block/sector.
This provided a reasonable estimate of free inodes, but on Linux
this meant that the free inodes count was underestimated by a large
amount, since 256 512-byte dnodes can fit into a 128kB block, and
more if the max blocksize is increased to 1MB or larger.

Also, the use of SPA_MINBLOCKSIZE is semantically incorrect since
DNODE_SIZE may change to a value other than SPA_MINBLOCKSIZE and
may even change per dataset, and devices with large sectors setting
ashift will also use a larger blocksize.

Correct the f_ffree calculation to use (availbytes >> DNODE_SHIFT)
to more accurately compute the maximum number of dnodes that can
be created.

Signed-off-by: Andreas Dilger <adilger@whamcloud.com>
Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
Closes #413
Closes #400
module/zfs/zfs_vfsops.c