Re: [Xen-devel] blkback shutdown problem ? (Re: [linux-4.9 test] 110513: regressions - FAIL)

2017-06-19 Thread Ian Jackson
Konrad Rzeszutek Wilk writes ("Re: blkback shutdown problem ? (Re: [linux-4.9 test] 110513: regressions - FAIL)"): > On Mon, Jun 19, 2017 at 12:27:33PM +0100, Roger Pau Monné wrote: > > This has already been noticed and fixed by Juergen [0], however AFAIK > > the patches are not yet in Jens/Linus

[Xen-devel] [PATCH for-4.9] xen/test/Makefile: Fix clean target, broken by pattern rule

2017-06-19 Thread Ian Jackson
component-qualified; clean targets (which are handled specially, even distclean); and dist, src-tarball-*, etc. (which are converted to install by an earlier Makefile). Reported-by: Andrew Cooper Signed-off-by: Ian Jackson --- xen/test/Makefile | 7 ++- 1 file changed, 6 insertions(+), 1

Re: [Xen-devel] [linux-4.9 test] 110371: regressions - FAIL

2017-06-19 Thread Ian Jackson
Julien Grall writes ("Re: [Xen-devel] [linux-4.9 test] 110371: regressions - FAIL"): > There are now in Linux 4.9 stable branch. Now the testing is blocked > because of again local migration [1]. The migration test seem less > reliable than the Arndale these days... is there anything we can do?

Re: [Xen-devel] [PATCH] ipxe: update to newer commit

2017-06-19 Thread Ian Jackson
ted-by: Zhongze Liu > > Signed-off-by: Wei Liu > > Ping? I don't think I have any basis to withhold an ack. Please update this in xen.git#staging:Config.mk. Obviously this isn't really suitable for staging-4.9. Acked-by: Ian Jackson Thanks, Ian. ___

Re: [Xen-devel] [PATCH for-4.9 0/4] Makefiles: Provide way to ship livepatch tests

2017-06-19 Thread Ian Jackson
Julien Grall writes ("Re: [Xen-devel] [PATCH for-4.9 0/4] Makefiles: Provide way to ship livepatch tests"): > I don't see the patches in staging. Do you still plan to have those > patches in Xen 4.9? As you'll have seen, they went to staging last week and are now in xen.git#master. I have just

[Xen-devel] blkback shutdown problem ? (Re: [linux-4.9 test] 110513: regressions - FAIL)

2017-06-19 Thread Ian Jackson
osstest service owner writes ("[linux-4.9 test] 110513: regressions - FAIL"): > flight 110513 linux-4.9 real [real] > http://logs.test-lab.xenproject.org/osstest/logs/110513/ > > Regressions :-( > > Tests which did not succeed and are blocking, > including tests which could not be run: > test-am

Re: [Xen-devel] [linux-4.9 test] 110535: regressions - FAIL

2017-06-19 Thread Ian Jackson
osstest service owner writes ("[linux-4.9 test] 110535: regressions - FAIL"): > flight 110535 linux-4.9 real [real] > http://logs.test-lab.xenproject.org/osstest/logs/110535/ > > Regressions :-( > > Tests which did not succeed and are blocking, > including tests which could not be run: > test-am

[Xen-devel] [OSSTEST PATCH 3/3] make-flight: Provide Windows 10 and Windows Server 16 guests with 3.5G of RAM

2017-06-16 Thread Ian Jackson
Paul advises that 4G may be needed, but we have some 8G hosts in various places. If this is not sufficient in practice, we should consider restricting the test to bigger hosts. Signed-off-by: Ian Jackson CC: Paul Durrant CC: Wei Liu CC: Dario Faggioli --- make-flight | 4 ++-- 1 file

[Xen-devel] [OSSTEST PATCH 1/3] ts-windows-install: Honour memsize and disksize guest runvars

2017-06-16 Thread Ian Jackson
Signed-off-by: Ian Jackson CC: Wei Liu CC: Paul Durrant --- ts-windows-install | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/ts-windows-install b/ts-windows-install index 5ee127d..4dc8f2a 100755 --- a/ts-windows-install +++ b/ts-windows-install @@ -27,17 +27,19

[Xen-devel] [OSSTEST PATCH 2/3] make-flight: do_hvm_win_test_one: honour extra args as runvars

2017-06-16 Thread Ian Jackson
No functional change as nothing passes extra args right now Signed-off-by: Ian Jackson --- make-flight | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/make-flight b/make-flight index 11cc01e..75afdb9 100755 --- a/make-flight +++ b/make-flight @@ -320,6 +320,7

Re: [Xen-devel] [OSSTEST PATCH 3/4] dhcp leases: Introduce new client/server leases query mechanism

