Re: Python in Domain Configurations

2023-07-31 Thread Ian Jackson
les - the main part of the existing parser - is only 183 loc including comments. Extending it (and the support code in libxlu_cfg.c) to do dictionaries as well as lists doesn't seem like it would make it too much bigger. Thanks, Ian. -- Ian JacksonThese opinions are my own. Pronouns: they/he. If I emailed you from @fyvzl.net or @evade.org.uk, that is a private address which bypasses my fierce spamfilter.

Re: [OSSTEST PATCH] PDU/IPMI: Be less aggressive with IPMI commands

2023-03-10 Thread Ian Jackson
instead of > sending the `power off` command again which we would never check the > result of. Reviewed-by: Ian Jackson -- Ian JacksonThese opinions are my own. Pronouns: they/he. If I emailed you from @fyvzl.net or @evade.org.uk, that is a private address which bypasses my fierce spamfilter.

Re: [PATCH 2/8] docs/process: branching-checklist: Remove reference to qemu-ijw.git

2022-12-12 Thread Ian Jackson
ag for the official tree. So remove > the section about qemu-ijw.git. > > [1] 25482.10006.140155.984...@chiark.greenend.org.uk > > Signed-off-by: Julien Grall > Cc: ijack...@chiark.greenend.org.uk Acked-by: Ian Jackson Ian.

RE: Release checklist and qemu-xen-traditional

