On 09/05/2025 18:16, Jakub Kicinski wrote:
On Fri, 9 May 2025 08:43:05 +0300 Tariq Toukan wrote:
+ -
+ name: devlink-rate-tc-index-max
+ header: uapi/linux/devlink.h
+ type: const
+ value: 7
Hi Jakub,
Ugh, still wrong, the user space headers don't have uAPI in the path
when installed. They go to /usr/include/linux/$name.h
But for defines "local" to the family you don't have to specify header:
at all, just drop it.
And please do build tests the next version:
make -C tools/net/ynl/ W=1 -j
Noted.
I'm going to also give you a hint that my next complaint will be that
there are no selftests in this series, and it extends uAPI.
I have some questions, looking for answers in an official source.
Most importantly, it is unclear to me when a selftest is required. What
is the guideline?
Please point me to any guidance for this selftest requirement. Is it
generic, or networking subsystem specific?
Let's make sure these things are well-defined, so we plan the extra
effort accordingly in future features, and improve predictability.
I reviewed
Documentation/process/maintainer-netdev.rst
Documentation/dev-tools/testing-overview.rst
Documentation/dev-tools/testing-devices.rst
Documentation/dev-tools/kselftest.rst
but couldn't find all answers.
BTW, some sources like the webpage link
https://kselftest.wiki.kernel.org/ in
Documentation/dev-tools/kselftest.rst is marked "OBSOLETE CONTENT".
Should be totally ignored?
Also, should the selftests require vendor-specific hardware to run? or
they imply adding netdevsim implementation?
Regards,
Tariq
