[PATCH 16/18] arm64: fpsimd: Move sve_flush_live() inline
Mark Rutland
mark.rutland at arm.com
Wed May 27 09:23:39 PDT 2026
On Wed, May 27, 2026 at 01:54:42PM +0100, Mark Brown wrote:
> On Thu, May 21, 2026 at 02:25:54PM +0100, Mark Rutland wrote:
> > Currently sve_flush_live() is written in out-of-line assembly. It would
> > be nice if we could move it inline such that control flow can be written
> > more clearly in C, and to permit the removal of otherwise unused
> > assembly macros.
>
> > +static inline void sve_flush_live(void)
> > +{
> > + unsigned long vl = sve_get_vl();
> > +
> > + if (vl > sizeof(__uint128_t)) {
>
> It feels odd to use sizeof(__unit128_t) here rather than u128. Whatever
> colour the bikeshed is:
FWIW, that was just to match __fpsimd_to_sve(). If there was nothing
else to be consistent with, I agree that using u128 woul2d be
preferable.
> Reviewed-by: Mark Brown <broonie at kernel.org>
Thanks!
Mark.
More information about the linux-arm-kernel
mailing list