diff mbox series

[meta-oe] thin-provisioning-tools: fix compile failure on 32bit x86

Message ID 20260512083810.1201751-1-hongxu.jia@windriver.com
State Under Review
Headers show
Series [meta-oe] thin-provisioning-tools: fix compile failure on 32bit x86 | expand

Commit Message

Hongxu Jia May 12, 2026, 8:38 a.m. UTC
$ echo 'MACHINE = "qemux86"' >> conf/local.conf
$ bitbake thin-provisioning-tools
```
error[E0080]: index out of bounds: the length is 1 but the index is 4
 --> tmp/work/core2-32-wrs-linux/thin-provisioning-tools/1.3.1/build/target/release/build/devicemapper-sys-f88f57f28cd965d2/out/dm-bindings.rs:3:8718
  |
3 | ...usize] ; ["Alignment of dm_ioctl"] [:: std :: mem :: align_of :: < dm_ioctl > () - 4usize] ; ["Offset of field: dm_ioctl::version"...
  |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ evaluation of `_` failed here
```

Tweak filter condition on TUNE_FEATURES to add m32,mx32 for 32bit x86

Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com>
---
 .../thin-provisioning-tools/thin-provisioning-tools_1.3.1.bb    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Khem Raj May 12, 2026, 3:43 p.m. UTC | #1
On Tue, May 12, 2026 at 1:38 AM hongxu via lists.openembedded.org
<hongxu.jia=windriver.com@lists.openembedded.org> wrote:

