Re: kern/41051: do_sys_rename mp resource race

[Antti Kantee <pooka%iki.fi@localhost>]

The following reply was made to PR kern/41051; it has been noted by GNATS.

From: Antti Kantee <pooka%iki.fi@localhost>
To: gnats-bugs%NetBSD.org@localhost
Cc: 
Subject: Re: kern/41051: do_sys_rename mp resource race
Date: Sat, 21 Mar 2009 10:36:09 +0200

 On Sat Mar 21 2009 at 08:10:06 +0000, Andrew Doran wrote:
 > The following reply was made to PR kern/41051; it has been noted by GNATS.
 > 
 > From: Andrew Doran <ad%netbsd.org@localhost>
 > To: gnats-bugs%NetBSD.org@localhost
 > Cc: 
 > Subject: Re: kern/41051: do_sys_rename mp resource race
 > Date: Sat, 21 Mar 2009 08:05:39 +0000
 > 
 >  On Sat, Mar 21, 2009 at 07:25:00AM +0000, pooka%iki.fi@localhost wrote:
 >  
 >  > >Fix:
 >  > take reference to mp around rename
 >  
 >  Disagree. I think VOPs should not, in general, be dropping refs to vnodes
 >  that the caller provides.
 
 do_sys_rename() is not a VOP, neither is vfs_destroy().  What do you
 mean by a VOP here?