diff options
author | Dave Chinner <dchinner@redhat.com> | 2014-07-15 07:28:41 +1000 |
---|---|---|
committer | Dave Chinner <david@fromorbit.com> | 2014-07-15 07:28:41 +1000 |
commit | 03e01349c654fbdea80d3d9b4ab599244eb55bb7 (patch) | |
tree | a2580ef8e0182c82f3dd84af81808ac33f42eb21 /fs/direct-io.c | |
parent | cf11da9c5d374962913ca5ba0ce0886b58286224 (diff) | |
download | op-kernel-dev-03e01349c654fbdea80d3d9b4ab599244eb55bb7.zip op-kernel-dev-03e01349c654fbdea80d3d9b4ab599244eb55bb7.tar.gz |
xfs: null unused quota inodes when quota is on
When quota is on, it is expected that unused quota inodes have a
value of NULLFSINO. The changes to support a separate project quota
in 3.12 broken this rule for non-project quota inode enabled
filesystem, as the code now refuses to write the group quota inode
if neither group or project quotas are enabled. This regression was
introduced by commit d892d58 ("xfs: Start using pquotaino from the
superblock").
In this case, we should be writing NULLFSINO rather than nothing to
ensure that we leave the group quota inode in a valid state while
quotas are enabled.
Failure to do so doesn't cause a current kernel to break - the
separate project quota inodes introduced translation code to always
treat a zero inode as NULLFSINO. This was introduced by commit
0102629 ("xfs: Initialize all quota inodes to be NULLFSINO") with is
also in 3.12 but older kernels do not do this and hence taking a
filesystem back to an older kernel can result in quotas failing
initialisation at mount time. When that happens, we see this in
dmesg:
[ 1649.215390] XFS (sdb): Mounting Filesystem
[ 1649.316894] XFS (sdb): Failed to initialize disk quotas.
[ 1649.316902] XFS (sdb): Ending clean mount
By ensuring that we write NULLFSINO to quota inodes that aren't
active, we avoid this problem. We have to be really careful when
determining if the quota inodes are active or not, because we don't
want to write a NULLFSINO if the quota inodes are active and we
simply aren't updating them.
Signed-off-by: Dave Chinner <dchinner@redhat.com>
Reviewed-by: Brian Foster <bfoster@redhat.com>
Signed-off-by: Dave Chinner <david@fromorbit.com>
Diffstat (limited to 'fs/direct-io.c')
0 files changed, 0 insertions, 0 deletions