Commit 26951ec8 authored by Peng Hao's avatar Peng Hao Committed by Sean Christopherson
Browse files

KVM: x86: Use octal for file permission 



Convert all module params to octal permissions to improve code readability
and to make checkpatch happy:

  WARNING: Symbolic permissions 'S_IRUGO' are not preferred. Consider using
           octal permissions '0444'.

Signed-off-by: default avatarPeng Hao <flyingpeng@tencent.com>
Link: https://lore.kernel.org/r/20231013113020.77523-1-flyingpeng@tencent.com


Signed-off-by: default avatarSean Christopherson <seanjc@google.com>
parent bf328e22

arch/x86/kvm/svm/svm.c

+1 −1
Original line number Diff line number Diff line
@@ -199,7 +199,7 @@ module_param_named(npt, npt_enabled, bool, 0444); 


/* allow nested virtualization in KVM/SVM */

static int nested = true;

module_param(nested, int, S_IRUGO);

module_param(nested, int, 0444);



/* enable/disable Next RIP Save */

int nrips = true;

arch/x86/kvm/vmx/vmx.c

+10 −10
Original line number Diff line number Diff line
@@ -82,28 +82,28 @@ bool __read_mostly enable_vpid = 1; 
module_param_named(vpid, enable_vpid, bool, 0444);



static bool __read_mostly enable_vnmi = 1;

module_param_named(vnmi, enable_vnmi, bool, S_IRUGO);

module_param_named(vnmi, enable_vnmi, bool, 0444);



bool __read_mostly flexpriority_enabled = 1;

module_param_named(flexpriority, flexpriority_enabled, bool, S_IRUGO);

module_param_named(flexpriority, flexpriority_enabled, bool, 0444);



bool __read_mostly enable_ept = 1;

module_param_named(ept, enable_ept, bool, S_IRUGO);

module_param_named(ept, enable_ept, bool, 0444);



bool __read_mostly enable_unrestricted_guest = 1;

module_param_named(unrestricted_guest,

			enable_unrestricted_guest, bool, S_IRUGO);

			enable_unrestricted_guest, bool, 0444);



bool __read_mostly enable_ept_ad_bits = 1;

module_param_named(eptad, enable_ept_ad_bits, bool, S_IRUGO);

module_param_named(eptad, enable_ept_ad_bits, bool, 0444);



static bool __read_mostly emulate_invalid_guest_state = true;

module_param(emulate_invalid_guest_state, bool, S_IRUGO);

module_param(emulate_invalid_guest_state, bool, 0444);



static bool __read_mostly fasteoi = 1;

module_param(fasteoi, bool, S_IRUGO);

module_param(fasteoi, bool, 0444);



module_param(enable_apicv, bool, S_IRUGO);

module_param(enable_apicv, bool, 0444);



bool __read_mostly enable_ipiv = true;

module_param(enable_ipiv, bool, 0444);
@@ -114,10 +114,10 @@ module_param(enable_ipiv, bool, 0444); 
 * use VMX instructions.

 */

static bool __read_mostly nested = 1;

module_param(nested, bool, S_IRUGO);

module_param(nested, bool, 0444);



bool __read_mostly enable_pml = 1;

module_param_named(pml, enable_pml, bool, S_IRUGO);

module_param_named(pml, enable_pml, bool, 0444);



static bool __read_mostly error_on_inconsistent_vmcs_config = true;

module_param(error_on_inconsistent_vmcs_config, bool, 0444);

arch/x86/kvm/x86.c

+9 −9
Original line number Diff line number Diff line
@@ -145,21 +145,21 @@ EXPORT_STATIC_CALL_GPL(kvm_x86_get_cs_db_l_bits); 
EXPORT_STATIC_CALL_GPL(kvm_x86_cache_reg);



static bool __read_mostly ignore_msrs = 0;

module_param(ignore_msrs, bool, S_IRUGO | S_IWUSR);

module_param(ignore_msrs, bool, 0644);



bool __read_mostly report_ignored_msrs = true;

module_param(report_ignored_msrs, bool, S_IRUGO | S_IWUSR);

module_param(report_ignored_msrs, bool, 0644);

EXPORT_SYMBOL_GPL(report_ignored_msrs);



unsigned int min_timer_period_us = 200;

module_param(min_timer_period_us, uint, S_IRUGO | S_IWUSR);

module_param(min_timer_period_us, uint, 0644);



static bool __read_mostly kvmclock_periodic_sync = true;

module_param(kvmclock_periodic_sync, bool, S_IRUGO);

module_param(kvmclock_periodic_sync, bool, 0444);



/* tsc tolerance in parts per million - default to 1/2 of the NTP threshold */

static u32 __read_mostly tsc_tolerance_ppm = 250;

module_param(tsc_tolerance_ppm, uint, S_IRUGO | S_IWUSR);

module_param(tsc_tolerance_ppm, uint, 0644);



/*

 * lapic timer advance (tscdeadline mode only) in nanoseconds.  '-1' enables
@@ -168,13 +168,13 @@ module_param(tsc_tolerance_ppm, uint, S_IRUGO | S_IWUSR); 
 * tuning, i.e. allows privileged userspace to set an exact advancement time.

 */

static int __read_mostly lapic_timer_advance_ns = -1;

module_param(lapic_timer_advance_ns, int, S_IRUGO | S_IWUSR);

module_param(lapic_timer_advance_ns, int, 0644);



static bool __read_mostly vector_hashing = true;

module_param(vector_hashing, bool, S_IRUGO);

module_param(vector_hashing, bool, 0444);



bool __read_mostly enable_vmware_backdoor = false;

module_param(enable_vmware_backdoor, bool, S_IRUGO);

module_param(enable_vmware_backdoor, bool, 0444);

EXPORT_SYMBOL_GPL(enable_vmware_backdoor);



/*
@@ -186,7 +186,7 @@ static int __read_mostly force_emulation_prefix; 
module_param(force_emulation_prefix, int, 0644);



int __read_mostly pi_inject_timer = -1;

module_param(pi_inject_timer, bint, S_IRUGO | S_IWUSR);

module_param(pi_inject_timer, bint, 0644);



/* Enable/disable PMU virtualization */

bool __read_mostly enable_pmu = true;