>>> On 28.10.15 at 04:21, <wei.w.w...@intel.com> wrote: > Signed-off-by: Wei Wang <wei.w.w...@intel.com>
Acked-by: Jan Beulich <jbeul...@suse.com> with two remarks: > --- a/docs/misc/xen-command-line.markdown > +++ b/docs/misc/xen-command-line.markdown > @@ -858,6 +858,13 @@ debug hypervisor only). > ### idle\_latency\_factor > > `= <integer>` > > +### intel\_pstate > +> `= <boolean>` > + > +> Default: `false` > + > +Enable the loading of the intel pstate driver. "Loading" is probably not the right term, as it gets always loaded. Perhaps just drop "the loading of"? > --- a/xen/include/asm-x86/cpufreq.h > +++ b/xen/include/asm-x86/cpufreq.h > @@ -22,6 +22,8 @@ > * ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > */ > > +extern int intel_pstate_init(void); Considering that I questioned the creation of this header in the previous patch, the addition of this declaration should not result in it now being created here. In fact if any new header was needed for this declaration, it would probably better live in xen/arch/x86/acpi/cpufreq/. Moving this to a _suitable_ existing header does not void the ack given. Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
