summaryrefslogtreecommitdiffstats
path: root/fs/direct-io.c
diff options
context:
space:
mode:
authorDave Chinner <dchinner@redhat.com>2014-07-15 07:28:41 +1000
committerDave Chinner <david@fromorbit.com>2014-07-15 07:28:41 +1000
commit03e01349c654fbdea80d3d9b4ab599244eb55bb7 (patch)
treea2580ef8e0182c82f3dd84af81808ac33f42eb21 /fs/direct-io.c
parentcf11da9c5d374962913ca5ba0ce0886b58286224 (diff)
downloadop-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
OpenPOWER on IntegriCloud