quota: make Q_XQUOTASYNC a noop
authorChristoph Hellwig <hch@infradead.org>
Mon, 20 Feb 2012 02:28:18 +0000 (02:28 +0000)
committerBen Myers <bpm@sgi.com>
Wed, 29 Feb 2012 20:10:42 +0000 (14:10 -0600)
Now that XFS takes quota reservations into account there is no need to flush
anything before reporting quotas - in addition to beeing fully transactional
all quota information is also 100% coherent with the rest of the filesystem
now.

Reviewed-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Ben Myers <bpm@sgi.com>
fs/quota/quota.c

index 7898cd688a0073b8caf6c193fc3deb1dc8722f5f..7a9bedeb1d571e1cd713f5c6fcb26547c9f46813 100644 (file)
@@ -282,10 +282,9 @@ static int do_quotactl(struct super_block *sb, int type, int cmd, qid_t id,
        case Q_XGETQUOTA:
                return quota_getxquota(sb, type, id, addr);
        case Q_XQUOTASYNC:
-               /* caller already holds s_umount */
                if (sb->s_flags & MS_RDONLY)
                        return -EROFS;
-               writeback_inodes_sb(sb, WB_REASON_SYNC);
+               /* XFS quotas are fully coherent now, making this call a noop */
                return 0;
        default:
                return -EINVAL;