From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heiko Carstens Subject: Re: [PATCH 02/12] KVM: s390: define GISA format-0 data structure Date: Fri, 19 Jan 2018 11:12:41 +0100 Message-ID: <20180119101241.GB4519@osiris> References: <20180116200217.211897-1-borntraeger@de.ibm.com> <20180116200217.211897-3-borntraeger@de.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: kvm-owner@vger.kernel.org List-Archive: List-Post: To: David Hildenbrand Cc: Christian Borntraeger , Cornelia Huck , KVM , linux-s390 , Janosch Frank , Michael Mueller List-ID: On Thu, Jan 18, 2018 at 09:47:01PM +0100, David Hildenbrand wrote: > Two minor things > > > > > +struct kvm_s390_gisa { > > + u32 next_alert; > > + u8 ipm; > > + u8 reserved01; > > + u8:6; > > Mind giving this also a reserved name And then all reserved fields have to be renamed as soon as one bit gets used? Please don't...