diff options
| author | Trond Myklebust <trond.myklebust@primarydata.com> | 2016-12-06 18:00:51 +0100 |
|---|---|---|
| committer | Trond Myklebust <trond.myklebust@primarydata.com> | 2016-12-07 19:41:58 +0100 |
| commit | cb067935175ca477380806dc80bf5f0bb51f6f71 (patch) | |
| tree | 12a1e4f89c0e18f47d83195d411639cecb9bc1c1 /fs/nfs/flexfilelayout | |
| parent | NFSv4: Add missing nfs_put_lock_context() (diff) | |
| download | linux-cb067935175ca477380806dc80bf5f0bb51f6f71.tar.xz linux-cb067935175ca477380806dc80bf5f0bb51f6f71.zip | |
pNFS/flexfiles: Fix ff_layout_add_ds_error_locked()
When we're merging an old entry into our new entry, we want to ensure that
we add the list entry in the correct place.
Signed-off-by: Trond Myklebust <trond.myklebust@primarydata.com>
Diffstat (limited to 'fs/nfs/flexfilelayout')
| -rw-r--r-- | fs/nfs/flexfilelayout/flexfilelayoutdev.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/fs/nfs/flexfilelayout/flexfilelayoutdev.c b/fs/nfs/flexfilelayout/flexfilelayoutdev.c index eb98395c3651..142bfd0b1663 100644 --- a/fs/nfs/flexfilelayout/flexfilelayoutdev.c +++ b/fs/nfs/flexfilelayout/flexfilelayoutdev.c @@ -254,8 +254,9 @@ ff_layout_add_ds_error_locked(struct nfs4_flexfile_layout *flo, } /* Entries match, so merge "err" into "dserr" */ extend_ds_error(dserr, err->offset, err->length); - list_del(&err->list); + list_replace(&err->list, &dserr->list); kfree(err); + return; } list_add_tail(&dserr->list, head); |