From: Namhyung Kim <namhyung@kernel.org>
To: Arnaldo Carvalho de Melo <acme@kernel.org>
Cc: Ian Rogers <irogers@google.com>, Jiri Olsa <jolsa@kernel.org>,
Adrian Hunter <adrian.hunter@intel.com>,
James Clark <james.clark@linaro.org>,
Peter Zijlstra <peterz@infradead.org>,
Ingo Molnar <mingo@kernel.org>,
LKML <linux-kernel@vger.kernel.org>,
linux-perf-users@vger.kernel.org
Subject: [PATCH v3 0/6] perf kvm: A small update in the default arch event
Date: Wed, 1 Jul 2026 12:41:47 -0700 [thread overview]
Message-ID: <20260701194153.401218-1-namhyung@kernel.org> (raw)
Hello,
Ian said there's an issue in the perf kvm record command line option parsing.
This is my attempt to address the issue, simplify the code and fix memory leaks.
v3 changes)
* remove unnecessary error check (Sashiko)
* update test code after rebase
v2 changes)
* handle kvm top subcommand (Sashiko)
* handle other arch/condition for the default events (Sashiko)
* add <stdlib.h> for musl compatibility (Sashiko)
* fix some memory leaks
Thanks,
Namhyung
Namhyung Kim (6):
perf kvm: Factor out kvm_need_default_arch_event()
perf kvm: Check kvm_need_default_arch_event() early
perf kvm: Kill STRDUP_FAIL_EXIT()
perf kvm: Do not copy filename string
perf kvm: Fix a memory leak in the usage string
perf test: Extend perf kvm tests to check default event
tools/perf/builtin-kvm.c | 109 ++++++++----------
tools/perf/tests/shell/kvm.sh | 10 ++
.../util/kvm-stat-arch/kvm-stat-powerpc.c | 31 +----
tools/perf/util/kvm-stat-arch/kvm-stat-x86.c | 37 +-----
tools/perf/util/kvm-stat.c | 41 +++++++
tools/perf/util/kvm-stat.h | 18 ++-
6 files changed, 116 insertions(+), 130 deletions(-)
--
2.55.0.rc0.799.gd6f94ed593-goog
next reply other threads:[~2026-07-01 19:41 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-07-01 19:41 Namhyung Kim [this message]
2026-07-01 19:41 ` [PATCH v3 1/6] perf kvm: Factor out kvm_need_default_arch_event() Namhyung Kim
2026-07-01 19:59 ` sashiko-bot
2026-07-01 19:41 ` [PATCH v3 2/6] perf kvm: Check kvm_need_default_arch_event() early Namhyung Kim
2026-07-01 19:41 ` [PATCH v3 3/6] perf kvm: Kill STRDUP_FAIL_EXIT() Namhyung Kim
2026-07-01 19:41 ` [PATCH v3 4/6] perf kvm: Do not copy filename string Namhyung Kim
2026-07-01 19:41 ` [PATCH v3 5/6] perf kvm: Fix a memory leak in the usage string Namhyung Kim
2026-07-01 19:41 ` [PATCH v3 6/6] perf test: Extend perf kvm tests to check default event Namhyung Kim
2026-07-01 20:11 ` sashiko-bot
2026-07-02 4:09 ` [PATCH v3 0/6] perf kvm: A small update in the default arch event Ian Rogers
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260701194153.401218-1-namhyung@kernel.org \
--to=namhyung@kernel.org \
--cc=acme@kernel.org \
--cc=adrian.hunter@intel.com \
--cc=irogers@google.com \
--cc=james.clark@linaro.org \
--cc=jolsa@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-perf-users@vger.kernel.org \
--cc=mingo@kernel.org \
--cc=peterz@infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox