Skip to content
Permalink
Browse files
mm/filemap: Convert page wait queues to be folios
Reinforce that if we're waiting for a bit in a struct page, that's
actually in the head page by changing the type from page to folio.
Increases the size of cachefiles by two bytes, but the kernel core
is unchanged in size.

Signed-off-by: Matthew Wilcox (Oracle) <willy@infradead.org>
  • Loading branch information
Matthew Wilcox (Oracle) authored and intel-lab-lkp committed Mar 20, 2021
1 parent 4e13f63 commit 93822cea6776a7c6c5b1341ed1c3fdbd1e5eeaab
Show file tree
Hide file tree
Showing 3 changed files with 29 additions and 28 deletions.
@@ -25,20 +25,20 @@ static int cachefiles_read_waiter(wait_queue_entry_t *wait, unsigned mode,
struct cachefiles_object *object;
struct fscache_retrieval *op = monitor->op;
struct wait_page_key *key = _key;
struct page *page = wait->private;
struct folio *folio = wait->private;

ASSERT(key);

_enter("{%lu},%u,%d,{%p,%u}",
monitor->netfs_page->index, mode, sync,
key->page, key->bit_nr);
key->folio, key->bit_nr);

if (key->page != page || key->bit_nr != PG_locked)
if (key->folio != folio || key->bit_nr != PG_locked)
return 0;

_debug("--- monitor %p %lx ---", page, page->flags);
_debug("--- monitor %p %lx ---", folio, folio->page.flags);

