svn commit: r218347 - stable/8/sys/dev/ata/chipsets

[Alexander Motin]

On 16.03.2011 15:51, Bob Willcox wrote:
> This change has broken SATA disk support on my Intel Atom D525 ITX system. By
> reverting this change 8.2-STABLE works again on this system.
>
> My 'uname -a' output is:
>
> FreeBSD maul.immure.com 8.2-STABLE FreeBSD 8.2-STABLE #6: Wed Mar 16 08:15:43 CDT 2011     bob at maul.immure.com:/usr/obj/usr/src/sys/MAUL  amd64
>
> When booting the system I get tons of these messages:
>
> Mar 16 07:38:19 maul kernel: ad4: WARNING - SETFEATURES ENABLE RCACHE requeued due to channel reset
> Mar 16 07:38:19 maul kernel: ad4: interrupt on idle channel ignored

As I can see, it means that channel has some active request, but it is 
in IDLE state. It is strange, but I won't be surprised much if it is the 
result of some locking problem in ata(4) in non-CAM mode.

> repeated over and over, and then lots of these:
>
> Mar 16 07:38:21 maul kernel: ad4: WARNING - READ_DMA48 requeued due to channel reset LBA=617964479
> Mar 16 07:38:21 maul kernel: ata2: FAILURE - already active DMA on this device
> Mar 16 07:38:21 maul kernel: ata2: setting up DMA failed
>
> for different LBA values.
>
> As one might expect, I then start seeing I/O errors on the disk and programs
> failing
>
> I've attached the 'pciconf -lv' output.

Send me please full verbose log, if you can save it. I am especially 
interested in place around first errors.

You may try to build kernel with `options ATA_CAM` to see if it helps. 
I've mostly tested this patch in that mode.

> On Sat, Feb 05, 2011 at 09:34:27PM +0000, Alexander Motin wrote:
>> Author: mav
>> Date: Sat Feb  5 21:34:26 2011
>> New Revision: 218347
>> URL: http://svn.freebsd.org/changeset/base/218347
>>
>> Log:
>>    MFC r217774:
>>    ICH7 SATA controller in legacy mode can provide access to SATA registers
>>    via AHCI-like memory resource at BAR(5). Use it if BIOS was so kind to
>>    allocate memory for that BAR. This allows hot-plug support and connection
>>    speed reporting.

-- 
Alexander Motin