From: Bartlomiej Zolnierkiewicz <bzolnier@xxxxxxxxx>
Date: Mon, 22 Jun 2009 23:35:06 +0200
> On Monday 22 June 2009 21:01:37 Frans Pop wrote:
>> yet have the DMA problem. The commit above is from the 2.6.30 development
>> cycle, so that fits. I expect you can verify it from the identify data.
>
> I had the same idea initially, unfortunately bit 1 is set for word 53 so this
> must be something else...
So this change should have had zero effect for Frans's case.
I double checked everything with test programs going over his
ID dump and it all checks out.
We might need to bisect this one. But Frans, just for the record
could you simply test reverting just that hunk? Thanks!
>> @@ -396,15 +393,14 @@ int ide_id_dma_bug(ide_drive_t *drive)
>>
>> if (id[ATA_ID_FIELD_VALID] & 4) {
>> if ((id[ATA_ID_UDMA_MODES] >> 8) &&
>> (id[ATA_ID_MWDMA_MODES] >> 8))
>> goto err_out;
>> - } else if (id[ATA_ID_FIELD_VALID] & 2) {
>> - if ((id[ATA_ID_MWDMA_MODES] >> 8) &&
>> - (id[ATA_ID_SWDMA_MODES] >> 8))
>> - goto err_out;
>> - }
>> + } else if ((id[ATA_ID_MWDMA_MODES] >> 8) &&
>> + (id[ATA_ID_SWDMA_MODES] >> 8))
>> + goto err_out;
--
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
