From mboxrd@z Thu Jan 1 00:00:00 1970 From: Michael Mueller Subject: Re: [PATCH v6 07/13] KVM: s390: introduce struct kvm_s390_gisa_interrupt Date: Thu, 24 Jan 2019 16:24:28 +0100 Message-ID: References: <20190124125939.130763-1-mimu@linux.ibm.com> <20190124125939.130763-8-mimu@linux.ibm.com> <20190124160637.0f8b47a7.cohuck@redhat.com> Reply-To: mimu@linux.ibm.com Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20190124160637.0f8b47a7.cohuck@redhat.com> Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org List-Archive: List-Post: To: Cornelia Huck Cc: KVM Mailing List , Linux-S390 Mailing List , linux-kernel@vger.kernel.org, Martin Schwidefsky , Heiko Carstens , Christian Borntraeger , Janosch Frank , David Hildenbrand , Halil Pasic , Pierre Morel List-ID: On 24.01.19 16:06, Cornelia Huck wrote: > On Thu, 24 Jan 2019 13:59:33 +0100 > Michael Mueller wrote: > >> Use this struct analog to the kvm interruption structs >> for kvm emulated floating and local interruptions. > > I guess that makes sense. > >> Further fields will be added with this series as >> required. > > A reference to "this series" will look a bit strange if you look at the > committed patch later. What about: > > "GIB handling will add further fields to this structure as required." Yes, that reads pretty well. I will replace the sentence. > > ? > >> >> Signed-off-by: Michael Mueller >> --- >> arch/s390/include/asm/kvm_host.h | 6 ++++- >> arch/s390/kvm/interrupt.c | 52 +++++++++++++++++++++++----------------- >> arch/s390/kvm/kvm-s390.c | 2 +- >> 3 files changed, 36 insertions(+), 24 deletions(-) > > Reviewed-by: Cornelia Huck >