On Tue, Mar 10, 2026 at 10:08 PM Brian Cain <[email protected]>
wrote:

> From: Brian Cain <[email protected]>
>
> Signed-off-by: Brian Cain <[email protected]>
> ---
>  target/hexagon/sys_macros.h |   3 +
>  target/hexagon/op_helper.c  | 117 ++++++++++++++++++++++++++++++++++++
>  2 files changed, 120 insertions(+)
>
> --- a/target/hexagon/op_helper.c
> +++ b/target/hexagon/op_helper.c
> @@ -1232,6 +1235,120 @@ void HELPER(modify_ssr)(CPUHexagonState *env,
> uint32_t new, uint32_t old)
> +            qemu_log_mask(LOG_GUEST_ERROR,
> +                          "Double k0lock at PC: 0x%x, thread may hang\n",
>

Don't use %x


> +        qemu_log_mask(LOG_GUEST_ERROR,
> +                      "thread %d attempted to unlock k0 without having
> the "
> +                      "lock, k0_lock state = %d, syscfg:k0 = %d\n",
> +                      env->threadId, env->k0_lock_state,
> +                      GET_SYSCFG_FIELD(SYSCFG_K0LOCK, syscfg));
>

Don't use %d

The patch description mentions TLB, but this patch only implements k0

Otherwise
Reviewed-by: Taylor Simpson <[email protected]>

Reply via email to