[meta-intel] [PATCH 4/5] fri2.conf: fix boot time kernel errors

[Darren Hart]

On 01/30/2013 01:59 PM, nitin.a.kamble at intel.com wrote:
> From: Nitin A Kamble <nitin.a.kamble at intel.com>
> 
> The fri2 kernel is hitting kernel BUGs while booting. See the YP
> bug #3810 for more details.

Please include details of the BUGs in the commit message.

> 
> FRI2 has 1GB of ram, hence the kernel sets 128MB for the vmalloc
> allocation memory. And the emgd driver needs at least 256MB vmalloc
> memory to work correctly. This commit specifies the vmalloc allocation
> size to the kernel by the vmalloc=256MB boot parameter.
> 
> Adjusting the kernel's vmalloc allocation size fixes the kernel errors.
> 

OK.... why haven't we seen this in the past? We shipped a ton of these
which didn't see this. Has something else changed recently? EMGD version
maybe?

> This fixes bug:
> [Yocto #3810]
> 
> Signed-off-by: Nitin A Kamble <nitin.a.kamble at intel.com>
> ---
>  meta-fri2/conf/machine/fri2.conf |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/meta-fri2/conf/machine/fri2.conf b/meta-fri2/conf/machine/fri2.conf
> index 40f2cdc..6ee2335 100644
> --- a/meta-fri2/conf/machine/fri2.conf
> +++ b/meta-fri2/conf/machine/fri2.conf
> @@ -37,4 +37,4 @@ SERIAL_CONSOLE = "115200 ttyPCH1"
>  APPEND += "console=ttyPCH1,115200 console=tty0"
>  
>  # EFI boot will ignore this and use the EFI framebuffer at 800x600

This comment needs to be updated. Perhaps adding vmalloc on a separate line
would make it clearer. The vmalloc line specifically needs to be documented.

> -APPEND += "video=vesafb vga=0x318"
> +APPEND += "video=vesafb vga=0x318 vmalloc=256MB"
> 

-- 
Darren Hart
Intel Open Source Technology Center
Yocto Project - Linux Kernel