[oe] [meta-qt5] qtscript, qtwebking: disable thumb issues on armv7

[Martin Jansa]

On Wed, Dec 23, 2015 at 10:57:18AM +0100, Andreas Müller wrote:
> On Wed, Dec 23, 2015 at 10:42 AM, Martin Jansa <martin.jansa at gmail.com> wrote:
> > On Wed, Dec 23, 2015 at 10:13:45AM +0100, Andreas Müller wrote:
> >> On Wed, Dec 23, 2015 at 9:19 AM, Andreas Müller
> >> <schnitzeltony at googlemail.com> wrote:
> >> > Hi
> >> >
> >> > The story occured on jethro but I think it is the same for master:
> >> On master fix for qtwebkit won't be necessary - qtwebkit will go away [1]
> >>
> >> [1] http://blog.qt.io/blog/2015/12/18/qt-5-6-beta-released/
> >
> > True, but there are some attempts to keep 5.5* qtwebkit compatible with
> > latest 5.6:
> > http://lists.qt-project.org/pipermail/releasing/2015-December/002026.html
> >
> > so I wouldn't hope for qtwebkit recipe removal soon.
> Yes - also in the blog I linked it was mentioned in the lower area but
> my mail was already out when I read that.
> >
> >>
> >> Andreas
> >> >
> >> > We have a simple application containing a QWebView which segfaulted
> >> > during loading a page on a CortexA9 (armv7 thumb2). After enabling
> >> > debug information for qtwebkit and remote debug session the point of
> >> > segfault was detected in
> >> >
> >> > qtwebkit: Source/JavaScriptCore/llint/LowLevelInterpreter.asm:1082
> >> >    callSlowPath(_llint_slow_path_new_func_exp)
> >> >
> >> > Further digging around lead to similar source in:
> >> >
> >> > * qtwebkit: Source/JavaScriptCore/assembler/ARMv7Assembler.h
> >> > * qtscript: src/3rdparty/javascriptcore/JavaScriptCore/assembler/ARMv7Assembler.h
> >> >
> >> > In these files a class ARMThumbImmediate is declared and used in many
> >> > places. This lead me to the assumption that compiling for armv7
> >> > expects thumb support.
> >> >
> >> > So I removed the
> >> >
> >> > ARM_INSTRUCTION_SET = "arm"
> >
> > You build it in distro with ARM_INSTRUCTION_SET set to "thumb", right?
> Correct
> 
> > Because the default is still "arm" so this issue you're describing is
> > still valid in default DISTRO-less builds.
> Ooh yes - so the right way for now (as long as other arm variants
> don't fail) would be something like
> 
> ARM_INSTRUCTION_SET = "arm"
> ARM_INSTRUCTION_SET_armv7 = "thumb"
> 
> If agreed I would send out patches for master and jethro with this.

I'm not sure if I would go that far to set it to "thumb" here, just add
comment (even better with link to upstream bug).

It would be IMHO first case where we force thumb from the recipe.

Also note that there isn't armv7 override only armv7a, armv7m, armv7r,
armv7ve.

> 
> Andreas
> -- 
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel at lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa at gmail.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://lists.openembedded.org/pipermail/openembedded-devel/attachments/20151223/ad6a6c90/attachment-0002.sig>