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;
Message-ID: <9952dc32-f464-c85a-d812-946d6b0ac734@intel.com>
Date:   Fri, 14 Apr 2023 17:18:27 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.9.1
Subject: Re: [PATCH v2 bpf 02/11] bpftool: define a local bpf_perf_link to fix
 accessing its fields
Content-Language: en-US
To:     =?UTF-8?Q?Michal_Such=c3=a1nek?= <msuchanek@suse.de>
CC:     Alexander Lobakin <alobakin@mailbox.org>,
        Alexei Starovoitov <ast@kernel.org>,
        Shung-Hsi Yu <shung-hsi.yu@suse.com>,
        Daniel Borkmann <daniel@iogearbox.net>,
        Andrii Nakryiko <andrii@kernel.org>,
        "Maciej Fijalkowski" <maciej.fijalkowski@intel.com>,
        Song Liu <songliubraving@fb.com>,
        Kumar Kartikeya Dwivedi <memxor@gmail.com>,
        <bpf@vger.kernel.org>, <netdev@vger.kernel.org>,
        <linux-kernel@vger.kernel.org>
References: <20220421003152.339542-1-alobakin@pm.me>
 <20220421003152.339542-3-alobakin@pm.me>
 <20230414095457.GG63923@kunlun.suse.cz>
From:   Alexander Lobakin <aleksander.lobakin@intel.com>
In-Reply-To: <20230414095457.GG63923@kunlun.suse.cz>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 8bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?OHRTa0J4NktDV0VaRlh6bHFKSU9HKzVYdjhWWldZbTJ1a21XaDlNQ2NHNC9o?=
 =?utf-8?B?TE15WW05dGtBZmxVMTVxekZkSW9uVWNrMUdXRVRSRVB1Mjg5aU1pN21ocUtO?=
 =?utf-8?B?SThKRUVUYVk0SzBsZ3pURjJnamY0cWd2eU1wZDRsZjVjUUdVRUc1bS84REZP?=
 =?utf-8?B?NkVyY2JtVW9rbDRLYmUzbkFBUnE2aTNaQzJ6NmorT1k5N2trTS9MNWFNVjEy?=
 =?utf-8?B?cGEybk13Vkk5TG54QTg0dUhDYVo3Z1o2QTE1OHc0bGJSUXFING5pL0pQdzB6?=
 =?utf-8?B?TVRpUE00Q0twbHNNRUF6NTJVZHUyOEpsOStpOUtWK0ZiRFhDNkZZS0duc1JD?=
 =?utf-8?B?cWV6WEExZWFId2dGdjNiK3IzdUV1QXVyKzlaVSt6aGI0cldIQmJjU1piazha?=
 =?utf-8?B?UmgvcTJZSUVqSkVVTjdBRXAzQjI3c25xUzBpUmt4WHJlSmNiODVjamR2VDZm?=
 =?utf-8?B?VXhZdTJ0aVBYUXBqOStJeXg5TEo3RjhMSVhFWmp5YmRVYTR2Ui82aXNtQm1Z?=
 =?utf-8?B?dEZpaElWVVNiQkxSeVZGUCt3TFpUQ1NQQk15YXFaZDNZdklzS0dRS2tRT0to?=
 =?utf-8?B?RkRhamx2bnlvMk5uS2kyMFQxOWFiSkdWTExrcEF6c2hjVjFBVGVIRWphbE1V?=
 =?utf-8?B?MGFsdUVxcGpDWlU5c2ZjNUpFbVlKdUU0dEhiZ3FlbEhGdlRMQUx6dUowMDk4?=
 =?utf-8?B?K3ltVitHL1Iwd1c0STZ1dHF4QkVUeitFMDkvb1JoVlBTSWtvd0tHTFBObXFV?=
 =?utf-8?B?SVppcDFWWG9xUEZVcW45SzNCS2xieWo2Mkc3ejEvUUZrNEIrbVZ3NkpGRmx2?=
 =?utf-8?B?a3J1M1NDY01lRnhpK2VVVU55cFFrdXRnek10T3hRNWp1MGRqOGl4SlZwdmgw?=
 =?utf-8?B?L2Y3ZmNXTWZEYXl0WTBJa3h0NC9TZ3ZLdXFWSnFrcFZLaVNDZ0IzU3hLWHEr?=
 =?utf-8?B?R016enorUndNdVRBb1BhTzhiUDlQSVNmdWZKMzhNbFJNN2QxZnZoazh1Z0I3?=
 =?utf-8?B?YTFnOFdldlg4U1I3bU0xZitGdCszaElZWnFab3ZWUlZkWHEvRE9UTHJDSkV2?=
 =?utf-8?B?alZvWDlrVmE1TVRDNXJSUEFKbVVkMkF6U2g4dU9KM3ZjaWtkK1p2TFBBOHh5?=
 =?utf-8?B?Y0VpbE9BaVVRc1RRaEcyalJlS0c1NVNzUG9wZERhcEVrNkVTWkFvNEk0b1Zk?=
 =?utf-8?B?OUNEejlVSkRJYmxoTnQxSW5CeW8vZm9oS0lyWkZCajdRZ00zb1hVTnB6SUht?=
 =?utf-8?B?b1o4bWh2Y0Vlb0ZiWXA0TWZUL1dHT3RzV0dDSzllUHc3NEV4RGR0NGtjMllB?=
 =?utf-8?B?WjVzempqRDE5a21PVHU4Zk1ZcUJqaE1TZnFQRkUwb0lBKys3MWRjSWo2TEsy?=
 =?utf-8?B?SXNKdG5DakVwYTVYdzdnLzIzOXZHUFdzV3VYV1hNOCtqMDRGanJWMFIyY0g1?=
 =?utf-8?B?RlZTeTQxUHJ3aWZWZVloa2F4YU9wOXcvdG9qaDd1NUdsdXVDMThjWjR5ZUNm?=
 =?utf-8?B?cnhZRUIrSkV3SlozeHFhYnNVdTVLbk9NVEJVa2FQeGRQNmp2b1R4UXAxblUv?=
 =?utf-8?B?bkNyTHZzTnZSNHU0cTJxWldwUjJRTFNuZEtRTWxFZ2NZZFVrNit6dTNUUUY0?=
 =?utf-8?B?U2NDa0k3WmVvVndrRWJYSm1oSVJhS2twQTg1b2IxclpkZzdyTTd3VUIyMlQz?=
 =?utf-8?B?MU5heGxmMm0zSHFtWGovNDhSTGJxTjgzdWhRNGswTHpHb0VKMG92eUo2RUwx?=
 =?utf-8?B?VzJ3TU0rMU9UNGV5dWFLZlowRUJadDRBUkthbVlobFBicGcyVzVsenphQlN1?=
 =?utf-8?B?L2JUdURUYnh6TDRoTVpINmIycUhUU2RDYWRsUHM4RlJHbHAxRDRxalNHdmx1?=
 =?utf-8?B?T1ZCelBKaVByNEFldkh1TFdIUElTa2JtMS92cExqcVNzekd5L3ZEVm1ZSUZr?=
 =?utf-8?B?a0NkeGhEWUw5dTBVZDRObnpORys0alhzaEUwSWlXSmQ5bCtsdDBTblRIZ1Iz?=
 =?utf-8?B?Q2oybUhXMENTNFJLTTQ2N2lnS2c0dDAzV0EyQUVnSzkySTM1Uk5ReUpybGd0?=
 =?utf-8?B?bklvVnFGenp1QnozNWs1a3ZqMVV0dThKcnRhYlMzOUxsOHRsU2VCUFBrRVk0?=
 =?utf-8?B?aEMrNFM4T2Mxd3FxcEV5dGdZNnBlU3NQWkpOVUpvbWlIancrNDNodjMxUkJF?=
 =?utf-8?B?TGc9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 4913d34b-8749-47a9-6788-08db3cfb8cb8
