From: Christoph Hellwig <hch@lst.de>
To: patches@groups.riscv.org
Cc: linux-kernel@vger.kernel.org, Arnd Bergmann <arnd@arndb.de>,
Olof Johansson <olof@lixom.net>,
Palmer Dabbelt <palmer@sifive.com>,
Adhemerval Zanella <adhemerval.zanella@linaro.org>
Subject: Re: [patches] [RFC] RISC-V: Don't set CLONE_BACKWARDS
Date: Tue, 9 Jan 2018 09:11:45 +0100 [thread overview]
Message-ID: <20180109081145.GA2094@lst.de> (raw)
In-Reply-To: <20180109012756.2401-1-palmer@sifive.com>
On Mon, Jan 08, 2018 at 05:27:56PM -0800, Palmer Dabbelt wrote:
> During the glibc upstreaming it was suggested that CLONE_BACKWARDS was a
> deprecated ABI decision. I think we just copied it from ARM, but I
> don't see any reason to favor one over the other.
>
> While we haven't released yet so I think it's still legal to change our
> ABI, I'd actually kind of prefer to avoid changing our ABI this late in
> the game. I guess this is more of an RFC than a patch: is there a
> reason to avoid CLONE_BACKWARDS?
>
> Note that I haven't tried any of this -- I'll give it some thourough
> testing and submit an actual patch if this is the way we want to go.
I see absolutely no reason to change this. Linux currently has 30
architecture port, out of which 10 (including riscv, i386, arm and arm64)
set CLONE_BACKWARDS.
There are no performance benefits of doing it one way or another, and
changing it now will break all the riscv enablement that's been going
on.
next prev parent reply other threads:[~2018-01-09 8:11 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-01-09 1:27 [RFC] RISC-V: Don't set CLONE_BACKWARDS Palmer Dabbelt
2018-01-09 8:11 ` Christoph Hellwig [this message]
2018-01-09 8:20 ` [patches] " Palmer Dabbelt
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=20180109081145.GA2094@lst.de \
--to=hch@lst.de \
--cc=adhemerval.zanella@linaro.org \
--cc=arnd@arndb.de \
--cc=linux-kernel@vger.kernel.org \
--cc=olof@lixom.net \
--cc=palmer@sifive.com \
--cc=patches@groups.riscv.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