[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 09/22] monitor: disable "info jit" and "info opc
From: |
Emilio G. Cota |
Subject: |
Re: [Qemu-devel] [PATCH 09/22] monitor: disable "info jit" and "info opcount" if !TCG |
Date: |
Sun, 9 Jul 2017 04:13:51 -0400 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
On Thu, Jul 06, 2017 at 16:26:52 -0400, Emilio G. Cota wrote:
> On Tue, Jul 04, 2017 at 13:12:02 +0200, Paolo Bonzini wrote:
> > Reviewed-by: Richard Henderson <address@hidden>
> > Signed-off-by: Paolo Bonzini <address@hidden>
(snip)
> > +++ b/accel/tcg/translate-all.c
> > @@ -1851,6 +1851,11 @@ void dump_exec_info(FILE *f, fprintf_function
> > cpu_fprintf)
> >
> > tb_lock();
> >
> > + if (!tcg_enabled()) {
> > + cpu_fprintf(f, "TCG not enabled\n");
> > + return;
> > + }
>
> Sorry I missed this before it was merged, but:
>
> - We're returning with tb_lock held
> - This check is redundant; see b7da97eef ("monitor: Check whether TCG
> is enabled before running the "info jit" code") which was merged
> in April (hmp_info_jit is the only caller of dump_exec_info).
>
> I suggest we get rid of the check, although moving it above tb_lock is
> also OK with me.
Patch here:
https://lists.nongnu.org/archive/html/qemu-devel/2017-07/msg02072.html
This other patch fixes another issue introduced in this series, please
have a look as well:
https://lists.nongnu.org/archive/html/qemu-devel/2017-07/msg02058.html
Cheers,
Emilio
- [Qemu-devel] [PATCH 02/22] configure: early test for supported targets, (continued)
- [Qemu-devel] [PATCH 02/22] configure: early test for supported targets, Paolo Bonzini, 2017/07/04
- [Qemu-devel] [PATCH 04/22] vl: convert -tb-size to qemu_strtoul, Paolo Bonzini, 2017/07/04
- [Qemu-devel] [PATCH 01/22] configure: factor out list of supported Xen/KVM/HAX targets, Paolo Bonzini, 2017/07/04
- [Qemu-devel] [PATCH 06/22] tcg: move page_size_init() function, Paolo Bonzini, 2017/07/04
- [Qemu-devel] [PATCH 05/22] vl: add tcg_enabled() for tcg related code, Paolo Bonzini, 2017/07/04
- [Qemu-devel] [PATCH 07/22] tcg: tcg_handle_interrupt() function, Paolo Bonzini, 2017/07/04
- [Qemu-devel] [PATCH 08/22] tcg: make tcg_allowed global, Paolo Bonzini, 2017/07/04
- [Qemu-devel] [PATCH 09/22] monitor: disable "info jit" and "info opcount" if !TCG, Paolo Bonzini, 2017/07/04
- [Qemu-devel] [PATCH 10/22] vapic: use tcg_enabled, Paolo Bonzini, 2017/07/04
- [Qemu-devel] [PATCH 11/22] tcg: add the tcg-stub.c file into accel/stubs/, Paolo Bonzini, 2017/07/04
- [Qemu-devel] [PATCH 12/22] tcg: move tb_lock out of translate-all.h, Paolo Bonzini, 2017/07/04
- [Qemu-devel] [PATCH 13/22] exec: elide calls to tb_lock and tb_unlock, Paolo Bonzini, 2017/07/04
- [Qemu-devel] [PATCH 14/22] tcg: add CONFIG_TCG guards in headers, Paolo Bonzini, 2017/07/04
- [Qemu-devel] [PATCH 15/22] tcg: add the CONFIG_TCG into Makefiles, Paolo Bonzini, 2017/07/04
- [Qemu-devel] [PATCH 16/22] target/i386: move cpu_sync_bndcs_hflags() function, Paolo Bonzini, 2017/07/04
- [Qemu-devel] [PATCH 17/22] target/i386: make cpu_get_fp80()/cpu_set_fp80() static, Paolo Bonzini, 2017/07/04