From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [RFC PATCH 17/17] KVM: PPC: Add an ioctl for userspace to select which platform to emulate Date: Mon, 04 Jul 2011 14:37:53 +0300 Message-ID: <4E11A611.3010904@redhat.com> References: <20110629101552.GA25406@bloggs.ozlabs.ibm.com> <20110629104103.GR25406@bloggs.ozlabs.ibm.com> <4E0C9077.2060608@suse.de> <4E0C9339.2080601@redhat.com> <4E0C94A0.3090301@suse.de> <4E0C9D98.5000904@redhat.com> <2EDDAC02-4BDC-4007-82E1-2C6D732C24D5@suse.de> <4E102516.1040805@redhat.com> <70A08140-B592-4B2F-985B-D8E5C78C743B@suse.de> <4E102ECE.1060004@redhat.com> <4E1030E1.6070101@redhat.com> <77E6DEE3-F00A-46EA-B8AF-58BC62A8143D@suse.de> <4E103270.4040506@redhat.com> <40F62A22-6A47-4414-8FF9-0534568353DA@suse.de> <4E11A293.2010505@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: Paul Mackerras , "linuxppc-dev@ozlabs.org" , "kvm@vger.kernel.org" , "kvm-ppc@vger.kernel.org" , Scott Wood To: Alexander Graf Return-path: In-Reply-To: Sender: kvm-ppc-owner@vger.kernel.org List-Id: kvm.vger.kernel.org On 07/04/2011 02:36 PM, Alexander Graf wrote: > > > > What intermediate steps? We can't add fields to the structure before we get the extensibility infrastructure, but that's all. > > If we add it now without extensibility code, we will have a kernel that only knows the ioctl with the size as it is now. If we later add another field, the ioctl size changes which would render new user space running on that particular kernel to fail, because it doesn't have the "just truncate the ioctl param to the size" logic yet. You're right. So let's add the padding now, and only use the great new stuff after it's actually in. Working ugly is better than broken pretty. -- error compiling committee.c: too many arguments to function