From owner-cvs-all Tue Jan 30 3:12:14 2001 Delivered-To: cvs-all@freebsd.org Received: from mailman.zeta.org.au (mailman.zeta.org.au [203.26.10.16]) by hub.freebsd.org (Postfix) with ESMTP id 1853837B503; Tue, 30 Jan 2001 03:11:52 -0800 (PST) Received: from bde.zeta.org.au (bde.zeta.org.au [203.2.228.102]) by mailman.zeta.org.au (8.9.3/8.8.7) with ESMTP id WAA07296; Tue, 30 Jan 2001 22:11:38 +1100 Date: Tue, 30 Jan 2001 22:11:24 +1100 (EST) From: Bruce Evans X-Sender: bde@besplex.bde.org To: Peter Wemm Cc: Soren Schmidt , Mike Smith , mjacob@feral.com, "[?X-UNKNOWN?]" , cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/dev/ata ata-disk.c In-Reply-To: <200101300213.f0U2DY459123@mobile.wemm.org> Message-ID: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Content-Transfer-Encoding: QUOTED-PRINTABLE Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.ORG 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