lkml.org 
[lkml]   [2016]   [Oct]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v4 2/4] x86/phi: Add enabling of the R3MWAIT during boot
On Tue, 18 Oct 2016, Grzegorz Andrejczuk wrote:
> +static void probe_xeon_phi_r3mwait(struct cpuinfo_x86 *c)
> +{
> + if (phi_r3mwait_disabled)
> + return;

So you return when the command line option to disable the feature was
given. That's not really a good choice if the feature is already enabled
due to BIOS featuritis. This can also happen when you kexec from a enabled
kernel into one which has the command line option set.

You really should make sure that the command line option results in
disabling the feature no matter what.

Thanks,

tglx

\
 
 \ /
  Last update: 2016-10-20 21:14    [W:0.055 / U:0.576 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site