Re: [PATCH V2 0/2] uapi: Add support for GENMASK_U128()

[Anshuman Khandual <anshuman.khandual@arm.com>]

On 7/25/24 11:18, Anshuman Khandual wrote:
> This adds support for GENMASK_U128() and some corresponding tests as well.
> GENMASK_U128() generated 128 bit masks will be required later on the arm64
> platform for enabling FEAT_SYSREG128 and FEAT_D128 features.
> 
> Because GENMAKS_U128() depends on __int128 data type being supported in the
> compiler, its usage needs to be protected with CONFIG_ARCH_SUPPORTS_INT128.
> 
> Cc: Andrew Morton <akpm@linux-foundation.org>
> Cc: Yury Norov <yury.norov@gmail.com>
> Cc: Rasmus Villemoes <linux@rasmusvillemoes.dk>
> Cc: Arnd Bergmann <arnd@arndb.de>>
> Cc: linux-kernel@vger.kernel.org
> Cc: linux-arch@vger.kernel.org
> 
> Changes in V2:
> 
> - Wrapped genmask_u128_test() with CONFIG_ARCH_SUPPORTS_INT128
> - Defined __BITS_PER_U128 unconditionally as 128
> - Defined __GENMASK_U128() via new _BIT128()
> - Dropped _U128() and _AC128()

Hello Arnd,

Does the changed series look good ? Please do let me know if something
further needs to be changed. Thank you.

- Anshuman