|
From: | Richard Henderson |
Subject: | Re: [PATCH v3 14/14] target/riscv: Compute mstatus.sd on demand |
Date: | Sun, 17 Oct 2021 22:31:53 -0700 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.13.0 |
On 10/17/21 9:52 PM, Alistair Francis wrote:
On Sun, Oct 17, 2021 at 3:32 AM Richard Henderson <richard.henderson@linaro.org> wrote:The position of this read-only field is dependent on the current cpu width. Rather than having to compute that difference in many places, compute it only on read. Signed-off-by: Richard Henderson <richard.henderson@linaro.org>This means that the value reported by riscv_cpu_dump_state() and GDB will both be incorrect though?
Yep. Missed those; should have added another accessor.Also, for the record, it changes the vmstate, but since a previous patch in the series bumped the version number for the split on misa, we can call all of a piece and ok.
r~
[Prev in Thread] | Current Thread | [Next in Thread] |