Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 20 Sep 2000 07:43:20 +1100 (EST)
From:      Bruce Evans <bde@zeta.org.au>
To:        Mike Smith <msmith@FreeBSD.org>
Cc:        cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sys/dev/aac aac.c
Message-ID:  <Pine.BSF.4.21.0009200739070.2436-100000@besplex.bde.org>
In-Reply-To: <200009190035.RAA80085@freefall.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, 18 Sep 2000, Mike Smith wrote:

> msmith      2000/09/18 17:35:03 PDT
> 
>   Modified files:
>     sys/dev/aac          aac.c 
>   Log:
>   Include <sys/proc.h> so that we can dereference something in the proc
>   structure.  Looks like someone removed a nested include somewhere else.

Looks like missing test coverage in LINT, and a missing option
(AAC_COMPAT_LINUX is defined in the module makefile but present in NOTES
or conf/options).

The include is not properly ifdefed.

Bruce



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?Pine.BSF.4.21.0009200739070.2436-100000>