RE: [PATCH] drm/radeon: fix runpm handling on APUs (v4)

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

 



> -----Original Message-----
> From: Christian König [mailto:deathsimple@xxxxxxxxxxx]
> Sent: Friday, April 11, 2014 9:10 AM
> To: Alex Deucher; dri-devel@xxxxxxxxxxxxxxxxxxxxx
> Cc: Deucher, Alexander; stable@xxxxxxxxxxxxxxx
> 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@xxxxxxx>
> > Cc: stable@xxxxxxxxxxxxxxx
> 
> 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);

--
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]