Commit d2155fe5 authored by Liu Xiang's avatar Liu Xiang Committed by Linus Torvalds

mm: compaction: remove duplicate !list_empty(&sublist) check

The list_splice_tail(&sublist, freelist) also do !list_empty(&sublist)
check, so remove the duplicate call.

Link: https://lkml.kernel.org/r/20210609095409.19920-1-liu.xiang@zlingsmart.comSigned-off-by: default avatarLiu Xiang <liu.xiang@zlingsmart.com>
Reviewed-by: default avatarDavid Hildenbrand <david@redhat.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 17adb230
...@@ -1297,8 +1297,7 @@ move_freelist_head(struct list_head *freelist, struct page *freepage) ...@@ -1297,8 +1297,7 @@ move_freelist_head(struct list_head *freelist, struct page *freepage)
if (!list_is_last(freelist, &freepage->lru)) { if (!list_is_last(freelist, &freepage->lru)) {
list_cut_before(&sublist, freelist, &freepage->lru); list_cut_before(&sublist, freelist, &freepage->lru);
if (!list_empty(&sublist)) list_splice_tail(&sublist, freelist);
list_splice_tail(&sublist, freelist);
} }
} }
...@@ -1315,8 +1314,7 @@ move_freelist_tail(struct list_head *freelist, struct page *freepage) ...@@ -1315,8 +1314,7 @@ move_freelist_tail(struct list_head *freelist, struct page *freepage)
if (!list_is_first(freelist, &freepage->lru)) { if (!list_is_first(freelist, &freepage->lru)) {
list_cut_position(&sublist, freelist, &freepage->lru); list_cut_position(&sublist, freelist, &freepage->lru);
if (!list_empty(&sublist)) list_splice_tail(&sublist, freelist);
list_splice_tail(&sublist, freelist);
} }
} }
......
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