* [PATCH v6 0/4] arm64,xen: add xen_boot support into grup-mkconfig
@ 2016-07-26 9:13 ` fu.wei
0 siblings, 0 replies; 23+ messages in thread
From: fu.wei @ 2016-07-26 9:13 UTC (permalink / raw)
To: grub-devel, arvidjaar, phcoder
Cc: linaro-uefi, Ian.Campbell, julien.grall, xen-devel, jcm,
leif.lindholm, sstabellini, Fu Wei
From: Fu Wei <fu.wei@linaro.org>
This patchset add xen_boot support into grup-mkconfig for
generating xen boot entrances automatically
Also update the docs/grub.texi for new xen_boot commands.
This patchset has been tested on Foudation model with a bug fix:
http://lists.gnu.org/archive/html/grub-devel/2016-02/msg00205.html
ChangeLog:
v6: http://lists.gnu.org/archive/html/grub-devel/2016-07/
Fix Coding style of util/grub.d/20_linux_xen.in, use soft tab.
v5: http://lists.gnu.org/archive/html/grub-devel/2016-07/msg00008.html
Update the introduction of xen_module commands in docs/grub.texi,
according to the suggestion from Julien Grall
v4: http://lists.gnu.org/archive/html/grub-devel/2016-05/
according to the XSM loading mechanism of Xen(upstreamed),
update the introduction of xen_module commands in docs/grub.texi
v3: http://lists.gnu.org/archive/html/grub-devel/2016-02/msg00314.html
reorder the patches
update the introduction of xen_module commands in docs/grub.texi
v2: http://lists.gnu.org/archive/html/grub-devel/2016-02/msg00282.html
add "--nounzip" option support in xen_module
use "feature_xen_boot" instead of "grub_xen_boot"
update the introduction of xen boot commands in docs/grub.texi
v1 :first upstream patchset:
http://lists.gnu.org/archive/html/grub-devel/2016-02/msg00264.html
Fu Wei (4):
i386,xen: Add xen_hypervisor and xen_module aliases for i386
arm64: add "--nounzip" option support in xen_module command
* util/grub.d/20_linux_xen.in: Add xen_boot command support
arm64: update the introduction of xen boot commands in docs/grub.texi
docs/grub.texi | 32 +++++++++-----------------------
grub-core/loader/arm64/xen_boot.c | 17 +++++++++++++++++
grub-core/loader/i386/xen.c | 7 +++++++
grub-core/normal/main.c | 2 +-
util/grub.d/20_linux_xen.in | 13 ++++++++++---
5 files changed, 44 insertions(+), 27 deletions(-)
--
2.5.5
^ permalink raw reply [flat|nested] 23+ messages in thread* [PATCH v6 1/4] i386, xen: Add xen_hypervisor and xen_module aliases for i386
2016-07-26 9:13 ` fu.wei
@ 2016-07-26 9:13 ` fu.wei
-1 siblings, 0 replies; 23+ messages in thread
From: fu.wei @ 2016-07-26 9:13 UTC (permalink / raw)
To: grub-devel, arvidjaar, phcoder
Cc: xen-devel, Ian.Campbell, jcm, leif.lindholm, julien.grall,
sstabellini, linaro-uefi, Fu Wei
From: Fu Wei <fu.wei@linaro.org>
Signed-off-by: Fu Wei <fu.wei@linaro.org>
---
grub-core/loader/i386/xen.c | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/grub-core/loader/i386/xen.c b/grub-core/loader/i386/xen.c
index c4d9689..15b0727 100644
--- a/grub-core/loader/i386/xen.c
+++ b/grub-core/loader/i386/xen.c
@@ -689,6 +689,7 @@ fail:
}
static grub_command_t cmd_xen, cmd_initrd, cmd_module, cmd_multiboot;
+static grub_command_t cmd_xen_hypervisor, cmd_xen_module;
GRUB_MOD_INIT (xen)
{
@@ -696,10 +697,14 @@ GRUB_MOD_INIT (xen)
0, N_("Load Linux."));
cmd_multiboot = grub_register_command ("multiboot", grub_cmd_xen,
0, N_("Load Linux."));
+ cmd_xen_hypervisor = grub_register_command ("xen_hypervisor", grub_cmd_xen,
+ 0, N_("Load Linux."));
cmd_initrd = grub_register_command ("initrd", grub_cmd_initrd,
0, N_("Load initrd."));
cmd_module = grub_register_command ("module", grub_cmd_module,
0, N_("Load module."));
+ cmd_xen_module = grub_register_command ("xen_module", grub_cmd_module,
+ 0, N_("Load module."));
my_mod = mod;
}
@@ -709,4 +714,6 @@ GRUB_MOD_FINI (xen)
grub_unregister_command (cmd_initrd);
grub_unregister_command (cmd_multiboot);
grub_unregister_command (cmd_module);
+ grub_unregister_command (cmd_xen_module);
+ grub_unregister_command (cmd_xen_hypervisor);
}
--
2.5.5
^ permalink raw reply related [flat|nested] 23+ messages in thread* [PATCH v6 1/4] i386, xen: Add xen_hypervisor and xen_module aliases for i386
@ 2016-07-26 9:13 ` fu.wei
0 siblings, 0 replies; 23+ messages in thread
From: fu.wei @ 2016-07-26 9:13 UTC (permalink / raw)
To: grub-devel, arvidjaar, phcoder
Cc: linaro-uefi, Ian.Campbell, julien.grall, xen-devel, jcm,
leif.lindholm, sstabellini, Fu Wei
From: Fu Wei <fu.wei@linaro.org>
Signed-off-by: Fu Wei <fu.wei@linaro.org>
---
grub-core/loader/i386/xen.c | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/grub-core/loader/i386/xen.c b/grub-core/loader/i386/xen.c
index c4d9689..15b0727 100644
--- a/grub-core/loader/i386/xen.c
+++ b/grub-core/loader/i386/xen.c
@@ -689,6 +689,7 @@ fail:
}
static grub_command_t cmd_xen, cmd_initrd, cmd_module, cmd_multiboot;
+static grub_command_t cmd_xen_hypervisor, cmd_xen_module;
GRUB_MOD_INIT (xen)
{
@@ -696,10 +697,14 @@ GRUB_MOD_INIT (xen)
0, N_("Load Linux."));
cmd_multiboot = grub_register_command ("multiboot", grub_cmd_xen,
0, N_("Load Linux."));
+ cmd_xen_hypervisor = grub_register_command ("xen_hypervisor", grub_cmd_xen,
+ 0, N_("Load Linux."));
cmd_initrd = grub_register_command ("initrd", grub_cmd_initrd,
0, N_("Load initrd."));
cmd_module = grub_register_command ("module", grub_cmd_module,
0, N_("Load module."));
+ cmd_xen_module = grub_register_command ("xen_module", grub_cmd_module,
+ 0, N_("Load module."));
my_mod = mod;
}
@@ -709,4 +714,6 @@ GRUB_MOD_FINI (xen)
grub_unregister_command (cmd_initrd);
grub_unregister_command (cmd_multiboot);
grub_unregister_command (cmd_module);
+ grub_unregister_command (cmd_xen_module);
+ grub_unregister_command (cmd_xen_hypervisor);
}
--
2.5.5
^ permalink raw reply related [flat|nested] 23+ messages in thread* Re: [PATCH v6 1/4] i386, xen: Add xen_hypervisor and xen_module aliases for i386
2016-07-26 9:13 ` fu.wei
@ 2016-11-03 14:37 ` Daniel Kiper
-1 siblings, 0 replies; 23+ messages in thread
From: Daniel Kiper @ 2016-11-03 14:37 UTC (permalink / raw)
To: grub-devel, fu.wei
Cc: xen-devel, arvidjaar, phcoder, linaro-uefi, leif.lindholm,
julien.grall, sstabellini, jcm
On Tue, Jul 26, 2016 at 05:13:40PM +0800, fu.wei@linaro.org wrote:
> From: Fu Wei <fu.wei@linaro.org>
>
> Signed-off-by: Fu Wei <fu.wei@linaro.org>
> ---
> grub-core/loader/i386/xen.c | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/grub-core/loader/i386/xen.c b/grub-core/loader/i386/xen.c
> index c4d9689..15b0727 100644
> --- a/grub-core/loader/i386/xen.c
> +++ b/grub-core/loader/i386/xen.c
> @@ -689,6 +689,7 @@ fail:
> }
>
> static grub_command_t cmd_xen, cmd_initrd, cmd_module, cmd_multiboot;
> +static grub_command_t cmd_xen_hypervisor, cmd_xen_module;
>
> GRUB_MOD_INIT (xen)
> {
> @@ -696,10 +697,14 @@ GRUB_MOD_INIT (xen)
> 0, N_("Load Linux."));
> cmd_multiboot = grub_register_command ("multiboot", grub_cmd_xen,
> 0, N_("Load Linux."));
> + cmd_xen_hypervisor = grub_register_command ("xen_hypervisor", grub_cmd_xen,
> + 0, N_("Load Linux."));
> cmd_initrd = grub_register_command ("initrd", grub_cmd_initrd,
> 0, N_("Load initrd."));
> cmd_module = grub_register_command ("module", grub_cmd_module,
> 0, N_("Load module."));
> + cmd_xen_module = grub_register_command ("xen_module", grub_cmd_module,
> + 0, N_("Load module."));
> my_mod = mod;
> }
>
> @@ -709,4 +714,6 @@ GRUB_MOD_FINI (xen)
> grub_unregister_command (cmd_initrd);
> grub_unregister_command (cmd_multiboot);
> grub_unregister_command (cmd_module);
> + grub_unregister_command (cmd_xen_module);
> + grub_unregister_command (cmd_xen_hypervisor);
Hmmm... This whole patch does not make sense for me. Why are you
adding aliases for ARM Xen hypervisor et consortes load commands
(AIUI) in i386 PV guest loader?
Daniel
^ permalink raw reply [flat|nested] 23+ messages in thread* Re: [PATCH v6 1/4] i386, xen: Add xen_hypervisor and xen_module aliases for i386
@ 2016-11-03 14:37 ` Daniel Kiper
0 siblings, 0 replies; 23+ messages in thread
From: Daniel Kiper @ 2016-11-03 14:37 UTC (permalink / raw)
To: grub-devel, fu.wei
Cc: arvidjaar, phcoder, xen-devel, jcm, leif.lindholm, julien.grall,
sstabellini, linaro-uefi
On Tue, Jul 26, 2016 at 05:13:40PM +0800, fu.wei@linaro.org wrote:
> From: Fu Wei <fu.wei@linaro.org>
>
> Signed-off-by: Fu Wei <fu.wei@linaro.org>
> ---
> grub-core/loader/i386/xen.c | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/grub-core/loader/i386/xen.c b/grub-core/loader/i386/xen.c
> index c4d9689..15b0727 100644
> --- a/grub-core/loader/i386/xen.c
> +++ b/grub-core/loader/i386/xen.c
> @@ -689,6 +689,7 @@ fail:
> }
>
> static grub_command_t cmd_xen, cmd_initrd, cmd_module, cmd_multiboot;
> +static grub_command_t cmd_xen_hypervisor, cmd_xen_module;
>
> GRUB_MOD_INIT (xen)
> {
> @@ -696,10 +697,14 @@ GRUB_MOD_INIT (xen)
> 0, N_("Load Linux."));
> cmd_multiboot = grub_register_command ("multiboot", grub_cmd_xen,
> 0, N_("Load Linux."));
> + cmd_xen_hypervisor = grub_register_command ("xen_hypervisor", grub_cmd_xen,
> + 0, N_("Load Linux."));
> cmd_initrd = grub_register_command ("initrd", grub_cmd_initrd,
> 0, N_("Load initrd."));
> cmd_module = grub_register_command ("module", grub_cmd_module,
> 0, N_("Load module."));
> + cmd_xen_module = grub_register_command ("xen_module", grub_cmd_module,
> + 0, N_("Load module."));
> my_mod = mod;
> }
>
> @@ -709,4 +714,6 @@ GRUB_MOD_FINI (xen)
> grub_unregister_command (cmd_initrd);
> grub_unregister_command (cmd_multiboot);
> grub_unregister_command (cmd_module);
> + grub_unregister_command (cmd_xen_module);
> + grub_unregister_command (cmd_xen_hypervisor);
Hmmm... This whole patch does not make sense for me. Why are you
adding aliases for ARM Xen hypervisor et consortes load commands
(AIUI) in i386 PV guest loader?
Daniel
^ permalink raw reply [flat|nested] 23+ messages in thread
* [PATCH v6 2/4] arm64: add "--nounzip" option support in xen_module command
2016-07-26 9:13 ` fu.wei
@ 2016-07-26 9:13 ` fu.wei
-1 siblings, 0 replies; 23+ messages in thread
From: fu.wei @ 2016-07-26 9:13 UTC (permalink / raw)
To: grub-devel, arvidjaar, phcoder
Cc: xen-devel, Ian.Campbell, jcm, leif.lindholm, julien.grall,
sstabellini, linaro-uefi, Fu Wei
From: Fu Wei <fu.wei@linaro.org>
This patch adds "--nounzip" option support in order to
be compatible with the module command of i386, then we can
simplify grub-mkconfig support code.
Signed-off-by: Fu Wei <fu.wei@linaro.org>
---
grub-core/loader/arm64/xen_boot.c | 17 +++++++++++++++++
1 file changed, 17 insertions(+)
diff --git a/grub-core/loader/arm64/xen_boot.c b/grub-core/loader/arm64/xen_boot.c
index a914eb8..0878364 100644
--- a/grub-core/loader/arm64/xen_boot.c
+++ b/grub-core/loader/arm64/xen_boot.c
@@ -20,6 +20,7 @@
#include <grub/charset.h>
#include <grub/command.h>
#include <grub/err.h>
+#include <grub/env.h>
#include <grub/file.h>
#include <grub/fdt.h>
#include <grub/list.h>
@@ -379,6 +380,20 @@ grub_cmd_xen_module (grub_command_t cmd __attribute__((unused)),
struct xen_boot_binary *module = NULL;
grub_file_t file = 0;
+ int nounzip = 0;
+
+ if (!argc)
+ {
+ grub_error (GRUB_ERR_BAD_ARGUMENT, N_("filename expected"));
+ goto fail;
+ }
+
+ if (grub_strcmp (argv[0], "--nounzip") == 0)
+ {
+ argv++;
+ argc--;
+ nounzip = 1;
+ }
if (!argc)
{
@@ -403,6 +418,8 @@ grub_cmd_xen_module (grub_command_t cmd __attribute__((unused)),
grub_dprintf ("xen_loader", "Init module and node info\n");
+ if (nounzip)
+ grub_file_filter_disable_compression ();
file = grub_file_open (argv[0]);
if (!file)
goto fail;
--
2.5.5
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
^ permalink raw reply related [flat|nested] 23+ messages in thread* [PATCH v6 2/4] arm64: add "--nounzip" option support in xen_module command
@ 2016-07-26 9:13 ` fu.wei
0 siblings, 0 replies; 23+ messages in thread
From: fu.wei @ 2016-07-26 9:13 UTC (permalink / raw)
To: grub-devel, arvidjaar, phcoder
Cc: linaro-uefi, Ian.Campbell, julien.grall, xen-devel, jcm,
leif.lindholm, sstabellini, Fu Wei
From: Fu Wei <fu.wei@linaro.org>
This patch adds "--nounzip" option support in order to
be compatible with the module command of i386, then we can
simplify grub-mkconfig support code.
Signed-off-by: Fu Wei <fu.wei@linaro.org>
---
grub-core/loader/arm64/xen_boot.c | 17 +++++++++++++++++
1 file changed, 17 insertions(+)
diff --git a/grub-core/loader/arm64/xen_boot.c b/grub-core/loader/arm64/xen_boot.c
index a914eb8..0878364 100644
--- a/grub-core/loader/arm64/xen_boot.c
+++ b/grub-core/loader/arm64/xen_boot.c
@@ -20,6 +20,7 @@
#include <grub/charset.h>
#include <grub/command.h>
#include <grub/err.h>
+#include <grub/env.h>
#include <grub/file.h>
#include <grub/fdt.h>
#include <grub/list.h>
@@ -379,6 +380,20 @@ grub_cmd_xen_module (grub_command_t cmd __attribute__((unused)),
struct xen_boot_binary *module = NULL;
grub_file_t file = 0;
+ int nounzip = 0;
+
+ if (!argc)
+ {
+ grub_error (GRUB_ERR_BAD_ARGUMENT, N_("filename expected"));
+ goto fail;
+ }
+
+ if (grub_strcmp (argv[0], "--nounzip") == 0)
+ {
+ argv++;
+ argc--;
+ nounzip = 1;
+ }
if (!argc)
{
@@ -403,6 +418,8 @@ grub_cmd_xen_module (grub_command_t cmd __attribute__((unused)),
grub_dprintf ("xen_loader", "Init module and node info\n");
+ if (nounzip)
+ grub_file_filter_disable_compression ();
file = grub_file_open (argv[0]);
if (!file)
goto fail;
--
2.5.5
^ permalink raw reply related [flat|nested] 23+ messages in thread* Re: [PATCH v6 2/4] arm64: add "--nounzip" option support in xen_module command
2016-07-26 9:13 ` fu.wei
@ 2016-11-03 15:00 ` Daniel Kiper
-1 siblings, 0 replies; 23+ messages in thread
From: Daniel Kiper @ 2016-11-03 15:00 UTC (permalink / raw)
To: grub-devel, fu.wei
Cc: xen-devel, arvidjaar, phcoder, linaro-uefi, leif.lindholm,
julien.grall, sstabellini, jcm
On Tue, Jul 26, 2016 at 05:13:41PM +0800, fu.wei@linaro.org wrote:
> From: Fu Wei <fu.wei@linaro.org>
>
> This patch adds "--nounzip" option support in order to
> be compatible with the module command of i386, then we can
> simplify grub-mkconfig support code.
>
> Signed-off-by: Fu Wei <fu.wei@linaro.org>
> ---
> grub-core/loader/arm64/xen_boot.c | 17 +++++++++++++++++
> 1 file changed, 17 insertions(+)
>
> diff --git a/grub-core/loader/arm64/xen_boot.c b/grub-core/loader/arm64/xen_boot.c
> index a914eb8..0878364 100644
> --- a/grub-core/loader/arm64/xen_boot.c
> +++ b/grub-core/loader/arm64/xen_boot.c
> @@ -20,6 +20,7 @@
> #include <grub/charset.h>
> #include <grub/command.h>
> #include <grub/err.h>
> +#include <grub/env.h>
Is it really needed?
Daniel
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH v6 2/4] arm64: add "--nounzip" option support in xen_module command
@ 2016-11-03 15:00 ` Daniel Kiper
0 siblings, 0 replies; 23+ messages in thread
From: Daniel Kiper @ 2016-11-03 15:00 UTC (permalink / raw)
To: grub-devel, fu.wei
Cc: arvidjaar, phcoder, xen-devel, jcm, leif.lindholm, julien.grall,
sstabellini, linaro-uefi
On Tue, Jul 26, 2016 at 05:13:41PM +0800, fu.wei@linaro.org wrote:
> From: Fu Wei <fu.wei@linaro.org>
>
> This patch adds "--nounzip" option support in order to
> be compatible with the module command of i386, then we can
> simplify grub-mkconfig support code.
>
> Signed-off-by: Fu Wei <fu.wei@linaro.org>
> ---
> grub-core/loader/arm64/xen_boot.c | 17 +++++++++++++++++
> 1 file changed, 17 insertions(+)
>
> diff --git a/grub-core/loader/arm64/xen_boot.c b/grub-core/loader/arm64/xen_boot.c
> index a914eb8..0878364 100644
> --- a/grub-core/loader/arm64/xen_boot.c
> +++ b/grub-core/loader/arm64/xen_boot.c
> @@ -20,6 +20,7 @@
> #include <grub/charset.h>
> #include <grub/command.h>
> #include <grub/err.h>
> +#include <grub/env.h>
Is it really needed?
Daniel
^ permalink raw reply [flat|nested] 23+ messages in thread
* [PATCH v6 3/4] * util/grub.d/20_linux_xen.in: Add xen_boot command support
2016-07-26 9:13 ` fu.wei
@ 2016-07-26 9:13 ` fu.wei
-1 siblings, 0 replies; 23+ messages in thread
From: fu.wei @ 2016-07-26 9:13 UTC (permalink / raw)
To: grub-devel, arvidjaar, phcoder
Cc: xen-devel, Ian.Campbell, jcm, leif.lindholm, julien.grall,
sstabellini, linaro-uefi, Fu Wei
From: Fu Wei <fu.wei@linaro.org>
This patch adds the support of xen_boot command:
xen_hypervisor
xen_module
Also add a new "feature_xen_boot" to indicate this grub support
xen_boot command.
Signed-off-by: Fu Wei <fu.wei@linaro.org>
---
grub-core/normal/main.c | 2 +-
util/grub.d/20_linux_xen.in | 13 ++++++++++---
2 files changed, 11 insertions(+), 4 deletions(-)
diff --git a/grub-core/normal/main.c b/grub-core/normal/main.c
index 78a70a8..3402a05 100644
--- a/grub-core/normal/main.c
+++ b/grub-core/normal/main.c
@@ -488,7 +488,7 @@ static const char *features[] = {
"feature_chainloader_bpb", "feature_ntldr", "feature_platform_search_hint",
"feature_default_font_path", "feature_all_video_module",
"feature_menuentry_id", "feature_menuentry_options", "feature_200_final",
- "feature_nativedisk_cmd", "feature_timeout_style"
+ "feature_nativedisk_cmd", "feature_timeout_style", "feature_xen_boot"
};
GRUB_MOD_INIT(normal)
diff --git a/util/grub.d/20_linux_xen.in b/util/grub.d/20_linux_xen.in
index c48af94..231e640 100644
--- a/util/grub.d/20_linux_xen.in
+++ b/util/grub.d/20_linux_xen.in
@@ -122,16 +122,23 @@ linux_entry ()
else
xen_rm_opts="no-real-mode edd=off"
fi
- multiboot ${rel_xen_dirname}/${xen_basename} placeholder ${xen_args} \${xen_rm_opts}
+ if [ "x\$feature_xen_boot" != xy ]; then
+ xen_loader="multiboot"
+ module_loader="module"
+ else
+ xen_loader="xen_hypervisor"
+ module_loader="xen_module"
+ fi
+ \${xen_loader} ${rel_xen_dirname}/${xen_basename} placeholder ${xen_args} \${xen_rm_opts}
echo '$(echo "$lmessage" | grub_quote)'
- module ${rel_dirname}/${basename} placeholder root=${linux_root_device_thisversion} ro ${args}
+ \${module_loader} ${rel_dirname}/${basename} placeholder root=${linux_root_device_thisversion} ro ${args}
EOF
if test -n "${initrd}" ; then
# TRANSLATORS: ramdisk isn't identifier. Should be translated.
message="$(gettext_printf "Loading initial ramdisk ...")"
sed "s/^/$submenu_indentation/" << EOF
echo '$(echo "$message" | grub_quote)'
- module --nounzip ${rel_dirname}/${initrd}
+ \${module_loader} --nounzip ${rel_dirname}/${initrd}
EOF
fi
sed "s/^/$submenu_indentation/" << EOF
--
2.5.5
^ permalink raw reply related [flat|nested] 23+ messages in thread* [PATCH v6 3/4] * util/grub.d/20_linux_xen.in: Add xen_boot command support
@ 2016-07-26 9:13 ` fu.wei
0 siblings, 0 replies; 23+ messages in thread
From: fu.wei @ 2016-07-26 9:13 UTC (permalink / raw)
To: grub-devel, arvidjaar, phcoder
Cc: linaro-uefi, Ian.Campbell, julien.grall, xen-devel, jcm,
leif.lindholm, sstabellini, Fu Wei
From: Fu Wei <fu.wei@linaro.org>
This patch adds the support of xen_boot command:
xen_hypervisor
xen_module
Also add a new "feature_xen_boot" to indicate this grub support
xen_boot command.
Signed-off-by: Fu Wei <fu.wei@linaro.org>
---
grub-core/normal/main.c | 2 +-
util/grub.d/20_linux_xen.in | 13 ++++++++++---
2 files changed, 11 insertions(+), 4 deletions(-)
diff --git a/grub-core/normal/main.c b/grub-core/normal/main.c
index 78a70a8..3402a05 100644
--- a/grub-core/normal/main.c
+++ b/grub-core/normal/main.c
@@ -488,7 +488,7 @@ static const char *features[] = {
"feature_chainloader_bpb", "feature_ntldr", "feature_platform_search_hint",
"feature_default_font_path", "feature_all_video_module",
"feature_menuentry_id", "feature_menuentry_options", "feature_200_final",
- "feature_nativedisk_cmd", "feature_timeout_style"
+ "feature_nativedisk_cmd", "feature_timeout_style", "feature_xen_boot"
};
GRUB_MOD_INIT(normal)
diff --git a/util/grub.d/20_linux_xen.in b/util/grub.d/20_linux_xen.in
index c48af94..231e640 100644
--- a/util/grub.d/20_linux_xen.in
+++ b/util/grub.d/20_linux_xen.in
@@ -122,16 +122,23 @@ linux_entry ()
else
xen_rm_opts="no-real-mode edd=off"
fi
- multiboot ${rel_xen_dirname}/${xen_basename} placeholder ${xen_args} \${xen_rm_opts}
+ if [ "x\$feature_xen_boot" != xy ]; then
+ xen_loader="multiboot"
+ module_loader="module"
+ else
+ xen_loader="xen_hypervisor"
+ module_loader="xen_module"
+ fi
+ \${xen_loader} ${rel_xen_dirname}/${xen_basename} placeholder ${xen_args} \${xen_rm_opts}
echo '$(echo "$lmessage" | grub_quote)'
- module ${rel_dirname}/${basename} placeholder root=${linux_root_device_thisversion} ro ${args}
+ \${module_loader} ${rel_dirname}/${basename} placeholder root=${linux_root_device_thisversion} ro ${args}
EOF
if test -n "${initrd}" ; then
# TRANSLATORS: ramdisk isn't identifier. Should be translated.
message="$(gettext_printf "Loading initial ramdisk ...")"
sed "s/^/$submenu_indentation/" << EOF
echo '$(echo "$message" | grub_quote)'
- module --nounzip ${rel_dirname}/${initrd}
+ \${module_loader} --nounzip ${rel_dirname}/${initrd}
EOF
fi
sed "s/^/$submenu_indentation/" << EOF
--
2.5.5
^ permalink raw reply related [flat|nested] 23+ messages in thread* Re: [PATCH v6 3/4] * util/grub.d/20_linux_xen.in: Add xen_boot command support
2016-07-26 9:13 ` fu.wei
@ 2016-11-03 15:19 ` Daniel Kiper
-1 siblings, 0 replies; 23+ messages in thread
From: Daniel Kiper @ 2016-11-03 15:19 UTC (permalink / raw)
To: grub-devel, fu.wei
Cc: xen-devel, arvidjaar, phcoder, linaro-uefi, leif.lindholm,
julien.grall, sstabellini, jcm
On Tue, Jul 26, 2016 at 05:13:42PM +0800, fu.wei@linaro.org wrote:
> From: Fu Wei <fu.wei@linaro.org>
>
> This patch adds the support of xen_boot command:
> xen_hypervisor
> xen_module
>
> Also add a new "feature_xen_boot" to indicate this grub support
> xen_boot command.
>
> Signed-off-by: Fu Wei <fu.wei@linaro.org>
> ---
> grub-core/normal/main.c | 2 +-
> util/grub.d/20_linux_xen.in | 13 ++++++++++---
> 2 files changed, 11 insertions(+), 4 deletions(-)
>
> diff --git a/grub-core/normal/main.c b/grub-core/normal/main.c
> index 78a70a8..3402a05 100644
> --- a/grub-core/normal/main.c
> +++ b/grub-core/normal/main.c
> @@ -488,7 +488,7 @@ static const char *features[] = {
> "feature_chainloader_bpb", "feature_ntldr", "feature_platform_search_hint",
> "feature_default_font_path", "feature_all_video_module",
> "feature_menuentry_id", "feature_menuentry_options", "feature_200_final",
> - "feature_nativedisk_cmd", "feature_timeout_style"
> + "feature_nativedisk_cmd", "feature_timeout_style", "feature_xen_boot"
I do not like it. This is not generic feature but ARM specific one.
Hence, we should not announce it as generic stuff. Then...
> };
>
> GRUB_MOD_INIT(normal)
> diff --git a/util/grub.d/20_linux_xen.in b/util/grub.d/20_linux_xen.in
> index c48af94..231e640 100644
> --- a/util/grub.d/20_linux_xen.in
> +++ b/util/grub.d/20_linux_xen.in
> @@ -122,16 +122,23 @@ linux_entry ()
> else
> xen_rm_opts="no-real-mode edd=off"
> fi
> - multiboot ${rel_xen_dirname}/${xen_basename} placeholder ${xen_args} \${xen_rm_opts}
> + if [ "x\$feature_xen_boot" != xy ]; then
...we should consult $grub_platform and/or $grub_cpu here.
Daniel
^ permalink raw reply [flat|nested] 23+ messages in thread* Re: [PATCH v6 3/4] * util/grub.d/20_linux_xen.in: Add xen_boot command support
@ 2016-11-03 15:19 ` Daniel Kiper
0 siblings, 0 replies; 23+ messages in thread
From: Daniel Kiper @ 2016-11-03 15:19 UTC (permalink / raw)
To: grub-devel, fu.wei
Cc: arvidjaar, phcoder, xen-devel, jcm, leif.lindholm, julien.grall,
sstabellini, linaro-uefi
On Tue, Jul 26, 2016 at 05:13:42PM +0800, fu.wei@linaro.org wrote:
> From: Fu Wei <fu.wei@linaro.org>
>
> This patch adds the support of xen_boot command:
> xen_hypervisor
> xen_module
>
> Also add a new "feature_xen_boot" to indicate this grub support
> xen_boot command.
>
> Signed-off-by: Fu Wei <fu.wei@linaro.org>
> ---
> grub-core/normal/main.c | 2 +-
> util/grub.d/20_linux_xen.in | 13 ++++++++++---
> 2 files changed, 11 insertions(+), 4 deletions(-)
>
> diff --git a/grub-core/normal/main.c b/grub-core/normal/main.c
> index 78a70a8..3402a05 100644
> --- a/grub-core/normal/main.c
> +++ b/grub-core/normal/main.c
> @@ -488,7 +488,7 @@ static const char *features[] = {
> "feature_chainloader_bpb", "feature_ntldr", "feature_platform_search_hint",
> "feature_default_font_path", "feature_all_video_module",
> "feature_menuentry_id", "feature_menuentry_options", "feature_200_final",
> - "feature_nativedisk_cmd", "feature_timeout_style"
> + "feature_nativedisk_cmd", "feature_timeout_style", "feature_xen_boot"
I do not like it. This is not generic feature but ARM specific one.
Hence, we should not announce it as generic stuff. Then...
> };
>
> GRUB_MOD_INIT(normal)
> diff --git a/util/grub.d/20_linux_xen.in b/util/grub.d/20_linux_xen.in
> index c48af94..231e640 100644
> --- a/util/grub.d/20_linux_xen.in
> +++ b/util/grub.d/20_linux_xen.in
> @@ -122,16 +122,23 @@ linux_entry ()
> else
> xen_rm_opts="no-real-mode edd=off"
> fi
> - multiboot ${rel_xen_dirname}/${xen_basename} placeholder ${xen_args} \${xen_rm_opts}
> + if [ "x\$feature_xen_boot" != xy ]; then
...we should consult $grub_platform and/or $grub_cpu here.
Daniel
^ permalink raw reply [flat|nested] 23+ messages in thread
* [PATCH v6 4/4] arm64: update the introduction of xen boot commands in docs/grub.texi
2016-07-26 9:13 ` fu.wei
@ 2016-07-26 9:13 ` fu.wei
-1 siblings, 0 replies; 23+ messages in thread
From: fu.wei @ 2016-07-26 9:13 UTC (permalink / raw)
To: grub-devel, arvidjaar, phcoder
Cc: xen-devel, Ian.Campbell, jcm, leif.lindholm, julien.grall,
sstabellini, linaro-uefi, Fu Wei
From: Fu Wei <fu.wei@linaro.org>
delete: xen_linux, xen_initrd, xen_xsm
add: xen_module
This update bases on
commit 0edd750e50698854068358ea53528100a9192902
Author: Vladimir Serbinenko <phcoder@gmail.com>
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 <fu.wei@linaro.org>
Reviewed-by: Julien Grall <julien.grall@arm.com>
---
docs/grub.texi | 32 +++++++++-----------------------
1 file changed, 9 insertions(+), 23 deletions(-)
diff --git a/docs/grub.texi b/docs/grub.texi
index 82f6fa4..85c913e 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,18 @@ 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:
+ - dom0 kernel image
+ - dom0 ramdisk if present
+ - XSM policy if present
@end deffn
-@node xen_initrd
-@subsection xen_initrd
-
-@deffn Command xen_initrd file
-Load a initrd image for dom0 kernel at the booting process of xen.
-@end deffn
-
-@node xen_xsm
-@subsection xen_xsm
-
-@deffn Command xen_xsm file
-Load a xen security module for xen hypervisor at the booting process of xen.
-See @uref{http://wiki.xen.org/wiki/XSM} for more detail.
-@end deffn
-
-
@node Networking commands
@section The list of networking commands
--
2.5.5
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
^ permalink raw reply related [flat|nested] 23+ messages in thread* [PATCH v6 4/4] arm64: update the introduction of xen boot commands in docs/grub.texi
@ 2016-07-26 9:13 ` fu.wei
0 siblings, 0 replies; 23+ messages in thread
From: fu.wei @ 2016-07-26 9:13 UTC (permalink / raw)
To: grub-devel, arvidjaar, phcoder
Cc: linaro-uefi, Ian.Campbell, julien.grall, xen-devel, jcm,
leif.lindholm, sstabellini, Fu Wei
From: Fu Wei <fu.wei@linaro.org>
delete: xen_linux, xen_initrd, xen_xsm
add: xen_module
This update bases on
commit 0edd750e50698854068358ea53528100a9192902
Author: Vladimir Serbinenko <phcoder@gmail.com>
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 <fu.wei@linaro.org>
Reviewed-by: Julien Grall <julien.grall@arm.com>
---
docs/grub.texi | 32 +++++++++-----------------------
1 file changed, 9 insertions(+), 23 deletions(-)
diff --git a/docs/grub.texi b/docs/grub.texi
index 82f6fa4..85c913e 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,18 @@ 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:
+ - dom0 kernel image
+ - dom0 ramdisk if present
+ - XSM policy if present
@end deffn
-@node xen_initrd
-@subsection xen_initrd
-
-@deffn Command xen_initrd file
-Load a initrd image for dom0 kernel at the booting process of xen.
-@end deffn
-
-@node xen_xsm
-@subsection xen_xsm
-
-@deffn Command xen_xsm file
-Load a xen security module for xen hypervisor at the booting process of xen.
-See @uref{http://wiki.xen.org/wiki/XSM} for more detail.
-@end deffn
-
-
@node Networking commands
@section The list of networking commands
--
2.5.5
^ permalink raw reply related [flat|nested] 23+ messages in thread* Re: [PATCH v6 4/4] arm64: update the introduction of xen boot commands in docs/grub.texi
2016-07-26 9:13 ` fu.wei
@ 2016-11-03 15:25 ` Daniel Kiper
-1 siblings, 0 replies; 23+ messages in thread
From: Daniel Kiper @ 2016-11-03 15:25 UTC (permalink / raw)
To: grub-devel, fu.wei
Cc: xen-devel, arvidjaar, phcoder, linaro-uefi, leif.lindholm,
julien.grall, sstabellini, jcm
On Tue, Jul 26, 2016 at 05:13:43PM +0800, fu.wei@linaro.org wrote:
> From: Fu Wei <fu.wei@linaro.org>
>
> delete: xen_linux, xen_initrd, xen_xsm
> add: xen_module
>
> This update bases on
> commit 0edd750e50698854068358ea53528100a9192902
> Author: Vladimir Serbinenko <phcoder@gmail.com>
> 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 <fu.wei@linaro.org>
> Reviewed-by: Julien Grall <julien.grall@arm.com>
> ---
> docs/grub.texi | 32 +++++++++-----------------------
> 1 file changed, 9 insertions(+), 23 deletions(-)
>
> diff --git a/docs/grub.texi b/docs/grub.texi
> index 82f6fa4..85c913e 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,18 @@ 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:
> + - dom0 kernel image
> + - dom0 ramdisk if present
> + - XSM policy if present
What happens if somebody wants to use XSM without ramdisk?
Please do not forget to mention somewhere that xen_hypervisor and
xen_module commands are ARM specific.
Daniel
^ permalink raw reply [flat|nested] 23+ messages in thread* Re: [PATCH v6 4/4] arm64: update the introduction of xen boot commands in docs/grub.texi
@ 2016-11-03 15:25 ` Daniel Kiper
0 siblings, 0 replies; 23+ messages in thread
From: Daniel Kiper @ 2016-11-03 15:25 UTC (permalink / raw)
To: grub-devel, fu.wei
Cc: arvidjaar, phcoder, xen-devel, jcm, leif.lindholm, julien.grall,
sstabellini, linaro-uefi
On Tue, Jul 26, 2016 at 05:13:43PM +0800, fu.wei@linaro.org wrote:
> From: Fu Wei <fu.wei@linaro.org>
>
> delete: xen_linux, xen_initrd, xen_xsm
> add: xen_module
>
> This update bases on
> commit 0edd750e50698854068358ea53528100a9192902
> Author: Vladimir Serbinenko <phcoder@gmail.com>
> 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 <fu.wei@linaro.org>
> Reviewed-by: Julien Grall <julien.grall@arm.com>
> ---
> docs/grub.texi | 32 +++++++++-----------------------
> 1 file changed, 9 insertions(+), 23 deletions(-)
>
> diff --git a/docs/grub.texi b/docs/grub.texi
> index 82f6fa4..85c913e 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,18 @@ 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:
> + - dom0 kernel image
> + - dom0 ramdisk if present
> + - XSM policy if present
What happens if somebody wants to use XSM without ramdisk?
Please do not forget to mention somewhere that xen_hypervisor and
xen_module commands are ARM specific.
Daniel
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH v6 0/4] arm64, xen: add xen_boot support into grup-mkconfig
2016-07-26 9:13 ` fu.wei
@ 2016-09-01 12:47 ` Julien Grall
-1 siblings, 0 replies; 23+ messages in thread
From: Julien Grall @ 2016-09-01 12:47 UTC (permalink / raw)
To: grub-devel, Daniel Kiper
Cc: xen-devel, jcm, leif.lindholm, sstabellini, linaro-uefi, fu.wei
Hi Daniel,
I have heard you became co-maintainer of GRUB. Congratulations for that :).
We have a couple series (this series and [1]) to allow grub booting Xen
on ARM that have been waiting on the GRUB ML for a while to be pushed.
Those patches have been tested and already integrated in some
distributions. It would be nice to get them in GRUB upstream.
Would it be possible for you to take a look?
Regards,
[1] https://lists.gnu.org/archive/html/grub-devel/2016-02/msg00205.html
On 26/07/16 10:13, fu.wei@linaro.org wrote:
> From: Fu Wei <fu.wei@linaro.org>
>
> This patchset add xen_boot support into grup-mkconfig for
> generating xen boot entrances automatically
>
> Also update the docs/grub.texi for new xen_boot commands.
>
> This patchset has been tested on Foudation model with a bug fix:
> http://lists.gnu.org/archive/html/grub-devel/2016-02/msg00205.html
>
> ChangeLog:
> v6: http://lists.gnu.org/archive/html/grub-devel/2016-07/
> Fix Coding style of util/grub.d/20_linux_xen.in, use soft tab.
>
> v5: http://lists.gnu.org/archive/html/grub-devel/2016-07/msg00008.html
> Update the introduction of xen_module commands in docs/grub.texi,
> according to the suggestion from Julien Grall
>
> v4: http://lists.gnu.org/archive/html/grub-devel/2016-05/
> according to the XSM loading mechanism of Xen(upstreamed),
> update the introduction of xen_module commands in docs/grub.texi
>
> v3: http://lists.gnu.org/archive/html/grub-devel/2016-02/msg00314.html
> reorder the patches
> update the introduction of xen_module commands in docs/grub.texi
>
> v2: http://lists.gnu.org/archive/html/grub-devel/2016-02/msg00282.html
> add "--nounzip" option support in xen_module
> use "feature_xen_boot" instead of "grub_xen_boot"
> update the introduction of xen boot commands in docs/grub.texi
>
> v1 :first upstream patchset:
> http://lists.gnu.org/archive/html/grub-devel/2016-02/msg00264.html
>
> Fu Wei (4):
> i386,xen: Add xen_hypervisor and xen_module aliases for i386
> arm64: add "--nounzip" option support in xen_module command
> * util/grub.d/20_linux_xen.in: Add xen_boot command support
> arm64: update the introduction of xen boot commands in docs/grub.texi
>
> docs/grub.texi | 32 +++++++++-----------------------
> grub-core/loader/arm64/xen_boot.c | 17 +++++++++++++++++
> grub-core/loader/i386/xen.c | 7 +++++++
> grub-core/normal/main.c | 2 +-
> util/grub.d/20_linux_xen.in | 13 ++++++++++---
> 5 files changed, 44 insertions(+), 27 deletions(-)
>
--
Julien Grall
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH v6 0/4] arm64, xen: add xen_boot support into grup-mkconfig
@ 2016-09-01 12:47 ` Julien Grall
0 siblings, 0 replies; 23+ messages in thread
From: Julien Grall @ 2016-09-01 12:47 UTC (permalink / raw)
To: grub-devel, Daniel Kiper
Cc: fu.wei, linaro-uefi, xen-devel, jcm, leif.lindholm, sstabellini
Hi Daniel,
I have heard you became co-maintainer of GRUB. Congratulations for that :).
We have a couple series (this series and [1]) to allow grub booting Xen
on ARM that have been waiting on the GRUB ML for a while to be pushed.
Those patches have been tested and already integrated in some
distributions. It would be nice to get them in GRUB upstream.
Would it be possible for you to take a look?
Regards,
[1] https://lists.gnu.org/archive/html/grub-devel/2016-02/msg00205.html
On 26/07/16 10:13, fu.wei@linaro.org wrote:
> From: Fu Wei <fu.wei@linaro.org>
>
> This patchset add xen_boot support into grup-mkconfig for
> generating xen boot entrances automatically
>
> Also update the docs/grub.texi for new xen_boot commands.
>
> This patchset has been tested on Foudation model with a bug fix:
> http://lists.gnu.org/archive/html/grub-devel/2016-02/msg00205.html
>
> ChangeLog:
> v6: http://lists.gnu.org/archive/html/grub-devel/2016-07/
> Fix Coding style of util/grub.d/20_linux_xen.in, use soft tab.
>
> v5: http://lists.gnu.org/archive/html/grub-devel/2016-07/msg00008.html
> Update the introduction of xen_module commands in docs/grub.texi,
> according to the suggestion from Julien Grall
>
> v4: http://lists.gnu.org/archive/html/grub-devel/2016-05/
> according to the XSM loading mechanism of Xen(upstreamed),
> update the introduction of xen_module commands in docs/grub.texi
>
> v3: http://lists.gnu.org/archive/html/grub-devel/2016-02/msg00314.html
> reorder the patches
> update the introduction of xen_module commands in docs/grub.texi
>
> v2: http://lists.gnu.org/archive/html/grub-devel/2016-02/msg00282.html
> add "--nounzip" option support in xen_module
> use "feature_xen_boot" instead of "grub_xen_boot"
> update the introduction of xen boot commands in docs/grub.texi
>
> v1 :first upstream patchset:
> http://lists.gnu.org/archive/html/grub-devel/2016-02/msg00264.html
>
> Fu Wei (4):
> i386,xen: Add xen_hypervisor and xen_module aliases for i386
> arm64: add "--nounzip" option support in xen_module command
> * util/grub.d/20_linux_xen.in: Add xen_boot command support
> arm64: update the introduction of xen boot commands in docs/grub.texi
>
> docs/grub.texi | 32 +++++++++-----------------------
> grub-core/loader/arm64/xen_boot.c | 17 +++++++++++++++++
> grub-core/loader/i386/xen.c | 7 +++++++
> grub-core/normal/main.c | 2 +-
> util/grub.d/20_linux_xen.in | 13 ++++++++++---
> 5 files changed, 44 insertions(+), 27 deletions(-)
>
--
Julien Grall
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH v6 0/4] arm64, xen: add xen_boot support into grup-mkconfig
2016-09-01 12:47 ` Julien Grall
@ 2016-09-01 21:51 ` Daniel Kiper
-1 siblings, 0 replies; 23+ messages in thread
From: Daniel Kiper @ 2016-09-01 21:51 UTC (permalink / raw)
To: Julien Grall
Cc: grub-devel, xen-devel, jcm, leif.lindholm, sstabellini,
linaro-uefi, fu.wei
Hey Julien,
On Thu, Sep 01, 2016 at 01:47:05PM +0100, Julien Grall wrote:
> Hi Daniel,
>
> I have heard you became co-maintainer of GRUB. Congratulations for that :).
Thanks a lot!
> We have a couple series (this series and [1]) to allow grub booting
> Xen on ARM that have been waiting on the GRUB ML for a while to be
> pushed.
>
> Those patches have been tested and already integrated in some
> distributions. It would be nice to get them in GRUB upstream.
As I can see at least some of them listed here:
https://docs.google.com/document/d/1O6wveo1_WsAyEHMD041xMYvhDTDp3jQYzmqkgBx57HM/
So, this is good sign. Currently, we are agreeing what should be
taken into 2.02 train. We are considering just most important things
and fixes. Hence, I am not able to promise that we include in 2.02
all ARM patches mentioned by you if they are not on the above list
right now. However, if it does not happen I will put them on the
list for release after 2.02 (assuming that they are OK).
> Would it be possible for you to take a look?
Sure thing. However, I am working on some stuff for Xen 4.8 and as
you may know deadline is quite close. So, I will take care of
GRUB2 things in the second half of September. Stay tuned...
Daniel
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH v6 0/4] arm64, xen: add xen_boot support into grup-mkconfig
@ 2016-09-01 21:51 ` Daniel Kiper
0 siblings, 0 replies; 23+ messages in thread
From: Daniel Kiper @ 2016-09-01 21:51 UTC (permalink / raw)
To: Julien Grall
Cc: grub-devel, fu.wei, linaro-uefi, xen-devel, jcm, leif.lindholm,
sstabellini
Hey Julien,
On Thu, Sep 01, 2016 at 01:47:05PM +0100, Julien Grall wrote:
> Hi Daniel,
>
> I have heard you became co-maintainer of GRUB. Congratulations for that :).
Thanks a lot!
> We have a couple series (this series and [1]) to allow grub booting
> Xen on ARM that have been waiting on the GRUB ML for a while to be
> pushed.
>
> Those patches have been tested and already integrated in some
> distributions. It would be nice to get them in GRUB upstream.
As I can see at least some of them listed here:
https://docs.google.com/document/d/1O6wveo1_WsAyEHMD041xMYvhDTDp3jQYzmqkgBx57HM/
So, this is good sign. Currently, we are agreeing what should be
taken into 2.02 train. We are considering just most important things
and fixes. Hence, I am not able to promise that we include in 2.02
all ARM patches mentioned by you if they are not on the above list
right now. However, if it does not happen I will put them on the
list for release after 2.02 (assuming that they are OK).
> Would it be possible for you to take a look?
Sure thing. However, I am working on some stuff for Xen 4.8 and as
you may know deadline is quite close. So, I will take care of
GRUB2 things in the second half of September. Stay tuned...
Daniel
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [Xen-devel] [PATCH v6 0/4] arm64, xen: add xen_boot support into grup-mkconfig
2016-09-01 21:51 ` Daniel Kiper
(?)
@ 2016-09-06 10:56 ` Julien Grall
-1 siblings, 0 replies; 23+ messages in thread
From: Julien Grall @ 2016-09-06 10:56 UTC (permalink / raw)
To: Daniel Kiper
Cc: grub-devel, xen-devel, jcm, leif.lindholm, sstabellini,
linaro-uefi, fu.wei
On 01/09/16 22:51, Daniel Kiper wrote:
> Hey Julien,
Hi Daniel,
> On Thu, Sep 01, 2016 at 01:47:05PM +0100, Julien Grall wrote:
>> We have a couple series (this series and [1]) to allow grub booting
>> Xen on ARM that have been waiting on the GRUB ML for a while to be
>> pushed.
>>
>> Those patches have been tested and already integrated in some
>> distributions. It would be nice to get them in GRUB upstream.
>
> As I can see at least some of them listed here:
> https://docs.google.com/document/d/1O6wveo1_WsAyEHMD041xMYvhDTDp3jQYzmqkgBx57HM/
>
> So, this is good sign. Currently, we are agreeing what should be
> taken into 2.02 train. We are considering just most important things
> and fixes. Hence, I am not able to promise that we include in 2.02
> all ARM patches mentioned by you if they are not on the above list
> right now. However, if it does not happen I will put them on the
> list for release after 2.02 (assuming that they are OK).
So it looks like that this series is in the list. However, the patch [1]
is missing. It is a requirement for this series, without it Xen will
crash when booting with GRUB.
>
>> Would it be possible for you to take a look?
>
> Sure thing. However, I am working on some stuff for Xen 4.8 and as
> you may know deadline is quite close. So, I will take care of
> GRUB2 things in the second half of September. Stay tuned...
It is fine by me.
Thank you,
[1] https://lists.gnu.org/archive/html/grub-devel/2016-02/msg00205.html
--
Julien Grall
^ permalink raw reply [flat|nested] 23+ messages in thread