From: "Yann E. MORIN" <yann.morin.1998@free.fr>
To: "José Luis Salvador Rufo" <salvador.joseluis@gmail.com>
Cc: Alexander Mukhin <alexander.i.mukhin@gmail.com>,
Michael Zimmermann <sigmaepsilon92@gmail.com>,
Philippe.Serbruyns@ugent.be, buildroot@buildroot.org
Subject: Re: [Buildroot] [PATCH] package/zfs: fix zfs autotools cross-compilation
Date: Sun, 26 Nov 2023 17:21:51 +0100 [thread overview]
Message-ID: <20231126162151.GW3177259@scaer> (raw)
In-Reply-To: <20231112221115.2289525-3-salvador.joseluis@gmail.com>
José, All,
On 2023-11-12 23:11 +0100, José Luis Salvador Rufo spake thusly:
> This commit addresses a long-standing bug encountered during ZFS
> compilation in cross-platform environments. The issue arises because ZFS
> autoconf triggers a `make modules` to detect if the kernel can compile
> modules [1]. The problem occurs when autoconf uses the host environment
> instead of the cross-platform environment.
>
> To fix this, we export necessary environment variables to ensure that ZFS
> autoconf utilizes the cross-platform environment correctly.
>
> This patch resolves ZFS cross-platform compilations:
> - http://autobuild.buildroot.net/results/ebeab256101bcba38c35fd55075c414e62f92caa/
> - http://autobuild.buildroot.net/results/03b9f12a106bf100eec695a92b83bf09b22c68b0/
> - http://autobuild.buildroot.net/results/c2da90337463607c2fadfeac7ad72e5c3899a61f/
> - http://autobuild.buildroot.net/results/465a249f92d2f5db7ac4b61b4111e6cbaaa15688/
> - http://autobuild.buildroot.net/results/7e2d3277e26fa5b0c8073a0e8b9e82f47ade9697/
> - http://autobuild.buildroot.net/results/a8fb87336b09fef8787a7889dfcccf14fe1215b9/
> - https://gitlab.com/kubu93/buildroot/-/jobs/1522848483
>
> And fix a few emails:
> - alpine.DEB.2.22.394.2108181630280.2028262@ridzo [build zfs into buildroot for raspberry pi 4]
> - https://lists.buildroot.org/pipermail/buildroot/2021-August/621696.html
> - https://lists.buildroot.org/pipermail/buildroot/2021-August/621345.html
> - https://lists.buildroot.org/pipermail/buildroot/2022-July/646379.html
> - https://lists.buildroot.org/pipermail/buildroot/2023-June/668467.html
>
> [1] This is the full callback, you can just check the last link:
> - https://github.com/openzfs/zfs/blob/zfs-2.1.12/config/kernel-declare-event-class.m4#L7C11-L7C11
> - https://github.com/openzfs/zfs/blob/zfs-2.1.12/config/kernel.m4#L883
> - https://github.com/openzfs/zfs/blob/zfs-2.1.12/config/kernel.m4#L868
> - https://github.com/openzfs/zfs/blob/zfs-2.1.12/config/kernel.m4#L668
>
> Signed-off-by: José Luis Salvador Rufo <salvador.joseluis@gmail.com>
Applied to master, thanks.
Regards,
Yann E. MORIN.
> ---
> package/zfs/zfs.mk | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/package/zfs/zfs.mk b/package/zfs/zfs.mk
> index f0461ea9ec..ab530cbcf0 100644
> --- a/package/zfs/zfs.mk
> +++ b/package/zfs/zfs.mk
> @@ -59,6 +59,14 @@ else
> ZFS_CONF_OPTS += --disable-pam
> endif
>
> +# Sets the environment for the `make` that will be run ZFS autotools checks.
> +ZFS_CONF_ENV += \
> + ARCH=$(KERNEL_ARCH) \
> + CROSS_COMPILE="$(TARGET_CROSS)"
> +ZFS_MAKE_ENV += \
> + ARCH=$(KERNEL_ARCH) \
> + CROSS_COMPILE="$(TARGET_CROSS)"
> +
> # ZFS userland tools are unfunctional without the Linux kernel modules.
> ZFS_MODULE_SUBDIRS = \
> module/avl \
> --
> 2.42.1
>
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
next prev parent reply other threads:[~2023-11-26 16:22 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-06-08 17:55 [Buildroot] [PATCH] package/zfs: fix autotools crosscompilations José Luis Salvador Rufo
2023-06-09 19:05 ` Alexander Mukhin
2023-06-12 17:57 ` Arnout Vandecappelle via buildroot
2023-07-26 21:30 ` Thomas Petazzoni via buildroot
2023-11-12 22:11 ` [Buildroot] [PATCH] package/zfs: fix zfs autotools cross-compilation José Luis Salvador Rufo
2023-11-26 16:21 ` Yann E. MORIN [this message]
2023-11-30 8:17 ` Peter Korsgaard
2023-11-12 22:17 ` [Buildroot] [PATCH] package/zfs: fix autotools crosscompilations José Luis Salvador Rufo
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20231126162151.GW3177259@scaer \
--to=yann.morin.1998@free.fr \
--cc=Philippe.Serbruyns@ugent.be \
--cc=alexander.i.mukhin@gmail.com \
--cc=buildroot@buildroot.org \
--cc=salvador.joseluis@gmail.com \
--cc=sigmaepsilon92@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox