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