Re: [Xen-devel] [PATCH v3 05/17] SUPPORT.md: Toolstack core

; +### systemd support for xl > + > +Status: Supported I think this should probably mention sysvinit if we are going to mention systemd. That is also Supported. With that extra stanza, Reviewed-by: Ian Jackson <ian.jack...@eu.citrix.com> Thanks, Ian.

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

osstest service owner writes ("[xen-unstable test] 116642: regressions - FAIL"): > flight 116642 xen-unstable real [real] > http://logs.test-lab.xenproject.org/osstest/logs/116642/ > > Regressions :-( > > Tests which did not succeed and are blocking, > including tests which could not be run: >

[Xen-devel] [OSSTEST PATCH 01/11] Debian: Factor out di_special_kernel from ts-host-install

No functional change. Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com> --- Osstest/Debian.pm | 31 +++ ts-host-install | 29 + 2 files changed, 36 insertions(+), 24 deletions(-) diff --git a/Osstest/Debian.pm b/Osstest/Deb

[Xen-devel] [OSSTEST PATCH 04/11] di_installer_path: Break out of preseed_create

No functional change. Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com> --- Osstest/Debian.pm | 16 1 file changed, 12 insertions(+), 4 deletions(-) diff --git a/Osstest/Debian.pm b/Osstest/Debian.pm index 6a1babf..7664194 100644 --- a/Osstest/Debian.pm +++ b/O

[Xen-devel] [OSSTEST PATCH 05/11] di_installer_path: Use in ts-host-install

No significant functional change. We now honour cfg_tftp_di_version if the $ho doesn't have the information. Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com> --- ts-host-install | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ts-host-install b/ts-host-install

[Xen-devel] [OSSTEST PATCH 02/11] di_special_kernel: Pass $kp to $fn

Adjust the one current caller. No functional change Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com> --- Osstest/Debian.pm | 4 ++-- ts-host-install | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/Osstest/Debian.pm b/Osstest/Debian.pm index ba39ad2..6a1babf

[Xen-devel] [OSSTEST PATCH 03/11] di_special_kernel: Replace open-coding in ts-host-install

. No overall functional change. Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com> --- ts-host-install | 9 +++-- 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/ts-host-install b/ts-host-install index 01d04d4..520983f 100755 --- a/ts-host-install +++ b/ts-host-install @@ -

[Xen-devel] [PATCH] docs/process/xen-release-management: Lesson to learn

is known and has been made and tested. Evidently our docs makes it insufficiently clear not to do that. CC: Lars Kurth <lars.ku...@citrix.com> CC: Julien Grall <julien.gr...@arm.com> CC: Juergen Gross <jgr...@suse.com> Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com&g

[Xen-devel] [PATCH] docs/process/release-checklist.txt: Say to push staging branch

it refers to commits they don't have. (Because of the strange rules git has about tag fetching.) Worse, the same may be true of people who "git clone". And anyway, those commits *should* be fed to staging-$x. CC: George Dunlap <george.dun...@citrix.com> Signed-off-by: Ian Jackson

Re: [Xen-devel] [PATCH] docs/process/xen-release-management: Lesson to learn

Julien Grall writes ("Re: [Xen-devel] [PATCH] docs/process/xen-release-management: Lesson to learn"): > That's why testing with XSAs was requested on the security-ml before > hand. However, this was mistakenly done on rc7 rather than rc8. If the proposed release had been constructed in advance,

Re: [Xen-devel] [PATCH] libxl/pvh: force PVH guests to use the xenstore shutdown

Roger Pau Monne writes ("[PATCH] libxl/pvh: force PVH guests to use the xenstore shutdown"): > There's no other way to shutdown a PVH guest at the moment. This commit message is rather confusing. Do you mean, instead: PVH guests are all required to support the xenstore-based shutdown

Re: [Xen-devel] [PATCH] libxl/pvh: force PVH guests to use the xenstore shutdown

Roger Pau Monné writes ("Re: [PATCH] libxl/pvh: force PVH guests to use the xenstore shutdown"): > Yes, that's exactly what I meant but failed to express. Feel free to > replace the commit message with yours. OK, thanks :-). I have Acked-by: Ian Jackson <ian.jack...@eu.citr

Re: [Xen-devel] [PATCH v2] libxl: put RSDP for PVH guest near 4GB

Juergen Gross writes ("[PATCH v2] libxl: put RSDP for PVH guest near 4GB"): > Instead of locating the RSDP table below 1MB put it just below 4GB > like the rest of the ACPI tables in case of PVH guests. This will > avoid punching more holes than necessary into the memory map. > > Signed-off-by:

