Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   "nakamura.shun@fujitsu.com" <nakamura.shun@fujitsu.com>
To:     'Jiri Olsa' <olsajiri@gmail.com>
CC:     "peterz@infradead.org" <peterz@infradead.org>,
        "mingo@redhat.com" <mingo@redhat.com>,
        "acme@kernel.org" <acme@kernel.org>,
        "mark.rutland@arm.com" <mark.rutland@arm.com>,
        "alexander.shishkin@linux.intel.com" 
        <alexander.shishkin@linux.intel.com>,
        "jolsa@redhat.com" <jolsa@redhat.com>,
        "namhyung@kernel.org" <namhyung@kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "linux-perf-users@vger.kernel.org" <linux-perf-users@vger.kernel.org>
Subject: RE: [RFC PATCH v2 2/7] libperf: Move 'open_flags' from tools/perf to
 evsel::open_flags
Thread-Topic: [RFC PATCH v2 2/7] libperf: Move 'open_flags' from tools/perf to
 evsel::open_flags
Thread-Index: AQHYRD0ZVj53OsTQsken6htt1zG29azqcpdg
Date:   Mon, 11 Apr 2022 08:27:56 +0000
Message-ID: <OSBPR01MB46006C8A00D51EFD498E0EACF7EA9@OSBPR01MB4600.jpnprd01.prod.outlook.com>
References: <20220325043829.224045-1-nakamura.shun@fujitsu.com>
 <20220325043829.224045-3-nakamura.shun@fujitsu.com> <YkRgFdWdW0HyqIno@krava>
In-Reply-To: <YkRgFdWdW0HyqIno@krava>
Accept-Language: ja-JP, en-US
Content-Language: ja-JP
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?iso-2022-jp?B?blpCcGRGWGNiRkFMRmVuQlU5aU96QWVBdHpEWFllSjlZWUU5Q09lMkZ0?=
 =?iso-2022-jp?B?b1AyWXJBSjRaaGkrTmozY2ZsRmowU0VGSlhrM3lWa2YwbWc4M1JWQWtY?=
 =?iso-2022-jp?B?ZlEwc1E1VU0wTEpZNUYwdUtNL3pvL05ZSnhyZWtnZ3RGU0cwbTg1MWEv?=
 =?iso-2022-jp?B?Uk56UVJDZCtLL0s1YW1LTVZhUWViU3gvUnhzVExCanhNR3hldlpkVGlZ?=
 =?iso-2022-jp?B?a0V6eHQ3bnFXVDRMNFhwRWpPRFNxWUVhd09XdktmMnBaSXVvUThvSjJ2?=
 =?iso-2022-jp?B?eUdHZ2lsVG1mMi9hTUNXZlBuU1prbU1vVkpUUzE5a01ZRlVocVBKaXpl?=
 =?iso-2022-jp?B?S1cxOVplaU9vai9BcUFMN3ZSekFrMVZCRkZhQWM4TVZuSnVzL3dQYSts?=
 =?iso-2022-jp?B?ZGJIQTY5MGZJeWVYWW5UOFlwdlA2T2lsYkxzcHh5bXoxcUxJeWZIdEor?=
 =?iso-2022-jp?B?U2ZFTzlCaTBzTUs0RXBiNEFkRVJvNTF2UmpCTzlDeFd4NTllWThWM1Yy?=
 =?iso-2022-jp?B?ZDRSY3RUSnVWOUk4c01CQmxFaEpwWWI1YWMyTXd5Qm0rRkNIS3dOd3hm?=
 =?iso-2022-jp?B?SVZGbjdIQURoL09ucXdSQTlVSnZieTlYbGtaSTRnWGNQNWM3d2FVVzhy?=
 =?iso-2022-jp?B?VWtYeVhNWU9kY2xNMnFuWWRXbnNiaWhvTHFVcnlQd3NCR2dpdkNWd3hP?=
 =?iso-2022-jp?B?MVYyM0F3VklvQWJQYkJUNHNQcmkyT29sY2g0TnJqZVpaYjlpK2h5Ulps?=
 =?iso-2022-jp?B?Q0VYcjU5NlZUNVlneDFreUZYbnFsL2ZQeEFYN09LYXJFR2pLWWpXc1lL?=
 =?iso-2022-jp?B?UmFqRHA5SjhCSnM0Tms2UnA0ckhZR045UHZKS1FsdityQjlrV1NhcUsz?=
 =?iso-2022-jp?B?QWVMd3RHa085YlRKSWw4dHU3VlVrdFFta2JaTExsR3RxcmdvWVIrL0lv?=
 =?iso-2022-jp?B?eE5BZVY4Y0FUd0JjS1VzZzBNUmY3VjdydEQvQ2Y5dGJKbU5MWXV2Smcv?=
 =?iso-2022-jp?B?Y21vdXpxOHlpZjNTR0pmeDd4bGwzMGtudmlBbGppSURLV3ZvL0xOVFJR?=
 =?iso-2022-jp?B?WGx3dG9OUWlCUFdGQnU3dFVHdjF3OWozck8yVHhuWUo0aUlvVU5oenFq?=
 =?iso-2022-jp?B?VmlueTZzVFcyUHRsREt0TVozaDJZc1dSb3o4MXdHZmdpNlhDcFZSMlA2?=
 =?iso-2022-jp?B?Uk5ZaitXMjJya3VhSWdNVmJMVnlaVW9JeGRsTzZnMmlMQUh2dFp3YUs4?=
 =?iso-2022-jp?B?MVgzVnlxRHNQaEwzWnM3eEQ2alJDT3BFKzNjdnpqUlQwUmRTR3ZnVFFy?=
 =?iso-2022-jp?B?Nnh1MVlGZzFOOGdPT1RhQUwrSlpzS3Nqb0NWTzVxdklkQWhPdW5rZis5?=
 =?iso-2022-jp?B?bnpFRDlQVXlqRGpGbjRHdFFoWGc0M1lHc2lkb1crWW9LODJ6U1JObWla?=
 =?iso-2022-jp?B?dVptSHRxS0FzcjhMNGZZWEo3LytTcDBJZUNMVEtjRmE1UVZwS2U0MFJw?=
 =?iso-2022-jp?B?UzNkakx1bE4vMVRLeTZWYlRaWmgwQXFNQ1dMRmFrVHZkTENaRmxWZ1Rr?=
 =?iso-2022-jp?B?a29jT1duL3RSR0pMSUp5cU5sQWduTWZ6bm8yYXhKc1NIQ09XenFKZFIx?=
 =?iso-2022-jp?B?cEd1VytRakRTUHZSeGEwd2J0eWhvZlBGVWYyQ21WTHd0cXp1NGFnMCtw?=
 =?iso-2022-jp?B?Ym94VHEvVW0yNUJTQUFKeW1Yclh4SEVSdmhieDlDNyt6MTdGSEJlSmJp?=
 =?iso-2022-jp?B?VkdiY1Q1MWU2eHFmUXNGN0RNYzVUZ1pjbnZ0bzMxS0c3dExVdHcrTFhu?=
 =?iso-2022-jp?B?L0lRaXNoVENKM1hCb0NjSnJIYXFqQjJPMjNGVlpGWjlUYlcrWFlIOGsx?=
 =?iso-2022-jp?B?R21Ra3ZuZ3VLMERsUnhQbDkrSWF1ZWpKYVhaSTlYZkpIWmZHSzZ0SDI4?=
 =?iso-2022-jp?B?eXJsTXdhOVRQaHJ4L2JBSng1V3lVaHFFdU10YUFpZnRzVzBWWUg0MXB2?=
 =?iso-2022-jp?B?M0wyRGZ6aXFvTkQ5TG5FYjRHVVE1Yld3bkJUbGk3bTU0ejAzYmJnL2NE?=
 =?iso-2022-jp?B?b3g5MmYvMDk1MlEwNXFlZStlYjVXdXl2ck1HVk9UR0RMc29sdVllVjBs?=
 =?iso-2022-jp?B?Q3cxN0VxVzZzTHZPYzF2R0d0NEh3R0lmaXNLeGU1Rnp6dVB3a0pwMDJ2?=
 =?iso-2022-jp?B?SktHMlhIME9sZytFd3VjR0tLQVBNMlFTSVN4V0F4ZUZnd3NSUm0rY01h?=
 =?iso-2022-jp?B?UGN2eFNMai94SXl1OHNudnNlSjdiTDJmdjBjRTlhZGhOL3liT1Awa0JK?=
 =?iso-2022-jp?B?eWVKRlNPWmlTOWU2clp5YUZXOVQ5Sk8vRmNJOUUxSllsK0R2dUNHSlZG?=
 =?iso-2022-jp?B?Z0NvdWo4Q0hYZS9WbEcrZzdWbmNNTkJJanJDVm9SZUVSKy8wZjBEc1JB?=
 =?iso-2022-jp?B?RllRUXdGeEI5Z2l1dXc3VzRuUEl2Q3V4bWFvYS8xc2V3SWlLZzFqcDVH?=
 =?iso-2022-jp?B?c25LTHpOR3pFSHIzcHVtUEl0WjV4ZWJBV3c5U25LQWVUaUtYU0hVTVFh?=
 =?iso-2022-jp?B?aUNsdDEyTT0=?=
