From mboxrd@z Thu Jan 1 00:00:00 1970 From: John Levon Subject: Re: [PATCH] shared_info size and padding fixes Date: Thu, 10 Aug 2006 13:13:48 +0100 Message-ID: <20060810121348.GA24928@totally.trollied.org> References: <20060809210528.GA31314@totally.trollied.org> <44DB0B5D.76E4.0078.0@novell.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <44DB0B5D.76E4.0078.0@novell.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Jan Beulich Cc: xen-devel@lists.xensource.com List-Id: xen-devel@lists.xenproject.org On Thu, Aug 10, 2006 at 10:33:01AM +0200, Jan Beulich wrote: > >sizeof(shared_info_t) can change even in compatible hypervisor > versions. > >Add a clear warning about this, and fix the few places relying on it. > >In addition, add some padding to arch_shared_info, to allow future > >additions to the structure without breaking known offsets of members > >following it in the shared info. > > Shouldn't you also bump the ABI version with these compatibility > breaking > changes to the shared info structure? No, see the previous discussion with Keir - changing the size of the shared info structure is apparently not part of the ABI (hence the reason for the patch and the explanation above!) Personally this seems unusual at best to me, but that's the way it is, and furthermore has already happened with "nmi_reason". john