Re: Pack of CAM improvements

Andriy Gapon wrote:
> on 29/01/2010 22:07 Alexander Motin said the following:
>> Juergen Lock wrote:
>>> Ok while we are talking about ahci(4) on IXP700...  Can anyone reproduce
>>> the `test unit ready' bug on one of those?  Since I saw no reply to
>>> my post,
>>> 	http://docs.freebsd.org/cgi/mid.cgi?201001231407.o0NE7l8j002620
>>> maybe the bug is controller-specific?  How to reproduce:  just try
>>> 	camcontrol tur adaX
>>> or
>>> 	cdrecord -scanbus
>>> or (at least I think this is the same issue) start xfburn without hal
>>> running, then watch for the bus to hang at the next disk access.
>>> (Also leaving the disk led on solid here.)  With the previous patch,
>>> 	http://people.freebsd.org/~mav/cam-ata.20100119.patch
>>> (haven't tested the latest one yet) at least it now seems to recover
>>> after some timeout, leaving this in dmesg:  (sorry I didn't notice
>>> when I first tried, guess I didn't wait long enough...)
>> It is controller specific. Intel and NVidia controllers just return
>> error immediately, as they should, and continue operation. ATI IXP700 -
>> doesn't:
> 
> I have this simple patch in my local tree:
> 
> I remember that this patch is not perfect, but it works for my simple desktop
> setup.  No bad side-effects from it either.

I've committed more complete version of this patch to the 9-CURRENT, as
part of r203376.

-- 
Alexander Motin