Re: [PATCH A or B] Fix for bug #853 - matchbox-panel segfaults, X becomes unusable

[Darren Hart <dvhart@linux.intel.com>]

On 03/16/2011 07:13 PM, Yu Ke wrote:
> On Mar 17, 00:13, Richard Purdie wrote:
>> On Wed, 2011-03-16 at 14:47 -0700, Darren Hart wrote:
>>> The root cause of the matchbox-panel segfault is the same as for
>>> the previous two bugs addressing the same issue:
>>>
>>> o Bug 224 - [Netbook/emenlow] matchbox-panel segfault after X
>>> startup o Bug 738 - matchbox-panel segfault after X startup
>>>
>>> There apparently is a bug in gcc which results in the omission of
>>> some code leading to the segfault on core2. The following flags
>>> resolve the problem.
>>>
>>> FULL_OPTIMIZATION = "-fexpensive-optimizations -frename-registers
>>> -O2 -ggdb -feliminate-unused-debug-types"
>>>
>>> We can continue to patch each new machine's gtk+ recipe that is
>>> based on atom-pc.conf, or we can just add the above to
>>> tune-atom.conf. I've tested both on both the n450 and a Toshiba
>>> NB305 netbook, and each solution is effective.
>>>
>>> Chaging it in tune-atom.conf likely fixes other bugs we either
>>> haven't solved yet or simply haven't hit. It is also much more
>>> invasive, and we may not be willing to accept that this close to
>>> 1.0. The patches follow, and we can include one or the other.
>>>
>>> Preferences?
>>
>> I'm in favour of C which is to add this to tune-atom.inc:
>>
>> FULL_OPTIMIZATION_pn-gtk+ = "-fexpensive-optimizations
>> -frename-registers -O2 -ggdb -feliminate-unused-debug-types"
>>
>> :)
>>
>> Cheers,
>>
>> Richard
>
> I have an option D :)
>
> i.e. in gtk+_2.22.1.bb, add: FULL_OPTIMIZATION_core2=
> "-fexpensive-optimizations -frename-registers -O2 -ggdb
> -feliminate-unused-debug-types"
>
> because this issue is caused by gcc optimization for core2 arch, so
> in theory, all machine with core2 arch setting will be affected, no
> only limited to atom machine

There are apparently at least 4 ways to do this :-) RP has merged his 
change (version C) to bernard and I've tested it on atom-pc. I've 
removed the existing changes from the original gtk+ recipe and the 
various bbappends in meta-intel. I'm testing on n450 now. If that works 
I'll send the patch against gtk+ and commit those to bernard and master 
for meta-intel.

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