[Xen-devel] [OSSTEST PATCH] crontab: Fix recipient of cron mails

Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com> --- crontab | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/crontab b/crontab index b710355..3c8c8cc 100755 --- a/crontab +++ b/crontab @@ -2,7 +2,7 @@ #@@ osst...@osstest.test-lab.xenproject.org PATH=/usr/loca

Re: [Xen-devel] [OSSTEST PATCH] linux-arm-xen: Get from shared arm/linux.git xenbits tree

Stefano Stabellini writes ("Re: [OSSTEST PATCH] linux-arm-xen: Get from shared arm/linux.git xenbits tree"): > On Tue, 5 Dec 2017, Julien Grall wrote: > > Acked-by: Julien Grall > > Acked-by: Stefano Stabellini Thanks. This queued for my next

Re: [Xen-devel] [PATCH v17 19/19] osstest: use -DWITHOUT_AUTO_OBJ with FreeBSD release targets

Roger Pau Monné writes ("Re: [PATCH v17 19/19] osstest: use -DWITHOUT_AUTO_OBJ with FreeBSD release targets"): > On Tue, Dec 05, 2017 at 03:09:47PM +, Ian Jackson wrote: > > Is that why you're not just updating osstest to expect the new > > behaviour ? > >

[Xen-devel] [OSSTEST PATCH] linux-arm-xen: Get from shared arm/linux.git xenbits tree

This drops the dependency on Stefano's personal git tree. The osstest output branch remains unmoved. CC: Stefano Stabellini <sstabell...@kernel.org> CC: Julien Grall <julien.gr...@linaro.org> Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com> --- ap-common | 4 ++--

[Xen-devel] [OSSTEST PATCH] Revert "make-flight etc.: drop arm64"

We have the laxtons back and they seem to be working. This reverts commit 329ef52c9866b2a8608a598836b7d5813de1e979. Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com> --- make-flight | 2 +- make-hosts-flight | 2 +- mfi-common| 6 +++--- 3 files changed, 5 insertions

[Xen-devel] [OSSTEST PATCH 2/2] README: Do not recommend cs-flight-create

This is not a normal way to carry on. Far too much like hard work. Recommend make-flight or cs-adjust-flight new: instead. CC: Julien Grall <julien.gr...@arm.com> Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com> --- README | 49 +

Re: [Xen-devel] Commit moratorium for branching Xen 4.10

Julien Grall writes ("Commit moratorium for branching Xen 4.10"): > Xen tree is going to branch at RC7. I don't want to branch when > master != staging, so please avoid committing new patches to staging now > to let master catch up with staging. Another announcement will be made > when the

Re: [Xen-devel] [OSSTEST RFC 13/16] WORKAROUND: Osstest/TestSupport: Make target_reboot works with systemd

Anthony PERARD writes ("[OSSTEST RFC 13/16] WORKAROUND: Osstest/TestSupport: Make target_reboot works with systemd"): > On host running with systemd as init, doing `ssh host reboot` will > result in ssh returning an error. > This patch works around by not waiting for the reboot command to return.

Re: [Xen-devel] [OSSTEST RFC 02/16] Osstest/TestSupport: In teditfileex, get the file with the requested user

Anthony PERARD writes ("[OSSTEST RFC 02/16] Osstest/TestSupport: In teditfileex, get the file with the requested user"): > e.g. If the file to be edited is only accessible to the root user, > target_getfile() would not work. Acked-by: Ian Jackson <ian.ja

Re: [Xen-devel] [OSSTEST RFC 01/16] JobDB-Standalone.tcl: Fix read-runvar

Anthony PERARD writes ("[OSSTEST RFC 01/16] JobDB-Standalone.tcl: Fix read-runvar"): > This fix the error bellow, when the runvar exist: > can't read "runvarinfo()": no such element in array *boggle* Acked-by: Ian Jackson

Re: [Xen-devel] [OSSTEST RFC 04/16] TestSupport: In host_install_postboot, don't call update-rc.d on CentOS

Anthony PERARD writes ("[OSSTEST RFC 04/16] TestSupport: In host_install_postboot, don't call update-rc.d on CentOS"): > Signed-off-by: Anthony PERARD At the very least, this needs an explanation of how, on CentOS, the same effect is achieved. Thanks, Ian.

Re: [Xen-devel] [OSSTEST RFC 09/16] CentOS: Setup osstest-confirm-booted.service

CentOS's provided systemd setup, then so be it. Acked-by: Ian Jackson <ian.jack...@eu.citrix.com> ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel

Re: [Xen-devel] [OSSTEST RFC 03/16] Osstest/TestSupport: Add centos to package_install_cmd

