Re: CVS commit: src

[Reinoud Zandijk <reinoud%NetBSD.org@localhost>]

On Fri, Aug 29, 2008 at 05:30:44PM +0300, Antti Kantee wrote:
> > Added Files:
> >     src/sys/fs: dirhash.c dirhash.h files.fs
> >     src/sys/modules/dirhash: Makefile
> > 
> > Log Message:
> > Add dirhash kernel module for sharing dirhashing support with other FS
> > other than UDF. Its FS agnostic and could be used for any FS
> 
> First, did I miss the discussion somewhere (highly possible)?

there was a discussion on the chat but i agree i should have posted on 
tech-kern... mea-culpa.

> Second, why did the outcome of the discussion decide this should be in
> sys/fs instead of kern/vfs_foo.c where all other fs subroutines are?

I used the analogy with `unicode.h' wich lives there since 2004. Its also 
optionally compiled if fs'ses use it and kern/vfs_foo.c would IMHO indicate 
that its always compiled in and a fixed part wich might not be a bad idea 
in hindsight.

With regards,
Reinoud