On 01/10/2011 07:17 PM, Jan Kiszka wrote:
Am 04.01.2011 14:14, Avi Kivity wrote: > 'goto done' leads to a maze of checks and actions, all pointless. Bail > out immediately if we can't decode. > > Signed-off-by: Avi Kivity<avi@xxxxxxxxxx> > --- > arch/x86/kvm/x86.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c > index b20499d..b085ac3 100644 > --- a/arch/x86/kvm/x86.c > +++ b/arch/x86/kvm/x86.c > @@ -4392,7 +4392,7 @@ int x86_emulate_instruction(struct kvm_vcpu *vcpu, > > r = x86_decode_insn(&vcpu->arch.emulate_ctxt, insn, insn_len); > if (r != EMULATION_OK) > - goto done; > + return EMULATE_FAIL; This leaves 'done' unused behind, and the compiler mourns.
It's already in 'next'. Marcelo can you amend it? -- error compiling committee.c: too many arguments to function -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
