>>> On 03.06.15 at 10:07, <wei.w.w...@intel.com> wrote:
> On 26/05/2015 22:02, Jan Beulich wrote
>> >>> On 13.05.16 at 09:51, <wei.w.w...@intel.com> wrote:
>> > --- a/xen/arch/x86/acpi/cpufreq/cpufreq.c
>> > +++ b/xen/arch/x86/acpi/cpufreq/cpufreq.c
>> > --- a/xen/arch/x86/acpi/cpufreq/intel_pstate.c
>> > +++ b/xen/arch/x86/acpi/cpufreq/intel_pstate.c
>> > @@ -766,6 +766,8 @@ static struct cpufreq_driver intel_pstate_driver = {
>> >      .name         = "intel_pstate",
>> >  };
>> >
>> > +int __initdata load_intel_pstate = 0;
>> 
>> static bool_t
> 
> I think we cannot use "static" here, since load_intel_pstate is also used in 
> cpufreq.c to select which driver to load.

Iirc I had also requested to deal with that (as it looks pretty hackish
right now).

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Reply via email to