[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] tcg: fix --disable-tcg build breakage introduce
From: |
Emilio G. Cota |
Subject: |
Re: [Qemu-devel] [PATCH] tcg: fix --disable-tcg build breakage introduced by tb_lock removal |
Date: |
Fri, 22 Jun 2018 13:03:49 -0400 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
On Fri, Jun 22, 2018 at 17:24:33 +0100, Peter Maydell wrote:
> On 22 June 2018 at 17:16, Emilio G. Cota <address@hidden> wrote:
> It's probably the --enable-debug that makes the difference:
> for instance cpu_reloading_memory_map() is referenced from
> tcg_commit(), which is only called from within an "if (tcg_enabled())"
> guard; with debug disabled the compiler probably figures out that
> tcg_commit() is unreachable and doesn't put it in the .o file.
Yes, I get the same errors with both --enable-debug and --disable-tcg.
> Possibly this has always been broken and wasn't a regression?
> If so I guess we should apply your patch and then fix this
> separately...
v2.12.0 doesn't build either with the options above, so the
regression is fixed with the patch.
I'll send a v2 patch as a separate thread with an updated commit
message to reflect the above.
Thanks,
Emilio