diff options
author | Trond Myklebust <Trond.Myklebust@netapp.com> | 2006-03-20 13:44:41 -0500 |
---|---|---|
committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2006-03-20 13:44:41 -0500 |
commit | 4c060b531006e0711db32a132d6ac7661594b280 (patch) | |
tree | 899f281dfec3da7e29ede2e76019b3861116714f /fs/lockd/clntlock.c | |
parent | 26bcbf965f857c710adafd16cf424f043006b5dd (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.c | 7 |
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. */ |
214 | restart: | 214 | restart: |
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 | ||