[meta-intel] [PATCH 02/10] emenlow.conf: specify a kernel boot param for emgd

[Kamble, Nitin A]

> -----Original Message-----
> From: Zanussi, Tom
> Sent: Wednesday, December 05, 2012 8:30 AM
> To: Kamble, Nitin A
> Cc: Hart, Darren; meta-intel at yoctoproject.org; Bodke, Kishore K; Saxena,
> Rahul; Liu, Song
> Subject: Re: [PATCH 02/10] emenlow.conf: specify a kernel boot param for
> emgd
> 
> On Tue, 2012-12-04 at 15:11 -0800, nitin.a.kamble at intel.com wrote:
> > From: Nitin A Kamble <nitin.a.kamble at intel.com>
> >
> > Via "vmalloc=256MB" boot parameter ask kernel to keep enough memory
> > available for the emgd kernel driver allocation.
> >
> 
> Hi Nitin,
> 
> I think this was asked about in a previous patchset awhile back, but was there
> a specific reason for needing this i.e. did you run into problems without it or
> is it required for emenlow specifically with emgd?
> 
> Tom

Hi Tom,
   This is needed in h/w configurations with low RAM. If system has 1GB RAM then the vmalloc set by kernel is about 128MB, and EMGD driver will see memory allocation issues. EMGD driver documentation specifies that this parameter need to be passed to the kernel.
    As my eMenlow h/w has 2GB memory, kernel by itself sets vmalloc of 256MB, so it is not an issue on my eMenlow. But for other h/w configurations this parameter is needed.

Thanks,
Nitin


> 
> > Signed-off-by: Nitin A Kamble <nitin.a.kamble at intel.com>
> > ---
> >  meta-emenlow/conf/machine/emenlow.conf |    2 +-
> >  1 files changed, 1 insertions(+), 1 deletions(-)
> >
> > diff --git a/meta-emenlow/conf/machine/emenlow.conf
> > b/meta-emenlow/conf/machine/emenlow.conf
> > index 235498c..de316b7 100644
> > --- a/meta-emenlow/conf/machine/emenlow.conf
> > +++ b/meta-emenlow/conf/machine/emenlow.conf
> > @@ -23,7 +23,7 @@ PREFERRED_VERSION_xserver-xorg ?= "1.9.3"
> >  PREFERRED_VERSION_mesa-dri ?= "7.11"
> >  PREFERRED_VERSION_xf86-input-evdev ?= "2.6.0"
> >
> > -APPEND += "video=vesafb vga=0x318"
> > +APPEND += "video=vesafb vga=0x318 vmalloc=256MB"
> >
> >  VA_FEATURES = "${@bb.utils.contains("LICENSE_FLAGS_WHITELIST", \
> >  	    "commercial", "gst-va-intel va-intel", "va-intel", d)}"
>