<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugzilla.yoctoproject.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugzilla.yoctoproject.org/"
          
          maintainer="it-coreprojects-helpdesk@linuxfoundation.org"
>

    <bug>
          <bug_id>9213</bug_id>
          
          <creation_ts>2016-03-04 11:39:09 +0000</creation_ts>
          <short_desc>Add an autobuilder run using ARM/Thumb.</short_desc>
          <delta_ts>2022-10-27 18:18:56 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>5</classification_id>
          <classification>Infrastructure</classification>
          <product>AutoBuilder</product>
          <component>autobuilder</component>
          <version>unspecified</version>
          <rep_platform>x86</rep_platform>
          <op_sys>Multiple</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>OBSOLETE</resolution>
          
          <see_also>https://bugzilla.yoctoproject.org/show_bug.cgi?id=7717</see_also>
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>Medium</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>Future</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Martin Jansa">Martin.Jansa</reporter>
          <assigned_to name="Ross Burton">ross.burton</assigned_to>
          <cc>elizabeth.flanagan</cc>
    
    <cc>infras.ab.watcher</cc>
    
    <cc>Infras.watcher</cc>
    
    <cc>jon.mason</cc>
    
    <cc>randy.macleod</cc>
    
    <cc>richard.purdie</cc>
    
    <cc>ross.burton</cc>
          
          
          <cf_os>---</cf_os>
          <cf_regression_type>---</cf_regression_type>
          
          <cf_docchange>No (bug/feature does not impact docs)</cf_docchange>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>59724</commentid>
    <comment_count>0</comment_count>
    <who name="Martin Jansa">Martin.Jansa</who>
    <bug_when>2016-03-04 11:39:09 +0000</bug_when>
    <thetext>See #7717</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>60084</commentid>
    <comment_count>1</comment_count>
    <who name="Richard Purdie">richard.purdie</who>
    <bug_when>2016-03-17 14:28:22 +0000</bug_when>
    <thetext>Just putting the other bug number doesn&apos;t really help here.

The issue is that qemuarm doesn&apos;t enable thumb and thumb in armv5 isn&apos;t well tested. The question is whether the qemuarm builds should select a thumb tune,  what if any complications that would cause and what if any downsides that would have.

Adding both a thumb and a non-thumb version of qemuarm isn&apos;t going to be that helpful since we&apos;d require more build time and more build resources which are already stretched and in limited supply.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>60159</commentid>
    <comment_count>2</comment_count>
    <who name="Martin Jansa">Martin.Jansa</who>
    <bug_when>2016-03-18 15:39:54 +0000</bug_when>
    <thetext>&gt; The issue is that qemuarm doesn&apos;t enable thumb and thumb in armv5 isn&apos;t well tested.

It was always enabled in my world builds.

Enabling it in one of the yocto builds or even by default in metadata should definitely improve it.

