From: Lukas Wunner <lukas-JFq808J9C/izQB+pC5nmwQ@public.gmane.org>
To: dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org,
nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org
Cc: Alex Deucher <alexander.deucher-5C7GfCeVMHo@public.gmane.org>,
Dave Airlie <airlied-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
Subject: [PATCH 0/9] Fix runtime pm ref leaks
Date: Tue, 24 May 2016 18:03:27 +0200 [thread overview]
Message-ID: <cover.1464103767.git.lukas@wunner.de> (raw)
In preparation for runtime pm on muxed dual GPU laptops,
I've fixed all runtime pm ref leaks I could find in nouveau,
radeon and amdgpu.
To ease reviewing, I've pushed this series to GitHub:
https://github.com/l1k/linux/commits/drm_runpm_fixes_v1
@Alex Deucher: I do not have an AMD GPU so couldn't test this
beyond verifying that it compiles. Please double-check the patches
and test them internally at AMD. By the way, I've noticed that
nouveau takes a runtime pm ref in ->preclose and releases it in
->postclose. This is missing in radeon and amdgpu. Please check
if it is needed.
Thanks,
Lukas
Lukas Wunner (9):
drm/nouveau: Don't leak runtime pm ref on driver unload
drm/nouveau: Forbid runtime pm on driver unload
drm/radeon: Don't leak runtime pm ref on driver unload
drm/radeon: Don't leak runtime pm ref on driver load
drm/radeon: Forbid runtime pm on driver unload
drm/amdgpu: Don't leak runtime pm ref on driver unload
drm/amdgpu: Don't leak runtime pm ref on driver load
drm/amdgpu: Forbid runtime pm on driver unload
drm: Turn off crtc before tearing down its data structure
drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c | 12 +++++++++---
drivers/gpu/drm/drm_crtc.c | 13 ++++++++++++-
drivers/gpu/drm/nouveau/nouveau_drm.c | 6 +++++-
drivers/gpu/drm/radeon/radeon_device.c | 4 ++++
drivers/gpu/drm/radeon/radeon_kms.c | 5 ++++-
5 files changed, 34 insertions(+), 6 deletions(-)
--
2.8.1
_______________________________________________
Nouveau mailing list
Nouveau@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/nouveau
next reply other threads:[~2016-05-24 16:03 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-05-24 16:03 Lukas Wunner [this message]
2016-05-24 16:03 ` [PATCH 4/9] drm/radeon: Don't leak runtime pm ref on driver load Lukas Wunner
2016-05-24 16:03 ` [PATCH 3/9] drm/radeon: Don't leak runtime pm ref on driver unload Lukas Wunner
2016-05-24 16:03 ` [PATCH 6/9] drm/amdgpu: " Lukas Wunner
[not found] ` <cover.1464103767.git.lukas-JFq808J9C/izQB+pC5nmwQ@public.gmane.org>
2016-05-24 16:03 ` [PATCH 9/9] drm: Turn off crtc before tearing down its data structure Lukas Wunner
2016-05-24 21:30 ` [Nouveau] " Daniel Vetter
2016-05-24 22:07 ` Lukas Wunner
[not found] ` <20160524220753.GA5941-JFq808J9C/izQB+pC5nmwQ@public.gmane.org>
2016-05-24 22:30 ` Daniel Vetter
[not found] ` <20160524213042.GC27098-dv86pmgwkMBes7Z6vYuT8azUEOm+Xw19@public.gmane.org>
2016-05-25 10:51 ` Lukas Wunner
2016-05-25 13:43 ` [Nouveau] " Daniel Vetter
[not found] ` <CAKMK7uGFb9ihRtjeK7s0ezPPv-C6S9GKbE4h9MLoPyHyN=9W5Q-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2016-06-01 12:36 ` Lukas Wunner
[not found] ` <20160601123641.GA15243-JFq808J9C/izQB+pC5nmwQ@public.gmane.org>
2016-06-01 14:40 ` Daniel Vetter
2016-06-03 7:30 ` [Nouveau] " Lukas Wunner
2016-06-03 18:21 ` Daniel Vetter
2016-06-08 16:55 ` Lukas Wunner
2016-05-24 16:03 ` [PATCH 1/9] drm/nouveau: Don't leak runtime pm ref on driver unload Lukas Wunner
[not found] ` <dd120a30cb769c93af8973cae41f61831d17e04b.1464103767.git.lukas-JFq808J9C/izQB+pC5nmwQ@public.gmane.org>
2016-05-27 1:07 ` Peter Wu
2016-05-29 15:50 ` Lukas Wunner
[not found] ` <20160529155006.GA12909-JFq808J9C/izQB+pC5nmwQ@public.gmane.org>
2016-05-30 17:03 ` Peter Wu
2016-05-31 11:34 ` Lukas Wunner
[not found] ` <20160531113443.GA14098-JFq808J9C/izQB+pC5nmwQ@public.gmane.org>
2016-05-31 11:41 ` Peter Wu
2016-05-24 16:03 ` [PATCH 2/9] drm/nouveau: Forbid runtime pm " Lukas Wunner
2016-05-24 16:03 ` [PATCH 8/9] drm/amdgpu: " Lukas Wunner
2016-05-24 16:03 ` [PATCH 5/9] drm/radeon: " Lukas Wunner
2016-05-24 16:03 ` [PATCH 7/9] drm/amdgpu: Don't leak runtime pm ref on driver load Lukas Wunner
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=cover.1464103767.git.lukas@wunner.de \
--to=lukas-jfq808j9c/izqb+pc5nmwq@public.gmane.org \
--cc=airlied-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
--cc=alexander.deucher-5C7GfCeVMHo@public.gmane.org \
--cc=dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org \
--cc=nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox