cvs commit: src/sys/kern vfs_syscalls.c
jhb at freebsd.org
Tue Feb 20 18:20:32 UTC 2007
On Tuesday 20 February 2007 09:49, Kostik Belousov wrote:
> On Mon, Feb 19, 2007 at 06:39:58PM -0500, John Baldwin wrote:
> > On Monday 19 February 2007 05:56, Konstantin Belousov wrote:
> > > kib 2007-02-19 10:56:10 UTC
> > >
> > > FreeBSD src repository
> > >
> > > Modified files:
> > > sys/kern vfs_syscalls.c
> > > Log:
> > > Remove union_dircheckp hook, it is not needed by new unionfs code anymore.
> > > As consequence, getdirentries() no longer needs to drop/reacquire
> > > directory vnode lock, that would allow it to be reclaimed in between.
> > I think there is at least one more copy of getdirentries() under sys/compat,
> > possibly multiple. Are you going to fix this in all of them?
> Could you, please, point out where is it ? My grep told me that I removed
> all references to the union_dircheckp, and I think that getdirentries() code
> had to have references to it. My change modified two syscalls: getdirentries()
> and ogetdirentries().
Well, I did find at least 3 ABIs (Linux, svr4, and ibcs2) that do their own
wrapper around VOP_READDIR(), but none of them had the union check in them.
More information about the cvs-all