Date:   Tue, 28 Feb 2023 16:51:49 +0800
From:   Leo Yan <leo.yan@linaro.org>
To:     Arnaldo Carvalho de Melo <acme@kernel.org>
Cc:     Peter Zijlstra <peterz@infradead.org>,
        Ingo Molnar <mingo@redhat.com>,
        Mark Rutland <mark.rutland@arm.com>,
        Alexander Shishkin <alexander.shishkin@linux.intel.com>,
        Jiri Olsa <jolsa@kernel.org>,
        Namhyung Kim <namhyung@kernel.org>,
        Ian Rogers <irogers@google.com>,
        Adrian Hunter <adrian.hunter@intel.com>,
        linux-perf-users@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v1 00/14] perf kvm: Support histograms and TUI mode
Message-ID: <Y/3ApYi3GLCVGEZr@leoy-yangtze.lan>
References: <20230226042053.1492409-1-leo.yan@linaro.org>
 <Y/0jPa9xu371PtRS@kernel.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <Y/0jPa9xu371PtRS@kernel.org>
Precedence: bulk

Hi Arnaldo,

On Mon, Feb 27, 2023 at 06:40:13PM -0300, Arnaldo Carvalho de Melo wrote:

[...]

> > And I verified the '--stdio' mode and confirmed it has the same result
> > with before applying this series.
> 
> Can you provide precise instructions for all the steps involved? from
> 'perf kvm record', I'm trying here with a simple 'perf kvm record' and
> then starting vm and having trouble.

Below are steps for verification this patch set.

In a terminal, I launch a KVM virtual machine with qemu command; I
downloaded an Ubuntu iso file [1] ahead and used it as the file system
image (or you could use any distro's iso file as file system image):

  $ qemu-system-x86_64 -M pc -enable-kvm -cpu host -m 4096 -hda ubuntu-22.04-desktop-amd64.iso

Then in another terminal, I can use below commands to capture KVM trace
data and report the result:

  # cd linux/tools/perf
  # ./perf kvm stat record
  # ./perf kvm stat report          => Output in TUI mode
  # ./perf kvm stat report --stdio  => Output in stdio mode

Note, I have sent out the patch set v2 to fix a potential building
failure in patch 13.  Please work on the new patch set [2].

Thanks,
Leo

[1] https://releases.ubuntu.com/22.04/ubuntu-22.04.2-desktop-amd64.iso
[2] https://lore.kernel.org/lkml/20230228084147.106167-1-leo.yan@linaro.org/