Date:   Tue, 11 Jan 2022 15:50:07 -0300
From:   Arnaldo Carvalho de Melo <arnaldo.melo@gmail.com>
To:     Ian Rogers <irogers@google.com>, John Garry <john.garry@huawei.com>
CC:     Andi Kleen <ak@linux.intel.com>, Jiri Olsa <jolsa@redhat.com>,
        Namhyung Kim <namhyung@kernel.org>,
        Kajol Jain <kjain@linux.ibm.com>,
        "Paul A . Clarke" <pc@us.ibm.com>,
        Arnaldo Carvalho de Melo <acme@kernel.org>,
        Riccardo Mancini <rickyman7@gmail.com>,
        Kan Liang <kan.liang@linux.intel.com>,
        Peter Zijlstra <peterz@infradead.org>,
        Ingo Molnar <mingo@redhat.com>,
        Mark Rutland <mark.rutland@arm.com>,
        Alexander Shishkin <alexander.shishkin@linux.intel.com>,
        linux-perf-users@vger.kernel.org, linux-kernel@vger.kernel.org,
        Vineet Singh <vineet.singh@intel.com>,
        James Clark <james.clark@arm.com>,
        Mathieu Poirier <mathieu.poirier@linaro.org>,
        Suzuki K Poulose <suzuki.poulose@arm.com>,
        Mike Leach <mike.leach@linaro.org>,
        Leo Yan <leo.yan@linaro.org>, coresight@lists.linaro.org,
        linux-arm-kernel@lists.infradead.org, zhengjun.xing@intel.com,
        eranian@google.com
Subject: Re: [PATCH v4 05/48] perf stat: Switch to cpu version of cpu_map__get
User-Agent: K-9 Mail for Android
In-Reply-To: <CAP-5=fW2JzePV=rQG=Ww1Omub0_r2gi38FUVyaG+a3L=44R9XA@mail.gmail.com>
References: <20220105061351.120843-1-irogers@google.com> <20220105061351.120843-6-irogers@google.com> <64d64526-7d3e-0c31-f43e-f03810f72ac2@huawei.com> <CAP-5=fW2JzePV=rQG=Ww1Omub0_r2gi38FUVyaG+a3L=44R9XA@mail.gmail.com>
Message-ID: <F6E1C874-F64D-4563-915B-F6420F987264@gmail.com>
MIME-Version: 1.0
Content-Type: text/plain;
 charset=utf-8
Content-Transfer-Encoding: quoted-printable
Precedence: bulk


On January 11, 2022 3:46:41 PM GMT-03:00, Ian Rogers <irogers@google=2Ecom=
> wrote:
>On Mon, Jan 10, 2022 at 11:04 AM John Garry <john=2Egarry@huawei=2Ecom> w=
rote:
>>
>> On 05/01/2022 06:13, Ian Rogers wrote:
>> > Avoid bugs where the wrong index is passed with the cpu_map=2E
>>
>> nit: Don't you mean possible bugs?
>
>You are right=2E I'll watch myself doing this in the future=2E

I'll update the comment,

I'm still testing it and giving more time for review, including to the adj=
ustments I made=2E=2E

- Arnaldo

>
>Thanks,
>Ian
>
>> >
>> > Reviewed-by: James Clark <james=2Eclark@arm=2Ecom>
>> > Signed-off-by: Ian Rogers <irogers@google=2Ecom>