Subject: Re: NAMECACHE_ENTER_REVERSE (Re: CVS commit: src/sys/kern)
To: None <tech-kern@netbsd.org>
From: Christos Zoulas <christos@astron.com>
List: tech-kern
Date: 11/28/2006 20:48:37
In article <20061128194624.95DA811705@yamt.dyndns.org>,
YAMAMOTO Takashi <yamt@mwd.biglobe.ne.jp> wrote:
>> Module Name: src
>> Committed By: christos
>> Date: Wed Oct 25 18:56:39 UTC 2006
>>
>> Modified Files:
>> src/sys/kern: vfs_cache.c
>>
>> Log Message:
>> enable namecache enter reverse for linux procfs lookups.
>> XXX: how expensive is that?
>>
>>
>> To generate a diff of this commit:
>> cvs rdiff -r1.65 -r1.66 src/sys/kern/vfs_cache.c
>>
>> Please note that diffs are not public domain; they are subject to the
>> copyright notices on the relevant files.
>
>it's wrong because our name cache is optional.
>ie. a filesystem is free to choose not to use it at all.
>please revert.
If the filesystem does not implement it, it fails with an error.
Actually I have a good idea how to deal with the failure in the
procfs "exe" case.
>(besides, i don't think it works as linux procfs expects,
>because there entries are not pinned.)
I think it is good enough for the purpose required
(to find any path to the vnode requested).
>if vnode -> name translation is really necessary, it should be
>implemented in upper layers. (caller of VOPs, not filesystems.)
how? can you please elaborate?
christos