diff options
| author | Greg Thelen <gthelen@google.com> | 2018-06-07 18:37:59 -0700 |
|---|---|---|
| committer | Jan Kara <jack@suse.cz> | 2018-06-20 11:04:26 +0200 |
| commit | 9560ba306df3e46b4b1037d101e2e4ca68610f55 (patch) | |
| tree | e480b217731cda6346d560cea44af7cd301a045a /fs | |
| parent | 81e97f01371f4e1701feeafe484665112cd9ddc2 (diff) | |
| download | lwn-9560ba306df3e46b4b1037d101e2e4ca68610f55.tar.gz lwn-9560ba306df3e46b4b1037d101e2e4ca68610f55.zip | |
quota: reclaim least recently used dquots
The dquots in the free_dquots list are not reclaimed in LRU way.
put_dquot_last() puts entries to the tail and dqcache_shrink_scan()
frees from the tail. Free unreferenced dquots in LRU order because it
seems more reasonable than freeing most recently used.
Signed-off-by: Greg Thelen <gthelen@google.com>
Signed-off-by: Shakeel Butt <shakeelb@google.com>
Signed-off-by: Jan Kara <jack@suse.cz>
Diffstat (limited to 'fs')
| -rw-r--r-- | fs/quota/dquot.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/quota/dquot.c b/fs/quota/dquot.c index d88231e3b2be..241b00f835b9 100644 --- a/fs/quota/dquot.c +++ b/fs/quota/dquot.c @@ -716,7 +716,7 @@ dqcache_shrink_scan(struct shrinker *shrink, struct shrink_control *sc) unsigned long freed = 0; spin_lock(&dq_list_lock); - head = free_dquots.prev; + head = free_dquots.next; while (head != &free_dquots && sc->nr_to_scan) { dquot = list_entry(head, struct dquot, dq_free); remove_dquot_hash(dquot); @@ -725,7 +725,7 @@ dqcache_shrink_scan(struct shrinker *shrink, struct shrink_control *sc) do_destroy_dquot(dquot); sc->nr_to_scan--; freed++; - head = free_dquots.prev; + head = free_dquots.next; } spin_unlock(&dq_list_lock); return freed; |