[oe] [meta-handheld 4/6] linux-kexecboot: don't disable DEBUG, PRINTK, BUG with CMDLINE_DEBUG not just loglevel=3

[Andrea Adami]

On Thu, Nov 24, 2011 at 12:01 PM, Andrea Adami <andrea.adami at gmail.com> wrote:
> On Thu, Nov 24, 2011 at 11:18 AM, Martin Jansa <martin.jansa at gmail.com> wrote:
>> Signed-off-by: Martin Jansa <Martin.Jansa at gmail.com>
>> ---
>>  recipes-kernel/linux/linux-kexecboot.inc |   14 +++++++++++---
>>  1 files changed, 11 insertions(+), 3 deletions(-)
>>
>> diff --git a/recipes-kernel/linux/linux-kexecboot.inc b/recipes-kernel/linux/linux-kexecboot.inc
>> index 83b7018..220bf36 100644
>> --- a/recipes-kernel/linux/linux-kexecboot.inc
>> +++ b/recipes-kernel/linux/linux-kexecboot.inc
>> @@ -108,9 +108,17 @@ do_configure_prepend() {
>>         kernel_conf_variable KEXEC y
>>         CONF_SED_SCRIPT="$CONF_SED_SCRIPT /=m/d;"
>>         kernel_conf_variable MODULES n
>> -        kernel_conf_variable DEBUG_ERRORS n
>> -        kernel_conf_variable DEBUG_BUGVERBOSE n
>> -        kernel_conf_variable DEBUG_KERNEL n
>> +        if [ "${CMDLINE_DEBUG}" = "loglevel=3" ] ; then
>> +                kernel_conf_variable DEBUG_ERRORS n
>> +                kernel_conf_variable DEBUG_BUGVERBOSE n
>> +                kernel_conf_variable DEBUG_KERNEL n
>> +       else
>> +                kernel_conf_variable PRINTK y
>> +                kernel_conf_variable BUG y
>> +                kernel_conf_variable DEBUG_ERRORS y
>> +                kernel_conf_variable DEBUG_BUGVERBOSE y
>> +                kernel_conf_variable DEBUG_KERNEL y
>> +       fi
>>
>>         # Force lzma for arm kernels only
>>         if [ "${ARCH}" = "arm" ] ; then
>> --
>> 1.7.8.rc3
>>
>>
>> _______________________________________________
>> Openembedded-devel mailing list
>> Openembedded-devel at lists.openembedded.org
>> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
>>
>
> The idea is good!
> The weak point is EXTRA_IMAGE_FEATURES = "debug-tweaks" , default in local.conf.

Hm.. and what about using  "tools-debug" instead?

Andrea


>
> Remember for collie we have to watch the kernel size! I'm not sure we
> respect the limit, probably yes.
> No problem on other Zaurus, unless we want to renable networking and usb.
>
> Regards
>
> Andrea
>