On 01/17/2015 09:35 PM, Dan Williams wrote:
>> Ronny reports: https://bugzilla.kernel.org/show_bug.cgi?id=87101 >> "Since commit 8a4aeec8d "libata/ahci: accommodate tag ordered >> controllers" the access to the harddisk on the first SATA-port is >> failing on its first access. The access to the harddisk on the >> second port is working normal.
>> When reverting the above commit, access to both harddisks is working >> fine again."
>> Maintain tag ordered submission as the default, but allow sata_sil24 to >> continue with the old behavior.
>> Cc: <stable@xxxxxxxxxxxxxxx <mailto:stable@xxxxxxxxxxxxxxx>> >> Cc: Tejun Heo <tj@xxxxxxxxxx <mailto:tj@xxxxxxxxxx>> >> Reported-by: Ronny Hegewald <Ronny.Hegewald@xxxxxxxxx> >> Signed-off-by: Dan Williams <dan.j.williams@xxxxxxxxx> >> --- >> drivers/ata/libata-core.c | 5 ++++- >> drivers/ata/sata_sil24.c | 2 +- >> include/linux/libata.h | 1 + >> 3 files changed, 6 insertions(+), 2 deletions(-)
>> diff --git a/drivers/ata/libata-core.c b/drivers/ata/libata-core.c >> index 5c84fb5c3372..e2085d4b5b93 100644 >> --- a/drivers/ata/libata-core.c >> +++ b/drivers/ata/libata-core.c >> @@ -4748,7 +4748,10 @@ static struct ata_queued_cmd *ata_qc_new(struct ata_port *ap) >> return NULL; >> >> for (i = 0, tag = ap->last_tag + 1; i < max_queue; i++, tag++) { >> - tag = tag < max_queue ? tag : 0; >> + if (ap->flags & ATA_FLAG_LOWTAG) >> + tag = i; >> + else >> + tag = tag < max_queue ? tag : 0;
> Ugh, this is clear abuse of the ?: operator... Why not simply:
> else if (tag >= max_queue) > tag = 0;
"Abuse"!? Let's just call it "creative differences adding in a minimal quirk while neglecting to refactor" ;-).
In fact, the old code had the same abuse, I didn't notice that...
Sure, that's cleaner.
Thanks. :-) MBR, Sergei -- 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