diff options
author | Julia Lawall <julia@diku.dk> | 2011-08-02 06:34:57 -0400 |
---|---|---|
committer | Avi Kivity <avi@redhat.com> | 2011-09-25 12:18:01 -0400 |
commit | cf3ace79c065d65e9636f719a9df1382725410e3 (patch) | |
tree | 26ad4454252fba25f059c320922af2da494f68be /arch/x86/kvm | |
parent | 742bc67042e34a9fe1fed0b46e4cb1431a72c4bf (diff) |
KVM: VMX: trivial: use BUG_ON
Use BUG_ON(x) rather than if(x) BUG();
The semantic patch that fixes this problem is as follows:
(http://coccinelle.lip6.fr/)
// <smpl>
@@ identifier x; @@
-if (x) BUG();
+BUG_ON(x);
@@ identifier x; @@
-if (!x) BUG();
+BUG_ON(!x);
// </smpl>
Signed-off-by: Julia Lawall <julia@diku.dk>
Signed-off-by: Marcelo Tosatti <mtosatti@redhat.com>
Diffstat (limited to 'arch/x86/kvm')
-rw-r--r-- | arch/x86/kvm/vmx.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/arch/x86/kvm/vmx.c b/arch/x86/kvm/vmx.c index e26629fbf1d7..03df703c8f20 100644 --- a/arch/x86/kvm/vmx.c +++ b/arch/x86/kvm/vmx.c | |||
@@ -4115,8 +4115,7 @@ static int handle_exception(struct kvm_vcpu *vcpu) | |||
4115 | error_code = vmcs_read32(VM_EXIT_INTR_ERROR_CODE); | 4115 | error_code = vmcs_read32(VM_EXIT_INTR_ERROR_CODE); |
4116 | if (is_page_fault(intr_info)) { | 4116 | if (is_page_fault(intr_info)) { |
4117 | /* EPT won't cause page fault directly */ | 4117 | /* EPT won't cause page fault directly */ |
4118 | if (enable_ept) | 4118 | BUG_ON(enable_ept); |
4119 | BUG(); | ||
4120 | cr2 = vmcs_readl(EXIT_QUALIFICATION); | 4119 | cr2 = vmcs_readl(EXIT_QUALIFICATION); |
4121 | trace_kvm_page_fault(cr2, error_code); | 4120 | trace_kvm_page_fault(cr2, error_code); |
4122 | 4121 | ||