]> granicus.if.org Git - zfs/commit
Fix statfs(2) for 32-bit user space
authorBrian Behlendorf <behlendorf1@llnl.gov>
Tue, 25 Sep 2018 00:11:25 +0000 (17:11 -0700)
committerGitHub <noreply@github.com>
Tue, 25 Sep 2018 00:11:25 +0000 (17:11 -0700)
commite897a23eb13bafcf9c38d2fc37ae57a7729d9a02
treec0370c4ecee2e54a40a5e4a273bee72ecdca8220
parent36e369ecb847570d6939073663c4e95406223c7f
Fix statfs(2) for 32-bit user space

When handling a 32-bit statfs() system call the returned fields,
although 64-bit in the kernel, must be limited to 32-bits or an
EOVERFLOW error will be returned.

This is less of an issue for block counts since the default
reported block size in 128KiB. But since it is possible to
set a smaller block size, these values will be scaled as
needed to fit in a 32-bit unsigned long.

Unlike most other filesystems the total possible file counts
are more likely to overflow because they are calculated based
on the available free space in the pool. In order to prevent
this the reported value must be capped at 2^32-1. This is
only for statfs(2) reporting, there are no changes to the
internal ZFS limits.

Reviewed-by: Andreas Dilger <andreas.dilger@whamcloud.com>
Reviewed-by: Richard Yao <ryao@gentoo.org>
Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
Issue #7927
Closes #7122
Closes #7937
config/kernel-in-compat-syscall.m4 [new file with mode: 0644]
config/kernel.m4
include/linux/vfs_compat.h
module/zfs/zfs_vfsops.c
module/zfs/zpl_super.c