[v3,0/6] Enable as many KUnit tests as possible
mbox series

Message ID 20200511131350.29638-1-anders.roxell@linaro.org
Headers show
Series
  • Enable as many KUnit tests as possible
Related show

Message

Anders Roxell May 11, 2020, 1:13 p.m. UTC
Hi,

This patchset will try to enable as many KUnit test fragments as
possible for the current .config file.
This will make it easier for both developers that tests their specific
feature and also for test-systems that would like to get as much as
possible for their current .config file.

I will send a separate KCSAN KUnit patch after this patchset since that
isn't in mainline yet.

Since v2:
Fixed David's comments. KUNIT_RUN_ALL -> KUNIT_ALL_TESTS, and he
suggested a great help text.

Since v1:
Marco commented to split up the patches, and change a "." to a ",".


Cheers,
Anders

Anders Roxell (6):
  kunit: Kconfig: enable a KUNIT_ALL_TESTS fragment
  kunit: default KUNIT_* fragments to KUNIT_ALL_TESTS
  lib: Kconfig.debug: default KUNIT_* fragments to KUNIT_ALL_TESTS
  drivers: base: default KUNIT_* fragments to KUNIT_ALL_TESTS
  fs: ext4: default KUNIT_* fragments to KUNIT_ALL_TESTS
  security: apparmor: default KUNIT_* fragments to KUNIT_ALL_TESTS

 drivers/base/Kconfig      |  3 ++-
 drivers/base/test/Kconfig |  3 ++-
 fs/ext4/Kconfig           |  3 ++-
 lib/Kconfig.debug         |  6 ++++--
 lib/kunit/Kconfig         | 23 ++++++++++++++++++++---
 security/apparmor/Kconfig |  3 ++-
 6 files changed, 32 insertions(+), 9 deletions(-)

Comments

Anders Roxell May 27, 2020, 11:49 a.m. UTC | #1
Hi all,

Friendly ping: who can take this?

Cheers,
Anders

On Mon, 11 May 2020 at 15:14, Anders Roxell <anders.roxell@linaro.org> wrote:
>
> Hi,
>
> This patchset will try to enable as many KUnit test fragments as
> possible for the current .config file.
> This will make it easier for both developers that tests their specific
> feature and also for test-systems that would like to get as much as
> possible for their current .config file.
>
> I will send a separate KCSAN KUnit patch after this patchset since that
> isn't in mainline yet.
>
> Since v2:
> Fixed David's comments. KUNIT_RUN_ALL -> KUNIT_ALL_TESTS, and he
> suggested a great help text.
>
> Since v1:
> Marco commented to split up the patches, and change a "." to a ",".
>
>
> Cheers,
> Anders
>
> Anders Roxell (6):
>   kunit: Kconfig: enable a KUNIT_ALL_TESTS fragment
>   kunit: default KUNIT_* fragments to KUNIT_ALL_TESTS
>   lib: Kconfig.debug: default KUNIT_* fragments to KUNIT_ALL_TESTS
>   drivers: base: default KUNIT_* fragments to KUNIT_ALL_TESTS
>   fs: ext4: default KUNIT_* fragments to KUNIT_ALL_TESTS
>   security: apparmor: default KUNIT_* fragments to KUNIT_ALL_TESTS
>
>  drivers/base/Kconfig      |  3 ++-
>  drivers/base/test/Kconfig |  3 ++-
>  fs/ext4/Kconfig           |  3 ++-
>  lib/Kconfig.debug         |  6 ++++--
>  lib/kunit/Kconfig         | 23 ++++++++++++++++++++---
>  security/apparmor/Kconfig |  3 ++-
>  6 files changed, 32 insertions(+), 9 deletions(-)
>
> --
> 2.20.1
>
Brendan Higgins May 28, 2020, 7:07 p.m. UTC | #2
On Wed, May 27, 2020 at 4:49 AM Anders Roxell <anders.roxell@linaro.org> wrote:
>
> Hi all,
>
> Friendly ping: who can take this?

Sorry, I just reviewed the last patch.

Shuah, do you mind picking this up for 5.8?

> Cheers,
> Anders
>
> On Mon, 11 May 2020 at 15:14, Anders Roxell <anders.roxell@linaro.org> wrote:
> >
> > Hi,
> >
> > This patchset will try to enable as many KUnit test fragments as
> > possible for the current .config file.
> > This will make it easier for both developers that tests their specific
> > feature and also for test-systems that would like to get as much as
> > possible for their current .config file.
> >
> > I will send a separate KCSAN KUnit patch after this patchset since that
> > isn't in mainline yet.
> >
> > Since v2:
> > Fixed David's comments. KUNIT_RUN_ALL -> KUNIT_ALL_TESTS, and he
> > suggested a great help text.
> >
> > Since v1:
> > Marco commented to split up the patches, and change a "." to a ",".
> >
> >
> > Cheers,
> > Anders
> >
> > Anders Roxell (6):
> >   kunit: Kconfig: enable a KUNIT_ALL_TESTS fragment
> >   kunit: default KUNIT_* fragments to KUNIT_ALL_TESTS
> >   lib: Kconfig.debug: default KUNIT_* fragments to KUNIT_ALL_TESTS
> >   drivers: base: default KUNIT_* fragments to KUNIT_ALL_TESTS
> >   fs: ext4: default KUNIT_* fragments to KUNIT_ALL_TESTS
> >   security: apparmor: default KUNIT_* fragments to KUNIT_ALL_TESTS
> >
> >  drivers/base/Kconfig      |  3 ++-
> >  drivers/base/test/Kconfig |  3 ++-
> >  fs/ext4/Kconfig           |  3 ++-
> >  lib/Kconfig.debug         |  6 ++++--
> >  lib/kunit/Kconfig         | 23 ++++++++++++++++++++---
> >  security/apparmor/Kconfig |  3 ++-
> >  6 files changed, 32 insertions(+), 9 deletions(-)
> >
> > --
> > 2.20.1
> >

Thanks!
Shuah Khan May 28, 2020, 7:13 p.m. UTC | #3
On 5/28/20 1:07 PM, Brendan Higgins wrote:
> On Wed, May 27, 2020 at 4:49 AM Anders Roxell <anders.roxell@linaro.org> wrote:
>>
>> Hi all,
>>
>> Friendly ping: who can take this?
> 
> Sorry, I just reviewed the last patch.
> 
> Shuah, do you mind picking this up for 5.8?
> 

Yup. Will do. I was watching this thread waiting for your
Ack. I will apply it once.

thanks,
-- Shuah
Shuah Khan June 2, 2020, 7:40 p.m. UTC | #4
On 5/28/20 1:13 PM, Shuah Khan wrote:
> On 5/28/20 1:07 PM, Brendan Higgins wrote:
>> On Wed, May 27, 2020 at 4:49 AM Anders Roxell 
>> <anders.roxell@linaro.org> wrote:
>>>
>>> Hi all,
>>>
>>> Friendly ping: who can take this?
>>
>> Sorry, I just reviewed the last patch.
>>
>> Shuah, do you mind picking this up for 5.8?
>>
> 
> Yup. Will do. I was watching this thread waiting for your
> Ack. I will apply it once.
> 

Applied to linux-kselftest kunit branch for Linux 5.8-rc1

thanks,
-- Shuah