From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from list by lists.gnu.org with archive (Exim 4.71) id 1b2God-0006yn-0f for mharc-grub-devel@gnu.org; Mon, 16 May 2016 07:36:03 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:35428) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1b2Goa-0006wg-67 for grub-devel@gnu.org; Mon, 16 May 2016 07:36:01 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1b2GoV-0007Xg-VN for grub-devel@gnu.org; Mon, 16 May 2016 07:36:00 -0400 Received: from foss.arm.com ([217.140.101.70]:44907) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1b2GoV-0007X0-QR for grub-devel@gnu.org; Mon, 16 May 2016 07:35:55 -0400 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.72.51.249]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 0CEA93A; Mon, 16 May 2016 04:28:37 -0700 (PDT) Received: from [10.1.215.28] (e108454-lin.cambridge.arm.com [10.1.215.28]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 031A43F218; Mon, 16 May 2016 04:28:18 -0700 (PDT) Subject: Re: [Xen-devel] [PATCH v4 4/4] arm64: update the introduction of xen boot commands in docs/grub.texi To: fu.wei@linaro.org, grub-devel@gnu.org, arvidjaar@gmail.com, phcoder@gmail.com References: <1462889006-23671-1-git-send-email-fu.wei@linaro.org> <1462889006-23671-5-git-send-email-fu.wei@linaro.org> Cc: xen-devel@lists.xensource.com, Ian.Campbell@citrix.com, jcm@redhat.com, julien.grall@linaro.org, leif.lindholm@linaro.org, linaro-uefi@lists.linaro.org, Stefano Stabellini From: Julien Grall Message-ID: <5739AED1.1020004@arm.com> Date: Mon, 16 May 2016 12:28:17 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.7.2 MIME-Version: 1.0 In-Reply-To: <1462889006-23671-5-git-send-email-fu.wei@linaro.org> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 217.140.101.70 X-BeenThere: grub-devel@gnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: The development of GNU GRUB List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 16 May 2016 11:36:01 -0000 Hi Fu Wei, (CC Stefano) On 10/05/16 15:03, fu.wei@linaro.org wrote: > From: Fu Wei > > delete: xen_linux, xen_initrd, xen_xsm > add: xen_module > > This update bases on > commit 0edd750e50698854068358ea53528100a9192902 > Author: Vladimir Serbinenko > Date: Fri Jan 22 10:18:47 2016 +0100 > > xen_boot: Remove obsolete module type distinctions. > > Also bases on the module loading mechanism of Xen code: > 488c2a8 docs/arm64: clarify the documention for loading XSM support > 67831c4 docs/arm64: update the documentation for loading XSM support > ca32012 xen/arm64: check XSM Magic from the second unknown module. > > Signed-off-by: Fu Wei > --- > docs/grub.texi | 33 ++++++++++----------------------- > 1 file changed, 10 insertions(+), 23 deletions(-) > > diff --git a/docs/grub.texi b/docs/grub.texi > index 82f6fa4..df6018a 100644 > --- a/docs/grub.texi > +++ b/docs/grub.texi > @@ -3861,9 +3861,7 @@ you forget a command, you can run the command @command{help} > * videoinfo:: List available video modes > @comment * xen_*:: Xen boot commands > * xen_hypervisor:: Load xen hypervisor binary > -* xen_linux:: Load dom0 kernel for xen hypervisor > -* xen_initrd:: Load dom0 initrd for dom0 kernel > -* xen_xsm:: Load xen security module for xen hypervisor > +* xen_module:: Load xen modules for xen hypervisor > @end menu > > > @@ -5141,30 +5139,19 @@ verbatim as the @dfn{kernel command-line}. Any other binaries must be > reloaded after using this command. > @end deffn > > -@node xen_linux > -@subsection xen_linux > +@node xen_module > +@subsection xen_module > > -@deffn Command xen_linux file [arguments] > -Load a dom0 kernel image for xen hypervisor at the booting process of xen. > +@deffn Command xen_module [--nounzip] file [arguments] > +Load a module for xen hypervisor at the booting process of xen. > The rest of the line is passed verbatim as the module command line. > +Modules should be loaded in the following order: > +The 1st module: dom0 kernel image > +The 2nd module: dom0 ramdisk (optional, if the ramdisk is present, it must > +always be the second module) > +If the XSM is present, it can be loaded at any time after the 1st module. I find the instruction confusing. You wrote that XSM can be loaded any time after the 1st module, but just before you wrote that ramdisk should always be the first one. What about: "Modules should be loaded in the following order: - dom0 kernel image - dom0 ramdisk if present - XSM policy if present" Regards, -- Julien Grall From mboxrd@z Thu Jan 1 00:00:00 1970 From: Julien Grall Subject: Re: [PATCH v4 4/4] arm64: update the introduction of xen boot commands in docs/grub.texi Date: Mon, 16 May 2016 12:28:17 +0100 Message-ID: <5739AED1.1020004@arm.com> References: <1462889006-23671-1-git-send-email-fu.wei@linaro.org> <1462889006-23671-5-git-send-email-fu.wei@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <1462889006-23671-5-git-send-email-fu.wei@linaro.org> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xen.org Sender: "Xen-devel" To: fu.wei@linaro.org, grub-devel@gnu.org, arvidjaar@gmail.com, phcoder@gmail.com Cc: xen-devel@lists.xensource.com, Ian.Campbell@citrix.com, jcm@redhat.com, julien.grall@linaro.org, leif.lindholm@linaro.org, Stefano Stabellini , linaro-uefi@lists.linaro.org List-Id: xen-devel@lists.xenproject.org SGkgRnUgV2VpLAoKKENDIFN0ZWZhbm8pCgpPbiAxMC8wNS8xNiAxNTowMywgZnUud2VpQGxpbmFy by5vcmcgd3JvdGU6Cj4gRnJvbTogRnUgV2VpIDxmdS53ZWlAbGluYXJvLm9yZz4KPgo+IGRlbGV0 ZTogeGVuX2xpbnV4LCB4ZW5faW5pdHJkLCB4ZW5feHNtCj4gYWRkOiB4ZW5fbW9kdWxlCj4KPiBU aGlzIHVwZGF0ZSBiYXNlcyBvbgo+ICAgICAgY29tbWl0IDBlZGQ3NTBlNTA2OTg4NTQwNjgzNThl YTUzNTI4MTAwYTkxOTI5MDIKPiAgICAgIEF1dGhvcjogVmxhZGltaXIgU2VyYmluZW5rbyA8cGhj b2RlckBnbWFpbC5jb20+Cj4gICAgICBEYXRlOiAgIEZyaSBKYW4gMjIgMTA6MTg6NDcgMjAxNiAr MDEwMAo+Cj4gICAgICAgICAgeGVuX2Jvb3Q6IFJlbW92ZSBvYnNvbGV0ZSBtb2R1bGUgdHlwZSBk aXN0aW5jdGlvbnMuCj4KPiBBbHNvIGJhc2VzIG9uIHRoZSBtb2R1bGUgbG9hZGluZyBtZWNoYW5p c20gb2YgWGVuIGNvZGU6Cj4gNDg4YzJhOCBkb2NzL2FybTY0OiBjbGFyaWZ5IHRoZSBkb2N1bWVu dGlvbiBmb3IgbG9hZGluZyBYU00gc3VwcG9ydAo+IDY3ODMxYzQgZG9jcy9hcm02NDogdXBkYXRl IHRoZSBkb2N1bWVudGF0aW9uIGZvciBsb2FkaW5nIFhTTSBzdXBwb3J0Cj4gY2EzMjAxMiB4ZW4v YXJtNjQ6IGNoZWNrIFhTTSBNYWdpYyBmcm9tIHRoZSBzZWNvbmQgdW5rbm93biBtb2R1bGUuCj4K PiBTaWduZWQtb2ZmLWJ5OiBGdSBXZWkgPGZ1LndlaUBsaW5hcm8ub3JnPgo+IC0tLQo+ICAgZG9j cy9ncnViLnRleGkgfCAzMyArKysrKysrKysrLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0KPiAgIDEg ZmlsZSBjaGFuZ2VkLCAxMCBpbnNlcnRpb25zKCspLCAyMyBkZWxldGlvbnMoLSkKPgo+IGRpZmYg LS1naXQgYS9kb2NzL2dydWIudGV4aSBiL2RvY3MvZ3J1Yi50ZXhpCj4gaW5kZXggODJmNmZhNC4u ZGY2MDE4YSAxMDA2NDQKPiAtLS0gYS9kb2NzL2dydWIudGV4aQo+ICsrKyBiL2RvY3MvZ3J1Yi50 ZXhpCj4gQEAgLTM4NjEsOSArMzg2MSw3IEBAIHlvdSBmb3JnZXQgYSBjb21tYW5kLCB5b3UgY2Fu IHJ1biB0aGUgY29tbWFuZCBAY29tbWFuZHtoZWxwfQo+ICAgKiB2aWRlb2luZm86OiAgICAgICAg ICAgICAgICAgICBMaXN0IGF2YWlsYWJsZSB2aWRlbyBtb2Rlcwo+ICAgQGNvbW1lbnQgKiB4ZW5f Kjo6ICAgICAgICAgICAgICBYZW4gYm9vdCBjb21tYW5kcwo+ICAgKiB4ZW5faHlwZXJ2aXNvcjo6 ICAgICAgICAgICAgICBMb2FkIHhlbiBoeXBlcnZpc29yIGJpbmFyeQo+IC0qIHhlbl9saW51eDo6 ICAgICAgICAgICAgICAgICAgIExvYWQgZG9tMCBrZXJuZWwgZm9yIHhlbiBoeXBlcnZpc29yCj4g LSogeGVuX2luaXRyZDo6ICAgICAgICAgICAgICAgICAgTG9hZCBkb20wIGluaXRyZCBmb3IgZG9t MCBrZXJuZWwKPiAtKiB4ZW5feHNtOjogICAgICAgICAgICAgICAgICAgICBMb2FkIHhlbiBzZWN1 cml0eSBtb2R1bGUgZm9yIHhlbiBoeXBlcnZpc29yCj4gKyogeGVuX21vZHVsZTo6ICAgICAgICAg ICAgICAgICAgTG9hZCB4ZW4gbW9kdWxlcyBmb3IgeGVuIGh5cGVydmlzb3IKPiAgIEBlbmQgbWVu dQo+Cj4KPiBAQCAtNTE0MSwzMCArNTEzOSwxOSBAQCB2ZXJiYXRpbSBhcyB0aGUgQGRmbntrZXJu ZWwgY29tbWFuZC1saW5lfS4gQW55IG90aGVyIGJpbmFyaWVzIG11c3QgYmUKPiAgIHJlbG9hZGVk IGFmdGVyIHVzaW5nIHRoaXMgY29tbWFuZC4KPiAgIEBlbmQgZGVmZm4KPgo+IC1Abm9kZSB4ZW5f bGludXgKPiAtQHN1YnNlY3Rpb24geGVuX2xpbnV4Cj4gK0Bub2RlIHhlbl9tb2R1bGUKPiArQHN1 YnNlY3Rpb24geGVuX21vZHVsZQo+Cj4gLUBkZWZmbiBDb21tYW5kIHhlbl9saW51eCBmaWxlIFth cmd1bWVudHNdCj4gLUxvYWQgYSBkb20wIGtlcm5lbCBpbWFnZSBmb3IgeGVuIGh5cGVydmlzb3Ig YXQgdGhlIGJvb3RpbmcgcHJvY2VzcyBvZiB4ZW4uCj4gK0BkZWZmbiBDb21tYW5kIHhlbl9tb2R1 bGUgWy0tbm91bnppcF0gZmlsZSBbYXJndW1lbnRzXQo+ICtMb2FkIGEgbW9kdWxlIGZvciB4ZW4g aHlwZXJ2aXNvciBhdCB0aGUgYm9vdGluZyBwcm9jZXNzIG9mIHhlbi4KPiAgIFRoZSByZXN0IG9m IHRoZSBsaW5lIGlzIHBhc3NlZCB2ZXJiYXRpbSBhcyB0aGUgbW9kdWxlIGNvbW1hbmQgbGluZS4K PiArTW9kdWxlcyBzaG91bGQgYmUgbG9hZGVkIGluIHRoZSBmb2xsb3dpbmcgb3JkZXI6Cj4gK1Ro ZSAxc3QgbW9kdWxlOiBkb20wIGtlcm5lbCBpbWFnZQo+ICtUaGUgMm5kIG1vZHVsZTogZG9tMCBy YW1kaXNrIChvcHRpb25hbCwgaWYgdGhlIHJhbWRpc2sgaXMgcHJlc2VudCwgaXQgbXVzdAo+ICth bHdheXMgYmUgdGhlIHNlY29uZCBtb2R1bGUpCj4gK0lmIHRoZSBYU00gaXMgcHJlc2VudCwgaXQg Y2FuIGJlIGxvYWRlZCBhdCBhbnkgdGltZSBhZnRlciB0aGUgMXN0IG1vZHVsZS4KCkkgZmluZCB0 aGUgaW5zdHJ1Y3Rpb24gY29uZnVzaW5nLiBZb3Ugd3JvdGUgdGhhdCBYU00gY2FuIGJlIGxvYWRl ZCBhbnkgCnRpbWUgYWZ0ZXIgdGhlIDFzdCBtb2R1bGUsIGJ1dCBqdXN0IGJlZm9yZSB5b3Ugd3Jv dGUgdGhhdCByYW1kaXNrIHNob3VsZCAKYWx3YXlzIGJlIHRoZSBmaXJzdCBvbmUuCgpXaGF0IGFi b3V0OgoiTW9kdWxlcyBzaG91bGQgYmUgbG9hZGVkIGluIHRoZSBmb2xsb3dpbmcgb3JkZXI6CiAg LSBkb20wIGtlcm5lbCBpbWFnZQogIC0gZG9tMCByYW1kaXNrIGlmIHByZXNlbnQKICAtIFhTTSBw b2xpY3kgaWYgcHJlc2VudCIKClJlZ2FyZHMsCgotLSAKSnVsaWVuIEdyYWxsCgpfX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpYZW4tZGV2ZWwgbWFpbGluZyBs aXN0Clhlbi1kZXZlbEBsaXN0cy54ZW4ub3JnCmh0dHA6Ly9saXN0cy54ZW4ub3JnL3hlbi1kZXZl bAo=