public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
From: Tsukasa OI <research_trasio@irq.a4lg.com>
To: Nelson Chu <nelson.chu@sifive.com>
Cc: Binutils <binutils@sourceware.org>
Subject: Re: [PATCH 0/5] RISC-V: Add Ratified Cache Management Operation ISA Extensions
Date: Sat, 18 Dec 2021 02:24:09 +0900	[thread overview]
Message-ID: <c9f116ee-3937-b355-bdb3-5c1610c6801c@irq.a4lg.com> (raw)
In-Reply-To: <CAJYME4Gzo4rLtSio-TU5Cn+eTVMsMR3v5vO=95mVoYD2KQ0OGA@mail.gmail.com>

Hi Nelson,

Thanks for quick review and ...I didn't expect that you can view
my copyright assignment status.

Anyway, you are right.  Considering I'm preparing far more larger patchset
(privileged specification 1.12 CSRs), I definitely need copyright
assignment including future changes.

I requested copyright assignment form based on request-assign.future and
legal procedures will complete in a few weeks (I guess).

Tsukasa

On 2021/12/18 0:23, Nelson Chu wrote:
> Hi Tsukasa,
> 
> Thanks for implementing these extensions, LGTM.  But I notice that
> your copyright assignment is only for GNU WGET, and it seems it
> doesn't cover the changes of binutils.  It would be great if you could
> check this for us, thanks.
> 
> Nelson
> 
> 
> On Thu, Dec 16, 2021 at 7:10 PM Tsukasa OI via Binutils
> <binutils@sourceware.org> wrote:
>>
>> This patchset adds support for three recently ratified RISC-V extensions:
>>
>> -   Zicbom (Cache-Block Management Instructions)
>> -   Zicbop (Cache-Block Prefetch hint instructions)
>> -   Zicboz (Cache-Block Zero Instructions)
>>
>> `prefetch.[irw]' hint instructions in Zicbop extension require new operand
>> type which is pseudo S-type immediate with low 5-bits set to zero
>> (32-byte aligned).  So, Zicbop changes are separate from Zicbom/Zicboz:
>>
>>     Patch 1:   Zicbom/z/p (common)
>>     Patch 2/3: Zicbom/z (regular CBO instructions)
>>     Patch 4/5: Zicbop   (prefetch hint instructions)
>>
>> cf. <https://github.com/riscv/riscv-CMOs/blob/fc8e97a9531ac9811971a182ae431976b86216e1/specifications/cmobase-v1.0-rc2.pdf>
>>
>>
>>
>>
>> Tsukasa OI (5):
>>   RISC-V: Add mininal support for Zicbo[mpz]
>>   RISC-V: Cache management instructions
>>   RISC-V: Cache management instruction testcases
>>   RISC-V: Prefetch hint instructions and operand set
>>   RISC-V: Prefetch hint instruction testcases
>>
>>  bfd/elfxx-riscv.c                     |  9 +++++++++
>>  gas/config/tc-riscv.c                 | 17 +++++++++++++++++
>>  gas/testsuite/gas/riscv/zicbom.d      | 15 +++++++++++++++
>>  gas/testsuite/gas/riscv/zicbom.s      |  7 +++++++
>>  gas/testsuite/gas/riscv/zicbop-fail.d |  3 +++
>>  gas/testsuite/gas/riscv/zicbop-fail.l |  4 ++++
>>  gas/testsuite/gas/riscv/zicbop-fail.s |  4 ++++
>>  gas/testsuite/gas/riscv/zicbop.d      | 12 ++++++++++++
>>  gas/testsuite/gas/riscv/zicbop.s      |  4 ++++
>>  gas/testsuite/gas/riscv/zicboz.d      | 11 +++++++++++
>>  gas/testsuite/gas/riscv/zicboz.s      |  3 +++
>>  include/opcode/riscv-opc.h            | 14 ++++++++++++++
>>  include/opcode/riscv.h                |  3 +++
>>  opcodes/riscv-dis.c                   |  4 ++++
>>  opcodes/riscv-opc.c                   |  9 +++++++++
>>  15 files changed, 119 insertions(+)
>>  create mode 100644 gas/testsuite/gas/riscv/zicbom.d
>>  create mode 100644 gas/testsuite/gas/riscv/zicbom.s
>>  create mode 100644 gas/testsuite/gas/riscv/zicbop-fail.d
>>  create mode 100644 gas/testsuite/gas/riscv/zicbop-fail.l
>>  create mode 100644 gas/testsuite/gas/riscv/zicbop-fail.s
>>  create mode 100644 gas/testsuite/gas/riscv/zicbop.d
>>  create mode 100644 gas/testsuite/gas/riscv/zicbop.s
>>  create mode 100644 gas/testsuite/gas/riscv/zicboz.d
>>  create mode 100644 gas/testsuite/gas/riscv/zicboz.s
>>
>>
>> base-commit: 23ff54c27d535727c1c467abdd4bed8fbd46d4a6
>> --
>> 2.32.0
>>
> 

  reply	other threads:[~2021-12-17 17:24 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-12-16 11:04 Tsukasa OI
2021-12-16 11:04 ` [PATCH 1/5] RISC-V: Add mininal support for Zicbo[mpz] Tsukasa OI
2021-12-16 11:04 ` [PATCH 2/5] RISC-V: Cache management instructions Tsukasa OI
2022-01-04  8:01   ` Jan Beulich
2022-01-04 22:15     ` Andrew Waterman
2022-01-05  3:17     ` Tsukasa OI
2021-12-16 11:04 ` [PATCH 3/5] RISC-V: Cache management instruction testcases Tsukasa OI
2021-12-16 11:04 ` [PATCH 4/5] RISC-V: Prefetch hint instructions and operand set Tsukasa OI
2021-12-17 15:15   ` Nelson Chu
2021-12-16 11:04 ` [PATCH 5/5] RISC-V: Prefetch hint instruction testcases Tsukasa OI
2021-12-17 15:23 ` [PATCH 0/5] RISC-V: Add Ratified Cache Management Operation ISA Extensions Nelson Chu
2021-12-17 17:24   ` Tsukasa OI [this message]
2021-12-17 17:39     ` Nelson Chu

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=c9f116ee-3937-b355-bdb3-5c1610c6801c@irq.a4lg.com \
    --to=research_trasio@irq.a4lg.com \
    --cc=binutils@sourceware.org \
    --cc=nelson.chu@sifive.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).