2022-12-02 Thread Ian Jackson
e-$v master > ``` I confess I answered without looking at the instructions again myself. I think you are right, and that branch instruction may be a duplicate. Later, when it comes to tagging, you'll probably want to have a local staging-$v branch in your working tree, so that the tag is in the right

Release checklist and qemu-xen-traditional

2022-12-02 Thread Ian Jackson
irc. I'm Diziet on oftc. I usually lurk on #xendevel (although I fell off at some point and only just rejoined), or you may send me a private message. Regards, Ian. -- Ian JacksonThese opinions are my own. Pronouns: they/he. If I emailed you from @fyvzl.net or @evade.org.uk, that is a private address which bypasses my fierce spamfilter.

Re: [PATCH] libvirt: disable Werror for non-libvirt flights

2022-09-15 Thread Ian Jackson
var like "libvrit_build_werror=false". But maybe that is gold-plating it. If you choose to keep that the way it is, then for either version of this patch: Acked-by: Ian Jackson -- Ian JacksonThese opinions are my own. Pronouns: they/he. If I emailed you from @fyvzl.net or @evade.or

osstest branch refs and self-push-gate Re: Sorting out osstest

2022-09-15 Thread Ian Jackson
orce push, ./mg-force-push is better since it can perform additional checks. xenbits:~osstest/osstest-massachusetts.git #refs/heads/flight-NNN Permanent record of the osstest code which ran some flight. Separated because of git's poor perf with trees with so many re

Re: [PATCH] libvirt: disable Werror for non-libvirt flights

2022-09-13 Thread Ian Jackson
ibvirt flights, > but this seems more conservative. Probably disabling it only for Xen is right. Ian. -- Ian JacksonThese opinions are my own. Pronouns: they/he. If I emailed you from @fyvzl.net or @evade.org.uk, that is a private address which bypasses my fierce spamfilter.

Re: [OSSTEST PATCH 2/2] ts-libvirt-build: Rework to build with meson

2022-08-02 Thread Ian Jackson
a "Makefile". That seems right to me. >build-essential bin86 bcc iasl bc > - flex bison cmake ninja-build > + flex bison cmake ninja-build meson *Some* version of meson is in even stretch so this is fine. A

Re: [OSSTEST PATCH 1/2] TestSupport: Add support for installing from backport repo

2022-08-02 Thread Ian Jackson
print ::EO < + > +# $suite backports > +deb $bp_url $suite-backports main > +EOF > + }); Nit: this last line looks misindented to me. Regardless, Acked-by: Ian Jackson Thanks, Ian. -- Ian JacksonThese opinions are my own. Pronouns: they/he. If I emailed you from

Re: [OSSTEST PATCH v2] ts-xen-build-prep: Install newer NASM version, to build OVMF

2022-05-20 Thread Ian Jackson
alled on Buster as it has few > dependencies and are already satisfied. > > [1] > https://github.com/tianocore/edk2/commit/6a890db161cd6d378bec3499a1e774db3f5a27a7 > ("BaseTools: Upgrade the version of NASM tool") LGTM assuming Roger is also happy... Acked-by: Ian Jackson

Re: [OSSTEST PATCH] ts-xen-build-prep: Grab newer NASM version, to build OVMF

2022-05-13 Thread Ian Jackson
ose using the some_extradebs method. Yes, I think this is better. -- Ian JacksonThese opinions are my own. Pronouns: they/he. If I emailed you from @fyvzl.net or @evade.org.uk, that is a private address which bypasses my fierce spamfilter.

Re: [OSSTEST PATCH] ts-xen-build-prep: Grab newer NASM version, to build OVMF

2022-05-13 Thread Ian Jackson
build-prep runs with the host all to itself, so I think you can use /root ? Ian. -- Ian JacksonThese opinions are my own. Pronouns: they/he. If I emailed you from @fyvzl.net or @evade.org.uk, that is a private address which bypasses my fierce spamfilter.

Re: [PATCH] osstest: update Debian Buster install CD media to 10.12

2022-05-12 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH] osstest: update Debian Buster install CD media to 10.12"): > Signed-off-by: Roger Pau Monné Acked-by: Ian Jackson Is this just a routine update, OOI, or was this to fix something ? Thanks, Ian.

Re: [PATCH] osstest: install irqbalance

2022-04-12 Thread Ian Jackson
able vCPUs, as the number of vCPUs offered to dom0 matches the > number of physical CPUs. Wow. This was missing all this time! I wonder if it was in the default install in some previous Debian release... Acked-by: Ian Jackson -- Ian JacksonThese opinions are my own. Pronouns: the

Re: [PATCH v2] osstest: stop anacron service

2022-04-07 Thread Ian Jackson
est > finishes fast enough: Well done for tracking that down. Acked-by: Ian Jackson Ian.

Re: [PATCH] osstest: update Debian installer version

2022-03-28 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH] osstest: update Debian installer version"): > Update the configuration file Debian version after having run > mg-debian-installer-update-all. Acked-by: Ian Jackson FTR, in future I think you should apply changes like this one without waiting for

Re: [PATCH] coverity: disable flight from crontab

2022-02-22 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH] coverity: disable flight from crontab"): > We are currently doing the Coverity Scans using a github workflow. Acked-by: Ian Jackson -- Ian JacksonThese opinions are my own. Pronouns: they/he. If I emailed you from @fyvzl.net or

Re: [PATCH] SUPPORT.md, MAINTAINERS: De-support qemu-xen-traditional

2022-01-04 Thread Ian Jackson
Yes, I'm afraid so. Regards, Ian. -- Ian JacksonThese opinions are my own. Pronouns: they/he. If I emailed you from @fyvzl.net or @evade.org.uk, that is a private address which bypasses my fierce spamfilter.

[OSSTEST PATCH 2/2] daily-cron-email-real-*: Temporarily drop osstest-admin

2021-12-14 Thread Ian Jackson
Roger has agreed to take on osstest admin for the moment. Someone who intents to take on the role long term will probably want to get CC's of these flight reports, but it's fairly noisy. So for now, send them only to the lists. CC: Roger Pau Monné Signed-off-by: Ian Jackson --- daily-cron

[OSSTEST PATCH 1/2] daily-cron-email-{play,adhoc}-*: Drop Citrix email

2021-12-14 Thread Ian Jackson
-by: Ian Jackson --- daily-cron-email-adhoc| 2 +- daily-cron-email-adhoc-bisectcomplete | 2 +- daily-cron-email-adhoc-bisectdone | 2 +- daily-cron-email-adhoc-bisectrun | 2 +- daily-cron-email-play | 2 +- daily-cron-email-play-bisectcomplete | 2

[PATCH] SUPPORT.md, MAINTAINERS: De-support qemu-xen-traditional

2021-12-09 Thread Ian Jackson
In 4.16 we changed to not build this by default. I think it is now time to explicitly desupport it, completely, in favour of Linux dm stub domains. Signed-off-by: Ian Jackson --- MAINTAINERS | 2 +- SUPPORT.md | 18 +- 2 files changed, 10 insertions(+), 10 deletions(-) diff

Re: [PATCH] MAINTAINERS: widen Anthony's area

2021-12-08 Thread Ian Jackson
igned-off-by: Jan Beulich FWIW Acked-by: Ian Jackson I have full confidence in Anthony here. Ian.

Re: [PATCH 1/7] xz: add fall-through comments to a switch statement

2021-12-07 Thread Ian Jackson
Jan Beulich writes ("Re: [PATCH 1/7] xz: add fall-through comments to a switch statement"): > I'm unwilling only as long as I don't understand the need for them. As > indicated, while I appreciate your "make verification easier for > reviewers", I assign that at least no higher priority than my

[OSSTEST PATCH 13/13] ts-xen-install: enable timestamp on guests logs

2021-12-06 Thread Ian Jackson
will override any options in XENCONSOLED_ARGS, so they can not be used in conjunction. Signed-off-by: Roger Pau Monné Acked-by: Ian Jackson --- ts-xen-install | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ts-xen-install b/ts-xen-install index 47865eb60..bf55d4e5c 100755

[OSSTEST PATCH 12/13] ts-memdisk-try-append: Fix for fact that broken() does not return

2021-12-06 Thread Ian Jackson
Signed-off-by: Ian Jackson --- ts-memdisk-try-append | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/ts-memdisk-try-append b/ts-memdisk-try-append index 18c4aaa5c..e34415b05 100755 --- a/ts-memdisk-try-append +++ b/ts-memdisk-try-append @@ -38,8 +38,6 @@ arch=`perl -I. -we

[OSSTEST PATCH 11/13] TestSupport: sub broken: document that it does not return.

