Re: [Xen-devel] Security response; public holidays

2017-05-02 Thread Ian Jackson
George Dunlap writes ("Re: Security response; public holidays"): > Some holidays will be fixed dates, while others (like Easter and > American Thanksgiving) will change every year. For the ones that > change, we need a mechanism for refreshing them every year. I suggest we set a reminder to send

[Xen-devel] [OSSTEST PATCH] Debian grub2 config: set TERMINAL to 'serial console'

2017-05-02 Thread Ian Jackson
7; provides the output to both (and would take input from whichever presses a key). Signed-off-by: Ian Jackson --- Osstest/Debian.pm | 3 +++ 1 file changed, 3 insertions(+) diff --git a/Osstest/Debian.pm b/Osstest/Debian.pm index acc4618..4155056 100644 --- a/Osstest/Debian.pm +++ b/Osstest/Debi

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

2017-05-02 Thread Ian Jackson
Paul Durrant writes ("RE: [Xen-devel] [xen-unstable test] 108068: regressions - FAIL"): > Ok, there really should be tests for Windows 10 and/or Server 2016 in there. > A lot of the newer viridian features only apply to those OS. I'll look into this. Ian. __

Re: [Xen-devel] [PATCH v1 0/2] libxl: add PV display device driver interface

2017-05-02 Thread Ian Jackson
Oleksandr Grytsov writes ("Re: [Xen-devel] [PATCH v1 0/2] libxl: add PV display device driver interface"): > On Thu, Apr 13, 2017 at 3:54 PM, Ian Jackson > wrote: > > Can you sketch out what the rest of the system does, then ? > > Presumably there has to be a differ

[Xen-devel] Security response; public holidays

2017-05-02 Thread Ian Jackson
listing all known public holidays throughout the world. But we are interested in public holidays affecting Xen Project community members. I was wondering if we should start a wiki page and invite community members to add their own local holidays. Ian. -- Ian JacksonThese opinions are my own

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

2017-05-02 Thread Ian Jackson
Paul Durrant writes ("RE: [Xen-devel] [xen-unstable test] 108068: regressions - FAIL"): > It's very hard to say. There's nothing particularly conclusive in the logs. > QEMU has definitely requested a reset as can be seen in > > http://logs.test-lab.xenproject.org/osstest/logs/108068/test-amd64-

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

2017-05-02 Thread Ian Jackson
Paul Durrant writes ("RE: [Xen-devel] [xen-unstable test] 108068: regressions - FAIL"): > In my experience Windows is usually consistent in its behaviour on power or > sleep events so it's unlikely that it would reboot sometime and not others. > BTW, is this test really testing XP SP3? It's been

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

2017-05-02 Thread Ian Jackson
Andrew Cooper writes ("Re: [Xen-devel] [xen-unstable test] 108068: regressions - FAIL"): > This is known, and has definitely been discussed before on xen-devel > before (although it involved IanC last time he looked at these tests, so > a while ago now). > > In an APCI view of the world, the two

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

2017-05-02 Thread Ian Jackson
Jan Beulich writes ("Re: [Xen-devel] [xen-unstable test] 108068: regressions - FAIL"): > On 01.05.17 at 20:49, wrote: > This has been recurring for the last few flights, but I wonder whether > > 2017-05-01 13:18:52 Z executing ssh ... root@172.16.144.40 readlink > /dev/italia0-vg/win.guest.osst

Re: [Xen-devel] [libvirt test] 107696: regressions - FAIL

2017-05-02 Thread Ian Jackson
Jim Fehlig writes ("Re: [Xen-devel] [libvirt test] 107696: regressions - FAIL"): > On 04/26/2017 02:34 PM, osstest service owner wrote: > > flight 107696 libvirt real [real] > > http://logs.test-lab.xenproject.org/osstest/logs/107696/ > > > > Regressions :-( > > > > Tests which did not succeed and

Re: [Xen-devel] Xen 4.9.0 RC3

2017-05-02 Thread Ian Jackson
Ian Jackson writes ("Xen 4.9.0 RC3"): > Xen 4.9.0 RC3 is now available. > > You can check it out from xen.git: > > git://xenbits.xen.org/xen.git#4.9.0-rc3 > > For your convenience there is also a tarball and singautre at: > https://downloads.xenproject.o

[Xen-devel] Xen 4.9.0 RC3

2017-04-28 Thread Ian Jackson
Xen 4.9.0 RC3 is now available. You can check it out from xen.git: git://xenbits.xen.org/xen.git#4.9.0-rc3 For your convenience there is also a tarball and singautre at: https://downloads.xenproject.org/release/xen/4.9.0-rc3/xen-4.9.0-rc3.tar.gz https://downloads.xenproject.org/release/

Re: [Xen-devel] [PATCH] x86emul: correct stub invocation constraints again

2017-04-28 Thread Ian Jackson
> > Reported-by: Boris Ostrovsky > Signed-off-by: Jan Beulich Release-acked-by: Ian Jackson ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

[Xen-devel] [OSSTEST PATCH] ts-libvirt-build: Cope with keycodemapdb submodule

2017-04-28 Thread Ian Jackson
libvirt.git#02fb15f introduces this new submodule. Submodules are awkward, and we need them to be explicitly listed. So add keycodemapdb to the submodulemap in ts-libvirt-build. Reported-by: Wei Liu Signed-off-by: Ian Jackson --- ts-libvirt-build | 3 ++- 1 file changed, 2 insertions(+), 1

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

