* [PATCH] drm/radeon: fix runpm handling on APUs (v4)
@ 2014-04-11 2:29 Alex Deucher
2014-04-11 2:29 ` [PATCH] drm/radeon: update CI DPM powertune settings Alex Deucher
` (2 more replies)
0 siblings, 3 replies; 5+ messages in thread
From: Alex Deucher @ 2014-04-11 2:29 UTC (permalink / raw)
To: dri-devel, deathsimple; +Cc: Alex Deucher, stable
Don't try and runtime suspend the APU in PX systems. We
only want to power down the dGPU.
v2: fix harder
v3: fix stupid typo
v4: consolidate runpm enablement to a single flag
bugs:
https://bugs.freedesktop.org/show_bug.cgi?id=75127
https://bugzilla.kernel.org/show_bug.cgi?id=72701
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Cc: stable@vger.kernel.org
---
drivers/gpu/drm/radeon/radeon.h | 1 +
drivers/gpu/drm/radeon/radeon_atpx_handler.c | 2 +-
drivers/gpu/drm/radeon/radeon_device.c | 19 ++++++++++---------
drivers/gpu/drm/radeon/radeon_drv.c | 24 ++++--------------------
drivers/gpu/drm/radeon/radeon_family.h | 1 +
drivers/gpu/drm/radeon/radeon_kms.c | 14 ++++++++++----
6 files changed, 27 insertions(+), 34 deletions(-)
diff --git a/drivers/gpu/drm/radeon/radeon.h b/drivers/gpu/drm/radeon/radeon.h
index f0fc2c8..3d94c0d 100644
--- a/drivers/gpu/drm/radeon/radeon.h
+++ b/drivers/gpu/drm/radeon/radeon.h
@@ -2318,6 +2318,7 @@ struct radeon_device {
bool have_disp_power_ref;
};
+bool radeon_is_px(struct drm_device *dev);
int radeon_device_init(struct radeon_device *rdev,
struct drm_device *ddev,
struct pci_dev *pdev,
diff --git a/drivers/gpu/drm/radeon/radeon_atpx_handler.c b/drivers/gpu/drm/radeon/radeon_atpx_handler.c
index fa9a9c0..dedea72 100644
--- a/drivers/gpu/drm/radeon/radeon_atpx_handler.c
+++ b/drivers/gpu/drm/radeon/radeon_atpx_handler.c
@@ -59,7 +59,7 @@ struct atpx_mux {
u16 mux;
} __packed;
-bool radeon_is_px(void) {
+bool radeon_has_atpx(void) {
return radeon_atpx_priv.atpx_detected;
}
diff --git a/drivers/gpu/drm/radeon/radeon_device.c b/drivers/gpu/drm/radeon/radeon_device.c
index 2e72dcd..00b19d0 100644
--- a/drivers/gpu/drm/radeon/radeon_device.c
+++ b/drivers/gpu/drm/radeon/radeon_device.c
@@ -102,11 +102,14 @@ static const char radeon_family_name[][16] = {
"LAST",
};
-#if defined(CONFIG_VGA_SWITCHEROO)
-bool radeon_is_px(void);
-#else
-static inline bool radeon_is_px(void) { return false; }
-#endif
+bool radeon_is_px(struct drm_device *dev)
+{
+ struct radeon_device *rdev = dev->dev_private;
+
+ if (rdev->flags & RADEON_IS_PX)
+ return true;
+ return false;
+}
/**
* radeon_program_register_sequence - program an array of registers.
@@ -1082,7 +1085,7 @@ static void radeon_switcheroo_set_state(struct pci_dev *pdev, enum vga_switchero
{
struct drm_device *dev = pci_get_drvdata(pdev);
- if (radeon_is_px() && state == VGA_SWITCHEROO_OFF)
+ if (radeon_is_px(dev) && state == VGA_SWITCHEROO_OFF)
return;
if (state == VGA_SWITCHEROO_ON) {
@@ -1301,9 +1304,7 @@ int radeon_device_init(struct radeon_device *rdev,
* ignore it */
vga_client_register(rdev->pdev, rdev, NULL, radeon_vga_set_decode);
- if (radeon_runtime_pm == 1)
- runtime = true;
- if ((radeon_runtime_pm == -1) && radeon_is_px())
+ if (rdev->flags & RADEON_IS_PX)
runtime = true;
vga_switcheroo_register_client(rdev->pdev, &radeon_switcheroo_ops, runtime);
if (runtime)
diff --git a/drivers/gpu/drm/radeon/radeon_drv.c b/drivers/gpu/drm/radeon/radeon_drv.c
index 2dc2cc4..3811812 100644
--- a/drivers/gpu/drm/radeon/radeon_drv.c
+++ b/drivers/gpu/drm/radeon/radeon_drv.c
@@ -114,6 +114,7 @@ extern int radeon_get_crtc_scanoutpos(struct drm_device *dev, int crtc,
unsigned int flags,
int *vpos, int *hpos, ktime_t *stime,
ktime_t *etime);
+extern bool radeon_is_px(struct drm_device *dev);
extern const struct drm_ioctl_desc radeon_ioctls_kms[];
extern int radeon_max_kms_ioctl;
int radeon_mmap(struct file *filp, struct vm_area_struct *vma);
@@ -143,11 +144,9 @@ void radeon_debugfs_cleanup(struct drm_minor *minor);
#if defined(CONFIG_VGA_SWITCHEROO)
void radeon_register_atpx_handler(void);
void radeon_unregister_atpx_handler(void);
-bool radeon_is_px(void);
#else
static inline void radeon_register_atpx_handler(void) {}
static inline void radeon_unregister_atpx_handler(void) {}
-static inline bool radeon_is_px(void) { return false; }
#endif
int radeon_no_wb;
@@ -404,12 +403,7 @@ static int radeon_pmops_runtime_suspend(struct device *dev)
struct drm_device *drm_dev = pci_get_drvdata(pdev);
int ret;
- if (radeon_runtime_pm == 0) {
- pm_runtime_forbid(dev);
- return -EBUSY;
- }
-
- if (radeon_runtime_pm == -1 && !radeon_is_px()) {
+ if (!radeon_is_px(drm_dev)) {
pm_runtime_forbid(dev);
return -EBUSY;
}
@@ -433,10 +427,7 @@ static int radeon_pmops_runtime_resume(struct device *dev)
struct drm_device *drm_dev = pci_get_drvdata(pdev);
int ret;
- if (radeon_runtime_pm == 0)
- return -EINVAL;
-
- if (radeon_runtime_pm == -1 && !radeon_is_px())
+ if (!radeon_is_px(drm_dev))
return -EINVAL;
drm_dev->switch_power_state = DRM_SWITCH_POWER_CHANGING;
@@ -461,14 +452,7 @@ static int radeon_pmops_runtime_idle(struct device *dev)
struct drm_device *drm_dev = pci_get_drvdata(pdev);
struct drm_crtc *crtc;
- if (radeon_runtime_pm == 0) {
- pm_runtime_forbid(dev);
- return -EBUSY;
- }
-
- /* are we PX enabled? */
- if (radeon_runtime_pm == -1 && !radeon_is_px()) {
- DRM_DEBUG_DRIVER("failing to power off - not px\n");
+ if (!radeon_is_px(drm_dev)) {
pm_runtime_forbid(dev);
return -EBUSY;
}
diff --git a/drivers/gpu/drm/radeon/radeon_family.h b/drivers/gpu/drm/radeon/radeon_family.h
index 614ad54..9da5da4 100644
--- a/drivers/gpu/drm/radeon/radeon_family.h
+++ b/drivers/gpu/drm/radeon/radeon_family.h
@@ -115,6 +115,7 @@ enum radeon_chip_flags {
RADEON_NEW_MEMMAP = 0x00400000UL,
RADEON_IS_PCI = 0x00800000UL,
RADEON_IS_IGPGART = 0x01000000UL,
+ RADEON_IS_PX = 0x02000000UL,
};
#endif
diff --git a/drivers/gpu/drm/radeon/radeon_kms.c b/drivers/gpu/drm/radeon/radeon_kms.c
index 3e49342..9337820 100644
--- a/drivers/gpu/drm/radeon/radeon_kms.c
+++ b/drivers/gpu/drm/radeon/radeon_kms.c
@@ -35,9 +35,9 @@
#include <linux/pm_runtime.h>
#if defined(CONFIG_VGA_SWITCHEROO)
-bool radeon_is_px(void);
+bool radeon_has_atpx(void);
#else
-static inline bool radeon_is_px(void) { return false; }
+static inline bool radeon_has_atpx(void) { return false; }
#endif
/**
@@ -107,6 +107,13 @@ int radeon_driver_load_kms(struct drm_device *dev, unsigned long flags)
flags |= RADEON_IS_PCI;
}
+ if (radeon_runtime_pm == 1)
+ flags |= RADEON_IS_PX;
+ else if ((radeon_runtime_pm == -1) &&
+ radeon_has_atpx() &&
+ ((flags & RADEON_IS_IGP) == 0))
+ flags |= RADEON_IS_PX;
+
/* radeon_device_init should report only fatal error
* like memory allocation failure or iomapping failure,
* or memory manager initialization failure, it must
@@ -137,8 +144,7 @@ int radeon_driver_load_kms(struct drm_device *dev, unsigned long flags)
"Error during ACPI methods call\n");
}
- if ((radeon_runtime_pm == 1) ||
- ((radeon_runtime_pm == -1) && radeon_is_px())) {
+ if (radeon_is_px(dev)) {
pm_runtime_use_autosuspend(dev->dev);
pm_runtime_set_autosuspend_delay(dev->dev, 5000);
pm_runtime_set_active(dev->dev);
--
1.8.3.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH] drm/radeon: update CI DPM powertune settings
2014-04-11 2:29 [PATCH] drm/radeon: fix runpm handling on APUs (v4) Alex Deucher
@ 2014-04-11 2:29 ` Alex Deucher
2014-04-11 2:29 ` [PATCH] drm/radeon: disable mclk dpm on R7 260X Alex Deucher
2014-04-11 13:10 ` [PATCH] drm/radeon: fix runpm handling on APUs (v4) Christian König
2 siblings, 0 replies; 5+ messages in thread
From: Alex Deucher @ 2014-04-11 2:29 UTC (permalink / raw)
To: dri-devel, deathsimple; +Cc: Alex Deucher
As per internal recommendations.
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
---
drivers/gpu/drm/radeon/ci_dpm.c | 25 +++++++++++++------------
1 file changed, 13 insertions(+), 12 deletions(-)
diff --git a/drivers/gpu/drm/radeon/ci_dpm.c b/drivers/gpu/drm/radeon/ci_dpm.c
index cad89a9..f92ac5e 100644
--- a/drivers/gpu/drm/radeon/ci_dpm.c
+++ b/drivers/gpu/drm/radeon/ci_dpm.c
@@ -202,24 +202,29 @@ static void ci_initialize_powertune_defaults(struct radeon_device *rdev)
struct ci_power_info *pi = ci_get_pi(rdev);
switch (rdev->pdev->device) {
+ case 0x6649:
case 0x6650:
+ case 0x6651:
case 0x6658:
case 0x665C:
+ case 0x665D:
default:
pi->powertune_defaults = &defaults_bonaire_xt;
break;
- case 0x6651:
- case 0x665D:
- pi->powertune_defaults = &defaults_bonaire_pro;
- break;
case 0x6640:
- pi->powertune_defaults = &defaults_saturn_xt;
- break;
case 0x6641:
- pi->powertune_defaults = &defaults_saturn_pro;
+ case 0x6646:
+ case 0x6647:
+ pi->powertune_defaults = &defaults_saturn_xt;
break;
case 0x67B8:
case 0x67B0:
+ pi->powertune_defaults = &defaults_hawaii_xt;
+ break;
+ case 0x67BA:
+ case 0x67B1:
+ pi->powertune_defaults = &defaults_hawaii_pro;
+ break;
case 0x67A0:
case 0x67A1:
case 0x67A2:
@@ -228,11 +233,7 @@ static void ci_initialize_powertune_defaults(struct radeon_device *rdev)
case 0x67AA:
case 0x67B9:
case 0x67BE:
- pi->powertune_defaults = &defaults_hawaii_xt;
- break;
- case 0x67BA:
- case 0x67B1:
- pi->powertune_defaults = &defaults_hawaii_pro;
+ pi->powertune_defaults = &defaults_bonaire_xt;
break;
}
--
1.8.3.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH] drm/radeon: disable mclk dpm on R7 260X
2014-04-11 2:29 [PATCH] drm/radeon: fix runpm handling on APUs (v4) Alex Deucher
2014-04-11 2:29 ` [PATCH] drm/radeon: update CI DPM powertune settings Alex Deucher
@ 2014-04-11 2:29 ` Alex Deucher
2014-04-11 13:10 ` [PATCH] drm/radeon: fix runpm handling on APUs (v4) Christian König
2 siblings, 0 replies; 5+ messages in thread
From: Alex Deucher @ 2014-04-11 2:29 UTC (permalink / raw)
To: dri-devel, deathsimple; +Cc: Alex Deucher, stable
Setting higher mclks seems to cause stability issues
on some R7 260X boards. Disable it for now for stability
until we find a proper fix.
bug:
https://bugs.freedesktop.org/show_bug.cgi?id=75992
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Cc: stable@vger.kernel.org
---
drivers/gpu/drm/radeon/ci_dpm.c | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/drivers/gpu/drm/radeon/ci_dpm.c b/drivers/gpu/drm/radeon/ci_dpm.c
index f92ac5e..18e91ee 100644
--- a/drivers/gpu/drm/radeon/ci_dpm.c
+++ b/drivers/gpu/drm/radeon/ci_dpm.c
@@ -5147,6 +5147,10 @@ int ci_dpm_init(struct radeon_device *rdev)
pi->mclk_dpm_key_disabled = 0;
pi->pcie_dpm_key_disabled = 0;
+ /* mclk dpm is unstable on some R7 260X cards */
+ if (rdev->pdev->device == 0x6658)
+ pi->mclk_dpm_key_disabled = 1;
+
pi->caps_sclk_ds = true;
pi->mclk_strobe_mode_threshold = 40000;
--
1.8.3.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] drm/radeon: fix runpm handling on APUs (v4)
2014-04-11 2:29 [PATCH] drm/radeon: fix runpm handling on APUs (v4) Alex Deucher
2014-04-11 2:29 ` [PATCH] drm/radeon: update CI DPM powertune settings Alex Deucher
2014-04-11 2:29 ` [PATCH] drm/radeon: disable mclk dpm on R7 260X Alex Deucher
@ 2014-04-11 13:10 ` Christian König
2014-04-11 13:12 ` Deucher, Alexander
2 siblings, 1 reply; 5+ messages in thread
From: Christian König @ 2014-04-11 13:10 UTC (permalink / raw)
To: Alex Deucher, dri-devel; +Cc: Alex Deucher, stable
Am 11.04.2014 04:29, schrieb Alex Deucher:
> Don't try and runtime suspend the APU in PX systems. We
> only want to power down the dGPU.
>
> v2: fix harder
> v3: fix stupid typo
> v4: consolidate runpm enablement to a single flag
>
> bugs:
> https://bugs.freedesktop.org/show_bug.cgi?id=75127
> https://bugzilla.kernel.org/show_bug.cgi?id=72701
>
> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
> Cc: stable@vger.kernel.org
All three patches added to my drm-fixes-3.15 queue.
Is it correct that "[PATCH] drm/radeon: update CI DPM powertune
settings" doesn't have a "CC: stable..." ?
Christian.
> ---
> drivers/gpu/drm/radeon/radeon.h | 1 +
> drivers/gpu/drm/radeon/radeon_atpx_handler.c | 2 +-
> drivers/gpu/drm/radeon/radeon_device.c | 19 ++++++++++---------
> drivers/gpu/drm/radeon/radeon_drv.c | 24 ++++--------------------
> drivers/gpu/drm/radeon/radeon_family.h | 1 +
> drivers/gpu/drm/radeon/radeon_kms.c | 14 ++++++++++----
> 6 files changed, 27 insertions(+), 34 deletions(-)
>
> diff --git a/drivers/gpu/drm/radeon/radeon.h b/drivers/gpu/drm/radeon/radeon.h
> index f0fc2c8..3d94c0d 100644
> --- a/drivers/gpu/drm/radeon/radeon.h
> +++ b/drivers/gpu/drm/radeon/radeon.h
> @@ -2318,6 +2318,7 @@ struct radeon_device {
> bool have_disp_power_ref;
> };
>
> +bool radeon_is_px(struct drm_device *dev);
> int radeon_device_init(struct radeon_device *rdev,
> struct drm_device *ddev,
> struct pci_dev *pdev,
> diff --git a/drivers/gpu/drm/radeon/radeon_atpx_handler.c b/drivers/gpu/drm/radeon/radeon_atpx_handler.c
> index fa9a9c0..dedea72 100644
> --- a/drivers/gpu/drm/radeon/radeon_atpx_handler.c
> +++ b/drivers/gpu/drm/radeon/radeon_atpx_handler.c
> @@ -59,7 +59,7 @@ struct atpx_mux {
> u16 mux;
> } __packed;
>
> -bool radeon_is_px(void) {
> +bool radeon_has_atpx(void) {
> return radeon_atpx_priv.atpx_detected;
> }
>
> diff --git a/drivers/gpu/drm/radeon/radeon_device.c b/drivers/gpu/drm/radeon/radeon_device.c
> index 2e72dcd..00b19d0 100644
> --- a/drivers/gpu/drm/radeon/radeon_device.c
> +++ b/drivers/gpu/drm/radeon/radeon_device.c
> @@ -102,11 +102,14 @@ static const char radeon_family_name[][16] = {
> "LAST",
> };
>
> -#if defined(CONFIG_VGA_SWITCHEROO)
> -bool radeon_is_px(void);
> -#else
> -static inline bool radeon_is_px(void) { return false; }
> -#endif
> +bool radeon_is_px(struct drm_device *dev)
> +{
> + struct radeon_device *rdev = dev->dev_private;
> +
> + if (rdev->flags & RADEON_IS_PX)
> + return true;
> + return false;
> +}
>
> /**
> * radeon_program_register_sequence - program an array of registers.
> @@ -1082,7 +1085,7 @@ static void radeon_switcheroo_set_state(struct pci_dev *pdev, enum vga_switchero
> {
> struct drm_device *dev = pci_get_drvdata(pdev);
>
> - if (radeon_is_px() && state == VGA_SWITCHEROO_OFF)
> + if (radeon_is_px(dev) && state == VGA_SWITCHEROO_OFF)
> return;
>
> if (state == VGA_SWITCHEROO_ON) {
> @@ -1301,9 +1304,7 @@ int radeon_device_init(struct radeon_device *rdev,
> * ignore it */
> vga_client_register(rdev->pdev, rdev, NULL, radeon_vga_set_decode);
>
> - if (radeon_runtime_pm == 1)
> - runtime = true;
> - if ((radeon_runtime_pm == -1) && radeon_is_px())
> + if (rdev->flags & RADEON_IS_PX)
> runtime = true;
> vga_switcheroo_register_client(rdev->pdev, &radeon_switcheroo_ops, runtime);
> if (runtime)
> diff --git a/drivers/gpu/drm/radeon/radeon_drv.c b/drivers/gpu/drm/radeon/radeon_drv.c
> index 2dc2cc4..3811812 100644
> --- a/drivers/gpu/drm/radeon/radeon_drv.c
> +++ b/drivers/gpu/drm/radeon/radeon_drv.c
> @@ -114,6 +114,7 @@ extern int radeon_get_crtc_scanoutpos(struct drm_device *dev, int crtc,
> unsigned int flags,
> int *vpos, int *hpos, ktime_t *stime,
> ktime_t *etime);
> +extern bool radeon_is_px(struct drm_device *dev);
> extern const struct drm_ioctl_desc radeon_ioctls_kms[];
> extern int radeon_max_kms_ioctl;
> int radeon_mmap(struct file *filp, struct vm_area_struct *vma);
> @@ -143,11 +144,9 @@ void radeon_debugfs_cleanup(struct drm_minor *minor);
> #if defined(CONFIG_VGA_SWITCHEROO)
> void radeon_register_atpx_handler(void);
> void radeon_unregister_atpx_handler(void);
> -bool radeon_is_px(void);
> #else
> static inline void radeon_register_atpx_handler(void) {}
> static inline void radeon_unregister_atpx_handler(void) {}
> -static inline bool radeon_is_px(void) { return false; }
> #endif
>
> int radeon_no_wb;
> @@ -404,12 +403,7 @@ static int radeon_pmops_runtime_suspend(struct device *dev)
> struct drm_device *drm_dev = pci_get_drvdata(pdev);
> int ret;
>
> - if (radeon_runtime_pm == 0) {
> - pm_runtime_forbid(dev);
> - return -EBUSY;
> - }
> -
> - if (radeon_runtime_pm == -1 && !radeon_is_px()) {
> + if (!radeon_is_px(drm_dev)) {
> pm_runtime_forbid(dev);
> return -EBUSY;
> }
> @@ -433,10 +427,7 @@ static int radeon_pmops_runtime_resume(struct device *dev)
> struct drm_device *drm_dev = pci_get_drvdata(pdev);
> int ret;
>
> - if (radeon_runtime_pm == 0)
> - return -EINVAL;
> -
> - if (radeon_runtime_pm == -1 && !radeon_is_px())
> + if (!radeon_is_px(drm_dev))
> return -EINVAL;
>
> drm_dev->switch_power_state = DRM_SWITCH_POWER_CHANGING;
> @@ -461,14 +452,7 @@ static int radeon_pmops_runtime_idle(struct device *dev)
> struct drm_device *drm_dev = pci_get_drvdata(pdev);
> struct drm_crtc *crtc;
>
> - if (radeon_runtime_pm == 0) {
> - pm_runtime_forbid(dev);
> - return -EBUSY;
> - }
> -
> - /* are we PX enabled? */
> - if (radeon_runtime_pm == -1 && !radeon_is_px()) {
> - DRM_DEBUG_DRIVER("failing to power off - not px\n");
> + if (!radeon_is_px(drm_dev)) {
> pm_runtime_forbid(dev);
> return -EBUSY;
> }
> diff --git a/drivers/gpu/drm/radeon/radeon_family.h b/drivers/gpu/drm/radeon/radeon_family.h
> index 614ad54..9da5da4 100644
> --- a/drivers/gpu/drm/radeon/radeon_family.h
> +++ b/drivers/gpu/drm/radeon/radeon_family.h
> @@ -115,6 +115,7 @@ enum radeon_chip_flags {
> RADEON_NEW_MEMMAP = 0x00400000UL,
> RADEON_IS_PCI = 0x00800000UL,
> RADEON_IS_IGPGART = 0x01000000UL,
> + RADEON_IS_PX = 0x02000000UL,
> };
>
> #endif
> diff --git a/drivers/gpu/drm/radeon/radeon_kms.c b/drivers/gpu/drm/radeon/radeon_kms.c
> index 3e49342..9337820 100644
> --- a/drivers/gpu/drm/radeon/radeon_kms.c
> +++ b/drivers/gpu/drm/radeon/radeon_kms.c
> @@ -35,9 +35,9 @@
> #include <linux/pm_runtime.h>
>
> #if defined(CONFIG_VGA_SWITCHEROO)
> -bool radeon_is_px(void);
> +bool radeon_has_atpx(void);
> #else
> -static inline bool radeon_is_px(void) { return false; }
> +static inline bool radeon_has_atpx(void) { return false; }
> #endif
>
> /**
> @@ -107,6 +107,13 @@ int radeon_driver_load_kms(struct drm_device *dev, unsigned long flags)
> flags |= RADEON_IS_PCI;
> }
>
> + if (radeon_runtime_pm == 1)
> + flags |= RADEON_IS_PX;
> + else if ((radeon_runtime_pm == -1) &&
> + radeon_has_atpx() &&
> + ((flags & RADEON_IS_IGP) == 0))
> + flags |= RADEON_IS_PX;
> +
> /* radeon_device_init should report only fatal error
> * like memory allocation failure or iomapping failure,
> * or memory manager initialization failure, it must
> @@ -137,8 +144,7 @@ int radeon_driver_load_kms(struct drm_device *dev, unsigned long flags)
> "Error during ACPI methods call\n");
> }
>
> - if ((radeon_runtime_pm == 1) ||
> - ((radeon_runtime_pm == -1) && radeon_is_px())) {
> + if (radeon_is_px(dev)) {
> pm_runtime_use_autosuspend(dev->dev);
> pm_runtime_set_autosuspend_delay(dev->dev, 5000);
> pm_runtime_set_active(dev->dev);
^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: [PATCH] drm/radeon: fix runpm handling on APUs (v4)
2014-04-11 13:10 ` [PATCH] drm/radeon: fix runpm handling on APUs (v4) Christian König
@ 2014-04-11 13:12 ` Deucher, Alexander
0 siblings, 0 replies; 5+ messages in thread
From: Deucher, Alexander @ 2014-04-11 13:12 UTC (permalink / raw)
To: Christian König, Alex Deucher, dri-devel; +Cc: stable
> -----Original Message-----
> From: Christian König [mailto:deathsimple@vodafone.de]
> Sent: Friday, April 11, 2014 9:10 AM
> To: Alex Deucher; dri-devel@lists.freedesktop.org
> Cc: Deucher, Alexander; stable@vger.kernel.org
> Subject: Re: [PATCH] drm/radeon: fix runpm handling on APUs (v4)
>
> Am 11.04.2014 04:29, schrieb Alex Deucher:
> > Don't try and runtime suspend the APU in PX systems. We
> > only want to power down the dGPU.
> >
> > v2: fix harder
> > v3: fix stupid typo
> > v4: consolidate runpm enablement to a single flag
> >
> > bugs:
> > https://bugs.freedesktop.org/show_bug.cgi?id=75127
> > https://bugzilla.kernel.org/show_bug.cgi?id=72701
> >
> > Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
> > Cc: stable@vger.kernel.org
>
> All three patches added to my drm-fixes-3.15 queue.
>
> Is it correct that "[PATCH] drm/radeon: update CI DPM powertune
> settings" doesn't have a "CC: stable..." ?
I want to let it get some wider testing before I send it back to stable.
Alex
>
> Christian.
>
> > ---
> > drivers/gpu/drm/radeon/radeon.h | 1 +
> > drivers/gpu/drm/radeon/radeon_atpx_handler.c | 2 +-
> > drivers/gpu/drm/radeon/radeon_device.c | 19 ++++++++++---------
> > drivers/gpu/drm/radeon/radeon_drv.c | 24 ++++--------------------
> > drivers/gpu/drm/radeon/radeon_family.h | 1 +
> > drivers/gpu/drm/radeon/radeon_kms.c | 14 ++++++++++----
> > 6 files changed, 27 insertions(+), 34 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/radeon/radeon.h
> b/drivers/gpu/drm/radeon/radeon.h
> > index f0fc2c8..3d94c0d 100644
> > --- a/drivers/gpu/drm/radeon/radeon.h
> > +++ b/drivers/gpu/drm/radeon/radeon.h
> > @@ -2318,6 +2318,7 @@ struct radeon_device {
> > bool have_disp_power_ref;
> > };
> >
> > +bool radeon_is_px(struct drm_device *dev);
> > int radeon_device_init(struct radeon_device *rdev,
> > struct drm_device *ddev,
> > struct pci_dev *pdev,
> > diff --git a/drivers/gpu/drm/radeon/radeon_atpx_handler.c
> b/drivers/gpu/drm/radeon/radeon_atpx_handler.c
> > index fa9a9c0..dedea72 100644
> > --- a/drivers/gpu/drm/radeon/radeon_atpx_handler.c
> > +++ b/drivers/gpu/drm/radeon/radeon_atpx_handler.c
> > @@ -59,7 +59,7 @@ struct atpx_mux {
> > u16 mux;
> > } __packed;
> >
> > -bool radeon_is_px(void) {
> > +bool radeon_has_atpx(void) {
> > return radeon_atpx_priv.atpx_detected;
> > }
> >
> > diff --git a/drivers/gpu/drm/radeon/radeon_device.c
> b/drivers/gpu/drm/radeon/radeon_device.c
> > index 2e72dcd..00b19d0 100644
> > --- a/drivers/gpu/drm/radeon/radeon_device.c
> > +++ b/drivers/gpu/drm/radeon/radeon_device.c
> > @@ -102,11 +102,14 @@ static const char radeon_family_name[][16] = {
> > "LAST",
> > };
> >
> > -#if defined(CONFIG_VGA_SWITCHEROO)
> > -bool radeon_is_px(void);
> > -#else
> > -static inline bool radeon_is_px(void) { return false; }
> > -#endif
> > +bool radeon_is_px(struct drm_device *dev)
> > +{
> > + struct radeon_device *rdev = dev->dev_private;
> > +
> > + if (rdev->flags & RADEON_IS_PX)
> > + return true;
> > + return false;
> > +}
> >
> > /**
> > * radeon_program_register_sequence - program an array of registers.
> > @@ -1082,7 +1085,7 @@ static void radeon_switcheroo_set_state(struct
> pci_dev *pdev, enum vga_switchero
> > {
> > struct drm_device *dev = pci_get_drvdata(pdev);
> >
> > - if (radeon_is_px() && state == VGA_SWITCHEROO_OFF)
> > + if (radeon_is_px(dev) && state == VGA_SWITCHEROO_OFF)
> > return;
> >
> > if (state == VGA_SWITCHEROO_ON) {
> > @@ -1301,9 +1304,7 @@ int radeon_device_init(struct radeon_device
> *rdev,
> > * ignore it */
> > vga_client_register(rdev->pdev, rdev, NULL,
> radeon_vga_set_decode);
> >
> > - if (radeon_runtime_pm == 1)
> > - runtime = true;
> > - if ((radeon_runtime_pm == -1) && radeon_is_px())
> > + if (rdev->flags & RADEON_IS_PX)
> > runtime = true;
> > vga_switcheroo_register_client(rdev->pdev,
> &radeon_switcheroo_ops, runtime);
> > if (runtime)
> > diff --git a/drivers/gpu/drm/radeon/radeon_drv.c
> b/drivers/gpu/drm/radeon/radeon_drv.c
> > index 2dc2cc4..3811812 100644
> > --- a/drivers/gpu/drm/radeon/radeon_drv.c
> > +++ b/drivers/gpu/drm/radeon/radeon_drv.c
> > @@ -114,6 +114,7 @@ extern int radeon_get_crtc_scanoutpos(struct
> drm_device *dev, int crtc,
> > unsigned int flags,
> > int *vpos, int *hpos, ktime_t *stime,
> > ktime_t *etime);
> > +extern bool radeon_is_px(struct drm_device *dev);
> > extern const struct drm_ioctl_desc radeon_ioctls_kms[];
> > extern int radeon_max_kms_ioctl;
> > int radeon_mmap(struct file *filp, struct vm_area_struct *vma);
> > @@ -143,11 +144,9 @@ void radeon_debugfs_cleanup(struct drm_minor
> *minor);
> > #if defined(CONFIG_VGA_SWITCHEROO)
> > void radeon_register_atpx_handler(void);
> > void radeon_unregister_atpx_handler(void);
> > -bool radeon_is_px(void);
> > #else
> > static inline void radeon_register_atpx_handler(void) {}
> > static inline void radeon_unregister_atpx_handler(void) {}
> > -static inline bool radeon_is_px(void) { return false; }
> > #endif
> >
> > int radeon_no_wb;
> > @@ -404,12 +403,7 @@ static int radeon_pmops_runtime_suspend(struct
> device *dev)
> > struct drm_device *drm_dev = pci_get_drvdata(pdev);
> > int ret;
> >
> > - if (radeon_runtime_pm == 0) {
> > - pm_runtime_forbid(dev);
> > - return -EBUSY;
> > - }
> > -
> > - if (radeon_runtime_pm == -1 && !radeon_is_px()) {
> > + if (!radeon_is_px(drm_dev)) {
> > pm_runtime_forbid(dev);
> > return -EBUSY;
> > }
> > @@ -433,10 +427,7 @@ static int radeon_pmops_runtime_resume(struct
> device *dev)
> > struct drm_device *drm_dev = pci_get_drvdata(pdev);
> > int ret;
> >
> > - if (radeon_runtime_pm == 0)
> > - return -EINVAL;
> > -
> > - if (radeon_runtime_pm == -1 && !radeon_is_px())
> > + if (!radeon_is_px(drm_dev))
> > return -EINVAL;
> >
> > drm_dev->switch_power_state =
> DRM_SWITCH_POWER_CHANGING;
> > @@ -461,14 +452,7 @@ static int radeon_pmops_runtime_idle(struct
> device *dev)
> > struct drm_device *drm_dev = pci_get_drvdata(pdev);
> > struct drm_crtc *crtc;
> >
> > - if (radeon_runtime_pm == 0) {
> > - pm_runtime_forbid(dev);
> > - return -EBUSY;
> > - }
> > -
> > - /* are we PX enabled? */
> > - if (radeon_runtime_pm == -1 && !radeon_is_px()) {
> > - DRM_DEBUG_DRIVER("failing to power off - not px\n");
> > + if (!radeon_is_px(drm_dev)) {
> > pm_runtime_forbid(dev);
> > return -EBUSY;
> > }
> > diff --git a/drivers/gpu/drm/radeon/radeon_family.h
> b/drivers/gpu/drm/radeon/radeon_family.h
> > index 614ad54..9da5da4 100644
> > --- a/drivers/gpu/drm/radeon/radeon_family.h
> > +++ b/drivers/gpu/drm/radeon/radeon_family.h
> > @@ -115,6 +115,7 @@ enum radeon_chip_flags {
> > RADEON_NEW_MEMMAP = 0x00400000UL,
> > RADEON_IS_PCI = 0x00800000UL,
> > RADEON_IS_IGPGART = 0x01000000UL,
> > + RADEON_IS_PX = 0x02000000UL,
> > };
> >
> > #endif
> > diff --git a/drivers/gpu/drm/radeon/radeon_kms.c
> b/drivers/gpu/drm/radeon/radeon_kms.c
> > index 3e49342..9337820 100644
> > --- a/drivers/gpu/drm/radeon/radeon_kms.c
> > +++ b/drivers/gpu/drm/radeon/radeon_kms.c
> > @@ -35,9 +35,9 @@
> > #include <linux/pm_runtime.h>
> >
> > #if defined(CONFIG_VGA_SWITCHEROO)
> > -bool radeon_is_px(void);
> > +bool radeon_has_atpx(void);
> > #else
> > -static inline bool radeon_is_px(void) { return false; }
> > +static inline bool radeon_has_atpx(void) { return false; }
> > #endif
> >
> > /**
> > @@ -107,6 +107,13 @@ int radeon_driver_load_kms(struct drm_device
> *dev, unsigned long flags)
> > flags |= RADEON_IS_PCI;
> > }
> >
> > + if (radeon_runtime_pm == 1)
> > + flags |= RADEON_IS_PX;
> > + else if ((radeon_runtime_pm == -1) &&
> > + radeon_has_atpx() &&
> > + ((flags & RADEON_IS_IGP) == 0))
> > + flags |= RADEON_IS_PX;
> > +
> > /* radeon_device_init should report only fatal error
> > * like memory allocation failure or iomapping failure,
> > * or memory manager initialization failure, it must
> > @@ -137,8 +144,7 @@ int radeon_driver_load_kms(struct drm_device
> *dev, unsigned long flags)
> > "Error during ACPI methods call\n");
> > }
> >
> > - if ((radeon_runtime_pm == 1) ||
> > - ((radeon_runtime_pm == -1) && radeon_is_px())) {
> > + if (radeon_is_px(dev)) {
> > pm_runtime_use_autosuspend(dev->dev);
> > pm_runtime_set_autosuspend_delay(dev->dev, 5000);
> > pm_runtime_set_active(dev->dev);
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2014-04-11 13:12 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-04-11 2:29 [PATCH] drm/radeon: fix runpm handling on APUs (v4) Alex Deucher
2014-04-11 2:29 ` [PATCH] drm/radeon: update CI DPM powertune settings Alex Deucher
2014-04-11 2:29 ` [PATCH] drm/radeon: disable mclk dpm on R7 260X Alex Deucher
2014-04-11 13:10 ` [PATCH] drm/radeon: fix runpm handling on APUs (v4) Christian König
2014-04-11 13:12 ` Deucher, Alexander
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.