diff options
author | Pavel Emelyanov <xemul@openvz.org> | 2007-09-19 08:44:07 -0400 |
---|---|---|
committer | J. Bruce Fields <bfields@citi.umich.edu> | 2007-10-09 18:32:45 -0400 |
commit | f0c1cd0eaf0b127356c2c09e40305453bc361b0f (patch) | |
tree | e522f8d83b439442aeb8c1b9c2a94941f6a95b1a | |
parent | 02888f41e9d7fa95d1f5b2f76e0f0af6ea8198cc (diff) |
Use list_first_entry in locks_wake_up_blocks
This routine deletes all the elements from the list
with the "while (!list_empty())" loop, and we already
have a list_first_entry() macro to help it look nicer :)
Signed-off-by: Pavel Emelyanov <xemul@openvz.org>
-rw-r--r-- | fs/locks.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/fs/locks.c b/fs/locks.c index c7c69d29a576..282b6c11670a 100644 --- a/fs/locks.c +++ b/fs/locks.c | |||
@@ -534,7 +534,9 @@ static void locks_insert_block(struct file_lock *blocker, | |||
534 | static void locks_wake_up_blocks(struct file_lock *blocker) | 534 | static void locks_wake_up_blocks(struct file_lock *blocker) |
535 | { | 535 | { |
536 | while (!list_empty(&blocker->fl_block)) { | 536 | while (!list_empty(&blocker->fl_block)) { |
537 | struct file_lock *waiter = list_entry(blocker->fl_block.next, | 537 | struct file_lock *waiter; |
538 | |||
539 | waiter = list_first_entry(&blocker->fl_block, | ||
538 | struct file_lock, fl_block); | 540 | struct file_lock, fl_block); |
539 | __locks_delete_block(waiter); | 541 | __locks_delete_block(waiter); |
540 | if (waiter->fl_lmops && waiter->fl_lmops->fl_notify) | 542 | if (waiter->fl_lmops && waiter->fl_lmops->fl_notify) |