Anthony PERARD writes ("[OSSTEST RFC 03/16] Osstest/TestSupport: Add centos to package_install_cmd"): > Signed-off-by: Anthony PERARD <anthony.per...@citrix.com> Acked-by: Ian Jackson <ian.jack...@eu.citrix.com> But, are we going to invent a package name transl

Re: [Xen-devel] [OSSTEST RFC 11/16] ts-centos-xen-pkg-install: Adjust daemons configuration

Anthony PERARD writes ("[OSSTEST RFC 11/16] ts-centos-xen-pkg-install: Adjust daemons configuration"): > Ajust configuration of xenconsoled and libvirtd. See my comments about ts-xen-install. It already does this. Please don't duplicate things. Thanks, Ian.

Re: [Xen-devel] [OSSTEST RFC 14/16] sg-run-job: Select host install script based on all_host_os runvar

Anthony PERARD writes ("[OSSTEST RFC 14/16] sg-run-job: Select host install script based on all_host_os runvar"): > This also select a different xen installation script. This is wrong, I'm afraid. In general, the recipes should not read runvars (with the exception perhaps of runvars that are

Re: [Xen-devel] [OSSTEST RFC 05/16] TestSupport: Adapt target_https_mitm_proxy_setup to CentOS

Anthony PERARD writes ("[OSSTEST RFC 05/16] TestSupport: Adapt target_https_mitm_proxy_setup to CentOS"): > The location for new certificates is different, and > update-ca-certificates is Debian specific. ... > +my $dest; > +my $update_ca_cmd; > +if ($ho->{OS} eq "centos" ) { I'm not

Re: [Xen-devel] [xen-unstable test] 116832: regressions - FAIL [and 1 more messages]

Jan Beulich writes ("Re: [Xen-devel] [xen-unstable test] 116832: regressions - FAIL [and 1 more messages]"): > Oh, that's pretty unfortunate. I think we'll then need a custom flight > tied to the box that this failure occurred on, to have a way to tell > whether the fix I'm about to prepare has

[Xen-devel] [OSSTEST PATCH] README.dev: Say how to do a commissioning flight for a new architecture

Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com> --- README.dev | 7 +++ 1 file changed, 7 insertions(+) diff --git a/README.dev b/README.dev index dc0dafd..37e2953 100644 --- a/README.dev +++ b/README.dev @@ -132,6 +132,13 @@ If that works, a more thorough test: This shoul

Re: [Xen-devel] [PATCH v17 17/19] osstest: remove the loader timeout from the install image

oxes. > > In order to cope with this remove the loader timeout on the install > image. Note that failure to boot will still drop the loader into a > manual prompt. I don't feel qualified to review this and it is confined to FreeBSD bootloader stuff, so Acked-by: Ian Jac

Re: [Xen-devel] [PATCH v17 19/19] osstest: use -DWITHOUT_AUTO_OBJ with FreeBSD release targets

Roger Pau Monne writes ("[PATCH v17 19/19] osstest: use -DWITHOUT_AUTO_OBJ with FreeBSD release targets"): > Due to a recent FreeBSD change the default output directory of the release > targets is changed to the object directory instead of the source > directory. Use WITHOUT_AUTO_OBJ to restore

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

osstest service owner writes ("[xen-unstable test] 116832: regressions - FAIL"): > flight 116832 xen-unstable real [real] > http://logs.test-lab.xenproject.org/osstest/logs/116832/ > > Regressions :-( > > Tests which did not succeed and are blocking, > including tests which could not be run: >

[Xen-devel] [for-4.10] Re: [PATCH v2] tools/libxl: mark special pages as reserved in e820 map for PVH

c from hvmloader and handling the creation of | the memory map for both HVM and PVH in libxl. But it seems to be a bugfix and has had review from the x86 perspective, so: Acked-by: Ian Jackson <ian.jack...@eu.citrix.com> If anyone is feeling up to doing some improvement, I would like to see a r

Re: [Xen-devel] [PATCH for-4.11 v3 1/1] Add new add_maintainers.pl script to optimise the workflow when using git format-patch with get_maintainer.pl

Thanks. This is much better :-). I have reviewed this for style, obvious bugs, and the semantics in the doc comment. I haven't tried to follow the algorithm in detail, but I reckon it's probably OK. I have reordered the patch (and hence the file) to make the grouping of my comments make more

Re: [Xen-devel] [xen-unstable test] 122660: regressions - trouble: blocked/broken/fail/pass

Juergen Gross writes ("Re: [Xen-devel] [xen-unstable test] 122660: regressions - trouble: blocked/broken/fail/pass"): > On 12/05/18 03:36, osstest service owner wrote: > > flight 122660 xen-unstable real [real] > > http://logs.test-lab.xenproject.org/osstest/logs/122660/ > > > > Regressions :-(

