From mboxrd@z Thu Jan 1 00:00:00 1970 Date: Fri, 12 Oct 2018 18:39:39 +0200 From: "Mark Brown" Subject: Re: [kernelci] add a new config fragment to kernel/configs Message-ID: <20181012163939.GD2340@sirena.org.uk> References: <7hin27fhkk.fsf@baylibre.com> MIME-Version: 1.0 In-Reply-To: <7hin27fhkk.fsf@baylibre.com> Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="jCrbxBqMcLqd4mOl" Content-Disposition: inline List-ID: To: Kevin Hilman Cc: Anders Roxell , kernelci@groups.io --jCrbxBqMcLqd4mOl Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Fri, Oct 12, 2018 at 02:57:47PM +0200, Kevin Hilman wrote: > Anders Roxell writes: > > Let me try to explain what the idea with this new fragment is about. Adding a > > fragment called all_abi.config (lets stick with this name throughout this > > email) in kernel/configs would be to enable as much as possible. Then we would > > be able to run any test suite with one kernel image. > Thanks for clarifing. To me, that doesn't sound like ABI, and > especially not *all* ABIs. It's just a set of interfaces used by common > test suites. IMO, using ABI is misleading, as when people see that, > they think of all the userspace/kernel interfaces. Right, so when we were discussing this internally the thinking was that we could meet the need for a config fragement that enabled interfaces for testsuites by enabling all the ABIs that the kernel offers. This is a useful thing to do anyway I think and it provides a concrete rule for what to put in there which is easy to follow and avoids debate about some of the tuning options that people like to twiddle. There was also some thought that we could have other config fragments like one to enable useful and cheap debug features which could be useful separately. I do agree that if it is just a "stuff people might want for testing" config then we shouldn't call it all_abi. > > 2. do allmodconfig and boot that, sadly that doesn't boot. I've tried to bisect > > the .config file and ended up with a pile of CONFIG_* options that has to be > > enabled/disabled so it booted. The idea was to add these options into the > > all_abi.config so we could do 'make allmodconfig all_abi.config'. There are > > different options that needs to be enabled/disabled between arm64 and x86 > > (those are the architectures I've tried on so far). > Having this "minimal boot" defconfig sounds like a good thing to have > independently of the test fragments. Definitely. > IMO, per test-suite fragments make the most sense, and will much easier > to maintain than one big fragment. It's also more self-documenting, and > test-suite maintainers might be more motivated to keep things up to > date. With one massive fragment, it would have to be well documented > (and kept that way) to understand why various options are present, and > which test suites required them etc. The idea with having this all_abi config which literally only has things that are ABIs was to have a simple, clear rule about what goes in which doesn't need input from most testsuite maintainers or particularly reference any individual testsuites but will do what's needed for most of them. If something's an ABI it should be enabled, if it's not an ABI then it should be in some other fragment. That sidesteps the issues with documentation. Per testsuite configs have the disadvantage that you've got to work with every single testsuite to add them (or maintain them separately) and deal with any churn in kernel config requirements as you move between kernels with new options or whatever somehow. Some of the testsuites will doubtless still end up needing them but hopefully most won't. --jCrbxBqMcLqd4mOl Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAlvAzkgACgkQJNaLcl1U h9DSSgf/ainJPAOiIu9Pxfl+dVLOyodCxIwVqye/wgQb6S4JLxxf2BsaNMv23Y9l rGYb7u73YIM4anGBjQBo66ypW8n2afcTqVQMc5sbO/aATvXAcGU0wQKSiMrXWHM3 08oOvp8SC0OO4O9T6mKDar9mgVL1U8EdSR8927MNewdIHFBbX/CwIXPdbicXqROe wgju8pRBhw+FvFAbK5TbxwgJKUNgxaQBCIGcU/UqfMvD+10li1q1QRHR8oH7NiN4 wHPOn0qwczbKGU8sqbpRL696ppQ6qRH5xQ0be7Jdi1UJ09JZCo1Kj3+IuferRcq8 CFdU6bXOFnREqSc6a+KJ+0+z35hJ2w== =FEfz -----END PGP SIGNATURE----- --jCrbxBqMcLqd4mOl--