From: Adhemerval Zanella <adhemerval.zanella@linaro.org>
To: Vineet Gupta <Vineet.Gupta1@synopsys.com>,
"libc-alpha@sourceware.org" <libc-alpha@sourceware.org>
Cc: arcml <linux-snps-arc@lists.infradead.org>
Subject: Re: [PATCH v6 01/13] ARC: ABI Implementation
Date: Fri, 29 May 2020 10:56:17 -0300 [thread overview]
Message-ID: <129265eb-c39a-6394-24ab-8734b04649bb@linaro.org> (raw)
In-Reply-To: <f36112c7-b7d6-243a-73eb-74d5b3772135@synopsys.com>
On 27/05/2020 19:15, Vineet Gupta wrote:
> On 5/27/20 11:26 AM, Adhemerval Zanella via Libc-alpha wrote:
>>
>>
>> On 22/04/2020 22:41, Vineet Gupta via Libc-alpha wrote:
>>> This code deals with the ARC ABI.
>>>
>>> Signed-off-by: Vineet Gupta <vgupta@synopsys.com>
>>
>> We do not use DCO, but rather copyright assignment.
>
> Right, removed that now.
>
>> Looks ok in general, with some comments below.
>
> Thx for taking a look.
>
>>> +;@ r1 = value that setjmp( ) will return due to this longjmp
>>
>> Since all .S files are processed by gcc assembly implementation usually
>> use C style comment (/* ... */). Same applies to other assembly
>> implementations.
>
> OK, I can update throughout, although I like the small assembler comments which
> are on the same line.
I don't have a strong preference and I am not sure if there is a strict
code guideline for comment in assembly implementations. It was more a
suggestion, since other assembly implementations tend to use C style
comment as well.
>>> diff --git a/sysdeps/arc/memusage.h b/sysdeps/arc/memusage.h
>
>>> +
>>> +#define GETSP() ({ register uintptr_t stack_ptr asm ("sp"); stack_ptr; })
>>> +
>>> +#define uatomic32_t unsigned int
>>
>> Not sure if this is really required now that we are moving to C11 atomic
>> model withing glibc itself. Maybe we could just use uint32_t on
>> malloc/memusage.c and rely on atomic macros instead.
>
> But that would be much bigger change, and orthogonal to the port. So perhaps we
> add it for now and then do the bigger/sweeping change.
>
Indeed, it was more a open note for a future cleanup. The current
definition is fine as is.
_______________________________________________
linux-snps-arc mailing list
linux-snps-arc@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-snps-arc
next prev parent reply other threads:[~2020-05-29 13:56 UTC|newest]
Thread overview: 48+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-04-23 1:41 [PATCH v6 00/13] glibc port to ARC processors Vineet Gupta
2020-04-23 1:41 ` [PATCH v6 01/13] ARC: ABI Implementation Vineet Gupta
[not found] ` <88508d10-2d29-026a-bb54-ad607154ab87@linaro.org>
2020-05-27 22:15 ` Vineet Gupta
2020-05-29 13:56 ` Adhemerval Zanella [this message]
[not found] ` <a56a35d4-3e9e-9a88-4be5-8553d5f11ad3@synopsys.com>
[not found] ` <87mu5jxkv7.fsf@oldenburg2.str.redhat.com>
2020-06-04 19:01 ` Vineet Gupta
2020-06-04 23:56 ` Vineet Gupta
2020-04-23 1:41 ` [PATCH v6 02/13] ARC: startup and dynamic linking code Vineet Gupta
[not found] ` <17957ee6-2bc1-f43b-f184-f0703ba2765f@linaro.org>
2020-05-28 1:14 ` Vineet Gupta
2020-04-23 1:41 ` [PATCH v6 03/13] ARC: Thread Local Storage support Vineet Gupta
[not found] ` <4f7a67fb-6f96-57e6-b827-d1ab5dd6733f@linaro.org>
2020-05-28 1:36 ` Vineet Gupta
2020-06-01 18:53 ` Adhemerval Zanella
2020-04-23 1:41 ` [PATCH v6 04/13] ARC: Atomics and Locking primitives Vineet Gupta
[not found] ` <03f4a9b3-b1ca-90fa-0b6a-609a3135267d@linaro.org>
2020-04-24 7:23 ` Vineet Gupta
[not found] ` <20200427215938.14136-1-vgupta@synopsys.com>
2020-04-27 22:13 ` [PATCH] semaphore: consolidate arch headers into a generic one Vineet Gupta
[not found] ` <ac93c301-36d3-b20a-d31c-50c1f3264c68@linaro.org>
2020-05-05 22:59 ` Vineet Gupta
2020-05-08 13:32 ` Adhemerval Zanella
2020-04-23 1:41 ` [PATCH v6 05/13] ARC: math soft float support Vineet Gupta
2020-04-23 1:41 ` [PATCH v6 06/13] ARC: hardware floating point support Vineet Gupta
2020-05-29 14:12 ` Adhemerval Zanella
2020-05-29 22:28 ` Vineet Gupta
2020-05-29 23:50 ` Vineet Gupta
2020-06-02 17:51 ` Joseph Myers
[not found] ` <07887c48-7e07-9f89-035d-3f336a16f2da@synopsys.com>
2020-06-02 18:13 ` static inline math functions (was Re: [PATCH v6 06/13] ARC: hardware floating point support) Joseph Myers
2020-06-02 18:35 ` Adhemerval Zanella
2020-06-05 4:44 ` [PATCH v6 06/13] ARC: hardware floating point support Vineet Gupta
2020-06-05 17:22 ` Adhemerval Zanella
2020-06-02 17:48 ` Joseph Myers
2020-04-23 1:41 ` [PATCH v6 07/13] ARC: Linux Syscall Interface Vineet Gupta
2020-05-29 16:49 ` Adhemerval Zanella
2020-05-30 2:02 ` Vineet Gupta
2020-06-03 19:46 ` Vineet Gupta
2020-06-03 20:04 ` Adhemerval Zanella
2020-06-03 20:17 ` Vineet Gupta
2020-06-04 11:06 ` Adhemerval Zanella
2020-04-23 1:41 ` [PATCH v6 08/13] ARC: Linux ABI Vineet Gupta
2020-04-23 1:41 ` [PATCH v6 09/13] ARC: Linux Startup and Dynamic Loading Vineet Gupta
2020-04-23 1:41 ` [PATCH v6 10/13] ARC: ABI lists Vineet Gupta
2020-04-23 1:41 ` [PATCH v6 11/13] ARC: Build Infrastructure Vineet Gupta
2020-06-03 19:58 ` Adhemerval Zanella
2020-06-04 15:25 ` Vineet Gupta
2020-06-04 17:05 ` Adhemerval Zanella
2020-06-08 4:18 ` Vineet Gupta
2020-04-23 1:41 ` [PATCH v6 12/13] build-many-glibcs.py: Enable ARC builds Vineet Gupta
2020-04-23 1:41 ` [PATCH v6 13/13] Documentation for ARC port Vineet Gupta
2020-05-04 21:21 ` [PATCH v6 00/13] glibc port to ARC processors Vineet Gupta
2020-05-15 0:45 ` Vineet Gupta
2020-05-27 1:49 ` Vineet Gupta
[not found] ` <d7f1176c-87c6-90c6-161c-4705a47837ea@linaro.org>
2020-05-27 18:38 ` Vineet Gupta
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=129265eb-c39a-6394-24ab-8734b04649bb@linaro.org \
--to=adhemerval.zanella@linaro.org \
--cc=Vineet.Gupta1@synopsys.com \
--cc=libc-alpha@sourceware.org \
--cc=linux-snps-arc@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).