2021-12-06 Thread Ian Jackson
Signed-off-by: Ian Jackson --- Osstest/TestSupport.pm | 1 + 1 file changed, 1 insertion(+) diff --git a/Osstest/TestSupport.pm b/Osstest/TestSupport.pm index ebe436275..8103ea1d3 100644 --- a/Osstest/TestSupport.pm +++ b/Osstest/TestSupport.pm @@ -261,6 +261,7 @@ sub fail ($) { sub broken

[OSSTEST PATCH 10/13] TestSupport: sub broken: exit with status 0 for skip

2021-12-06 Thread Ian Jackson
Otherwise, whatever calls us will set the step to fail (or something). Signed-off-by: Ian Jackson --- Osstest/TestSupport.pm | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/Osstest/TestSupport.pm b/Osstest/TestSupport.pm index ff890f96c..ebe436275 100644 --- a/Osstest

[OSSTEST PATCH 02/13] ts-memdisk-try-append: Reindent (nfc)

2021-12-06 Thread Ian Jackson
Signed-off-by: Ian Jackson Release-Acked-by: Ian Jackson --- ts-memdisk-try-append | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/ts-memdisk-try-append b/ts-memdisk-try-append index 4b54eff01..a35751eb3 100755 --- a/ts-memdisk-try-append +++ b/ts-memdisk

[OSSTEST PATCH 03/13] ts-memdisk-try-append: Enable perl warnings

2021-12-06 Thread Ian Jackson
Signed-off-by: Ian Jackson Release-Acked-by: Ian Jackson --- ts-memdisk-try-append | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ts-memdisk-try-append b/ts-memdisk-try-append index a35751eb3..bfb36caf7 100755 --- a/ts-memdisk-try-append +++ b/ts-memdisk-try-append @@ -18,7

[OSSTEST PATCH 06/13] starvation: Only count "pass" and "fail" as done jobs

2021-12-06 Thread Ian Jackson
The main effect is to treat "broken" and "blocked" jobs as not part of the flight for starvation calculations. This is correct for commissioning flights and OK for other flights. Signed-off-by: Ian Jackson Release-Acked-by: Ian Jackson --- Osstest/Executive.pm

[OSSTEST PATCH 04/13] ts-memdisk-try-append: More defensive case test

2021-12-06 Thread Ian Jackson
Signed-off-by: Ian Jackson Release-Acked-by: Ian Jackson --- ts-memdisk-try-append | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/ts-memdisk-try-append b/ts-memdisk-try-append index bfb36caf7..d5c12866f 100755 --- a/ts-memdisk-try-append +++ b/ts-memdisk-try

[OSSTEST PATCH 00/13] Miscellaneous improvements

2021-12-06 Thread Ian Jackson
These patches were accumulated during the Xen 4.16 freeze and as RM I chose to defer them. I am submitting them to osstest pretest now. Ian Jackson (12): make-flight: do_examine_one: add firmware argument ts-memdisk-try-append: Reindent (nfc) ts-memdisk-try-append: Enable perl warnings

[OSSTEST PATCH 09/13] TestSupport: sub broken: Add a newline

2021-12-06 Thread Ian Jackson
This suppresses printing of the the line number, which is not useful. Signed-off-by: Ian Jackson --- Osstest/TestSupport.pm | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/Osstest/TestSupport.pm b/Osstest/TestSupport.pm index b17bf08a6..ff890f96c 100644 --- a/Osstest

[OSSTEST PATCH 05/13] examination: skip memdisk on non-BIOS hosts, run per-firmware on x86

2021-12-06 Thread Ian Jackson
the corresponding plain job, namely osstest linux-* xen-unstable These jobs are just like the plain jobs, except that one of ,PropEq:Firmware:bios:bios ,PropEq:Firmware:bios:uefi has been added to the end of all_hostflags. Signed-off-by: Ian Jackson Release-Acked-by: Ian

[OSSTEST PATCH 01/13] make-flight: do_examine_one: add firmware argument

2021-12-06 Thread Ian Jackson
Nothing sets this yet. No functional change (checked with standalone-generate-dump-flight-runvars) Signed-off-by: Ian Jackson Release-Acked-by: Ian Jackson --- make-flight | 15 +-- 1 file changed, 13 insertions(+), 2 deletions(-) diff --git a/make-flight b/make-flight index

[OSSTEST PATCH 08/13] TestSupport: sub broken: break out "$msg"

2021-12-06 Thread Ian Jackson
No functional change. Signed-off-by: Ian Jackson --- Osstest/TestSupport.pm | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/Osstest/TestSupport.pm b/Osstest/TestSupport.pm index a0ca69437..b17bf08a6 100644 --- a/Osstest/TestSupport.pm +++ b/Osstest/TestSupport.pm

[OSSTEST PATCH 07/13] sg-run-job: Do not set a step status to pass unless it's running

2021-12-06 Thread Ian Jackson
This avoids skip etc. being treated as pass. Signed-off-by: Ian Jackson --- tcl/JobDB-Executive.tcl | 1 + 1 file changed, 1 insertion(+) diff --git a/tcl/JobDB-Executive.tcl b/tcl/JobDB-Executive.tcl index 38248823e..8e0062f59 100644 --- a/tcl/JobDB-Executive.tcl +++ b/tcl/JobDB-Executive.tcl

Re: [PATCH 1/7] xz: add fall-through comments to a switch statement

2021-12-06 Thread Ian Jackson
Julien Grall writes ("Re: [PATCH 1/7] xz: add fall-through comments to a switch statement"): > On 06/12/2021 16:12, Jan Beulich wrote: > > Hmm, I did address both your and Ian's concerns in v2, admittedly by only > > going as far as minimally necessary. I therefore wouldn't call this an > > "open

Xen staging-4.16 branch now open

2021-12-06 Thread Ian Jackson
Ian Jackson writes ("Xen 4.16 is released"): > Xen 4.16, the product of 9 months' work by the Xen Project community, > is now released. I have now done the administrivia to staging-4.16 and it is open for backports according to the usual protocol. Ian.

[PATCH for-xen-stable] MAINTAINERS: Resign from my tools stable branch maintainership

2021-12-06 Thread Ian Jackson
:00:00 2001 From: Ian Jackson To: xen-devel@lists.xenproject.org Date: Mon, 6 Dec 2021 14:40:24 + Subject: [PATCH for-xen-stable] MAINTAINERS: Resign from tools stable branch maintainership Signed-off-by: Ian Jackson Signed-off-by: Ian Jackson --- MAINTAINERS | 2 +- 1 file changed

[PATCH] MAINTAINERS: Resign from my maintainership roles

2021-12-06 Thread Ian Jackson
. I have really appreciated working with such good co-maintainers and colleagues. Thanks, Ian. >From 608531a0cc34a5bc096ccf585e16f182b5ed83e1 Mon Sep 17 00:00:00 2001 From: Ian Jackson To: xen-devel@lists.xenproject.org Cc: Andrew Cooper Cc: George Dunlap Cc: Jan Beulich Cc: Julien Grall

[OSSTEST PATCH] production-config: renumbering the test lab

2021-12-04 Thread Ian Jackson
Our dhcp pool exhausted. I am going to force push this. Signed-off-by: Ian Jackson --- production-config | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/production-config b/production-config index 9a4054445..023b81622 100644 --- a/production-config +++ b/production-config

Re: Xen 4.16 is released

2021-12-02 Thread Ian Jackson
Correction: I wrote: > Xen 4.16, the product of 9 months' work by the Xen Project community, > is now released. > > You can find it here: > git clone -b RELEASE-4.16.0 https://xenbits.xen.org/git-http/xen.git > https://downloads.xenproject.org/release/xen/4.16.0/ > For more information see

Xen 4.16 is released

2021-12-02 Thread Ian Jackson
Xen 4.16, the product of 9 months' work by the Xen Project community, is now released. You can find it here: git clone -b RELEASE-4.16.0 https://xenbits.xen.org/git-http/xen.git https://downloads.xenproject.org/release/xen/4.16.0/ For more information see the release notes:

Xen 4.16 release party, Cambridge

2021-12-01 Thread Ian Jackson
We will shortly release Xen 4.16.0. Some of us will be celebrating in person in Cambridge: in the pub, next Thursday (9th December). We'll be at the Haymakers [1], in the gazebo, from 17:00. https://www.individualpubs.co.uk/haymakers/ https://www.openstreetmap.org/way/131306510 In addition,

[PATCH] CHANGELOG.md: Start new "unstable" section

2021-12-01 Thread Ian Jackson
I'm committing this now, along with the forward-ports mentioned. I think this is mechanical release technician stuff, so I don't feel the need for an ack. Ian. >From 6c1c97e24f830a921a23e3b9694e20493c9986ee Mon Sep 17 00:00:00 2001 From: Ian Jackson To: xen-devel@lists.xenproject.org Cc:

[PATCH for-4.16] SUPPORT.md: Define support lifetime

2021-11-30 Thread Ian Jackson
Signed-off-by: Ian Jackson --- SUPPORT.md | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/SUPPORT.md b/SUPPORT.md index b1367e2b22..85726102ea 100644 --- a/SUPPORT.md +++ b/SUPPORT.md @@ -10,9 +10,9 @@ for the definitions of the support status levels etc. # Release

Re: [xen-unstable test] 166378: regressions - FAIL

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

Re: [PATCH 1/7] xz: add fall-through comments to a switch statement

2021-11-26 Thread Ian Jackson
Jan Beulich writes ("Re: [PATCH 1/7] xz: add fall-through comments to a switch statement"): > On 26.11.2021 11:04, Julien Grall wrote: > > For this case, you provided some sort of an explanation but so far, I am > > still waiting for a link to confirm that the signed-off-by match the one > > on

Re: [PATCH] osstest: enable timestamp on guests logs

2021-11-25 Thread Ian Jackson
setting both the > log and the timestamp options. Note that setting XENCONSOLED_TRACE > will override any options in XENCONSOLED_ARGS, so they can not be used > in conjunction. > > Signed-off-by: Roger Pau Monné Acked-by: Ian Jackson Thanks! I have queued this on my "to be pushed after 4.16" branch. Ian.

Re: [xen-4.15-testing test] 166311: regressions - FAIL

2021-11-25 Thread Ian Jackson
Jan Beulich writes ("Re: [xen-4.15-testing test] 166311: regressions - FAIL"): > On 24.11.2021 03:00, osstest service owner wrote: > > flight 166311 xen-4.15-testing real [real] > > http://logs.test-lab.xenproject.org/osstest/logs/166311/ > > > > Regressions :-( > > > > Tests which did not

Re: [PATCH for-4.16] Revert "x86/CPUID: shrink max_{,sub}leaf fields according to actual leaf contents"

2021-11-25 Thread Ian Jackson
Ian Jackson writes ("Re: [PATCH for-4.16] Revert "x86/CPUID: shrink max_{,sub}leaf fields according to actual leaf contents""): > Jan Beulich writes ("Re: [PATCH for-4.16] Revert "x86/CPUID: shrink > max_{,sub}leaf fields according to actual leaf conten

Re: [PATCH for-4.16] Revert "x86/CPUID: shrink max_{,sub}leaf fields according to actual leaf contents"

2021-11-25 Thread Ian Jackson
th the course of action taken. > Nevertheless I fear this is going to become yet one more case where > future action is promised, but things then die out. > > Ian - I guess all this now needs is your R-a. Thanks everyone. Yes. For the record, Release-Acked-by: Ian Jackson I will commit it myself now. Ian.

Re: [PATCH for-4.16] x86/cpuid: do not shrink number of leaves in max policies

2021-11-24 Thread Ian Jackson
Andrew Cooper writes ("Re: [PATCH for-4.16] x86/cpuid: do not shrink number of leaves in max policies"): > It may revert cleanly, but it won't build because of the first hunk in > 81da2b544cbb00.  That hunk needs reverting too, because it too breaks > some cpuid= settings in VM config files.

Re: [PATCH for-4.16] x86/cpuid: do not shrink number of leaves in max policies

2021-11-24 Thread Ian Jackson
(Hoisting Roger and Jan to the To:) Andrew Cooper writes ("Re: [PATCH for-4.16] x86/cpuid: do not shrink number of leaves in max policies"): > On 24/11/2021 16:24, Ian Jackson wrote: > > Questions from my RM hat: > > > > Is there a workaround ? > > No

Re: [PATCH for-4.16] x86/cpuid: do not shrink number of leaves in max policies

2021-11-24 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH for-4.16] x86/cpuid: do not shrink number of leaves in max policies"): > Shrinking max policies can lead to failures in migration as previous > versions of Xen didn't shrink the number of leaves in any case, so > it's possible for a guest created on previous

[OSSTEST PATCH] mg-repro-setup: Make ordinary alloc: work again

2021-11-24 Thread Ian Jackson
omote an error test to before builds (nfc)" CC: Roger Pau Monné Signed-off-by: Ian Jackson Release-Acked-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 895f29963..b00e182f3 100755 --- a/mg-repro-setu

Xen 4.16 branched, commit moratorium lifted, freeze still in force

2021-11-23 Thread Ian Jackson
Xen 4.16 has now branched. The staging-4.16 branch et al exists. The commit moratorium is lifted. However: * 4.16 is in deep freeze now. Nothing should be committed without a release ack. Roger, you have a blanket release ack for your CHANGELOG update(s). * Until 4.16 is released,

[PATCH for-4.17 3/3] Set version to 4.17: rerun autogen.sh

2021-11-23 Thread Ian Jackson
Signed-off-by: Ian Jackson --- configure | 18 +- docs/configure| 18 +- stubdom/configure | 18 +- tools/configure | 18 +- 4 files changed, 36 insertions(+), 36 deletions(-) diff --git a/configure b/configure index

[PATCH for-4.17 2/3] Set version to 4.17; 4.16 has branched

2021-11-23 Thread Ian Jackson
Signed-off-by: Ian Jackson --- README | 10 +- xen/Makefile | 4 ++-- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/README b/README index a626e56436..562b808080 100644 --- a/README +++ b/README @@ -1,9 +1,9

[PATCH for-4.17 1/3] Revert "Config.mk: pin QEMU_UPSTREAM_REVISION (prep for Xen 4.16 RC1)"

2021-11-23 Thread Ian Jackson
This branch is unstable again now. This reverts commit c9ce6afbf2d7772f47fc572bb7fc9555724927ed. --- Config.mk | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Config.mk b/Config.mk index 6be010c7ce..6587c7d626 100644 --- a/Config.mk +++ b/Config.mk @@ -239,7 +239,7 @@

[PATCH for-4.16 3/3] Turn off debug by default

2021-11-23 Thread Ian Jackson
Signed-off-by: Ian Jackson --- tools/Rules.mk| 2 +- xen/Kconfig.debug | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/Rules.mk b/tools/Rules.mk index b022da3336..051a5d3555 100644 --- a/tools/Rules.mk +++ b/tools/Rules.mk @@ -25,7 +25,7 @@ CFLAGS_xeninclude = -I

[PATCH for-4.16 2/3] SUPPORT.md: Set Release Notes link

2021-11-23 Thread Ian Jackson
Signed-off-by: Ian Jackson --- SUPPORT.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/SUPPORT.md b/SUPPORT.md index 3a34933c89..b1367e2b22 100644 --- a/SUPPORT.md +++ b/SUPPORT.md @@ -9,13 +9,13 @@ for the definitions of the support status levels etc. # Release

[PATCH for-4.16 1/3] Config.mk: switch to named tags (for stable branch)

2021-11-23 Thread Ian Jackson
Signed-off-by: Ian Jackson --- Config.mk | 8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/Config.mk b/Config.mk index 6be010c7ce..b32a185efc 100644 --- a/Config.mk +++ b/Config.mk @@ -239,17 +239,15 @@ SEABIOS_UPSTREAM_URL ?= git://xenbits.xen.org/seabios.git

[PATCH for-4.16 0/3] Changes from release technician checklist

2021-11-23 Thread Ian Jackson
I am about to push these to xen.git#staging-4.16. Roger, I have not messed with the furniture in CHANGELOG.md on either staging-4.16 or staging. Ian. Ian Jackson (3): Config.mk: switch to named tags (for stable branch) SUPPORT.md: Set Release Notes link Turn off debug by default

Xen 4.16 RC4

2021-11-23 Thread Ian Jackson
Xen 4.16 RC4 is now available. It is available from git: git clone https://xenbits.xenproject.org/git-http/xen.git -b 4.16.0-rc4 For your convenience a tarball is available: https://downloads.xenproject.org/release/xen/4.16.0-rc4/xen-4.16.0-rc4.tar.gz

[OSSTEST PATCH 1/2] README.dev: Document to use updated osstest for commissioning

2021-11-22 Thread Ian Jackson
Signed-off-by: Ian Jackson Release-Acked-by: Ian Jackson --- README.dev | 2 ++ 1 file changed, 2 insertions(+) diff --git a/README.dev b/README.dev index 3d09b3c6b..132772fb4 100644 --- a/README.dev +++ b/README.dev @@ -108,6 +108,8 @@ Create the tftp directory (must be done after firmware

[OSSTEST PATCH 2/2] allow.all: Treat freeBSD memdisk-try-append failures as nonblocking

2021-11-22 Thread Ian Jackson
In practice this currently fails on some of our hosts. We don't seem to have resources to investigate properly, and this will be blocking recomissioning of hosts if we don't get it out of the way. Signed-off-by: Ian Jackson Release-Acked-by: Ian Jackson CC: Roger Pau Monné --- allow.all | 1

Commit Moratorium (was Re: Xen 4.16 development update - tree status)

2021-11-22 Thread Ian Jackson
tl;dr: Commit moratorium is now in force until further notice (which will be given tomorrow afternoon). Jan Beulich writes ("Re: Xen 4.16 development update - tree status"): > On 19.11.2021 16:15, Ian Jackson wrote: > > Tree status > > === > > >

Re: Xen 4.16 development update - tree status

2021-11-19 Thread Ian Jackson
Andrew Cooper writes ("Re: Xen 4.16 development update - tree status"): > On 19/11/2021 15:15, Ian Jackson wrote: > > Open issues and potential blockers > > == Thanks, Andrew, for this helpful information. > > I am aware of thre

Re: [PATCH for-4.16 v3] efi: fix alignment of function parameters in compat mode

2021-11-19 Thread Ian Jackson
nt the special casing. Signed-off-by: Roger Pau Monné Release-Acked-by: Ian Jackson Reviewed-by: Ian Jackson Signed-off-by: Ian Jackson Reviewed-by: Jan Beulich --- Changes since v3: - Remove hard tabs. Apply Jan's r-b as authorised in email. Changes since v2: - Adjust the commentary a

[PATCH for-4.16 v3] efi: fix alignment of function parameters in compat mode

2021-11-19 Thread Ian Jackson
copy of 3 variables doesn't seem to warrant the special casing. Signed-off-by: Roger Pau Monné Release-Acked-by: Ian Jackson Reviewed-by: Ian Jackson Signed-off-by: Ian Jackson --- Changes since v2: - Adjust the commentary as per discussion. Changes since v1: - Copy back the results. --- x

Xen 4.16 development update - tree status

2021-11-19 Thread Ian Jackson
Tree status === We are now in deep code freeze, during which we will try to discover and eliminate serious bugs and regressions. All patches other than documentation patches need a Release-Ack. Fixes for serious bugs, and test improvements, will get such an ack. I have decided to branch

Re: [XEN PATCH for-4.16] golang/xenlight: regen generated code

2021-11-19 Thread Ian Jackson
Nick Rosbrook writes ("Re: [XEN PATCH for-4.16] golang/xenlight: regen generated code"): > Acked-by: Nick Rosbrook Thanks all. Acked-by: Ian Jackson Release-Acked-by: Ian Jackson and pushed. Ian.

Re: [PATCH 0/7] (mainly) xz imports from Linux

2021-11-19 Thread Ian Jackson
Jan Beulich writes ("[PATCH 0/7] (mainly) xz imports from Linux"): > While going through their 5.15.3 log I did notice two changes, which made > me go check what else we might be missing. The series here is the result. > Linux has also updated zstd, but that includes a pretty large change which >

Re: [PATCH for-4.16] efi: fix alignment of function parameters in compat mode [and 1 more messages]

2021-11-17 Thread Ian Jackson
Roger Pau Monné writes ("Re: [PATCH for-4.16] efi: fix alignment of function parameters in compat mode [and 1 more messages]"): > On Wed, Nov 17, 2021 at 04:05:43PM +, Ian Jackson wrote: > > Andrew Cooper writes ("Re: [PATCH for-4.16] efi: fix alignment of function

Re: [PATCH for-4.16] efi: fix alignment of function parameters in compat mode [and 1 more messages]

2021-11-17 Thread Ian Jackson
Andrew Cooper writes ("Re: [PATCH for-4.16] efi: fix alignment of function parameters in compat mode"): > This will compile, but the caller won't get any data back unless you > copy the opposite way here... Well spotted. I feel quite the fool! Ian.

Re: [PATCH for-4.16] efi: fix alignment of function parameters in compat mode

2021-11-17 Thread Ian Jackson
on our stack instead of doing it in xenpf_efi_runtime_call. Thanks. I have double-checked the variable names etc. Reviewed-by: Ian Jackson I agree with your analysis vis-a-vis 4.16. The current code is technically UB[1] and it seems possible that it might trigger bugs in firmware. I would like a third

Re: [PATCH for-4.16] MAINTAINERS: declare REMUS support orphaned

2021-11-17 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH for-4.16] MAINTAINERS: declare REMUS support orphaned"): > The designated maintainer email address for the remus entry is > bouncing, so remove it and declare the entry as orphaned as there's no > other maintainer. Hrm, :-(. Acked-by: Ian Jackson

Re: [PATCH for-4.16 1/2] CHANGELOG: set Xen 4.15 release date

2021-11-17 Thread Ian Jackson
Jan Beulich writes ("Re: [PATCH for-4.16 1/2] CHANGELOG: set Xen 4.15 release date"): > I'm puzzled how we managed to have missed that. Manual faff and eyes glazing over I think... > > --- a/CHANGELOG.md > > +++ b/CHANGELOG.md > > @@ -22,7 +22,7 @@ The format is based on [Keep a > >

Re: [PATCH for-4.16 0/2] CHANGELOG: start to populate entries for 4.16

2021-11-17 Thread Ian Jackson
maintainers in this cover letter and also in patch #2 for > further feedback on missing items. Thanks! Dropped the CC list for this mail: Release-Acked-by: Ian Jackson for both patches (including future versions). Ian.

Re: [PATCH for-4.16 4/4] tools/python: fix python libxc bindings to pass a max grant version

2021-11-17 Thread Ian Jackson
ifying a max version of 2, which is > > inline with the current defaults in the hypervisor. > > > > Fixes: 7379f9e10a ('gnttab: allow setting max version per-domain') > > Signed-off-by: Roger Pau Monné > > Acked-by: Andrew Cooper FTR Acked-by: Ian Jackson

Xen 4.16 RC3

2021-11-16 Thread Ian Jackson
Xen 4.16 RC3 is now available. It is available from git: git clone https://xenbits.xenproject.org/git-http/xen.git -b 4.16.0-rc3 For your convenience a tarball is available: https://downloads.xenproject.org/release/xen/4.16.0-rc3/xen-4.16.0-rc3.tar.gz

Re: [PATCH for-4.16 3/4] test/tsx: set grant version for created domains

2021-11-16 Thread Ian Jackson
ich > Signed-off-by: Roger Pau Monné Release-Acked-by: Ian Jackson

Re: [PATCH for-4.16 1/4] domctl: introduce a macro to set the grant table max version

2021-11-16 Thread Ian Jackson
; > > > Requested-by: Jan Beulich > > Signed-off-by: Roger Pau Monné > > Acked-by: Julien Grall Reviewed-by: Ian Jackson Release-Acked-by: Ian Jackson

Re: [PATCH for-4.16 4/4] tools/python: fix python libxc bindings to pass a max grant version

2021-11-16 Thread Ian Jackson
he current defaults in the hypervisor. > > Fixes: 7379f9e10a ('gnttab: allow setting max version per-domain') > Signed-off-by: Roger Pau Monné > --- > Cc: Ian Jackson > > Without this fix the python bindings won't be usable, as they will > attempt to pass a max version of 0 which wi

Re: [linux-linus test] 166151: regressions - FAIL

2021-11-16 Thread Ian Jackson
Jan Beulich writes ("Re: [linux-linus test] 166151: regressions - FAIL"): > On 16.11.2021 05:42, osstest service owner wrote: > > flight 166151 linux-linus real [real] > > http://logs.test-lab.xenproject.org/osstest/logs/166151/ > > > > Regressions :-( > > > > Tests which did not succeed and are

Re: [PATCH-4.16] arm/efi: Improve performance requesting filesystem handle

2021-11-16 Thread Ian Jackson
Luca Fancellu writes ("[PATCH-4.16] arm/efi: Improve performance requesting filesystem handle"): > Currently, the code used to handle and possibly load from the filesystem > modules defined in the DT is allocating and closing the filesystem handle > for each module to be loaded. > > To improve

Re: [PATCH for-4.16] CHANGELOG: set Xen 4.15 release date

2021-11-16 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH for-4.16] CHANGELOG: set Xen 4.15 release date"): > Signed-off-by: Roger Pau Monné Release-Acked-by: Ian Jackson

Re: [PATCH v2 1/6][4.16?] x86/x2APIC: defer probe until after IOMMU ACPI table parsing [and 2 more messages]

2021-11-16 Thread Ian Jackson
t of > users we believe to be empty. > > As ever, feel free to contradict me if I have the wrong end of one of > the many sticks here... On the question of backporting: Jan Beulich: > >Ian Jackson: > >>> Nevertheless it will then be a backporting candidate, so > &

Re: [PATCH for-4.16] freebsd/privcmd: fix MMAP_RESOURCE ioctl definition

2021-11-16 Thread Ian Jackson
as originally the > MMAP_RESOURCE ioctl didn't need to copy out any data. > > Signed-off-by: Roger Pau Monné > --- > Cc: Ian Jackson > > The change only affects FreeBSD, and it's only a change in a > definition of an ioctl, so it's unlikely to break existing code logic. &g

Re: [PATCH-4.16 v2] xen/efi: Fix Grub2 boot on arm64

2021-11-16 Thread Ian Jackson
Luca Fancellu writes ("Re: [PATCH-4.16 v2] xen/efi: Fix Grub2 boot on arm64"): > > On 15 Nov 2021, at 22:00, Stefano Stabellini wrote: > > On Mon, 15 Nov 2021, Julien Grall wrote: > >> That would indeed be better. I'd like this patch to be merged in 4.16. > >> Would > >> you be able to send a

Re: [PATCH for-4.16] Revert "domctl: improve locking during domain destruction" [and 2 more messages]

2021-11-16 Thread Ian Jackson
Roger Pau Monné writes ("Re: [PATCH for-4.16] Revert "domctl: improve locking during domain destruction""): > On Tue, Nov 09, 2021 at 03:04:56PM +, Ian Jackson wrote: > > So I am going to treat this as an effectively new change. > > > > AIU

Re: [PATCH for-4.16] tests/resource: set grant version for created domains

2021-11-16 Thread Ian Jackson
s cannot be created. > > Fixes: 7379f9e10a ('gnttab: allow setting max version per-domain') > Reported-by: Andrew Cooper > Signed-off-by: Roger Pau Monné Release-Acked-by: Ian Jackson

Re: [PATCH v2] public/gnttab: relax v2 recommendation

2021-11-16 Thread Ian Jackson
Jan Beulich writes ("[PATCH v2] public/gnttab: relax v2 recommendation"): > With there being a way to disable v2 support, telling new guests to use > v2 exclusively is not a good suggestion. > > Signed-off-by: Jan Beulich > Reviewed-by: Luca Fancellu Rele

Re: [PATCH RESEND 0/3][4.16?] VT-d: misc (regression) fixes [and 2 more messages]

2021-11-12 Thread Ian Jackson
Jan Beulich writes ("[PATCH RESEND 0/3][4.16?] VT-d: misc (regression) fixes"): > (re-sending upon Ian's request with his address adjusted, including > Kevin's R-b at this occasion) > > 1: per-domain IOMMU bitmap needs to have dynamic size ... > As to patch 1: Without the earlier change, systems

Re: [PATCH RESEND 1/3][4.16] VT-d: per-domain IOMMU bitmap needs to have dynamic size

2021-11-12 Thread Ian Jackson
ixes: 27713fa2aa21 ("VT-d: improve save/restore of registers across S3") > Signed-off-by: Jan Beulich > Reviewed-by: Kevin Tian Release-Acked-by: Ian Jackson provided it can be committed today. Thanks, Ian.

Re: [PATCH v2] MAINTAINERS: add Bertrand to the ARM reviewers

2021-11-11 Thread Ian Jackson
ase-acked-by for this? No, it's fine, but for the record: Release-Acked-by: Ian Jackson

Re: [PATCH v3] xen/arm: don't assign domU static-mem to dom0 as reserved-memory

2021-11-11 Thread Ian Jackson
Julien Grall writes ("Re: [PATCH v3] xen/arm: don't assign domU static-mem to dom0 as reserved-memory"): > The first version of the patch [1] was tagged with for-4.16 and contains > the rationale. > > I was thinking to commit it because it already contains your > release-acked-by. Can you let

  1   2   3   4   5   6   7   8   9   10   >