Date: Mon, 21 Jun 1999 15:12:11 +0800 From: Peter Wemm <peter@netplex.com.au> To: Doug Rabson <dfr@nlsystems.com> Cc: cvs-committers@freebsd.org, cvs-all@freebsd.org Subject: Re: cvs commit: src/sys/isa sio.c Message-ID: <19990621071211.988BB75@overcee.netplex.com.au> In-Reply-To: Your message of "Sun, 20 Jun 1999 14:28:22 %2B0100." <Pine.BSF.4.05.9906201426210.80685-100000@herring.nlsystems.com>
next in thread | previous in thread | raw e-mail | index | archive | help
Doug Rabson wrote: > On Sun, 20 Jun 1999, Peter Wemm wrote: > > > peter 1999/06/20 06:10:11 PDT > > > > Modified files: > > sys/isa sio.c > > Log: > > Use the proper interfaces to find if a device is enabled, don't dig into > > the bus mechanism's private internals! > > Well spotted. For future reference, bus_private.h may not be included by > any drivers. Any driver which includes this header should be considered > broken. Well, I wouldn't go quite that far.. Usually this would be the case, the whole point of the bus interface system is to avoid structures for this sort of thing and diving in severely impairs binary upwards compatability. If a driver or bus genuinely *does* need to get to the internals, then either the programmer is attacking the problem the wrong way or the public interfaces are inadequate. For actual bus devices, it is not all that unlikely that there are weaknesses. Cheers, -Peter 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?19990621071211.988BB75>