diff options
author | ShihPo Hung | 2020-01-15 07:17:31 +0100 |
---|---|---|
committer | Palmer Dabbelt | 2020-01-16 19:02:58 +0100 |
commit | 613fa160e19abe8e1fe44423fcfa8ec73d3d48e5 (patch) | |
tree | 2ab0fc22e44548385a5fda5c5f7afaca5b106e93 /target/sh4/helper.c | |
parent | riscv: Set xPIE to 1 after xRET (diff) | |
download | qemu-613fa160e19abe8e1fe44423fcfa8ec73d3d48e5.tar.gz qemu-613fa160e19abe8e1fe44423fcfa8ec73d3d48e5.tar.xz qemu-613fa160e19abe8e1fe44423fcfa8ec73d3d48e5.zip |
target/riscv: Fix tb->flags FS status
It was found that running libquantum on riscv-linux qemu produced an
incorrect result. After investigation, FP registers are not saved
during context switch due to incorrect mstatus.FS.
In current implementation tb->flags merges all non-disabled state to
dirty. This means the code in mark_fs_dirty in translate.c that
handles initial and clean states is unreachable.
This patch fixes it and is successfully tested with:
libquantum
Thanks to Richard for pointing out the actual bug.
v3: remove the redundant condition
v2: root cause FS problem
Suggested-by: Richard Henderson <richard.henderson@linaro.org>
Signed-off-by: ShihPo Hung <shihpo.hung@sifive.com>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Signed-off-by: Palmer Dabbelt <palmerdabbelt@google.com>
Diffstat (limited to 'target/sh4/helper.c')
0 files changed, 0 insertions, 0 deletions