* [PATCH v2 6/6] security: apparmor: default KUNIT_* fragments to KUNIT_RUN_ALL
@ 2020-05-05 10:27 Anders Roxell
2020-05-07 3:23 ` David Gow
0 siblings, 1 reply; 2+ messages in thread
From: Anders Roxell @ 2020-05-05 10:27 UTC (permalink / raw)
To: john.johansen, jmorris, serge
Cc: gregkh, tytso, adilger.kernel, akpm, brendanhiggins,
=linux-kselftest, kunit-dev, linux-kernel, linux-ext4,
linux-security-module, elver, davidgow, Anders Roxell
This makes it easier to enable all KUnit fragments.
Adding 'if !KUNIT_RUN_ALL' so individual test can be turned of if
someone wants that even though KUNIT_RUN_ALL is enabled.
Signed-off-by: Anders Roxell <anders.roxell@linaro.org>
---
security/apparmor/Kconfig | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/security/apparmor/Kconfig b/security/apparmor/Kconfig
index 0fe336860773..c4648426ea5d 100644
--- a/security/apparmor/Kconfig
+++ b/security/apparmor/Kconfig
@@ -70,8 +70,9 @@ config SECURITY_APPARMOR_DEBUG_MESSAGES
the kernel message buffer.
config SECURITY_APPARMOR_KUNIT_TEST
- bool "Build KUnit tests for policy_unpack.c"
+ bool "Build KUnit tests for policy_unpack.c" if !KUNIT_RUN_ALL
depends on KUNIT=y && SECURITY_APPARMOR
+ default KUNIT_RUN_ALL
help
This builds the AppArmor KUnit tests.
--
2.20.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH v2 6/6] security: apparmor: default KUNIT_* fragments to KUNIT_RUN_ALL
2020-05-05 10:27 [PATCH v2 6/6] security: apparmor: default KUNIT_* fragments to KUNIT_RUN_ALL Anders Roxell
@ 2020-05-07 3:23 ` David Gow
0 siblings, 0 replies; 2+ messages in thread
From: David Gow @ 2020-05-07 3:23 UTC (permalink / raw)
To: Anders Roxell
Cc: John Johansen, jmorris, serge, Greg Kroah-Hartman,
Theodore Ts'o, adilger.kernel, Andrew Morton, Brendan Higgins,
=linux-kselftest, KUnit Development, Linux Kernel Mailing List,
linux-ext4, linux-security-module, Marco Elver
On Tue, May 5, 2020 at 6:27 PM Anders Roxell <anders.roxell@linaro.org> wrote:
>
> This makes it easier to enable all KUnit fragments.
>
> Adding 'if !KUNIT_RUN_ALL' so individual test can be turned of if
> someone wants that even though KUNIT_RUN_ALL is enabled.
Again, as with the other patches, might be worth revising this
description when changing the config name.
>
> Signed-off-by: Anders Roxell <anders.roxell@linaro.org>
Reviewed-by: David Gow <davidgow@google.com>
Thanks again for the patchset, it's working well for me.
-- David
> ---
> security/apparmor/Kconfig | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/security/apparmor/Kconfig b/security/apparmor/Kconfig
> index 0fe336860773..c4648426ea5d 100644
> --- a/security/apparmor/Kconfig
> +++ b/security/apparmor/Kconfig
> @@ -70,8 +70,9 @@ config SECURITY_APPARMOR_DEBUG_MESSAGES
> the kernel message buffer.
>
> config SECURITY_APPARMOR_KUNIT_TEST
> - bool "Build KUnit tests for policy_unpack.c"
> + bool "Build KUnit tests for policy_unpack.c" if !KUNIT_RUN_ALL
> depends on KUNIT=y && SECURITY_APPARMOR
> + default KUNIT_RUN_ALL
> help
> This builds the AppArmor KUnit tests.
>
> --
> 2.20.1
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-05-07 3:23 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-05-05 10:27 [PATCH v2 6/6] security: apparmor: default KUNIT_* fragments to KUNIT_RUN_ALL Anders Roxell
2020-05-07 3:23 ` David Gow
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).