NFSv4/pNFS: Always allow update of a zero valued layout barrier

A zero value for the layout barrier indicates that it has been cleared
(since seqid '0' is an illegal value), so we should always allow it to
be updated.

Fixes: d29b468da4f9 ("pNFS/NFSv4: Improve rejection of out-of-order layouts")
Signed-off-by: Trond Myklebust <trond.myklebust@hammerspace.com>
Signed-off-by: Anna Schumaker <Anna.Schumaker@Netapp.com>

diff --git a/fs/nfs/pnfs.c b/fs/nfs/pnfs.c
index 856706180d338e22fbcec8710667b715ff106999..7775f6b5a53adc13971c49b5398bbf2f6d481a0b 100644 (file)
--- a/fs/nfs/pnfs.c
+++ b/fs/nfs/pnfs.c
@@ -335,7 +335,7 @@ static bool pnfs_seqid_is_newer(u32 s1, u32 s2)
 
 static void pnfs_barrier_update(struct pnfs_layout_hdr *lo, u32 newseq)
 {
-       if (pnfs_seqid_is_newer(newseq, lo->plh_barrier))
+       if (pnfs_seqid_is_newer(newseq, lo->plh_barrier) || !lo->plh_barrier)
                lo->plh_barrier = newseq;
 }