[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v2 1/3] compiler: define QEMU_CACHELINE_SIZE
From: |
Emilio G. Cota |
Subject: |
[Qemu-devel] [PATCH v2 1/3] compiler: define QEMU_CACHELINE_SIZE |
Date: |
Mon, 5 Jun 2017 18:49:37 -0400 |
This is a constant used as a hint for padding structs to hopefully avoid
false cache line sharing.
The constant can be set at configure time by defining QEMU_CACHELINE_SIZE
via --extra-cflags. If not set there, we try to obtain the value from
the machine running the configure script. If we fail, we default to
reasonable values, i.e. 128 bytes for ppc64 and 64 bytes for all others.
Note: the configure script only picks up the cache line size when run
on Linux hosts because I have no other platforms (e.g. Windows, BSD's)
to test on.
Signed-off-by: Emilio G. Cota <address@hidden>
---
configure | 38 ++++++++++++++++++++++++++++++++++++++
include/qemu/compiler.h | 17 +++++++++++++++++
2 files changed, 55 insertions(+)
diff --git a/configure b/configure
index 13e040d..6a68cb2 100755
--- a/configure
+++ b/configure
@@ -4832,6 +4832,41 @@ EOF
fi
fi
+# Find out the size of a cache line on the host
+# TODO: support more platforms
+cat > $TMPC<<EOF
+#ifdef __linux__
+
+#include <stdio.h>
+
+#define SYSFS "/sys/devices/system/cpu/cpu0/cache/index0/coherency_line_size"
+
+int main(int argc, char *argv[])
+{
+ unsigned int size;
+ FILE *fp;
+
+ fp = fopen(SYSFS, "r");
+ if (fp == NULL) {
+ return -1;
+ }
+ if (!fscanf(fp, "%u", &size)) {
+ return -1;
+ }
+ return size;
+}
+#else
+#error Cannot find host cache line size
+#endif
+EOF
+
+host_cacheline_size=0
+if compile_prog "" "" ; then
+ ./$TMPE
+ host_cacheline_size=$?
+fi
+
+
##########################################
# check for _Static_assert()
@@ -5284,6 +5319,9 @@ fi
if test "$bigendian" = "yes" ; then
echo "HOST_WORDS_BIGENDIAN=y" >> $config_host_mak
fi
+if test "$host_cacheline_size" -gt 0 ; then
+ echo "HOST_CACHELINE_SIZE=$host_cacheline_size" >> $config_host_mak
+fi
if test "$mingw32" = "yes" ; then
echo "CONFIG_WIN32=y" >> $config_host_mak
rc_version=$(cat $source_path/VERSION)
diff --git a/include/qemu/compiler.h b/include/qemu/compiler.h
index 340e5fd..178d831 100644
--- a/include/qemu/compiler.h
+++ b/include/qemu/compiler.h
@@ -40,6 +40,23 @@
# define QEMU_PACKED __attribute__((packed))
#endif
+/*
+ * Cache line size of the host. Can be overriden.
+ * Note that this is just a compile-time hint to hopefully avoid false sharing
+ * of cache lines; code must be correct regardless of the constant's value.
+ */
+#ifndef QEMU_CACHELINE_SIZE
+# ifdef HOST_CACHELINE_SIZE
+# define QEMU_CACHELINE_SIZE HOST_CACHELINE_SIZE
+# else
+# if defined(__powerpc64__)
+# define QEMU_CACHELINE_SIZE 128
+# else
+# define QEMU_CACHELINE_SIZE 64
+# endif
+# endif
+#endif
+
#define QEMU_ALIGNED(X) __attribute__((aligned(X)))
#ifndef glue
--
2.7.4
- [Qemu-devel] [PATCH v2 0/3] tcg: allocate TB structs preceding translated code, Emilio G. Cota, 2017/06/05
- [Qemu-devel] [PATCH v2 1/3] compiler: define QEMU_CACHELINE_SIZE,
Emilio G. Cota <=
- Re: [Qemu-devel] [PATCH v2 1/3] compiler: define QEMU_CACHELINE_SIZE, Pranith Kumar, 2017/06/06
- Re: [Qemu-devel] [PATCH v2 1/3] compiler: define QEMU_CACHELINE_SIZE, Richard Henderson, 2017/06/06
- Re: [Qemu-devel] [PATCH v2 1/3] compiler: define QEMU_CACHELINE_SIZE, Emilio G. Cota, 2017/06/06
- Re: [Qemu-devel] [PATCH v2 1/3] compiler: define QEMU_CACHELINE_SIZE, Richard Henderson, 2017/06/06
- Re: [Qemu-devel] [PATCH v2 1/3] compiler: define QEMU_CACHELINE_SIZE, Geert Martin Ijewski, 2017/06/06
- Re: [Qemu-devel] [PATCH v2 1/3] compiler: define QEMU_CACHELINE_SIZE, Emilio G. Cota, 2017/06/06
- Re: [Qemu-devel] [PATCH v2 1/3] compiler: define QEMU_CACHELINE_SIZE, Geert Martin Ijewski, 2017/06/06
[Qemu-devel] [PATCH v2 2/3] tests: use QEMU_CACHELINE_SIZE instead of hard-coding it, Emilio G. Cota, 2017/06/05
[Qemu-devel] [PATCH v2 3/3] tcg: allocate TB structs before the corresponding translated code, Emilio G. Cota, 2017/06/05