Commit ec87b42f authored by Nikolay Borisov's avatar Nikolay Borisov Committed by David Sterba

btrfs: use list_last_entry in add_falloc_range

Instead of calling list_entry with head->prev simply call
list_last_entry which makes it obvious which member of the list is
being referred. This allows to remove the extra 'prev' pointer.
Signed-off-by: default avatarNikolay Borisov <nborisov@suse.com>
Reviewed-by: default avatarDavid Sterba <dsterba@suse.com>
Signed-off-by: default avatarDavid Sterba <dsterba@suse.com>
parent 4183abf6
...@@ -3034,7 +3034,6 @@ struct falloc_range { ...@@ -3034,7 +3034,6 @@ struct falloc_range {
*/ */
static int add_falloc_range(struct list_head *head, u64 start, u64 len) static int add_falloc_range(struct list_head *head, u64 start, u64 len)
{ {
struct falloc_range *prev = NULL;
struct falloc_range *range = NULL; struct falloc_range *range = NULL;
if (list_empty(head)) if (list_empty(head))
...@@ -3044,9 +3043,9 @@ static int add_falloc_range(struct list_head *head, u64 start, u64 len) ...@@ -3044,9 +3043,9 @@ static int add_falloc_range(struct list_head *head, u64 start, u64 len)
* As fallocate iterate by bytenr order, we only need to check * As fallocate iterate by bytenr order, we only need to check
* the last range. * the last range.
*/ */
prev = list_entry(head->prev, struct falloc_range, list); range = list_last_entry(head, struct falloc_range, list);
if (prev->start + prev->len == start) { if (range->start + range->len == start) {
prev->len += len; range->len += len;
return 0; return 0;
} }
insert: insert:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment