pNFS/flexfiles: Fix layoutcommit after a commit to DS
authorTrond Myklebust <trond.myklebust@primarydata.com>
Sun, 26 Jun 2016 16:39:49 +0000 (12:39 -0400)
committerTrond Myklebust <trond.myklebust@primarydata.com>
Tue, 5 Jul 2016 22:52:26 +0000 (18:52 -0400)
We should always do a layoutcommit after commit to DS, except if
the layout segment we're using has set FF_FLAGS_NO_LAYOUTCOMMIT.

Fixes: d67ae825a59d ("pnfs/flexfiles: Add the FlexFile Layout Driver")
Signed-off-by: Trond Myklebust <trond.myklebust@primarydata.com>
fs/nfs/flexfilelayout/flexfilelayout.c

index 0e8018bc98808bf29828e4861aee0c1ed9df3b31..2689c9e9dc3c23de4e5556183bdef3cd03d16d9b 100644 (file)
@@ -1530,8 +1530,7 @@ static int ff_layout_commit_done_cb(struct rpc_task *task,
                return -EAGAIN;
        }
 
-       if (data->verf.committed == NFS_UNSTABLE
-           && ff_layout_need_layoutcommit(data->lseg))
+       if (ff_layout_need_layoutcommit(data->lseg))
                pnfs_set_layoutcommit(data->inode, data->lseg, data->lwb);
 
        return 0;