2017-06-16 Thread Ian Jackson
Ian Jackson writes ("[OSSTEST PATCH 3/4] dhcp leases: Introduce new client/server leases query mechanism"): > * A Python program `ms-leases-omapiproxy' which speaks OMAPI to the >DHCP server, and a simple synchronous query protocol on its >stdin/stdout. OMAPI au

[Xen-devel] [OSSTEST PATCH] ts-windows-install: Provide guest with 32G of disk rather than 10G

2017-06-16 Thread Ian Jackson
Paul Durrant examined a VM after a migration failure and found it short of free space with some essential services not running, and conjectures that this is cauxed by disk shortage. Signed-off-by: Ian Jackson CC: Paul Durrant --- ts-windows-install | 2 +- 1 file changed, 1 insertion(+), 1

[Xen-devel] [OSSTEST PATCH 4/6] ts-syslog-server: New test script

2017-06-16 Thread Ian Jackson
n with the new "|" spawn-ts mode: it will keep running until it is reaped. Currently it is not part of any jobs. Signed-off-by: Ian Jackson --- ts-syslog-server | 85 1 file changed, 85 insertions(+) create mode 100755 ts-syslog-

[Xen-devel] [OSSTEST PATCH 1/6] sg-run-job: Document run-ts and spawn-ts argument syntaxes

2017-06-16 Thread Ian Jackson
Signed-off-by: Ian Jackson --- sg-run-job | 93 ++ 1 file changed, 93 insertions(+) diff --git a/sg-run-job b/sg-run-job index 4b98e2b..fd109e1 100755 --- a/sg-run-job +++ b/sg-run-job @@ -146,6 +146,99 @@ proc testid_matches_globs

[Xen-devel] [OSSTEST PATCH 6/6] Debian: Pass sylog server details to debian-installer

2017-06-16 Thread Ian Jackson
This logs the installer output, even during a failed install. Signed-off-by: Ian Jackson --- Osstest/Debian.pm | 5 + 1 file changed, 5 insertions(+) diff --git a/Osstest/Debian.pm b/Osstest/Debian.pm index 8ba48bf..845027a 100644 --- a/Osstest/Debian.pm +++ b/Osstest/Debian.pm @@ -690,6

[Xen-devel] [OSSTEST PATCH 2/6] sg-run-job: Support scripts which need to be told when to quit

2017-06-16 Thread Ian Jackson
We give them a pipe on stdin, whose writing end we close when we want to reap them. Signed-off-by: Ian Jackson --- sg-run-job | 19 +-- 1 file changed, 17 insertions(+), 2 deletions(-) diff --git a/sg-run-job b/sg-run-job index fd109e1..34462c7 100755 --- a/sg-run-job +++ b/sg

[Xen-devel] [OSSTEST PATCH 5/6] sg-run-job: Spawn ts-syslog-server

2017-06-16 Thread Ian Jackson
Every job now gets a syslog server, which starts up after host allocation. The server's address and port are recorded by ts-syslog-server in a runvar and can be used by subsequent test steps. Signed-off-by: Ian Jackson --- sg-run-job | 4 1 file changed, 4 insertions(+) diff --git

[Xen-devel] [OSSTEST PATCH 3/6] TestSupport: Provide controller_ipaddr

2017-06-16 Thread Ian Jackson
Signed-off-by: Ian Jackson --- Osstest/TestSupport.pm | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/Osstest/TestSupport.pm b/Osstest/TestSupport.pm index ce9ffaa..8f8013e 100644 --- a/Osstest/TestSupport.pm +++ b/Osstest/TestSupport.pm @@ -92,7 +92,7 @@ BEGIN

Re: [Xen-devel] questions on mem_sharing_op*'s and tools/tests/mem-sharing/memshrtool

2017-06-16 Thread Ian Jackson
Zhongze Liu writes ("Re: questions on mem_sharing_op*'s and tools/tests/mem-sharing/memshrtool"): > 2017-06-16 11:50 GMT+08:00 Zhongze Liu : > > Hi there, > > > > I was experimenting with the mem_sharing_op and I found a handy tool: > > tools/tests/mem-sharing/memshrtool > > I set up two bare meta

[Xen-devel] [OSSTEST PATCH 3/4] dhcp leases: Introduce new client/server leases query mechanism

2017-06-15 Thread Ian Jackson
to *, so does not need to know its hostname. So the config has only the port, and the on-DHCP-server settings. Signed-off-by: Ian Jackson --- Osstest.pm| 4 ++ Osstest/DhcpWatch/leasesdaemon.pm | 98 +++ ms-leases-omapiproxy | 43

[Xen-devel] [OSSTEST PATCH 2/4] tcl/daemonlib: tolerate no $c(...Host)

2017-06-15 Thread Ian Jackson
d-off-by: Ian Jackson --- tcl/daemonlib.tcl | 17 + 1 file changed, 13 insertions(+), 4 deletions(-) diff --git a/tcl/daemonlib.tcl b/tcl/daemonlib.tcl index 1e86d5f..b23caea 100644 --- a/tcl/daemonlib.tcl +++ b/tcl/daemonlib.tcl @@ -195,7 +195,7 @@ proc newconn {chan addr port} {

[Xen-devel] [OSSTEST PATCH 4/4] production-config: Use the new leases server for all hosts by default

2017-06-15 Thread Ian Jackson
Signed-off-by: Ian Jackson --- production-config | 2 ++ 1 file changed, 2 insertions(+) diff --git a/production-config b/production-config index 96831c7..63d6641 100644 --- a/production-config +++ b/production-config @@ -94,6 +94,8 @@ TftpNetbootGroup osstest TftpDiVersion_wheezy 2016-06-08

[Xen-devel] [OSSTEST PATCH 1/4] mg-repro-setup: Use new syntax for cs-adjust-flight

2017-06-15 Thread Ian Jackson
in-tree user of that feature. Signed-off-by: Ian Jackson --- mg-repro-setup | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/mg-repro-setup b/mg-repro-setup index 4f2fe54..9663497 100755 --- a/mg-repro-setup +++ b/mg-repro-setup @@ -199,7 +199,7 @@ if $skipcapture; then adjrunv

Re: [Xen-devel] [RFC PATCH 0/4] Per-domain locking in xl

2017-06-15 Thread Ian Jackson
Wei Liu writes ("[RFC PATCH 0/4] Per-domain locking in xl"): > It has always been the case that different xl processes can > manipulate the same domain at the same time. This could be > problematic. > > This series attempts to provide facility for xl to have a per-domain > lock. This lock should b

Re: [Xen-devel] [PATCH v2 1/1] xl.cfg man page cleanup and fixes

2017-06-13 Thread Ian Jackson
x27;s I could see > - general styling and makeup fixes to make everything look more consistent > > Signed-off-by: Armando Vega > Reviewed-by: Dario Faggioli Thanks to both of you. Acked-by: Ian Jackson (without doing a thorough review) ___

Re: [Xen-devel] [linux-4.9 test] 110371: regressions - FAIL

2017-06-13 Thread Ian Jackson
Dario Faggioli writes ("Re: [Xen-devel] [linux-4.9 test] 110371: regressions - FAIL"): > So, this appears to be something related to linux-4.9 and that > particular board, I guess? Yes, indeed. This has been broken for some time. > (If this was known and being looked up already, sorry for the n

Re: [Xen-devel] [PATCH v2] tools: fix several "format-truncation" warnings with GCC 7

2017-06-13 Thread Ian Jackson
Zhongze Liu writes ("[PATCH v2] tools: fix several "format-truncation" warnings with GCC 7"): > GCC 7.1.1 complains that several buffers passed to snprintf() in > xenpmd and tools/ocmal/xc are too small to hold the largest possible > resulting string. > > enlarge the size of these buffers to fix

Re: [Xen-devel] tags in backport commits

2017-06-13 Thread Ian Jackson
Stefano Stabellini writes ("tags in backport commits"): > Specifically, I would like to see a tag that specifies the id of the > original commit, like "master commit" that Jan is using today. I would > prefer if the tag didn't have any spaces, thus, I suggest > master-commit-id or upstream-commit-i

[Xen-devel] FTR: osstest load spikes, countermeasures

2017-06-12 Thread Ian Jackson
I wrote on IRC: 11:07 andyhhp: Looking at your xtf failure in 110095. 11:08 Diziet: yes - it is odd 11:08 the test itself didn't fail 11:10 andyhhp: Jun 8 07:26:08 osstest uptime: 07:26:08 up 168 days, 19:53, 9 users, load average: 19.87, 10.37, 9.17 11:10 So your failure coi

[Xen-devel] [OSSTEST PATCH 2/2] fixup! ts-xen-build: Build livepatches test-cases

2017-06-12 Thread Ian Jackson
Use substep_eval, so we cope with xen.git revisions which do not support `make dist-tests'. Signed-off-by: Ian Jackson --- ts-xen-build | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/ts-xen-build b/ts-xen-build index eaf44b1..097ac0a 100755 --- a/ts-xen-build +++

