diff options
author | Rusty Russell <rusty@rustcorp.com.au> | 2012-01-12 18:02:18 -0500 |
---|---|---|
committer | Rusty Russell <rusty@rustcorp.com.au> | 2012-01-12 18:02:18 -0500 |
commit | 476bc0015bf09dad39d36a8b19f76f0c181d1ec9 (patch) | |
tree | 054f322e639affab256cf7849f59f1dcdf117094 /arch/x86/kvm | |
parent | 2329abfa344a9a824bc4c71f2415528777265510 (diff) |
module_param: make bool parameters really bool (arch)
module_param(bool) used to counter-intuitively take an int. In
fddd5201 (mid-2009) we allowed bool or int/unsigned int using a messy
trick.
It's time to remove the int/unsigned int option. For this version
it'll simply give a warning, but it'll break next kernel version.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Diffstat (limited to 'arch/x86/kvm')
-rw-r--r-- | arch/x86/kvm/mmu.c | 2 | ||||
-rw-r--r-- | arch/x86/kvm/vmx.c | 18 | ||||
-rw-r--r-- | arch/x86/kvm/x86.c | 4 |
3 files changed, 12 insertions, 12 deletions
diff --git a/arch/x86/kvm/mmu.c b/arch/x86/kvm/mmu.c index 2a2a9b40db19..224b02c3cda9 100644 --- a/arch/x86/kvm/mmu.c +++ b/arch/x86/kvm/mmu.c | |||
@@ -74,7 +74,7 @@ enum { | |||
74 | #endif | 74 | #endif |
75 | 75 | ||
76 | #ifdef MMU_DEBUG | 76 | #ifdef MMU_DEBUG |
77 | static int dbg = 0; | 77 | static bool dbg = 0; |
78 | module_param(dbg, bool, 0644); | 78 | module_param(dbg, bool, 0644); |
79 | #endif | 79 | #endif |
80 | 80 | ||
diff --git a/arch/x86/kvm/vmx.c b/arch/x86/kvm/vmx.c index 906a7e84200f..d29216c462b3 100644 --- a/arch/x86/kvm/vmx.c +++ b/arch/x86/kvm/vmx.c | |||
@@ -51,29 +51,29 @@ | |||
51 | MODULE_AUTHOR("Qumranet"); | 51 | MODULE_AUTHOR("Qumranet"); |
52 | MODULE_LICENSE("GPL"); | 52 | MODULE_LICENSE("GPL"); |
53 | 53 | ||
54 | static int __read_mostly enable_vpid = 1; | 54 | static bool __read_mostly enable_vpid = 1; |
55 | module_param_named(vpid, enable_vpid, bool, 0444); | 55 | module_param_named(vpid, enable_vpid, bool, 0444); |
56 | 56 | ||
57 | static int __read_mostly flexpriority_enabled = 1; | 57 | static bool __read_mostly flexpriority_enabled = 1; |
58 | module_param_named(flexpriority, flexpriority_enabled, bool, S_IRUGO); | 58 | module_param_named(flexpriority, flexpriority_enabled, bool, S_IRUGO); |
59 | 59 | ||
60 | static int __read_mostly enable_ept = 1; | 60 | static bool __read_mostly enable_ept = 1; |
61 | module_param_named(ept, enable_ept, bool, S_IRUGO); | 61 | module_param_named(ept, enable_ept, bool, S_IRUGO); |
62 | 62 | ||
63 | static int __read_mostly enable_unrestricted_guest = 1; | 63 | static bool __read_mostly enable_unrestricted_guest = 1; |
64 | module_param_named(unrestricted_guest, | 64 | module_param_named(unrestricted_guest, |
65 | enable_unrestricted_guest, bool, S_IRUGO); | 65 | enable_unrestricted_guest, bool, S_IRUGO); |
66 | 66 | ||
67 | static int __read_mostly emulate_invalid_guest_state = 0; | 67 | static bool __read_mostly emulate_invalid_guest_state = 0; |
68 | module_param(emulate_invalid_guest_state, bool, S_IRUGO); | 68 | module_param(emulate_invalid_guest_state, bool, S_IRUGO); |
69 | 69 | ||
70 | static int __read_mostly vmm_exclusive = 1; | 70 | static bool __read_mostly vmm_exclusive = 1; |
71 | module_param(vmm_exclusive, bool, S_IRUGO); | 71 | module_param(vmm_exclusive, bool, S_IRUGO); |
72 | 72 | ||
73 | static int __read_mostly yield_on_hlt = 1; | 73 | static bool __read_mostly yield_on_hlt = 1; |
74 | module_param(yield_on_hlt, bool, S_IRUGO); | 74 | module_param(yield_on_hlt, bool, S_IRUGO); |
75 | 75 | ||
76 | static int __read_mostly fasteoi = 1; | 76 | static bool __read_mostly fasteoi = 1; |
77 | module_param(fasteoi, bool, S_IRUGO); | 77 | module_param(fasteoi, bool, S_IRUGO); |
78 | 78 | ||
79 | /* | 79 | /* |
@@ -81,7 +81,7 @@ module_param(fasteoi, bool, S_IRUGO); | |||
81 | * VMX and be a hypervisor for its own guests. If nested=0, guests may not | 81 | * VMX and be a hypervisor for its own guests. If nested=0, guests may not |
82 | * use VMX instructions. | 82 | * use VMX instructions. |
83 | */ | 83 | */ |
84 | static int __read_mostly nested = 0; | 84 | static bool __read_mostly nested = 0; |
85 | module_param(nested, bool, S_IRUGO); | 85 | module_param(nested, bool, S_IRUGO); |
86 | 86 | ||
87 | #define KVM_GUEST_CR0_MASK_UNRESTRICTED_GUEST \ | 87 | #define KVM_GUEST_CR0_MASK_UNRESTRICTED_GUEST \ |
diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c index 1171def5f96b..14d6cadc4ba6 100644 --- a/arch/x86/kvm/x86.c +++ b/arch/x86/kvm/x86.c | |||
@@ -88,8 +88,8 @@ static void process_nmi(struct kvm_vcpu *vcpu); | |||
88 | struct kvm_x86_ops *kvm_x86_ops; | 88 | struct kvm_x86_ops *kvm_x86_ops; |
89 | EXPORT_SYMBOL_GPL(kvm_x86_ops); | 89 | EXPORT_SYMBOL_GPL(kvm_x86_ops); |
90 | 90 | ||
91 | int ignore_msrs = 0; | 91 | static bool ignore_msrs = 0; |
92 | module_param_named(ignore_msrs, ignore_msrs, bool, S_IRUGO | S_IWUSR); | 92 | module_param(ignore_msrs, bool, S_IRUGO | S_IWUSR); |
93 | 93 | ||
94 | bool kvm_has_tsc_control; | 94 | bool kvm_has_tsc_control; |
95 | EXPORT_SYMBOL_GPL(kvm_has_tsc_control); | 95 | EXPORT_SYMBOL_GPL(kvm_has_tsc_control); |