Re: [Xen-devel] staging-4.6 seems to be broken

Jan Beulich writes ("Re: staging-4.6 seems to be broken"): > On 11.05.18 at 17:02, wrote: > > osstest service owner writes ("[xen-4.6-testing test] 122657: regressions - ... > > At least some of these are surely real regressions. > > Are you saying that just because of

Re: [Xen-devel] staging-4.6 seems to be broken

Jan Beulich writes ("Re: staging-4.6 seems to be broken"): > Hrrrm indeed. 4.8 flight 12658 and 4.9 flight 12659 look about as bad as > the 4.6 one you've commented on. I took a look at the host histories for some of the hosts involved and these failures do not seem to have occurred at the same

[Xen-devel] [PATCH 3/4] tools: xencall, xengnttab, xengntshr: Provide access to internal fds

I want this to support my qemu depriv descriptor audit tool. Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com> --- tools/libs/call/core.c| 5 + tools/libs/call/include/xencall.h | 8 tools/libs/call/libxencall.map| 1 + tools/libs/

[Xen-devel] [PATCH 4/4] libxl: Provide better error message when qemu restrict user not found

Add mention of LIBXL_QEMU_USER_RANGE_BASE, in case that is what the user was intending. Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com> --- tools/libxl/libxl_dm.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/tools/libxl/libxl_dm.c b/tools/libxl/libxl_dm.c

[Xen-devel] [PATCH for-4.12 0/4] tools: Internal fd access, etc.

I am working on auditing whether deprvileging qemu has actually worked. The approach I have chosen is to fish the descriptors out of qemu (by using debugging facilities), and try to make hypercalls etc. using them. To take making a hypercall as an example: this is not easily done without libxc.

[Xen-devel] [PATCH 1/4] libxc: Drop declarations of osdep_privcmd_open and _close

These functions are no longer defined or used anywhere. The declarations should have been deleted when the definitions were. Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com> --- tools/libxc/xc_private.h | 3 --- 1 file changed, 3 deletions(-) diff --git a/tools/libxc/xc_private.h b

[Xen-devel] [PATCH 2/4] libxc: Provide access to internal handles

is not significant. Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com> --- tools/libxc/include/xenctrl.h | 10 ++ tools/libxc/xc_private.c | 5 + 2 files changed, 15 insertions(+) diff --git a/tools/libxc/include/xenctrl.h b/tools/libxc/include/xenctrl.h index 408fa1c..d7733aa

[Xen-devel] [OSSTEST PATCH] ts-xen-build: run `make all' before `make', by default

other build arguments). In the future perhaps we should do tools and hypervisor builds entirely separately. Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com> --- ts-xen-build | 6 ++ 1 file changed, 6 insertions(+) diff --git a/ts-xen-build b/ts-xen-build index c5d2a1d..018bc6a

[Xen-devel] [OSSTEST PATCH] sg-report-flight: Provide correct links to test history pages

Grall <julien.gr...@linaro.org> Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com> --- sg-report-flight | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/sg-report-flight b/sg-report-flight index dba3641..e69efb7 100755 --- a/sg-report-flight +++ b/sg-r

Re: [Xen-devel] [PATCH] docs/process/xen-release-management: Lesson to learn

Julien Grall writes ("Re: [Xen-devel] [PATCH] docs/process/xen-release-management: Lesson to learn"): > While I understand this was not ideal, I still think it was the best > solution we had at that time. I would be interested to know what you > would have chosen with the situation we had. >

Re: [Xen-devel] [PATCH] docs/process/xen-release-management: Lesson to learn

George Dunlap writes ("Re: [Xen-devel] [PATCH] docs/process/xen-release-management: Lesson to learn"): > How would you apply this directive to the particular situation we > found ourselves in this time? > > As a reminder: > > * Around 3 December, we didn't think we'd be ready to release until

[Xen-devel] [OSSTEST PATCH] ts-xen-build: run `make build' before `make', by default

arguments). In the future perhaps we should do tools and hypervisor builds entirely separately. Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com> --- v2: Use `make build' instead of `make all' since the former actually detects the bug in a buggy unpatched Xen 4.8. Fix a syntax

[Xen-devel] staging-4.6 seems to be broken

osstest service owner writes ("[xen-4.6-testing test] 122657: regressions - FAIL"): > flight 122657 xen-4.6-testing real [real] > http://logs.test-lab.xenproject.org/osstest/logs/122657/ > > Regressions :-( > > Tests which did not succeed and are blocking, > including tests which could not be

