Re: [PATCH -next v19 01/24] riscv: Rename __switch_to_aux() -> fpu

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, May 12, 2023 at 6:56 AM Palmer Dabbelt <palmer@xxxxxxxxxxx> wrote:
>
> On Tue, 09 May 2023 03:30:10 PDT (-0700), andy.chiu@xxxxxxxxxx wrote:
> > From: Guo Ren <ren_guo@xxxxxxxxx>
> >
> > The name of __switch_to_aux() is not clear and rename it with the
> > determine function: __switch_to_fpu(). Next we could add other regs'
> > switch.
> >
> > Signed-off-by: Guo Ren <ren_guo@xxxxxxxxx>
> > Signed-off-by: Guo Ren <guoren@xxxxxxxxxxxxxxxxx>
> > Signed-off-by: Greentime Hu <greentime.hu@xxxxxxxxxx>
> > Reviewed-by: Anup Patel <anup@xxxxxxxxxxxxxx>
> > Reviewed-by: Palmer Dabbelt <palmer@xxxxxxxxxxxx>
> > Signed-off-by: Andy Chiu <andy.chiu@xxxxxxxxxx>
> > Tested-by: Heiko Stuebner <heiko.stuebner@xxxxxxxx>
> > Reviewed-by: Heiko Stuebner <heiko.stuebner@xxxxxxxx>
> > Reviewed-by: Conor Dooley <conor.dooley@xxxxxxxxxxxxx>
> > ---
> >  arch/riscv/include/asm/switch_to.h | 6 +++---
> >  1 file changed, 3 insertions(+), 3 deletions(-)
> >
> > diff --git a/arch/riscv/include/asm/switch_to.h b/arch/riscv/include/asm/switch_to.h
> > index 60f8ca01d36e..4b96b13dee27 100644
> > --- a/arch/riscv/include/asm/switch_to.h
> > +++ b/arch/riscv/include/asm/switch_to.h
> > @@ -46,7 +46,7 @@ static inline void fstate_restore(struct task_struct *task,
> >       }
> >  }
> >
> > -static inline void __switch_to_aux(struct task_struct *prev,
> > +static inline void __switch_to_fpu(struct task_struct *prev,
> >                                  struct task_struct *next)
> >  {
> >       struct pt_regs *regs;
> > @@ -66,7 +66,7 @@ static __always_inline bool has_fpu(void)
> >  static __always_inline bool has_fpu(void) { return false; }
> >  #define fstate_save(task, regs) do { } while (0)
> >  #define fstate_restore(task, regs) do { } while (0)
> > -#define __switch_to_aux(__prev, __next) do { } while (0)
> > +#define __switch_to_fpu(__prev, __next) do { } while (0)
> >  #endif
> >
> >  extern struct task_struct *__switch_to(struct task_struct *,
> > @@ -77,7 +77,7 @@ do {                                                        \
> >       struct task_struct *__prev = (prev);            \
> >       struct task_struct *__next = (next);            \
> >       if (has_fpu())                                  \
> > -             __switch_to_aux(__prev, __next);        \
> > +             __switch_to_fpu(__prev, __next);        \
> >       ((last) = __switch_to(__prev, __next));         \
> >  } while (0)
>
> Reviewed-by: Palmer Dabbelt <palmer@xxxxxxxxxxxx>

I noticed that your R-b has been here for a while (at least since v13
where I started to handle this series). Do you want me to keep the
original, or the last one?

Thanks,
Andy




[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux