Re: [PATCH bpf] selftests/bpf: Re-add kfunc declarations to qdisc tests

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 2025-07-02 at 10:28 -0700, Andrii Nakryiko wrote:

[...]

> Ok, if I'm the only one who thinks we shouldn't duplicate kfunc
> definitions because we have an established approach that works, so be
> it, not such a big deal.
> 
> I'm curious to see if the next step would be someone asking to do
> something about enum or struct that is defined only with some kernel
> configuration that selftest relies on. Are we going to add extra
> #defines just to be able to do #ifndef-#define-#endif guarding in
> selftest source code just to accommodate someone wanting to build BPF
> selftests, but not wanting to follow prescribed build setup? Or start
> adding feature detection in Makefile and exclude some tests from being
> built? Will that start to be a maintenance burden?

In my personal opinion adding #ifndef-#define-#endif and feature
detection in the makefile would constitute maintenance burden, yes.

> I guess I just don't understand the direction here.





[Index of Archives]     [Linux Samsung SoC]     [Linux Rockchip SoC]     [Linux Actions SoC]     [Linux for Synopsys ARC Processors]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]


  Powered by Linux