Emilio G. Cota <c...@braap.org> writes:
> This is a thread-local variable and therefore all changes
> to it will be seen in order by the owning thread. There is
> no need for it to be volatile.
>
> Signed-off-by: Emilio G. Cota <c...@braap.org>
> ---
> translate-all.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/translate-all.c b/translate-all.c
> index 901a35e..31239db 100644
> --- a/translate-all.c
> +++ b/translate-all.c
> @@ -130,7 +130,7 @@ static void *l1_map[V_L1_SIZE];
> TCGContext tcg_ctx;
>
> /* translation block context */
> -__thread volatile int have_tb_lock;
> +__thread int have_tb_lock;
Reviewed-by: Alex Bennée <alex.ben...@linaro.org>
but really should be folded into the original patches.
>
> void tb_lock(void)
> {
--
Alex Bennée
