Re: [PATCH/RFC 14/21] KVM: s390: clear the pfault queue if user space sets the invalid token
Am 17.01.2015 um 11:50 schrieb Heiko Carstens: [...] >> +/* for KVM_SYNC_PFAULT and KVM_REG_S390_PFTOKEN */ >> +#define KVM_S390_PFAULT_TOKEN_INVALID 0xUL > > uapi is for both 64 bit and (theoretically) also 32 bit. So this > should be 0xfff...ffULL instead of UL to prevent a different compat > ABI. Thanks. Will fix up. Christian -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH/RFC 14/21] KVM: s390: clear the pfault queue if user space sets the invalid token
On Thu, Jan 15, 2015 at 02:43:27PM +0100, Christian Borntraeger wrote: > From: David Hildenbrand > > We need a way to clear the async pfault queue from user space (e.g. > for resets and SIGP SET ARCHITECTURE). > > This patch simply clears the queue as soon as user space sets the > invalid pfault token. The definition of the invalid token is moved > to uapi. > > Signed-off-by: David Hildenbrand > Acked-by: Cornelia Huck > Signed-off-by: Christian Borntraeger > --- > arch/s390/include/asm/kvm_host.h | 1 - > arch/s390/include/uapi/asm/kvm.h | 3 +++ > arch/s390/kvm/kvm-s390.c | 4 > 3 files changed, 7 insertions(+), 1 deletion(-) > > diff --git a/arch/s390/include/asm/kvm_host.h > b/arch/s390/include/asm/kvm_host.h > index 4de479e..b617052 100644 > --- a/arch/s390/include/asm/kvm_host.h > +++ b/arch/s390/include/asm/kvm_host.h > @@ -469,7 +469,6 @@ struct kvm_vcpu_arch { > }; > struct gmap *gmap; > struct kvm_guestdbg_info_arch guestdbg; > -#define KVM_S390_PFAULT_TOKEN_INVALID(-1UL) > unsigned long pfault_token; > unsigned long pfault_select; > unsigned long pfault_compare; > diff --git a/arch/s390/include/uapi/asm/kvm.h > b/arch/s390/include/uapi/asm/kvm.h > index 9c01159..ac3000d 100644 > --- a/arch/s390/include/uapi/asm/kvm.h > +++ b/arch/s390/include/uapi/asm/kvm.h > @@ -108,6 +108,9 @@ struct kvm_guest_debug_arch { > struct kvm_hw_breakpoint __user *hw_bp; > }; > > +/* for KVM_SYNC_PFAULT and KVM_REG_S390_PFTOKEN */ > +#define KVM_S390_PFAULT_TOKEN_INVALID0xUL uapi is for both 64 bit and (theoretically) also 32 bit. So this should be 0xfff...ffULL instead of UL to prevent a different compat ABI. -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH/RFC 14/21] KVM: s390: clear the pfault queue if user space sets the invalid token
From: David Hildenbrand We need a way to clear the async pfault queue from user space (e.g. for resets and SIGP SET ARCHITECTURE). This patch simply clears the queue as soon as user space sets the invalid pfault token. The definition of the invalid token is moved to uapi. Signed-off-by: David Hildenbrand Acked-by: Cornelia Huck Signed-off-by: Christian Borntraeger --- arch/s390/include/asm/kvm_host.h | 1 - arch/s390/include/uapi/asm/kvm.h | 3 +++ arch/s390/kvm/kvm-s390.c | 4 3 files changed, 7 insertions(+), 1 deletion(-) diff --git a/arch/s390/include/asm/kvm_host.h b/arch/s390/include/asm/kvm_host.h index 4de479e..b617052 100644 --- a/arch/s390/include/asm/kvm_host.h +++ b/arch/s390/include/asm/kvm_host.h @@ -469,7 +469,6 @@ struct kvm_vcpu_arch { }; struct gmap *gmap; struct kvm_guestdbg_info_arch guestdbg; -#define KVM_S390_PFAULT_TOKEN_INVALID (-1UL) unsigned long pfault_token; unsigned long pfault_select; unsigned long pfault_compare; diff --git a/arch/s390/include/uapi/asm/kvm.h b/arch/s390/include/uapi/asm/kvm.h index 9c01159..ac3000d 100644 --- a/arch/s390/include/uapi/asm/kvm.h +++ b/arch/s390/include/uapi/asm/kvm.h @@ -108,6 +108,9 @@ struct kvm_guest_debug_arch { struct kvm_hw_breakpoint __user *hw_bp; }; +/* for KVM_SYNC_PFAULT and KVM_REG_S390_PFTOKEN */ +#define KVM_S390_PFAULT_TOKEN_INVALID 0xUL + #define KVM_SYNC_PREFIX (1UL << 0) #define KVM_SYNC_GPRS (1UL << 1) #define KVM_SYNC_ACRS (1UL << 2) diff --git a/arch/s390/kvm/kvm-s390.c b/arch/s390/kvm/kvm-s390.c index 9bdc9f9..bfb2b99 100644 --- a/arch/s390/kvm/kvm-s390.c +++ b/arch/s390/kvm/kvm-s390.c @@ -936,6 +936,8 @@ static int kvm_arch_vcpu_ioctl_set_one_reg(struct kvm_vcpu *vcpu, case KVM_REG_S390_PFTOKEN: r = get_user(vcpu->arch.pfault_token, (u64 __user *)reg->addr); + if (vcpu->arch.pfault_token == KVM_S390_PFAULT_TOKEN_INVALID) + kvm_clear_async_pf_completion_queue(vcpu); break; case KVM_REG_S390_PFCOMPARE: r = get_user(vcpu->arch.pfault_compare, @@ -1408,6 +1410,8 @@ static void sync_regs(struct kvm_vcpu *vcpu, struct kvm_run *kvm_run) vcpu->arch.pfault_token = kvm_run->s.regs.pft; vcpu->arch.pfault_select = kvm_run->s.regs.pfs; vcpu->arch.pfault_compare = kvm_run->s.regs.pfc; + if (vcpu->arch.pfault_token == KVM_S390_PFAULT_TOKEN_INVALID) + kvm_clear_async_pf_completion_queue(vcpu); } kvm_run->kvm_dirty_regs = 0; } -- 1.9.3 -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html