Revert "FROMLIST: fs/buffer.c: Revoke LRU when trying to drop buffers"
This reverts commit 43edfc892e
.
Reason for revert: broke various vts tests
Bug: 174118021
Change-Id: I5b0c802c5da476a78d7a3e125043156e3eb4b58d
Signed-off-by: Alistair Delva <adelva@google.com>
This commit is contained in:
parent
18b294fa4c
commit
fefe6520ad
81
fs/buffer.c
81
fs/buffer.c
@ -48,7 +48,6 @@
|
||||
#include <linux/sched/mm.h>
|
||||
#include <trace/events/block.h>
|
||||
#include <linux/fscrypt.h>
|
||||
#include <linux/xarray.h>
|
||||
|
||||
#include "internal.h"
|
||||
|
||||
@ -1435,59 +1434,12 @@ static bool has_bh_in_lru(int cpu, void *dummy)
|
||||
return false;
|
||||
}
|
||||
|
||||
static void __evict_bhs_lru(void *arg)
|
||||
{
|
||||
struct bh_lru *b = &get_cpu_var(bh_lrus);
|
||||
struct xarray *busy_bhs = arg;
|
||||
struct buffer_head *bh;
|
||||
unsigned long i, xarray_index;
|
||||
|
||||
xa_for_each(busy_bhs, xarray_index, bh) {
|
||||
for (i = 0; i < BH_LRU_SIZE; i++) {
|
||||
if (b->bhs[i] == bh) {
|
||||
brelse(b->bhs[i]);
|
||||
b->bhs[i] = NULL;
|
||||
break;
|
||||
}
|
||||
}
|
||||
|
||||
bh = bh->b_this_page;
|
||||
}
|
||||
|
||||
put_cpu_var(bh_lrus);
|
||||
}
|
||||
|
||||
static bool page_has_bhs_in_lru(int cpu, void *arg)
|
||||
{
|
||||
struct bh_lru *b = per_cpu_ptr(&bh_lrus, cpu);
|
||||
struct xarray *busy_bhs = arg;
|
||||
struct buffer_head *bh;
|
||||
unsigned long i, xarray_index;
|
||||
|
||||
xa_for_each(busy_bhs, xarray_index, bh) {
|
||||
for (i = 0; i < BH_LRU_SIZE; i++) {
|
||||
if (b->bhs[i] == bh)
|
||||
return true;
|
||||
}
|
||||
|
||||
bh = bh->b_this_page;
|
||||
}
|
||||
|
||||
return false;
|
||||
|
||||
}
|
||||
void invalidate_bh_lrus(void)
|
||||
{
|
||||
on_each_cpu_cond(has_bh_in_lru, invalidate_bh_lru, NULL, 1);
|
||||
}
|
||||
EXPORT_SYMBOL_GPL(invalidate_bh_lrus);
|
||||
|
||||
static void evict_bh_lrus(struct xarray *busy_bhs)
|
||||
{
|
||||
on_each_cpu_cond(page_has_bhs_in_lru, __evict_bhs_lru,
|
||||
busy_bhs, 1);
|
||||
}
|
||||
|
||||
void set_bh_page(struct buffer_head *bh,
|
||||
struct page *page, unsigned long offset)
|
||||
{
|
||||
@ -3248,36 +3200,14 @@ drop_buffers(struct page *page, struct buffer_head **buffers_to_free)
|
||||
{
|
||||
struct buffer_head *head = page_buffers(page);
|
||||
struct buffer_head *bh;
|
||||
struct xarray busy_bhs;
|
||||
int bh_count = 0;
|
||||
int xa_ret, ret = 0;
|
||||
|
||||
xa_init(&busy_bhs);
|
||||
|
||||
bh = head;
|
||||
do {
|
||||
if (buffer_busy(bh)) {
|
||||
xa_ret = xa_err(xa_store(&busy_bhs, bh_count++,
|
||||
bh, GFP_ATOMIC));
|
||||
if (xa_ret)
|
||||
goto out;
|
||||
}
|
||||
if (buffer_busy(bh))
|
||||
goto failed;
|
||||
bh = bh->b_this_page;
|
||||
} while (bh != head);
|
||||
|
||||
if (bh_count) {
|
||||
/*
|
||||
* Check if the busy failure was due to an outstanding
|
||||
* LRU reference
|
||||
*/
|
||||
evict_bh_lrus(&busy_bhs);
|
||||
do {
|
||||
if (buffer_busy(bh))
|
||||
goto out;
|
||||
} while (bh != head);
|
||||
}
|
||||
|
||||
ret = 1;
|
||||
do {
|
||||
struct buffer_head *next = bh->b_this_page;
|
||||
|
||||
@ -3287,10 +3217,9 @@ drop_buffers(struct page *page, struct buffer_head **buffers_to_free)
|
||||
} while (bh != head);
|
||||
*buffers_to_free = head;
|
||||
detach_page_private(page);
|
||||
out:
|
||||
xa_destroy(&busy_bhs);
|
||||
|
||||
return ret;
|
||||
return 1;
|
||||
failed:
|
||||
return 0;
|
||||
}
|
||||
|
||||
int try_to_free_buffers(struct page *page)
|
||||
|
Loading…
Reference in New Issue
Block a user