mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Joseph Myers <joseph@codesourcery.com>
To: Yury Norov <ynorov@caviumnetworks.com>
Cc: <libc-alpha@sourceware.org>, <linux-kernel@vger.kernel.org>,
	<arnd@arndb.de>, <catalin.marinas@arm.com>,
	<marcus.shawcroft@arm.com>, <philb@gnu.org>,
	<davem@davemloft.net>, <szabolcs.nagy@arm.com>,
	<maxim.kuvyrkov@linaro.org>, <pinskia@gmail.com>,
	<bamvor.zhangjian@huawei.com>, <schwab@suse.de>,
	<fweimer@redhat.com>, <Prasun.Kapoor@cavium.com>,
	<cmetcalf@mellanox.com>, <hjl.tools@gmail.com>,
	<adhemerval.zanella@linaro.org>,
	Yury Norov <yury.norov@gmail.com>
Subject: Re: [PATCH 17/23] [AARCH64] ILP32: introduce syscalls that pass off_t
Date: Tue, 28 Jun 2016 17:13:07 +0000	[thread overview]
Message-ID: <alpine.DEB.2.20.1606281711290.25214@digraph.polyomino.org.uk> (raw)
In-Reply-To: <1467131978-669-18-git-send-email-ynorov@caviumnetworks.com>

On Tue, 28 Jun 2016, Yury Norov wrote:

> ILP32 has 64-bit off_t, to follow modern requirements.
> But kernel clears top-halves of input registers. It means
> we have to pass corresponding arguments in a pair, like
> aarch32 does. In this patch all affected syscalls are redefined.
> Most of them are taken from arm code.

Please follow the spirit of Adhemerval's recent work.  Determine the 
appropriate abstractions for the architecture differences for these 
syscalls, and set up unified implementations used on all architectures, 
rather than increasing the number of architecture-specific variants.

> +#ifdef __ILP32__
> +/* Define this if your 32-bit syscall API requires 64-bit register
> +   pairs to start with an even-number register.  */
> +#define __ASSUME_ALIGNED_REGISTER_PAIRS	1
> +#endif /* __ILP32__ */

Preprocessor indentation.

-- 
Joseph S. Myers
joseph@codesourcery.com

  reply	other threads:[~2016-06-28 17:13 UTC|newest]

