perf auxtrace arm: Support compat_auxtrace_mmap__{read_head|write_tail}
authorLeo Yan <leo.yan@linaro.org>
Sun, 29 Aug 2021 10:22:38 +0000 (18:22 +0800)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Tue, 31 Aug 2021 18:12:00 +0000 (15:12 -0300)
When the tool runs with compat mode on Arm platform, the kernel is in
64-bit mode and user space is in 32-bit mode; the user space can use
instructions "ldrd" and "strd" for 64-bit value atomicity.

This patch adds compat_auxtrace_mmap__{read_head|write_tail} for arm
building, it uses "ldrd" and "strd" instructions to ensure accessing
atomicity for aux head and tail.  The file arch/arm/util/auxtrace.c is
built for arm and arm64 building, these two functions are not needed for
arm64, so check the compiler macro "__arm__" to only include them for
arm building.

Signed-off-by: Leo Yan <leo.yan@linaro.org>
Reviewed-by: James Clark <james.clark@arm.com>
Tested-by: James Clark <james.clark@arm.com>
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Cc: Andi Kleen <ak@linux.intel.com>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: John Garry <john.garry@huawei.com>
Cc: Mark Rutland <mark.rutland@arm.com>
Cc: Mathieu Poirier <mathieu.poirier@linaro.org>
Cc: Mike Leach <mike.leach@linaro.org>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: "Russell King (oracle)" <linux@armlinux.org.uk>
Cc: Suzuki Poulouse <suzuki.poulose@arm.com>
Cc: Will Deacon <will@kernel.org>
Cc: coresight@lists.linaro.org
Cc: linux-arm-kernel@lists.infradead.org
Link: http://lore.kernel.org/lkml/20210829102238.19693-3-leo.yan@linaro.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/arch/arm/util/auxtrace.c

index b187bddbd01ac9576a862d0efbbb76296b8d7b12..c7c7ec0812d5aef4c4399ca3e86ecdf69d0c9815 100644 (file)
@@ -107,3 +107,35 @@ struct auxtrace_record
        *err = 0;
        return NULL;
 }
+
+#if defined(__arm__)
+u64 compat_auxtrace_mmap__read_head(struct auxtrace_mmap *mm)
+{
+       struct perf_event_mmap_page *pc = mm->userpg;
+       u64 result;
+
+       __asm__ __volatile__(
+"      ldrd    %0, %H0, [%1]"
+       : "=&r" (result)
+       : "r" (&pc->aux_head), "Qo" (pc->aux_head)
+       );
+
+       return result;
+}
+
+int compat_auxtrace_mmap__write_tail(struct auxtrace_mmap *mm, u64 tail)
+{
+       struct perf_event_mmap_page *pc = mm->userpg;
+
+       /* Ensure all reads are done before we write the tail out */
+       smp_mb();
+
+       __asm__ __volatile__(
+"      strd    %2, %H2, [%1]"
+       : "=Qo" (pc->aux_tail)
+       : "r" (&pc->aux_tail), "r" (tail)
+       );
+
+       return 0;
+}
+#endif