Re: [PATCH] powerpc/mm: add devmem_is_allowed() for STRICT_DEVMEM checking

2011-06-29 Thread Sukadev Bhattiprolu
On 06/14/2011 12:04 PM, Scott Wood wrote: On Tue, 14 Jun 2011 14:17:01 -0400 Steve Best wrote: On Tue, 2011-06-14 at 12:30 -0500, Nathan Lynch wrote: Hi Steve, On Tue, 2011-06-14 at 12:58 -0400, Steve Best wrote: +/* + * devmem_is_allowed() checks to see if /dev/mem access to a certain addr

Re: [PATCH] powerpc/mm: add devmem_is_allowed() for STRICT_DEVMEM checking

2011-06-14 Thread Scott Wood
On Tue, 14 Jun 2011 14:17:01 -0400 Steve Best wrote: > On Tue, 2011-06-14 at 12:30 -0500, Nathan Lynch wrote: > > Hi Steve, > > > > On Tue, 2011-06-14 at 12:58 -0400, Steve Best wrote: > > > +/* > > > + * devmem_is_allowed() checks to see if /dev/mem access to a certain > > > address > > > + *

Re: [PATCH] powerpc/mm: add devmem_is_allowed() for STRICT_DEVMEM checking

2011-06-14 Thread Steve Best
On Tue, 2011-06-14 at 12:30 -0500, Nathan Lynch wrote: > Hi Steve, > > On Tue, 2011-06-14 at 12:58 -0400, Steve Best wrote: > > diff --git a/arch/powerpc/Kconfig.debug b/arch/powerpc/Kconfig.debug > > index e72dcf6..e1aab6b 100644 > > --- a/arch/powerpc/Kconfig.debug > > +++ b/arch/powerpc/Kconfi

Re: [PATCH] powerpc/mm: add devmem_is_allowed() for STRICT_DEVMEM checking

2011-06-14 Thread Nathan Lynch
Hi Steve, On Tue, 2011-06-14 at 12:58 -0400, Steve Best wrote: > diff --git a/arch/powerpc/Kconfig.debug b/arch/powerpc/Kconfig.debug > index e72dcf6..e1aab6b 100644 > --- a/arch/powerpc/Kconfig.debug > +++ b/arch/powerpc/Kconfig.debug > @@ -283,4 +283,15 @@ config PPC_EARLY_DEBUG_CPM_ADDR >

[PATCH] powerpc/mm: add devmem_is_allowed() for STRICT_DEVMEM checking

2011-06-14 Thread Steve Best
Provide devmem_is_allowed() routine to restrict access to kernel memory from userspace. Set CONFIG_STRICT_DEVMEM config option to switch on checking. Signed-off-by: Steve Best diff --git a/arch/powerpc/Kconfig.debug b/arch/powerpc/Kconfig.debug index e72dcf6..e1aab6b 1

Re: [PATCH] powerpc/mm: add devmem_is_allowed() for STRICT_DEVMEM checking

2011-02-01 Thread Scott Wood
On Tue, 1 Feb 2011 12:21:45 -0500 Steve Best wrote: > > On Mon, 2011-01-31 at 13:40 -0600, Scott Wood wrote: > > I don't see how this is a sane thing to turn on by default (you're not > > restricting it, BTW -- you're completely disabling it with that > > implementation of devmem_is_allowed). I

Re: [PATCH] powerpc/mm: add devmem_is_allowed() for STRICT_DEVMEM checking

2011-02-01 Thread Steve Best
On Mon, 2011-01-31 at 13:40 -0600, Scott Wood wrote: > On Mon, 31 Jan 2011 14:16:00 -0500 > Steve Best wrote: > > > Provide devmem_is_allowed() routine to restrict access to kernel > > memory from userspace. > > Set CONFIG_STRICT_DEVMEM config option to switch on checking. > > > > S

Re: [PATCH] powerpc/mm: add devmem_is_allowed() for STRICT_DEVMEM checking

2011-01-31 Thread Scott Wood
On Mon, 31 Jan 2011 14:16:00 -0500 Steve Best wrote: > Provide devmem_is_allowed() routine to restrict access to kernel > memory from userspace. > Set CONFIG_STRICT_DEVMEM config option to switch on checking. > > Signed-off-by: Steve Best > > diff --git a/arch/powerpc/Kconfig.debug

Re: [PATCH] powerpc/mm: add devmem_is_allowed() for STRICT_DEVMEM checking

2011-01-31 Thread Josh Boyer
On Mon, Jan 31, 2011 at 2:25 PM, Josh Boyer wrote: > On Mon, Jan 31, 2011 at 2:16 PM, Steve Best wrote: >>    Provide devmem_is_allowed() routine to restrict access to kernel >>    memory from userspace. >>    Set CONFIG_STRICT_DEVMEM config option to switch on checking. >> >> Signed-off-by: Stev

Re: [PATCH] powerpc/mm: add devmem_is_allowed() for STRICT_DEVMEM checking

2011-01-31 Thread Josh Boyer
On Mon, Jan 31, 2011 at 2:16 PM, Steve Best wrote: >    Provide devmem_is_allowed() routine to restrict access to kernel >    memory from userspace. >    Set CONFIG_STRICT_DEVMEM config option to switch on checking. > > Signed-off-by: Steve Best > > diff --git a/arch/powerpc/Kconfig.debug b/arch/

[PATCH] powerpc/mm: add devmem_is_allowed() for STRICT_DEVMEM checking

2011-01-31 Thread Steve Best
Provide devmem_is_allowed() routine to restrict access to kernel memory from userspace. Set CONFIG_STRICT_DEVMEM config option to switch on checking. Signed-off-by: Steve Best diff --git a/arch/powerpc/Kconfig.debug b/arch/powerpc/Kconfig.debug index 2d38a50..6805d5d 100644 --- a/arc