Re: [Xen-devel] [OSSTEST PATCH] Drop build-*-oldkern [and 1 more messages]

2017-06-12 Thread Ian Jackson
Wei Liu writes ("[OSSTEST PATCH] Drop build-*-oldkern"): > That is for testing the in-xen.git kernel build machinery which we > surely don't care about anymore. Thanks. I will push this in a moment. Acked-by: Ian Jackson Wei Liu writes ("Re: [OSSTEST PATCH] Drop bu

[Xen-devel] [OSSTEST PATCH v4-v4.1 delta 0/2] livepatch test support

2017-06-12 Thread Ian Jackson
I found that this series broke (of course) when run on xen.git trees without the dist-test target. These two patches fix this. The second patch will be folded into `ts-xen-build: Build livepatches test-cases' and I am then going to try to push it (again). Ian. __

[Xen-devel] [OSSTEST PATCH 1/2] substep handling improvements

2017-06-12 Thread Ian Jackson
$@. Signed-off-by: Ian Jackson --- Osstest/TestSupport.pm | 33 +++-- sg-run-job | 3 ++- 2 files changed, 33 insertions(+), 3 deletions(-) diff --git a/Osstest/TestSupport.pm b/Osstest/TestSupport.pm index fc1aa7d..ce9ffaa 100644 --- a/Osstest/TestSupport.pm

Re: [Xen-devel] [PATCH] tools: fix several "format-truncation" errors with GCC 7

2017-06-12 Thread Ian Jackson
Zhongze Liu writes ("[PATCH] tools: fix several "format-truncation" errors with GCC 7"): > replace several snprintf with asprintf in xenpmd and tools/ocmal/xc > to fix the "format-truncation" errors caused by incorrect size of buffers. Thanks for paying attention to the quality of our code, but:

Re: [Xen-devel] [xen-4.7-testing bisection] complete build-amd64

2017-06-12 Thread Ian Jackson
Jan Beulich writes ("Re: [Xen-devel] [xen-4.7-testing bisection] complete build-amd64"): > On 12.06.17 at 12:45, wrote: > > Jan Beulich writes ("Re: [Xen-devel] [xen-4.7-testing bisection] complete > > build-amd64"): ... > > Normally, the debug build of some software would (i) generate better >

Re: [Xen-devel] [xen-4.7-testing bisection] complete build-amd64

2017-06-12 Thread Ian Jackson
Jan Beulich writes ("Re: [Xen-devel] [xen-4.7-testing bisection] complete build-amd64"): > On 11.06.17 at 17:55, wrote: > > commit 164c34dd23bc3ea8d5285752d9270627a93c91f5 > > Author: Jan Beulich > > Date: Fri Jun 9 13:51:34 2017 +0200 > > > > hvmloader: avoid tests when they wo

Re: [Xen-devel] [OSSTEST PATCH v4 00/11] livepatch test support

2017-06-09 Thread Ian Jackson
Konrad Rzeszutek Wilk writes ("Re: [OSSTEST PATCH v4 00/11] livepatch test support"): > Done. I am satisfied with them. Great, thanks. I have pushed this to osstest pretest. I think it should go through without regressions; the new tests will fail, of course, on Xen branches that don't have the

Re: [Xen-devel] [PATCH for-4.9 0/4] Makefiles: Provide way to ship livepatch tests

2017-06-09 Thread Ian Jackson
Julien Grall writes ("Re: [PATCH for-4.9 0/4] Makefiles: Provide way to ship livepatch tests"): > Release-acked-by: Julien Grall Thanks. I have pushed this to staging and will wait for an osstest test report to check I didn't break the build, before applying the same commits to staging-4.9. Ia

Re: [Xen-devel] [OSSTEST PATCH v4 00/11] livepatch test support

2017-06-09 Thread Ian Jackson
Konrad Rzeszutek Wilk writes ("Re: [OSSTEST PATCH v4 00/11] livepatch test support"): > On Wed, Jun 07, 2017 at 06:48:51PM +0100, Ian Jackson wrote: > > - 06 ts-livepatch: Initial test-cases. > > - 07 sg-run-job: Add the test-livepatch. > > m 08 make-f

Re: [Xen-devel] [OSSTEST PATCH v4 00/11] livepatch test support

2017-06-07 Thread Ian Jackson
Ian Jackson writes ("[OSSTEST PATCH v4 00/11] livepatch test support"): > Thanks to Konrad, who passed me his git branch under the table. > I have tidied up some loose ends, and tested it with my xen.git > Makefile patches. Oh, I should probably share this test report. (So

[Xen-devel] [OSSTEST PATCH 02/11] TestSupport: target_cmd_*: Add some doc comments

2017-06-07 Thread Ian Jackson
Signed-off-by: Konrad Rzeszutek Wilk --- v4: Split off from the patch introducing target_cmd_output_root_status. Rewrote all the comments to be true. --- Osstest/TestSupport.pm | 11 +++ 1 file changed, 11 insertions(+) diff --git a/Osstest/TestSupport.pm b/Osstest/TestSupport.pm ind

[Xen-devel] [OSSTEST PATCH 04/11] mfi-common: Add an enable_livepatch runvar to the Xen build jobs

2017-06-07 Thread Ian Jackson
ff-by: Konrad Rzeszutek Wilk Signed-off-by: Ian Jackson --- v3: New patch v4: Regularise branch_wants_livepatch exit status (shell functions return 0 for true and non-0 for false). Use the `livepatch_runvars' pattern, so that we do not add enable_livepatch=false to any jobs.

