From: Joseph Myers <joseph@codesourcery.com>
To: Lukasz Majewski <lukma@denx.de>
Cc: Paul Eggert <eggert@cs.ucla.edu>,
Adhemerval Zanella <adhemerval.zanella@linaro.org>,
Florian Weimer <fweimer@redhat.com>,
GNU C Library <libc-alpha@sourceware.org>,
Siddhesh Poyarekar <siddhesh@redhat.com>,
Andreas Schwab <schwab@suse.de>,
Stepan Golosunov <stepan@golosunov.pp.ru>,
Alistair Francis <alistair.francis@wdc.com>
Subject: Re: [RFC 0/6] y2038: Prepare glibc to be Y2038 safe for 32 bit ports
Date: Sat, 5 Dec 2020 00:01:04 +0000 [thread overview]
Message-ID: <alpine.DEB.2.22.394.2012042356550.240600@digraph.polyomino.org.uk> (raw)
In-Reply-To: <20201204233604.7430-1-lukma@denx.de>
On Sat, 5 Dec 2020, Lukasz Majewski wrote:
> This patch series is the RFC for preparing glibc to be Y2038 safe on
> ports with __WORDSIZE=32 && __TIMESIZE!=64 (e.g. 32 bit ARM).
What exactly does "preparing" mean here?
I'd expect support for _TIME_BITS to go along with public symbol versions
for all the new symbols that functions get redirected to, because
otherwise people building with _TIME_BITS=64 will get either link failures
or broken binaries depending on GLIBC_PRIVATE symbols. But as far as I
can see, this patch series only adds GLIBC_PRIVATE symbols, not any public
symbols or corresponding abilist updates.
I'd also expect support for _TIME_BITS to go along with tests covering all
the new interfaces (that is, for every time-affected interface, there
should be a test that it works properly with _TIME_BITS=64, typically a
test that defines _TIME_BITS to 64 then includes an existing test for that
interface, possibly with other tests to verify that large time values work
properly where possible).
You don't say how this series was tested, but this may be around the point
where build-many-glibcs.py testing is needed. Certainly any patch in such
a series that adds a new public symbol should be tested with
build-many-glibcs.py to make sure that all the ABI test baseline updates
are correct.
--
Joseph S. Myers
joseph@codesourcery.com
next prev parent reply other threads:[~2020-12-05 0:01 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-12-04 23:35 Lukasz Majewski
2020-12-04 23:35 ` [RFC 1/6] y2038: Introduce _TIME_BITS flag to support 64 bit time on 32 bit systems Lukasz Majewski
2020-12-05 0:12 ` Joseph Myers
2020-12-07 13:00 ` Lukasz Majewski
2020-12-04 23:36 ` [RFC 2/6] y2038: stat: {f}stat{at}64_time64 redirection to be used on Y2038 systems Lukasz Majewski
2020-12-05 0:04 ` Joseph Myers
2020-12-07 10:28 ` Lukasz Majewski
2020-12-07 18:07 ` Joseph Myers
2020-12-04 23:36 ` [RFC 3/6] y2038: Export struct_stat_time64_helper.h with Y2038 safe stat{64} content Lukasz Majewski
2020-12-29 10:30 ` Lukasz Majewski
2020-12-04 23:36 ` [RFC 4/6] y2038: Enhance struct msqid_ds to support 64 bit time Lukasz Majewski
2020-12-29 10:34 ` Lukasz Majewski
2020-12-04 23:36 ` [RFC 5/6] msqid: Provide internal copy of struct __msqid64_ds Lukasz Majewski
2020-12-29 10:54 ` Lukasz Majewski
2020-12-04 23:36 ` [RFC 6/6] msg: provide glibc local copy of struct msqid_ds Lukasz Majewski
2020-12-05 0:01 ` Joseph Myers [this message]
2020-12-07 10:22 ` [RFC 0/6] y2038: Prepare glibc to be Y2038 safe for 32 bit ports Lukasz Majewski
2020-12-07 18:01 ` Joseph Myers
2020-12-08 10:07 ` Lukasz Majewski
2020-12-08 15:25 ` Joseph Myers
2020-12-13 11:49 ` Lukasz Majewski
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=alpine.DEB.2.22.394.2012042356550.240600@digraph.polyomino.org.uk \
--to=joseph@codesourcery.com \
--cc=adhemerval.zanella@linaro.org \
--cc=alistair.francis@wdc.com \
--cc=eggert@cs.ucla.edu \
--cc=fweimer@redhat.com \
--cc=libc-alpha@sourceware.org \
--cc=lukma@denx.de \
--cc=schwab@suse.de \
--cc=siddhesh@redhat.com \
--cc=stepan@golosunov.pp.ru \
/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).