From: Stafford Horne <shorne@gmail.com>
To: Adhemerval Zanella <adhemerval.zanella@linaro.org>
Cc: GLIBC patches <libc-alpha@sourceware.org>,
Openrisc <openrisc@lists.librecores.org>
Subject: Re: [PATCH v5 00/13] Glibc OpenRISC port - PUSHED
Date: Thu, 6 Jan 2022 06:34:25 +0900 [thread overview]
Message-ID: <YdYO4eW2sjiIYKPg@antec> (raw)
In-Reply-To: <YdS92XAL/U+57Qfx@antec>
Thank you.
On Wed, Jan 05, 2022 at 06:36:25AM +0900, Stafford Horne wrote:
> On Tue, Jan 04, 2022 at 09:34:47AM -0300, Adhemerval Zanella wrote:
> >
> >
> > On 04/01/2022 00:14, Stafford Horne via Libc-alpha wrote:
> > > This is the OpenRISC port for glibc that I have been working on.
> > >
> > > Changes since v4:
> > > - Fix suggestion from Joseph Myers
> > > - Remove libnsl.abilist
> > > - Fixed suggestions from Adhemerval:
> > > - Use Elf32_Addr and Elf32_Sym on link.h and sotruss-lib.c.
> > > - Remove the superflous atomic typedefs on atomic-machine.h.
> > > - Add the _dl_find_object on libc.abilist.
> > > - Update wording for the NEWS entry.
> > > - Other fixes
> > > - Update date to 2022
> > > - Remove uneeded REGSIZE
> > > - Remove trailing newlines in Makefile
> > > - Fix space vs tabs indentation in sysdep.h
> > >
> > > Changes since v3:
> > > - Fix suggestion from Paul Eggert
> > > - Fix typo pore/port
> > > - Fixed suggestions from Adhemerval:
> > > - Used ENTRY/END asm macros where missing
> > > - Remove block in dl-machine.h suggesting LD_PROFILE
> > > - Remove jmp_buf-macros.h macros and use new default.
> > > - Remove tcb-offsets.sym file and Makefile reference.
> > > - Remove tcb-offsets.h include in tls.h.
> > > - Define lib-audit register layout inline with other architectures.
> > > - Fix usage of locally defined register in or1k_get_got. Also, fixup
> > > indentation when using or1k_get_got.
> > > - Cleanup abilists after rebasing, adding rseq and removing
> > > empty files.
> > > - Remove prctl and other unused entries in the linux Makefile.
> > > - Fix STACK_ALIGN to 4 bytes, from incorrect copy-n-paste 16 bytes.
> > > - Use default pthreadtypes-arch.h.
> > > - Mention only soft-float supported in NEWS.
> > > - Remove sysdeps/unix/sysv/linux/or1k/bits/timesize.h and use new
> > > default.
> > >
> > > Changes since v2:
> > > - Fixed suggestions from Joseph Myers:
> > > - Fix comment style, and description on top of each file
> > > - Make sure macros have parentheses when needed,
> > > - Bump required kernel down to 5.4.0 and document
> > > - Regenerate arch-syscall.h
> > > - Fixed suggestions from Adhemerval:
> > > - Remove kernel_stat.h
> > > - Just set MMAP2_PAGE_UNIT to 8K
> > > - Remove ioctl.c and syscall.c files
> > > - Update TCB alignment to 32 bytes
> > >
> > > Changes since v1:
> > > - Update api's as suggested by Florian
> > > - Remove hard float support
> > > - Updates to get all tests passing
> > > - Split patch into managable bits similar to recent ARC port
> > >
> > > Documentation:
> > >
> > > Architecture / ABI docs:
> > > https://raw.githubusercontent.com/openrisc/doc/master/openrisc-arch-1.3-rev1.pdf
> > >
> > > Test Results:
> > >
> > > build-many-glibcs.py:
> > >
> > > PASS with mainline ang gcc-11.
> > >
> > > Full test suite:
> > >
> > > This time I ran the test suite with timeout of 300 and with patch for forcing
> > > O_LARGEFILE in open64 [0]. The test has one failure but it seems like flaky
> > > test as it passed after the second run.
> > This is already approved, so I take these tests were done against and slight order
> > tree (which is fine btw).
>
> Sorry, I forgot to update this bit for v5. The patch for open64 is already
> upstream.
>
> > >
> > > # results in file: /home/shorne/work/gnu-toolchain/log/test--20211226-162029.log
> > > # test start: 2021-12-26T16:20:29+09:00
> > >
> > > # failures
> > > FAIL: posix/tst-execveat /* Running second time passes */
> > >
> > > # test finish: 2021-12-29T12:54:08+09:00
> > > # test duration: 2 days 20 hours 33 minutes and 39 seconds
> > > # test for file: tests.sum
> > > # test wrapper: /home/shorne/work/gnu-toolchain/glibc/scripts/cross-test-ssh.sh --timeoutfactor 300 10.0.0.5
> > >
> > > # summary
> > > 1 FAIL
> > > 4148 PASS
> > > 31 UNSUPPORTED
> > > 16 XFAIL
> > > 2 XPASS
> > >
> > > posix/tst-execveat.out:
> > > Timed out: killed the child process
> > > Termination time: 2021-12-29T01:02:42.433964340
> > > Last write to standard output: 2021-12-28T23:22:40.266363000
> > >
> > > tests.sum: https://gist.github.com/stffrdhrn/88c1fce0c54350258f1f5869068d015a
> > >
> > > [0] https://sourceware.org/pipermail/libc-alpha/2021-December/134634.html
> > >
> >
> > I approved the missing patch and I think the patches are ok to go in. I am
> > curious about the posix/tst-execveat issues, but since it does not really
> > have any arch-specific implementation I think it might be related to the
> > test environment (it would be good to know exactly what has caused the
> > failure so we can improve it).
>
> It fails with what looks like a deadlock but I cannot reproduce it. When it
> fails it times our after several minutes, when I run it again it passes right
> away.
>
> The platform I run on loads binaries over NFS. There may be some kind of race
> condition caused when binaries are loading slowly.
>
> I will wait a day or so for comments then push this upstream. If no objections,
> thanks for all your help reviewing. Note, after pushing up stream I will also
> publish all of the updates to the glibc wiki.
>
> -Stafford
next prev parent reply other threads:[~2022-01-05 21:34 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-01-04 3:14 [PATCH v5 00/13] Glibc OpenRISC port Stafford Horne
2022-01-04 3:14 ` [PATCH v5 01/13] elf: Add reloc for OpenRISC Stafford Horne
2022-01-04 3:14 ` [PATCH v5 02/13] linux/syscalls: Add or1k_atomic syscall " Stafford Horne
2022-01-04 3:14 ` [PATCH v5 03/13] or1k: ABI Implementation Stafford Horne
2022-01-04 3:14 ` [PATCH v5 04/13] or1k: startup and dynamic linking code Stafford Horne
2022-01-04 3:14 ` [PATCH v5 05/13] or1k: Thread Local Storage support Stafford Horne
2022-01-04 3:14 ` [PATCH v5 06/13] or1k: Atomics and Locking primitives Stafford Horne
2022-01-04 3:14 ` [PATCH v5 07/13] or1k: math soft float support Stafford Horne
2022-01-04 3:14 ` [PATCH v5 08/13] or1k: Linux Syscall Interface Stafford Horne
2022-01-04 3:14 ` [PATCH v5 09/13] or1k: Linux ABI Stafford Horne
2022-01-04 3:14 ` [PATCH v5 10/13] or1k: ABI lists Stafford Horne
2022-01-04 12:27 ` Adhemerval Zanella
2022-01-04 21:37 ` Stafford Horne
2022-01-04 3:14 ` [PATCH v5 11/13] or1k: Build Infrastructure Stafford Horne
2022-01-04 3:14 ` [PATCH v5 12/13] build-many-glibcs.py: add OpenRISC support Stafford Horne
2022-01-04 3:14 ` [PATCH v5 13/13] Documentation for OpenRISC port Stafford Horne
2022-01-04 12:28 ` Adhemerval Zanella
2022-01-04 21:38 ` Stafford Horne
2022-01-04 12:34 ` [PATCH v5 00/13] Glibc " Adhemerval Zanella
2022-01-04 21:36 ` Stafford Horne
2022-01-05 21:34 ` Stafford Horne [this message]
2022-01-24 13:23 ` [OpenRISC] " Florian Weimer
2022-01-24 13:54 ` Adhemerval Zanella
2022-01-25 1:19 ` Stafford Horne
2022-01-25 17:07 ` Adhemerval Zanella
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=YdYO4eW2sjiIYKPg@antec \
--to=shorne@gmail.com \
--cc=adhemerval.zanella@linaro.org \
--cc=libc-alpha@sourceware.org \
--cc=openrisc@lists.librecores.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 read-only IMAP folder(s) and NNTP newsgroup(s).