[Xen-devel] [OSSTEST PATCH 07/11] sg-run-job: Add the test-livepatch.

2017-06-07 Thread Ian Jackson
From: Konrad Rzeszutek Wilk Signed-off-by: Konrad Rzeszutek Wilk Signed-off-by: Ian Jackson --- v4: Tiny style change. --- sg-run-job | 6 ++ 1 file changed, 6 insertions(+) diff --git a/sg-run-job b/sg-run-job index ceb7980..35bc785 100755 --- a/sg-run-job +++ b/sg-run-job @@ -441,6

[Xen-devel] [OSSTEST PATCH 10/11] cs-adjust-flight: Rework runvar-build-set new value handling

2017-06-07 Thread Ian Jackson
ich is not desirable, nor even possible with new:) Instead, we adopt the convention that a trailing . completes the value with the old job name. Values without a . are taken literally as a job name, resulting in intra-flight references to that job. Signed-off-by: Ian Jackson --- v4: New patch, w

[Xen-devel] [OSSTEST PATCH 11/11] sg-check-tested: Provide --flight option

2017-06-07 Thread Ian Jackson
This is mostly useful in ad-hoc situations. For example, it can be usefully used with ./mg-adjust-flight-makexrefs. Signed-off-by: Ian Jackson --- v4: New patch, which helps with issues I tripped over while trying to ad-hoc test the livepatch series. --- sg-check-tested | 4 1 file

[Xen-devel] [OSSTEST PATCH 05/11] ts-xen-build: Build livepatches test-cases

2017-06-07 Thread Ian Jackson
on a corresponding change to xen.git. Finally, split the livepatch tests into their own stashed deliverable from this job. Signed-off-by: Konrad Rzeszutek Wilk Signed-off-by: Ian Jackson --- v1: New posting v2: Put the livepatch test-cases in xentlpdist.tar.gz file v3: Use enable_livepatch to gate the

[Xen-devel] [OSSTEST PATCH 03/11] TestSupport: target_cmd_output_root_status: New sub

2017-06-07 Thread Ian Jackson
From: Konrad Rzeszutek Wilk Similar to target_cmd_root_status except it outputs both output _and_ status. Signed-off-by: Konrad Rzeszutek Wilk Signed-off-by: Ian Jackson --- v2: Initial posting v3: Sprinkle some comments. v4: Move doc comments out of this patch (and rewrite them). Edit

[Xen-devel] [OSSTEST PATCH 08/11] make-flight: Add livepatch build/test target in the matrix.

2017-06-07 Thread Ian Jackson
or the Xen build configuration in those flights, and try building the livepatch test cases (since those things are cheap). Signed-off-by: Konrad Rzeszutek Wilk Signed-off-by: Ian Jackson --- v4: Added flight changes info to the commit message. --- make-flight | 12 1 file changed, 12

[Xen-devel] [OSSTEST PATCH 06/11] ts-livepatch: Initial test-cases.

2017-06-07 Thread Ian Jackson
From: Konrad Rzeszutek Wilk There are 37 of the test-cases in there. Before we run any of them we verify that the payload files are present in /usr/lib/debug. If so we go through all of the test-cases. Signed-off-by: Konrad Rzeszutek Wilk Signed-off-by: Ian Jackson --- v1: Initial submission

