diff options
author | J. Bruce Fields <bfields@fieldses.org> | 2006-03-26 04:37:24 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-03-26 11:56:56 -0500 |
commit | 6dc0fe8f8b40854982929e4f24d8c65115769b60 (patch) | |
tree | a3489bf4765799778238e1fee3de71ec6a0e1d73 /fs/locks.c | |
parent | fa3536cc144c1298f2ed9416c33f3b77fa2cd37a (diff) |
[PATCH] VFS,fs/locks.c: cleanup locks_insert_block
BUG instead of handling a case that should never happen.
Signed-off-by: J. Bruce Fields <bfields@citi.umich.edu>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'fs/locks.c')
-rw-r--r-- | fs/locks.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/fs/locks.c b/fs/locks.c index 709450a7b89d..4badf6a0e7b6 100644 --- a/fs/locks.c +++ b/fs/locks.c | |||
@@ -533,12 +533,7 @@ static void locks_delete_block(struct file_lock *waiter) | |||
533 | static void locks_insert_block(struct file_lock *blocker, | 533 | static void locks_insert_block(struct file_lock *blocker, |
534 | struct file_lock *waiter) | 534 | struct file_lock *waiter) |
535 | { | 535 | { |
536 | if (!list_empty(&waiter->fl_block)) { | 536 | BUG_ON(!list_empty(&waiter->fl_block)); |
537 | printk(KERN_ERR "locks_insert_block: removing duplicated lock " | ||
538 | "(pid=%d %Ld-%Ld type=%d)\n", waiter->fl_pid, | ||
539 | waiter->fl_start, waiter->fl_end, waiter->fl_type); | ||
540 | __locks_delete_block(waiter); | ||
541 | } | ||
542 | list_add_tail(&waiter->fl_block, &blocker->fl_block); | 537 | list_add_tail(&waiter->fl_block, &blocker->fl_block); |
543 | waiter->fl_next = blocker; | 538 | waiter->fl_next = blocker; |
544 | if (IS_POSIX(blocker)) | 539 | if (IS_POSIX(blocker)) |