mbox series

[v1,00/13] riscv: report more ISA extensions through hwprobe

Message ID 20231011111438.909552-1-cleger@rivosinc.com
Headers
Series riscv: report more ISA extensions through hwprobe |

Message

Clément Léger Oct. 11, 2023, 11:14 a.m. UTC 
  In order to be able to gather more information about the supported ISA
extensions from userspace using the hwprobe syscall, add more ISA extensions
report. This series adds the following ISA extensions support:

- Zfh[min]
- Zvfh[min]
- Zihintntl
- Zvbb
- Zvbc
- Zvkb
- Zvkg
- Zvkned
- Zvknh[ab]
- Zvksed
- Zvksh
- Zvkn
- Zvknc
- Zvkng
- Zvks
- Zvksc
- Zvksg
- Zvkt

Clément Léger (13):
  riscv: fatorize hwprobe ISA extension reporting
  riscv: add ISA extension probing for Zv* extensions
  riscv: hwprobe: export Zv* ISA extensions
  dt-bindings: riscv: add Zv* ratified crypto ISA extensions description
  riscv: add ISA extension probing for Zfh/Zfhmin
  riscv: hwprobe: export Zfh/Zfhmin ISA extensions
  dt-bindings: riscv: add Zfh/Zfhmin ISA extensions description
  riscv: add ISA extension probing for Zihintntl
  riscv: hwprobe: export Zhintntl ISA extension
  dt-bindings: riscv: add Zihintntl ISA extension description
  riscv: add ISA extension probing for Zvfh[min]
  riscv: hwprobe: export Zvfh[min] ISA extensions
  dt-bindings: riscv: add Zvfh[min] ISA extension description

 .../devicetree/bindings/riscv/extensions.yaml | 127 ++++++++++++++++++
 Documentation/riscv/hwprobe.rst               |  65 +++++++++
 arch/riscv/include/asm/hwcap.h                |  21 +++
 arch/riscv/include/uapi/asm/hwprobe.h         |  21 +++
 arch/riscv/kernel/cpufeature.c                |  21 +++
 arch/riscv/kernel/sys_riscv.c                 |  53 ++++++--
 6 files changed, 294 insertions(+), 14 deletions(-)

Comments

Clément Léger Oct. 12, 2023, 7:15 a.m. UTC | #1 
On second thought, maybe it would make more sense to squash all logical
commits together (dt-bindings, hwporobe, etc) with all the ISA
extensions in each. Tell me if you think it would be better.

Clément

On 11/10/2023 13:14, Clément Léger wrote:
> In order to be able to gather more information about the supported ISA
> extensions from userspace using the hwprobe syscall, add more ISA extensions
> report. This series adds the following ISA extensions support:
> 
> - Zfh[min]
> - Zvfh[min]
> - Zihintntl
> - Zvbb
> - Zvbc
> - Zvkb
> - Zvkg
> - Zvkned
> - Zvknh[ab]
> - Zvksed
> - Zvksh
> - Zvkn
> - Zvknc
> - Zvkng
> - Zvks
> - Zvksc
> - Zvksg
> - Zvkt
> 
> Clément Léger (13):
>   riscv: fatorize hwprobe ISA extension reporting
>   riscv: add ISA extension probing for Zv* extensions
>   riscv: hwprobe: export Zv* ISA extensions
>   dt-bindings: riscv: add Zv* ratified crypto ISA extensions description
>   riscv: add ISA extension probing for Zfh/Zfhmin
>   riscv: hwprobe: export Zfh/Zfhmin ISA extensions
>   dt-bindings: riscv: add Zfh/Zfhmin ISA extensions description
>   riscv: add ISA extension probing for Zihintntl
>   riscv: hwprobe: export Zhintntl ISA extension
>   dt-bindings: riscv: add Zihintntl ISA extension description
>   riscv: add ISA extension probing for Zvfh[min]
>   riscv: hwprobe: export Zvfh[min] ISA extensions
>   dt-bindings: riscv: add Zvfh[min] ISA extension description
> 
>  .../devicetree/bindings/riscv/extensions.yaml | 127 ++++++++++++++++++
>  Documentation/riscv/hwprobe.rst               |  65 +++++++++
>  arch/riscv/include/asm/hwcap.h                |  21 +++
>  arch/riscv/include/uapi/asm/hwprobe.h         |  21 +++
>  arch/riscv/kernel/cpufeature.c                |  21 +++
>  arch/riscv/kernel/sys_riscv.c                 |  53 ++++++--
>  6 files changed, 294 insertions(+), 14 deletions(-)
>
Conor Dooley Oct. 12, 2023, 8:21 a.m. UTC | #2 
On Thu, Oct 12, 2023 at 09:15:46AM +0200, Clément Léger wrote:
> On second thought, maybe it would make more sense to squash all logical
> commits together (dt-bindings, hwporobe, etc) with all the ISA
> extensions in each. Tell me if you think it would be better.

I don't think there's anything wrong with the current approach.
Clément Léger Oct. 12, 2023, 8:25 a.m. UTC | #3 
On 12/10/2023 10:21, Conor Dooley wrote:
> On Thu, Oct 12, 2023 at 09:15:46AM +0200, Clément Léger wrote:
>> On second thought, maybe it would make more sense to squash all logical
>> commits together (dt-bindings, hwporobe, etc) with all the ISA
>> extensions in each. Tell me if you think it would be better.
> 
> I don't think there's anything wrong with the current approach.

Hi Conor,

Ok then, let's keep like this if there are no objection, I thought it
was a bit "too many" commits just for a few lines, but at least it's
unitary.

Thanks

Clément