[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC PATCH V7 14/19] cpu: introduce tlb_flush*_all.
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [RFC PATCH V7 14/19] cpu: introduce tlb_flush*_all. |
Date: |
Mon, 10 Aug 2015 17:54:11 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.1.0 |
On 10/08/2015 17:27, address@hidden wrote:
> From: KONRAD Frederic <address@hidden>
>
> Some architectures allow to flush the tlb of other VCPUs. This is not a
> problem
> when we have only one thread for all VCPUs but it definitely needs to be an
> asynchronous work when we are in true multithreaded work.
>
> TODO: Some test case, I fear some bad results in case a VCPUs execute a
> barrier
> or something like that.
>
> Signed-off-by: KONRAD Frederic <address@hidden>
> ---
> cputlb.c | 76
> +++++++++++++++++++++++++++++++++++++++++++++++++
> include/exec/exec-all.h | 2 ++
> 2 files changed, 78 insertions(+)
I still believe this should be a target-specific change. This would
also make it easier to do the remote TLB flush synchronously, as is the
case on ARM (if I understand correctly).
Paolo
> diff --git a/cputlb.c b/cputlb.c
> index 79fff1c..e5853fd 100644
> --- a/cputlb.c
> +++ b/cputlb.c
> @@ -72,6 +72,45 @@ void tlb_flush(CPUState *cpu, int flush_global)
> tlb_flush_count++;
> }
>
> +struct TLBFlushParams {
> + CPUState *cpu;
> + int flush_global;
> +};
> +
> +static void tlb_flush_async_work(void *opaque)
> +{
> + struct TLBFlushParams *params = opaque;
> +
> + tlb_flush(params->cpu, params->flush_global);
> + g_free(params);
> +}
> +
> +void tlb_flush_all(int flush_global)
> +{
> + CPUState *cpu;
> + struct TLBFlushParams *params;
> +
> +#if 0 /* MTTCG */
> + CPU_FOREACH(cpu) {
> + tlb_flush(cpu, flush_global);
> + }
> +#else
> + CPU_FOREACH(cpu) {
> + if (qemu_cpu_is_self(cpu)) {
> + /* async_run_on_cpu handle this case but this just avoid a malloc
> + * here.
> + */
> + tlb_flush(cpu, flush_global);
> + } else {
> + params = g_malloc(sizeof(struct TLBFlushParams));
> + params->cpu = cpu;
> + params->flush_global = flush_global;
> + async_run_on_cpu(cpu, tlb_flush_async_work, params);
> + }
> + }
> +#endif /* MTTCG */
> +}
> +
> static inline void tlb_flush_entry(CPUTLBEntry *tlb_entry, target_ulong addr)
> {
> if (addr == (tlb_entry->addr_read &
> @@ -124,6 +163,43 @@ void tlb_flush_page(CPUState *cpu, target_ulong addr)
> tb_flush_jmp_cache(cpu, addr);
> }
>
> +struct TLBFlushPageParams {
> + CPUState *cpu;
> + target_ulong addr;
> +};
> +
> +static void tlb_flush_page_async_work(void *opaque)
> +{
> + struct TLBFlushPageParams *params = opaque;
> +
> + tlb_flush_page(params->cpu, params->addr);
> + g_free(params);
> +}
> +
> +void tlb_flush_page_all(target_ulong addr)
> +{
> + CPUState *cpu;
> + struct TLBFlushPageParams *params;
> +
> + CPU_FOREACH(cpu) {
> +#if 0 /* !MTTCG */
> + tlb_flush_page(cpu, addr);
> +#else
> + if (qemu_cpu_is_self(cpu)) {
> + /* async_run_on_cpu handle this case but this just avoid a malloc
> + * here.
> + */
> + tlb_flush_page(cpu, addr);
> + } else {
> + params = g_malloc(sizeof(struct TLBFlushPageParams));
> + params->cpu = cpu;
> + params->addr = addr;
> + async_run_on_cpu(cpu, tlb_flush_page_async_work, params);
> + }
> +#endif /* MTTCG */
> + }
> +}
> +
> /* update the TLBs so that writes to code in the virtual page 'addr'
> can be detected */
> void tlb_protect_code(ram_addr_t ram_addr)
> diff --git a/include/exec/exec-all.h b/include/exec/exec-all.h
> index 9f1c1cb..e9512df 100644
> --- a/include/exec/exec-all.h
> +++ b/include/exec/exec-all.h
> @@ -97,7 +97,9 @@ bool qemu_in_vcpu_thread(void);
> void cpu_reload_memory_map(CPUState *cpu);
> void tcg_cpu_address_space_init(CPUState *cpu, AddressSpace *as);
> /* cputlb.c */
> +void tlb_flush_page_all(target_ulong addr);
> void tlb_flush_page(CPUState *cpu, target_ulong addr);
> +void tlb_flush_all(int flush_global);
> void tlb_flush(CPUState *cpu, int flush_global);
> void tlb_set_page(CPUState *cpu, target_ulong vaddr,
> hwaddr paddr, int prot,
>
- Re: [Qemu-devel] [RFC PATCH V7 09/19] Drop global lock during TCG code execution, (continued)
[Qemu-devel] [RFC PATCH V7 11/19] tcg: switch on multithread., fred . konrad, 2015/08/10
[Qemu-devel] [RFC PATCH V7 13/19] add a callback when tb_invalidate is called., fred . konrad, 2015/08/10
[Qemu-devel] [RFC PATCH V7 10/19] cpu: remove exit_request global., fred . konrad, 2015/08/10
[Qemu-devel] [RFC PATCH V7 12/19] Use atomic cmpxchg to atomically check the exclusive value in a STREX, fred . konrad, 2015/08/10
[Qemu-devel] [RFC PATCH V7 14/19] cpu: introduce tlb_flush*_all., fred . konrad, 2015/08/10
- Re: [Qemu-devel] [RFC PATCH V7 14/19] cpu: introduce tlb_flush*_all.,
Paolo Bonzini <=
[Qemu-devel] [RFC PATCH V7 15/19] arm: use tlb_flush*_all, fred . konrad, 2015/08/10
[Qemu-devel] [RFC PATCH V7 18/19] mttcg: signal the associated cpu anyway., fred . konrad, 2015/08/10
[Qemu-devel] [RFC PATCH V7 16/19] translate-all: introduces tb_flush_safe., fred . konrad, 2015/08/10
[Qemu-devel] [RFC PATCH V7 17/19] translate-all: (wip) use tb_flush_safe when we can't alloc more tb., fred . konrad, 2015/08/10