From: Jungseok Lee <jungseoklee85@gmail.com>
To: Julien Grall <julien.grall@arm.com>
Cc: sstabellini@kernel.org, wei.liu2@citrix.com,
ian.jackson@eu.citrix.com,
Andrii Anisov <andrii.anisov@gmail.com>,
jbeulich@suse.com, xen-devel@lists.xen.org
Subject: Re: [PATCH for-4.8] tools/libacpi: Fix compilation when cross building the tools
Date: Mon, 28 Nov 2016 21:53:08 +0900 [thread overview]
Message-ID: <AEB02566-951C-4298-9FE5-57403FC24686@gmail.com> (raw)
In-Reply-To: <1480100781-2915-1-git-send-email-julien.grall@arm.com>
On Nov 26, 2016, at 4:06 AM, Julien Grall wrote:
Hi Julien and Xen folks,
> The tools (such as mk_dsdt) can be cross-built when it may not be
> desirable to build them on the target.
>
> The commit c4ac1077 "libxl/arm: Generate static ACPI DSDT table"
> introduced support of ARM64 in mk_dsdt but also break cross-building
> tools because the ACPI tables are not correct.
>
> While mk_dsdt should generate ACPI table for the target architecture, it
> currently generates the one for the host. This is because the source
> code contains reference to the host architecture (__aarch64__,
> __x86_64__, __i386__) when it should be the target architecture.
>
> Replace all __aarch64__, __x86_64__, __i386__ by the corresponding
> CONFIG_*.
>
> Also expose the CONFIG_* to the source code as the currently only
> exposed to the Makefile.
>
> Reported-by: Andrii Anisov <andrii.anisov@gmail.com>
> Suggested-by: Wei Liu <wei.liu2@citrix.com>
> Signed-off-by: Julien Grall <julien.grall@arm.com>
>
> ---
> This was reported on the ML recently (see [1]) and affects only Xen
> 4.8. Without this patch, cross-building the tools will not work.
>
> I think this patch is quite important for embedded users where they
> tend to cross-build the rootfs (for instance using yocto).
I've experienced the same build breakage when using Yocto. This patch
resolves the problem in my case. Thanks a lot!
Best Regards
Jungseok Lee
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
prev parent reply other threads:[~2016-11-28 12:53 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-11-25 19:06 [PATCH for-4.8] tools/libacpi: Fix compilation when cross building the tools Julien Grall
2016-11-25 19:12 ` Andrew Cooper
2016-11-28 13:00 ` Julien Grall
2016-11-28 13:01 ` Andrew Cooper
2016-11-28 8:10 ` Jan Beulich
2016-11-28 12:53 ` Julien Grall
2016-11-28 12:53 ` Jungseok Lee [this message]
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=AEB02566-951C-4298-9FE5-57403FC24686@gmail.com \
--to=jungseoklee85@gmail.com \
--cc=andrii.anisov@gmail.com \
--cc=ian.jackson@eu.citrix.com \
--cc=jbeulich@suse.com \
--cc=julien.grall@arm.com \
--cc=sstabellini@kernel.org \
--cc=wei.liu2@citrix.com \
--cc=xen-devel@lists.xen.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.