if (!PageUptodate(page) && !PageError(page)) {
if (!FolioUptodate(folio) && !FolioError(folio)) {
/* unlocked, not uptodate and not erronous? */
_debug("page probably truncated");
}
@@ -107,7 +107,7 @@ static int cachefiles_read_reissue(struct cachefiles_object *object,
put_page(backpage2);

INIT_LIST_HEAD(&monitor->op_link);
add_page_wait_queue(backpage, &monitor->monitor);
add_folio_wait_queue(page_folio(backpage), &monitor->monitor);

if (trylock_page(backpage)) {
ret = -EIO;
@@ -294,7 +294,7 @@ static int cachefiles_read_backing_file_one(struct cachefiles_object *object,
get_page(backpage);
monitor->back_page = backpage;
monitor->monitor.private = backpage;
add_page_wait_queue(backpage, &monitor->monitor);
add_folio_wait_queue(page_folio(backpage), &monitor->monitor);
monitor = NULL;

/* but the page may have been read before the monitor was installed, so
@@ -548,7 +548,7 @@ static int cachefiles_read_backing_file(struct cachefiles_object *object,
get_page(backpage);
monitor->back_page = backpage;
monitor->monitor.private = backpage;
add_page_wait_queue(backpage, &monitor->monitor);
add_folio_wait_queue(page_folio(backpage), &monitor->monitor);
monitor = NULL;

/* but the page may have been read before the monitor was
@@ -669,21 +669,21 @@ static inline pgoff_t linear_page_index(struct vm_area_struct *vma,
}

struct wait_page_key {
struct page *page;
struct folio *folio;
int bit_nr;
int page_match;
};

struct wait_page_queue {
struct page *page;
struct folio *folio;
int bit_nr;
wait_queue_entry_t wait;
};

static inline bool wake_page_match(struct wait_page_queue *wait_page,
struct wait_page_key *key)
{
if (wait_page->page != key->page)
if (wait_page->folio != key->folio)
return false;
key->page_match = 1;

@@ -820,7 +820,7 @@ void page_endio(struct page *page, bool is_write, int err);
/*
* Add an arbitrary waiter to a page's wait queue
*/
extern void add_page_wait_queue(struct page *page, wait_queue_entry_t *waiter);
void add_folio_wait_queue(struct folio *folio, wait_queue_entry_t *waiter);

/*
* Fault everything in given userspace address range in.
@@ -1019,19 +1019,19 @@ EXPORT_SYMBOL(__page_cache_alloc);
*/
#define PAGE_WAIT_TABLE_BITS 8
#define PAGE_WAIT_TABLE_SIZE (1 << PAGE_WAIT_TABLE_BITS)
static wait_queue_head_t page_wait_table[PAGE_WAIT_TABLE_SIZE] __cacheline_aligned;
static wait_queue_head_t folio_wait_table[PAGE_WAIT_TABLE_SIZE] __cacheline_aligned;

static wait_queue_head_t *page_waitqueue(struct page *page)
static wait_queue_head_t *folio_waitqueue(struct folio *folio)
{
return &page_wait_table[hash_ptr(page, PAGE_WAIT_TABLE_BITS)];
return &folio_wait_table[hash_ptr(folio, PAGE_WAIT_TABLE_BITS)];
}

void __init pagecache_init(void)
{
int i;

for (i = 0; i < PAGE_WAIT_TABLE_SIZE; i++)
init_waitqueue_head(&page_wait_table[i]);
init_waitqueue_head(&folio_wait_table[i]);

page_writeback_init();
}
@@ -1086,10 +1086,11 @@ static int wake_page_function(wait_queue_entry_t *wait, unsigned mode, int sync,
*/
flags = wait->flags;
if (flags & WQ_FLAG_EXCLUSIVE) {
if (test_bit(key->bit_nr, &key->page->flags))
if (test_bit(key->bit_nr, &key->folio->page.flags))
return -1;
if (flags & WQ_FLAG_CUSTOM) {
if (test_and_set_bit(key->bit_nr, &key->page->flags))
if (test_and_set_bit(key->bit_nr,
&key->folio->page.flags))
return -1;
flags |= WQ_FLAG_DONE;
}
@@ -1123,12 +1124,12 @@ static int wake_page_function(wait_queue_entry_t *wait, unsigned mode, int sync,

static void wake_up_folio_bit(struct folio *folio, int bit_nr)
{
wait_queue_head_t *q = page_waitqueue(&folio->page);
wait_queue_head_t *q = folio_waitqueue(folio);
struct wait_page_key key;
unsigned long flags;
wait_queue_entry_t bookmark;

key.page = &folio->page;
key.folio = folio;
key.bit_nr = bit_nr;
key.page_match = 0;

@@ -1220,7 +1221,7 @@ int sysctl_page_lock_unfairness = 5;
static inline int wait_on_folio_bit_common(struct folio *folio, int bit_nr,
int state, enum behavior behavior)
{
wait_queue_head_t *q = page_waitqueue(&folio->page);
wait_queue_head_t *q = folio_waitqueue(folio);
int unfairness = sysctl_page_lock_unfairness;
struct wait_page_queue wait_page;
wait_queue_entry_t *wait = &wait_page.wait;
@@ -1240,7 +1241,7 @@ static inline int wait_on_folio_bit_common(struct folio *folio, int bit_nr,

init_wait(wait);
wait->func = wake_page_function;
wait_page.page = &folio->page;
wait_page.folio = folio;
wait_page.bit_nr = bit_nr;

repeat:
@@ -1395,17 +1396,17 @@ int put_and_wait_on_page_locked(struct page *page, int state)
*
* Add an arbitrary @waiter to the wait queue for the nominated @page.
*/
void add_page_wait_queue(struct page *page, wait_queue_entry_t *waiter)
void add_folio_wait_queue(struct folio *folio, wait_queue_entry_t *waiter)
{
wait_queue_head_t *q = page_waitqueue(page);
wait_queue_head_t *q = folio_waitqueue(folio);
unsigned long flags;

spin_lock_irqsave(&q->lock, flags);
__add_wait_queue_entry_tail(q, waiter);
SetPageWaiters(page);
SetFolioWaiters(folio);
spin_unlock_irqrestore(&q->lock, flags);
}
EXPORT_SYMBOL_GPL(add_page_wait_queue);
EXPORT_SYMBOL_GPL(add_folio_wait_queue);

#ifndef clear_bit_unlock_is_negative_byte

@@ -1550,10 +1551,10 @@ EXPORT_SYMBOL_GPL(__lock_folio_killable);

static int __lock_folio_async(struct folio *folio, struct wait_page_queue *wait)
{
struct wait_queue_head *q = page_waitqueue(&folio->page);
struct wait_queue_head *q = folio_waitqueue(folio);
int ret = 0;

wait->page = &folio->page;
wait->folio = folio;
wait->bit_nr = PG_locked;

spin_lock_irq(&q->lock);

0 comments on commit 93822ce

Please sign in to comment.