Re: [Xen-devel] xenbits DSA ssh keys to be disabled

Ian Jackson writes ("xenbits DSA ssh keys to be disabled"): > DSA keys ("dss") are 1024-bit and not really considered good practice > any more. By default in Debian's openssh-server, they are now > disabled. > > We are going to disable these soon. Can you p

Re: [Xen-devel] [PATCH v3 0/2] fix several issues in documentation

Juergen Gross writes ("Re: [Xen-devel] [PATCH v3 0/2] fix several issues in documentation"): > On 08/05/18 11:25, George Dunlap wrote: > > But maybe that's a discussion to have when we open the 4.12 development > > window. > > Another point is to add a call of "make -C docs all" to the OSStest >

[Xen-devel] [PATCH for-4.11 0/3] Support matrix: add missing caveat footnotes

From: Ian Jackson <ian.jack...@eu.citrix.com> Lars spotted that the support matrix https://xenbits.xen.org/docs/unstable/support-matrix.html is missing some footnotes. These three patches fix this. I think this is release-critical. Ian Jackson (3): docs/parse-support-md: Rename Re

[Xen-devel] [PATCH 3/3] docs/parse-support-md: Correctly process caveats in multi-status sections

there for them. Reported-by: Lars Kurth <lars.ku...@citrix.com> Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com> --- docs/parse-support-md | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/docs/parse-support-md b/docs/parse-support-md index 278072f..99ce547 10075

[Xen-devel] [PATCH 1/3] docs/parse-support-md: Rename RealSect to RealInSect

This makes the distinction between insections and sectnodes clearer. No functional change. Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com> --- docs/parse-support-md | 44 ++-- 1 file changed, 22 insertions(+), 22 deletions(-) diff --git

Re: [Xen-devel] [PATCH 3/4] tools: xencall, xengnttab, xengntshr: Provide access to internal fds

Andrew Cooper writes ("Re: [Xen-devel] [PATCH 3/4] tools: xencall, xengnttab, xengntshr: Provide access to internal fds"): > These are ABI breakages. Thanks for the review and sorry to miss that. You are right. I have another question, RFC: I have a test C program which links against Xen

Re: [Xen-devel] [PATCH for-4.11 0/3] Support matrix: add missing caveat footnotes

Juergen Gross writes ("Re: [PATCH for-4.11 0/3] Support matrix: add missing caveat footnotes"): > For the series: > Release-acked-by: Juergen Gross Thanks. FYI, example output is here: https://xenbits.xen.org/people/iwj/2018/support-matrix-example-E/t.html For my notes, I

Re: [Xen-devel] Test for osstest, features used in Qubes OS

Marek Marczykowski-Górecki writes ("Test for osstest, features used in Qubes OS"): > As discussed some time ago, I'd like to help with adding tests for some > features we use in Qubes OS. > > IMO the easiest thing to test is host suspend. You just need to execute > "rtcwake -s 30 -m mem", and

[Xen-devel] [OSSTEST PATCH 01/10] Osstest/TestSupport: Use right arch for UEFI grub setup

Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com> --- Osstest/TestSupport.pm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Osstest/TestSupport.pm b/Osstest/TestSupport.pm index 6d5e667..983bb17 100644 --- a/Osstest/TestSupport.pm +++ b/Osstest/TestSupp

[Xen-devel] [OSSTEST PATCH 06/10] Perl @INC path: fix a few more scripts to use BEGIN

Three more files which missed out on dea987c5ab11 "PERLLIB, @INC: Use BEGIN { }" Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com> CC: Roger Pau Monné <roger@citrix.com> --- mg-anoint | 2 +- ts-examine-hostprops-save | 2 +- ts-freebsd-host-ins

[Xen-devel] [OSSTEST PATCH 02/10] Osstest/Debian: preseed: Force UEFI install regardless

n existing operating system, you should choose NOT to force UEFI installation here. Force UEFI installation? 1: Yes 2: No Prompt: '?' for help> Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com> --- Osstest/Debian.pm | 1 + 1 file changed, 1 insertion(+) diff --git a/

[Xen-devel] [OSSTEST PATCH 03/10] README.dev: Fix a typo

Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com> --- README.dev | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/README.dev b/README.dev index 95fc66c..0f7fccd 100644 --- a/README.dev +++ b/README.dev @@ -30,8 +30,8 @@ Keeps running for the duration,

[Xen-devel] [OSSTEST PATCH 04/10] README.dev: Discuss setting Firmware for UEFI machines

This must occur before mknetbootdir, or mknetbootdir does not set things up for UEFI booting. Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com> --- README.dev | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/README.dev b/README.dev index 0f7fccd..a2b93e0

