Re: [Xen-devel] [PATCH v6 4/6] x86/intel_pstate: add a booting param to select the driver to load

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 28.10.15 at 04:21, <wei.w.wang@xxxxxxxxx> wrote:
> Signed-off-by: Wei Wang <wei.w.wang@xxxxxxxxx>

Acked-by: Jan Beulich <jbeulich@xxxxxxxx>
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@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel