| Message ID | 20170905170628.10211-1-bp@alien8.de (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On 05.09.2017 19:06, Borislav Petkov wrote: > From: Borislav Petkov <bp@suse.de> > > There's no need to issue that everytime during boot - we have the > /proc/cpuinfo flag for people and software to query. > > Signed-off-by: Borislav Petkov <bp@suse.de> > Cc: Janakarajan Natarajan <Janakarajan.Natarajan@amd.com> > Cc: Paolo Bonzini <pbonzini@redhat.com> > Cc: Radim Krčmář <rkrcmar@redhat.com> > --- > arch/x86/kvm/svm.c | 5 +---- > 1 file changed, 1 insertion(+), 4 deletions(-) > > diff --git a/arch/x86/kvm/svm.c b/arch/x86/kvm/svm.c > index 8dbd8dbc83eb..f25e5b930932 100644 > --- a/arch/x86/kvm/svm.c > +++ b/arch/x86/kvm/svm.c > @@ -1101,11 +1101,8 @@ static __init int svm_hardware_setup(void) > if (vls) { > if (!npt_enabled || > !boot_cpu_has(X86_FEATURE_V_VMSAVE_VMLOAD) || > - !IS_ENABLED(CONFIG_X86_64)) { > + !IS_ENABLED(CONFIG_X86_64)) > vls = false; > - } else { > - pr_info("Virtual VMLOAD VMSAVE supported\n"); > - } > } > > return 0; > Maybe go one step further and incorporate everything (+vls) into a single if statement?
diff --git a/arch/x86/kvm/svm.c b/arch/x86/kvm/svm.c index 8dbd8dbc83eb..f25e5b930932 100644 --- a/arch/x86/kvm/svm.c +++ b/arch/x86/kvm/svm.c @@ -1101,11 +1101,8 @@ static __init int svm_hardware_setup(void) if (vls) { if (!npt_enabled || !boot_cpu_has(X86_FEATURE_V_VMSAVE_VMLOAD) || - !IS_ENABLED(CONFIG_X86_64)) { + !IS_ENABLED(CONFIG_X86_64)) vls = false; - } else { - pr_info("Virtual VMLOAD VMSAVE supported\n"); - } } return 0;