[Xen-devel] [OSSTEST PATCH 10/10] mfi-common: Fall back to anointed builds in Executive mode

Previously, `make-*-flight' would not work unless FREEBSD_*_BUILDJOB was set. Now we use the anointed values if we can find them. If we can't, mg-anoint retrieve will print a warning. Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com> CC: Roger Pau Monné <roger@citrix.com&

[Xen-devel] [OSSTEST PATCH 05/10] README.dev: Make example commisioning runes use $hn variable

This makes them better for cutting-and-pasting. Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com> --- README.dev | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/README.dev b/README.dev index a2b93e0..441eee8 100644 --- a/README.dev +++ b/README.dev @@

[Xen-devel] [OSSTEST PATCH 00/10] UEFI and commissioning fixes

From: Ian Jackson <ian.jack...@eu.citrix.com> I am trying to commission some new hosts. These patches fix problems I encountered. There are fixes to: * Better support UEFI x86 hosts. * Improve the commissioning instructions in README.dev. * Handle the new FreeBSD anointments system i

[Xen-devel] [OSSTEST PATCH 07/10] mg-anoint: Make readonly operations "work" in standalone mode

This makes `mg-anoint' in standalone mode a view onto an empty set of anointments. So now it becomes ok to call mg-anoint in make-*-flight. Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com> CC: Roger Pau Monné <roger@citrix.com> --- Osstest/JobDB/Executive.pm | 2 ++ O

[Xen-devel] [OSSTEST PATCH 08/10] mg-anoint: Support mg-anoint retrieve --tolerate-unprepared

make-*-flight is going to want this. Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com> CC: Roger Pau Monné <roger@citrix.com> --- mg-anoint | 12 ++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/mg-anoint b/mg-anoint index 522cbdd..d09124b 100

[Xen-devel] [OSSTEST PATCH 09/10] mfi-common: set_freebsd_runvars: Never set freebsd_distpath to `/amd64' etc.

the freebsd_* runvars, rather than setting them to wrong values (eg, `freebsd_distpath=/i386'. Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com> CC: Roger Pau Monné <roger@citrix.com> --- mfi-common | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/mfi-

Re: [Xen-devel] [xen-unstable test] 122804: FAIL

Juergen Gross writes ("Re: [Xen-devel] [xen-unstable test] 122804: FAIL"): > On 17/05/18 01:57, osstest service owner wrote: > > Tests which are failing intermittently (not blocking): > > test-amd64-amd64-xl-qemut-debianhvm-amd64 16 guest-localmigrate/x10 fail > > pass in 122715 > >

Re: [Xen-devel] Test for osstest, features used in Qubes OS

Marek Marczykowski-Górecki writes ("Re: Test for osstest, features used in Qubes OS"): > On Thu, May 17, 2018 at 01:26:30PM +0100, Ian Jackson wrote: > > Is it likely that this will depend on non-buggy host firmware ? If so > > then we need to make arrangements to test i

[Xen-devel] XSM in osstest, grub config, outstanding patch

Hi, I'm emailing you because I know you have an interest in XSM (and therefore in its testing in osstest). osstest manages the booting of its test hosts using the distro-supplied bootloader arrangements for its dom0s. For Debian that is update-grub. Currently, osstest has a hacked-up local copy

Re: [Xen-devel] [PATCH v6] scripts/add_maintainers.pl: New script

George Dunlap writes ("Re: [PATCH v6] scripts/add_maintainers.pl: New script"): > I've looked over the help and approve of the functionality, and tested > it for my use case and it seems to work as advertised. > > On that basis: > > Acked-by: George Dunlap Thanks,

Re: [Xen-devel] [PATCH v6] scripts/add_maintainers.pl: New script

George Dunlap writes ("Re: [PATCH v6] scripts/add_maintainers.pl: New script"): > > Changes since v5: > > - Add mention of --get-maintainers, and its best use case, to --help > > output. (Move $get_maintainer up so that it can be used here.) > > Do you actually want to check this massive

[Xen-devel] [PATCH] scripts/add_maintainers.pl: New script

ei Liu <wei.l...@citrix.com> Signed-off-by: Lars Kurth <lars.ku...@citrix.com> Release-acked-by: Juergen Gross <jgr...@suse.com> Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com> --- scripts/add_maintainers.pl | 548 + 1 file c

Re: [Xen-devel] [PATCH for-4.11 v4 1/1] Add new add_maintainers.pl script to optimise the workflow when using git format-patch with get_maintainer.pl

