Re: [PATCH 2/3] loongarch: Introduce sys_loongarch_flush_icache syscall

[Xi Ruoyao <xry111@xry111.site>]

On Thu, 2025-01-02 at 18:34 +0000, Jiaxun Yang wrote:

/* snip */

> Sadly many user space applications are assuming ICACHET support, we can't
> recall those binaries. So we'd better get UAPI for cacheflush ready soonish
> and encourage application to start using it.

To encourage the developers changing ibar to loongarch_flush_icache, we
should minimize the extra overhead on mainstream systems.  We can add an
vDSO layer so if the CPU has ICACHET:

int vdso_loongarch_flush_icache(...)
{
  asm ("ibar 0");
  return 0;
}

And otherwise the vDSO wrapper invokes the real syscall.  I've
implemented the boot-time alternative runtime patching for vDSO at
https://lore.kernel.org/loongarch/20240816110717.10249-3-xry111@xry111.site/.

> The syscall resolves to a ibar for now, it should be revised when we have
> actual non-ICACHET support in kernel.

/* snip */

> diff --git a/arch/loongarch/include/asm/cacheflush.h b/arch/loongarch/include/asm/cacheflush.h
> index f8754d08a31ab07490717c31b9253871668b9a76..94f4a47f00860977db0b360965a22ff0a461c098 100644
> --- a/arch/loongarch/include/asm/cacheflush.h
> +++ b/arch/loongarch/include/asm/cacheflush.h
> @@ -80,6 +80,12 @@ static inline void flush_cache_line(int leaf, unsigned long addr)
>  	}
>  }
>  
> +/*
> + * Bits in sys_loongarch_flush_icache()'s flags argument.
> + */
> +#define SYS_LOONGARCH_FLUSH_ICACHE_LOCAL 1UL
> +#define SYS_LOONGARCH_FLUSH_ICACHE_ALL   (SYS_LOONGARCH_FLUSH_ICACHE_LOCAL)

Not a UAPI header so not usable by the user?  How would they specify
flags then?

If you meant to add them for UAPI, it would be very problematic.  When a
new cache type emerges in the hardware implementations, we need to grow
SYS_LOONGARCH_FLUSH_ICACHE_ALL in the UAPI header, but we cannot change
the already compiled JIT applications.  Thus all JIT applications have
to be recompiled with the latest UAPI header.  This just seems an
unnecessary severe burden to the packagers.

Instead IMO it's better not to expose so much details to the userspace.
Just remove the flags argument and flush all the icaches the kernel
knows, so with a new cache type the user (and distro) just need to
update or patch their kernel, w/o recompiling all JIT apps.


-- 
Xi Ruoyao <xry111@xry111.site>
School of Aerospace Science and Technology, Xidian University