It should also prevent people from removing ARM_INSTRUCTION_SET overrides when they feel it&apos;s something old and not needed anymore (see examples in #7717).

Changing the default DISTRO config to enable thumb when available should help people which often don&apos;t understand that seeing &quot;thumb&quot; in TUNE_FEATURES isn&apos;t enough to get the toolchain to actually use it - but this discussion probably belongs back to closed #7717.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>86149</commentid>
    <comment_count>3</comment_count>
    <who name="Ross Burton">ross.burton</who>
    <bug_when>2020-01-29 16:39:47 +0000</bug_when>
    <thetext>Jon: this bug is about adding a new build to the autobuilder to exercise ARM builds with Thumb enabled.  Seems more sensible for you to own this than me!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>88489</commentid>
    <comment_count>4</comment_count>
    <who name="Ross Burton">ross.burton</who>
    <bug_when>2020-10-26 15:07:22 +0000</bug_when>
    <thetext>Oh the irony in me@Intel passing this to Jon when me@Arm is likely to work on this.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>94283</commentid>
    <comment_count>5</comment_count>
    <who name="Randy MacLeod">randy.macleod</who>
    <bug_when>2022-10-27 15:30:16 +0000</bug_when>
    <thetext>Ross plans to add a minimal build.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>94286</commentid>
    <comment_count>6</comment_count>
    <who name="Ross Burton">ross.burton</who>
    <bug_when>2022-10-27 15:40:06 +0000</bug_when>
    <thetext>Turns out that since 3e7d26e31a724009e69788460737e07163a14051 (2018), qemuarm uses thumb:

$ MACHINE=qemuarm bitbake-getvar ARM_INSTRUCTION_SET
ARM_INSTRUCTION_SET=&quot;thumb&quot;

$ MACHINE=qemuarm bitbake-getvar DEFAULTTUNE
DEFAULTTUNE=&quot;cortexa15thf-neon&quot;

$ MACHINE=qemuarm bitbake-getvar TUNE_FEATURES
TUNE_FEATURES=&quot;arm vfp cortexa15 neon thumb callconvention-hard&quot;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>94288</commentid>
    <comment_count>7</comment_count>
    <who name="Randy MacLeod">randy.macleod</who>
    <bug_when>2022-10-27 16:00:01 +0000</bug_when>
    <thetext>That was a poky commit, the oe-core version is:
   https://git.openembedded.org/openembedded-core/commit/?id=c88304a78e528596ca481cabe273749c286c352a

Thanks Ross.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>94289</commentid>
    <comment_count>8</comment_count>
    <who name="Martin Jansa">Martin.Jansa</who>
    <bug_when>2022-10-27 16:18:58 +0000</bug_when>
    <thetext>This was originally for enabling thumb1 on qemuarm when it still was armv5 based (because that failed to build much more often than with thumb2).

The referenced commit enables it (thumb2) for armv7a, but nowadays I don&apos;t know if anyone still cares about armv5 (I don&apos;t run world builds anymore) and I don&apos;t see any qemuarmv5 jobs on https://autobuilder.yoctoproject.org/typhoon/#/schedulers as well.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>94290</commentid>
    <comment_count>9</comment_count>
    <who name="Ross Burton">ross.burton</who>
    <bug_when>2022-10-27 16:22:20 +0000</bug_when>
    <thetext>The meta-arm CI has qemuarmv5, and that uses the thumb feature by default too. Does that cover your original needs?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>94291</commentid>
    <comment_count>10</comment_count>
    <who name="Ross Burton">ross.burton</who>
    <bug_when>2022-10-27 16:23:21 +0000</bug_when>
    <thetext>The proper CI is internal for now, but Jon has a mirror which is public: https://gitlab.com/jonmason00/meta-arm/-/jobs/3234500301</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>94292</commentid>
    <comment_count>11</comment_count>
    <who name="Martin Jansa">Martin.Jansa</who>
    <bug_when>2022-10-27 16:33:38 +0000</bug_when>
    <thetext>&gt; The meta-arm CI has qemuarmv5, and that uses the thumb feature by default too.

feature in TUNE_FEATURES or does it set ARM_INSTRUCTION_SET in the job config as well? I don&apos;t see ARM_INSTRUCTION_SET in https://gitlab.com/jonmason00/meta-arm/-/blob/master/ci/qemuarmv5.yml</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>94293</commentid>
    <comment_count>12</comment_count>
    <who name="Ross Burton">ross.burton</who>
    <bug_when>2022-10-27 16:53:55 +0000</bug_when>
    <thetext>$ MACHINE=qemuarmv5 bitbake-getvar TUNE_FEATURES
TUNE_FEATURES=&quot;arm armv5 thumb dsp&quot;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>94295</commentid>
    <comment_count>13</comment_count>
    <who name="Martin Jansa">Martin.Jansa</who>
    <bug_when>2022-10-27 17:27:09 +0000</bug_when>
    <thetext>&gt; TUNE_FEATURES=&quot;arm armv5 thumb dsp&quot;

That only means that the qemuarmv5 is compatible with thumb, but it&apos;s not used without setting the default ARM_INSTRUCTION_SET to thumb as well.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>94296</commentid>
    <comment_count>14</comment_count>
    <who name="Ross Burton">ross.burton</who>
    <bug_when>2022-10-27 18:18:56 +0000</bug_when>
    <thetext>Of course, sorry.  Indeed, that isn&apos;t set.  We&apos;ll look at adding it to meta-arm :)</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>