Re: [RFC PATCH 4/7] accel/kvm: Simplify kvm_set_sigmask_len()

2020-07-05 Thread Philippe Mathieu-Daudé
On 7/5/20 9:12 AM, Paolo Bonzini wrote: > On 23/06/20 12:50, Philippe Mathieu-Daudé wrote: >> The sigmask_len is a property of the accelerator, not the VM. >> Simplify by directly using the global kvm_state, remove the >> unnecessary KVMState* argument. > > This is not entirely true, if there were

Re: [RFC PATCH 4/7] accel/kvm: Simplify kvm_set_sigmask_len()

2020-07-05 Thread Paolo Bonzini
On 23/06/20 12:50, Philippe Mathieu-Daudé wrote: > The sigmask_len is a property of the accelerator, not the VM. > Simplify by directly using the global kvm_state, remove the > unnecessary KVMState* argument. This is not entirely true, if there were multiple KVMStates how would you know which one

[RFC PATCH 4/7] accel/kvm: Simplify kvm_set_sigmask_len()

2020-06-23 Thread Philippe Mathieu-Daudé
The sigmask_len is a property of the accelerator, not the VM. Simplify by directly using the global kvm_state, remove the unnecessary KVMState* argument. Signed-off-by: Philippe Mathieu-Daudé --- include/sysemu/kvm.h | 2 +- accel/kvm/kvm-all.c | 4 ++-- target/mips/kvm.c| 2 +- 3 files cha