> $ echo 'MACHINE = "qemux86"' >> conf/local.conf
> $ bitbake thin-provisioning-tools
> ```
> error[E0080]: index out of bounds: the length is 1 but the index is 4
>  -->
> tmp/work/core2-32-wrs-linux/thin-provisioning-tools/1.3.1/build/target/release/build/devicemapper-sys-f88f57f28cd965d2/out/dm-bindings.rs:3:8718
>   |
> 3 | ...usize] ; ["Alignment of dm_ioctl"] [:: std :: mem :: align_of :: <
> dm_ioctl > () - 4usize] ; ["Offset of field: dm_ioctl::version"...
>   |
>  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
> evaluation of `_` failed here
> ```
>
> Tweak filter condition on TUNE_FEATURES to add m32,mx32 for 32bit x86
>
> Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com>
> ---
>  .../thin-provisioning-tools/thin-provisioning-tools_1.3.1.bb    | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta-oe/recipes-support/thin-provisioning-tools/
> thin-provisioning-tools_1.3.1.bb
> b/meta-oe/recipes-support/thin-provisioning-tools/
> thin-provisioning-tools_1.3.1.bb
> index 4d8d2a04df..6a58d090e8 100644
> --- a/meta-oe/recipes-support/thin-provisioning-tools/
> thin-provisioning-tools_1.3.1.bb
> +++ b/meta-oe/recipes-support/thin-provisioning-tools/
> thin-provisioning-tools_1.3.1.bb
> @@ -10,7 +10,7 @@ SRC_URI = " \
>      git://
> github.com/device-mapper-utils/thin-provisioning-tools;branch=main;protocol=https;tag=v${PV}
> <http://github.com/device-mapper-utils/thin-provisioning-tools;branch=main;protocol=https;tag=v$%7BPV%7D>
> \
>      file://0001-Use-portable-atomics-crate.patch \
>      file://disable-cargo-metadata.patch \
> -    ${@bb.utils.contains('TUNE_FEATURES', '32', \
> +    ${@bb.utils.contains_any('TUNE_FEATURES', '32 m32 mx32', \
>
>  'file://dms-no-layout-check.patch;patchdir=${CARGO_VENDORING_DIRECTORY}/devicemapper-sys-0.3.3',
> \
>          '', d)} \
>

would something like
${@oe.utils.conditional('SITEINFO_BITS', '32',
'file://dms-no-layout-check.patch;patchdir=${CARGO_VENDORING_DIRECTORY}/devicemapper-sys-0.3.3',
'', d)}
 cover all 32bit cases ?

>      "
> --
> 2.34.1
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#126864):
> https://lists.openembedded.org/g/openembedded-devel/message/126864
> Mute This Topic: https://lists.openembedded.org/mt/119271402/1997914
> Group Owner: openembedded-devel+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub [
> raj.khem@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
>
Hongxu Jia May 13, 2026, 2:42 a.m. UTC | #2
On 5/12/26 23:43, Khem Raj wrote:
> **
> *CAUTION: This email comes from a non Wind River email account!*
> Do not click links or open attachments unless you recognize the sender 
> and know the content is safe.
>
>
> On Tue, May 12, 2026 at 1:38 AM hongxu via lists.openembedded.org 
> <https://urldefense.com/v3/__http://lists.openembedded.org__;!!AjveYdw8EvQ!ZNjmxGK0Ko-e44Pl88M3wIFp_TgwLte4BBS_Mg_MWqeWG1CaMtFrYf3zHpNGQL8u31s4OojLxs1ER4HbXFfP$> 
> <hongxu.jia=windriver.com@lists.openembedded.org> wrote:
>
>     $ echo 'MACHINE = "qemux86"' >> conf/local.conf
>     $ bitbake thin-provisioning-tools
>     ```
>     error[E0080]: index out of bounds: the length is 1 but the index is 4
>      -->
>     tmp/work/core2-32-wrs-linux/thin-provisioning-tools/1.3.1/build/target/release/build/devicemapper-sys-f88f57f28cd965d2/out/dm-bindings.rs
>     <https://urldefense.com/v3/__http://dm-bindings.rs__;!!AjveYdw8EvQ!ZNjmxGK0Ko-e44Pl88M3wIFp_TgwLte4BBS_Mg_MWqeWG1CaMtFrYf3zHpNGQL8u31s4OojLxs1ER_XZVo3H$>:3:8718
>       |
>     3 | ...usize] ; ["Alignment of dm_ioctl"] [:: std :: mem ::
>     align_of :: < dm_ioctl > () - 4usize] ; ["Offset of field:
>     dm_ioctl::version"...
>       |
>      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
>     evaluation of `_` failed here
>     ```
>
>     Tweak filter condition on TUNE_FEATURES to add m32,mx32 for 32bit x86
>
>     Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com>
>     ---
>      .../thin-provisioning-tools/thin-provisioning-tools_1.3.1.bb
>     <https://urldefense.com/v3/__http://thin-provisioning-tools_1.3.1.bb__;!!AjveYdw8EvQ!ZNjmxGK0Ko-e44Pl88M3wIFp_TgwLte4BBS_Mg_MWqeWG1CaMtFrYf3zHpNGQL8u31s4OojLxs1ER13ufRPS$>
>       | 2 +-
>      1 file changed, 1 insertion(+), 1 deletion(-)
>
>     diff --git
>     a/meta-oe/recipes-support/thin-provisioning-tools/thin-provisioning-tools_1.3.1.bb
>     <https://urldefense.com/v3/__http://thin-provisioning-tools_1.3.1.bb__;!!AjveYdw8EvQ!ZNjmxGK0Ko-e44Pl88M3wIFp_TgwLte4BBS_Mg_MWqeWG1CaMtFrYf3zHpNGQL8u31s4OojLxs1ER13ufRPS$>
>     b/meta-oe/recipes-support/thin-provisioning-tools/thin-provisioning-tools_1.3.1.bb
>     <https://urldefense.com/v3/__http://thin-provisioning-tools_1.3.1.bb__;!!AjveYdw8EvQ!ZNjmxGK0Ko-e44Pl88M3wIFp_TgwLte4BBS_Mg_MWqeWG1CaMtFrYf3zHpNGQL8u31s4OojLxs1ER13ufRPS$>
>     index 4d8d2a04df..6a58d090e8 100644
>     ---
>     a/meta-oe/recipes-support/thin-provisioning-tools/thin-provisioning-tools_1.3.1.bb
>     <https://urldefense.com/v3/__http://thin-provisioning-tools_1.3.1.bb__;!!AjveYdw8EvQ!ZNjmxGK0Ko-e44Pl88M3wIFp_TgwLte4BBS_Mg_MWqeWG1CaMtFrYf3zHpNGQL8u31s4OojLxs1ER13ufRPS$>
>     +++
>     b/meta-oe/recipes-support/thin-provisioning-tools/thin-provisioning-tools_1.3.1.bb
>     <https://urldefense.com/v3/__http://thin-provisioning-tools_1.3.1.bb__;!!AjveYdw8EvQ!ZNjmxGK0Ko-e44Pl88M3wIFp_TgwLte4BBS_Mg_MWqeWG1CaMtFrYf3zHpNGQL8u31s4OojLxs1ER13ufRPS$>
>     @@ -10,7 +10,7 @@ SRC_URI = " \
>        
>      git://github.com/device-mapper-utils/thin-provisioning-tools;branch=main;protocol=https;tag=v${PV}
>     <https://urldefense.com/v3/__http://github.com/device-mapper-utils/thin-provisioning-tools;branch=main;protocol=https;tag=v$*7BPV*7D__;JSU!!AjveYdw8EvQ!ZNjmxGK0Ko-e44Pl88M3wIFp_TgwLte4BBS_Mg_MWqeWG1CaMtFrYf3zHpNGQL8u31s4OojLxs1ER_9QYG3K$>
>     \
>     file://0001-Use-portable-atomics-crate.patch \
>     file://disable-cargo-metadata.patch \
>     -    ${@bb.utils.contains('TUNE_FEATURES', '32', \
>     +    ${@bb.utils.contains_any('TUNE_FEATURES', '32 m32 mx32', \
>      'file://dms-no-layout-check.patch;patchdir=${CARGO_VENDORING_DIRECTORY}/devicemapper-sys-0.3.3',
>     \
>              '', d)} \
>
>
> would something like
> ${@oe.utils.conditional('SITEINFO_BITS', '32', 
> 'file://dms-no-layout-check.patch;patchdir=${CARGO_VENDORING_DIRECTORY}/devicemapper-sys-0.3.3', 
> '', d)}
>  cover all 32bit cases ?

Good idea, v2 incoming

//Hongxu

>          "
>     -- 
>     2.34.1
>
>
>     -=-=-=-=-=-=-=-=-=-=-=-
>     Links: You receive all messages sent to this group.
>     View/Reply Online (#126864):
>     https://lists.openembedded.org/g/openembedded-devel/message/126864
>     <https://urldefense.com/v3/__https://lists.openembedded.org/g/openembedded-devel/message/126864__;!!AjveYdw8EvQ!ZNjmxGK0Ko-e44Pl88M3wIFp_TgwLte4BBS_Mg_MWqeWG1CaMtFrYf3zHpNGQL8u31s4OojLxs1ER1OnUMOy$>
>     Mute This Topic:
>     https://lists.openembedded.org/mt/119271402/1997914
>     <https://urldefense.com/v3/__https://lists.openembedded.org/mt/119271402/1997914__;!!AjveYdw8EvQ!ZNjmxGK0Ko-e44Pl88M3wIFp_TgwLte4BBS_Mg_MWqeWG1CaMtFrYf3zHpNGQL8u31s4OojLxs1ER6iBVuEP$>
>     Group Owner: openembedded-devel+owner@lists.openembedded.org
>     <mailto:openembedded-devel%2Bowner@lists.openembedded.org>
>     Unsubscribe:
>     https://lists.openembedded.org/g/openembedded-devel/unsub
>     <https://urldefense.com/v3/__https://lists.openembedded.org/g/openembedded-devel/unsub__;!!AjveYdw8EvQ!ZNjmxGK0Ko-e44Pl88M3wIFp_TgwLte4BBS_Mg_MWqeWG1CaMtFrYf3zHpNGQL8u31s4OojLxs1ER0JpsS6j$>
>     [raj.khem@gmail.com]
>     -=-=-=-=-=-=-=-=-=-=-=-
>
diff mbox series

Patch

diff --git a/meta-oe/recipes-support/thin-provisioning-tools/thin-provisioning-tools_1.3.1.bb b/meta-oe/recipes-support/thin-provisioning-tools/thin-provisioning-tools_1.3.1.bb
index 4d8d2a04df..6a58d090e8 100644
--- a/meta-oe/recipes-support/thin-provisioning-tools/thin-provisioning-tools_1.3.1.bb
+++ b/meta-oe/recipes-support/thin-provisioning-tools/thin-provisioning-tools_1.3.1.bb
@@ -10,7 +10,7 @@  SRC_URI = " \
     git://github.com/device-mapper-utils/thin-provisioning-tools;branch=main;protocol=https;tag=v${PV} \
     file://0001-Use-portable-atomics-crate.patch \
     file://disable-cargo-metadata.patch \
-    ${@bb.utils.contains('TUNE_FEATURES', '32', \
+    ${@bb.utils.contains_any('TUNE_FEATURES', '32 m32 mx32', \
         'file://dms-no-layout-check.patch;patchdir=${CARGO_VENDORING_DIRECTORY}/devicemapper-sys-0.3.3', \
         '', d)} \
     "