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.
Attachment:
signature.asc
Description: PGP signature
