On Wed, Sep 1, 2021 at 8:51 PM LIU Zhiwei <[email protected]> wrote:
>
> These variables should be target_ulong. If truncated to int,
> the bool conditions they indicate will be wrong.
>
> As satp is very important for Linux, this bug almost fails every boot.
Could you please describe which Linux configuration is broken? I have
a 64-bit 5.10 kernel and it boots fine.
Please add:
Fixes: 419ddf00ed78 ("target/riscv: Remove the hardcoded SATP_MODE macro")
> Signed-off-by: LIU Zhiwei <[email protected]>
> ---
> target/riscv/csr.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/target/riscv/csr.c b/target/riscv/csr.c
> index 50a2c3a3b4..ba9818f6a5 100644
> --- a/target/riscv/csr.c
> +++ b/target/riscv/csr.c
> @@ -986,7 +986,7 @@ static RISCVException read_satp(CPURISCVState *env, int
> csrno,
> static RISCVException write_satp(CPURISCVState *env, int csrno,
> target_ulong val)
> {
> - int vm, mask, asid;
> + target_ulong vm, mask, asid;
>
> if (!riscv_feature(env, RISCV_FEATURE_MMU)) {
> return RISCV_EXCP_NONE;
Reviewed-by: Bin Meng <[email protected]>
