From a599936c14d078eca1412cac003a9b8c4ef5501f Mon Sep 17 00:00:00 2001 From: Al Viro Date: Sat, 9 Jun 2012 11:55:20 -0400 Subject: [PATCH] --- yaml --- r: 312875 b: refs/heads/master c: 049b3c10eecd0a5f3605fa3cd13f638593213ccb h: refs/heads/master i: 312873: 5c5755c45962dd8c27f2086f0ecf7c24bef69c0e 312871: 7e0c4e3a271e4dc691bd44298ce6e20bd3df9c5c v: v3 --- [refs] | 2 +- trunk/Documentation/filesystems/porting | 10 ++++------ 2 files changed, 5 insertions(+), 7 deletions(-) diff --git a/[refs] b/[refs] index 882e4947521f..4cbe6e572afa 100644 --- a/[refs] +++ b/[refs] @@ -1,2 +1,2 @@ --- -refs/heads/master: 7968ce12e9645c5eb5bb3f4320e43c2e402d580c +refs/heads/master: 049b3c10eecd0a5f3605fa3cd13f638593213ccb diff --git a/trunk/Documentation/filesystems/porting b/trunk/Documentation/filesystems/porting index 8c91d1057d9a..ed9fbc23ece0 100644 --- a/trunk/Documentation/filesystems/porting +++ b/trunk/Documentation/filesystems/porting @@ -355,12 +355,10 @@ protects *all* the dcache state of a given dentry. via rcu-walk path walk (basically, if the file can have had a path name in the vfs namespace). - i_dentry and i_rcu share storage in a union, and the vfs expects -i_dentry to be reinitialized before it is freed, so an: - - INIT_LIST_HEAD(&inode->i_dentry); - -must be done in the RCU callback. + Even though i_dentry and i_rcu share storage in a union, we will +initialize the former in inode_init_always(), so just leave it alone in +the callback. It used to be necessary to clean it there, but not anymore +(starting at 3.2). -- [recommended]