Hi, On 08-07-16 01:38, Peter Wu wrote:
Hi, Here are two patches to fix an issue reported on kernel bugzilla (infinite loop due to unchecked function) and a more important fix to fix hanging Optimus machines when runtime PM is enabled (with pm/pci patches). See the first version[1] for a background on the fixed problems. This is the second revision of incorporating feedback from Emil Velikov (patch 1), Mika Westerberg (patch 4). Patches 2 and 3 are unchanged. The previous patchset had R-b from Hans de Goede, I think they are still valid. Noteworthy is that the fourth patch now checks directly for _PR3. The commit message is updated to emphasize that memory/disk corruption is fixed for some machines. This patchset can be merged before or after the pci/pm changes[2] (expected to be merged for 4.8), see the original posting[1] for consequences. I have tested it on top of v4.7-rc5. To make patch four work properly, Lukas' RPM refcounting patches should be included. A similar (open/new) RPM refcounting issue in snd-hda-intel should also be fixed. Otherwise the bridge will not really sleep. There is another minor patch for nouveau_pr3_present, but it is not included here because it depends on visibility of pci_bridge_d3_possible(). I'll send a separate mail for this to linux-pci.
Patches 1 - 3 are: Reviewed-by: Hans de Goede <hdegoede@xxxxxxxxxx> Patch 4 looks good to me too, but I'm not familiar enough with the pci-e pm stuff to feel comfortable acking it. Regards, Hans
Kind regards, Peter [1]: https://lists.freedesktop.org/archives/nouveau/2016-May/025116.html [2]: https://git.kernel.org/cgit/linux/kernel/git/helgaas/pci.git/?h=pci/pm Peter Wu (4): drm/nouveau/acpi: ensure matching ACPI handle and supported functions drm/nouveau/acpi: return supported DSM functions drm/nouveau/acpi: check for function 0x1B before using it drm/nouveau/acpi: fix lockup with PCIe runtime PM drivers/gpu/drm/nouveau/nouveau_acpi.c | 103 +++++++++++++++++++++------------ 1 file changed, 66 insertions(+), 37 deletions(-)
-- 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