b writefile ($$) { my ($content, $file) = @_; my $fh; open($fh, ">", "$file.tmp") or die "Could not open file '$file.tmp' $!"; print $fh $content or die $!; close $fh or die $!; ranem "$file.tmp", $file or die

Re: [Xen-devel] [PATCH v5] scripts/add_maintainers.pl: New script

(Fixed the Subject line.) Ian Jackson writes ("[PATCH] scripts/add_maintainers.pl: New script"): > Changes since v4: > - Strip some trailing whitespace from the code > - writefile() now uses the .tmp-and-rename pattern to avoid data loss > - Provide --get-maintain

Re: [Xen-devel] Test for osstest, features used in Qubes OS

Dario Faggioli writes ("Re: [Xen-devel] Test for osstest, features used in Qubes OS"): > On Mon, 2018-05-21 at 12:04 +0100, George Dunlap wrote: > > What if we 1) have two versions of the test -- "Fake suspend" and > > "Real Suspend"; 2) only run "Real suspend" on hardware specifically > > marked

[Xen-devel] [PATCH v2 1/2] docs/process: Add RUBRIC

Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com> --- docs/process/RUBRIC | 15 +++ 1 file changed, 15 insertions(+) create mode 100644 docs/process/RUBRIC diff --git a/docs/process/RUBRIC b/docs/process/RUBRIC new file mode 100644 index 000..78aca80 --- /dev/null

[Xen-devel] [PATCH v2 2/2] docs/process/xen-release-management: Lesson to learn

is known and has been made and tested. Evidently our docs makes it insufficiently clear not to do that. CC: Lars Kurth <lars.ku...@citrix.com> CC: Julien Grall <julien.gr...@arm.com> Acked-by: Juergen Gross <jgr...@suse.com> Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.c

Re: [Xen-devel] [PATCH] tools: prepend to PKG_CONFIG_PATH when configuring qemu

Ian Jackson writes ("Re: [PATCH] tools: prepend to PKG_CONFIG_PATH when configuring qemu"): > Juergen Gross writes ("Re: [PATCH] tools: prepend to PKG_CONFIG_PATH when > configuring qemu"): > > On 26/04/18 19:41, Stewart Hildebrand wrote: > >

[Xen-devel] Mysterious one-off libvirt build failure

I am trying to commission some new test boxes. One of my test flights experienced a weird build failure: http://logs.test-lab.xenproject.org/osstest/logs/123046/build-i386-libvirt/6.ts-libvirt-build.log For your convenience, log snippet below. I think this is probably a make -j race in the

Re: [Xen-devel] [PATCH v2 2/2] docs/process/xen-release-management: Lesson to learn

Juergen Gross writes ("Re: [Xen-devel] [PATCH v2 2/2] docs/process/xen-release-management: Lesson to learn"): > On 22/05/18 18:45, Ian Jackson wrote: > > +7. Do not commit to a release date until > > + > > +* The exact xen.git commit id to be released is known

Re: [Xen-devel] [PATCH v2 0/2] Release process: lesson to learn

Ian Jackson writes ("[PATCH v2 0/2] Release process: lesson to learn"): > This is my third attempt to get consensus for this. We had a > discussion about this in December and again in April. I think I have enough acks now, so I intend to push this staging, subject to checking wi

Re: [Xen-devel] Likely build race, "/usr/bin/ld: cannot find -lvirt"

Ian Jackson writes ("Likely build race, "/usr/bin/ld: cannot find -lvirt""): > tl;dr: > > I think there is a bug in libvirt's build system which, with > low probability, causes a build failure containing this message: > /usr/bin/ld: cannot find -lvirt >

Re: [Xen-devel] [PATCH] libxc/x86/PV: don't hand through CPUID leaf 0x80000008 as is

Andrew Cooper writes ("Re: [PATCH] libxc/x86/PV: don't hand through CPUID leaf 0x8008 as is"): > On 22/05/18 12:41, Wei Liu wrote: > > On Tue, May 22, 2018 at 05:40:02AM -0600, Jan Beulich wrote: > >> Just like for HVM the feature set should be used for EBX output, while > >> EAX should be

[Xen-devel] [OSSTEST PATCH] ap-common: Switch to Linux 4.14 by default on X86.

;roger@citrix.com> CC: Julien Grall <julien.gr...@linaro.org> Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com> --- ap-common | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ap-common b/ap-common index ec6e010..5e7cfde 100644 --- a/ap-common +++ b/ap

Re: [Xen-devel] [PATCH for-4.11] tools: set DEBUG_DIR from configure

Olaf Hering writes ("Re: [Xen-devel] [PATCH for-4.11] tools: set DEBUG_DIR from configure"): > On Tue, Mar 27, Roger Pau Monne wrote: > > > Allow the path to be set from a configure command line option. > > Please backport 641f9ce2fa to 4.10 ASAP. See >

