All of lore.kernel.org
 help / color / mirror / Atom feed
From: Arnaldo Carvalho de Melo <acme@kernel.org>
To: Ian Rogers <irogers@google.com>
Cc: John Garry <john.garry@huawei.com>,
	will@kernel.org, mathieu.poirier@linaro.org, leo.yan@linaro.org,
	peterz@infradead.org, mingo@redhat.com, mark.rutland@arm.com,
	alexander.shishkin@linux.intel.com, jolsa@redhat.com,
	namhyung@kernel.org, linux-arm-kernel@lists.infradead.org,
	linux-perf-users@vger.kernel.org, linux-kernel@vger.kernel.org,
	linuxarm@huawei.com, zhangshaokun@hisilicon.com,
	liuqi115@huawei.com
Subject: Re: [PATCH 1/5] perf parse-events: Set numeric term config
Date: Tue, 28 Sep 2021 16:16:51 -0300	[thread overview]
Message-ID: <YVNqI+YFGbX+xiuk@kernel.org> (raw)
In-Reply-To: <CAP-5=fVG3_p3Xf3s6FnpSRFwt3+CxitPZwRK0fMbz2t8WZaSEw@mail.gmail.com>

Em Tue, Sep 28, 2021 at 10:59:42AM -0700, Ian Rogers escreveu:
> On Thu, Sep 16, 2021 at 5:39 AM John Garry <john.garry@huawei.com> wrote:
> >
> > For numeric terms, the config field may be NULL as it is not set from the
> > l+y parsing.
> >
> > Fix by setting the term config from the term type name.
> >
> > Also fix up the pmu-events test to set the alias strings to set the period
> > term properly, and fix up parse-events test to check the term config
> > string.
> >
> > Signed-off-by: John Garry <john.garry@huawei.com>
> 
> Acked-by: Ian Rogers <irogers@google.com>
> 
> Having this would be very useful with an issue I'm looking into.

Thanks, applied the series.

- Arnaldo


WARNING: multiple messages have this Message-ID (diff)
From: Arnaldo Carvalho de Melo <acme@kernel.org>
To: Ian Rogers <irogers@google.com>
Cc: John Garry <john.garry@huawei.com>,
	will@kernel.org, mathieu.poirier@linaro.org, leo.yan@linaro.org,
	peterz@infradead.org, mingo@redhat.com, mark.rutland@arm.com,
	alexander.shishkin@linux.intel.com, jolsa@redhat.com,
	namhyung@kernel.org, linux-arm-kernel@lists.infradead.org,
	linux-perf-users@vger.kernel.org, linux-kernel@vger.kernel.org,
	linuxarm@huawei.com, zhangshaokun@hisilicon.com,
	liuqi115@huawei.com
Subject: Re: [PATCH 1/5] perf parse-events: Set numeric term config
Date: Tue, 28 Sep 2021 16:16:51 -0300	[thread overview]
Message-ID: <YVNqI+YFGbX+xiuk@kernel.org> (raw)
In-Reply-To: <CAP-5=fVG3_p3Xf3s6FnpSRFwt3+CxitPZwRK0fMbz2t8WZaSEw@mail.gmail.com>

Em Tue, Sep 28, 2021 at 10:59:42AM -0700, Ian Rogers escreveu:
> On Thu, Sep 16, 2021 at 5:39 AM John Garry <john.garry@huawei.com> wrote:
> >
> > For numeric terms, the config field may be NULL as it is not set from the
> > l+y parsing.
> >
> > Fix by setting the term config from the term type name.
> >
> > Also fix up the pmu-events test to set the alias strings to set the period
> > term properly, and fix up parse-events test to check the term config
> > string.
> >
> > Signed-off-by: John Garry <john.garry@huawei.com>
> 
> Acked-by: Ian Rogers <irogers@google.com>
> 
> Having this would be very useful with an issue I'm looking into.

Thanks, applied the series.

- Arnaldo


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  reply	other threads:[~2021-09-28 19:16 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-09-16 12:34 [PATCH 0/5] Improve perf list support for hisi uncore PMUs John Garry
2021-09-16 12:34 ` John Garry
2021-09-16 12:34 ` [PATCH 1/5] perf parse-events: Set numeric term config John Garry
2021-09-16 12:34   ` John Garry
2021-09-28 17:59   ` Ian Rogers
2021-09-28 17:59     ` Ian Rogers
2021-09-28 19:16     ` Arnaldo Carvalho de Melo [this message]
2021-09-28 19:16       ` Arnaldo Carvalho de Melo
2021-09-16 12:34 ` [PATCH 2/5] perf jevents: Support ConfigCode John Garry
2021-09-16 12:34   ` John Garry
2021-09-28 18:00   ` Ian Rogers
2021-09-28 18:00     ` Ian Rogers
2021-09-16 12:34 ` [PATCH 3/5] perf test: Verify more event members in pmu-events test John Garry
2021-09-16 12:34   ` John Garry
2021-09-28 18:02   ` Ian Rogers
2021-09-28 18:02     ` Ian Rogers
2021-09-16 12:34 ` [PATCH 4/5] perf test: Add pmu-event test for event described as "config=" John Garry
2021-09-16 12:34   ` John Garry
2021-09-28 18:03   ` Ian Rogers
2021-09-28 18:03     ` Ian Rogers
2021-09-16 12:34 ` [PATCH 5/5] perf vendor events arm64: Revise hip08 uncore events John Garry
2021-09-16 12:34   ` John Garry
2021-09-28 18:04   ` Ian Rogers
2021-09-28 18:04     ` 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=YVNqI+YFGbX+xiuk@kernel.org \
    --to=acme@kernel.org \
    --cc=alexander.shishkin@linux.intel.com \
    --cc=irogers@google.com \
    --cc=john.garry@huawei.com \
    --cc=jolsa@redhat.com \
    --cc=leo.yan@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-perf-users@vger.kernel.org \
    --cc=linuxarm@huawei.com \
    --cc=liuqi115@huawei.com \
    --cc=mark.rutland@arm.com \
    --cc=mathieu.poirier@linaro.org \
    --cc=mingo@redhat.com \
    --cc=namhyung@kernel.org \
    --cc=peterz@infradead.org \
    --cc=will@kernel.org \
    --cc=zhangshaokun@hisilicon.com \
    /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.