2017-04-28 Thread Ian Jackson
Jan Beulich writes ("Re: [Xen-devel] [xen-unstable test] 107791: regressions - FAIL"): > I don't suppose there is any memory sharing being set up for any > tests in osstest, Indeed, there isn't. Ian. ___ Xen-devel mailing list Xen-devel@lists.xen.org

[Xen-devel] OCaml xenstore: max value size is configure to be 2048 bytes - consistent with C xenstore?

2017-04-27 Thread Ian Jackson
(dropping Lars; CCing MAINTAINERs for affected areas.) Christian Lindig writes ("OCaml xenstore: max value size is configure to be 2048 bytes - consistent with C xenstore?"): > We recently had a case where XenServer installation showed errors because the > data stored in OCaml xenstore exceeded

Re: [Xen-devel] [PATCH for-4.9] seabios: run olddefconfig

2017-04-26 Thread Ian Jackson
Wei Liu writes ("[PATCH for-4.9] seabios: run olddefconfig"): > We provided a base config file in 970f8de3e. To generate a full config > file, running olddefconfig is required. > > Signed-off-by: Wei Liu > --- > Cc: Ian Jackson > Cc: Jan Beulich Release-acke

Re: [Xen-devel] [seabios bisection] complete build-amd64-xsm

2017-04-26 Thread Ian Jackson
Jan Beulich writes ("Re: [Xen-devel] [seabios bisection] complete build-amd64-xsm"): > On 26.04.17 at 04:27, wrote: > > Bug is in tree: xen git://xenbits.xen.org/xen.git > > Bug introduced: 99704f26360ee8d4f85081c6c50ce64f47961f6d > > Bug not present: 6f9b62ca57322197e26d3b22ff11b62969714

Re: [Xen-devel] [RFC PATCH 9/9] xen: Add use_iommu flag to createdomain domctl

2017-04-26 Thread Ian Jackson
Oleksandr Tyshchenko writes ("Re: [RFC PATCH 9/9] xen: Add use_iommu flag to createdomain domctl"): > On Tue, Apr 25, 2017 at 6:23 PM, Wei Liu wrote: > > Let me explain where I'm coming from: > > > > 1. if not populating the iommu page table causes Xen to malfunction > >(crash?), it is a bug.

Re: [Xen-devel] [OSSTEST PATCH] ts-openstack-deploy: Set http proxy

2017-04-25 Thread Ian Jackson
Anthony PERARD writes ("Re: [OSSTEST PATCH] ts-openstack-deploy: Set http proxy"): > The patch itself looks good, but I think http_proxy_envsettings only set > http_proxy, twice, but does not set https_proxy. Indeed. See my patch "proxy config: Actually set https_proxy too" posted today, when I

[Xen-devel] [OSSTEST PATCH] ts-openstack-deploy: Set http proxy

2017-04-25 Thread Ian Jackson
This allows ./stack.sh to access the global internet. CC: Anthony PERARD Signed-off-by: Ian Jackson --- ts-openstack-deploy | 3 +++ 1 file changed, 3 insertions(+) diff --git a/ts-openstack-deploy b/ts-openstack-deploy index a1c974f..d4c041d 100755 --- a/ts-openstack-deploy +++ b/ts

[Xen-devel] [OSSTEST PATCH 1/3] mg-repro-setup: Cope with flights referencing other builds

2017-04-25 Thread Ian Jackson
We attempted to do this by trying to adjust build runvars only if they didn't have a `.'. But cs-adjust-flight runvar-build-set has special handling for : it is always qualified with the flight. So change the match to look for the current flight number. Signed-off-by: Ian Jackson ---

[Xen-devel] [OSSTEST PATCH 2/3] proxy config: Actually set https_proxy too

2017-04-25 Thread Ian Jackson
lity. Signed-off-by: Ian Jackson --- Osstest/TestSupport.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Osstest/TestSupport.pm b/Osstest/TestSupport.pm index d482e1d..19bdd23 100644 --- a/Osstest/TestSupport.pm +++ b/Osstest/TestSupport.pm @@ -1911,7 +1911,7 @@ sub ht

[Xen-devel] [OSSTEST PATCH 3/3] mg-hosts: -l: print subtask info too

