On Mon, Mar 6, 2023 at 11:15 AM Randy Dunlap <rdunlap@xxxxxxxxxxxxx> wrote: > > Hi, > > > On 3/5/23 19:12, Huacai Chen wrote: > > +void kernel_fpu_begin(void) > > +{ > > + if(this_cpu_read(in_kernel_fpu)) > > if ( > > + return; > > + > > + preempt_disable(); > > + this_cpu_write(in_kernel_fpu, true); > > + > > + if (!is_fpu_owner()) > > + enable_fpu(); > > + else > > + _save_fp(¤t->thread.fpu); > > +} > > +EXPORT_SYMBOL(kernel_fpu_begin); > > + > > +void kernel_fpu_end(void) > > +{ > > + if(!this_cpu_read(in_kernel_fpu)) > > if ( > > i.e., add a space after "if". OK, thanks. Huacai > > > + return; > > + > > + if (!is_fpu_owner()) > > + disable_fpu(); > > + else > > + _restore_fp(¤t->thread.fpu); > > + > > + this_cpu_write(in_kernel_fpu, false); > > + preempt_enable(); > > +} > > +EXPORT_SYMBOL(kernel_fpu_end); > > -- > ~Randy