|
From: | Richard Henderson |
Subject: | Re: [PATCH 5/5] target/arm: only perform TCG cpu and machine inits if TCG enabled |
Date: | Fri, 16 Dec 2022 16:20:09 -0800 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.4.2 |
On 12/16/22 13:29, Fabiano Rosas wrote:
- /* - * Misaligned thumb pc is architecturally impossible. - * We have an assert in thumb_tr_translate_insn to verify this. - * Fail an incoming migrate to avoid this assert. - */ - if (!is_a64(env) && env->thumb && (env->regs[15] & 1)) { - return -1; - } + /* + * Misaligned thumb pc is architecturally impossible. + * We have an assert in thumb_tr_translate_insn to verify this. + * Fail an incoming migrate to avoid this assert. + */ + if (!is_a64(env) && env->thumb && (env->regs[15] & 1)) { + return -1; + }
This is a sanity check rejecting malformed vmsave. While hw virt won't have the same assert as mentioned in the comment, it won't be happy and will raise some sort of cpu exception later. I think it's better to reject the bad vmload early. I suppose we could expand the comment to that effect, so that it doesn't appear to be wholly tcg inspired.
Otherwise, Reviewed-by: Richard Henderson <richard.henderson@linaro.org> r~
[Prev in Thread] | Current Thread | [Next in Thread] |