[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <20221105072311.8214-1-leo.yan@linaro.org>
Date: Sat, 5 Nov 2022 07:23:08 +0000
From: Leo Yan <leo.yan@...aro.org>
To: Marc Zyngier <maz@...nel.org>, James Morse <james.morse@....com>,
Alexandru Elisei <alexandru.elisei@....com>,
Suzuki K Poulose <suzuki.poulose@....com>,
Oliver Upton <oliver.upton@...ux.dev>,
Catalin Marinas <catalin.marinas@....com>,
Will Deacon <will@...nel.org>,
Arnaldo Carvalho de Melo <acme@...nel.org>,
John Garry <john.garry@...wei.com>,
James Clark <james.clark@....com>,
Mike Leach <mike.leach@...aro.org>,
Peter Zijlstra <peterz@...radead.org>,
Ingo Molnar <mingo@...hat.com>,
Mark Rutland <mark.rutland@....com>,
Alexander Shishkin <alexander.shishkin@...ux.intel.com>,
Jiri Olsa <jolsa@...nel.org>,
Namhyung Kim <namhyung@...nel.org>,
linux-arm-kernel@...ts.infradead.org, kvmarm@...ts.linux.dev,
kvmarm@...ts.cs.columbia.edu, linux-kernel@...r.kernel.org,
linux-perf-users@...r.kernel.org
Cc: Leo Yan <leo.yan@...aro.org>
Subject: [PATCH v1 0/3] KVM: arm64: Support tracing virtual CPU ID
Before there have some efforts and discussion for supprot tracing
virtual CPU ID in Arm64 KVM, see [1][2].
The issue was blocked with a main concern that we cannot change the
existed trace events to avoid ABI breakage. So the question is how
we add new trace events with tracing virtual CPU ID and also need to
keep backward compatibility.
This patch set is to restart the work, it's inspired by Qais Yousef's
work for adding scheduler tracepoints [3].
The first patch changes to register tracepoint callbacks, this can allow
us to support multiple trace events with a single call site, it's a
preparation to add new trace events.
The second patch is to add two new trace events kvm_entry_v2 and
kvm_exit_v2, and these two trace events contain the field "vcpu_id" for
virtual CPU ID.
For more complete view, the third patch is the change in perf tool.
It dynamically detects trace nodes under sysfs and decide to use the
version 2's trace events or rollback to use original events.
This patch set has been tested with mainline kernel on Arm64 Ampere
Altra platform.
Note: I used checkpatch.pl to validate patches format and observed it
reports error for second patch for adding trace events; since the trace
event definition uses its own coding style, I just keep as it is.
[1] https://lore.kernel.org/lkml/1560330526-15468-2-git-send-email-yuzenghui@huawei.com/
[2] https://lore.kernel.org/lkml/20200917003645.689665-1-sergey.senozhatsky@gmail.com/
[3] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/kernel/sched/core.c?id=a056a5bed7fa67706574b00cf1122c38596b2be1
Leo Yan (3):
KVM: arm64: Dynamically register callback for tracepoints
KVM: arm64: Add trace events with field 'vcpu_id'
perf arm64: Support virtual CPU ID for kvm-stat
arch/arm64/kvm/Makefile | 2 +-
arch/arm64/kvm/arm.c | 4 +-
arch/arm64/kvm/trace.c | 35 +++++++++++++++++
arch/arm64/kvm/trace_arm.h | 53 ++++++++++++++++++++++++++
tools/perf/arch/arm64/util/kvm-stat.c | 54 ++++++++++++++++++++++++---
5 files changed, 140 insertions(+), 8 deletions(-)
create mode 100644 arch/arm64/kvm/trace.c
--
2.34.1
Powered by blists - more mailing lists