nfs: __rcu annotations
in [Kernel]
|
Prev: BUG: unable to handle kernel NULL pointer dereference, i915_gem_object_move_to_active
Next: [Patch v10 0/3] net: reserve ports for applications using fixed port numbers
From: David Howells on 5 May 2010 06:20 Paul E. McKenney <paulmck(a)linux.vnet.ibm.com> wrote: > struct nfs_delegation { > struct list_head super_list; > - struct rpc_cred *cred; > + struct rpc_cred __rcu *cred; > struct inode *inode; > nfs4_stateid stateid; > fmode_t type; That's unnecessary. I posted a patch that removed the usage of RCU on that pointer: Date: Tue, 20 Apr 2010 11:26:13 +0100 Subject: NFS: Fix RCU issues in the NFSv4 delegation code Did Trond take it? David -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo(a)vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
From: Trond Myklebust on 5 May 2010 08:50 On Wed, 2010-05-05 at 11:14 +0100, David Howells wrote: > Paul E. McKenney <paulmck(a)linux.vnet.ibm.com> wrote: > > > struct nfs_delegation { > > struct list_head super_list; > > - struct rpc_cred *cred; > > + struct rpc_cred __rcu *cred; > > struct inode *inode; > > nfs4_stateid stateid; > > fmode_t type; > > That's unnecessary. I posted a patch that removed the usage of RCU on that > pointer: > > Date: Tue, 20 Apr 2010 11:26:13 +0100 > Subject: NFS: Fix RCU issues in the NFSv4 delegation code > > Did Trond take it? > > David Yes. I haven't pushed it to Linus yet, but I'm planning to do so in the next 2 days. Cheers Trond -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo(a)vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
From: Paul E. McKenney on 5 May 2010 17:10
On Wed, May 05, 2010 at 08:44:55AM -0400, Trond Myklebust wrote: > On Wed, 2010-05-05 at 11:14 +0100, David Howells wrote: > > Paul E. McKenney <paulmck(a)linux.vnet.ibm.com> wrote: > > > > > struct nfs_delegation { > > > struct list_head super_list; > > > - struct rpc_cred *cred; > > > + struct rpc_cred __rcu *cred; > > > struct inode *inode; > > > nfs4_stateid stateid; > > > fmode_t type; > > > > That's unnecessary. I posted a patch that removed the usage of RCU on that > > pointer: > > > > Date: Tue, 20 Apr 2010 11:26:13 +0100 > > Subject: NFS: Fix RCU issues in the NFSv4 delegation code > > > > Did Trond take it? > > > > David > > Yes. I haven't pushed it to Linus yet, but I'm planning to do so in the > next 2 days. I have dropped this change from this commit. Are the rest of the changes in this commit (nfs_fs.h and auth_gss.h) OK? Thanx, Paul -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo(a)vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/ |