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 <[email protected]>
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(-)
_______________________________________________
Nouveau mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/nouveau
