Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 03 Jul 2005 01:10:40 -0000
From:      "Poul-Henning Kamp" <phk@phk.freebsd.dk>
To:        Jeff Roberson <jroberson@chesapeake.net>
Cc:        cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sys/kern imgact_elf.c kern_descrip.c kern_exec.c uipc_syscalls.c vfs_bio.c vfs_subr.c vfs_syscalls.c 
Message-ID:  <79410.1106645301@critter.freebsd.dk>
In-Reply-To: Your message of "Tue, 25 Jan 2005 04:24:18 EST." <20050125042317.D18864@mail.chesapeake.net> 

next in thread | previous in thread | raw e-mail | index | archive | help
In message <20050125042317.D18864@mail.chesapeake.net>, Jeff Roberson writes:
>
>On Tue, 25 Jan 2005, Poul-Henning Kamp wrote:
>
>> phk         2005-01-25 00:39:00 UTC
>>
>>   FreeBSD src repository
>>
>>   Modified files:
>>     sys/kern             imgact_elf.c kern_descrip.c kern_exec.c
>>                          uipc_syscalls.c vfs_bio.c vfs_subr.c
>>                          vfs_syscalls.c
>>   Log:
>>   Don't use VOP_GETVOBJECT, use vp->v_object directly.
>
>How does this effect stacking?  Originally this was done so we could get
>the appropriate level of vnode object.  Was this discussed somewhere that
>I missed?

It was discussed on arch@ recently.

Short summary:  Instead of the code which calls VOP_OPEN being
responsible for vnode_pager object creation it is now the vnode's
own filesystem which must do so in vop_open() (and vfs_fhtovp() if
exportable).  This also shaved 200 lines of code out.

Stacking filesystems do this, for instance nullfs sets v_object to that
of the lower filesystem in vop_open() and sets it to NULL in vop_close().

That way vp->v_object can be used directly.

-- 
Poul-Henning Kamp       | UNIX since Zilog Zeus 3.20
phk@FreeBSD.ORG         | TCP/IP since RFC 956
FreeBSD committer       | BSD since 4.3-tahoe    
Never attribute to malice what can adequately be explained by incompetence.




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?79410.1106645301>