2017-04-25 Thread Ian Jackson
Signed-off-by: Ian Jackson --- mg-allocate | 12 +--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/mg-allocate b/mg-allocate index 2dfbdb1..4f02ce5 100755 --- a/mg-allocate +++ b/mg-allocate @@ -612,13 +612,19 @@ END qw(restype resname

Re: [Xen-devel] linux 4.9 fails with nested Xen

2017-04-21 Thread Ian Jackson
Boris Ostrovsky writes ("Re: linux 4.9 fails with nested Xen"): > This is the same failure that was discussed earlier --- lack of commit > da72ff5bfcb02 (partially revert "xen: Remove event channel notification > through Xen PCI platform device") in 4.9 tree. > > This actual offending commit (72a9

[Xen-devel] linux 4.9 fails with nested Xen

2017-04-21 Thread Ian Jackson
osstest service owner writes ("[linux-4.9 test] 107573: regressions - FAIL"): > flight 107573 linux-4.9 real [real] > http://logs.test-lab.xenproject.org/osstest/logs/107573/ ... > test-amd64-amd64-qemuu-nested-intel 13 xen-boot/l1 fail never pass > test-amd64-amd64-qemuu-nested-amd 13 xen-

[Xen-devel] Xen Project Security Team considering becoming a CNA

2017-04-20 Thread Ian Jackson
(Resending with the correct CC (!)) We are in discussions with MITRE with a view to potentially becoming a CVE Numbering Authority. This would probably smooth the process of getting CVE numbers for XSAs. If anyone has any opinions/representations/concerns/whatever about this, please do share the

[Xen-devel] Xen Project Security Team considering becoming a CNA

2017-04-20 Thread Ian Jackson
We are in discussions with MITRE with a view to potentially becoming a CVE Numbering Authority. This would probably smooth the process of getting CVE numbers for XSAs. If anyone has any opinions/representations/concerns/whatever about this, please do share them (here in this thread, or privately

[Xen-devel] [PATCH 1/2] sg-report-flight: Do not try to print nonexistent FirstTip info

2017-04-20 Thread Ian Jackson
The use of $info->{FirstTip}{flight} autovivifies $info->{FirstTip}. Defend $pinfo against the use of an autovivified empty hashref. Signed-off-by: Ian Jackson --- sg-report-flight |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sg-report-flight b/sg-report-flight

[Xen-devel] [PATCH 2/2] sg-report-flight: Do not report many non-regressions as allowable

2017-04-20 Thread Ian Jackson
variable, which was never used, was removed. However, what ought to happen, is that $failv->{Allow} would be set to 1 if the entry matched an allow pattern, and only if it was a blocker. That makes the logic in "report ... separately" correct. Signed-off-by: Ian Jackson --- sg-

Re: [Xen-devel] [PATCH for-4.9 v2 0/3] oxenstored: make it work on FreeBSD

2017-04-20 Thread Ian Jackson
;; > +esac > +AC_SUBST(XENSTORED_KVA) > + > +case "$host_os" in > +*freebsd*) XENSTORED_PORT=/dev/xen/xenstored ;; > +*) XENSTORED_PORT=/proc/xen/xsd_port ;; > +esac > +AC_SUBST(XENSTORED_PORT) I'm not sure why one wouldn't combine these two case statement

Re: [Xen-devel] [PATCH RFC 07/20] migration: defer precopy policy to libxl

2017-04-18 Thread Ian Jackson
Wei Liu writes ("Re: [PATCH RFC 07/20] migration: defer precopy policy to libxl"): > On Thu, Mar 30, 2017 at 01:19:41AM -0400, Joshua Otto wrote: > > Is the memcpy() really significant here? If this were a tight > > loop, sure, but every invocation of the policy callback implies > > both a 4MB ne

[Xen-devel] [OSSTEST PATCH 10/11] mg-execute-flight: Support --progress-fd

2017-04-18 Thread Ian Jackson
Signed-off-by: Ian Jackson --- mg-execute-flight | 7 +++ 1 file changed, 7 insertions(+) diff --git a/mg-execute-flight b/mg-execute-flight index bd09df3..5a861b0 100755 --- a/mg-execute-flight +++ b/mg-execute-flight @@ -35,6 +35,8 @@ publish=false badusage () { echo >&2 &qu

[Xen-devel] [OSSTEST PATCH 02/11] ts-coverity-upload: Pass HttpsProxyMITMCert to curl

2017-04-18 Thread Ian Jackson
"proxy config: Add ability to install MITM TLS cert" sets this up on the test hosts, but not the controller. We don't really want this odd configuration globally on the controller. Instead, we have to pass it where it is needed. Signed-off-by: Ian Jackson --- ts-coverity-uplo

[Xen-devel] [OSSTEST PATCH 04/11] sg-run-job: Break out testid_matches_globs

2017-04-18 Thread Ian Jackson
No functional change. Signed-off-by: Ian Jackson --- sg-run-job | 18 -- 1 file changed, 12 insertions(+), 6 deletions(-) diff --git a/sg-run-job b/sg-run-job index cc24b73..8f7c36f 100755 --- a/sg-run-job +++ b/sg-run-job @@ -133,6 +133,15 @@ proc setstatus {st} { jobdb

[Xen-devel] [OSSTEST PATCH 08/11] mg-allocate: Document --info-base in head comment

2017-04-18 Thread Ian Jackson
This documentation was omitted from "mg-repro-setup: Provide useful info for allocation system" Signed-off-by: Ian Jackson --- mg-allocate | 2 ++ 1 file changed, 2 insertions(+) diff --git a/mg-allocate b/mg-allocate index 803abb6..87a4e71 100755 --- a/mg-allocate +++ b/mg-allocat

[Xen-devel] [OSSTEST PATCH 06/11] sg-run-job: Honour skip_testids runvar

2017-04-18 Thread Ian Jackson
p matching takes place against the unsubstituted value (since it happens before a stepno is allocated). There is no way to skip only some of a set of steps whose testids are distinguished only by stepno. Signed-off-by: Ian Jackson --- sg-run-job | 15 --- 1 file changed, 12 insert

[Xen-devel] [OSSTEST PATCH 05/11] sg-run-job: Start step a bit later

2017-04-18 Thread Ian Jackson
We are going to want to process the nearly-finished testid. So allocate a stepno at the last possible moment. No overall functional chagne. Signed-off-by: Ian Jackson --- sg-run-job | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/sg-run-job b/sg-run-job index 8f7c36f

[Xen-devel] [OSSTEST PATCH 00/11] Coverity fix and mg-repro-setup improvements

2017-04-18 Thread Ian Jackson
This should hopefully actually fix the coverity upload failures. Also, I tripped over some infelicities in mg-repro-setup. I'm going to push this to pretest now. ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

[Xen-devel] [OSSTEST PATCH 03/11] truncation: Support globs, and multiple patterns

2017-04-18 Thread Ian Jackson
le to cope with space-separated lists. Signed-off-by: Ian Jackson --- cs-bisection-step | 4 +++- mg-repro-setup| 2 +- sg-run-job| 15 +-- 3 files changed, 13 insertions(+), 8 deletions(-) diff --git a/cs-bisection-step b/cs-bisection-step index 162e068..76f5010 100755

[Xen-devel] [OSSTEST PATCH 07/11] mg-repro-setup: skip log capture (by default)

2017-04-18 Thread Ian Jackson
Signed-off-by: Ian Jackson --- mg-repro-setup | 5 + 1 file changed, 5 insertions(+) diff --git a/mg-repro-setup b/mg-repro-setup index bb429b4..af679d0 100755 --- a/mg-repro-setup +++ b/mg-repro-setup @@ -34,6 +34,7 @@ usage () { cat < estimated duration (default = 28d) --ro

[Xen-devel] [OSSTEST PATCH 09/11] mg-allocate: Support --progress-fd

2017-04-18 Thread Ian Jackson
Signed-off-by: Ian Jackson --- mg-allocate | 14 ++ 1 file changed, 14 insertions(+) diff --git a/mg-allocate b/mg-allocate index 87a4e71..2dfbdb1 100755 --- a/mg-allocate +++ b/mg-allocate @@ -57,6 +57,8 @@ # # --info-base= Replaces `manual' at start of info put into

[Xen-devel] [OSSTEST PATCH 01/11] TestSupport: Provide stashfilecontents

2017-04-18 Thread Ian Jackson
Like target_putfilecontents_* but for files on controller. No callers yet. Signed-off-by: Ian Jackson --- Osstest/TestSupport.pm | 11 +++ 1 file changed, 11 insertions(+) diff --git a/Osstest/TestSupport.pm b/Osstest/TestSupport.pm index 2b4a1f4..d482e1d 100644 --- a/Osstest

[Xen-devel] [OSSTEST PATCH 11/11] mg-repro-setup: Use --progress-fd

2017-04-18 Thread Ian Jackson
Also, print a sensible message if allocation fails. Signed-off-by: Ian Jackson --- mg-repro-setup | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/mg-repro-setup b/mg-repro-setup index af679d0..0b3137a 100755 --- a/mg-repro-setup +++ b/mg-repro-setup @@ -48,6

[Xen-devel] [for-4.9] Re: [PATCH 2/2] tools: Use POSIX signal.h instead of sys/signal.h

2017-04-18 Thread Ian Jackson
Alistair Francis writes ("[PATCH 2/2] tools: Use POSIX signal.h instead of sys/signal.h"): > The POSIX spec specifies to use: > #include > instead of: > #include > as seen here: >http://pubs.opengroup.org/onlinepubs/009695399/functions/signal.html

Re: [Xen-devel] [PATCH for-4.9 2/2] oxenstored: make it work on FreeBSD

2017-04-18 Thread Ian Jackson
Wei Liu writes ("[PATCH for-4.9 2/2] oxenstored: make it work on FreeBSD"): > Call the uname syscall to determine sysname and return device names > accordingly. ... > -let xenstored_proc_kva = "/proc/xen/xsd_kva" > -let xenstored_proc_port = "/proc/xen/xsd_port" > +let xenstored_proc_kva = > + let

Re: [Xen-devel] [PATCH for-4.9] oxenstored: remove "_proc" in names

2017-04-18 Thread Ian Jackson
Wei Liu writes ("[PATCH for-4.9] oxenstored: remove "_proc" in names"): > They aren't limited to proc file system anymore. Use more generic names. > > No functional change. Acked-by: Ian Jackson ___ Xen-devel ma

Re: [Xen-devel] [PATCH for-4.9 1/2] oxenstored: add an Unix syscall C extension

2017-04-18 Thread Ian Jackson
Wei Liu writes ("[PATCH for-4.9 1/2] oxenstored: add an Unix syscall C extension"): > Currently there is only uname syscall in it. I agree that this would be good to have in 4.9. The ocaml FFI has tricky memory management, so this needs review by an ocaml expert. Ian. _

Re: [Xen-devel] [PATCH v1 0/2] libxl: add PV display device driver interface

2017-04-13 Thread Ian Jackson
Oleksandr Grytsov writes ("Re: [Xen-devel] [PATCH v1 0/2] libxl: add PV display device driver interface"): > After internal discussion we think that putting positions and > z-orders of virtual connectors to the Xen store and libxl > configuration is not so good idea. Because their composition depe

[Xen-devel] [PATCH 0/5] Release administrivia for 4.9.0-rc1*

2017-04-12 Thread Ian Jackson
This series is being posted pro-format. I've pushed them already. The first patch is routine administrivia. The next two are the result of me messing up. The final two are release checklist updates, including attempts to make this kind of mistake less likely. Thanks, Ian.

[Xen-devel] [PATCH 1/5] Config.mk, etc.: Prepare 4.9.0-rc1

2017-04-12 Thread Ian Jackson
* Update Config.mk REVISION values to refer to relevant tags. * Update README version number * Update xen/Makefile version number Signed-off-by: Ian Jackson --- Config.mk| 6 +++--- README | 10 +- xen/Makefile | 2 +- 3 files changed, 9 insertions(+), 9 deletions(-) diff

[Xen-devel] [PATCH 3/5] Config.mk: Update for 4.9.0-rc1.2

2017-04-12 Thread Ian Jackson
.) Signed-off-by: Ian Jackson --- Config.mk | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Config.mk b/Config.mk index c2e4e37..8642fec 100644 --- a/Config.mk +++ b/Config.mk @@ -269,7 +269,7 @@ MINIOS_UPSTREAM_URL ?= git://xenbits.xen.org/mini-os.git endif OVMF_UPSTREA

[Xen-devel] [PATCH 2/5] Config.mk: Update for 4.9.0-rc1.1

2017-04-12 Thread Ian Jackson
In qemu-trad, I made xen-4.9.0-rc1 refer erroneously to the 4.8 branch. That doesn't build. So we are burning the version number 4.9.0-rc1 in xen.git and qemu-trad. (The other trees can remain as they are.) Signed-off-by: Ian Jackson --- Config.mk | 2 +- 1 file changed, 1 insertion(

[Xen-devel] [PATCH 4/5] misc/release-checklist.txt: Preemptive updates

2017-04-12 Thread Ian Jackson
These are things I noticed should be fixed, while trying to make 4.9.0-rc1. Signed-off-by: Ian Jackson --- misc/release-checklist.txt | 43 ++- 1 file changed, 18 insertions(+), 25 deletions(-) diff --git a/misc/release-checklist.txt b/misc/release

[Xen-devel] [PATCH 5/5] misc/release-checklist.txt: Try to avoid wrong-tag mistakes

2017-04-12 Thread Ian Jackson
Add some better checking and make the runes a bit more robust. Signed-off-by: Ian Jackson --- misc/release-checklist.txt | 13 + 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/misc/release-checklist.txt b/misc/release-checklist.txt index f9e7e06..b96964e 100644 --- a

Re: [Xen-devel] Please apply "partially revert "xen: Remove event channel..."

2017-04-12 Thread Ian Jackson
Greg KH writes ("Re: Please apply "partially revert "xen: Remove event channel...""): > On Wed, Apr 12, 2017 at 03:26:53PM +0200, Juergen Gross wrote: > > On 12/04/17 15:13, Greg KH wrote: > > > Is this still true? This long thread is totally confusing, is that what > > > you really want to have

[Xen-devel] [PATCH 1/8] selecthost: Default pdu to "unsupported" rather than nothing

2017-04-10 Thread Ian Jackson
Signed-off-by: Ian Jackson --- Osstest/TestSupport.pm |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Osstest/TestSupport.pm b/Osstest/TestSupport.pm index 4ced6c0..a498ddd 100644 --- a/Osstest/TestSupport.pm +++ b/Osstest/TestSupport.pm @@ -814,7 +814,7 @@ sub

[Xen-devel] [PATCH 3/8] mg-allocate: Redirect a few messages to stderr

2017-04-10 Thread Ian Jackson
These were, anomalously, printed to stdout. We are going to want to reserve stdout for programmatically-useful output. Signed-off-by: Ian Jackson --- mg-allocate |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/mg-allocate b/mg-allocate index 7b1eb62..f895af2 100755

[Xen-devel] [PATCH 5/8] ts-hosts-allocate-Executive: Honour OSSTEST_NOALLOCATE

2017-04-10 Thread Ian Jackson
This allows host allocation to be skipped even though the rest of the job is being run. Signed-off-by: Ian Jackson --- ts-hosts-allocate-Executive |5 + 1 file changed, 5 insertions(+) diff --git a/ts-hosts-allocate-Executive b/ts-hosts-allocate-Executive index 0b83365..9417226 100755

[Xen-devel] [PATCH 8/8] serial logs capture: Detect errors from glob

2017-04-10 Thread Ian Jackson
Empirically, if glob cannot do its work, it sets $! and returns an empty list. Signed-off-by: Ian Jackson --- Osstest/Serial/sympathy.pm |5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/Osstest/Serial/sympathy.pm b/Osstest/Serial/sympathy.pm index 84a1e09..7461e07

[Xen-devel] [PATCH 4/8] mg-allocate: Provide --stdout-output

2017-04-10 Thread Ian Jackson
This prints RESTYPE/RESNAME/SHAREIX to stdout, after allocation is successful (outside the db retry loop). This means that a programmatic caller can now tell what was allocated, even if the spec was complex. Signed-off-by: Ian Jackson --- mg-allocate |8 1 file changed, 8

[Xen-devel] [PATCH 6/8] mg-repro-setup: New script for setting up repros automatically

2017-04-10 Thread Ian Jackson
l the user Signed-off-by: Ian Jackson --- mg-repro-setup | 200 1 file changed, 200 insertions(+) create mode 100755 mg-repro-setup diff --git a/mg-repro-setup b/mg-repro-setup new file mode 100755 index 000..c673cb6 --- /dev/null +++

[Xen-devel] [PATCH 7/8] serial logs capture: chomp parameters to the perl script

2017-04-10 Thread Ian Jackson
For reasons that aren't clear, perl's glob operator ignores the \n. We can gloss over that for now, but let's chomp it in case they fix that. Signed-off-by: Ian Jackson --- Osstest/Serial/sympathy.pm |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a

[Xen-devel] [PATCH 2/8] selecthost: Honour IDENT_hostflagadjust runvar

2017-04-10 Thread Ian Jackson
This allows runvars to override hostflags from the resource database or configuration. Signed-off-by: Ian Jackson --- Osstest/TestSupport.pm |8 1 file changed, 8 insertions(+) diff --git a/Osstest/TestSupport.pm b/Osstest/TestSupport.pm index a498ddd..88b0a65 100644 --- a/Osstest

Re: [Xen-devel] [linux-arm-xen test] 107256: trouble: broken/pass

2017-04-10 Thread Ian Jackson
Julien Grall writes ("Re: [Xen-devel] [linux-arm-xen test] 107256: trouble: broken/pass"): > IIRC Ian decommissioned the Arndale on Friday on my request because they > are too unreliable for testing (we are using a dongle for network that > often fail). Unfortunately, I have found that decommis

Re: [Xen-devel] [xen-4.8-testing test] 107259: trouble: blocked/broken/fail/pass

2017-04-10 Thread Ian Jackson
osstest service owner writes ("[xen-4.8-testing test] 107259: trouble: blocked/broken/fail/pass"): > flight 107259 xen-4.8-testing real [real] > http://logs.test-lab.xenproject.org/osstest/logs/107259/ > > Failures and problems with tests :-( > > Tests which did not succeed and are blocking, > i

[Xen-devel] [OSSTEST PATCH] cri-getplatforms: Require a tested platform to have some purpose-test hosts

2017-04-07 Thread Ian Jackson
Signed-off-by: Ian Jackson --- Osstest/HostDB/Executive.pm | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/Osstest/HostDB/Executive.pm b/Osstest/HostDB/Executive.pm index 3e5eca2..300178b 100644 --- a/Osstest/HostDB/Executive.pm +++ b/Osstest/HostDB/Executive.pm

[Xen-devel] Please apply "partially revert "xen: Remove event channel..."

2017-04-07 Thread Ian Jackson
tl;dr: Please apply da72ff5bfcb02c6ac8b169a7cf597a3c8e6c4de1 partially revert "xen: Remove event channel notification through Xen PCI platform device" to all stable branches which have a version of the original broken commit. This includes at least 4.9.y. Background: osstest s

Re: [Xen-devel] [OSSTEST PATCH v9 1/3] ts-openstack-deploy: Deploy OpenStack on a host with devstack

2017-04-07 Thread Ian Jackson
Anthony PERARD writes ("Re: [OSSTEST PATCH v9 1/3] ts-openstack-deploy: Deploy OpenStack on a host with devstack"): > On Thu, Apr 06, 2017 at 05:32:32PM +0100, Ian Jackson wrote: > > Also, it would be worth grepping for other occurrences of target_cmd > > with missing s

Re: [Xen-devel] [OSSTEST PATCH] ts-xtf-run: Understand ./xtf-runner returning CRASH

2017-04-07 Thread Ian Jackson
Andrew Cooper writes ("Re: [OSSTEST PATCH] ts-xtf-run: Understand ./xtf-runner returning CRASH"): > Ping on this getting accepted? I have some XTF improvements queued > behind not breaking OSSTest. Sorry for dropping this. I have queued it up and will push it with the TLS MITM series. Ian. __

[Xen-devel] [OSSTEST PATCH 2/4] proxy config: Add ability to install MITM TLS cert

2017-04-06 Thread Ian Jackson
lling the cert on each test box. (The squid becomes part of the TCB for our coverity upload password, but that is fine.) Signed-off-by: Ian Jackson --- Osstest/TestSupport.pm | 10 ++ 1 file changed, 10 insertions(+) diff --git a/Osstest/TestSupport.pm b/Osstest/TestSupport.pm index 1c

[Xen-devel] [OSSTEST PATCH 3/4] proxy config: Add MITM cert for Massachusetts

2017-04-06 Thread Ian Jackson
Signed-off-by: Ian Jackson --- production-config | 25 + 1 file changed, 25 insertions(+) diff --git a/production-config b/production-config index a95e243..07cf5a6 100644 --- a/production-config +++ b/production-config @@ -140,3 +140,28 @@ ssh-rsa

[Xen-devel] [OSSTEST PATCH 4/4] proxy config: Set https_proxy too

2017-04-06 Thread Ian Jackson
Clients including curl honour https_proxy (not http_proxy) in the environment for https:// urls. Signed-off-by: Ian Jackson --- Osstest/TestSupport.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Osstest/TestSupport.pm b/Osstest/TestSupport.pm index ac9726c..4ced6c0

[Xen-devel] [OSSTEST PATCH 1/4] proxy config: Factor out http_proxy_envsettings

2017-04-06 Thread Ian Jackson
No overall functional change, other than slight syntax changes to generated shell runes. Signed-off-by: Ian Jackson --- Osstest/TestSupport.pm | 22 -- 1 file changed, 16 insertions(+), 6 deletions(-) diff --git a/Osstest/TestSupport.pm b/Osstest/TestSupport.pm index

Re: [Xen-devel] [OSSTEST PATCH v9 1/3] ts-openstack-deploy: Deploy OpenStack on a host with devstack

2017-04-06 Thread Ian Jackson
Anthony PERARD writes ("[OSSTEST PATCH v9 1/3] ts-openstack-deploy: Deploy OpenStack on a host with devstack"): > This script installs any necessary packages and clones all of the OpenStack > trees which are used by devstack to deploy OpenStack. > > Signed-off-by: Anthony P

Re: [Xen-devel] [PATCH v1 0/2] libxl: add PV display device driver interface

2017-04-06 Thread Ian Jackson
Oleksandr Grytsov writes ("Re: [Xen-devel] [PATCH v1 0/2] libxl: add PV display device driver interface"): > On Thu, Apr 6, 2017 at 2:17 PM, Ian Jackson wrote: > > Oleksandr Grytsov writes ("Re: [Xen-devel] [PATCH v1 0/2] libxl: add PV > > display device driver i

Re: [Xen-devel] [PATCH v1 0/2] libxl: add PV display device driver interface

2017-04-06 Thread Ian Jackson
Oleksandr Grytsov writes ("Re: [Xen-devel] [PATCH v1 0/2] libxl: add PV display device driver interface"): > Backend and frontend operate with connectors (virtual displays). They > are identified by connector index. > The virtual display layouts, orders, on which physical display they > are locate

Re: [Xen-devel] [PATCH v2 for-4.9 1/7] MAINTAINERS: Move the x86 instruction emulator under x86 maintainership

2017-04-06 Thread Ian Jackson
Andrew Cooper writes ("[PATCH v2 for-4.9 1/7] MAINTAINERS: Move the x86 instruction emulator under x86 maintainership"): > Requested-by: Jan Beulich > Signed-off-by: Andrew Cooper Acked-by: Ian Jackson ___ Xen-devel mailin

Re: [Xen-devel] [PATCHv2] libxc: fix segfault on uninitialized xch->fmem

2017-04-05 Thread Ian Jackson
Wei Liu writes ("Re: [PATCHv2] libxc: fix segfault on uninitialized xch->fmem"): > Ian, this is a backport candidate. Queued, thanks (until after 4.8.1, given that currently this is in staging only). Ian. ___ Xen-devel mailing list Xen-devel@lists.xen.

Re: [Xen-devel] [PATCH v1 0/2] libxl: add PV display device driver interface

2017-04-05 Thread Ian Jackson
Oleksandr Grytsov writes ("Re: [Xen-devel] [PATCH v1 0/2] libxl: add PV display device driver interface"): > On Wed, Apr 5, 2017 at 5:28 PM, Ian Jackson wrote: > > Who starts the display backend ? I'm obviously missing some of the > > plumbing. Sorry again for a

Re: [Xen-devel] [PATCH v1 0/2] libxl: add PV display device driver interface

2017-04-05 Thread Ian Jackson
Oleksandr Grytsov writes ("Re: [Xen-devel] [PATCH v1 0/2] libxl: add PV display device driver interface"): > The display backend is a user-space application which is DRM or > Wayland client. Generally it can be run on any domain. Who starts the display backend ? I'm obviously missing some of th

[Xen-devel] [PATCH] xenstore: dont increment bool variable

2017-04-05 Thread Ian Jackson
Juergen Gross writes ("[PATCH] xenstore: dont increment bool variable"): > Instead of incrementing a bool variable just set it to true. Acked-by: Ian Jackson I will commit this to staging. Also, I will squash it into the XSA206 fixes as I push them to the stable b

Re: [Xen-devel] [PATCH v1 0/2] libxl: add PV display device driver interface

2017-04-05 Thread Ian Jackson
al1img . writes ("Re: [PATCH v1 0/2] libxl: add PV display device driver interface"): > On Wed, Apr 5, 2017 at 2:05 PM, Ian Jackson wrote: > > Sorry to show my ignorance, but what is a "PV display device" ? > > Where is the backend ? What protocol do the front

Re: [Xen-devel] [linux-arm-xen test] 107164: regressions - FAIL

2017-04-05 Thread Ian Jackson
Stefano Stabellini writes ("Re: [Xen-devel] [linux-arm-xen test] 107164: regressions - FAIL"): > On Tue, 4 Apr 2017, Ian Jackson wrote: > > 2. When that passes the push gate (now done) and is being used > > everywhere (not quite yet), we change the XenDTUARTPath settin

Re: [Xen-devel] [PATCH] tools/xenstore: Add -Wno-bool-operation

2017-04-05 Thread Ian Jackson
Andrew Cooper writes ("Re: [Xen-devel] [PATCH] tools/xenstore: Add -Wno-bool-operation"): > I'm fairly sure this will break the Clang build. This seems unlikely. I assume that clang has the same behaviour as gcc, for -Wno-unknown-warning. > Furthermore, the proper fix here is to make less "clev

Re: [Xen-devel] [PATCH v2 1/5] tools/libxenctrl: fix error check after opening libxenforeignmemory

2017-04-05 Thread Ian Jackson
Wei Liu writes ("Re: [PATCH v2 1/5] tools/libxenctrl: fix error check after opening libxenforeignmemory"): > On Wed, Feb 22, 2017 at 01:27:34PM +, Paul Durrant wrote: > > Checking the value of xch->xcall is clearly incorrect. The code should be > > checking xch->fmem (i.e. the return of the pr

Re: [Xen-devel] Backport request

2017-04-05 Thread Ian Jackson
Juergen Gross writes ("Backport request"): > I'd like to request backport of upstream commit bc6a499ebcf1 > ("libxl: correct xenstore entry for empty cdrom") to at least > 4.8 and 4.7. > > As libxl sources have been split in 4.9 the patch doesn't apply. > Attached patch applies cleanly to 4.8 and

[Xen-devel] [PATCH] tools/xenstore: Add -Wno-bool-operation

2017-04-05 Thread Ian Jackson
ly the XSA-206 patches to stable trees. In the stable trees, this should be applied _before_ the XSA-206 series. Reported-by: Michael Young CC: Wei Liu CC: Jan Beulich Signed-off-by: Ian Jackson --- tools/xenstore/Makefile | 1 + 1 file changed, 1 insertion(+) diff --git a/tools/xenstore/

Re: [Xen-devel] Xen Security Advisory 206 - xenstore denial of service via repeated update

2017-04-05 Thread Ian Jackson
M A Young writes ("Re: [Xen-devel] Xen Security Advisory 206 - xenstore denial of service via repeated update"): > On Tue, 4 Apr 2017, Ian Jackson wrote: > > Michael, can you confirm whether the patch below fixes it for you ? > > > > If so we will commit it to st

Re: [Xen-devel] [PATCH v1 0/2] libxl: add PV display device driver interface

2017-04-05 Thread Ian Jackson
Oleksandr Grytsov writes ("[PATCH v1 0/2] libxl: add PV display device driver interface"): > From: Oleksandr Grytsov > > These patches add PV display device to libxl and xl. > > Changes since initial: > * re-work libxl displ related functions to use libxl_device_type > > Oleksandr Grytsov (2)

Re: [Xen-devel] Xen Security Advisory 206 - xenstore denial of service via repeated update

2017-04-04 Thread Ian Jackson
Ian Jackson writes ("Re: [Xen-devel] Xen Security Advisory 206 - xenstore denial of service via repeated update"): > > [-Werror=bool-operation] > > if (!domain->wrl_delay_logged++) { > > I think this warning is wrong. ... > It's a shame that we can'

Re: [Xen-devel] [linux-arm-xen test] 107164: regressions - FAIL

2017-04-04 Thread Ian Jackson
Julien Grall writes ("Re: [Xen-devel] [linux-arm-xen test] 107164: regressions - FAIL"): > On 04/04/2017 04:14 AM, osstest service owner wrote: > > flight 107164 linux-arm-xen real [real] > > http://logs.test-lab.xenproject.org/osstest/logs/107164/ ... > > test-armhf-armhf-libvirt-xsm 13 saverest

[Xen-devel] [OSSTEST PATCH] branches: add linux-4.9

2017-04-04 Thread Ian Jackson
Signed-off-by: Ian Jackson --- cr-for-branches | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cr-for-branches b/cr-for-branches index 48576d8..ab080d7 100755 --- a/cr-for-branches +++ b/cr-for-branches @@ -31,7 +31,7 @@ scriptoptions="$1"; shift LOGFILE=

Re: [Xen-devel] [PATCH] tools: ocaml: In configure, check for ocamlopt

2017-04-03 Thread Ian Jackson
Ian Jackson writes ("Re: [PATCH] tools: ocaml: In configure, check for ocamlopt"): > Wei Liu writes ("Re: [PATCH] tools: ocaml: In configure, check for ocamlopt"): > > On Mon, Apr 03, 2017 at 12:34:13PM +0100, Ian Jackson wrote: > > > CC: Julien Grall

Re: [Xen-devel] [PATCH] tools: ocaml: In configure, check for ocamlopt

2017-04-03 Thread Ian Jackson
Wei Liu writes ("Re: [PATCH] tools: ocaml: In configure, check for ocamlopt"): > On Mon, Apr 03, 2017 at 12:34:13PM +0100, Ian Jackson wrote: > > CC: Julien Grall > > CC: Christian Lindig > > CC: Jonathan Ludlam > > CC: David Scott > > CC: Wei Liu &g

[Xen-devel] [PATCH 0/2] xensore: fixing two bugs

2017-04-03 Thread Ian Jackson
ct error code when violating quota > xenstore: add missing checks for allocation failure Both, Acked-by: Ian Jackson ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

[Xen-devel] [PATCH v3 0/4] xenstore: rework of transaction handling

2017-04-03 Thread Ian Jackson
to XSA 206 which has been disclosed only today. So V1 > and V2 have been non-public in order to speed up review process without > disclosing the XSA. I meant to look over these again and ack them. I see they have been committed, but FTR: Acked-by: Ian Jackson __

[Xen-devel] [PATCH] tools: ocaml: In configure, check for ocamlopt

2017-04-03 Thread Ian Jackson
Christian Lindig CC: Jonathan Ludlam CC: David Scott CC: Wei Liu Tested-by: Ian Jackson Signed-off-by: Ian Jackson --- tools/configure| 4 ++-- tools/configure.ac | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/tools/configure b/tools/configure index 7a57e65..6ea8

Re: [Xen-devel] Xen Security Advisory 206 - xenstore denial of service via repeated update

2017-03-31 Thread Ian Jackson
(dropping some of the lists) Michael Young writes ("Re: [Xen-devel] Xen Security Advisory 206 - xenstore denial of service via repeated update"): > On Wed, 29 Mar 2017, Xen.org security team wrote: > >Xen Security Advisory XSA-206 > > version 9 > >

Re: [Xen-devel] [PATCH] xl: Add 'pvh' config option

2017-03-31 Thread Ian Jackson
Roger Pau Monné writes ("Re: [PATCH] xl: Add 'pvh' config option"): > On Fri, Mar 31, 2017 at 09:59:09AM -0400, Boris Ostrovsky wrote: > > That would make the toolstack have three guest types while the > > hypervisor has two. Is that a problem, conceptually ? It seems to me to be accurate here.

<    4   5   6   7   8   9   10   11   12   13   >