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

Re: [Xen-devel] [PATCH v2 7/9] x86/intel_pstate: add a booting param to select the driver to load



>>> On 03.06.15 at 10:07, <wei.w.wang@xxxxxxxxx> wrote:
> On 26/05/2015 22:02, Jan Beulich wrote
>> >>> On 13.05.16 at 09:51, <wei.w.wang@xxxxxxxxx> 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@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.