Re: [PATCH] x86/kvm/nVMX: avoid redundant double assignment of nested_run_pending

2018-08-24 Thread Krish Sadhukhan
On 08/23/2018 09:24 AM, Vitaly Kuznetsov wrote: nested_run_pending is set 20 lines above and check_vmentry_prereqs()/ check_vmentry_postreqs() don't seem to be resetting it (the later, however, checks it). Signed-off-by: Vitaly Kuznetsov --- arch/x86/kvm/vmx.c | 3 --- 1 file changed, 3

Re: [PATCH] x86/kvm/nVMX: avoid redundant double assignment of nested_run_pending

2018-08-24 Thread Krish Sadhukhan
On 08/23/2018 09:24 AM, Vitaly Kuznetsov wrote: nested_run_pending is set 20 lines above and check_vmentry_prereqs()/ check_vmentry_postreqs() don't seem to be resetting it (the later, however, checks it). Signed-off-by: Vitaly Kuznetsov --- arch/x86/kvm/vmx.c | 3 --- 1 file changed, 3

Re: [PATCH] x86/kvm/nVMX: avoid redundant double assignment of nested_run_pending

2018-08-23 Thread Jim Mattson
On Thu, Aug 23, 2018 at 9:26 AM, Paolo Bonzini wrote: > On 23/08/2018 18:24, Vitaly Kuznetsov wrote: >> nested_run_pending is set 20 lines above and check_vmentry_prereqs()/ >> check_vmentry_postreqs() don't seem to be resetting it (the later, however, >> checks it). >> >> Signed-off-by: Vitaly

Re: [PATCH] x86/kvm/nVMX: avoid redundant double assignment of nested_run_pending

2018-08-23 Thread Jim Mattson
On Thu, Aug 23, 2018 at 9:26 AM, Paolo Bonzini wrote: > On 23/08/2018 18:24, Vitaly Kuznetsov wrote: >> nested_run_pending is set 20 lines above and check_vmentry_prereqs()/ >> check_vmentry_postreqs() don't seem to be resetting it (the later, however, >> checks it). >> >> Signed-off-by: Vitaly

Re: [PATCH] x86/kvm/nVMX: avoid redundant double assignment of nested_run_pending

2018-08-23 Thread Eduardo Valentin
On Thu, Aug 23, 2018 at 06:24:24PM +0200, Vitaly Kuznetsov wrote: > nested_run_pending is set 20 lines above and check_vmentry_prereqs()/ > check_vmentry_postreqs() don't seem to be resetting it (the later, however, > checks it). > Reviewed-by: Eduardo Valentin > Signed-off-by: Vitaly

Re: [PATCH] x86/kvm/nVMX: avoid redundant double assignment of nested_run_pending

2018-08-23 Thread Eduardo Valentin
On Thu, Aug 23, 2018 at 06:24:24PM +0200, Vitaly Kuznetsov wrote: > nested_run_pending is set 20 lines above and check_vmentry_prereqs()/ > check_vmentry_postreqs() don't seem to be resetting it (the later, however, > checks it). > Reviewed-by: Eduardo Valentin > Signed-off-by: Vitaly

Re: [PATCH] x86/kvm/nVMX: avoid redundant double assignment of nested_run_pending

2018-08-23 Thread Paolo Bonzini
On 23/08/2018 18:24, Vitaly Kuznetsov wrote: > nested_run_pending is set 20 lines above and check_vmentry_prereqs()/ > check_vmentry_postreqs() don't seem to be resetting it (the later, however, > checks it). > > Signed-off-by: Vitaly Kuznetsov > --- > arch/x86/kvm/vmx.c | 3 --- > 1 file

Re: [PATCH] x86/kvm/nVMX: avoid redundant double assignment of nested_run_pending

2018-08-23 Thread Paolo Bonzini
On 23/08/2018 18:24, Vitaly Kuznetsov wrote: > nested_run_pending is set 20 lines above and check_vmentry_prereqs()/ > check_vmentry_postreqs() don't seem to be resetting it (the later, however, > checks it). > > Signed-off-by: Vitaly Kuznetsov > --- > arch/x86/kvm/vmx.c | 3 --- > 1 file

[PATCH] x86/kvm/nVMX: avoid redundant double assignment of nested_run_pending

2018-08-23 Thread Vitaly Kuznetsov
nested_run_pending is set 20 lines above and check_vmentry_prereqs()/ check_vmentry_postreqs() don't seem to be resetting it (the later, however, checks it). Signed-off-by: Vitaly Kuznetsov --- arch/x86/kvm/vmx.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/arch/x86/kvm/vmx.c

[PATCH] x86/kvm/nVMX: avoid redundant double assignment of nested_run_pending

2018-08-23 Thread Vitaly Kuznetsov
nested_run_pending is set 20 lines above and check_vmentry_prereqs()/ check_vmentry_postreqs() don't seem to be resetting it (the later, however, checks it). Signed-off-by: Vitaly Kuznetsov --- arch/x86/kvm/vmx.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/arch/x86/kvm/vmx.c