linux-fsdevel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] fs: VALIDATE_FS_PARSER should default to n
@ 2019-04-01 11:53 Geert Uytterhoeven
  2019-04-30  8:56 ` Geert Uytterhoeven
  0 siblings, 1 reply; 2+ messages in thread
From: Geert Uytterhoeven @ 2019-04-01 11:53 UTC (permalink / raw)
  To: David Howells
  Cc: Alexander Viro, linux-fsdevel, linux-kernel, Geert Uytterhoeven

CONFIG_VALIDATE_FS_PARSER is a debugging tool to check that the parser
tables are vaguely sane.  It was set to default to 'Y' for the moment to
catch errors in upcoming fs conversion development.

Make sure it is not enabled by default in the final release of v5.1.

Fixes: 31d921c7fb969172 ("vfs: Add configuration parser helpers")
Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
---
 fs/Kconfig | 1 -
 1 file changed, 1 deletion(-)

diff --git a/fs/Kconfig b/fs/Kconfig
index 3e6d3101f3ff0dba..db921dc267d3b784 100644
--- a/fs/Kconfig
+++ b/fs/Kconfig
@@ -10,7 +10,6 @@ config DCACHE_WORD_ACCESS
 
 config VALIDATE_FS_PARSER
 	bool "Validate filesystem parameter description"
-	default y
 	help
 	  Enable this to perform validation of the parameter description for a
 	  filesystem when it is registered.
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] fs: VALIDATE_FS_PARSER should default to n
  2019-04-01 11:53 [PATCH] fs: VALIDATE_FS_PARSER should default to n Geert Uytterhoeven
@ 2019-04-30  8:56 ` Geert Uytterhoeven
  0 siblings, 0 replies; 2+ messages in thread
From: Geert Uytterhoeven @ 2019-04-30  8:56 UTC (permalink / raw)
  To: David Howells; +Cc: Alexander Viro, Linux FS Devel, Linux Kernel Mailing List

Hi David,

On Mon, Apr 1, 2019 at 1:54 PM Geert Uytterhoeven <geert@linux-m68k.org> wrote:
> CONFIG_VALIDATE_FS_PARSER is a debugging tool to check that the parser
> tables are vaguely sane.  It was set to default to 'Y' for the moment to
> catch errors in upcoming fs conversion development.
>
> Make sure it is not enabled by default in the final release of v5.1.
>
> Fixes: 31d921c7fb969172 ("vfs: Add configuration parser helpers")
> Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>

Ping?
Final release of v5.1 is imminent.

> --- a/fs/Kconfig
> +++ b/fs/Kconfig
> @@ -10,7 +10,6 @@ config DCACHE_WORD_ACCESS
>
>  config VALIDATE_FS_PARSER
>         bool "Validate filesystem parameter description"
> -       default y
>         help
>           Enable this to perform validation of the parameter description for a
>           filesystem when it is registered.

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2019-04-30  8:57 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-04-01 11:53 [PATCH] fs: VALIDATE_FS_PARSER should default to n Geert Uytterhoeven
2019-04-30  8:56 ` Geert Uytterhoeven

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).