From: Ian Campbell <ian.campbell@citrix.com>
To: ian.jackson@eu.citrix.com, xen-devel@lists.xen.org
Cc: Ian Campbell <ian.campbell@citrix.com>
Subject: [PATCH OSSTEST 6/8] ts-kernel-build: Include dtbs in dist file
Date: Fri, 2 Oct 2015 10:26:16 +0100 [thread overview]
Message-ID: <1443777978-11119-6-git-send-email-ian.campbell@citrix.com> (raw)
In-Reply-To: <1443777956.11707.61.camel@citrix.com>
These are installed to $(INSTALL_PATH)/dtbs/$(KERNEL_RELEASE) where
$(INSTALL_PATH) defaults to /boot but we override it to our staging
/boot.
Note that ts-host-install will install the OS dtbs directly into
/boot/dtbs without the subdirectory, so this won't clash and could be
considered a fallback hence I don't propose to move those ones.
The install_dtbs target has been available since v3.14, wherease we
only test v3.16 onwards on ARM, hence no arrangements are needed to
conditionalise this installation over and above the per-arch
arrangements made here.
Having now set $(INSTALL_PATH) I think the "install" target could now
take over the installation of System.map, vmlinux and .config into
/boot but I've not checked this with all historical kernel versions
and don't intend to make this change now.
Remove any previous dist dir on install, otherwise the kernel tends to
create dist/boot/dtbs.old with the previous contents on repeated use.
Seems like good hygiene anyway.
Signed-off-by: Ian Campbell <ian.campbell@citrix.com>
---
ts-kernel-build | 17 ++++++++++++-----
1 file changed, 12 insertions(+), 5 deletions(-)
diff --git a/ts-kernel-build b/ts-kernel-build
index e30b1c9..8d906a5 100755
--- a/ts-kernel-build
+++ b/ts-kernel-build
@@ -42,10 +42,10 @@ selectbuildhost(\@ARGV);
builddirsprops();
my $archparms = {
- 'arm64' => { Image => 'Image' },
- 'armhf' => { DefConfig => 'multi_v7', Image => 'zImage' },
- 'i386' => { Image => 'bzImage' },
- 'amd64' => { Image => 'bzImage' }
+ 'arm64' => { Image => 'Image', Dtbs => 1 },
+ 'armhf' => { DefConfig => 'multi_v7', Image => 'zImage', Dtbs => 1 },
+ 'i386' => { Image => 'bzImage' },
+ 'amd64' => { Image => 'bzImage' }
};
my $parms = $archparms->{ $r{arch} };
@@ -384,10 +384,17 @@ sub kinstall () {
$kernfile = "arch/$karch/boot/$parms->{Image}"
if !defined $kernfile;
+ my $dtbs_install = '';
+ $dtbs_install = 'dtbs_install' if $parms->{Dtbs};
+
target_cmd_build($ho, 300, $builddir, <<END);
+ rm -rf dist
mkdir -p dist/boot dist/lib/modules
cd linux
- make $makeflags INSTALL_MOD_PATH=$builddir/dist modules_install
+ make $makeflags \\
+ INSTALL_PATH=$builddir/dist/boot \\
+ INSTALL_MOD_PATH=$builddir/dist modules_install \\
+ $dtbs_install
cp System.map $builddir/dist/boot/
cp $kernfile $builddir/dist/boot/vmlinuz
cp .config $builddir/dist/boot/config
--
2.5.3
next prev parent reply other threads:[~2015-10-02 9:26 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-10-02 9:25 [PATCH OSSTEST 0/8] Use the DTB provided by the kernel we are trying to boot Ian Campbell
2015-10-02 9:26 ` [PATCH OSSTEST 1/8] Osstest: Add %arch_debian2linux (and inverse) Ian Campbell
2015-10-05 16:18 ` Ian Jackson
2015-10-02 9:26 ` [PATCH OSSTEST 2/8] ts-kernel-build: Introduce global $parms Ian Campbell
2015-10-05 16:18 ` Ian Jackson
2015-10-02 9:26 ` [PATCH OSSTEST 3/8] ts-kernel-build: Refactor $archparams::DefConfig selection Ian Campbell
2015-10-05 16:19 ` Ian Jackson
2015-10-02 9:26 ` [PATCH OSSTEST 4/8] ts-kernel-build: Refactor $archparams KernelImage handling Ian Campbell
2015-10-05 16:19 ` Ian Jackson
2015-10-02 9:26 ` [PATCH OSSTEST 5/8] ts-kernel-build: Add arm64 support Ian Campbell
2015-10-05 16:20 ` Ian Jackson
2015-10-05 16:28 ` Ian Campbell
2015-10-06 13:35 ` Ian Jackson
2015-10-02 9:26 ` Ian Campbell [this message]
2015-10-05 16:22 ` [PATCH OSSTEST 6/8] ts-kernel-build: Include dtbs in dist file Ian Jackson
2015-10-05 16:29 ` Ian Campbell
2015-10-06 13:24 ` Ian Campbell
2015-10-06 13:36 ` Ian Jackson
2015-10-06 13:36 ` Ian Jackson
2015-10-02 9:26 ` [PATCH OSSTEST 7/8] Debian: Enable interpolation in uboot_scr_load_dtb here doc Ian Campbell
2015-10-05 16:24 ` Ian Jackson
2015-10-05 16:30 ` Ian Campbell
2015-10-02 9:26 ` [PATCH OSSTEST 8/8] Debian: Use dtbs from kernel dist when booting that kernel Ian Campbell
2015-10-05 16:29 ` Ian Jackson
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=1443777978-11119-6-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).