From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ian Campbell Subject: Re: stubdom vtpm build failure in staging Date: Fri, 30 Jan 2015 10:39:56 +0000 Message-ID: <1422614396.12551.5.camel@citrix.com> References: <20150128150850.GA4285@aepfle.de> <1422458693.5187.19.camel@citrix.com> <945CA011AD5F084CBEA3E851C0AB28890E87F6C4@SHSMSX101.ccr.corp.intel.com> <20150129082706.GB1410@aepfle.de> <945CA011AD5F084CBEA3E851C0AB28890E87F775@SHSMSX101.ccr.corp.intel.com> <20150129093649.GA20318@aepfle.de> <20150129102736.GA1106@aepfle.de> <1422528392.30641.24.camel@citrix.com> <20150130080921.GA15882@aepfle.de> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20150130080921.GA15882@aepfle.de> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Olaf Hering Cc: Ian Jackson , "Xu, Quan" , "xen-devel@lists.xen.org" List-Id: xen-devel@lists.xenproject.org On Fri, 2015-01-30 at 09:09 +0100, Olaf Hering wrote: > On Thu, Jan 29, Ian Campbell wrote: > > > This compiles for me on Debian with gcc 4.7.2-5. > > Same here. > > > I'm honestly not sure what the C standard says about duplicate but > > identical typedefs -- Ian? > > How would that matter now? ;-) Old gcc can not deal with it, the commits > cause a regression for gcc-4.5 and older. If it is allowed then strictly speaking a bug in gcc-4.5 and older has caused a regression with C compliant code. In some cases we might then argue that people should fix their compiler, I doubt that will be needed in this case, but it is an option. Anyway, in more practical terms right now this might have an impact on how invasive a fix we would be prepared to accept. In particular if the current code is valid then we might want to make the fix conditional on the gcc version so that we can eventually drop it. If the current code is invalid then the first fix is to make it valid and see if that also helps with the older gcc. Ian.