From mboxrd@z Thu Jan 1 00:00:00 1970 X-Msuck: nntp://news.gmane.org/gmane.linux.lib.musl.general/11626 Path: news.gmane.org!.POSTED!not-for-mail From: Rich Felker Newsgroups: gmane.linux.lib.musl.general Subject: Re: [PATCH] powerpc64le: Add single instruction math functions Date: Thu, 29 Jun 2017 13:59:26 -0400 Message-ID: <20170629175926.GF1627@brightrain.aerifal.cx> References: <594DB66E.7030009@adelielinux.org> <594DDEC6.8030200@adelielinux.org> <594EFC63.3000707@adelielinux.org> <20170625001024.GA1627@brightrain.aerifal.cx> <20170629160544.GE1627@brightrain.aerifal.cx> Reply-To: musl@lists.openwall.com NNTP-Posting-Host: blaine.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-Trace: blaine.gmane.org 1498759760 17623 195.159.176.226 (29 Jun 2017 18:09:20 GMT) X-Complaints-To: usenet@blaine.gmane.org NNTP-Posting-Date: Thu, 29 Jun 2017 18:09:20 +0000 (UTC) User-Agent: Mutt/1.5.21 (2010-09-15) To: musl@lists.openwall.com Original-X-From: musl-return-11639-gllmg-musl=m.gmane.org@lists.openwall.com Thu Jun 29 20:09:16 2017 Return-path: Envelope-to: gllmg-musl@m.gmane.org Original-Received: from mother.openwall.net ([195.42.179.200]) by blaine.gmane.org with smtp (Exim 4.84_2) (envelope-from ) id 1dQdsR-0004Ke-Dw for gllmg-musl@m.gmane.org; Thu, 29 Jun 2017 20:09:15 +0200 Original-Received: (qmail 13363 invoked by uid 550); 29 Jun 2017 18:09:18 -0000 Mailing-List: contact musl-help@lists.openwall.com; run by ezmlm Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-ID: Original-Received: (qmail 13340 invoked from network); 29 Jun 2017 18:09:17 -0000 Content-Disposition: inline In-Reply-To: Original-Sender: Rich Felker Xref: news.gmane.org gmane.linux.lib.musl.general:11626 Archived-At: On Thu, Jun 29, 2017 at 01:00:51PM -0400, David Edelsohn wrote: > >> A colleague of mine reminded me that ELFv2 ABI specifies POWER8 as the > >> minimum hardware (not little-endian). > > > > This is a gratuitous requirement and has nothing to do with the > > meaning of ELFv2 we're using (and likewise not with the gcc > > --with-abi=elfv2). > > 2.1.1. Processor Architecture > > This ABI is predicated on, at a minimum, Power ISA version 2.7 and > contains additional implementation characteristics. Yes, I understand that it's there but this "requirement" is orthogonal to the actual interface boundaries the ABI defines. ARM's EABI has a similar gratuitous baseline of v4t; the compiler can satisfy all the interface boundary requirements even on v4 and probably lower if it wants to, but nobody has implemented that. > >> The implementation of ELFv2 can > >> operate on earlier hardware, but binaries may not be forward > >> compatible because of VSX. Because of the calling convention of VSX > >> registers in ELFv2, the stack may be corrupted if an application built > >> without VSX support is linked with a library that does support VSX. > >> One cannot mix and match musl libc built for POWER4 or PPC970 and musl > >> libc built for POWER7. > > > > I don't think this is accurate. If it is then it's a serious bug we > > need to fix, and it should have been discussed at the time the port > > was added... > > This is not an implementation detail in the library, it is the calling > convention in the compilers. > > > > > Can you provide a citation for the usage of VSX registers in the > > calling convention, and how you think that affects the stack? > > Table 2.22 Vector Register Roles in Section 2.2.1.1 Register Roles. > The definition of volatile and non-volatile registers for vector > registers affects the amount of stack allocated and the saving of > non-volatile registers. Are you saying the caller has to allocate space that the callee might use to preseve call-saved registers it wants to use, and that the amount saved depends on ISA level? If so I'll look for the associated logic in GCC and see what it's doing. If that's the case it should be reserving space regardless of whether it's built for an ISA level with the registers or not. > What is the status of the PPC64LE math optimization patch? It's merged, I just forgot to push. So is the s390x one. Pushing now. Rich