Thread overview: 59+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-06-28 16:39 [RFC2 PATCH 00/23] ARM64: support ILP32 Yury Norov
2016-06-28 16:39 ` [PATCH 01/23] [AARCH64] define word size for lp64 and ilp32 Yury Norov
2016-06-28 17:00   ` Joseph Myers
2016-06-28 17:30     ` Joseph Myers
2016-06-28 16:39 ` [PATCH 02/23] [AARCH64] Add header guards to sysdep.h headers Yury Norov
2016-06-28 16:39 ` [PATCH 03/23] Add dynamic ILP32 AARCH64 relocations to elf.h Yury Norov
2016-06-28 17:03   ` Joseph Myers
2016-06-28 16:39 ` [PATCH 04/23] [AARCH64] Add PTR_REG, PTR_LOG_SIZE, and PTR_SIZE. Use it in LDST_PCREL and LDST_GLOBAL Yury Norov
2016-06-28 16:39 ` [PATCH 05/23] [AARCH64] Use PTR_REG in crti.S Yury Norov
2016-06-28 17:05   ` Joseph Myers
2016-06-28 16:39 ` [PATCH 06/23] [AARCH64] Use PTR_REG/PTR_SIZE/PTR_SIZE_LOG in dl-tlsesc.S Yury Norov
2016-06-28 17:06   ` Joseph Myers
2016-06-28 16:39 ` [PATCH 07/23] [AARCH64] Use PTR_* macros in dl-trampoline.S Yury Norov
2016-06-28 16:39 ` [PATCH 08/23] [AARCH64] Use PTR_* in start.S Yury Norov
2016-06-28 16:39 ` [PATCH 09/23] [AARCH64] Use PTR_REG in getcontext.S Yury Norov
2016-06-28 16:39 ` [PATCH 10/23] [AARCH64] Detect ILP32 in configure scripts Yury Norov
2016-06-28 17:07   ` Joseph Myers
2016-06-28 17:26     ` Yury Norov
2016-06-28 17:35       ` Joseph Myers
2016-06-28 16:39 ` [PATCH 11/23] [AARCH64] Add ILP32 support to elf_machine_load_address Yury Norov
2016-06-28 16:39 ` [PATCH 12/23] [AARCH64] Add ILP32 to makefiles Yury Norov
2016-06-28 16:39 ` [PATCH 13/23] [AARCH64] Add support to ldconfig for ILP32 and libilp32 Yury Norov
2016-06-28 16:39 ` [PATCH 14/23] [AARCH64] Add ILP32 ld.so to the known interpreter names Yury Norov
2016-06-28 16:39 ` [PATCH 15/23] [AARCH64] Add ldd-rewrite.sed so that ilp32 ld.so can be found Yury Norov
2016-06-28 16:39 ` [PATCH 16/23] [AARCH64] Make lp64 and ilp32 directories Yury Norov
2016-06-28 17:10   ` Joseph Myers
2016-06-29  8:43   ` Andreas Schwab
2016-06-28 16:39 ` [PATCH 17/23] [AARCH64] ILP32: introduce syscalls that pass off_t Yury Norov
2016-06-28 17:13   ` Joseph Myers [this message]
2016-06-29 10:10   ` Andreas Schwab
2016-06-29 12:04     ` Andreas Schwab
2016-06-28 16:39 ` [PATCH 18/23] [AARCH64] ILP32: support stat syscall family Yury Norov
2016-06-28 17:15   ` Joseph Myers
2016-06-28 19:08     ` Yury Norov
2016-06-28 20:41       ` Adhemerval Zanella
2016-06-28 20:46       ` Joseph Myers
2016-06-29  6:57   ` Andreas Schwab
2016-06-29  9:20   ` Andreas Schwab
2016-06-28 16:39 ` [PATCH 19/23] [AARCH64] delouse input arguments in system functions Yury Norov
2016-06-28 17:15   ` Joseph Myers
2016-06-29  7:49   ` Andreas Schwab
2016-06-28 16:39 ` [PATCH 20/23] [AARCH64] Fix ILP32 warning Yury Norov
2016-06-28 16:39 ` [PATCH 21/23] [AARCH64] Make __SIZEOF_SEM_T 16 for ILP32 Yury Norov
2016-06-28 17:16   ` Joseph Myers
2016-06-29  7:55   ` Andreas Schwab
2016-06-28 16:39 ` [PATCH 22/23] off_t: fix register pair calculation for 64-bit case Yury Norov
2016-06-28 17:17   ` Joseph Myers
2016-06-28 16:39 ` [PATCH 23/23] [AARCH64] Take utmp{,x}.h from s390 port Yury Norov
2016-06-28 17:18   ` Joseph Myers
2016-06-28 17:59     ` Yury Norov
2016-06-28 20:38       ` Joseph Myers
2016-06-28 20:43       ` Adhemerval Zanella
2016-06-28 16:57 ` [RFC2 PATCH 00/23] ARM64: support ILP32 Joseph Myers
2016-06-28 17:31 ` Adhemerval Zanella
2016-07-05  7:33 ` Andreas Schwab
2016-07-08 15:12 ` Yury Norov
2016-09-08  4:25 ` Zhangjian (Bamvor)
2016-09-08 11:37   ` Yury Norov
2016-09-08 11:51     ` Zhangjian (Bamvor)

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.20.1606281711290.25214@digraph.polyomino.org.uk \
    --to=joseph@codesourcery.com \
    --cc=Prasun.Kapoor@cavium.com \
    --cc=adhemerval.zanella@linaro.org \
    --cc=arnd@arndb.de \
    --cc=bamvor.zhangjian@huawei.com \
    --cc=catalin.marinas@arm.com \
    --cc=cmetcalf@mellanox.com \
    --cc=davem@davemloft.net \
    --cc=fweimer@redhat.com \
    --cc=hjl.tools@gmail.com \
    --cc=libc-alpha@sourceware.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=marcus.shawcroft@arm.com \
    --cc=maxim.kuvyrkov@linaro.org \
    --cc=philb@gnu.org \
    --cc=pinskia@gmail.com \
    --cc=schwab@suse.de \
    --cc=szabolcs.nagy@arm.com \
    --cc=ynorov@caviumnetworks.com \
    --cc=yury.norov@gmail.com \
    /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

all inboxes | Powered by JetHome®