This is the mail archive of the mailing list for the glibc project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH v2 03/15] RISC-V: Startup and Dynamic Loading Code

On Fri, 22 Dec 2017, Palmer Dabbelt wrote:

> On Wed, 20 Dec 2017 08:38:48 PST (-0800), wrote:
> > On Tue, 19 Dec 2017, Palmer Dabbelt wrote:
> > 
> > > +extern ElfW(Addr) la_riscv_gnu_pltenter (ElfW(Sym) *__sym, unsigned int
> > > __ndx,
> > 
> > With reference to my previous comments, ElfW is an exception to the rule
> > of spaces before '(' in macro and function calls, because it locally
> > expands to a single identifier.  So this code is correct not to have a
> > space there.
> > 
> > > +  extern ElfW(Addr) _GLOBAL_OFFSET_TABLE_
> > > __attribute__((visibility("hidden")));
> Ah, luckily I missed those when going through you last mail :)

I advise reading 
<>, which discusses 
exceptions such as ElfW and GLRO.

> I haven't tried it, so I assume it does not work.  It looks like the static
> PIE section's header is broken on that site, I'm seeing
>    == Static PIE ===
> in normal text.  I assume it's supposed to look more like "=== Static PIE ==="
> to make the text large?

If you don't already have wiki write access you should create an account 
and ask on libc-alpha for someone to add it to 
<>.  Then you can fix typos 
yourself, as well as adding RISC-V entries to wiki pages where 

> Based on looking through the requirement list, there's enough things there in
> toolchain land that "should just work" that I'm sure something will end up
> being broken.  I assume it's OK to just add the various RISC-V targets to the
> unsupported list for now?

Yes (once in glibc).  RISC-V entries will also be needed on the list of 
ABIs and dynamic linker names at 
<> - again, only once actually in 

Apart from those wiki pages mentioned just now, every release cycle, 
during the one-month release freeze, architecture maintainers should add 
test results to the per-release wiki page (see 
<> for the last release as 
an example, <> for the next 
one, <> for the template 
for future releases which RISC-V will need adding to once in glibc).

Joseph S. Myers

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]