NFS: Background flush should not be low priority
authorTrond Myklebust <trond.myklebust@primarydata.com>
Mon, 28 Dec 2015 18:30:42 +0000 (13:30 -0500)
committerTrond Myklebust <trond.myklebust@primarydata.com>
Mon, 28 Dec 2015 18:30:42 +0000 (13:30 -0500)
Background flush is needed in order to satisfy the global page limits.
Don't subvert by reducing the priority.
This should also address a write starvation issue that was reported by
Neil Brown.

Signed-off-by: Trond Myklebust <trond.myklebust@primarydata.com>
fs/nfs/write.c

index 7b93164069307346ea43f130e3883959ae6576b6..7a4fe7d82e65e4463d290131998e8b429971bb53 100644 (file)
@@ -247,8 +247,6 @@ static int wb_priority(struct writeback_control *wbc)
                return FLUSH_HIGHPRI | FLUSH_STABLE;
        if (wbc->sync_mode == WB_SYNC_ALL)
                ret = FLUSH_COND_STABLE;
-       if (wbc->for_kupdate || wbc->for_background)
-               ret |= FLUSH_LOWPRI;
        return ret;
 }