From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 122819 invoked by alias); 8 May 2015 14:40:38 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Received: (qmail 122809 invoked by uid 89); 8 May 2015 14:40:37 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-0.3 required=5.0 tests=AWL,BAYES_00,KAM_LAZY_DOMAIN_SECURITY,RDNS_DYNAMIC,TVD_RCVD_IP autolearn=no version=3.3.2 X-HELO: brightrain.aerifal.cx Received: from 216-12-86-13.cv.mvl.ntelos.net (HELO brightrain.aerifal.cx) (216.12.86.13) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Fri, 08 May 2015 14:40:36 +0000 Received: from dalias by brightrain.aerifal.cx with local (Exim 3.15 #2) id 1YqjRq-0002Ox-00; Fri, 08 May 2015 14:40:18 +0000 Date: Fri, 08 May 2015 14:40:00 -0000 From: Rich Felker To: Matthew Fortune Cc: "H.J. Lu" , Szabolcs Nagy , Gregor Richards , Szabolcs Nagy , "gcc-patches@gcc.gnu.org" Subject: Re: [PATCH 6/13] mips musl support Message-ID: <20150508144018.GY17573@brightrain.aerifal.cx> References: <55354A4A.4060702@arm.com> <55354B79.3050201@arm.com> <6D39441BF12EF246A7ABCE6654B023532103AB78@LEMAIL01.le.imgtec.org> <20150421144240.GU6817@brightrain.aerifal.cx> <6D39441BF12EF246A7ABCE6654B023532103AECA@LEMAIL01.le.imgtec.org> <553E4A45.1070107@arm.com> <6D39441BF12EF246A7ABCE6654B023532104FBE2@LEMAIL01.le.imgtec.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <6D39441BF12EF246A7ABCE6654B023532104FBE2@LEMAIL01.le.imgtec.org> User-Agent: Mutt/1.5.21 (2010-09-15) X-SW-Source: 2015-05/txt/msg00674.txt.bz2 On Fri, May 08, 2015 at 02:25:11PM +0000, Matthew Fortune wrote: > H.J. Lu writes: > > On Mon, Apr 27, 2015 at 7:40 AM, Szabolcs Nagy > > wrote: > > > > > > > > > On 21/04/15 15:59, Matthew Fortune wrote: > > >> Rich Felker writes: > > >>> On Tue, Apr 21, 2015 at 01:58:02PM +0000, Matthew Fortune wrote: > > >>>> There does however appear to be both soft and hard float variants > > > > > > Patch v2. > > > > > > Now all the ABI variants musl plans to support are represented. > > > > > > gcc/Changelog: > > > > > > 2015-04-27 Gregor Richards > > > Szabolcs Nagy > > > > > > * config/mips/linux.h (MUSL_DYNAMIC_LINKER32): Define. > > > (MUSL_DYNAMIC_LINKER64, MUSL_DYNAMIC_LINKERN32): Define. > > > (GNU_USER_DYNAMIC_LINKERN32): Update. > > > > You checked in config/linux.h CHOOSE_DYNAMIC_LINKER change without > > config/mips/linux.h change. Now linux-mips is broken. > > The MIPS patch is OK. I am concerned that you are aiming for one > dynamic linker per ABI variant in musl but are not accounting for > soft-float up front in n32/n64. There is time to reconsider this > before any of this code gets to a versioned GCC release though. I'm not aware of whether there are mips64 chips for which softfloat would be desirable, so I don't know if it's an ABI we'll ever have, but I'm not opposed to adding it here just to be safe (in case we need it). > I.e. as it stands this patch is not OK for backporting to GCC 5 > without further discussion. > > There is also the perspective that we should be able to aim for > an ABI variant agnostic dynamic linker at some point over the next > year by working towards a build that truly uses no float and is > hence compatible with all the ABI variants. For musl that's not going to happen. The dynamic linker and shared libc are one file, which therefore has lots of public interfaces that depend on the argument passing ABI. Rich