linux-kernel - Re: [PATCH] VFS: Go through the LRU list of inode from head

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Thu, 21 Jun 2012 17:47:55 +0800
From:	Wanpeng Li <liwp.linux@...il.com>
To:	Cong Meng <mc@...ux.vnet.ibm.com>
Cc:	Al Viro <viro@...iv.linux.org.uk>, linux-fsdevel@...r.kernel.org,
	linux-kernel@...r.kernel.org, Wanpeng Li <liwp.linux@...il.com>
Subject: Re: [PATCH] VFS: Go through the LRU list of inode from head

On Thu, Jun 21, 2012 at 05:00:27PM +0800, Cong Meng wrote:
>Go through the LRU list of inode from head.

IIRC, prune_icache_sb is called(as a shrinker in shrink_slab) in page reclaim 
path, it is used to free icache. Tail entry in LRU(Least Recently Used) list 
is the least recently used inode object. When the inode cache has to shrink, 
the kernel removes entries from the tail.
>
>(I'm not sure whether there is any trick here I doesn't get.  If yes, 
>any one could explain it)
>
>Signed-off-by: Cong Meng <mc@...ux.vnet.ibm.com>
>---
> fs/inode.c |    2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
>diff --git a/fs/inode.c b/fs/inode.c
>index 775cbab..aac8449 100644
>--- a/fs/inode.c
>+++ b/fs/inode.c
>@@ -704,7 +704,7 @@ void prune_icache_sb(struct super_block *sb, int nr_to_scan)
> 		if (list_empty(&sb->s_inode_lru))
> 			break;
> 
>-		inode = list_entry(sb->s_inode_lru.prev, struct inode, i_lru);
>+		inode = list_entry(sb->s_inode_lru.next, struct inode, i_lru);
> 
> 		/*
> 		 * we are inverting the sb->s_inode_lru_lock/inode->i_lock here,
>-- 
>1.7.5.4
>
>--
>To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
>the body of a message to majordomo@...r.kernel.org
>More majordomo info at  http://vger.kernel.org/majordomo-info.html
>Please read the FAQ at  http://www.tux.org/lkml/
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/