From owner-freebsd-current@FreeBSD.ORG Tue Aug 3 10:31:04 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 23FFC16A4CE for ; Tue, 3 Aug 2004 10:31:04 +0000 (GMT) Received: from will.iki.fi (will.iki.fi [217.169.64.20]) by mx1.FreeBSD.org (Postfix) with ESMTP id DAC3643D64 for ; Tue, 3 Aug 2004 10:31:03 +0000 (GMT) (envelope-from will+freebsd-current@will.iki.fi) Received: from [10.0.20.162] (fa-3-0-0.fw.exomi.com [217.169.64.99]) by will.iki.fi (Postfix) with ESMTP id 701AA3E; Tue, 3 Aug 2004 13:33:14 +0300 (EEST) Message-ID: <410F695F.2060907@will.iki.fi> Date: Tue, 03 Aug 2004 13:30:55 +0300 From: Ville-Pertti Keinonen User-Agent: Mozilla Thunderbird 0.7.1 (X11/20040708) X-Accept-Language: en-us, en MIME-Version: 1.0 To: =?ISO-8859-1?Q?S=F8ren_Schmidt?= References: <410E688D.7020709@will.iki.fi> <410E74F7.1070000@will.iki.fi> <20040802132802.3d7kgoow0c80ss0s@www.sweetdreamsracing.biz> <410E7B8B.3080407@will.iki.fi> <410E81B8.1000206@DeepCore.dk> <410E8594.7070600@will.iki.fi> <410EA92C.6090506@DeepCore.dk> <410F3DD0.5030104@will.iki.fi> <410F4F66.8040201@DeepCore.dk> <410F5A99.8070005@will.iki.fi> <410F5D2D.4080305@DeepCore.dk> In-Reply-To: <410F5D2D.4080305@DeepCore.dk> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 8bit cc: freebsd-current@freebsd.org Subject: Re: ATA driver races with interrupts 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: Tue, 03 Aug 2004 10:31:04 -0000 Søren Schmidt wrote: > Reliable solutions needs reliable HW, we can narrow down the race > window here, but without HW support we cannot close it completely > unless we serialises access to the channels (what lots of OS's does > btw).. For now, I moved my SATA disks to the Promise controller, which seems to work with an unpatched -current. If serialization is the only reliable solution, shouldn't it be enabled for all controllers with multiple channels on the same irq that use ata_generic_intr? It isn't going to hurt performance when only using one channel, and when using multiple channels...currently it's pretty much guaranteed to blow up.