summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRafael J. Wysocki <rafael.j.wysocki@intel.com>2016-12-24 00:29:56 +0100
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>2016-12-27 03:30:11 +0100
commit366430b5c2f61a75d45b9fc00886ffff12f395ab (patch)
tree729f466e442a1113a71fec3811f0bbf19e7bdd4d
parentadec57c61c2421d9d06c1fa8dd1ff7ed4fd2ca1b (diff)
downloadlinux-366430b5c2f61a75d45b9fc00886ffff12f395ab.tar.gz
linux-366430b5c2f61a75d45b9fc00886ffff12f395ab.tar.bz2
linux-366430b5c2f61a75d45b9fc00886ffff12f395ab.zip
cpufreq: intel_pstate: Do not expose PID parameters in passive mode
If intel_pstate works in the passive mode in which it acts as a regular cpufreq driver and collaborates with generic cpufreq governors, the PID parameters are not used, so do not expose them via debugfs in that case. Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
-rw-r--r--drivers/cpufreq/intel_pstate.c9
1 files changed, 4 insertions, 5 deletions
diff --git a/drivers/cpufreq/intel_pstate.c b/drivers/cpufreq/intel_pstate.c
index 7cd0177ddeaf..0d01cb2dc6d2 100644
--- a/drivers/cpufreq/intel_pstate.c
+++ b/drivers/cpufreq/intel_pstate.c
@@ -964,10 +964,6 @@ static void __init intel_pstate_debug_expose_params(void)
struct dentry *debugfs_parent;
int i = 0;
- if (hwp_active ||
- pstate_funcs.get_target_pstate == get_target_pstate_use_cpu_load)
- return;
-
debugfs_parent = debugfs_create_dir("pstate_snb", NULL);
if (IS_ERR_OR_NULL(debugfs_parent))
return;
@@ -2433,7 +2429,10 @@ hwp_cpu_matched:
if (rc)
goto out;
- intel_pstate_debug_expose_params();
+ if (intel_pstate_driver == &intel_pstate && !hwp_active &&
+ pstate_funcs.get_target_pstate != get_target_pstate_use_cpu_load)
+ intel_pstate_debug_expose_params();
+
intel_pstate_sysfs_expose_params();
if (hwp_active)