[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH v5 28/33] cputlb: make tlb_flush_by_mmuidx safe

From: Pranith Kumar
Subject: Re: [Qemu-devel] [PATCH v5 28/33] cputlb: make tlb_flush_by_mmuidx safe for MTTCG
Date: Tue, 01 Nov 2016 01:20:32 -0400

Hi Alex,

Alex Bennée writes:

> These flushes allow a per-mmuidx granularity to the TLB flushing and are
> currently only used by the ARM model. As it is possible to hammer the
> other vCPU threads with flushes (and build up long queues of identical
> flushes) we extend mechanism used for the global tlb_flush and set a
> bitmap describing all the pending flushes. The updates are done
> atomically to avoid corruption of the bitmap but repeating a flush is
> certainly not a problem.
> Signed-off-by: Alex Bennée <address@hidden>


>  static inline void tlb_flush_entry(CPUTLBEntry *tlb_entry, target_ulong addr)
> @@ -233,16 +288,50 @@ void tlb_flush_page(CPUState *cpu, target_ulong addr)
>      }
>  }
> -void tlb_flush_page_by_mmuidx(CPUState *cpu, target_ulong addr, ...)
> +/* As we are going to hijack the bottom bits of the page address for a
> + * mmuidx bit mask we need to fail to build if we can't do that
> + */
> +

FYI, this is causing a build error on my system with gcc 6.2.

  CC      aarch64-softmmu/cputlb.o
In file included from /home/pranith/devops/code/qemu/include/qemu/osdep.h:36:0,
                 from /home/pranith/devops/code/qemu/cputlb.c:20:
/home/pranith/devops/code/qemu/include/exec/cpu-all.h:196:26: error: 
braced-group within expression allowed only inside a function
 #define TARGET_PAGE_BITS ({ assert(target_page_bits_decided); \
/home/pranith/devops/code/qemu/include/qemu/compiler.h:89:54: note: in 
definition of macro ‘QEMU_BUILD_BUG_ON’
     typedef char glue(qemu_build_bug_on__,__LINE__)[(x)?-1:1] 
/home/pranith/devops/code/qemu/cputlb.c:293:34: note: in expansion of macro 
/home/pranith/devops/code/qemu/rules.mak:60: recipe for target 'cputlb.o' failed
make[1]: *** [cputlb.o] Error 1
Makefile:202: recipe for target 'subdir-aarch64-softmmu' failed
make: *** [subdir-aarch64-softmmu] Error 2


reply via email to

[Prev in Thread] Current Thread [Next in Thread]