[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 20/25] tcg: Save insn data and use it in cpu_
From: |
Richard Henderson |
Subject: |
Re: [Qemu-devel] [PATCH v3 20/25] tcg: Save insn data and use it in cpu_restore_state_from_tb |
Date: |
Fri, 25 Sep 2015 16:05:00 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.2.0 |
On 09/25/2015 02:10 PM, Aurelien Jarno wrote:
>> + if (more)
>> + byte |= 0x80;
>
> You are missing braces here.
Gah. I thought I fixed that...
> Given we save both the host and the guest PC in this structure, one
> obvious optimization would be to skip saving data for host instructions
> which can not generate exception. It means that all the TCG ops in this
> instruction do not generate exceptions either. We can easily test that
> for all TCG instructions except all by looking at the
> TCG_OPF_SIDE_EFFECTS flag. For the call op, we have to look at the
> TCG_CALL_NO_SIDE_EFFECTS flag, even if it doesn't necessary means the
> helper might generate exception.
>
> That should significantly save space on load/store architectures. That
> said we can probably do that in a latter time.
Yes, Alex Bennee mentioned this during round 1. I decided to not try to do
that all at once.
When we do get there, we also have to add an additional column for icount.
It's currently inferred that each entry is 1 insn. This will expand the size
of the table in any case that every insn might raise an exception, but I expect
the normal case to be a slight decrease.
r~
- Re: [Qemu-devel] [PATCH v3 12/25] target-sparc: Tidy gen_branch_a interface, (continued)
- [Qemu-devel] [PATCH v3 11/25] target-cris: Mirror gen_opc_pc into insn_start, Richard Henderson, 2015/09/22
- [Qemu-devel] [PATCH v3 13/25] target-sparc: Split out gen_branch_n, Richard Henderson, 2015/09/22
- [Qemu-devel] [PATCH v3 16/25] tcg: Merge cpu_gen_code into tb_gen_code, Richard Henderson, 2015/09/22
- [Qemu-devel] [PATCH v3 20/25] tcg: Save insn data and use it in cpu_restore_state_from_tb, Richard Henderson, 2015/09/22
- [Qemu-devel] [PATCH v3 19/25] tcg: Pass data argument to restore_state_to_opc, Richard Henderson, 2015/09/22
- [Qemu-devel] [PATCH v3 22/25] tcg: Remove tcg_gen_code_search_pc, Richard Henderson, 2015/09/22
- [Qemu-devel] [PATCH v3 18/25] tcg: Add TCG_MAX_INSNS, Richard Henderson, 2015/09/22
- [Qemu-devel] [PATCH v3 23/25] tcg: Emit prologue to the beginning of code_gen_buffer, Richard Henderson, 2015/09/22