summaryrefslogtreecommitdiffstats
path: root/fs/nfs/flexfilelayout/flexfilelayoutdev.c
diff options
context:
space:
mode:
authorTrond Myklebust <trond.myklebust@primarydata.com>2016-12-06 12:00:51 -0500
committerTrond Myklebust <trond.myklebust@primarydata.com>2016-12-07 13:41:58 -0500
commitcb067935175ca477380806dc80bf5f0bb51f6f71 (patch)
tree12a1e4f89c0e18f47d83195d411639cecb9bc1c1 /fs/nfs/flexfilelayout/flexfilelayoutdev.c
parent7a0566b38c518e98df2359c8c15c2b3f91a4d67e (diff)
downloadop-kernel-dev-cb067935175ca477380806dc80bf5f0bb51f6f71.zip
op-kernel-dev-cb067935175ca477380806dc80bf5f0bb51f6f71.tar.gz
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/flexfilelayoutdev.c')
-rw-r--r--fs/nfs/flexfilelayout/flexfilelayoutdev.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/fs/nfs/flexfilelayout/flexfilelayoutdev.c b/fs/nfs/flexfilelayout/flexfilelayoutdev.c
index eb98395..142bfd0 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);
OpenPOWER on IntegriCloud