Re: [PATCH] target/riscv: Use TARGET_FMT_lx for env->mhartid

2023-01-15 Thread Alistair Francis
On Tue, Jan 10, 2023 at 1:28 AM Bin Meng wrote: > > env->mhartid is currently casted to long before printed, which drops > the high 32-bit for rv64 on 32-bit host. Use TARGET_FMT_lx instead. > > Signed-off-by: Bin Meng Thanks! Applied to riscv-to-apply.next Alistair > --- > > target/riscv/cp

Re: [PATCH] target/riscv: Use TARGET_FMT_lx for env->mhartid

2023-01-15 Thread Alistair Francis
On Tue, Jan 10, 2023 at 1:28 AM Bin Meng wrote: > > env->mhartid is currently casted to long before printed, which drops > the high 32-bit for rv64 on 32-bit host. Use TARGET_FMT_lx instead. > > Signed-off-by: Bin Meng Reviewed-by: Alistair Francis Alistair > --- > > target/riscv/cpu.c | 6 +

Re: [PATCH] target/riscv: Use TARGET_FMT_lx for env->mhartid

2023-01-09 Thread Richard Henderson
On 1/9/23 07:47, Philippe Mathieu-Daudé wrote: On 9/1/23 16:26, Bin Meng wrote: env->mhartid is currently casted to long before printed, which drops the high 32-bit for rv64 on 32-bit host. Use TARGET_FMT_lx instead. Oh, a 32-bit host user! Signed-off-by: Bin Meng ---   target/riscv/cpu.c

Re: [PATCH] target/riscv: Use TARGET_FMT_lx for env->mhartid

2023-01-09 Thread Bin Meng
On Mon, Jan 9, 2023 at 11:48 PM Philippe Mathieu-Daudé wrote: > > On 9/1/23 16:26, Bin Meng wrote: > > env->mhartid is currently casted to long before printed, which drops > > the high 32-bit for rv64 on 32-bit host. Use TARGET_FMT_lx instead. > > Oh, a 32-bit host user! > > > Signed-off-by: Bin M

Re: [PATCH] target/riscv: Use TARGET_FMT_lx for env->mhartid

2023-01-09 Thread Philippe Mathieu-Daudé
On 9/1/23 16:26, Bin Meng wrote: env->mhartid is currently casted to long before printed, which drops the high 32-bit for rv64 on 32-bit host. Use TARGET_FMT_lx instead. Oh, a 32-bit host user! Signed-off-by: Bin Meng --- target/riscv/cpu.c | 6 +++--- 1 file changed, 3 insertions(+), 3

[PATCH] target/riscv: Use TARGET_FMT_lx for env->mhartid

2023-01-09 Thread Bin Meng
env->mhartid is currently casted to long before printed, which drops the high 32-bit for rv64 on 32-bit host. Use TARGET_FMT_lx instead. Signed-off-by: Bin Meng --- target/riscv/cpu.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/target/riscv/cpu.c b/target/riscv/cpu