| commit | d40d3da00c10f0169a26985ecb65033bff536f2c | |
| author | Emilio G. Cota <cota@braap.org> | |
| Sun, 9 Jul 2017 07:49:54 +0000 (9 03:49 -0400) | ||
| committer | Paolo Bonzini <pbonzini@redhat.com> | |
| Fri, 14 Jul 2017 10:04:42 +0000 (14 12:04 +0200) | ||
| tree | 4515110b83fddadc029af20db60e7760ff0c7025 | treesnapshot (tar.gz zip) |
| parent | f940488feaca394ccc68e0595b4c3a9017f16e45 | commitdiff |
translate-all: remove redundant !tcg_enabled check in dump_exec_info
This check is redundant because it is already performed by the only
caller of dump_exec_info -- the caller was updated by b7da97eef
("monitor: Check whether TCG is enabled before running the "info jit"
code").
Checking twice wouldn't necessarily be too bad, but here the check also
returns with tb_lock held. So we can either do the check before tb_lock is
acquired, or just get rid of it. Given that it is redundant, I am going
for the latter option.
Signed-off-by: Emilio G. Cota <cota@braap.org>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
This check is redundant because it is already performed by the only
caller of dump_exec_info -- the caller was updated by b7da97eef
("monitor: Check whether TCG is enabled before running the "info jit"
code").
Checking twice wouldn't necessarily be too bad, but here the check also
returns with tb_lock held. So we can either do the check before tb_lock is
acquired, or just get rid of it. Given that it is redundant, I am going
for the latter option.
Signed-off-by: Emilio G. Cota <cota@braap.org>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
| accel/tcg/translate-all.c | diffblobblamehistory |