Re: [PATCH 07/13] libata-acpi: add ATA_FLAG_ACPI_SATA port flag

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

 



Alan Cox wrote:
> You can end up with _SDD methods for SFF format controllers (or what we
> think of as SFF format controllers) when we don't have the knowledge to
> drive them in their full super-whizzo method (eg Marvell)
> 
>> The ACPI spec says the layout is dependent on controller interface and I
>>  can see reasons why we need to follow that but not the other way
>> around.  Do you have counter-examples?
> 
> The first problem is the words "the spec". 
> 
> Agreed that STM/GTM are not safe except when expected (and not always safe
> when they are) but that ought to be ok as we then set all the modes
> properly ourselves afterwards.

I agree that there are cases where we drive a controller in different
way than how the BIOS configured it and as a result we can end up with
unmatching ACPI nodes, but I think that it's safer to ignore unmatched
ACPI nodes than to do what seems likely without really knowing what's
going on.

There just isn't a generic way to determine which node matches which
device.  The first two ports might map to the first IDE channel or port
1 and 3 (ata_piix).  Or each port might map to a separate IDE channel
with slave spot disabled.  Even if we support 'best effort' matching, it
is something that only the LLD knows how it should be done and whether
it's safe or not.  IMHO, we're better off just ignoring ATA ACPI
hierarchy if it doesn't match the expectation of the driver.

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