mm/percpu: use list_first_entry_or_null in pcpu_reclaim_populated()

To replace list_empty()/list_first_entry() pair to simplify code. Signed-off-by: Baoquan He <[email protected]> Acked-by: Dennis Zhou <[email protected]> Signed-off-by: Dennis Zhou <[email protected]>
wafgo · Nov 8, 2022 · c1f6688 · c1f6688
1 parent 5a7d596
commit c1f6688
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/mm/percpu.c b/mm/percpu.c
@@ -2143,9 +2143,9 @@ static void pcpu_reclaim_populated(void)
 	 * other accessor is the free path which only returns area back to the
 	 * allocator not touching the populated bitmap.
 	 */
-	while (!list_empty(&pcpu_chunk_lists[pcpu_to_depopulate_slot])) {
-		chunk = list_first_entry(&pcpu_chunk_lists[pcpu_to_depopulate_slot],
-					 struct pcpu_chunk, list);
+	while ((chunk = list_first_entry_or_null(
+			&pcpu_chunk_lists[pcpu_to_depopulate_slot],
+			struct pcpu_chunk, list))) {
 		WARN_ON(chunk->immutable);
 
 		/*