From: Ian Jackson <ian.jackson@eu.citrix.com>
To: xen-devel@lists.xenproject.org
Cc: Ian Jackson <Ian.Jackson@eu.citrix.com>,
Ian Campbell <ian.campbell@citrix.com>
Subject: [OSSTEST PATCH 10/13] rump kernel tests: Specify kernel via runvar
Date: Tue, 15 Jul 2014 16:22:24 +0100 [thread overview]
Message-ID: <1405437747-12753-11-git-send-email-ian.jackson@eu.citrix.com> (raw)
In-Reply-To: <1405437747-12753-1-git-send-email-ian.jackson@eu.citrix.com>
The runvar <gn>_builtimage is set to <distpart>:<distpath>.
Previously we hardcoded `/usr/local/lib/xen/rump-kernel' in
`rumpuserxen'.
The ffs filesystem images, and the demo config template, still come
from a single particular build, which we find from the
`guests_rumpuserxenbuildjob' runvar. This runvar is then doing double
duty as the build job for the ffs filesystem images and example
configs, and the `default' build job for guest kernels.
The subpath /usr/local/lib/xen/rump-kernel is duplicated, because it
appears once (in make-flight, for setting a runvar) to find the demo
guest's kernel, and once (hardcoded in ts-rumpuserxen-demo-setup) for
the filesystem images and config file.
Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>
---
make-flight | 1 +
ts-rumpuserxen-demo-setup | 17 +++++++++++++----
2 files changed, 14 insertions(+), 4 deletions(-)
diff --git a/make-flight b/make-flight
index 8aa5396..cede756 100755
--- a/make-flight
+++ b/make-flight
@@ -123,6 +123,7 @@ do_rumpkernel_tests () {
test-rumpuserxen xl \
$xenarch $dom0arch \
guests_rumpuserxenbuildjob=build-$rumparch-rumpuserxen \
+ rump_builtimage=rumpuserxen:/usr/local/lib/xen/rump-kernel \
all_hostflags=$most_hostflags
}
diff --git a/ts-rumpuserxen-demo-setup b/ts-rumpuserxen-demo-setup
index 68ec7d4..34da7cb 100755
--- a/ts-rumpuserxen-demo-setup
+++ b/ts-rumpuserxen-demo-setup
@@ -35,10 +35,19 @@ our $rkdist;
sub prep () {
$gho = prepareguest($ho, $gn, $gn, 4096, undef, 30);
+ my $cfg_rkdist = target_extract_jobdistpath_subdir
+ ($ho, 'rumpuserxen--base', 'rumpuserxen',
+ $r{guests_rumpuserxenbuildjob});
+ $cfg_rkdist .= '/usr/local/lib/xen/rump-kernel';
+
my $buildjob = guest_var($gho, 'rumpuserxenbuildjob', undef);
+ my $builtimage = guest_var($gho, 'builtimage', undef);
+ $builtimage =~ m/\:/ or die "$builtimage ?";
+ my $builtimage_part = $`;
+ my $builtimage_subpath = $'; #';
$rkdist = target_extract_jobdistpath_subdir
- ($ho, 'rumpuserxen', 'rumpuserxen', $buildjob);
- $rkdist .= '/usr/local/lib/xen/rump-kernel';
+ ($ho, "rumpuserxen-$gn", $builtimage_part, $buildjob);
+ my $imagefile = $rkdist.$builtimage_subpath;
my @images;
@@ -46,7 +55,7 @@ sub prep () {
my $cfgfile = "$gn.cfg";
my $cfgpath = "/etc/xen/$cfgfile";
- target_editfile_root($ho, "$rkdist/domain_config", $cfgfile, $cfgpath,
+ target_editfile_root($ho, "$cfg_rkdist/domain_config", $cfgfile, $cfgpath,
sub {
while (<EI>) {
next if m/^\s*\#/;
@@ -55,7 +64,7 @@ sub prep () {
my $in = $& if m/^\w\S*/;
s# = .* # " = '$gn'" #xe if $in eq 'name';
- s# =\s* ['"](.*)['"] # " = '$rkdist/$1'" #xe if $in eq 'kernel';
+ s# = .* # " = '$imagefile'" #xe if $in eq 'kernel';
s#\b mac=[0-9a-f:]+ # "mac=$gho->{Ether}" #xe if $in eq 'vif';
s#\b 4 \b# 3 #xe if $in eq 'extra';
--
1.7.10.4
next prev parent reply other threads:[~2014-07-15 15:22 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-07-15 15:22 [OSSTEST PATCH 00/13] Build and test rump kernel xenstore-ls Ian Jackson
2014-07-15 15:22 ` [OSSTEST PATCH 01/13] selectguest: log a message Ian Jackson
2014-07-15 15:22 ` [OSSTEST PATCH 02/13] BuildSupport: make selectbuildhost modify @ARGV Ian Jackson
2014-07-15 15:22 ` [OSSTEST PATCH 03/13] sg-run-job: Support toggling `adding' by repeating `+' Ian Jackson
2014-07-16 9:17 ` Ian Campbell
2014-07-15 15:22 ` [OSSTEST PATCH 04/13] building: Delete when cloning, not when preparing $builddir Ian Jackson
2014-07-16 9:19 ` Ian Campbell
2014-07-16 9:58 ` Ian Jackson
2014-07-16 10:06 ` Ian Campbell
2014-07-16 10:18 ` Ian Jackson
2014-07-16 10:20 ` Ian Campbell
2014-07-16 10:29 ` Ian Jackson
2014-07-16 11:25 ` Ian Campbell
2014-07-15 15:22 ` [OSSTEST PATCH 05/13] make-flight: rumpuserxen tests: use same buildjob for all guests Ian Jackson
2014-07-16 9:19 ` Ian Campbell
2014-07-16 10:10 ` Ian Jackson
2014-07-15 15:22 ` [OSSTEST PATCH 06/13] ts-xen-build: Allow specification of make targets Ian Jackson
2014-07-15 15:22 ` [OSSTEST PATCH 07/13] ts-xen-build: Honour cmdprefix_configure, cmdprefix_make Ian Jackson
2014-07-15 15:22 ` [OSSTEST PATCH 08/13] rump kernel tests: Do a xen.git tools build Ian Jackson
2014-07-15 15:22 ` [OSSTEST PATCH 09/13] rump kernel tests: Use guest-specific fs images location Ian Jackson
2014-07-15 15:22 ` Ian Jackson [this message]
2014-07-15 15:22 ` [OSSTEST PATCH 11/13] rump kernel tests: Specify cmdline via runvar Ian Jackson
2014-07-15 15:22 ` [OSSTEST PATCH 12/13] ts-guest-destroy-hard: New script Ian Jackson
2014-07-16 9:24 ` Ian Campbell
2014-07-16 10:10 ` Ian Jackson
2014-07-16 10:12 ` Ian Campbell
2014-07-16 10:20 ` Ian Jackson
2014-07-16 10:21 ` Ian Campbell
2014-07-16 10:30 ` Ian Jackson
2014-07-15 15:22 ` [OSSTEST PATCH 13/13] rump kernel tests: Run xenstore-ls demo Ian Jackson
2014-07-16 9:26 ` Ian Campbell
2014-07-16 10:15 ` Ian Jackson
2014-07-16 10:17 ` Ian Campbell
2014-07-16 9:26 ` [OSSTEST PATCH 00/13] Build and test rump kernel xenstore-ls 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=1405437747-12753-11-git-send-email-ian.jackson@eu.citrix.com \
--to=ian.jackson@eu.citrix.com \
--cc=ian.campbell@citrix.com \
--cc=xen-devel@lists.xenproject.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).