All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ian Jackson <Ian.Jackson@eu.citrix.com>
To: Julien Grall <julien.grall@arm.com>
Cc: xen-devel@lists.xensource.com, Wei Liu <wei.liu2@citrix.com>,
	jcm@redhat.com, George Dunlap <george.dunlap@citrix.com>,
	leif.lindholm@linaro.org, xen-devel@lists.xen.org,
	sstabellini@kernel.org, linaro-uefi@lists.linaro.org,
	dgdegra@tycho.nsa.gov, fu.wei@linaro.org
Subject: [PATCH v3] docs/arm64: update the documention for loading XSM support
Date: Fri, 22 Apr 2016 18:29:34 +0100	[thread overview]
Message-ID: <22298.24446.56007.535207@mariner.uk.xensource.com> (raw)
In-Reply-To: <571A53E2.2020603@arm.com>

Julien Grall writes ("Re: [PATCH v2] docs/arm64: update the documention for loading XSM support"):
> The new version looks good to me:
> Acked-by: Julien Grall <julien.grall@arm.com>
> 
> Can a native speaker (Ian, Konrad, George) double-check the wording)?

I found it rather difficult to read.  See updated version, attached.

I dropped your ack because I want to be sure that the new version
still describes the actual behaviour.  I kept Wei's release-ack.

Thanks,
Ian.

From bd8f24667d353a4c90203d51c1fdb42a66b79973 Mon Sep 17 00:00:00 2001
From: Fu Wei <fu.wei@linaro.org>
Date: Thu, 21 Apr 2016 19:07:09 +0800
Subject: [PATCH] docs/arm64: update the documention for loading XSM support

This patch updates the documentation for allowing detection of an XSM
module that lacks a specific compatible string.

(This mechanism was added in commit ca32012341f3,
 "xen/arm64: check XSM Magic from the second unknown module.")

Signed-off-by: Fu Wei <fu.wei@linaro.org>
Signed-off-by: Ian Jackson <ian.jackson@eu.citrix.com>
Release-acked-by: Wei Liu <wei.liu2@citrix.com>
---
 docs/misc/arm/device-tree/booting.txt |   27 +++++++++++++++++++++++----
 1 file changed, 23 insertions(+), 4 deletions(-)

diff --git a/docs/misc/arm/device-tree/booting.txt b/docs/misc/arm/device-tree/booting.txt
index ad98bf3..f3179d6 100644
--- a/docs/misc/arm/device-tree/booting.txt
+++ b/docs/misc/arm/device-tree/booting.txt
@@ -24,10 +24,29 @@ Each node contains the following properties:
 	string (which must always be present).
 
 	Xen will assume that the first module which lacks a more
-	specific compatible string is a "multiboot,kernel" and that
-	the second such is a "multiboot,ramdisk". Any subsequent
-	modules which lack a specific compatiblity string will not
-	receive any special treatment.
+	specific compatible string is a "multiboot,kernel".
+
+	Xen will examine each module, starting from the second
+	module that lacks a specific compatible string.  Xen will
+        check each such module for the XSM Magic number:
+
+	- For a module which has the XSM Magic number: it will be
+          treated by Xen as if its compatible string was
+          "xen,xsm-policy";
+
+	- For a module which does not have the XSM Magic: the second
+          module lacking a compatible string will be treated by Xen as
+          if its compatible string was "multiboot,ramdisk"; for the
+          third and subsequent modules which lack a specific
+          compatible string, Xen will not apply any special treatment.
+
+	This means if the ramdisk module is present and does not have the
+	compatible string "multiboot,ramdisk", then it must always be the
+	second module.
+
+	Note: This XSM Magic detection behavior was introduced by Xen 4.7.
+	Xen 4.6 (and downwards) still requires the XSM module to have the
+	compatible string "xen,xsm-policy".
 
 	Xen 4.4 supported a different set of legacy compatible strings
 	which remain supported such that systems supporting both 4.4
-- 
1.7.10.4


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

  parent reply	other threads:[~2016-04-22 17:29 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-04-21 11:07 [PATCH v2] docs/arm64: update the documention for loading XSM support fu.wei
2016-04-21 11:40 ` Jan Beulich
2016-04-26 14:00   ` Fu Wei
2016-04-22 16:40 ` Julien Grall
2016-04-22 16:41   ` Julien Grall
2016-04-22 16:43   ` Wei Liu
2016-04-22 17:28     ` Stefano Stabellini
2016-04-22 17:37       ` Ian Jackson
2016-04-22 17:29   ` Ian Jackson [this message]
2016-04-22 17:37     ` [PATCH v3] " Konrad Rzeszutek Wilk
2016-04-25  9:42     ` Julien Grall
2016-04-25 15:12       ` Ian Jackson
2016-04-25 15:25         ` Stefano Stabellini
2016-04-25 15:35           ` [PATCH] docs/arm64: clarify " Ian Jackson
2016-04-25 16:16             ` Julien Grall
2016-04-25 16:35               ` Ian Jackson
2016-04-25 16:38                 ` [PATCH v2] " Ian Jackson
2016-04-25 16:45                   ` Julien Grall
2016-04-25 16:49                     ` Wei Liu
2016-04-26  9:47             ` [PATCH] " Stefano Stabellini
2016-04-26  9:56               ` Wei Liu
2016-04-26  9:57                 ` Stefano Stabellini
2016-04-26 14:03           ` [PATCH v3] docs/arm64: update " Fu Wei

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=22298.24446.56007.535207@mariner.uk.xensource.com \
    --to=ian.jackson@eu.citrix.com \
    --cc=dgdegra@tycho.nsa.gov \
    --cc=fu.wei@linaro.org \
    --cc=george.dunlap@citrix.com \
    --cc=jcm@redhat.com \
    --cc=julien.grall@arm.com \
    --cc=leif.lindholm@linaro.org \
    --cc=linaro-uefi@lists.linaro.org \
    --cc=sstabellini@kernel.org \
    --cc=wei.liu2@citrix.com \
    --cc=xen-devel@lists.xen.org \
    --cc=xen-devel@lists.xensource.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 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.