-
Notifications
You must be signed in to change notification settings - Fork 600
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[AdditionalVectorCrypto] adding section on addition vector crypto extensions #1306
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Nicolas Brunie <82109999+nibrunieAtSi5@users.noreply.github.com>
Will it be reserved or not for instructions (vghsh and vgmul) , if the vd register group overlaps with the vs2 register group. |
Good question @QJtaibai , this is not listed but should be to aligned with the other |
Hello! I have a question that in the instruction part of document vghsh.vs shares same encoding with vghsh.vv, and has func6:101100, while in Appendix A it looks vghsh.vs have func6:100011. I wonder which one is correct? |
Signed-off-by: Nicolas Brunie <nibrunie@gmail.com>
Appendix A and the riscv-v opcode Pull request (https://github.com/nibrunieAtSi5/riscv-opcodes/pull/1/files) are right, the instruciton part of the document was incorrect. I have fixed it. Note that those opcodes are simply suggestion at this point (but that does not mean the suggestions should not be consistent). Thank you for pointing that out. |
I have developed a small example to demonstrate how The current implementation provides "performance" of about 0.65 instruction per bytes (on 1MB input buffer). |
This pull request is the
riscv-isa-manual
version of a pull request started onriscv-crypto
: riscv/riscv-crypto#362./!\ This pull request is a draft for the future fast track extensions.
This pull requests draft the changes associated with two fast track extensions for vector crypto.
During the specification process for vector crypto 1.0.0 a few items had to be discarded because they appeared too late in the process. This fast track extension tries to address some of them.
The official demand that will be discussed in the Task Group and submitted to the Unpriv Committee is being drafter here: https://docs.google.com/document/d/1zpYhnZi2NxhjfcBGvPOy0oDhx6lTXchscG17Qcl6wv8/edit?usp=sharing
New features:
Zvbc32e
: Extendingvclmul[h].v[vh]
instruction to supportSEW=32-bit
valueELEN >= 32
) or in addition toZvbc
(ELEN >= 64
)Zvkgs
: Adding.vs
variants tovghsh
andvghmul
Zvkgs
Open questions:
Zvbc32e
be allowed whenELEN >= 32
without depending onZvbc
? (Answer: YES)Zvbc32e
support SEW=16 ? (SEW=8 ?)How to name the two new extensionsZvkt(bc/bc32e)
to extendZvkt
to the extension ofvclmul[h/]
defined inZvbc32e
?Related changes:
spike-isa-sim
modificationsZvbc32e
andZvkgs
: Vector crypto additional riscv-software-src/riscv-isa-sim#1748Zvbce32
https://github.com/riscv/riscv-crypto/blob/main/doc/vector/code-samples/zvbc-test.cZvkgs
https://github.com/riscv/riscv-crypto/blob/main/doc/vector/code-samples/zvkg-test.cZvkgs
Zvbc32e
Draft versions:
Changelogs
vs2
/vd
overlap as reserved encoding for newvghsh.vs
/vgmul.vs
instructions (review feedback from @QJtaibai )Original Plan for the fast track schedule
References