Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1496984

Re: Change CONFIG_DEVKMEM default value to n

From Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Newsgroups linux.kernel
Subject Re: Change CONFIG_DEVKMEM default value to n
Date 2016-10-07 06:10 +0200
Message-ID <spuDU-2QL-17@gated-at.bofh.it> (permalink)
References <spsLM-1l4-5@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On Fri, Oct 07, 2016 at 10:04:11AM +0800, Dave Young wrote:
> Kconfig comment suggests setting it as "n" if in doubt thus move the
> default value to 'n'.
> 
> Signed-off-by: Dave Young <dyoung@redhat.com>
> Suggested-by: Kees Cook <keescook@chromium.org>
> ---
>  drivers/char/Kconfig |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> --- linux-x86.orig/drivers/char/Kconfig
> +++ linux-x86/drivers/char/Kconfig
> @@ -17,7 +17,7 @@ config DEVMEM
>  
>  config DEVKMEM
>  	bool "/dev/kmem virtual device support"
> -	default y
> +	default n

If you remove the "default" line, it defaults to 'n'.

And is it really "safe" to default this to n now?

thanks,

greg k-h

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

Change CONFIG_DEVKMEM default value to n Dave Young <dyoung@redhat.com> - 2016-10-07 04:10 +0200
  Re: Change CONFIG_DEVKMEM default value to n Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2016-10-07 06:10 +0200
    Re: Change CONFIG_DEVKMEM default value to n Kees Cook <keescook@chromium.org> - 2016-10-07 19:10 +0200
    Re: Change CONFIG_DEVKMEM default value to n Dave Young <dyoung@redhat.com> - 2016-10-10 04:50 +0200
      Re: Change CONFIG_DEVKMEM default value to n Dave Young <dyoung@redhat.com> - 2016-10-10 05:00 +0200
        Re: Change CONFIG_DEVKMEM default value to n Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2016-10-10 07:20 +0200
          Re: Change CONFIG_DEVKMEM default value to n Dave Young <dyoung@redhat.com> - 2016-10-10 07:40 +0200

csiph-web