From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paolo Bonzini Subject: Re: Clang/LLVM version requirements Date: Thu, 13 Sep 2012 16:13:37 +0200 Message-ID: <5051EA11.4080907@redhat.com> References: <20120907085036.GA71093@ocelot.phlegethon.org> <5049E2C70200007800099B21@nat28.tlf.novell.com> <20120913101120.GA12881@ocelot.phlegethon.org> <5051E531020000780009B083@nat28.tlf.novell.com> <5051D440.4000007@redhat.com> <505203F0020000780009B12B@nat28.tlf.novell.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <505203F0020000780009B12B@nat28.tlf.novell.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Jan Beulich Cc: Tim Deegan , Jeffrey Karrels , xen-devel@lists.xen.org List-Id: xen-devel@lists.xenproject.org Il 13/09/2012 16:04, Jan Beulich ha scritto: >>> >> So the patch is fine by me if it covers that misalignment case. >>> >> But it seems a little heavy handed - I'd think that instead of the >>> >> sub-section, we could just create an arbitrary other section, or >>> >> even allow uninitialized variable (it's unclear to me why Paolo >>> >> wrote the comment - in c/s 25479:61dfb3da56b - regarding BSS >>> >> the way it is now) - after all we only need to make sure that >>> >> - the space gets properly allocated in trampoline.S, i.e. also in >>> >> reloc.bin >>> >> - all accesses are PC-relative >>> >> Neither has anything to do with use of uninitialized variables. >> > >> > We cannot use BSS because it doesn't appear in reloc.S. > That, imo, would be a binutils bug. It's just because the flags say so. Tim's flag magic should do it (my reply crossed with his). Paolo