tech-kern archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: Decomposing vfs_subr.c



David Holland <dholland-tech%netbsd.org@localhost> wrote:
>  > > 
>  > > - I think it should be vfs_vnode.c?
>  > 
>  > OK, unless somebody will come up with a better name.
> 
> Since AIUI from chat this is going to contain the vnode lifecycle and
> code and not e.g. stuff like vn_lock, I think I'd prefer vfs_vncache.c.
> But, vfs_vnode.c is definitely better than vfs_node.c.

vfs_vnode.c then.

>  > Speaking of structural clean ups - I am thinking about moving vfs_*.c
>  > into a separate src/sys/vfs directory.  Given that clean code history
>  > of vfs_subr.c is already damaged (*cough*pooka*cough*) and decomposing
>  > will do more - it might be worth going all the way.
> 
> Well, forcibly moving vfs_lookup.c right now (or anytime in the near
> future) would be a bad idea, so let's not. After that stuff
> stabilizes, perhaps we can. Though I'd kind of prefer having real
> rename support before launching on major reorgs.

I would say better move early, so the history of your further changes will
be preserved.  Rather than you make changes, and then history gets cut.

-- 
Mindaugas


Home | Main Index | Thread Index | Old Index