Subject: Re: CVS commit: src/sys/fs/smbfs
To: Jaromir Dolecek <jdolecek@netbsd.org>
From: Bill Studenmund <wrstuden@netbsd.org>
List: source-changes
Date: 04/02/2003 11:19:30
On Wed, 2 Apr 2003, Jaromir Dolecek wrote:

> Bill Studenmund wrote:
> >
> > Why not put this in the LKM code that loads the FS?
>
> Do you mean lkm/vfs/smbfs/lkminit_vfs.c or the generic VFS LKM load code?

After sending, I realized I was unclear.

> I'd rather keep it in fs/smbfs/smbfs_vfsops.c, so that it's
> less likely to get out of sync with rest of smbfs code.

Then may I suggest smbfs_lkm.c and smbfs_builtin.c (name needs help)?

My desire is to partition the LKM-specific and not-LKM-specific code as
much as possible. That way the main body of the code stays as environment-
agnostic as possible.

> If the latter, is there any way for the (kernel) LKM load code to
> determine which malloc types the LKM uses?

That might be a good thing to add. The recent LKM changes have helped a
lot, and we could add something like this as a further extension.

Take care,

Bill