[Xen-devel] [OSSTEST PATCH 09/11] PDU/lab: Similar to xenuse

2017-06-07 Thread Ian Jackson
, 1200, 2400, 4800, 9600, 19200, 38400] -v for verbose Signed-off-by: Konrad Rzeszutek Wilk Signed-off-by: Ian Jackson --- v4: Change config setting names from XenUse* to OracleLab* Signed-off-by: Ian Jackson --- Osstest/PDU/lab.pm | 56

[Xen-devel] [OSSTEST PATCH v4 00/11] livepatch test support

2017-06-07 Thread Ian Jackson
Thanks to Konrad, who passed me his git branch under the table. I have tidied up some loose ends, and tested it with my xen.git Makefile patches. Konrad, would you please take a look at this to make sure you don't think I've broken anything and that my changes look sane ? In particlar the patches

[Xen-devel] [OSSTEST PATCH 01/11] TestSupport: target_cmd_root_status: New sub which returns return code.

2017-06-07 Thread Ian Jackson
zutek Wilk Acked-by: Ian Jackson --- v1: New submission v2: Change to target_cmd_root_status Continue doing the 'die' in tcmdex, but only if $badstatusok is zero. Plumb the $badstatusok through to tcmd. v3: Removed the extra space in tcmdex Added Ian's Ack v4: Ed

Re: [Xen-devel] [OSSTEST PATCH] standalone-reset: Check for SSH keys.

2017-06-07 Thread Ian Jackson
Hi. I found this patch in the git branch you passed me. I'm not sure if you intended this patch for upstream, but it has your s-o-b so I guess so. For reasons that will become clear I have chosen to git-send-email it to myself, so that I may reply. Ian Jackson writes ("[OSS

Re: [Xen-devel] [PATCH for-4.9 0/4] Makefiles: Provide way to ship livepatch tests [and 1 more messages]

