Re: [PATCH 2/2] debugfs: Add access restriction option

2020-07-20 Thread Enderborg, Peter
On 7/15/20 11:39 AM, Greg Kroah-Hartman wrote: > On Wed, Jul 15, 2020 at 10:42:07AM +0200, Peter Enderborg wrote: >> Since debugfs include sensitive information it need to be treated >> carefully. But it also has many very useful debug functions for userspace. >> With this option we can have same c

Re: [PATCH 2/2] debugfs: Add access restriction option

2020-07-16 Thread Randy Dunlap
On 7/16/20 12:15 AM, Peter Enderborg wrote: > --- > .../admin-guide/kernel-parameters.txt | 15 +++ > fs/debugfs/inode.c| 39 +++ > fs/debugfs/internal.h | 14 +++ > lib/Kconfig.debug |

[PATCH 2/2] debugfs: Add access restriction option

2020-07-16 Thread Peter Enderborg
Since debugfs include sensitive information it need to be treated carefully. But it also has many very useful debug functions for userspace. With this option we can have same configuration for system with need of debugfs and a way to turn it off. This gives a extra protection for exposure on system

Re: [PATCH 2/2] debugfs: Add access restriction option

2020-07-15 Thread Randy Dunlap
Hi Peter, Here are a few more comments/corrections. On 7/15/20 9:54 PM, Peter Enderborg wrote: > Since debugfs include sensitive information it need to be treated > carefully. But it also has many very useful debug functions for userspace. > With this option we can have same configuration for sys

[PATCH 2/2] debugfs: Add access restriction option

2020-07-15 Thread Peter Enderborg
Since debugfs include sensitive information it need to be treated carefully. But it also has many very useful debug functions for userspace. With this option we can have same configuration for system with need of debugfs and a way to turn it off. This gives a extra protection for exposure on system

Re: [PATCH 2/2] debugfs: Add access restriction option

2020-07-15 Thread Randy Dunlap
Hi, On 7/15/20 8:25 AM, Peter Enderborg wrote: > diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug > index 9ad9210d70a1..aec81f38bfce 100644 > --- a/lib/Kconfig.debug > +++ b/lib/Kconfig.debug > @@ -476,6 +476,38 @@ config DEBUG_FS > > If unsure, say N. > > +choice > + prompt "Deb

[PATCH 2/2] debugfs: Add access restriction option

2020-07-15 Thread Peter Enderborg
Since debugfs include sensitive information it need to be treated carefully. But it also has many very useful debug functions for userspace. With this option we can have same configuration for system with need of debugfs and a way to turn it off. This gives a extra protection for exposure on system

Re: [PATCH 2/2] debugfs: Add access restriction option

2020-07-15 Thread Enderborg, Peter
On 7/15/20 12:35 PM, Greg Kroah-Hartman wrote: > On Wed, Jul 15, 2020 at 10:03:19AM +, Enderborg, Peter wrote: >> On 7/15/20 11:39 AM, Greg Kroah-Hartman wrote: >>> On Wed, Jul 15, 2020 at 10:42:07AM +0200, Peter Enderborg wrote: Since debugfs include sensitive information it need to be tr

Re: [PATCH 2/2] debugfs: Add access restriction option

2020-07-15 Thread Greg Kroah-Hartman
On Wed, Jul 15, 2020 at 10:03:19AM +, Enderborg, Peter wrote: > On 7/15/20 11:39 AM, Greg Kroah-Hartman wrote: > > On Wed, Jul 15, 2020 at 10:42:07AM +0200, Peter Enderborg wrote: > >> Since debugfs include sensitive information it need to be treated > >> carefully. But it also has many very us

Re: [PATCH 2/2] debugfs: Add access restriction option

2020-07-15 Thread Enderborg, Peter
On 7/15/20 11:39 AM, Greg Kroah-Hartman wrote: > On Wed, Jul 15, 2020 at 10:42:07AM +0200, Peter Enderborg wrote: >> Since debugfs include sensitive information it need to be treated >> carefully. But it also has many very useful debug functions for userspace. >> With this option we can have same c

Re: [PATCH 2/2] debugfs: Add access restriction option

2020-07-15 Thread Greg Kroah-Hartman
On Wed, Jul 15, 2020 at 10:42:07AM +0200, Peter Enderborg wrote: > Since debugfs include sensitive information it need to be treated > carefully. But it also has many very useful debug functions for userspace. > With this option we can have same configuration for system with > need of debugfs and a

[PATCH 2/2] debugfs: Add access restriction option

2020-07-15 Thread Peter Enderborg
Since debugfs include sensitive information it need to be treated carefully. But it also has many very useful debug functions for userspace. With this option we can have same configuration for system with need of debugfs and a way to turn it off. This gives a extra protection for exposure on system

Re: [PATCH 2/2] debugfs: Add access restriction option

2020-07-10 Thread Greg Kroah-Hartman
On Mon, Jun 22, 2020 at 10:30:19AM +0200, Peter Enderborg wrote: > Since debugfs include sensitive information it need to be treated > carefully. But it also has many very useful debug functions for userspace. > With this option we can have same configuration for system with > need of debugfs and a

[PATCH 2/2] debugfs: Add access restriction option

2020-06-22 Thread Peter Enderborg
Since debugfs include sensitive information it need to be treated carefully. But it also has many very useful debug functions for userspace. With this option we can have same configuration for system with need of debugfs and a way to turn it off. This gives a extra protection for exposure on system