[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