X-MS-Exchange-CrossTenant-AuthSource: DM6PR11MB3625.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 14 Apr 2023 15:18:50.1086
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: yXlHJ7yRbFnrxZ/ALYETRyrMluwWPyZun0zlxPmrlwshSwSWcXyCUAd+/d9+dCFfl4L7+pjsCPw6XaawJjfTz5u/VJBsl7y6GYGrhN2ZmBs=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL1PR11MB5288
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-6.7 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,
        RCVD_IN_DNSWL_MED,RCVD_IN_MSPIKE_H3,RCVD_IN_MSPIKE_WL,SPF_HELO_NONE,
        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

From: Michal Suchánek <msuchanek@suse.de>
Date: Fri, 14 Apr 2023 11:54:57 +0200

> Hello,

Hey-hey,

> 
> On Thu, Apr 21, 2022 at 12:38:58AM +0000, Alexander Lobakin wrote:
>> When building bpftool with !CONFIG_PERF_EVENTS:
>>
>> skeleton/pid_iter.bpf.c:47:14: error: incomplete definition of type 'struct bpf_perf_link'
>>         perf_link = container_of(link, struct bpf_perf_link, link);
>>                     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>> tools/bpf/bpftool/bootstrap/libbpf/include/bpf/bpf_helpers.h:74:22: note: expanded from macro 'container_of'
>>                 ((type *)(__mptr - offsetof(type, member)));    \
>>                                    ^~~~~~~~~~~~~~~~~~~~~~
>> tools/bpf/bpftool/bootstrap/libbpf/include/bpf/bpf_helpers.h:68:60: note: expanded from macro 'offsetof'
>>  #define offsetof(TYPE, MEMBER)  ((unsigned long)&((TYPE *)0)->MEMBER)
>>                                                   ~~~~~~~~~~~^
>> skeleton/pid_iter.bpf.c:44:9: note: forward declaration of 'struct bpf_perf_link'
>>         struct bpf_perf_link *perf_link;
>>                ^
>>
>> &bpf_perf_link is being defined and used only under the ifdef.
>> Define struct bpf_perf_link___local with the `preserve_access_index`
>> attribute inside the pid_iter BPF prog to allow compiling on any
>> configs. CO-RE will substitute it with the real struct bpf_perf_link
>> accesses later on.
>> container_of() is not CO-REd, but it is a noop for
>> bpf_perf_link <-> bpf_link and the local copy is a full mirror of
>> the original structure.
>>
>> Fixes: cbdaf71f7e65 ("bpftool: Add bpf_cookie to link output")
> 
> This does not solve the problem completely. Kernels that don't have
> CONFIG_PERF_EVENTS in the first place are also missing the enum value
> BPF_LINK_TYPE_PERF_EVENT which is used as the condition for handling the
> cookie.

Sorry, I haven't been working with my home/private stuff for more than a
year already. I may get back to it some day when I'm tired of Lua (curse
words, sorry :D), but for now the series is "a bit" abandoned.
I think there was alternative solution proposed there, which promised to
be more flexible. But IIRC it also doesn't touch the enum (was it added
recently? Because it was building just fine a year ago on config without
perf events).

> 
> Thanks
> 
> Michal
> 
Thanks,
Olek