Hello,

I noticed that we give thread_info.cpu two different names in the
assembly, load it as a wider type, and store it suboptimally.

v2 is split for easy backporting.

Thanks.

v2:
* transform [v1 2/3] into [v2 1-2/4] to simplify backports [Alex]
* tested and reviewed by Alex
v1: https://lore.kernel.org/lkml/20250722160556.2216925-2-rkrcmar@ventanamicro.com/

Radim Krčmář (4):
  riscv: use lw when reading int cpu in new_vmalloc_check
  riscv: use lw when reading int cpu in asm_per_cpu
  riscv: use TASK_TI_CPU instead of TASK_TI_CPU_NUM
  riscv: pack rv64 thread_info better

 arch/riscv/include/asm/asm.h         | 2 +-
 arch/riscv/include/asm/thread_info.h | 2 +-
 arch/riscv/kernel/asm-offsets.c      | 1 -
 arch/riscv/kernel/entry.S            | 2 +-
 4 files changed, 3 insertions(+), 4 deletions(-)

-- 
2.50.0

Hi Radim,

On 7/25/25 18:54, Radim Krčmář wrote:
> Hello,
>
> I noticed that we give thread_info.cpu two different names in the
> assembly, load it as a wider type, and store it suboptimally.
>
> v2 is split for easy backporting.
>
> Thanks.
>
> v2:
> * transform [v1 2/3] into [v2 1-2/4] to simplify backports [Alex]
> * tested and reviewed by Alex
> v1: https://lore.kernel.org/lkml/20250722160556.2216925-2-rkrcmar@ventanamicro.com/
>
> Radim Krčmář (4):
>    riscv: use lw when reading int cpu in new_vmalloc_check
>    riscv: use lw when reading int cpu in asm_per_cpu
>    riscv: use TASK_TI_CPU instead of TASK_TI_CPU_NUM
>    riscv: pack rv64 thread_info better
>
>   arch/riscv/include/asm/asm.h         | 2 +-
>   arch/riscv/include/asm/thread_info.h | 2 +-
>   arch/riscv/kernel/asm-offsets.c      | 1 -
>   arch/riscv/kernel/entry.S            | 2 +-
>   4 files changed, 3 insertions(+), 4 deletions(-)
>

Thanks for the quick respin, I'll take that for 6.17.

Thanks,

Alex