[PATCH] resolve collision of generic ATA_FLAG_LOWTAG and driver specific flag

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



The patch "libata: allow sata_sil24 to opt-out of tag ordered submission" 
(72dd299d5039a336493993dcc63413cf31d0e662) introduces a regression with the 
sata_sil24 driver.

The new flag ATA_FLAG_LOWTAG accidentially uses the same bit as 
SIL24_FLAG_PCIX_IRQ_WOC in the driver. This activates code for Silicon Image 
3132, which is only suppossed to run under 3124.

ATA_FLAG_LOWTAG is only used in sata_sil24 and is planned to be removed soon, 
so lets just use another bit for the flag in sata_sil24. 

Signed-off-by: Ronny Hegewald <Ronny.Hegewald@xxxxxxxxx>
Cc: stable@xxxxxxxxxxxxxxx

--- linux-3.18.5/drivers/ata/sata_sil24.c.org
+++ linux-3.18.5/drivers/ata/sata_sil24.c
@@ -247,7 +247,7 @@
 	SIL24_COMMON_FLAGS	= ATA_FLAG_SATA | ATA_FLAG_PIO_DMA |
 				  ATA_FLAG_NCQ | ATA_FLAG_ACPI_SATA |
 				  ATA_FLAG_AN | ATA_FLAG_PMP | ATA_FLAG_LOWTAG,
-	SIL24_FLAG_PCIX_IRQ_WOC	= (1 << 24), /* IRQ loss errata on PCI-X */
+	SIL24_FLAG_PCIX_IRQ_WOC	= (1 << 25), /* IRQ loss errata on PCI-X */
 
 	IRQ_STAT_4PORTS		= 0xf,
 };

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




[Index of Archives]     [Linux Filesystems]     [Linux SCSI]     [Linux RAID]     [Git]     [Kernel Newbies]     [Linux Newbie]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Samba]     [Device Mapper]

  Powered by Linux