From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from list by lists.gnu.org with archive (Exim 4.71) id 1ZFP2m-0006Wa-EP for mharc-grub-devel@gnu.org; Wed, 15 Jul 2015 11:56:24 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:60846) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZFP2j-0006WA-My for grub-devel@gnu.org; Wed, 15 Jul 2015 11:56:22 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZFP2i-0005JQ-EP for grub-devel@gnu.org; Wed, 15 Jul 2015 11:56:21 -0400 Received: from mail-wg0-x234.google.com ([2a00:1450:400c:c00::234]:33652) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZFP2i-0005JH-6E for grub-devel@gnu.org; Wed, 15 Jul 2015 11:56:20 -0400 Received: by wgmn9 with SMTP id n9so37316091wgm.0 for ; Wed, 15 Jul 2015 08:56:19 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=message-id:date:from:user-agent:mime-version:to:cc:subject :references:in-reply-to:content-type; bh=CV9gGCj6LMhLwKpysTZh5KhzWR8H8aqUhLDQUcaWYBU=; b=lw+mSqN+B5FMmltCGxI/egoSnzyviE7DK/UdFZRNyghbpPw1fwwo0JgB2IqmiGd6Dd KgE773Sw5lYFPu89taNj6p/YpfxR3xsPk913ggZfeqQsogRGrpTf1p2iPA7FoRnY6cra eHCyoJ2+LVz9egx8UtvFuWnA7N4dryKAa1DV9WhhjABhf116W9qiMvsqOQ+ZV5D9u5Dy NpImISQhaPPHjM9XNQpW+PSsz+SeYn8nY4VaLG+f5Fn119ur+bvoR/LwDi8toaWYO7rp 4EzQflq1y5ay1SsLNn6ZJQzPYiXqxLzSyIBp98gb8j8poH9+NAzCUAXj+n1w89olw9L6 4TIA== X-Received: by 10.195.11.3 with SMTP id ee3mr9786127wjd.89.1436975779394; Wed, 15 Jul 2015 08:56:19 -0700 (PDT) Received: from ?IPv6:2620:0:105f:fd00:863a:4bff:fe50:abc4? ([2620:0:105f:fd00:863a:4bff:fe50:abc4]) by smtp.gmail.com with ESMTPSA id r19sm398749wib.7.2015.07.15.08.56.18 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 15 Jul 2015 08:56:18 -0700 (PDT) Message-ID: <55A682A2.8000500@gmail.com> Date: Wed, 15 Jul 2015 17:56:18 +0200 From: =?UTF-8?B?VmxhZGltaXIgJ8+GLWNvZGVyL3BoY29kZXInIFNlcmJpbmVua28=?= User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Icedove/31.7.0 MIME-Version: 1.0 To: fu.wei@linaro.org, grub-devel@gnu.org, arvidjaar@gmail.com Subject: Re: [PATCH v2 0/3] arm64: Add multiboot support (via fdt) for Xen boot References: <=fu.wei@linaro.org> <1436777640-31871-1-git-send-email-fu.wei@linaro.org> In-Reply-To: <1436777640-31871-1-git-send-email-fu.wei@linaro.org> Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="S5LBfWDMpTbP42GgwQuWDE6VqlqU45VFD" X-detected-operating-system: by eggs.gnu.org: Error: Malformed IPv6 address (bad octet value). X-Received-From: 2a00:1450:400c:c00::234 Cc: xen-devel@lists.xensource.com, Ian.Campbell@citrix.com, stefano.stabellini@eu.citrix.com, jcm@redhat.com, leif.lindholm@linaro.org, ryan.harkin@linaro.org, linaro-uefi@lists.linaro.org X-BeenThere: grub-devel@gnu.org X-Mailman-Version: 2.1.14 Precedence: list Reply-To: The development of GNU GRUB List-Id: The development of GNU GRUB List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 15 Jul 2015 15:56:23 -0000 This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --S5LBfWDMpTbP42GgwQuWDE6VqlqU45VFD Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable On 13.07.2015 10:53, fu.wei@linaro.org wrote: > From: Fu Wei >=20 > - This adds support for the Xen boot on ARM specification for arm64. >=20 > - The implementation for Xen is following : > http://wiki.xen.org/wiki/Xen_ARM_with_Virtualization_Extensions/Mul= tiboot > and xen/docs/misc/arm/device-tree/booting.txt in Xen source code. >=20 > - The multiboot/module commands have existed, so we use=20 > xen_hypervisor/xen_module instead. >=20 > - This Xen boot support is built into linux module for aarch64, > and can not be used alone. >=20 > - Adding this functionality to the existing "linux" module is for > reusing the existing code of devicetree. >=20 This is a misguided decision. Modules can depend on other modules. Ideally shared functionality should be in a separate module but having xen depend on linux is an OK stopgap solution. Putting everything in one module is bad. > - Add the support of xen_hypervisor/xen_module commands in util/grub.= d/20_linux_xen.in >=20 > - Add the introduction of xen_hypervisor/xen_module commands in docs/= grub.texi >=20 > - The example of this support is > https://wiki.linaro.org/LEG/Engineering/Grub2/Xen_booting_on_Founda= tion_FVP_model_by_GRUB >=20 > Changelog: > v2: remove the patches which have been accepted. > according to Vladimir's suggestion, change the command manes > and relevant code: > multiboot-->xen_hypervisor > module-->xen_module > improve the option parsing support for xen_hypervisor/xen_module co= mmands. > add a patch for adding xen_hypervisor/xen_module support > in util/grub.d/20_linux_xen.in. > update docs/grub.texi patch for the new command names. >=20 > v1: The first version upstream patchset to grub-devel mailing list >=20 >=20 > Fu Wei (3): > arm64: Add Xen boot support file > * util/grub.d/20_linux_xen.in: Add support of the XEN boot on aarch64= > arm64: Add the introduction of xen_hypervisor/xen_module command in > docs/grub.texi >=20 > docs/grub.texi | 27 ++ > grub-core/Makefile.core.def | 1 + > grub-core/loader/arm64/linux.c | 6 + > grub-core/loader/arm64/xen_boot.c | 615 ++++++++++++++++++++++++++++++= ++++++++ > include/grub/arm64/xen_boot.h | 115 +++++++ > util/grub.d/20_linux_xen.in | 14 +- > 6 files changed, 775 insertions(+), 3 deletions(-) > create mode 100644 grub-core/loader/arm64/xen_boot.c > create mode 100644 include/grub/arm64/xen_boot.h >=20 --S5LBfWDMpTbP42GgwQuWDE6VqlqU45VFD Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iF4EAREKAAYFAlWmgqIACgkQmBXlbbo5nOufEwD7BBfs05LWxgwIQw/LiCM6Dl6w MVkNZek5qm9GLYQHJyIA/00v4218Iv3pBV4RwWQFx+u+7fvPkLvub/ha8N8dbvrS =w+gk -----END PGP SIGNATURE----- --S5LBfWDMpTbP42GgwQuWDE6VqlqU45VFD--