public inbox for libc-alpha@sourceware.org
 help / color / mirror / Atom feed
From: Paul Eggert <eggert@cs.ucla.edu>
To: Adhemerval Zanella Netto <adhemerval.zanella@linaro.org>
Cc: libc-alpha@sourceware.org
Subject: Re: time64 / Large File Support: 2) default time64 breaks legacy 32bit binaries
Date: Thu, 26 Jan 2023 18:38:08 -0800	[thread overview]
Message-ID: <aaa0cf5d-e6ca-627d-5450-4da0d2a2b27c@cs.ucla.edu> (raw)
In-Reply-To: <2342ab66-6ac6-17d8-3693-8e2fd93fc8a1@linaro.org>

On 1/26/23 05:21, Adhemerval Zanella Netto via Libc-alpha wrote:
> So although I am not quite against --enable-hard-sys-types64, I personally
> think we should do something more drastically (which not all other glibc
> developers agree) and flip the switch to enable 64-bit time_t *as default*
> and document 32-bit is opt-in.

Just to clarify, there are two issues here:

A. Should _FILE_OFFSET_BITS and _TIME_BITS default to 64 instead of to 32?

B. Should _FILE_OFFSET_BITS=64 and _TIME_BITS=64 be the *only* values 
supported? That is, should the old API (with 32-bit time_t, off_t, etc.) 
be available only to old 32-bit executables, and not to newly-built 
32-bit executables?

As I understood it, the proposal was that (A) and (B) both default to 
"no", but '.../configure --enable-hard-sys-types64' would change both 
(A) and (B) to "yes".

It sounds like you're proposing a more drastic approach, in which (A) 
and (B) both default to "yes", but '.../configure 
--disable-hard-sys-types64' changes them both to "no". Is that right?

Another possibility is to adopt the milder approach for glibc version N, 
and switch to the more drastic approach in glibc version N+1.This would 
give distros time to test this new configure-time option before it's the 
default.

  parent reply	other threads:[~2023-01-27  2:38 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-01-25 23:57 The time64 and Large File Support mess Andreas K. Huettel
2023-01-25 23:58 ` time64 / Large File Support: 1) [2.28 Regression]: New getdents{64} implementation breaks qemu-user Andreas K. Huettel
2023-01-26 12:21   ` Adhemerval Zanella Netto
2023-01-27 20:08     ` Andreas K. Huettel
2023-01-25 23:59 ` time64 / Large File Support: 2) default time64 breaks legacy 32bit binaries Andreas K. Huettel
2023-01-26  4:13   ` Paul Eggert
2023-01-26 13:21     ` Adhemerval Zanella Netto
2023-01-26 23:35       ` Sam James
2023-01-27 17:33         ` Adhemerval Zanella Netto
2023-02-01 16:26         ` Florian Weimer
2023-02-01 19:47           ` Sam James
2023-02-01 19:54             ` Sam James
2023-02-03 17:52             ` Florian Weimer
2023-02-01 22:22           ` Michael Hudson-Doyle
2023-02-03 14:17             ` Adhemerval Zanella Netto
2023-02-03 18:56               ` Florian Weimer
2023-01-27  2:38       ` Paul Eggert [this message]
2023-01-27 17:40         ` Adhemerval Zanella Netto
2023-01-27 23:51           ` Paul Eggert
2023-01-27 23:58             ` Joseph Myers
2023-02-01 12:27               ` Adhemerval Zanella Netto
2023-01-26 10:43   ` Florian Weimer

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=aaa0cf5d-e6ca-627d-5450-4da0d2a2b27c@cs.ucla.edu \
    --to=eggert@cs.ucla.edu \
    --cc=adhemerval.zanella@linaro.org \
    --cc=libc-alpha@sourceware.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).