xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: Ian Campbell <ian.campbell@citrix.com>
To: ian.jackson@eu.citrix.com
Cc: Ian Campbell <ian.campbell@citrix.com>, xen-devel@lists.xen.org
Subject: [PATCH OSSTEST] arm boot fixes and improvements
Date: Fri, 25 Oct 2013 11:40:31 +0100	[thread overview]
Message-ID: <1382697631-7104-1-git-send-email-ian.campbell@citrix.com> (raw)

Correct prototype for setupboot_uboot.

Add some additional logging to the u-boot script creation process as well as
the resulting script, to aid debugging
---
 Osstest/Debian.pm |   10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/Osstest/Debian.pm b/Osstest/Debian.pm
index 517fa61..c675e0d 100644
--- a/Osstest/Debian.pm
+++ b/Osstest/Debian.pm
@@ -120,11 +120,12 @@ sub lvm_lv_name($$) {
     return "/dev/mapper/$vg-$lv";
 }
 
-sub setupboot_uboot ($$$$) {
+sub setupboot_uboot ($$$) {
     my ($ho,$want_kernver,$xenhopt,$xenkopt) = @_;
     my $bl= { };
 
     $bl->{UpdateConfig}= sub {
+	my ( $ho ) = @_;
 
 	my $xen = "xen";
 	my $kern = "vmlinuz-$want_kernver";
@@ -132,6 +133,9 @@ sub setupboot_uboot ($$$$) {
 
 	my $root= lvm_lv_name($ho,"root");
 
+	logm("Xen options: $xenhopt");
+	logm("Linux options: $xenkopt");
+
 	target_cmd_root($ho, <<END);
 if test ! -f /boot/$kern ; then
     exit 1
@@ -160,7 +164,7 @@ setenv xen_addr_r 0x01000000
 ext2load scsi 0 \\\${xen_addr_r} \$xen
 setenv bootargs "$xenhopt"
 echo Loaded \$xen to \\\${xen_addr_r} (\\\${filesize})
-echo command line: $xenhopt
+echo command line: \\\${bootargs}
 
 ext2load scsi 0 \\\${kernel_addr_r} $kern
 fdt mknod /chosen module\@0
@@ -176,6 +180,8 @@ fdt set /chosen/module\@1 compatible "xen,linux-initrd" "xen,multiboot-module"
 fdt set /chosen/module\@1 reg <\\\${ramdisk_addr_r} \\\${filesize}>
 echo Loaded $initrd to \\\${ramdisk_addr_r} (\\\${filesize})
 
+fdt print /chosen
+
 echo Booting \\\${xen_addr_r} - \\\${fdt_addr}
 bootz \\\${xen_addr_r} - \\\${fdt_addr}
 EOF
-- 
1.7.10.4

             reply	other threads:[~2013-10-25 10:40 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-10-25 10:40 Ian Campbell [this message]
2013-10-28 15:47 ` [PATCH OSSTEST] arm boot fixes and improvements Ian Jackson
2013-10-31 17:53   ` Ian Campbell
2013-10-31 18:38     ` Ian Jackson
2013-10-31 18:50       ` Ian Campbell

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=1382697631-7104-1-git-send-email-ian.campbell@citrix.com \
    --to=ian.campbell@citrix.com \
    --cc=ian.jackson@eu.citrix.com \
    --cc=xen-devel@lists.xen.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).