Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id ; Mon, 11 Nov 2002 12:04:55 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id ; Mon, 11 Nov 2002 12:04:55 -0500 Received: from tmr-02.dsl.thebiz.net ([216.238.38.204]:24082 "EHLO gatekeeper.tmr.com") by vger.kernel.org with ESMTP id ; Mon, 11 Nov 2002 12:04:54 -0500 Date: Mon, 11 Nov 2002 12:10:13 -0500 (EST) From: Bill Davidsen To: Alan Cox cc: Jens Axboe , Torben Mathiasen , Linux Kernel Mailing List Subject: Re: [PATCH-2.5.46] IDE BIOS timings In-Reply-To: <1036780850.16651.105.camel@irongate.swansea.linux.org.uk> Message-ID: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1457 Lines: 34 On 8 Nov 2002, Alan Cox wrote: > On Fri, 2002-11-08 at 16:56, Jens Axboe wrote: > > > Linus please drop this patch for now. Its not been tested on enough > > > controllers, its making things unneccessarily ugly and its also just > > > going to make updates hard. > > > > Alan, the patch is pretty much straight forward. Cleaning up the magic > > numbers and ->autotune consistencies is a good thing, imo. > > You can clean up the naming but it still hasn't been tested, not all > bioses neccessarily give us timings we can trust either. I'm not > opposed to the concept but after the previous IDE mess in 2.5 merging > something that isnt tested on lots of controllers and might have weird > effects does both me a bit This is one of those things which we should allow at user risk. After all, you can shoot yourself in the foot with hdparm as well, there are many unwise things allowed. Having seen all the warnings from bad setup of MPS and ACPI in dmesg, I would say it's more likely that the BIOS get these settings right, since they may be used by that other operating system. -- bill davidsen CTO, TMR Associates, Inc Doing interesting things with little computers since 1979. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/