KVM: x86 emulator: tighen up ->read_std() and ->write_std() error checks
authorAvi Kivity <avi@redhat.com>
Mon, 22 Nov 2010 15:53:24 +0000 (17:53 +0200)
committerAvi Kivity <avi@redhat.com>
Wed, 12 Jan 2011 09:29:56 +0000 (11:29 +0200)
Instead of checking for X86EMUL_PROPAGATE_FAULT, check for any error,
making the callers more reliable.

Signed-off-by: Avi Kivity <avi@redhat.com>
Signed-off-by: Marcelo Tosatti <mtosatti@redhat.com>
arch/x86/kvm/emulate.c

index 345aa4d3614a4c23df7ead61effcb20b901a34ef..223c536b7f71e34cf8a5825b77a093a3817bb70a 100644 (file)
@@ -1915,7 +1915,7 @@ static int task_switch_16(struct x86_emulate_ctxt *ctxt,
 
        ret = ops->read_std(old_tss_base, &tss_seg, sizeof tss_seg, ctxt->vcpu,
                            &ctxt->exception);
-       if (ret == X86EMUL_PROPAGATE_FAULT)
+       if (ret != X86EMUL_CONTINUE)
                /* FIXME: need to provide precise fault address */
                return ret;
 
@@ -1923,13 +1923,13 @@ static int task_switch_16(struct x86_emulate_ctxt *ctxt,
 
        ret = ops->write_std(old_tss_base, &tss_seg, sizeof tss_seg, ctxt->vcpu,
                             &ctxt->exception);
-       if (ret == X86EMUL_PROPAGATE_FAULT)
+       if (ret != X86EMUL_CONTINUE)
                /* FIXME: need to provide precise fault address */
                return ret;
 
        ret = ops->read_std(new_tss_base, &tss_seg, sizeof tss_seg, ctxt->vcpu,
                            &ctxt->exception);
-       if (ret == X86EMUL_PROPAGATE_FAULT)
+       if (ret != X86EMUL_CONTINUE)
                /* FIXME: need to provide precise fault address */
                return ret;
 
@@ -1940,7 +1940,7 @@ static int task_switch_16(struct x86_emulate_ctxt *ctxt,
                                     &tss_seg.prev_task_link,
                                     sizeof tss_seg.prev_task_link,
                                     ctxt->vcpu, &ctxt->exception);
-               if (ret == X86EMUL_PROPAGATE_FAULT)
+               if (ret != X86EMUL_CONTINUE)
                        /* FIXME: need to provide precise fault address */
                        return ret;
        }
@@ -2049,7 +2049,7 @@ static int task_switch_32(struct x86_emulate_ctxt *ctxt,
 
        ret = ops->read_std(old_tss_base, &tss_seg, sizeof tss_seg, ctxt->vcpu,
                            &ctxt->exception);
-       if (ret == X86EMUL_PROPAGATE_FAULT)
+       if (ret != X86EMUL_CONTINUE)
                /* FIXME: need to provide precise fault address */
                return ret;
 
@@ -2057,13 +2057,13 @@ static int task_switch_32(struct x86_emulate_ctxt *ctxt,
 
        ret = ops->write_std(old_tss_base, &tss_seg, sizeof tss_seg, ctxt->vcpu,
                             &ctxt->exception);
-       if (ret == X86EMUL_PROPAGATE_FAULT)
+       if (ret != X86EMUL_CONTINUE)
                /* FIXME: need to provide precise fault address */
                return ret;
 
        ret = ops->read_std(new_tss_base, &tss_seg, sizeof tss_seg, ctxt->vcpu,
                            &ctxt->exception);
-       if (ret == X86EMUL_PROPAGATE_FAULT)
+       if (ret != X86EMUL_CONTINUE)
                /* FIXME: need to provide precise fault address */
                return ret;
 
@@ -2074,7 +2074,7 @@ static int task_switch_32(struct x86_emulate_ctxt *ctxt,
                                     &tss_seg.prev_task_link,
                                     sizeof tss_seg.prev_task_link,
                                     ctxt->vcpu, &ctxt->exception);
-               if (ret == X86EMUL_PROPAGATE_FAULT)
+               if (ret != X86EMUL_CONTINUE)
                        /* FIXME: need to provide precise fault address */
                        return ret;
        }