Re: RFC: ATA to CAM integration patch

Ilya Zhuravlev wrote:
> Alexander Motin wrote:
>> Ilya Zhuravlev wrote:
>>> ahci cannot attach drives
>>> 8.0-beta2, laptop asus k50in, nvidia MCP75L-based
>>>
>>> ahci0: [THREAD]
>>> ahci0: AHCI v1.20 with 2 3Gbps ports, Port Multiplier supported
>>> ahcich0: <AHCI channel> at channel 0 on ahci0
>>> ahcich0: [THREAD]
>>> ahcich1: <AHCI channel> at channel 1 on ahci0
>>> ahcich1: [THREAD]
>>> ......
>>> (aprobe0:ahcich0:0:15:0): SIGNATURE: 0000
>>> (aprobe0:ahcich0:0:0:0): SIGNATURE: 0000
>>> (aprobe0:ahcich0:0:0:0): Uncorrected Parity Error
>>> (aprobe0:ahcich0:0:0:0): Retrying Command
>>> (aprobe0:ahcich0:0:0:0): Uncoreccted Parity Error
>>> (aprobe0:ahcich0:0:0:0): error 5
>>> (aprobe0:ahcich0:0:0:0): Retries Exhausted
>>> (aprobe1:ahcich1:0:15:0): SIGNATURE: eb14
>>> (aprobe0:ahcich1:0:0:0): SIGNATURE: eb14
>>> (aprobe0:ahcich1:0:0:0): Uncoreccted Parity Error
>>> (aprobe0:ahcich1:0:0:0): Retrying Command
>>> (aprobe0:ahcich1:0:0:0): Uncoreccted Parity Error
>>> (aprobe0:ahcich1:0:0:0): error 5
>>> (aprobe0:ahcich1:0:0:0): Retries Exhausted
>>>
>>> pciconf with ata-driver and ata-compat enabled in bios:
>>> atapci0_at_pci0:0:11:0:    class=0x010185 card=0x1cf71043
>>> chip=0x0ab510de rev=0xb1 hdr=0x00
>>>     vendor     = 'Nvidia Corp'
>>>     class      = mass storage
>>>     subclass   = ATA
>>>     bar   [10] = type I/O Port, range 32, base 0xc080, size  8, enabled
>>>     bar   [14] = type I/O Port, range 32, base 0xc000, size  4, enabled
>>>     bar   [18] = type I/O Port, range 32, base 0xbc00, size  8, enabled
>>>     bar   [1c] = type I/O Port, range 32, base 0xb880, size  4, enabled
>>>     bar   [20] = type I/O Port, range 32, base 0xb800, size 16, enabled
>>>     bar   [24] = type Memory, range 32, base 0xfae7c000, size 8192,
>>> enabled
>>>     cap 01[44] = powerspec 2  supports D0 D3  current D0
>>>     cap 12[8c] = SATA Index-Data Pair
>>>     cap 05[b0] = MSI supports 8 messages, 64 bit
>>>
>>> atacontrol for devices on channels attached
>>
>> Try please to uncomment device_printf() lines inside ahci_ch_intr()
>> function. It could give some ideas about what's going on there.
>>
> Sorry for long delay.
> boot -v, pciconf attached

I don't see that you've uncommented
//device_printf(dev, "%s ERROR is %08x cs %08x...
lines in ahci_ch_intr() and rebuilt kernel as I've said.

-- 
Alexander Motin