Date: Tue, 30 Jan 2001 22:11:24 +1100 (EST) From: Bruce Evans <bde@zeta.org.au> To: Peter Wemm <peter@netplex.com.au> Cc: Soren Schmidt <sos@freebsd.dk>, Mike Smith <msmith@FreeBSD.org>, mjacob@feral.com, "[?X-UNKNOWN?]" <sos@FreeBSD.org>, cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/dev/ata ata-disk.c Message-ID: <Pine.BSF.4.21.0101302120060.826-100000@besplex.bde.org> In-Reply-To: <200101300213.f0U2DY459123@mobile.wemm.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, 29 Jan 2001, Peter Wemm wrote:
> Replicating the hw.atamodes dma/pio thing is probably the ticket though.
Blech. Sysctl should never be used in device drivers. It doesn't scale
well, and encourages poor user interfaces.
My hd.dev.atamodes is:
hw.atamodes: dma,pio,---,dma,dma,---,---,---,
If I had 512 channels (the maximum number of ad disks supported), it
might be:
hw.atamodes: dma,pio,dma,dma,dma,dma,dma,dma,dma,pio,dma,dma,dma,dma,dm=
a,:-8,dma,dma,dma,dma,dma,dma,dma,dma,dma,pio,dma,dma,dma,dma,dma,dma,dma,p=
io,dma,dma,dma,dma,dma,dma,dma,pio,dma,dma,dma,dma,dma,dma,dma,pio,dma,dma,=
dma,dma,dma,dma,dma,pio,dma,dma,dma,dma,dma,dma,dma,pio,dma,dma,dma,dma,dma=
,dma,dma,pio,dma,dma,dma,dma,dma,dma,dma,pio,dma,dma,dma,dma,dma,dma,dma,pi=
o,dma,dma,dma,dma,dma,dma,dma,pio,dma,dma,dma,dma,dma,dma,dma,pio,dma,dma,d=
ma,dma,dma,dma,dma,pio,dma,dma,dma,dma,dma,dma,dma,pio,dma,dma,dma,dma,dma,=
dma,dma,pio,dma,dma,dma,dma,dma,dma,dma,pio,dma,dma,dma,dma,dma,dma,dma,pio=
,dma,dma,dma,dma,dma,dma,dma,pio,dma,dma,dma,dma,dma,dma,dma,pio,dma,dma,dm=
a,dma,dma,dma,dma,pio,dma,dma,dma,dma,dma,dma,dma,pio,dma,dma,dma,dma,dma,d=
ma,dma,pio,dma,dma,dma,dma,dma,dma,dma,pio,dma,dma,dma,dma,dma,dma,dma,pio,=
dma,dma,dma,dma,dma,dma,dma,pio,dma,dma,dma,dma,dma,dma,dma,pio,dma,dma,dma=
,dma,dma,dma,dma,pio,dma,dma,dma,dma,dma,dma,dma,pio,dma,dma,dma,dma,dma,dm=
a,dma,pio,dma,dma,dma,dma,dma,dma,dma,pio,dma,dma,dma,dma,dma,:-%,
Quick now, change the mode for ad345 (hint: it's probably not just the 345t=
h
element in the string).
Supporting this poor user interface isn't even easy for the driver -- it ta=
kes
about 40 lines of code to translate the string.
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.0101302120060.826-100000>