On Wed, Apr 25, 2018 at 03:23:34PM +0100, Chris Wilson wrote: > A more complete, and more importantly stable, interface for controlling > the RPS frequency range is available in sysfs, obsoleting the unstable > debugfs. > > It's presence seems to trick people into using it, forgetting it is not > ABI. > > References: https://bugs.freedesktop.org/show_bug.cgi?id=106237 > Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > Reviewed-by: Sagar Arun Kamble <sagar.a.kamble@xxxxxxxxx> after CI gets happy: Acked-by: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx> > --- > drivers/gpu/drm/i915/i915_debugfs.c | 115 ---------------------------- > 1 file changed, 115 deletions(-) > > diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c > index 2f05f5262bba..1c88805d3354 100644 > --- a/drivers/gpu/drm/i915/i915_debugfs.c > +++ b/drivers/gpu/drm/i915/i915_debugfs.c > @@ -4204,119 +4204,6 @@ DEFINE_SIMPLE_ATTRIBUTE(i915_drop_caches_fops, > i915_drop_caches_get, i915_drop_caches_set, > "0x%08llx\n"); > > -static int > -i915_max_freq_get(void *data, u64 *val) > -{ > - struct drm_i915_private *dev_priv = data; > - > - if (INTEL_GEN(dev_priv) < 6) > - return -ENODEV; > - > - *val = intel_gpu_freq(dev_priv, dev_priv->gt_pm.rps.max_freq_softlimit); > - return 0; > -} > - > -static int > -i915_max_freq_set(void *data, u64 val) > -{ > - struct drm_i915_private *dev_priv = data; > - struct intel_rps *rps = &dev_priv->gt_pm.rps; > - u32 hw_max, hw_min; > - int ret; > - > - if (INTEL_GEN(dev_priv) < 6) > - return -ENODEV; > - > - DRM_DEBUG_DRIVER("Manually setting max freq to %llu\n", val); > - > - ret = mutex_lock_interruptible(&dev_priv->pcu_lock); > - if (ret) > - return ret; > - > - /* > - * Turbo will still be enabled, but won't go above the set value. > - */ > - val = intel_freq_opcode(dev_priv, val); > - > - hw_max = rps->max_freq; > - hw_min = rps->min_freq; > - > - if (val < hw_min || val > hw_max || val < rps->min_freq_softlimit) { > - mutex_unlock(&dev_priv->pcu_lock); > - return -EINVAL; > - } > - > - rps->max_freq_softlimit = val; > - > - if (intel_set_rps(dev_priv, val)) > - DRM_DEBUG_DRIVER("failed to update RPS to new softlimit\n"); > - > - mutex_unlock(&dev_priv->pcu_lock); > - > - return 0; > -} > - > -DEFINE_SIMPLE_ATTRIBUTE(i915_max_freq_fops, > - i915_max_freq_get, i915_max_freq_set, > - "%llu\n"); > - > -static int > -i915_min_freq_get(void *data, u64 *val) > -{ > - struct drm_i915_private *dev_priv = data; > - > - if (INTEL_GEN(dev_priv) < 6) > - return -ENODEV; > - > - *val = intel_gpu_freq(dev_priv, dev_priv->gt_pm.rps.min_freq_softlimit); > - return 0; > -} > - > -static int > -i915_min_freq_set(void *data, u64 val) > -{ > - struct drm_i915_private *dev_priv = data; > - struct intel_rps *rps = &dev_priv->gt_pm.rps; > - u32 hw_max, hw_min; > - int ret; > - > - if (INTEL_GEN(dev_priv) < 6) > - return -ENODEV; > - > - DRM_DEBUG_DRIVER("Manually setting min freq to %llu\n", val); > - > - ret = mutex_lock_interruptible(&dev_priv->pcu_lock); > - if (ret) > - return ret; > - > - /* > - * Turbo will still be enabled, but won't go below the set value. > - */ > - val = intel_freq_opcode(dev_priv, val); > - > - hw_max = rps->max_freq; > - hw_min = rps->min_freq; > - > - if (val < hw_min || > - val > hw_max || val > rps->max_freq_softlimit) { > - mutex_unlock(&dev_priv->pcu_lock); > - return -EINVAL; > - } > - > - rps->min_freq_softlimit = val; > - > - if (intel_set_rps(dev_priv, val)) > - DRM_DEBUG_DRIVER("failed to update RPS to new softlimit\n"); > - > - mutex_unlock(&dev_priv->pcu_lock); > - > - return 0; > -} > - > -DEFINE_SIMPLE_ATTRIBUTE(i915_min_freq_fops, > - i915_min_freq_get, i915_min_freq_set, > - "%llu\n"); > - > static int > i915_cache_sharing_get(void *data, u64 *val) > { > @@ -4878,8 +4765,6 @@ static const struct i915_debugfs_files { > const struct file_operations *fops; > } i915_debugfs_files[] = { > {"i915_wedged", &i915_wedged_fops}, > - {"i915_max_freq", &i915_max_freq_fops}, > - {"i915_min_freq", &i915_min_freq_fops}, > {"i915_cache_sharing", &i915_cache_sharing_fops}, > {"i915_ring_missed_irq", &i915_ring_missed_irq_fops}, > {"i915_ring_test_irq", &i915_ring_test_irq_fops}, > -- > 2.17.0 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@xxxxxxxxxxxxxxxxxxxxx > https://lists.freedesktop.org/mailman/listinfo/intel-gfx _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx