From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Return-Path: From: Peter Wu To: nouveau@lists.freedesktop.org, dri-devel@lists.freedesktop.org Cc: Dave Airlie , Mika Westerberg , Bjorn Helgaas , linux-pci@vger.kernel.org Subject: [PATCH v2 0/4] nouveau RPM fixes for Optimus Date: Fri, 8 Jul 2016 01:38:23 +0200 Message-Id: <20160707233827.2100-1-peter@lekensteyn.nl> List-ID: 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. 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(-) -- 2.9.0 From mboxrd@z Thu Jan 1 00:00:00 1970 From: Peter Wu Subject: [PATCH v2 0/4] nouveau RPM fixes for Optimus Date: Fri, 8 Jul 2016 01:38:23 +0200 Message-ID: <20160707233827.2100-1-peter@lekensteyn.nl> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: nouveau-bounces-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org Sender: "Nouveau" To: nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org, dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org Cc: Dave Airlie , Bjorn Helgaas , Mika Westerberg , linux-pci-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: nouveau.vger.kernel.org SGksCgpIZXJlIGFyZSB0d28gcGF0Y2hlcyB0byBmaXggYW4gaXNzdWUgcmVwb3J0ZWQgb24ga2Vy bmVsIGJ1Z3ppbGxhIChpbmZpbml0ZSBsb29wCmR1ZSB0byB1bmNoZWNrZWQgZnVuY3Rpb24pIGFu ZCBhIG1vcmUgaW1wb3J0YW50IGZpeCB0byBmaXggaGFuZ2luZyBPcHRpbXVzCm1hY2hpbmVzIHdo ZW4gcnVudGltZSBQTSBpcyBlbmFibGVkICh3aXRoIHBtL3BjaSBwYXRjaGVzKS4KClNlZSB0aGUg Zmlyc3QgdmVyc2lvblsxXSBmb3IgYSBiYWNrZ3JvdW5kIG9uIHRoZSBmaXhlZCBwcm9ibGVtcy4g VGhpcyBpcyB0aGUKc2Vjb25kIHJldmlzaW9uIG9mIGluY29ycG9yYXRpbmcgZmVlZGJhY2sgZnJv bSBFbWlsIFZlbGlrb3YgKHBhdGNoIDEpLCBNaWthCldlc3RlcmJlcmcgKHBhdGNoIDQpLiBQYXRj aGVzIDIgYW5kIDMgYXJlIHVuY2hhbmdlZC4KVGhlIHByZXZpb3VzIHBhdGNoc2V0IGhhZCBSLWIg ZnJvbSBIYW5zIGRlIEdvZWRlLCBJIHRoaW5rIHRoZXkgYXJlIHN0aWxsIHZhbGlkLgoKTm90ZXdv cnRoeSBpcyB0aGF0IHRoZSBmb3VydGggcGF0Y2ggbm93IGNoZWNrcyBkaXJlY3RseSBmb3IgX1BS My4gVGhlIGNvbW1pdAptZXNzYWdlIGlzIHVwZGF0ZWQgdG8gZW1waGFzaXplIHRoYXQgbWVtb3J5 L2Rpc2sgY29ycnVwdGlvbiBpcyBmaXhlZCBmb3Igc29tZQptYWNoaW5lcy4KCgpUaGlzIHBhdGNo c2V0IGNhbiBiZSBtZXJnZWQgYmVmb3JlIG9yIGFmdGVyIHRoZSBwY2kvcG0gY2hhbmdlc1syXSAo ZXhwZWN0ZWQgdG8KYmUgbWVyZ2VkIGZvciA0LjgpLCBzZWUgdGhlIG9yaWdpbmFsIHBvc3Rpbmdb MV0gZm9yIGNvbnNlcXVlbmNlcy4gSSBoYXZlIHRlc3RlZAppdCBvbiB0b3Agb2YgdjQuNy1yYzUu IFRvIG1ha2UgcGF0Y2ggZm91ciB3b3JrIHByb3Blcmx5LCBMdWthcycgUlBNIHJlZmNvdW50aW5n CnBhdGNoZXMgc2hvdWxkIGJlIGluY2x1ZGVkLiBBIHNpbWlsYXIgKG9wZW4vbmV3KSBSUE0gcmVm Y291bnRpbmcgaXNzdWUgaW4Kc25kLWhkYS1pbnRlbCBzaG91bGQgYWxzbyBiZSBmaXhlZC4gT3Ro ZXJ3aXNlIHRoZSBicmlkZ2Ugd2lsbCBub3QgcmVhbGx5IHNsZWVwLgoKVGhlcmUgaXMgYW5vdGhl ciBtaW5vciBwYXRjaCBmb3Igbm91dmVhdV9wcjNfcHJlc2VudCwgYnV0IGl0IGlzIG5vdCBpbmNs dWRlZApoZXJlIGJlY2F1c2UgaXQgZGVwZW5kcyBvbiB2aXNpYmlsaXR5IG9mIHBjaV9icmlkZ2Vf ZDNfcG9zc2libGUoKS4gSSdsbCBzZW5kIGEKc2VwYXJhdGUgbWFpbCBmb3IgdGhpcyB0byBsaW51 eC1wY2kuCgpLaW5kIHJlZ2FyZHMsClBldGVyCgogWzFdOiBodHRwczovL2xpc3RzLmZyZWVkZXNr dG9wLm9yZy9hcmNoaXZlcy9ub3V2ZWF1LzIwMTYtTWF5LzAyNTExNi5odG1sCiBbMl06IGh0dHBz Oi8vZ2l0Lmtlcm5lbC5vcmcvY2dpdC9saW51eC9rZXJuZWwvZ2l0L2hlbGdhYXMvcGNpLmdpdC8/ aD1wY2kvcG0KClBldGVyIFd1ICg0KToKICBkcm0vbm91dmVhdS9hY3BpOiBlbnN1cmUgbWF0Y2hp bmcgQUNQSSBoYW5kbGUgYW5kIHN1cHBvcnRlZCBmdW5jdGlvbnMKICBkcm0vbm91dmVhdS9hY3Bp OiByZXR1cm4gc3VwcG9ydGVkIERTTSBmdW5jdGlvbnMKICBkcm0vbm91dmVhdS9hY3BpOiBjaGVj ayBmb3IgZnVuY3Rpb24gMHgxQiBiZWZvcmUgdXNpbmcgaXQKICBkcm0vbm91dmVhdS9hY3BpOiBm aXggbG9ja3VwIHdpdGggUENJZSBydW50aW1lIFBNCgogZHJpdmVycy9ncHUvZHJtL25vdXZlYXUv bm91dmVhdV9hY3BpLmMgfCAxMDMgKysrKysrKysrKysrKysrKysrKysrLS0tLS0tLS0tLS0tCiAx IGZpbGUgY2hhbmdlZCwgNjYgaW5zZXJ0aW9ucygrKSwgMzcgZGVsZXRpb25zKC0pCgotLSAKMi45 LjAKCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCk5vdXZl YXUgbWFpbGluZyBsaXN0Ck5vdXZlYXVAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlz dHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vbm91dmVhdQo=