<?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>14470</bug_id>
          
          <creation_ts>2021-07-04 13:38:55 +0000</creation_ts>
          <short_desc>AB-INT PTEST ARM: parted &apos;t1100-busy-label.sh&apos;, &apos;t1101-busy-partition.sh&apos; ptest intermittent failure</short_desc>
          <delta_ts>2021-08-24 14:46:04 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>10</classification_id>
          <classification>QA/Testing</classification>
          <product>Package Testing (ptest)</product>
          <component>ptest</component>
          <version>3.4</version>
          <rep_platform>x86</rep_platform>
          <op_sys>Multiple</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard>AB-INT AB-INT-TMPFS</status_whiteboard>
          <keywords></keywords>
          <priority>Medium+</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>3.4 M3</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Alexandre Belloni">alexandre.belloni</reporter>
          <assigned_to name="Ross Burton">ross.burton</assigned_to>
          <cc>bruce.ashfield</cc>
    
    <cc>leonardo.sandoval</cc>
    
    <cc>randy.macleod</cc>
    
    <cc>rcalhoun-bugzilla</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>90845</commentid>
    <comment_count>0</comment_count>
    <who name="Alexandre Belloni">alexandre.belloni</who>
    <bug_when>2021-07-04 13:38:55 +0000</bug_when>
    <thetext>&apos;parted&apos;: [&apos;t1100-busy-label.sh&apos;, &apos;t1101-busy-partition.sh&apos;]

https://autobuilder.yoctoproject.org/typhoon/#/builders/82/builds/1957/steps/12/logs/stdio

qemuarm64-ptest ubuntu2004-arm-1

this build had the glibc2.34 update.


https://autobuilder.yoctoproject.org/typhoon/#/builders/82/builds/1959/steps/12/logs/stdio

qemuarm64-ptest ubuntu1804-arm-1</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>90847</commentid>
    <comment_count>1</comment_count>
    <who name="Alexandre Belloni">alexandre.belloni</who>
    <bug_when>2021-07-04 13:46:34 +0000</bug_when>
    <thetext>https://autobuilder.yoctoproject.org/typhoon/#/builders/82/builds/1961/steps/12/logs/stdio

qemuarm64-ptest ubuntu2004-arm-1</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>90852</commentid>
    <comment_count>2</comment_count>
    <who name="Alexandre Belloni">alexandre.belloni</who>
    <bug_when>2021-07-04 13:59:44 +0000</bug_when>
    <thetext>https://autobuilder.yoctoproject.org/typhoon/#/builders/82/builds/1956/steps/12/logs/stdio

qemuarm64-ptest ubuntu2004-arm-1</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>90858</commentid>
    <comment_count>3</comment_count>
    <who name="Leonardo Sandoval">leonardo.sandoval</who>
    <bug_when>2021-07-05 18:46:20 +0000</bug_when>
    <thetext>https://autobuilder.yoctoproject.org/typhoon/#/builders/82/builds/1962/steps/12/logs/stdio

qemuarm64-ptest ubuntu2004-arm-1</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>90887</commentid>
    <comment_count>4</comment_count>
    <who name="Alexandre Belloni">alexandre.belloni</who>
    <bug_when>2021-07-08 15:11:01 +0000</bug_when>
    <thetext>https://autobuilder.yoctoproject.org/typhoon/#/builders/82/builds/1970/steps/12/logs/stdio

qemuarm64-ptest ubuntu1804-arm-1

https://autobuilder.yoctoproject.org/typhoon/#/builders/82/builds/1968/steps/12/logs/stdio

qemuarm64-ptest ubuntu2004-arm-1

https://autobuilder.yoctoproject.org/typhoon/#/builders/82/builds/1966/steps/12/logs/stdio

qemuarm64-ptest ubuntu1804-arm-1</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>90902</commentid>
    <comment_count>5</comment_count>
    <who name="Ross Burton">ross.burton</who>
    <bug_when>2021-07-09 11:16:04 +0000</bug_when>
    <thetext>Patch on the list to fix most issues in the tests, but those tests fail as we don&apos;t have vfat in the qemu kernels.

