* Change CONFIG_DEVKMEM default value to n
@ 2016-10-07 2:04 Dave Young
2016-10-07 3:57 ` Greg Kroah-Hartman
0 siblings, 1 reply; 7+ messages in thread
From: Dave Young @ 2016-10-07 2:04 UTC (permalink / raw)
To: Arnd Bergmann, Greg Kroah-Hartman; +Cc: Kees Cook, linux-kernel
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
help
Say Y here if you want to support the /dev/kmem device. The
/dev/kmem device is rarely used, but can be used for certain
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: Change CONFIG_DEVKMEM default value to n
2016-10-07 2:04 Change CONFIG_DEVKMEM default value to n Dave Young
@ 2016-10-07 3:57 ` Greg Kroah-Hartman
2016-10-07 17:02 ` Kees Cook
2016-10-10 2:44 ` Dave Young
0 siblings, 2 replies; 7+ messages in thread
From: Greg Kroah-Hartman @ 2016-10-07 3:57 UTC (permalink / raw)
To: Dave Young; +Cc: Arnd Bergmann, Kees Cook, linux-kernel
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
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: Change CONFIG_DEVKMEM default value to n
2016-10-07 3:57 ` Greg Kroah-Hartman
@ 2016-10-07 17:02 ` Kees Cook
2016-10-10 2:44 ` Dave Young
1 sibling, 0 replies; 7+ messages in thread
From: Kees Cook @ 2016-10-07 17:02 UTC (permalink / raw)
To: Greg Kroah-Hartman; +Cc: Dave Young, Arnd Bergmann, LKML
On Thu, Oct 6, 2016 at 8:57 PM, Greg Kroah-Hartman
<gregkh@linuxfoundation.org> wrote:
> 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?
This is DEVKMEM which distros haven't (shouldn't have?) used for a
while now. I'm not so bold as to suggest making DEVMEM default-n yet.
;)
-Kees
--
Kees Cook
Nexus Security
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: Change CONFIG_DEVKMEM default value to n
2016-10-07 3:57 ` Greg Kroah-Hartman
2016-10-07 17:02 ` Kees Cook
@ 2016-10-10 2:44 ` Dave Young
2016-10-10 2:50 ` Dave Young
1 sibling, 1 reply; 7+ messages in thread
From: Dave Young @ 2016-10-10 2:44 UTC (permalink / raw)
To: Greg Kroah-Hartman; +Cc: Arnd Bergmann, Kees Cook, linux-kernel
On 10/07/16 at 05:57am, Greg Kroah-Hartman wrote:
> 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'.
I personally perfer a "default n", but I can update it..
>
> And is it really "safe" to default this to n now?
There is an old article here:
https://lwn.net/Articles/147901/
AFAIK Distributions like Fedora/Debian has disabled it for long time.
If one really need it he can still enable it in his own config file.
>
> thanks,
>
> greg k-h
Thanks
Dave
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: Change CONFIG_DEVKMEM default value to n
2016-10-10 2:44 ` Dave Young
@ 2016-10-10 2:50 ` Dave Young
2016-10-10 5:12 ` Greg Kroah-Hartman
0 siblings, 1 reply; 7+ messages in thread
From: Dave Young @ 2016-10-10 2:50 UTC (permalink / raw)
To: Greg Kroah-Hartman; +Cc: Arnd Bergmann, Kees Cook, linux-kernel
On 10/10/16 at 10:44am, Dave Young wrote:
> On 10/07/16 at 05:57am, Greg Kroah-Hartman wrote:
> > 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'.
>
> I personally perfer a "default n", but I can update it..
Greg, here is an update with dropping the default line:
Move CONFIG_DEVKMEM default to n
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 | 1 -
1 file changed, 1 deletion(-)
--- linux-x86.orig/drivers/char/Kconfig
+++ linux-x86/drivers/char/Kconfig
@@ -17,7 +17,6 @@ config DEVMEM
config DEVKMEM
bool "/dev/kmem virtual device support"
- default y
help
Say Y here if you want to support the /dev/kmem device. The
/dev/kmem device is rarely used, but can be used for certain
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: Change CONFIG_DEVKMEM default value to n
2016-10-10 2:50 ` Dave Young
@ 2016-10-10 5:12 ` Greg Kroah-Hartman
2016-10-10 5:35 ` Dave Young
0 siblings, 1 reply; 7+ messages in thread
From: Greg Kroah-Hartman @ 2016-10-10 5:12 UTC (permalink / raw)
To: Dave Young; +Cc: Arnd Bergmann, Kees Cook, linux-kernel
On Mon, Oct 10, 2016 at 10:50:50AM +0800, Dave Young wrote:
> On 10/10/16 at 10:44am, Dave Young wrote:
> > On 10/07/16 at 05:57am, Greg Kroah-Hartman wrote:
> > > 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'.
> >
> > I personally perfer a "default n", but I can update it..
>
> Greg, here is an update with dropping the default line:
<snip>
Can you resend it in a format I can apply it in?
thanks,
greg k-h
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: Change CONFIG_DEVKMEM default value to n
2016-10-10 5:12 ` Greg Kroah-Hartman
@ 2016-10-10 5:35 ` Dave Young
0 siblings, 0 replies; 7+ messages in thread
From: Dave Young @ 2016-10-10 5:35 UTC (permalink / raw)
To: Greg Kroah-Hartman; +Cc: Arnd Bergmann, Kees Cook, linux-kernel
On 10/10/16 at 07:12am, Greg Kroah-Hartman wrote:
> On Mon, Oct 10, 2016 at 10:50:50AM +0800, Dave Young wrote:
> > On 10/10/16 at 10:44am, Dave Young wrote:
> > > On 10/07/16 at 05:57am, Greg Kroah-Hartman wrote:
> > > > 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'.
> > >
> > > I personally perfer a "default n", but I can update it..
> >
> > Greg, here is an update with dropping the default line:
>
> <snip>
>
> Can you resend it in a format I can apply it in?
Done, thanks you!
>
> thanks,
>
> greg k-h
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2016-10-10 5:35 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-10-07 2:04 Change CONFIG_DEVKMEM default value to n Dave Young
2016-10-07 3:57 ` Greg Kroah-Hartman
2016-10-07 17:02 ` Kees Cook
2016-10-10 2:44 ` Dave Young
2016-10-10 2:50 ` Dave Young
2016-10-10 5:12 ` Greg Kroah-Hartman
2016-10-10 5:35 ` Dave Young
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.