From: Leo Yan <leo.yan@linaro.org>
To: Arnaldo Carvalho de Melo <acme@kernel.org>
Cc: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>,
Mark Rutland <mark.rutland@arm.com>,
Peter Zijlstra <peterz@infradead.org>,
Will Deacon <will@kernel.org>, John Garry <john.garry@huawei.com>,
Mathieu Poirier <mathieu.poirier@linaro.org>,
Namhyung Kim <namhyung@kernel.org>,
Suleiman Souhlal <suleiman@google.com>,
linux-kernel@vger.kernel.org,
linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCHv4] perf kvm: add kvm-stat for arm64
Date: Tue, 13 Oct 2020 23:29:34 +0800 [thread overview]
Message-ID: <20201013152934.GC24974@leoy-ThinkPad-X240s> (raw)
In-Reply-To: <20201013143709.GD1063281@kernel.org>
On Tue, Oct 13, 2020 at 11:37:09AM -0300, Arnaldo Carvalho de Melo wrote:
> Em Tue, Sep 29, 2020 at 12:34:50PM +0900, Sergey Senozhatsky escreveu:
> > On (20/09/17 19:02), Sergey Senozhatsky wrote:
> > > Add support for perf kvm stat on arm64 platform.
>
> > > Example:
> > > # perf kvm stat report
>
> > > Analyze events for all VMs, all VCPUs:
>
> > > VM-EXIT Samples Samples% Time% Min Time Max Time Avg time
> > >
> > > DABT_LOW 661867 98.91% 40.45% 2.19us 3364.65us 6.24us ( +- 0.34% )
> > > IRQ 4598 0.69% 57.44% 2.89us 3397.59us 1276.27us ( +- 1.61% )
> > > WFx 1475 0.22% 1.71% 2.22us 3388.63us 118.31us ( +- 8.69% )
> > > IABT_LOW 1018 0.15% 0.38% 2.22us 2742.07us 38.29us ( +- 12.55% )
> > > SYS64 180 0.03% 0.01% 2.07us 112.91us 6.57us ( +- 14.95% )
> > > HVC64 17 0.00% 0.01% 2.19us 322.35us 42.95us ( +- 58.98% )
>
> > > Total Samples:669155, Total events handled time:10216387.86us.
>
> > > Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
> > > Reviewed-by: Leo Yan <leo.yan@linaro.org>
> > > Tested-by: Leo Yan <leo.yan@linaro.org>
>
> > Arnaldo, any opinion on this?
>
> I'm not finding the actual patch, just this reply from you, lets try
> with b4 using this message Message-Id... Magic! But it isn't applying,
> can you please refresh the patch to what is in my perf/core branch?
I did a quick trying, the merge confliction is caused by the file
tools/perf/arch/arm64/util/Build (for the new added tsc.c).
@Sergey, please help to rebase on the latest perf/core branch [1].
Thanks,
Leo
[1] https://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git branch: perf/core
WARNING: multiple messages have this Message-ID (diff)
From: Leo Yan <leo.yan@linaro.org>
To: Arnaldo Carvalho de Melo <acme@kernel.org>
Cc: Mark Rutland <mark.rutland@arm.com>,
Mathieu Poirier <mathieu.poirier@linaro.org>,
Peter Zijlstra <peterz@infradead.org>,
John Garry <john.garry@huawei.com>,
linux-kernel@vger.kernel.org,
Sergey Senozhatsky <sergey.senozhatsky@gmail.com>,
Suleiman Souhlal <suleiman@google.com>,
Namhyung Kim <namhyung@kernel.org>, Will Deacon <will@kernel.org>,
linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCHv4] perf kvm: add kvm-stat for arm64
Date: Tue, 13 Oct 2020 23:29:34 +0800 [thread overview]
Message-ID: <20201013152934.GC24974@leoy-ThinkPad-X240s> (raw)
In-Reply-To: <20201013143709.GD1063281@kernel.org>
On Tue, Oct 13, 2020 at 11:37:09AM -0300, Arnaldo Carvalho de Melo wrote:
> Em Tue, Sep 29, 2020 at 12:34:50PM +0900, Sergey Senozhatsky escreveu:
> > On (20/09/17 19:02), Sergey Senozhatsky wrote:
> > > Add support for perf kvm stat on arm64 platform.
>
> > > Example:
> > > # perf kvm stat report
>
> > > Analyze events for all VMs, all VCPUs:
>
> > > VM-EXIT Samples Samples% Time% Min Time Max Time Avg time
> > >
> > > DABT_LOW 661867 98.91% 40.45% 2.19us 3364.65us 6.24us ( +- 0.34% )
> > > IRQ 4598 0.69% 57.44% 2.89us 3397.59us 1276.27us ( +- 1.61% )
> > > WFx 1475 0.22% 1.71% 2.22us 3388.63us 118.31us ( +- 8.69% )
> > > IABT_LOW 1018 0.15% 0.38% 2.22us 2742.07us 38.29us ( +- 12.55% )
> > > SYS64 180 0.03% 0.01% 2.07us 112.91us 6.57us ( +- 14.95% )
> > > HVC64 17 0.00% 0.01% 2.19us 322.35us 42.95us ( +- 58.98% )
>
> > > Total Samples:669155, Total events handled time:10216387.86us.
>
> > > Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
> > > Reviewed-by: Leo Yan <leo.yan@linaro.org>
> > > Tested-by: Leo Yan <leo.yan@linaro.org>
>
> > Arnaldo, any opinion on this?
>
> I'm not finding the actual patch, just this reply from you, lets try
> with b4 using this message Message-Id... Magic! But it isn't applying,
> can you please refresh the patch to what is in my perf/core branch?
I did a quick trying, the merge confliction is caused by the file
tools/perf/arch/arm64/util/Build (for the new added tsc.c).
@Sergey, please help to rebase on the latest perf/core branch [1].
Thanks,
Leo
[1] https://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git branch: perf/core
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2020-10-13 15:29 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-09-17 10:02 [PATCHv4] perf kvm: add kvm-stat for arm64 Sergey Senozhatsky
2020-09-17 10:02 ` Sergey Senozhatsky
2020-09-29 3:34 ` Sergey Senozhatsky
2020-09-29 3:34 ` Sergey Senozhatsky
2020-10-13 14:37 ` Arnaldo Carvalho de Melo
2020-10-13 14:37 ` Arnaldo Carvalho de Melo
2020-10-13 15:29 ` Leo Yan [this message]
2020-10-13 15:29 ` Leo Yan
2020-10-27 5:56 ` Sergey Senozhatsky
2020-10-27 5:56 ` Sergey Senozhatsky
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=20201013152934.GC24974@leoy-ThinkPad-X240s \
--to=leo.yan@linaro.org \
--cc=acme@kernel.org \
--cc=john.garry@huawei.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=mathieu.poirier@linaro.org \
--cc=namhyung@kernel.org \
--cc=peterz@infradead.org \
--cc=sergey.senozhatsky@gmail.com \
--cc=suleiman@google.com \
--cc=will@kernel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.