Message ID | 54539D07.7070107@arm.com |
---|---|
State | Committed |
Headers |
Received: (qmail 17333 invoked by alias); 31 Oct 2014 14:30:37 -0000 Mailing-List: contact libc-alpha-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: <libc-alpha.sourceware.org> List-Unsubscribe: <mailto:libc-alpha-unsubscribe-##L=##H@sourceware.org> List-Subscribe: <mailto:libc-alpha-subscribe@sourceware.org> List-Archive: <http://sourceware.org/ml/libc-alpha/> List-Post: <mailto:libc-alpha@sourceware.org> List-Help: <mailto:libc-alpha-help@sourceware.org>, <http://sourceware.org/ml/#faqs> Sender: libc-alpha-owner@sourceware.org Delivered-To: mailing list libc-alpha@sourceware.org Received: (qmail 17315 invoked by uid 89); 31 Oct 2014 14:30:36 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.8 required=5.0 tests=AWL, BAYES_00, SPF_PASS autolearn=ham version=3.3.2 X-HELO: service87.mimecast.com Message-ID: <54539D07.7070107@arm.com> Date: Fri, 31 Oct 2014 14:30:31 +0000 From: Renlin Li <renlin.li@arm.com> User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.1.2 MIME-Version: 1.0 To: libc-alpha@sourceware.org CC: marcus Shawcroft <Marcus.Shawcroft@arm.com> Subject: [PATCH][AARCH64]End frame record chain correctly X-MC-Unique: 114103114303300601 Content-Type: multipart/mixed; boundary="------------080400000400070908010504" |
Commit Message
Renlin Li
Oct. 31, 2014, 2:30 p.m. UTC
Hi all, This is a simple patch to make the end of frame record chain correctly according to AArch64 AAPCS. Currently, the FP(x29) is overwritten by SP in the very beginning of _start. aarch64-none-linux-gnueabi target has been tested on the model. No new issues. Is this Okay to commit? From the Procedure Call Standard for ARM 64-bit Architecture documentation: "Conforming code shall construct a linked list of stack-frames. Each frame shall link to the frame of its caller by means of a frame record of two 64-bit values on the stack. The frame record for the innermost frame (belonging to the most recent routine invocation) shall be pointed to by the Frame Pointer register (FP). The lowest addressed double-word shall point to the previous frame record and the highest addressed double-word shall contain the value passed in LR on entry to the current function. The end of the frame record chain is indicated by the address zero in the address for the previous frame." ChangeLog: 2014-10-31 Renlin Li <Renlin.Li@arm.com> * sysdeps/aarch64/start.S: Delete x29 overwritten assignment.
Comments
On 31 October 2014 14:30, Renlin Li <renlin.li@arm.com> wrote: > Hi all, > > This is a simple patch to make the end of frame record chain correctly > according to AArch64 AAPCS. > Currently, the FP(x29) is overwritten by SP in the very beginning of _start. > > aarch64-none-linux-gnueabi target has been tested on the model. No new > issues. Surely aarch64-none-linux-gnu ;-) The bugzilla # for this is 17555: > ChangeLog: > > 2014-10-31 Renlin Li <Renlin.Li@arm.com> > > * sysdeps/aarch64/start.S: Delete x29 overwritten assignment. Don't forget the () part of the ChangeLog entry. Add a reference to the bugzilla ticket: [BZ #17555] * sysdeps/aarch64/start.S (_start): Delete x29 overwritten assignment. Please also add 17555 to the 2.21 section of the NEWS file. Can you back port this to 2.20 and 2.19 please. Thanks /Marcus
Hi all, can anybody help to review this? Regards, Renlin Li On 31/10/14 14:30, Renlin Li wrote: > Hi all, > > This is a simple patch to make the end of frame record chain correctly > according to AArch64 AAPCS. > Currently, the FP(x29) is overwritten by SP in the very beginning of > _start. > > aarch64-none-linux-gnueabi target has been tested on the model. No new > issues. > > Is this Okay to commit? > > > From the Procedure Call Standard for ARM 64-bit Architecture > documentation: > "Conforming code shall construct a linked list of stack-frames. Each > frame shall link to the frame of its caller by means of a frame record > of two 64-bit values on the stack. The frame record for the innermost > frame (belonging to the most recent routine invocation) shall be > pointed to by the Frame Pointer register (FP). The lowest addressed > double-word shall point to the previous frame record and the highest > addressed double-word shall contain the value passed in LR on entry to > the current function. The end of the frame record chain is indicated > by the address zero in the address for the previous frame." > > > > ChangeLog: > > 2014-10-31 Renlin Li <Renlin.Li@arm.com> > > * sysdeps/aarch64/start.S: Delete x29 overwritten assignment.
On 11/06/2014 09:51 AM, Renlin Li wrote:
> can anybody help to review this?
Please avoid top posting.
Marcus seems to have reviewed it yesterday.
You need to repost your patch with the changes requested
and ask the AArch64 machine maintainer to check it in for
you if you don't have commit access.
Marcus also asked you to provide 2.19 and 2.20 patches.
Cheers,
Carlos.
diff --git a/sysdeps/aarch64/start.S b/sysdeps/aarch64/start.S index 35d603a..69b45ea 100644 --- a/sysdeps/aarch64/start.S +++ b/sysdeps/aarch64/start.S @@ -47,7 +47,6 @@ _start: /* Create an initial frame with 0 LR and FP */ mov x29, #0 mov x30, #0 - mov x29, sp /* Setup rtld_fini in argument register */ mov x5, x0