Subject: Re: CVS commit: src/sys/miscfs/genfs
To: None <reinoud@netbsd.org>
From: YAMAMOTO Takashi <yamt@mwd.biglobe.ne.jp>
List: source-changes
Date: 12/01/2005 01:25:23
> On Wed, Nov 30, 2005 at 03:45:16AM +0000, YAMAMOTO Takashi wrote:
> > Modified Files:
> > src/sys/miscfs/genfs: genfs_vnops.c
> >
> > Log Message:
> > revert rev.1.111 as it isn't necessary or correct.
> > - currently no one in tree has a problem with zero b_lblkno, afaik.
> > - this buf is used for "devvp", so it doesn't make sense to
> > use lbn in the "vp".
>
> i'll have to disagree. Some filingsystems like ntfs, arla and others that
> do 1:1 translation in their vop_bmap function DONT pass their "devvp" but
> their "vp" and thus it makes sense for them.
so? do you mean all filesystems should pass their "vp"?
> As a compromise we could
> adjust the value of b_lblkno the same way as b_blkno is adjusted. In that
> way it would allways point to the correct lbn whatever scheme or method is
> used.
>
> Reinoud
can you explain "the same way"?
YAMAMOTO Takashi