[OE-core] [PATCH] busybox: add devmem

[Richard Purdie]

On Wed, 2019-01-30 at 14:18 -0800, Khem Raj wrote:
> 
> 
> On Wed, Jan 30, 2019 at 12:31 PM Adrian Bunk <bunk at stusta.de> wrote:
> > On Wed, Jan 30, 2019 at 08:50:02AM -0800, Khem Raj wrote:
> > > On Wed, Jan 30, 2019 at 1:34 AM Adrian Bunk <bunk at stusta.de>
> > wrote:
> > > >
> > > > This is a tiny but pretty useful tool.
> > > >
> > > 
> > > question is, do we need this enabled in default config, or could
> > be add it via
> > > some DISTRO_FEATURE meant for validation etc. May be if you
> > explain your
> > > usecase then we might be able to make a better assessment.
> > 
> > Sorry for being too terse.
> > 
> > devmem allows reading and writing hardware configuration registers,
> > which is useful both for debugging and for scripts.
> > 
> 
> Thanks for the info this seems useful can you also report how much
> does it increase size of busybox 

One reason I'm a little nervous of devmem in busybox is security attack
surface. You can of course argue that there are 101 ways to mimic
devmem so this isn't a concern. When you consider some configurations
can have suid busybox, it becomes more of a worry. Our default
separates out the suid pieces for the attack surface reason.

It is useful so I am torn but its worth keeping this in mind...

Cheers,

Richard