cvs commit: src/sys/gnu/ext2fs ext2_vfsops.c src/sys/ufs/ffs ffs_vfsops.c

Alexander Kabaev ak03 at gte.com
Mon Nov 3 11:04:09 PST 2003


On Mon, 3 Nov 2003 10:55:22 -0800
Alfred Perlstein <bright at mu.org> wrote:

> * Alexander Kabaev <kan at FreeBSD.org> [031102 20:46] wrote:
> > kan         2003/11/02 20:46:19 PST
> > 
> >   FreeBSD src repository
> > 
> >   Modified files:
> >     sys/gnu/ext2fs       ext2_vfsops.c 
> >     sys/ufs/ffs          ffs_vfsops.c 
> >   Log:
> >   Use VOP_UNLOCK/vrele instead of vput. td was erecived as a
> >   parameter and one cannot be sure it is equal to curthread.
> 
> If there is one place (boot()) where this isn't the case, can't that
> place be fixed?  My guess is that a lot of other stuff breaks when
> the thread vop parameter isn't curthread in other cases.
> 

There is a number of issues with fixing sync to pass a real curthread
and Bruce listed them in his email. Going forward, I think we should
strip 'td' parameters from VOPs altogether, but I think this is a 6.0
material.

-- 
Alexander Kabaev


More information about the cvs-all mailing list