Subject: Re: yamt-vop branch
To: None <chuq@chuq.com>
From: YAMAMOTO Takashi <yamt@mwd.biglobe.ne.jp>
List: tech-kern
Date: 10/30/2005 00:22:32
> I looked at the changes, I have just a few comments:

thanks.

> cd9660_setattr
> 	should just return EOPNOTSUPP immediately, the extra checks are silly.

i thought it's written in the manner to support r/w support in future.
isn't it the case?

> ntfs_fsync
> 	should take a (void *) argument like all the other VOPs.

i've followed the style in which most of ntfs code is written.
i'd like to leave it for jdolecek@.

> tmpfs_update
> 	should return void, since it can't fail.

i'll change.

> ffs_*
> 	should just call ffs_* directly instead of UFS_*.
> 
> lfs_*
> 	should just call lfs_* directly instead of UFS_*.

i think it's a matter of style.  but ok, i'll change them.

YAMAMOTO Takashi