2017-06-07 Thread Ian Jackson
Wei Liu writes ("Re: [PATCH for-4.9 0/4] Makefiles: Provide way to ship livepatch tests"): > Acked-by: Wei Liu Thanks to everyone for the review and acks. > +1 for this to go into 4.9 and be backported to 4.8. Thanks. Jan Beulich writes ("Re: [PATCH 3/4] xen/test/livepatch: Add xen_nop.livepa

[Xen-devel] [PATCH 1/4] xen/test/livepatch/Makefile: Install in DESTDIR/usr/lib/debug/xen-livepatch

2017-06-07 Thread Ian Jackson
Dumping these patch files in /usr/lib/debug/xen-*.livepatch is a bit ugly. Also, refactor the Makefile to have a LIVEPATCHES variable, to reduce repetition. CC: Konrad Rzeszutek Wilk Signed-off-by: Ian Jackson --- xen/test/livepatch/Makefile | 19 +++ 1 file changed, 11

[Xen-devel] [PATCH 3/4] xen/test/livepatch: Add xen_nop.livepatch to .gitignore

2017-06-07 Thread Ian Jackson
CC: Konrad Rzeszutek Wilk Signed-off-by: Ian Jackson --- .gitignore | 1 + 1 file changed, 1 insertion(+) diff --git a/.gitignore b/.gitignore index 74747cb..5cae740 100644 --- a/.gitignore +++ b/.gitignore @@ -300,6 +300,7 @@ xen/test/livepatch/config.h xen/test/livepatch

[Xen-devel] [PATCH 2/4] xen/test/livepatch: Regularise Makefiles

2017-06-07 Thread Ian Jackson
ed any more and we can't build livepatches for it either. After this, the xen/test subdirectory is somewhere were make can be invoked in the way which is conventional for xen.git/xen/ subdirs. None of this is yet invoked from the top-level Makefile. CC: Konrad Rzeszutek Wilk Signed-off-by: I

[Xen-devel] [PATCH 4/4] Makefile: Provide way to ship livepatch test files

2017-06-07 Thread Ian Jackson
uild syste. CC: Konrad Rzeszutek Wilk Signed-off-by: Ian Jackson --- Makefile | 11 +++ 1 file changed, 11 insertions(+) diff --git a/Makefile b/Makefile index 2ff33df..0b5f57d 100644 --- a/Makefile +++ b/Makefile @@ -144,6 +144,17 @@ tools/firmware/ovmf-dir-force-update: install-docs:

[Xen-devel] [PATCH for-4.9 0/4] Makefiles: Provide way to ship livepatch tests

2017-06-07 Thread Ian Jackson
I am trying to get the livepatches tested in osstest. As discussed, I would like the xen.git Makefiles to be able to ship the livepatch test files, so that osstest does not need to have too much special knowledge of the xen.git build system. There are three preliminary patches which tidy up the x

Re: [Xen-devel] [PATCH] tools: bump some library version numbers to 4.10

2017-06-07 Thread Ian Jackson
Wei Liu writes ("[PATCH] tools: bump some library version numbers to 4.10"): > Signed-off-by: Wei Liu Acked-by: Ian Jackson ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

Re: [Xen-devel] [xen-unstable test] 110009: regressions - FAIL

2017-06-06 Thread Ian Jackson
Jan Beulich writes ("Re: [Xen-devel] [xen-unstable test] 110009: regressions - FAIL"): > So finally we have some output from the debugging code added by > 933f966bcd ("x86/mm: add temporary debugging code to > get_page_from_gfn_p2m()"), i.e. the migration heisenbug we hope > to hunt down: > > (XE

Re: [Xen-devel] Nested virt broken in Linux 4.9 (was Re: [OSSTEST PATCH 2/3] ap-common: Switch to Linux 4.9 by default [and 1 more messages])

2017-06-06 Thread Ian Jackson
Ian Jackson writes ("Re: Nested virt broken in Linux 4.9 (was Re: [OSSTEST PATCH 2/3] ap-common: Switch to Linux 4.9 by default [and 1 more messages])"): > But osstest does not want to update its linux-4.9 branch to include > those x86 nested fixes, because doing so would

Re: [Xen-devel] [PATCH 0/1] xl.cfg man page cleanup and fixes

2017-06-06 Thread Ian Jackson
Armando Vega writes ("[PATCH 0/1] xl.cfg man page cleanup and fixes"): > so I've made a new round of cleaning and fixing. There was quite some work to > be done with this one. And there are a few issues that are left still, but > at least not with the general correctness and style of the manual. Mo

Re: [Xen-devel] [PATCH for 4.9] vif-common.sh: Have iptables wait for the xtables lock

2017-06-05 Thread Ian Jackson
xit out rather than waiting for it. > This means that when starting a large number of guests in parallel, > many will fail out with messages like this: What a mess, eh ? Acked-by: Ian Jackson ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v2 1/1] xl man page cleanup and fixes

2017-05-31 Thread Ian Jackson
Armando Vega writes ("Re: [PATCH v2 1/1] xl man page cleanup and fixes"): > It could be from how I broke the lines to 80 chars, it kept the spaces > between the words at the end, could that be the problem? I can watch out > for that in the future. Yes, that will be it. However, in general, it is

Re: [Xen-devel] [PATCH v2 0/1] xl man page cleanup and fixes proposal

2017-05-31 Thread Ian Jackson
Armando Vega writes ("[PATCH v2 0/1] xl man page cleanup and fixes proposal"): > thanks for the ack Ian, I'm glad you found it helpful! > I amended the commit message to include my sign-off and Ian's ack. > I'll send a PR on github once I get your okay. Sorry, I confused you by talking about pull

Re: [Xen-devel] Nested virt broken in Linux 4.9 (was Re: [OSSTEST PATCH 2/3] ap-common: Switch to Linux 4.9 by default [and 1 more messages])

2017-05-30 Thread Ian Jackson
Julien Grall writes ("Re: Nested virt broken in Linux 4.9 (was Re: [OSSTEST PATCH 2/3] ap-common: Switch to Linux 4.9 by default [and 1 more messages])"): > On 30/05/17 17:13, Ian Jackson wrote: ... > > I see. Can you do that please ? It's blocking moving our testing to

Re: [Xen-devel] Nested virt broken in Linux 4.9 (was Re: [OSSTEST PATCH 2/3] ap-common: Switch to Linux 4.9 by default [and 1 more messages])

2017-05-30 Thread Ian Jackson
Julien Grall writes ("Re: Nested virt broken in Linux 4.9 (was Re: [OSSTEST PATCH 2/3] ap-common: Switch to Linux 4.9 by default [and 1 more messages])"): > There are two missing patches in Linux 4.9 to be able to boot on the > Arndale: > - c827283586a4 "ARM: 8636/1: Cleanup sanity_check_me

Re: [Xen-devel] Nested virt broken in Linux 4.9 (was Re: [OSSTEST PATCH 2/3] ap-common: Switch to Linux 4.9 by default [and 1 more messages])

2017-05-30 Thread Ian Jackson
Boris Ostrovsky writes ("Re: Nested virt broken in Linux 4.9 (was Re: [OSSTEST PATCH 2/3] ap-common: Switch to Linux 4.9 by default [and 1 more messages])"): > On 05/30/2017 10:28 AM, Ian Jackson wrote: > > osstest service owner writes ("[osstest test] 109837: regressi

[Xen-devel] Nested virt broken in Linux 4.9 (was Re: [OSSTEST PATCH 2/3] ap-common: Switch to Linux 4.9 by default [and 1 more messages])

2017-05-30 Thread Ian Jackson
Ian Jackson writes ("[OSSTEST PATCH 2/3] ap-common: Switch to Linux 4.9 by default"): > I ran a special report[1] to see what to expect and: ... It seems I ran or read this wrong. In fact, my change is stuck in the osstest self push gate because: osstest service owner writes (&

Re: [Xen-devel] [PATCH 0/1] xl man page cleanup and fixes proposal

2017-05-30 Thread Ian Jackson
Armando Vega writes ("[PATCH 0/1] xl man page cleanup and fixes proposal"): > If you accept this patch I would gladly donate my time to fixing the > rest of the man pages, as I did notice the same problems in some of > them. I'd love more patches like that one. > I have a personal fork of this re

Re: [Xen-devel] [PATCH 1/1] xl man page cleanup and fixes

2017-05-30 Thread Ian Jackson
the patch and it was good. Good enough, indeed, that I am inclined to Acked-by: Ian Jackson without reading the second half :-). We have a standing RM exception for docs patches so this can go in to both branches when the commit moratorium (in pla

[Xen-devel] [for-4.9] Re: HVM guest performance regression

2017-05-26 Thread Ian Jackson
Juergen Gross writes ("HVM guest performance regression"): > Looking for the reason of a performance regression of HVM guests under > Xen 4.7 against 4.5 I found the reason to be commit > c26f92b8fce3c9df17f7ef035b54d97cbe931c7a ("libxl: remove freemem_slack") > in Xen 4.6. > > The problem occurre

[Xen-devel] [PATCH] tools/firmware: pass EXTRAVERSION to seabios build

2017-05-26 Thread Ian Jackson
Olaf Hering writes ("[PATCH] tools/firmware: pass EXTRAVERSION to seabios build"): > Having a fixed version string in the seabios binary is cumbersome: > - it is required to write something to '.version' in the toplevel >directory of seabios-dir-remove. > - it is required to pass EXTRAVERSIO

[Xen-devel] [OSSTEST PATCH] arm64, armhf: Use dtuart for console by default

2017-05-26 Thread Ian Jackson
ulien Grall Signed-off-by: Ian Jackson --- ts-xen-install | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/ts-xen-install b/ts-xen-install index 9eb2131..3ab6090 100755 --- a/ts-xen-install +++ b/ts-xen-install @@ -145,7 +145,9 @@ sub adjustconfig () { sub setupboot () {

Re: [Xen-devel] [PATCH for-4.9 2/2] Makefile: Regularise subdir targets and their dependencies

2017-05-25 Thread Ian Jackson
given that this is a comment-only patch I guess Julien's exception for docs applies, so I will probably commit it tomorrow unless someone objects. Ian. From 352f4a1f9797d1eee94121565fb4e3d8fc698371 Mon Sep 17 00:00:00 2001 From: Ian Jackson Date: Thu, 25 May 2017 16:42:12 +0100 Subject: [P

Re: [Xen-devel] Xen 4.9 rc6 [and 1 more messages]

2017-05-25 Thread Ian Jackson
Thanks for the test reports and the release-ack. Ideally I would like a review but I may not get one. Julien, or Michael, do you feel qualified to review this ? Looking at the REST MAINTAINERS: I know Andrew, Jan and Konrad are extremely busy right now so I don't want to ask them to stare at th

[Xen-devel] [OSSTEST PATCH 2/2] ts-hosts-allocate-Executive: Make build failures not be host-sticky

2017-05-25 Thread Ian Jackson
defaulting $prevfail_bonus to 0 in that case, before the week-long later default is applied. This means that a build job runvar would override the built-in default whether that's 7 days or 0. Reported-by: Roger Pau Monné Signed-off-by: Ian Jackson --- ts-hosts-allocate-Executive | 1 + 1 file chang

[Xen-devel] [OSSTEST PATCH 1/2] ts-hosts-allocate-Executive: Move $prevfail_bonus initial setting

2017-05-25 Thread Ian Jackson
No functional change. This is going to make the next patch easy. Reported-by: Roger Pau Monné Signed-off-by: Ian Jackson --- ts-hosts-allocate-Executive | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/ts-hosts-allocate-Executive b/ts-hosts-allocate-Executive index

[Xen-devel] [PATCH for-4.9 2/2] Makefile: Regularise subdir targets and their dependencies

2017-05-24 Thread Ian Jackson
running -C. suffix=-tools configure install-tools ok. running -C. suffix=-stubdom configure install-stubdom ok. running -C. suffix=-docs configure install-docs ok. running -Cxen suffix= install ok. running -Cdocs suffix= configure install ok. running -Ctools suffix= configure inst

[Xen-devel] [PATCH for-4.9 1/2] tools/include/Makefile: Support `build' target

2017-05-24 Thread Ian Jackson
This is the only one of the Makefiles invoked with -C from the toplevel which lacks this target. CC: Julien Grall CC: M A Young CC: Andrew Cooper CC: Wei Liu Signed-off-by: Ian Jackson --- tools/include/Makefile | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tools

Re: [Xen-devel] Xen 4.9 rc6

2017-05-24 Thread Ian Jackson
I'm about to post a pair of patches, against staging, which I think fix this problem. Ian. ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH for-4.9] tools/build: Fix installation of public headers

2017-05-24 Thread Ian Jackson
Wei Liu writes ("Re: [PATCH for-4.9] tools/build: Fix installation of public headers"): > Reviewed-by: Wei Liu FAOD I think this patch is broken and am developing a replacement. Ian. ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xe

Re: [Xen-devel] [PATCH 5/7] osstest: introduce a FreeBSD build script

2017-05-24 Thread Ian Jackson
Roger Pau Monne writes ("Re: [PATCH 5/7] osstest: introduce a FreeBSD build script"): > On Tue, May 23, 2017 at 06:58:59PM +0100, Ian Jackson wrote: > > Suppose we have ts-freebsd-build set > > path_freebsddist=$stash/build/freebsd/ > > and have it put the f

Re: [Xen-devel] [PATCH 4/7] osstest: add a FreeBSD host install recipe

2017-05-24 Thread Ian Jackson
Roger Pau Monne writes ("Re: [PATCH 4/7] osstest: add a FreeBSD host install recipe"): > OK, so then I will just drop FreeBSDBase and just use > $ho->{Tftp}{TmpDir}/freebsd-images. I don't think there's a lot of > value in have it in a standard folder, since those are just random > builds. I guess

[Xen-devel] [PATCH for-4.9] docs: replace xm with xl in xen-tscmode [and 1 more messages]

2017-05-24 Thread Ian Jackson
Olaf Hering writes ("[PATCH] docs: replace xm with xl in xen-tscmode"): > Signed-off-by: Olaf Hering Olaf Hering writes ("[PATCH] docs: correct paragraph indention in xen-tscmode"): > Signed-off-by: Olaf Hering Both: Acked-by: Ian Jackson I think these good

[Xen-devel] [OSSTEST PATCH] ts-examine-logs-save: Do not do readdir $! check

2017-05-23 Thread Ian Jackson
version of perl. Signed-off-by: Ian Jackson --- ts-examine-logs-save | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ts-examine-logs-save b/ts-examine-logs-save index 014522a..b9c9941 100755 --- a/ts-examine-logs-save +++ b/ts-examine-logs-save @@ -97,7 +97,7 @@ sub save_logs

Re: [Xen-devel] [PATCH 5/7] osstest: introduce a FreeBSD build script

2017-05-23 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH 5/7] osstest: introduce a FreeBSD build script"): > -my $path= "path_${part}dist"; > -logm("checking $k $path"); > -get_stashed($path, $r{$k}); > +if ("$part" eq "freebsd") { > +foreach (qw(base kernel manifest image)) { > +my $pa

Re: [Xen-devel] [PATCH 4/7] osstest: add a FreeBSD host install recipe

2017-05-23 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH 4/7] osstest: add a FreeBSD host install recipe"): > The installation is performed using the bsdinstall tool, which is > part of the FreeBSD base system. The installer image is setup with > the osstest ssh keys and sshd enabled by default, which allows the > test ha

Re: [Xen-devel] [OSSTEST PATCH 08/24] cs-hosts-list: Support --suites

2017-05-23 Thread Ian Jackson
Roger Pau Monné writes ("Re: [Xen-devel] [OSSTEST PATCH 08/24] cs-hosts-list: Support --suites"): > I should add a --freebsd=... option. Yes. Although currently my host examination uses only Linux and Xen. Do you think there's any value in doing it with FreeBSD as well ? Ian.

Re: [Xen-devel] Xen 4.9 rc6

2017-05-23 Thread Ian Jackson
Julien Grall writes ("Re: [Xen-devel] Xen 4.9 rc6"): > On 23/05/17 10:24, M A Young wrote: > > I had build problems with these two patches added (and before I added > > them). It looks like at the stage in my build process where > > make -j2 prefix=/usr tools > > gets run it is running > > make -C

Re: [Xen-devel] [PATCH 4/7] osstest: add a FreeBSD host install recipe

2017-05-23 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH 4/7] osstest: add a FreeBSD host install recipe"): > The installation is performed using the bsdinstall tool, which is > part of the FreeBSD base system. The installer image is setup with > the osstest ssh keys and sshd enabled by default, which allows the > test ha

Re: [Xen-devel] [PATCH 3/7] osstest: fix regular expression used to match buildjob in ts-build-check

2017-05-23 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH 3/7] osstest: fix regular expression used to match buildjob in ts-build-check"): > Current regular expression used to match the buildjob works correctly when the > buildjob runvar has the buildjob format, but not when the format is > _buildjob (the first match group

Re: [Xen-devel] [PATCH 2/7] osstest: move known_hosts generation to TestSupport

2017-05-23 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH 2/7] osstest: move known_hosts generation to TestSupport"): > This is equivalent to the already existing authorized_keys function, and > generates the contents of the known_hosts file that should be installed on > targets. Ac

Re: [Xen-devel] [PATCH 1/7] osstest: make built_stash_file store a path_ runvar for each file

2017-05-23 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH 1/7] osstest: make built_stash_file store a path_ runvar for each file"): > And introduce built_stash_debugfile in order the keep the previous behavior of > built_stash_file. Acked-by: Ian Jackson ___ X

Re: [Xen-devel] [OSSTEST PATCH 06/24] cs-hosts-list: New utility

2017-05-23 Thread Ian Jackson
Roger Pau Monné writes ("Re: [Xen-devel] [OSSTEST PATCH 06/24] cs-hosts-list: New utility"): > On Thu, May 18, 2017 at 12:01:16PM +0100, Ian Jackson wrote: > > +foreach my $flag (grep { length} split /\,/, $flags) { > > +$qtxt.= < > + AND EX

Re: [Xen-devel] [OSSTEST PATCH 05/24] tcmd: Work around ssh bug with `ssh host ""'

2017-05-23 Thread Ian Jackson
Roger Pau Monné writes ("Re: [Xen-devel] [OSSTEST PATCH 05/24] tcmd: Work around ssh bug with `ssh host ""'"): > On Thu, May 18, 2017 at 12:01:15PM +0100, Ian Jackson wrote: > > This runs an interactive shell session on the host, rathern than > > running

Re: [Xen-devel] [PATCH] XenBus: Don't wait for producer to fill the ring if the ring

2017-05-23 Thread Ian Jackson
Jan Beulich writes ("Re: [Xen-devel] [PATCH] XenBus: Don't wait for producer to fill the ring if the ring"): > On 23.05.17 at 14:42, wrote: > > Thanks, and sorry to tetch. [ much deleted because I really don't see much point me me arguing this further ] > While I can vaguely guess what you me

Re: [Xen-devel] [PATCH] XenBus: Don't wait for producer to fill the ring if the ring

2017-05-23 Thread Ian Jackson
Jan Beulich writes ("Re: [Xen-devel] [PATCH] XenBus: Don't wait for producer to fill the ring if the ring"): > On 17.05.17 at 16:57, wrote: > > --- a/tools/firmware/hvmloader/xenbus.c > > +++ b/tools/firmware/hvmloader/xenbus.c > > @@ -141,7 +141,18 @@ static void ring_read(char *data, uint32_t l

[Xen-devel] [OSSTEST PATCH 2/3] ap-common: Switch to Linux 4.9 by default

2017-05-22 Thread Ian Jackson
Monné Signed-off-by: Ian Jackson --- ap-common | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ap-common b/ap-common index cbb815c..bc7c03c 100644 --- a/ap-common +++ b/ap-common @@ -60,7 +60,7 @@ : ${PUSH_TREE_LINUX:=$XENBITS:/home/xen/git/linux-pvops.git} : ${BASE_TREE_

<    2   3   4   5   6   7   8   9   10   11   >