Bruce, can we get vfat in the qemu kernels please?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>90904</commentid>
    <comment_count>6</comment_count>
    <who name="Bruce Ashfield">bruce.ashfield</who>
    <bug_when>2021-07-09 14:23:45 +0000</bug_when>
    <thetext>As Ross mentioned on IRC, we have some distro/machine features that indicate vfat support.

I&apos;ve added a KERNEL_FEATURE to linux-yocto.inc that triggers off of the vfat MACHINE_FEATURE and will configure things for vfat support.

I&apos;ll include it in my next pull request.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>90930</commentid>
    <comment_count>7</comment_count>
    <who name="Alexandre Belloni">alexandre.belloni</who>
    <bug_when>2021-07-15 13:42:59 +0000</bug_when>
    <thetext>https://autobuilder.yoctoproject.org/typhoon/#/builders/82/builds/1974/steps/12/logs/stdio
https://autobuilder.yoctoproject.org/typhoon/#/builders/82/builds/1975/steps/12/logs/stdio
https://autobuilder.yoctoproject.org/typhoon/#/builders/82/builds/1976/steps/12/logs/stdio
https://autobuilder.yoctoproject.org/typhoon/#/builders/82/builds/1977/steps/12/logs/stdio
https://autobuilder.yoctoproject.org/typhoon/#/builders/82/builds/1980/steps/12/logs/stdio
https://autobuilder.yoctoproject.org/typhoon/#/builders/82/builds/1981/steps/12/logs/stdio
https://autobuilder.yoctoproject.org/typhoon/#/builders/82/builds/1982/steps/12/logs/stdio
https://autobuilder.yoctoproject.org/typhoon/#/builders/82/builds/1985/steps/12/logs/stdio</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>90931</commentid>
    <comment_count>8</comment_count>
    <who name="Alexandre Belloni">alexandre.belloni</who>
    <bug_when>2021-07-15 13:48:14 +0000</bug_when>
    <thetext>https://autobuilder.yoctoproject.org/typhoon/#/builders/82/builds/1986/steps/12/logs/stdio
https://autobuilder.yoctoproject.org/typhoon/#/builders/82/builds/1987/steps/12/logs/stdio
https://autobuilder.yoctoproject.org/typhoon/#/builders/82/builds/1988/steps/12/logs/stdio
https://autobuilder.yoctoproject.org/typhoon/#/builders/82/builds/1992/steps/12/logs/stdio</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>90952</commentid>
    <comment_count>9</comment_count>
    <who name="Ross Burton">ross.burton</who>
    <bug_when>2021-07-16 10:09:13 +0000</bug_when>
    <thetext>My fault: I failed to specify ordering.

Those tests will fail until Bruce&apos;s &quot;linux-yocto: add vfat KERNEL_FEATURE when MACHINE_FEATURES include vfat&apos; merges.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>90953</commentid>
    <comment_count>10</comment_count>
    <who name="Ross Burton">ross.burton</who>
    <bug_when>2021-07-16 12:10:51 +0000</bug_when>
    <thetext>Patches on the list to skip those if vfat isn&apos;t present.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>90962</commentid>
    <comment_count>11</comment_count>
    <who name="Ross Burton">ross.burton</who>
    <bug_when>2021-07-20 08:59:56 +0000</bug_when>
    <thetext>Closing: the tests now skip if vfat isn&apos;t enabled, and vfat is enabled in the qemu builds.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>91252</commentid>
    <comment_count>12</comment_count>
    <who name="Rob Calhoun">rcalhoun-bugzilla</who>
    <bug_when>2021-08-24 01:34:21 +0000</bug_when>
    <thetext>This commit:

https://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?id=2b44ac076c7068aa0316d15fff49f02491eccb69

broke our build with the following error:

