On Mon, 8 Jan 2018 20:37:47 +0100 David Hildenbrand <david@xxxxxxxxxx> wrote: > "wq" is not used at all. That seems to date back to 66933b78e3204057bfc26343afcd0d463c0e8e55. > "cpuflags" can be access directly via the vcpu, > just as "float_int" via vcpu->kvm. I'm trying to remember whether there were any caching/locality issues, but I don't think so. > > Signed-off-by: David Hildenbrand <david@xxxxxxxxxx> > --- > > v1 -> v2: > - Properly make use of dst_vcpu instead of vcpu in sigp.c > > arch/s390/include/asm/kvm_host.h | 3 --- > arch/s390/kvm/interrupt.c | 25 +++++++++++-------------- > arch/s390/kvm/kvm-s390.c | 3 --- > arch/s390/kvm/kvm-s390.h | 2 +- > arch/s390/kvm/sigp.c | 12 ++++-------- > 5 files changed, 16 insertions(+), 29 deletions(-) > > diff --git a/arch/s390/kvm/interrupt.c b/arch/s390/kvm/interrupt.c > index 818aa4248b0f..f8eb2cfa763a 100644 > --- a/arch/s390/kvm/interrupt.c > +++ b/arch/s390/kvm/interrupt.c > @@ -107,12 +107,11 @@ static int sca_inject_ext_call(struct kvm_vcpu *vcpu, int src_id) > > static void sca_clear_ext_call(struct kvm_vcpu *vcpu) > { > - struct kvm_s390_local_interrupt *li = &vcpu->arch.local_int; > int rc, expect; > > if (!kvm_s390_use_sca_entries()) > return; > - atomic_andnot(CPUSTAT_ECALL_PEND, li->cpuflags); > + atomic_andnot(CPUSTAT_ECALL_PEND, &vcpu->arch.sie_block->cpuflags); We have __set_cpuflag() to set a flag; do we also want __clear_cpuflag()? Would make the code nicer, I think. (There are more possible callers for that.) > read_lock(&vcpu->kvm->arch.sca_lock); > if (vcpu->kvm->arch.use_esca) { > struct esca_block *sca = vcpu->kvm->arch.sca; -- To unsubscribe from this list: send the line "unsubscribe linux-s390" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html