Re: [yocto] [meta-security][PATCH] layer: add more memory for Qemu machines

[Mikko Rapeli <mikko.rapeli@linaro.org>]

Hi,

On Wed, Jun 28, 2023 at 05:47:21PM +0100, Richard Purdie wrote:
> On Wed, 2023-06-28 at 08:56 -0400, Armin Kuster wrote:
> > Signed-off-by: Armin Kuster <akuster808@gmail.com>
> > ---
> >  conf/layer.conf | 3 +++
> >  1 file changed, 3 insertions(+)
> > 
> > diff --git a/conf/layer.conf b/conf/layer.conf
> > index 334a945..5f289cb 100644
> > --- a/conf/layer.conf
> > +++ b/conf/layer.conf
> > @@ -28,4 +28,7 @@ INHERIT += "sanity-meta-security"
> >  
> >  QB_KERNEL_CMDLINE_APPEND = " ${@bb.utils.contains('DISTRO_FEATURES', 'apparmor', 'apparmor=1 security=apparmor', '', d)}"
> >  
> > +# We need more mem to run many apps in this layer
> > +QB_MEM="-m 2048"
> > +
> >  addpylib ${LAYERDIR}/lib oeqa
> 
> Putting that unconditionally in a layer.conf is a pretty poor thing to
> do for usability IMO as it effectively forces that decision on anyone
> including the layer. There has to be a better way to handle that, at
> least conditionally on some override?

Currently these belong logically to machine config but what about generic
machine targets. Could an image config define how much memory is needed
to run the image on qemu?

Cheers,

-Mikko