[PATCH v1 0/3] Remove toolchain dependencies for Zicbom

[Andrew Jones]

On Sun, Jan 08, 2023 at 04:33:53PM +0000, Conor Dooley wrote:
> From: Conor Dooley <conor.dooley at microchip.com>
> 
> Hey all,
> 
> I've yoinked patch 1 from Drew's series adding support for Zicboz &
> attached two more patches here that remove the need for, and then drop
> the toolchain support checks for Zicbom. The goal is to remove the need
> for checking the presence of toolchain Zicbom support in the work being
> done to support non instruction based CMOs [1].
> 
> I've tested compliation on a number of different configurations with
> the Zicbom config option enabled. The important ones to call out I
> guess are:
> - clang/llvm 14 w/ LLVM=1 which doesn't support Zicbom atm.
> - gcc 11 w/ binutils 2.37 which doesn't support Zicbom atm either.
> - clang/llvm 15 w/ LLVM=1 BUT with binutils 2.37's ld. This is the
>   configuration that prompted adding the LD checks as cc/as supports
>   Zicbom, but ld doesn't [2].
> - gcc 12 w/ binutils 2.39 & clang 15 w/ LLVM=1, both of these supported
>   Zicbom before and still do.
> 
> I also checked building the THEAD errata etc with
> CONFIG_RISCV_ISA_ZICBOM disabled, and there were no build issues there
> either.
> 
> Drew - I took the liberty of dropping the cbo.zero addition from your
> patch.

Thanks, LGTM.

For the series,

Reviewed-by: Andrew Jones <ajones at ventanamicro.com>

Thanks,
drew