From owner-freebsd-current Wed Feb 12 12: 2:40 2003 Delivered-To: freebsd-current@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 580E937B401 for ; Wed, 12 Feb 2003 12:02:39 -0800 (PST) Received: from delivery.infowest.com (delivery.infowest.com [204.17.177.5]) by mx1.FreeBSD.org (Postfix) with ESMTP id 75BD143F85 for ; Wed, 12 Feb 2003 12:02:38 -0800 (PST) (envelope-from agifford@infowest.com) Received: from infowest.com (eq.net [208.186.104.163]) by delivery.infowest.com (Postfix) with ESMTP id 6CECFE3CD0A; Wed, 12 Feb 2003 13:02:37 -0700 (MST) Message-ID: <3E4AA85A.8010908@infowest.com> Date: Wed, 12 Feb 2003 13:02:34 -0700 From: "Aaron D. Gifford" User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.2.1) Gecko/20021130 X-Accept-Language: en-us, en MIME-Version: 1.0 To: Soeren Schmidt , current@freebsd.org Subject: Re: kern/43345: Support for the SiS 651 ATA controller References: <200302121809.h1CI9Zdx054097@spider.deepcore.dk> In-Reply-To: <200302121809.h1CI9Zdx054097@spider.deepcore.dk> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 8bit Sender: owner-freebsd-current@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG Soeren Schmidt wrote: > It seems Aaron D. Gifford wrote: > >>Then on one of the messages, I noticed a link to Patrick Bihan-Faou's >>problem report, read it, and tried out his patch under 5.0-CURRENT >>(having completed my install of 5.0-RELEASE and updated to -CURRENT). >>Of course the line numbers were a bit different, but a hand patch of the >>two files was quick and easy. A quick kernel recompile, BIOS re-enable >>of UDMA, and boot proved the patch worked! > > The patch does *not* work, the reason you have it sortof working is that > the driver writes to the wrong regs, and your BIOS set up the mode > correctly.. > I have a working solution here, and it will get committed to -current > as soon as I have it tested some more. This cannot easily be backported > to -stable since it is part of a major rewrite of parts of the ATA > driver. However when it has proven to be working, I'll try to get the > SiS part backported to the older ATA driver in -stable, time permitting... > > -Søren Thanks for the clarification. I'd better disable UDMA on my system then to avoid any data corruption. Would it be a good idea to annotate the PR with your above comments? My initial reading of the PR made it sound like the patch would work in a kludgy way without problems while your complete update was in the works. From your above comments, this interpretation was wrong. The patch is buggy and dangerous. I'll keep my eyes open for your fix. Thanks again for your work and for taking the time to let me know this. Aaron out. To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-current" in the body of the message