From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933333AbdBPT0q (ORCPT ); Thu, 16 Feb 2017 14:26:46 -0500 Received: from mx1.redhat.com ([209.132.183.28]:58926 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932699AbdBPT0n (ORCPT ); Thu, 16 Feb 2017 14:26:43 -0500 Subject: Re: [PATCH] KVM: race-free exit from KVM_RUN without POSIX signals To: Paolo Bonzini , linux-kernel@vger.kernel.org, kvm@vger.kernel.org References: <1487169821-14806-1-git-send-email-pbonzini@redhat.com> Cc: rkrcmar@redhat.com, christoffer.dall@linaro.org, marc.zyngier@arm.com, james.hogan@imgtec.com, paulus@samba.org, borntraeger@de.ibm.com, cornelia.huck@de.ibm.com, kvmarm@lists.cs.columbia.edu, kvm-ppc@vger.kernel.org From: David Hildenbrand Organization: Red Hat GmbH Message-ID: <8a0f0818-07eb-c33e-9488-5a69c6bb67b1@redhat.com> Date: Thu, 16 Feb 2017 20:26:38 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.6.0 MIME-Version: 1.0 In-Reply-To: <1487169821-14806-1-git-send-email-pbonzini@redhat.com> Content-Type: text/plain; charset=iso-8859-15 Content-Transfer-Encoding: 7bit X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.29]); Thu, 16 Feb 2017 19:26:43 +0000 (UTC) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org > post_kvm_run_save(vcpu); > diff --git a/include/uapi/linux/kvm.h b/include/uapi/linux/kvm.h > index 7964b970b9ad..f51d5082a377 100644 > --- a/include/uapi/linux/kvm.h > +++ b/include/uapi/linux/kvm.h > @@ -218,7 +218,8 @@ struct kvm_hyperv_exit { > struct kvm_run { > /* in */ > __u8 request_interrupt_window; > - __u8 padding1[7]; > + __u8 immediate_exit; As mentioned already on IRC, maybe something like "block_vcpu_run" would fit better now. But this is also ok and looks good to me. Reviewed-by: David Hildenbrand -- Thanks, David From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Hildenbrand Subject: Re: [PATCH] KVM: race-free exit from KVM_RUN without POSIX signals Date: Thu, 16 Feb 2017 20:26:38 +0100 Message-ID: <8a0f0818-07eb-c33e-9488-5a69c6bb67b1@redhat.com> References: <1487169821-14806-1-git-send-email-pbonzini@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: james.hogan@imgtec.com, marc.zyngier@arm.com, kvm-ppc@vger.kernel.org, borntraeger@de.ibm.com, paulus@samba.org, cornelia.huck@de.ibm.com, kvmarm@lists.cs.columbia.edu To: Paolo Bonzini , linux-kernel@vger.kernel.org, kvm@vger.kernel.org Return-path: In-Reply-To: <1487169821-14806-1-git-send-email-pbonzini@redhat.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: kvmarm-bounces@lists.cs.columbia.edu Sender: kvmarm-bounces@lists.cs.columbia.edu List-Id: kvm.vger.kernel.org > post_kvm_run_save(vcpu); > diff --git a/include/uapi/linux/kvm.h b/include/uapi/linux/kvm.h > index 7964b970b9ad..f51d5082a377 100644 > --- a/include/uapi/linux/kvm.h > +++ b/include/uapi/linux/kvm.h > @@ -218,7 +218,8 @@ struct kvm_hyperv_exit { > struct kvm_run { > /* in */ > __u8 request_interrupt_window; > - __u8 padding1[7]; > + __u8 immediate_exit; As mentioned already on IRC, maybe something like "block_vcpu_run" would fit better now. But this is also ok and looks good to me. Reviewed-by: David Hildenbrand -- Thanks, David From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Hildenbrand Date: Thu, 16 Feb 2017 19:26:38 +0000 Subject: Re: [PATCH] KVM: race-free exit from KVM_RUN without POSIX signals Message-Id: <8a0f0818-07eb-c33e-9488-5a69c6bb67b1@redhat.com> List-Id: References: <1487169821-14806-1-git-send-email-pbonzini@redhat.com> In-Reply-To: <1487169821-14806-1-git-send-email-pbonzini@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Paolo Bonzini , linux-kernel@vger.kernel.org, kvm@vger.kernel.org Cc: rkrcmar@redhat.com, christoffer.dall@linaro.org, marc.zyngier@arm.com, james.hogan@imgtec.com, paulus@samba.org, borntraeger@de.ibm.com, cornelia.huck@de.ibm.com, kvmarm@lists.cs.columbia.edu, kvm-ppc@vger.kernel.org > post_kvm_run_save(vcpu); > diff --git a/include/uapi/linux/kvm.h b/include/uapi/linux/kvm.h > index 7964b970b9ad..f51d5082a377 100644 > --- a/include/uapi/linux/kvm.h > +++ b/include/uapi/linux/kvm.h > @@ -218,7 +218,8 @@ struct kvm_hyperv_exit { > struct kvm_run { > /* in */ > __u8 request_interrupt_window; > - __u8 padding1[7]; > + __u8 immediate_exit; As mentioned already on IRC, maybe something like "block_vcpu_run" would fit better now. But this is also ok and looks good to me. Reviewed-by: David Hildenbrand -- Thanks, David