lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Thu, 13 Dec 2012 10:44:41 -0800 From: Greg KH <gregkh@...uxfoundation.org> To: "Kirill A. Shutemov" <kirill.shutemov@...ux.intel.com> Cc: Miles J Penner <miles.j.penner@...el.com>, John Ronciak <john.ronciak@...el.com>, Mika Westerberg <mika.westerberg@...ux.intel.com>, Tushar N Dave <tushar.n.dave@...el.com>, "Kirill A. Shutemov" <kirill@...temov.name>, Bjorn Helgaas <bhelgaas@...gle.com>, Rusty Russell <rusty@...tcorp.com.au>, Mauro Carvalho Chehab <mchehab@...hat.com>, linux-pci@...r.kernel.org, linux-kernel@...r.kernel.org Subject: Re: [PATCH 1/3] PCI Hotplug: workaround for Thunderbolt on Acer Aspire S5 On Thu, Dec 13, 2012 at 05:31:46PM +0200, Kirill A. Shutemov wrote: > From: "Kirill A. Shutemov" <kirill.shutemov@...ux.intel.com> > > Correct ACPI PCI hotplug imeplementation should have _RMV method in a > PCI slot (device under pci bridge). In Acer Aspire S5 case we have it > deeper in hierarchy: > > Device (RP05) > { > // ... > Device (HRUP) > { > // ... > Device (HRDN) > { > // ... > Device (EPUP) > { > // ... > Method (_RMV, 0, NotSerialized) // _RMV: Removal Status > { > Return (One) > } > } > } > } > } > > Signed-off-by: Kirill A. Shutemov <kirill.shutemov@...ux.intel.com> > --- > drivers/pci/hotplug/acpi_pcihp.c | 13 +++++++++++++ > 1 file changed, 13 insertions(+) > > diff --git a/drivers/pci/hotplug/acpi_pcihp.c b/drivers/pci/hotplug/acpi_pcihp.c > index 2a47e82..d92ebfb 100644 > --- a/drivers/pci/hotplug/acpi_pcihp.c > +++ b/drivers/pci/hotplug/acpi_pcihp.c > @@ -422,6 +422,19 @@ static int pcihp_is_ejectable(acpi_handle handle) > status = acpi_evaluate_integer(handle, "_RMV", NULL, &removable); > if (ACPI_SUCCESS(status) && removable) > return 1; > + > + /* > + * Workaround for Thunderbolt implementation on Acer Aspire S5. > + * > + * Correct ACPI PCI hotplug imeplementation has _RMV method in a PCI > + * slot (device under pci bridge). In Acer Aspire S5 case we have it > + * deeper in hierarchy. > + */ > + status = acpi_evaluate_integer(handle, "HRDN.EPUP._RMV", NULL, > + &removable); > + if (ACPI_SUCCESS(status) && removable) > + return 1; I have no objection to this patch as-is, but I wonder how will other BIOSes implement this "incorrectly" in the future. Should we always just try to walk the whole PCI slot heirachy looking for the _RMV attribute? That should solve the problem where someone else places this at another location for the slot, right? Is there any test for Windows that ensures that this gets placed in the "correct" location that we can rely on? thanks, greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists