Hi Michal,
> -----Original Message-----
> From: Xen-devel <xen-devel-boun...@lists.xenproject.org> On Behalf Of
> Michal Orzel
> Sent: Tuesday, September 27, 2022 5:47 PM
> To: xen-devel@lists.xenproject.org
> Cc: Michal Orzel <michal.or...@amd.com>; Doug Goldstein
> <car...@cardoe.com>; Stefano Stabellini <sstabell...@kernel.org>
> Subject: [PATCH v3 05/10] automation: Add Arm containers to containerize
> script
>
> Script automation/scripts/containerize makes it easy to build Xen within
> predefined containers from gitlab container registry. This script is
> currently missing the helpers to select Arm containers, so populate the
> necessary entries.
>
> Signed-off-by: Michal Orzel <michal.or...@amd.com>
> Acked-by: Stefano Stabellini <sstabell...@kernel.org>
> ---
> Changes in v3:
> - none
> Changes in v2:
> - modify commit msg to reflect that we are missing helpers but in reality
> it could be possible to use Arm containers by specifying the full path
> to gitlab container registry. However, such usage is annoying.
> ---
> automation/scripts/containerize | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/automation/scripts/containerize
> b/automation/scripts/containerize
> index 9d4beca4fa4b..0f4645c4cccb 100755
> --- a/automation/scripts/containerize
> +++ b/automation/scripts/containerize
> @@ -25,6 +25,7 @@ die() {
> BASE="registry.gitlab.com/xen-project/xen"
> case "_${CONTAINER}" in
> _alpine) CONTAINER="${BASE}/alpine:3.12" ;;
> + _alpine-arm64v8) CONTAINER="${BASE}/alpine:3.12-arm64v8" ;;
> _archlinux|_arch) CONTAINER="${BASE}/archlinux:current" ;;
> _riscv64) CONTAINER="${BASE}/archlinux:riscv64" ;;
> _centos7) CONTAINER="${BASE}/centos:7" ;;
> @@ -35,6 +36,8 @@ case "_${CONTAINER}" in
> _stretch|_) CONTAINER="${BASE}/debian:stretch" ;;
> _buster-gcc-ibt) CONTAINER="${BASE}/debian:buster-gcc-ibt" ;;
> _unstable|_) CONTAINER="${BASE}/debian:unstable" ;;
> + _unstable-arm32-gcc) CONTAINER="${BASE}/debian:unstable-arm32-
> gcc" ;;
> + _unstable-arm64v8) CONTAINER="${BASE}/debian:unstable-arm64v8" ;;
> _trusty) CONTAINER="${BASE}/ubuntu:trusty" ;;
> _xenial) CONTAINER="${BASE}/ubuntu:xenial" ;;
> _opensuse-leap|_leap) CONTAINER="${BASE}/suse:opensuse-leap" ;;
> --
> 2.25.1
[Jiamei Xie]
I wonder if an default container for arm can be added. For example, if
"CONTAINER=arm64 automation/scripts/containerize bash",
set the default CONTAINER as
"registry.gitlab.com/xen-project/xen/alpine:3.12-arm64v8"
Best wishes
Jiamei Xie
