Use list_for_each_entry_safe() instead of list_for_each_safe() to
simplify the code.

Signed-off-by: Geliang Tang <geliangt...@163.com>
---
 mm/swapfile.c | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/mm/swapfile.c b/mm/swapfile.c
index 7c714c6..31dc94f 100644
--- a/mm/swapfile.c
+++ b/mm/swapfile.c
@@ -2957,11 +2957,10 @@ static void free_swap_count_continuations(struct 
swap_info_struct *si)
                struct page *head;
                head = vmalloc_to_page(si->swap_map + offset);
                if (page_private(head)) {
-                       struct list_head *this, *next;
-                       list_for_each_safe(this, next, &head->lru) {
-                               struct page *page;
-                               page = list_entry(this, struct page, lru);
-                               list_del(this);
+                       struct page *page, *next;
+
+                       list_for_each_entry_safe(page, next, &head->lru, lru) {
+                               list_del(&page->lru);
                                __free_page(page);
                        }
                }
-- 
2.5.0


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to