[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v6 15/19] target-arm/cpu: don't reset TLB struct
From: |
Alex Bennée |
Subject: |
Re: [Qemu-devel] [PATCH v6 15/19] target-arm/cpu: don't reset TLB structures, use cputlb to do it |
Date: |
Thu, 10 Nov 2016 18:08:41 +0000 |
User-agent: |
mu4e 0.9.17; emacs 25.1.50.16 |
Richard Henderson <address@hidden> writes:
> On 11/09/2016 03:57 PM, Alex Bennée wrote:
>> +#ifdef CONFIG_SOFTMMU
>> + memset(env, 0, offsetof(CPUARMState, tlb_table));
>> + tlb_flush(s, 0);
>> +#else
>> memset(env, 0, offsetof(CPUARMState, features));
>> +#endif
>
> I'd really prefer to see the tlb_flush be moved into parent_reset, so that we
> handle it identically for all targets.
Yeah I'll prepare a series to do that separate from MTTCG.
>
> As for the memset, do we really need to distinguish softmmu? I don't like you
> picking out a variable name within CPU_COMMON. Better to use empty struct
> markers, like the
>
> struct {} start_init_save;
>
> that x86 uses.
OK fair enough.
--
Alex Bennée
- Re: [Qemu-devel] [PATCH v6 12/19] cputlb: tweak qemu_ram_addr_from_host_nofail reporting, (continued)
- [Qemu-devel] [PATCH v6 15/19] target-arm/cpu: don't reset TLB structures, use cputlb to do it, Alex Bennée, 2016/11/09
- [Qemu-devel] [PATCH v6 16/19] target-arm: ensure BQL taken for ARM_CP_IO register access, Alex Bennée, 2016/11/09
- [Qemu-devel] [PATCH v6 18/19] target-arm: don't generate WFE/YIELD calls for MTTCG, Alex Bennée, 2016/11/09
- [Qemu-devel] [PATCH v6 14/19] target-arm/powerctl: defer cpu reset work to CPU context, Alex Bennée, 2016/11/09
- [Qemu-devel] [PATCH v6 13/19] cputlb: atomically update tlb fields used by tlb_reset_dirty, Alex Bennée, 2016/11/09