[PATCH] Documentation: Add detailed explanation for 'N' taint flag

Benjamin Poirier posted 1 patch 1 year, 5 months ago
Documentation/admin-guide/tainted-kernels.rst | 2 ++
1 file changed, 2 insertions(+)
[PATCH] Documentation: Add detailed explanation for 'N' taint flag
Posted by Benjamin Poirier 1 year, 5 months ago
Every taint flag has an entry in the "More detailed explanation" section
except for the 'N' flag. That omission was probably just an oversight so
add an entry for that flag.

Fixes: 2852ca7fba9f ("panic: Taint kernel if tests are run")
Signed-off-by: Benjamin Poirier <bpoirier@nvidia.com>
---
 Documentation/admin-guide/tainted-kernels.rst | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/Documentation/admin-guide/tainted-kernels.rst b/Documentation/admin-guide/tainted-kernels.rst
index f92551539e8a..700aa72eecb1 100644
--- a/Documentation/admin-guide/tainted-kernels.rst
+++ b/Documentation/admin-guide/tainted-kernels.rst
@@ -182,3 +182,5 @@ More detailed explanation for tainting
      produce extremely unusual kernel structure layouts (even performance
      pathological ones), which is important to know when debugging. Set at
      build time.
+
+ 18) ``N`` if an in-kernel test, such as a KUnit test, has been run.
-- 
2.45.2
Re: [PATCH] Documentation: Add detailed explanation for 'N' taint flag
Posted by Jonathan Corbet 1 year, 4 months ago
Benjamin Poirier <bpoirier@nvidia.com> writes:

> Every taint flag has an entry in the "More detailed explanation" section
> except for the 'N' flag. That omission was probably just an oversight so
> add an entry for that flag.
>
> Fixes: 2852ca7fba9f ("panic: Taint kernel if tests are run")
> Signed-off-by: Benjamin Poirier <bpoirier@nvidia.com>
> ---
>  Documentation/admin-guide/tainted-kernels.rst | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/Documentation/admin-guide/tainted-kernels.rst b/Documentation/admin-guide/tainted-kernels.rst
> index f92551539e8a..700aa72eecb1 100644
> --- a/Documentation/admin-guide/tainted-kernels.rst
> +++ b/Documentation/admin-guide/tainted-kernels.rst
> @@ -182,3 +182,5 @@ More detailed explanation for tainting
>       produce extremely unusual kernel structure layouts (even performance
>       pathological ones), which is important to know when debugging. Set at
>       build time.
> +
> + 18) ``N`` if an in-kernel test, such as a KUnit test, has been run.

Applied, thanks.

jon
Re: [PATCH] Documentation: Add detailed explanation for 'N' taint flag
Posted by David Gow 1 year, 5 months ago
On Thu, 18 Jul 2024 at 04:36, Benjamin Poirier <bpoirier@nvidia.com> wrote:
>
> Every taint flag has an entry in the "More detailed explanation" section
> except for the 'N' flag. That omission was probably just an oversight so
> add an entry for that flag.
>
> Fixes: 2852ca7fba9f ("panic: Taint kernel if tests are run")
> Signed-off-by: Benjamin Poirier <bpoirier@nvidia.com>
> ---

Nice catch, thanks!

Reviewed-by: David Gow <davidgow@google.com>

Cheers,
-- David


>  Documentation/admin-guide/tainted-kernels.rst | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/Documentation/admin-guide/tainted-kernels.rst b/Documentation/admin-guide/tainted-kernels.rst
> index f92551539e8a..700aa72eecb1 100644
> --- a/Documentation/admin-guide/tainted-kernels.rst
> +++ b/Documentation/admin-guide/tainted-kernels.rst
> @@ -182,3 +182,5 @@ More detailed explanation for tainting
>       produce extremely unusual kernel structure layouts (even performance
>       pathological ones), which is important to know when debugging. Set at
>       build time.
> +
> + 18) ``N`` if an in-kernel test, such as a KUnit test, has been run.
> --
> 2.45.2
>
Re: [PATCH] Documentation: Add detailed explanation for 'N' taint flag
Posted by Luis Chamberlain 1 year, 5 months ago
On Wed, Jul 17, 2024 at 04:35:21PM -0400, Benjamin Poirier wrote:
> Every taint flag has an entry in the "More detailed explanation" section
> except for the 'N' flag. That omission was probably just an oversight so
> add an entry for that flag.
> 
> Fixes: 2852ca7fba9f ("panic: Taint kernel if tests are run")
> Signed-off-by: Benjamin Poirier <bpoirier@nvidia.com>
> ---

Acked-by: Luis Chamberlain <mcgrof@kernel.org>

  Luis