ERROR: linux-stable-5.4+gitAUTOINC+82ffbc138a-r1 do_kernel_metadata: Feature &apos;cfg/fs/vfat.scc&apos; not found, this will cause configuration failures.
ERROR: linux-stable-5.4+gitAUTOINC+82ffbc138a-r1 do_kernel_metadata: Check the SRC_URI for meta-data repositories or directories that may be missing
ERROR: linux-stable-5.4+gitAUTOINC+82ffbc138a-r1 do_kernel_metadata: Set KERNEL_DANGLING_FEATURES_WARN_ONLY to ignore this issue

The KERNEL_FEATURES check seems overly tied to a specific implementation and can error out in situations where nothing is wrong. 

Our situation looks like this:

* We use https://github.com/gregkh/linux as kernel source
* We use our own kernel recipe.
* That recipe that applies a few platform-specific patches plus a defconfig
* The recipe includes recipes-kernel/linux/linux-yocto.inc 
* The defconfig includes CONFIG_VFAT_FS=y
* DISTRO_FEATURES includes vfat

Because DISTRO_FEATURES includes vfat, KERNEL_FEATURES includes vfat, so CONFIG_VFAT_FS needs to be set. But it is set in the defconfig, so a config fragment isn&apos;t needed. 

I worked around this by replacing linux-yocto.inc in our kernel recipe with a subset of directives from that file:

-require recipes-kernel/linux/linux-yocto.inc
+inherit kernel
+inherit kernel-yocto
+B = &quot;${WORKDIR}/linux-${PACKAGE_ARCH}-${LINUX_KERNEL_TYPE}-build&quot;
+
+do_install_append(){
+       if [ -n &quot;${KMETA}&quot; ]; then
+               rm -rf ${STAGING_KERNEL_DIR}/${KMETA}
+       fi
+}
+
+LIC_FILES_CHKSUM ?= &quot;file://COPYING;md5=d7810fab7487fb0aad327b76f1be7cd7&quot;
+
+DEPENDS += &quot;xz-native bc-native&quot;

I didn&apos;t want to use the KERNEL_DANGLING_FEATURES_WARN_ONLY flag because I prefer to build without warnings, but that is what this project did:  https://gerrit.openbmc-project.xyz/c/openbmc/openbmc/+/36159 

I think it would be better for kconfig checks to be done against the .config file that will be used in the build, rather than against the intermediate files that generate that .config.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>91253</commentid>
    <comment_count>13</comment_count>
    <who name="Rob Calhoun">rcalhoun-bugzilla</who>
    <bug_when>2021-08-24 01:42:53 +0000</bug_when>
    <thetext>Correction, this is the commit caused our problem:

https://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/meta/recipes-kernel/linux/linux-yocto.inc?h=hardknott&amp;id=06af9f81e973894482a9c5136ecc267bca0c2f30</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>91254</commentid>
    <comment_count>14</comment_count>
    <who name="Ross Burton">ross.burton</who>
    <bug_when>2021-08-24 09:13:52 +0000</bug_when>
    <thetext>If you&apos;re just using a defconfig and your own recipe, then you don&apos;t really need to include linux-yocto, as that&apos;s where the fragment handling comes in.

FWIW, you can trivially tell linux-yocto to use a defconfig and mainline kernel release, or alternatively there is a set of &apos;pure mainline kernel&apos; recipes in https://git.sr.ht/~pbarker/meta-linux-mainline/.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>91257</commentid>
    <comment_count>15</comment_count>
    <who name="Rob Calhoun">rcalhoun-bugzilla</who>
    <bug_when>2021-08-24 14:46:04 +0000</bug_when>
    <thetext>Thanks, Ross. That&apos;s pretty much what I did, and it works fine. (I don&apos;t see any need to re-open this ticket; I just wanted to post the error in case it helps someone else out.)

Thanks also for the link to Paul Barker&apos;s linux-stable recipe. My recipe is a modification of the linux-yocto recipe; Paul&apos;s is cleaner.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>