[PATCH] mm: PageLRU no testset
PG_lru is protected by zone->lru_lock. It does not need TestSet/TestClear operations. Signed-off-by: Nick Piggin <npiggin@suse.de> Signed-off-by: Andrew Morton <akpm@osdl.org> Signed-off-by: Linus Torvalds <torvalds@osdl.org>
This commit is contained in:
parent
46453a6e19
commit
8d438f96d2
|
@ -239,10 +239,9 @@ extern void __mod_page_state_offset(unsigned long offset, unsigned long delta);
|
|||
#define __ClearPageDirty(page) __clear_bit(PG_dirty, &(page)->flags)
|
||||
#define TestClearPageDirty(page) test_and_clear_bit(PG_dirty, &(page)->flags)
|
||||
|
||||
#define SetPageLRU(page) set_bit(PG_lru, &(page)->flags)
|
||||
#define PageLRU(page) test_bit(PG_lru, &(page)->flags)
|
||||
#define TestSetPageLRU(page) test_and_set_bit(PG_lru, &(page)->flags)
|
||||
#define TestClearPageLRU(page) test_and_clear_bit(PG_lru, &(page)->flags)
|
||||
#define SetPageLRU(page) set_bit(PG_lru, &(page)->flags)
|
||||
#define ClearPageLRU(page) clear_bit(PG_lru, &(page)->flags)
|
||||
|
||||
#define PageActive(page) test_bit(PG_active, &(page)->flags)
|
||||
#define SetPageActive(page) set_bit(PG_active, &(page)->flags)
|
||||
|
|
16
mm/swap.c
16
mm/swap.c
|
@ -214,8 +214,8 @@ void fastcall __page_cache_release(struct page *page)
|
|||
struct zone *zone = page_zone(page);
|
||||
|
||||
spin_lock_irqsave(&zone->lru_lock, flags);
|
||||
if (!TestClearPageLRU(page))
|
||||
BUG();
|
||||
BUG_ON(!PageLRU(page));
|
||||
ClearPageLRU(page);
|
||||
del_page_from_lru(zone, page);
|
||||
spin_unlock_irqrestore(&zone->lru_lock, flags);
|
||||
}
|
||||
|
@ -265,8 +265,8 @@ void release_pages(struct page **pages, int nr, int cold)
|
|||
zone = pagezone;
|
||||
spin_lock_irq(&zone->lru_lock);
|
||||
}
|
||||
if (!TestClearPageLRU(page))
|
||||
BUG();
|
||||
BUG_ON(!PageLRU(page));
|
||||
ClearPageLRU(page);
|
||||
del_page_from_lru(zone, page);
|
||||
}
|
||||
|
||||
|
@ -345,8 +345,8 @@ void __pagevec_lru_add(struct pagevec *pvec)
|
|||
zone = pagezone;
|
||||
spin_lock_irq(&zone->lru_lock);
|
||||
}
|
||||
if (TestSetPageLRU(page))
|
||||
BUG();
|
||||
BUG_ON(PageLRU(page));
|
||||
SetPageLRU(page);
|
||||
add_page_to_inactive_list(zone, page);
|
||||
}
|
||||
if (zone)
|
||||
|
@ -372,8 +372,8 @@ void __pagevec_lru_add_active(struct pagevec *pvec)
|
|||
zone = pagezone;
|
||||
spin_lock_irq(&zone->lru_lock);
|
||||
}
|
||||
if (TestSetPageLRU(page))
|
||||
BUG();
|
||||
BUG_ON(PageLRU(page));
|
||||
SetPageLRU(page);
|
||||
if (TestSetPageActive(page))
|
||||
BUG();
|
||||
add_page_to_active_list(zone, page);
|
||||
|
|
20
mm/vmscan.c
20
mm/vmscan.c
|
@ -1042,9 +1042,10 @@ int isolate_lru_page(struct page *page)
|
|||
if (PageLRU(page)) {
|
||||
struct zone *zone = page_zone(page);
|
||||
spin_lock_irq(&zone->lru_lock);
|
||||
if (TestClearPageLRU(page)) {
|
||||
if (PageLRU(page)) {
|
||||
ret = 1;
|
||||
get_page(page);
|
||||
ClearPageLRU(page);
|
||||
if (PageActive(page))
|
||||
del_page_from_active_list(zone, page);
|
||||
else
|
||||
|
@ -1085,6 +1086,8 @@ static int isolate_lru_pages(int nr_to_scan, struct list_head *src,
|
|||
page = lru_to_page(src);
|
||||
prefetchw_prev_lru_page(page, src, flags);
|
||||
|
||||
BUG_ON(!PageLRU(page));
|
||||
|
||||
list_del(&page->lru);
|
||||
if (unlikely(get_page_testone(page))) {
|
||||
/*
|
||||
|
@ -1100,8 +1103,7 @@ static int isolate_lru_pages(int nr_to_scan, struct list_head *src,
|
|||
* the page is not being freed elsewhere -- the page release
|
||||
* code relies on it.
|
||||
*/
|
||||
if (!TestClearPageLRU(page))
|
||||
BUG();
|
||||
ClearPageLRU(page);
|
||||
list_add(&page->lru, dst);
|
||||
nr_taken++;
|
||||
}
|
||||
|
@ -1156,8 +1158,8 @@ static void shrink_cache(struct zone *zone, struct scan_control *sc)
|
|||
*/
|
||||
while (!list_empty(&page_list)) {
|
||||
page = lru_to_page(&page_list);
|
||||
if (TestSetPageLRU(page))
|
||||
BUG();
|
||||
BUG_ON(PageLRU(page));
|
||||
SetPageLRU(page);
|
||||
list_del(&page->lru);
|
||||
if (PageActive(page))
|
||||
add_page_to_active_list(zone, page);
|
||||
|
@ -1276,8 +1278,8 @@ refill_inactive_zone(struct zone *zone, struct scan_control *sc)
|
|||
while (!list_empty(&l_inactive)) {
|
||||
page = lru_to_page(&l_inactive);
|
||||
prefetchw_prev_lru_page(page, &l_inactive, flags);
|
||||
if (TestSetPageLRU(page))
|
||||
BUG();
|
||||
BUG_ON(PageLRU(page));
|
||||
SetPageLRU(page);
|
||||
if (!TestClearPageActive(page))
|
||||
BUG();
|
||||
list_move(&page->lru, &zone->inactive_list);
|
||||
|
@ -1305,8 +1307,8 @@ refill_inactive_zone(struct zone *zone, struct scan_control *sc)
|
|||
while (!list_empty(&l_active)) {
|
||||
page = lru_to_page(&l_active);
|
||||
prefetchw_prev_lru_page(page, &l_active, flags);
|
||||
if (TestSetPageLRU(page))
|
||||
BUG();
|
||||
BUG_ON(PageLRU(page));
|
||||
SetPageLRU(page);
|
||||
BUG_ON(!PageActive(page));
|
||||
list_move(&page->lru, &zone->active_list);
|
||||
pgmoved++;
|
||||
|
|
Loading…
Reference in New Issue