On Sat, Jul 16, 2016 at 12:48 PM, Bandan Das <bsd@xxxxxxxxxx> wrote: > > This doesn't seem to be ambiguous in the context of setup_shadow_vmcs since > it has one error path and it always returns 0 on success. Just a nit to get rid of > that local variable... I think it's slightly better to keep the -ENOMEM close to the allocation failure, but I don't feel strongly either way. -- 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