Backport request for commit 579b7c582 (drm/nouveau/pmu: do not assume a PMU is present)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



This commit should be backported to kernel 4.3 (apologies for
line-wrapping). Without it, we get a null deref (i.e. oops) on GF117
GPUs which for one reason or another don't have the PMU hooked up.
That should be fixed in its own right, but that's a separate matter.
The issue was introduced in commit e2ca4e7d6e (drm/nouveau/pmu:
convert to new-style nvkm_subdev)

commit 579b7c58215329803ce184704463de09f0f310ac
Author: Alexandre Courbot <acourbot@xxxxxxxxxx>
Date:   Thu Sep 3 17:39:52 2015 +0900

    drm/nouveau/pmu: do not assume a PMU is present

    Some devices may not have a PMU. Avoid a NULL pointer dereference in
    such cases by checking whether the pointer given to nvkm_pmu_pgob() is
    valid.

    Signed-off-by: Alexandre Courbot <acourbot@xxxxxxxxxx>
    Signed-off-by: Ben Skeggs <bskeggs@xxxxxxxxxx>

diff --git a/drivers/gpu/drm/nouveau/nvkm/subdev/pmu/base.c
b/drivers/gpu/drm/nouveau/nvkm/subdev/pmu/base.c
index 27a79c0..d95eb86 100644
--- a/drivers/gpu/drm/nouveau/nvkm/subdev/pmu/base.c
+++ b/drivers/gpu/drm/nouveau/nvkm/subdev/pmu/base.c
@@ -28,7 +28,7 @@
 void
 nvkm_pmu_pgob(struct nvkm_pmu *pmu, bool enable)
 {
-       if (pmu->func->pgob)
+       if (pmu && pmu->func->pgob)
                pmu->func->pgob(pmu, enable);
 }
--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]