On Fri, Oct 22, 2021 at 12:23:20PM +0100, Catalin Marinas wrote:
On Tue, Oct 19, 2021 at 06:22:12PM +0100, Mark Brown wrote:
-static void set_sve_default_vl(int val) +static void set_default_vl(enum vec_type type, int val) {
- WRITE_ONCE(__sve_default_vl, val);
- WRITE_ONCE(vl_config[type].__default_vl, val);
}
Same here, it should probably be static inline.
TBH I'd be worried if setting the default VL is sufficiently performance critical that we have to insist on it being inlined - it seems as well to let the compiler make the choice if it wants to. It's only useful for the gets because we need a stub.