Re: [Xen-devel] [OSSTEST PATCH] ap-common: Switch to Linux 4.14 by default on X86.

Julien Grall writes ("Re: [Xen-devel] [OSSTEST PATCH] ap-common: Switch to Linux 4.14 by default on X86."): > Would it be possible to update your CC with my Arm e-mail from now on? Sorry, I copied it from an old commit. > FWIW, linux-arm-xen branch is using 4.14.9. I had to upgrade it in order

Re: [Xen-devel] [xen-unstable-smoke baseline test] 122919: tolerable all pass

Jan Beulich writes ("Re: [Xen-devel] [xen-unstable-smoke baseline test] 122919: tolerable all pass"): > On 18.05.18 at 15:48, wrote: > > But in globbing, ! inside [ ] is a character class complement, not a > > literal. The result is that mg-adjust-flight-makexrefs would

Re: [Xen-devel] [xen-unstable test] 122715: trouble: broken/fail/pass

osstest service owner writes ("[xen-unstable test] 122715: trouble: broken/fail/pass"): > flight 122715 xen-unstable real [real] > http://logs.test-lab.xenproject.org/osstest/logs/122715/ > > Failures and problems with tests :-( > > Tests which did not succeed and are blocking, > including

Re: [Xen-devel] [xen-unstable-smoke baseline test] 122919: tolerable all pass

cent actually tested revision. This is a lie. In fact, the problem is that many of these flights are rather ill-constructed. I have just force pushed the commit below to osstest to fix it. Ian. From 86662216840ce20c2e6dd248e4a04492d11b27dd Mon Sep 17 00:00:00 2001 From: Ian Jackson <ian.jack...@eu.citri

[Xen-devel] [PATCH v6] scripts/add_maintainers.pl: New script

(Move $get_maintainer up so that it can be used here.) Cc: Andrew Cooper <andrew.coop...@citrix.com> Cc: George Dunlap <george.dun...@eu.citrix.com> Cc: Ian Jackson <ian.jack...@eu.citrix.com> Cc: Jan Beulich <jbeul...@suse.com> Cc: Julien Grall <julien.gr...@arm.com>

Re: [Xen-devel] [PATCH v5] scripts/add_maintainers.pl: New script

Lars Kurth writes ("Re: [PATCH v5] scripts/add_maintainers.pl: New script"): > Acked-by: Lars Kurth > Although it should probably mention --get-maintainers in the help message Good point. Ian. ___ Xen-devel mailing list

Re: [Xen-devel] Mysterious one-off libvirt build failure

Ian Jackson writes ("Mysterious one-off libvirt build failure"): > I am trying to commission some new test boxes. One of my test flights > experienced a weird build failure: > > > http://logs.test-lab.xenproject.org/osstest/logs/123046/build-i386-libvirt/6.ts-libvir

[Xen-devel] [OSSTEST PATCH v2 23/23] freebsd: Expect a freebsd-10 hostflag for freebsd tests

This will allow us to exclude certain hosts. For example, right now, we have no support for booting FreeBSD on UEFI hosts. Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com> CC: Roger Pau Monné <roger@citrix.com> --- make-flight | 2 +- 1 file changed, 1 insertion(+)

[Xen-devel] [OSSTEST PATCH v2 09/23] mg-anoint: Make readonly operations "work" in standalone mode

This makes `mg-anoint' in standalone mode a view onto an empty set of anointments. So now it becomes ok to call mg-anoint in make-*-flight. Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com> CC: Roger Pau Monné <roger@citrix.com> --- Osstest/JobDB/Executive.pm | 2 ++ O

[Xen-devel] [OSSTEST PATCH v2 10/23] mg-anoint: Support mg-anoint retrieve --tolerate-unprepared

make-*-flight is going to want this. Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com> CC: Roger Pau Monné <roger@citrix.com> --- mg-anoint | 12 ++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/mg-anoint b/mg-anoint index 522cbdd..d09124b 100

[Xen-devel] [OSSTEST PATCH v2 20/23] grub: Move 20_linux_xen to suite-specific overlays

stretch still needs some of this, in particular the XSM changes. Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com> --- overlay-jessie/etc/grub.d/20_linux_xen | 220 overlay-stretch/etc/grub.d/20_linux_xen | 220 o

[Xen-devel] [OSSTEST PATCH v2 06/23] README.dev: Discuss setting Firmware for UEFI machines

This must occur before mknetbootdir, or mknetbootdir does not set things up for UEFI booting. Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com> --- README.dev | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/README.dev b/README.dev index de04b98..a2e46f9

  1   2   3   4   5   6   7   8   9   10   >