From owner-freebsd-current@FreeBSD.ORG Sun Apr 4 13:03:03 2004 Return-Path: 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 8FFBF16A4CE for ; Sun, 4 Apr 2004 13:03:03 -0700 (PDT) Received: from www.ambrisko.com (adsl-64-174-51-42.dsl.snfc21.pacbell.net [64.174.51.42]) by mx1.FreeBSD.org (Postfix) with ESMTP id 2C6A443D58 for ; Sun, 4 Apr 2004 13:03:03 -0700 (PDT) (envelope-from ambrisko@ambrisko.com) Received: from ambrisko.com (localhost [127.0.0.1]) by www.ambrisko.com (8.12.9p2/8.12.9) with ESMTP id i34K2nCf038809; Sun, 4 Apr 2004 13:02:49 -0700 (PDT) (envelope-from ambrisko@ambrisko.com) Received: (from ambrisko@localhost) by ambrisko.com (8.12.9p2/8.12.9/Submit) id i34K2nop038808; Sun, 4 Apr 2004 13:02:49 -0700 (PDT) (envelope-from ambrisko) From: Doug Ambrisko Message-Id: <200404042002.i34K2nop038808@ambrisko.com> In-Reply-To: <406EDED3.90708@DeepCore.dk> To: =?ISO-8859-1?Q?S=F8ren_Schmidt?= Date: Sun, 4 Apr 2004 13:02:49 -0700 (PDT) X-Mailer: ELM [version 2.4ME+ PL94b (25)] MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain; charset=UNKNOWN-8BIT cc: current@freebsd.org Subject: Re: Intel SATA ICH5/5R 6300ESB support patches X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 04 Apr 2004 20:03:03 -0000 Søren Schmidt writes: [ Charset ISO-8859-1 unsupported, converting... ] | Doug Ambrisko wrote: | > Here are some patches to deal with SATA devices causing errors and | > resulting in wedged systems. Deals with drives removed/powered down, | > drives powered up or plugged in and media errors. We need to | > read the SATA registers and twiddle the port enable on device | > departures. If we don't then the ata code execution will result in | > a lock-up. | > | > I put in a patch for geom for bio_taskqueue_remove. Since ata code | > schedules bio_task it need to be cancelled when we abort and call | > biodone. If we don't cancel this task then when the task is | > run later we get a double free in UMA since we have cleaned up | > twice and called biodone twice for the same request. It seems | > like in biodone we should clean up tasks there. Suggestions | > appreciated. This band-aids it for: | > 1) dd if=/dev/ad2 of=/dev/null | > 2) then power down the drive. | > | > There are other issues on media recovery that I've hit but | > I'm going to do that separate to this HW support. Promise | > patches to deal with SATA issues should be coming soon. | > | > These are based on my -stable patches but are enhanced a little to | > deal with some other cases. | > | > This should make -current systems more stable with SATA drives when | > something goes wrong with them. | > | > I'd like to commit them to -current after a review. | | Well, looks promising, but unfortunatly they collide with the WIP I have | here in the lab, so I'd appreciate if you would not commit it right now. | I'll look into getting the confilicting areas merged into what I have | here, then have you test it on the ICH5 as I dont have any of those | around... Sounds good. If you send me stuff I can test it and/or merge it myself. I have both ICH5 and 6300ESB. The 6300ESB is basically the same with different IDs then the ICH5. BTW the SATA sstatus/scontrol registers are available even though the doc's only claim serror. I have a test setup where I can remotely turn off drives and have a drive with known media errors to test system recovery. Next I'll work on getting the Promise SATA patches that prevent hangs into -current from my -stable patches. This will fix the hangs with thoses cards. I should be able to get the done and tested on Mon. Thanks, Doug A.