From: arnd@arndb.de (Arnd Bergmann)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v6 13/17] arm64:ilp32: add sys_ilp32.c and a separate table (in entry.S) to use it
Date: Mon, 16 Nov 2015 14:15:58 +0100 [thread overview]
Message-ID: <4164423.46Jfedud5Y@wuerfel> (raw)
In-Reply-To: <alpine.DEB.2.10.1511161230220.25844@digraph.polyomino.org.uk>
On Monday 16 November 2015 12:34:55 Joseph Myers wrote:
> On Mon, 16 Nov 2015, Arnd Bergmann wrote:
>
> > > It's not a matter of leaving anything out - these would simply use 64-bit
> > > off_t (__off_t and __off64_t would be the same type) and the *64 versions
> > > would be aliases, exactly the same as on 64-bit architectures. (And
> > > _FILE_OFFSET_BITS handling would also be exactly the same as on 64-bit
> > > architectures.) I see no reason for the set of off_t-related symbols that
> > > exist, or which symbols are aliases of which others, to vary between pure
> > > 64-bit systems and ILP32 ABIs (for 32-bit or 64-bit architectures) that
> > > simply happen to have had 64-bit off_t from the start.
> >
> > Ok, fair enough. So we just change the global __OFF_T_TYPE definition
> > in bits/typesizes.h and override it for all the existing 32-bit ports,
> > correct?
>
> Well, it's sysdeps/unix/sysv/linux/generic/bits/typesizes.h that's
> relevant - so if future generic architectures will use 64-bit off_t, I
> suppose the existing file could be cloned for existing generic
> architectures with 32-bit support.
Ok, got it.
> And all the types involved in struct stat are affected (e.g. ino_t),
> not just off_t.
ino_t seems to be the only other type in 'struct stat' that depends
on _FILE_OFFSET_BITS in glibc. On the kernel side, we don't care about
__kernel_ino_t any more, we just leave that defined as 'unsigned long'
while using a plain 'unsigned long long' for 'st_ino' in struct stat64
(and don't use __kernel_ino_t anywhere else either).
> And getting the aliases
> right may involve disentangling the different meanings of wordsize-64 into
> different sysdeps directories. ("off_t is off64_t" and "stat is stat64"
> are not the same thing. See MIPS n64.) And the design work needs to be
> done on libc-alpha, not in a random discussion elsewhere.
Sure. For the moment, we have all the information we need for the kernel
side at least: we will keep using only 64-bit __kernel_loff_t on the
system call side in new architecture ports and let you figure out how
to work with that on the glibc side whenever the next 32-bit port arrives,
which I assume will be arm64-ilp32.
The 'struct stat' discussion will of course come back soon when we get to
the 64-bit time_t patches, or when we introduce the extended stat syscall,
whichever happens first.
Thanks a lot for your help!
Arnd
next prev parent reply other threads:[~2015-11-16 13:15 UTC|newest]
Thread overview: 60+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-11-02 23:30 [RFC PATCH v6 00/17] ILP32 for ARM64 Yury Norov
2015-11-02 23:30 ` [PATCH v6 01/17] arm64:ilp32: add documentation on the ILP32 ABI " Yury Norov
2015-11-05 14:33 ` Arnd Bergmann
2015-11-02 23:30 ` [PATCH v6 02/17] arm64: ensure the kernel is compiled for LP64 Yury Norov
2015-11-02 23:30 ` [PATCH v6 03/17] arm64: rename COMPAT to AARCH32_EL0 in Kconfig Yury Norov
2015-11-02 23:30 ` [PATCH v6 04/17] arm64: change some CONFIG_COMPAT over to use CONFIG_AARCH32_EL0 instead Yury Norov
2015-11-02 23:30 ` [PATCH v6 05/17] arm64:ilp32: share signal structures between ILP32 and LP64 ABIs Yury Norov
2015-11-02 23:30 ` [PATCH v6 06/17] arm64: introduce is_a32_task and is_a32_thread (for AArch32 compat) Yury Norov
2015-11-02 23:30 ` [PATCH v6 07/17] arm64:ilp32: add is_ilp32_compat_{task, thread} and TIF_32BIT_AARCH64 Yury Norov
2015-11-02 23:30 ` [PATCH v6 08/17] arm64:ilp32: share HWCAP between LP64 and ILP32 Yury Norov
2015-11-02 23:30 ` [PATCH v6 09/17] arm64:ilp32 use the native LP64 'start_thread' for ILP32 threads Yury Norov
2015-11-02 23:30 ` [PATCH v6 10/17] arm64:ilp32: support core dump generation for ILP32 Yury Norov
2015-11-02 23:30 ` [PATCH v6 11/17] ptrace: Allow compat to use the native siginfo Yury Norov
2015-11-05 13:53 ` Arnd Bergmann
2015-11-02 23:30 ` [PATCH v6 12/17] ilp32: common 32-bit wrappers Yury Norov
2015-11-02 23:30 ` [PATCH v6 13/17] arm64:ilp32: add sys_ilp32.c and a separate table (in entry.S) to use it Yury Norov
2015-11-05 13:50 ` Arnd Bergmann
2015-11-11 17:54 ` Andreas Schwab
2015-11-11 20:40 ` Arnd Bergmann
2015-11-11 23:07 ` Andreas Schwab
2015-11-11 23:23 ` Arnd Bergmann
2015-11-12 8:58 ` Andreas Schwab
2015-11-12 9:22 ` Arnd Bergmann
2015-11-12 9:44 ` Andreas Schwab
2015-11-12 13:24 ` Arnd Bergmann
2015-11-12 13:47 ` Andreas Schwab
2015-11-13 15:34 ` Arnd Bergmann
2015-11-13 15:38 ` Andrew Pinski
2015-11-13 16:10 ` Arnd Bergmann
2015-11-15 15:18 ` Arnd Bergmann
2015-11-15 16:42 ` Chris Metcalf
2015-11-16 10:16 ` Joseph Myers
2015-11-16 11:00 ` Arnd Bergmann
2015-11-16 11:12 ` Joseph Myers
2015-11-16 11:40 ` Arnd Bergmann
2015-11-16 12:03 ` Joseph Myers
2015-11-16 12:12 ` Arnd Bergmann
2015-11-16 12:34 ` Joseph Myers
2015-11-16 13:15 ` Arnd Bergmann [this message]
2015-11-16 13:34 ` Andreas Schwab
2015-11-16 13:54 ` Arnd Bergmann
2015-11-02 23:30 ` [PATCH v6 14/17] aarch64: ilp32: use generic stat64 structure Yury Norov
2015-11-05 14:03 ` Arnd Bergmann
2015-11-02 23:30 ` [PATCH v6 15/17] arm64:ilp32: use the native siginfo instead of the compat siginfo Yury Norov
2015-11-02 23:30 ` [PATCH v6 16/17] arm64:ilp32: change COMPAT_ELF_PLATFORM to report a a subplatform for ILP32 Yury Norov
2015-11-02 23:30 ` [PATCH v6 17/17] arm64:ilp32: add ARM64_ILP32 to Kconfig Yury Norov
2015-11-05 11:36 ` [RFC PATCH v6 00/17] ILP32 for ARM64 Andreas Schwab
2015-11-05 13:39 ` Andrew Pinski
2015-11-05 22:18 ` Yury Norov
2015-11-09 10:01 ` Andreas Schwab
2015-11-11 7:16 ` Zhangjian (Bamvor)
2015-11-11 8:44 ` Andreas Schwab
2015-11-09 13:23 ` Andreas Schwab
2015-11-09 14:21 ` Arnd Bergmann
2015-11-09 14:33 ` Andreas Schwab
2015-11-09 14:36 ` Arnd Bergmann
2015-11-09 14:52 ` pinskia at gmail.com
2015-11-09 15:50 ` Yury Norov
2015-11-09 16:04 ` Yury Norov
[not found] ` <1449010556-31699-1-git-send-email-iharutyunov@sonicwall.com>
2015-12-03 17:05 ` Yury Norov
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=4164423.46Jfedud5Y@wuerfel \
--to=arnd@arndb.de \
--cc=linux-arm-kernel@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).