With commit 96ae556569 ("x86/HVM: fix forwarding of internally cached requests") rc doesn't change anymore in the respective preceding switch() statements.
Coverity ID: 1358080 Signed-off-by: Jan Beulich <jbeul...@suse.com> --- a/xen/arch/x86/hvm/intercept.c +++ b/xen/arch/x86/hvm/intercept.c @@ -151,8 +151,6 @@ int hvm_process_io_intercept(const struc domain_crash(current->domain); return X86EMUL_UNHANDLEABLE; } - if ( rc != X86EMUL_OKAY ) - break; } else p->data = data; @@ -181,8 +181,6 @@ int hvm_process_io_intercept(const struc domain_crash(current->domain); return X86EMUL_UNHANDLEABLE; } - if ( rc != X86EMUL_OKAY ) - break; } else data = p->data;
x86/HVM: remove dead code With commit 96ae556569 ("x86/HVM: fix forwarding of internally cached requests") rc doesn't change anymore in the respective preceding switch() statements. Coverity ID: 1358080 Signed-off-by: Jan Beulich <jbeul...@suse.com> --- a/xen/arch/x86/hvm/intercept.c +++ b/xen/arch/x86/hvm/intercept.c @@ -151,8 +151,6 @@ int hvm_process_io_intercept(const struc domain_crash(current->domain); return X86EMUL_UNHANDLEABLE; } - if ( rc != X86EMUL_OKAY ) - break; } else p->data = data; @@ -181,8 +181,6 @@ int hvm_process_io_intercept(const struc domain_crash(current->domain); return X86EMUL_UNHANDLEABLE; } - if ( rc != X86EMUL_OKAY ) - break; } else data = p->data;
_______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel