public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
From: Jim Wilson <jim.wilson.gcc@gmail.com>
To: lkcl <luke.leighton@gmail.com>
Cc: binutils@sourceware.org
Subject: Re: [PATCH v1] RISC-V: Support XVentanaCondOps extension
Date: Wed, 20 Apr 2022 15:15:55 -0700	[thread overview]
Message-ID: <CALNwTfxuzqtN0YCL=nuQxyvafhkupaoJ72vc0BE5u5A+t0Tq7w@mail.gmail.com> (raw)
In-Reply-To: <C1BE526B-6CF1-4929-8AA2-8FD7BB67093C@gmail.com>

On Wed, Apr 20, 2022 at 10:55 AM lkcl via Binutils <binutils@sourceware.org>
wrote:

> my understanding of custom extensions is that they are intended never,
> under any circumstances, to hit "upstream", due to the risk of creating
> massive conflict and confusion due to dominance of one extension over
> another, particularly if that custom extension achieves extremely
> commonly-used and high profile public status.
>

The RISC-V ISA has always planned for support for custom extensions.  We
have a part of the opcode space reserved for custom extensions, and we have
an arch string syntax for specifying custom extensions.   We are also
trying to standardize the naming conventions for custom extensions in
riscv-toolchain-conventions to avoid conflict.  See pull requests #17 and
#19.

Custom extensions are a fact of life.  All major RISC-V vendors have them.
You are suggesting that each vendor should maintain their own toolchain
source tree.  But if they all do that, then there is a risk that none will
contribute patches upstream.  We are better off if we allow vendor custom
extensions upstream, and that will encourage vendors to work upstream.  We
already have support for SiFive custom extensions on
the users/riscv/binutils-integration-branch.  The plan was to move that to
the master branch after the last release, but Nelson has been busy and
hasn't gotten around to it yet.  Ventana custom extensions are just more of
the same.  Hopefully we can also get Alibaba/T-Head and others to add their
custom extensions upstream too.

If you don't want custom extensions, it is easy enough to avoid them by not
enabling them in the arch string.  I expect that vendor independent linux
distros will be built without any vendor custom extensions enabled.

Jim

  reply	other threads:[~2022-04-20 22:16 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <mailman.26390.1650459569.2100866.binutils@sourceware.org>
2022-04-20 17:55 ` lkcl
2022-04-20 22:15   ` Jim Wilson [this message]
2022-04-20 23:17     ` lkcl
2022-01-09 19:29 Philipp Tomsich
2022-01-17 12:57 ` Philipp Tomsich
2022-04-20 11:37 ` Philipp Tomsich
2022-04-20 12:42   ` Kito Cheng
2022-04-22 10:37     ` Philipp Tomsich
2022-04-23  1:42       ` Kito Cheng
2022-04-25  9:38         ` Nelson Chu
2022-05-12 19:59           ` Philipp Tomsich
2022-05-12 22:23             ` Palmer Dabbelt
2022-05-13  7:55               ` Philipp Tomsich
2022-05-26 19:50               ` Philipp Tomsich
2022-05-30 19:33                 ` Palmer Dabbelt
2022-05-30 20:09                   ` Philipp Tomsich
2023-04-25 10:48     ` Philipp Tomsich
2023-04-26 20:34       ` Jeff Law

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='CALNwTfxuzqtN0YCL=nuQxyvafhkupaoJ72vc0BE5u5A+t0Tq7w@mail.gmail.com' \
    --to=jim.wilson.gcc@gmail.com \
    --cc=binutils@sourceware.org \
    --cc=luke.leighton@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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).