Re: [PATCH 0/2] pciehp: workaround for slot mis-detection problem

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

 



Alex Chiang wrote:
> Hi Kenji-san,
> 
> * Kenji Kaneshige <kaneshige.kenji@xxxxxxxxxxxxxx>:
>> There had been reported a problem that pciehp driver mis-detect the
>> non hot-pluggable slot as hot-pluggable on some platforms. The cause
>> of this problem is hot-plug capable bit in Slot Capabilities register
>> is set improperly by BIOS or hardware. It seems BIOS/hardware problem,
>> but I think pciehp driver needs workaround for this problem.
> 
> Do you know the magnitude of how many machines have broekn BIOS?
> 
> I see from your patch 2/2 that there is a strong correlation
> between duplicate slot names and incorrect hot-plug capable bit.
> If that's true, then I guess there are a lot of broken machines
> out there...
> 
> 	http://www.kerneloops.org/searchweek.php?search=pci_create_slot
> 
> I was just wondering if we could quirk this somehow, but if you
> think there are too many broken machines out there, then your
> approach seems pretty reasonable.
> 

Please note that pciehp and ACPI PCI slot detection driver have
different naming method. The pciehp driver uses Physical Slot Number
bit in Slot Capabilities register for slot naming. On the other hand,
ACPI PCI slot detection driver uses ACPI _SUN method for slot naming.
So, pciehp's duplicate slot name problem is different from the problem
reported in the above URL, I think.

By the way, I don't know the problem reported in the above URL very
much. Could you tell me what is the cause of this problem briefly?
Broken ACPI namespace?

Thanks,
Kenji Kaneshige


--
To unsubscribe from this list: send the line "unsubscribe linux-pci" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [DMA Engine]     [Linux Coverity]     [Linux USB]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Greybus]

  Powered by Linux