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]
> -=-=-=-=-=-=-=-=-=-=-=-
>
>
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)} \
     "