From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from xry111.site (xry111.site [IPv6:2001:470:683e::1]) by sourceware.org (Postfix) with ESMTPS id 39BA93858C5F for ; Sat, 22 Jul 2023 07:18:20 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 39BA93858C5F Authentication-Results: sourceware.org; dmarc=pass (p=reject dis=none) header.from=xry111.site Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=xry111.site DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=xry111.site; s=default; t=1690010296; bh=nHld6BUhiOTGf04JHB5zNKIRcSDF+gimssxznbXRlkA=; h=Subject:From:To:Cc:Date:In-Reply-To:References:From; b=burrDYpWjqyaFHdKDhnWvK9FrE3FqX6+yDFvEaxa4FHx81K5C9/n6LFK1IV7jVgu/ BqRqwHc83AFflo+IWxAmgDZrDvaTGqx4NhSWjyxpZsLum/kohkM4+OXXAqUjUkiX5N Kun8GWeheHxRyLWerqLW43Od2h+f0Hm6QFwfLo1w= Received: from [IPv6:240e:358:117b:1c00:dc73:854d:832e:4] (unknown [IPv6:240e:358:117b:1c00:dc73:854d:832e:4]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (P-256) server-signature ECDSA (P-384) server-digest SHA384) (Client did not present a certificate) (Authenticated sender: xry111@xry111.site) by xry111.site (Postfix) with ESMTPSA id ABA956599C; Sat, 22 Jul 2023 03:18:11 -0400 (EDT) Message-ID: Subject: Re: [PATCH v4 1/2] MIPS: support mips*64 as CPU and gnuabi64 as ABI From: Xi Ruoyao To: YunQiang Su , "Maciej W. Rozycki" , "Andreas K. Huettel" Cc: Richard Sandiford , Nick Clifton , YunQiang Su , binutils@sourceware.org, jiaxun.yang@flygoat.com Date: Sat, 22 Jul 2023 15:18:05 +0800 In-Reply-To: References: <20230414072046.1639896-1-yunqiang.su@cipunited.com> <20230418140019.2195551-1-yunqiang.su@cipunited.com> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable User-Agent: Evolution 3.48.4 MIME-Version: 1.0 X-Spam-Status: No, score=0.2 required=5.0 tests=BAYES_00,BODY_8BITS,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,LIKELY_SPAM_FROM,SPF_HELO_PASS,SPF_PASS,TXREP,T_SCC_BODY_TEXT_LINE autolearn=no autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: On Fri, 2023-07-21 at 23:01 +0800, YunQiang Su wrote: > Debian, as a *REAL* OS, instead of anything you are imagining, is *USING*= it. > I don't agree that you make any "features" heavenly. > If so, we cannot fix any bug. >=20 > More and more software in the *REAL* world, use the CPU section (here mip= sisa64) > to determine the 64bit OS/env. > Anyway, the *REAL* world is much more important the the world only in > your *MIND*. I'm not sure about this debate (I've stopped development of Linux From Scratch on MIPS64 because I don't have a good hardware now and I don't like developing something purely on an emulator). But as a LFS editor: please make an end to this instead of delaying the release indefinitely. The delay will impact our plans for testing and releasing LFS 12.0. I'm pulling Andreas (as Gentoo MIPS maintainer and the RM of the upcoming Glibc release) into the discussion. > Maciej W. Rozycki =E4=BA=8E2023=E5=B9=B47=E6=9C=8821= =E6=97=A5=E5=91=A8=E4=BA=94 22:31=E5=86=99=E9=81=93=EF=BC=9A > >=20 > > On Fri, 21 Jul 2023, YunQiang Su wrote: > >=20 > > > > > > =C2=A0So this has changed the default ABI from o32 to n32 for `= mipsisa64-*-*' > > > > >=20 > > > > > I think that maybe you have a misunderstanding. > > > > > 1. It is *not* for `mipsisa64-*-*', it is for `mipsisa64-*-gnuabi= 64' > > > > > 2. It is not from O32 to N32. It is from N32 to N64. > > > >=20 > > > > =C2=A0Well test results say otherwise.=C2=A0 Please build yourself = binutils for > > > > `mipsisa64-linux' and see what output format it uses with and witho= ut this > > > > change. > > >=20 > > > Ohh you are right, while, does mips*-linux without ABI section really= matter? > > > I never hear about it is used for any real system. > >=20 > > =C2=A0You may not have heard about it, but someone has added these mach= ine > > specifiers for a reason and I reckon seeing at least `mipsisa64-*-*' on= es > > used at one point, maybe at MTI back in ~2005.=C2=A0 Here the "isa64" s= uffix > > only specifies the ISA level (just as with "isa32", etc.) and not the A= BI > > such as with the "64" suffix. > >=20 > > =C2=A0It may have even been before we had NewABI support in the first p= lace, so > > the semantics for `mipsisa64-*-*' had to stay for backwards compatibili= ty > > already back then.=C2=A0 And I do remember the days and the issues arou= nd > > merging NewABI stuff; if MTI weren't so slow with defining it, we could > > have ended up with NUBI instead rather than this old IRIX stuff, not > > necessarily adequate for embedded use and with its sole advantage that = it > > was already well-defined then, so the community took lead and merged > > NewABI support behind MTI's back. > >=20 > > > Ohh, yes, my patch switch the default output of mips*isa64-linux to > > > N32 from O32. > > > I believe that it is an correct modification. > >=20 > > =C2=A0Maybe, but this is changing what we've had for 20+ years, so it m= ust not > > be done without proper consideration, documentation, and likely a NEWS > > entry, as this is a user-visible change of semantics.=C2=A0 You can't j= ust > > decide unilaterally, especially on something that's been out there for > > such a long time. > >=20 > > =C2=A0I think this close to 2.41 release the best course of action may = be > > reverting the change after all on trunk/2.41, and then re-applying the > > uncontroversial part on trunk only.=C2=A0 This will prevent Nick from b= eing > > held with the release and we can decide what to do with the rest.=C2=A0= It > > would unblock me with accepting your outstanding bug fix as well, also = for > > 2.41. > >=20 > > =C2=A0And I wonder if there's more that needs to be undone for 2.41, so= that it > > can be then handled properly with 2.42, especially given the large > > quantity of regressions introduced, which may not necessarily be solely > > due to the lack of update to the testsuite and could be a sign of actua= l > > problems introduced with the tools themselves.=C2=A0 I'd prefer to have > > results with 2.41 that are no worse than as at 32f1c80375eb^ TBH. > >=20 > > =C2=A0 Maciej --=20 Xi Ruoyao School of Aerospace Science and Technology, Xidian University