On Thu, 5 Dec 2024 at 21:05, Mark Brown <broonie@xxxxxxxxxx> wrote: > Should this be a build dependency or only a runtime dependency, or > should these be separate options for cases where the selftest builds a > module? Hello, thanks for looking through my patch. Some tests fail to compile and throw errors in case their required config options are not enabled. This optional check was created to prevent such issues. > If people are building the selftests once and then using them > with a bunch of kernel builds it might be surprising if some of the > binaries vanish. I'm not really familiar with packaging selftests, but this patch does not remove existing binaries when running tests. If I misunderstood what you are referring to, please let me know. > Shouldn't we try the current kernel tree, and for runtime checks > /proc/config.gz would be good to check when it's enabled? When I looked into this, it seems (according to the config.gz man page) that the contents of /proc/config.gz are the same as /lib/modules/$(uname -r)/build/.config, but /proc/config.gz is only available if the kernel was compiled with CONFIG_IKCONFIG_PROC enabled. It does seem like /lib/modules/$(uname -r)/build/scripts is not always available though, so will send in a new patch directly checking build/.config after checking it out on a few more systems.
