Date: Mon, 31 Mar 2003 20:08:33 -0500 From: Jake Burkholder <jake@locore.ca> To: Nate Lawson <nate@root.org> Cc: John Baldwin <jhb@FreeBSD.org> Subject: Re: cvs commit: src/sys/pci if_dc.c if_pcn.c if_rl.c if_sf.c if_sis.c if_ste.c if_tl.c if_vr.c if_wb.c if_xl.c Message-ID: <20030401010833.GA39531@locore.ca> In-Reply-To: <Pine.BSF.4.21.0303311635490.8123-100000@root.org> References: <XFMail.20030331155020.jhb@FreeBSD.org> <Pine.BSF.4.21.0303311635490.8123-100000@root.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Apparently, On Mon, Mar 31, 2003 at 04:37:15PM -0800, Nate Lawson said words to the effect of; > On Mon, 31 Mar 2003, John Baldwin wrote: > > On 31-Mar-2003 John Baldwin wrote: > > > jhb 2003/03/31 12:22:00 PST > > > > > > FreeBSD src repository > > > > > > Modified files: > > > sys/pci if_dc.c if_pcn.c if_rl.c if_sf.c if_sis.c > > > if_ste.c if_tl.c if_vr.c if_wb.c if_xl.c > > > Log: > > > Add missing ()'s so that these drivers all compile again. > > > > > > Noticed by: jake > > > Tested on: i386 (compile) > > > > Also, the pcn(4) driver still has some possibly bogus locking in > > its probe routine. Not sure if other drivers have locks in their > > probe routines. > > Thank you jhb and jake for fixing the KASSERT ()'s. I had options > INVARIANTS in my kernel so I'm not sure why this was not caught. I was > compiling them as modules so does INVARIANTS not enable KASSERT in > modules? It does not, option headers and not created automatically for modules. Jake
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20030401010833.GA39531>