Re: [RFC][PATCH] cpufreq: intel_pstate: Delete intel_pstate sysfs if failed to register the driver

2020-10-16 Thread Rafael J. Wysocki
On Fri, Oct 9, 2020 at 5:29 AM Chen Yu wrote: > > There is a corner case that if the intel_pstate driver failed to be > registered(might be due to invalid MSR access) and with the acpi_cpufreq > loaded, the intel_pstate sysfs might still be created, which makes the > user confusing(turbostat for

Re: [RFC][PATCH] cpufreq: intel_pstate: Delete intel_pstate sysfs if failed to register the driver

2020-10-13 Thread Chen Yu
Hi Srinivas, On Mon, Oct 12, 2020 at 06:22:40AM -0700, srinivas pandruvada wrote: > On Fri, 2020-10-09 at 11:30 +0800, Chen Yu wrote: > > There is a corner case that if the intel_pstate driver failed to be > > registered(might be due to invalid MSR access) > Do you have logs why it is not loaded?

Re: [RFC][PATCH] cpufreq: intel_pstate: Delete intel_pstate sysfs if failed to register the driver

2020-10-12 Thread srinivas pandruvada
On Fri, 2020-10-09 at 11:30 +0800, Chen Yu wrote: > There is a corner case that if the intel_pstate driver failed to be > registered(might be due to invalid MSR access) Do you have logs why it is not loaded? On supported platforms MSRs should be invalid. It may be a case when we are trying to

[RFC][PATCH] cpufreq: intel_pstate: Delete intel_pstate sysfs if failed to register the driver

2020-10-08 Thread Chen Yu
There is a corner case that if the intel_pstate driver failed to be registered(might be due to invalid MSR access) and with the acpi_cpufreq loaded, the intel_pstate sysfs might still be created, which makes the user confusing(turbostat for example): grep .