Content-Type: text/plain; charset="iso-2022-jp"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: fujitsu.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: OSBPR01MB4600.jpnprd01.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 65117eb2-8caf-4d50-50cf-08da1b952e73
X-MS-Exchange-CrossTenant-originalarrivaltime: 11 Apr 2022 08:27:56.9264
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: a19f121d-81e1-4858-a9d8-736e267fd4c7
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: YW+tCYn9XnmetmR59rSN0RSCVkhbnWfBbcVBXP1kEP6NFSyMi9ti9TozUDbz7EUyFwciQXQoaxEFnB5P8+K8tspHRxBGOt+P1GBrOl2jkZg=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: TYCPR01MB8568
X-Spam-Status: No, score=-5.2 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,RCVD_IN_DNSWL_MED,RCVD_IN_MSPIKE_H5,RCVD_IN_MSPIKE_WL,
        SPF_HELO_PASS,SPF_NONE,T_SCC_BODY_TEXT_LINE autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi jirka

> >  struct perf_evsel *perf_evsel__new(struct perf_event_attr *attr) @@
> > -158,7 +159,7 @@ int perf_evsel__open(struct perf_evsel *evsel, struct
> > perf_cpu_map *cpus,
> >
> >  			fd =3D sys_perf_event_open(&evsel->attr,
> >  						 threads->map[thread].pid,
> > -						 cpu, group_fd, 0);
> > +						 cpu, group_fd,
> evsel->open_flags);
>=20
> please keep this change just open_flags field move, this change should go=
 to
> later patch
>=20
I'll fix it.


Best Regards
Shunsuke