Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 03 Aug 2004 13:35:07 +0200
From:      =?ISO-8859-1?Q?S=F8ren_Schmidt?= <sos@DeepCore.dk>
To:        Ville-Pertti Keinonen <will+freebsd-current@will.iki.fi>
Cc:        freebsd-current@freebsd.org
Subject:   Re: ATA driver races with interrupts
Message-ID:  <410F786B.1060206@DeepCore.dk>
In-Reply-To: <410F695F.2060907@will.iki.fi>
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> <410F695F.2060907@will.iki.fi>

next in thread | previous in thread | raw e-mail | index | archive | help
Ville-Pertti Keinonen wrote:
> 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.

Yeah, Promise makes HW thats designed to work ;)

> 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.

Well, until very recently such controllers didn't exist. I've fixed the 
Intel SATA part, but I have no docs for the VIA SATA part so I have 
nothing to go by there so maybe I should mark that one as broken and 
serialize the accesses...


-- 
-Søren



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?410F786B.1060206>