Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 13 Dec 2001 15:30:28 +0200
From:      Sheldon Hearn <sheldonh@starjuice.net>
To:        cvs-committers@FreeBSD.org
Cc:        cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sys/fs/smbfs smbfs_vfsops.c 
Message-ID:  <781.1008250228@axl.seasidesoftware.co.za>
In-Reply-To: Your message of "Thu, 13 Dec 2001 15:19:18 %2B0200." <603.1008249558@axl.seasidesoftware.co.za> 

next in thread | previous in thread | raw e-mail | index | archive | help


On Thu, 13 Dec 2001 15:19:18 +0200, Sheldon Hearn wrote:

> I included <sys/module.h>, which is already included by <sys/kernel.h>.
> 
> I did this because I'm guessing it's _not_ okay to assume that
> <sys/kernel.h> will include <sys/module.h>.
> 
> Am I right, or should I remove the explicit inclusion of <sys/module.h>?

Nevermind, I found my answer in the CVS history for src/sys/sys/kernel.h:

|revision 1.82
|date: 2001/02/04 11:46:17;  author: peter;  state: Exp;  lines: +4 -28
|Pull the rug from under the 'LKM Compatability' macro - PSEUDO_SET().
|There are two 3rd party code chunks using this still - the IPv6 stuff
|and
|i4b.  Give them a private copy as an alternative to changing them too
|much.
|
|XXX sys/kernel.h still has a #include <sys/module.h> in it.  I will be
|taking this out shortly - this affects a number of drivers.

Ciao,
Sheldon.

To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?781.1008250228>