mirror of
git://git.kernel.org/pub/scm/linux/kernel/git/xiang/erofs.git
synced 2026-10-31 22:34:33 +01:00
[PATCH] dcache: avoid RCU for never-hashed dentries
Some dentries don't need to be globally visible in dentry hashtable.
(pipes & sockets)
Such dentries dont need to wait for a RCU grace period at delete time.
Being able to free them permits a better CPU cache use (hot cache)
This patch combined with (dont insert pipe dentries into dentry_hashtable)
reduced time of { pipe(p); close(p[0]); close(p[1]);} on my UP machine (1.6
GHz Pentium-M) from 3.23 us to 2.86 us (But this patch does not depend on
other patches, only bench results)
Signed-off-by: Eric Dumazet <[email protected]>
Cc: Al Viro <[email protected]>
Cc: Maneesh Soni <[email protected]>
Cc: "Paul E. McKenney" <[email protected]>
Cc: Dipankar Sarma <[email protected]>
Acked-by: David Miller <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
This commit is contained in:
1 parent
d18de5a272
commit
b3423415fb
1 file changed
+12
-4
+12
-4
@@ -68,15 +68,19 @@ struct dentry_stat_t dentry_stat = {
|
||||
.age_limit = 45,
|
||||
};
|
||||
|
||||
static void d_callback(struct rcu_head *head)
|
||||
static void __d_free(struct dentry *dentry)
|
||||
{
|
||||
struct dentry * dentry = container_of(head, struct dentry, d_u.d_rcu);
|
||||
|
||||
if (dname_external(dentry))
|
||||
kfree(dentry->d_name.name);
|
||||
kmem_cache_free(dentry_cache, dentry);
|
||||
}
|
||||
|
||||
static void d_callback(struct rcu_head *head)
|
||||
{
|
||||
struct dentry * dentry = container_of(head, struct dentry, d_u.d_rcu);
|
||||
__d_free(dentry);
|
||||
}
|
||||
|
||||
/*
|
||||
* no dcache_lock, please. The caller must decrement dentry_stat.nr_dentry
|
||||
* inside dcache_lock.
|
||||
@@ -85,7 +89,11 @@ static void d_free(struct dentry *dentry)
|
||||
{
|
||||
if (dentry->d_op && dentry->d_op->d_release)
|
||||
dentry->d_op->d_release(dentry);
|
||||
call_rcu(&dentry->d_u.d_rcu, d_callback);
|
||||
/* if dentry was never inserted into hash, immediate free is OK */
|
||||
if (dentry->d_hash.pprev == NULL)
|
||||
__d_free(dentry);
|
||||
else
|
||||
call_rcu(&dentry->d_u.d_rcu, d_callback);
|
||||
}
|
||||
|
||||
/*
|
||||
|
||||
Reference in new issue
Block a user