aboutsummaryrefslogtreecommitdiffstats
path: root/fs/lockd/clntlock.c
diff options
context:
space:
mode:
authorTrond Myklebust <Trond.Myklebust@netapp.com>2006-03-20 13:44:41 -0500
committerTrond Myklebust <Trond.Myklebust@netapp.com>2006-03-20 13:44:41 -0500
commit4c060b531006e0711db32a132d6ac7661594b280 (patch)
tree899f281dfec3da7e29ede2e76019b3861116714f /fs/lockd/clntlock.c
parent26bcbf965f857c710adafd16cf424f043006b5dd (diff)
lockd: Fix Oopses due to list manipulation errors.
The patch "stop abusing file_lock_list introduces a couple of bugs since the locks may be copied and need to be removed from the lists when they are destroyed. Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Diffstat (limited to 'fs/lockd/clntlock.c')
-rw-r--r--fs/lockd/clntlock.c7
1 files changed, 4 insertions, 3 deletions
diff --git a/fs/lockd/clntlock.c b/fs/lockd/clntlock.c
index 0fc0ee267b04..7cf41c1e1a88 100644
--- a/fs/lockd/clntlock.c
+++ b/fs/lockd/clntlock.c
@@ -213,11 +213,12 @@ reclaimer(void *ptr)
213 /* First, reclaim all locks that have been marked. */ 213 /* First, reclaim all locks that have been marked. */
214restart: 214restart:
215 list_for_each_entry_safe(fl, next, &host->h_reclaim, fl_u.nfs_fl.list) { 215 list_for_each_entry_safe(fl, next, &host->h_reclaim, fl_u.nfs_fl.list) {
216 list_del(&fl->fl_u.nfs_fl.list); 216 list_del_init(&fl->fl_u.nfs_fl.list);
217 217
218 nlmclnt_reclaim(host, fl);
219 if (signalled()) 218 if (signalled())
220 break; 219 continue;
220 if (nlmclnt_reclaim(host, fl) == 0)
221 list_add_tail(&fl->fl_u.nfs_fl.list, &host->h_granted);
221 goto restart; 222 goto restart;
222 } 223 }
223 224