Re: [Xen-devel] [PATCH 4/6] x86/cpuid: Handle leaf 0x80000000 in guest_cpuid()

2017-01-20 Thread Andrew Cooper
On 20/01/17 15:58, Jan Beulich wrote: On 18.01.17 at 20:40, wrote: >> --- a/xen/arch/x86/cpuid.c >> +++ b/xen/arch/x86/cpuid.c >> @@ -163,6 +163,24 @@ static void recalculate_xstate(struct cpuid_policy *p) >> } >> } >> >> +static void

Re: [Xen-devel] [PATCH 5/6] x86/cpuid: Handle the long vendor string in guest_cpuid()

2017-01-20 Thread Jan Beulich
>>> On 18.01.17 at 20:40, wrote: > Leaves 0x8002 through 0x8004 are plain ASCII text, and require no > specific recalculation. Do they not? We don't currently get them in line with a perhaps overridden vendor, but maybe we should? > @@ -936,6 +938,7 @@ void

Re: [Xen-devel] [PATCH v4 1/8] public / x86: Introduce __HYPERCALL_dm_op...

2017-01-20 Thread Paul Durrant
> -Original Message- [snip] > > + > > +va_start(args, nr_bufs); > > +for (idx = 0; idx < nr_bufs; idx++) > > Coding style. Ah, yes. > > > + > > +int compat_dm_op(domid_t domid, > > + unsigned int nr_bufs, > > +

Re: [Xen-devel] [PATCH 4/6] x86/cpuid: Handle leaf 0x80000000 in guest_cpuid()

2017-01-20 Thread Jan Beulich
>>> On 18.01.17 at 20:40, wrote: > --- a/xen/arch/x86/cpuid.c > +++ b/xen/arch/x86/cpuid.c > @@ -163,6 +163,24 @@ static void recalculate_xstate(struct cpuid_policy *p) > } > } > > +static void recalculate_common(struct cpuid_policy *p) > +{ > +switch (

Re: [Xen-devel] memory hotplug for domUs

2017-01-20 Thread Ian Jackson
Juergen Gross writes ("memory hotplug for domUs"): > We first thought to enhance "xl mem-set", but after some more thinking > about it I'd rather add a new xl command, e.g. "mem-add" (we could later > even add "mem-remove" to support memory unplug). Why ? Why would xl mem-set not automatically

Re: [Xen-devel] [PATCH v4 1/8] public / x86: Introduce __HYPERCALL_dm_op...

2017-01-20 Thread Wei Liu
On Tue, Jan 17, 2017 at 05:29:49PM +, Paul Durrant wrote: > ...as a set of hypercalls to be used by a device model. > > As stated in the new docs/designs/dm_op.markdown: > > "The aim of DMOP is to prevent a compromised device model from > compromising domains other then the one it is

Re: [Xen-devel] [PATCH] docs: clarify xl mem-max semantics

2017-01-20 Thread Ian Jackson
Konrad Rzeszutek Wilk writes ("Re: [Xen-devel] [PATCH] docs: clarify xl mem-max semantics"): > On Fri, Jan 20, 2017 at 10:54:18AM +0100, Juergen Gross wrote: > > The mem-max value may not correspond to the actual memory used in the > > domain, as it may balloon down its memory to give more back

Re: [Xen-devel] [PATCH] x86/emul: Fix clang build following BMI1/BMI2/TBM instruction support

2017-01-20 Thread Jan Beulich
>>> On 20.01.17 at 16:37, wrote: > Travis reports that Clang objects to integer truncation during assignments to > a bitfield: > > ./x86_emulate/x86_emulate.c:6150:19: error: implicit truncation from 'int' > to bitfield changes value from -1 to 15 >

Re: [Xen-devel] [PATCH 3/6] x86/cpuid: Handle leaf 0 in guest_cpuid()

2017-01-20 Thread Jan Beulich
>>> On 18.01.17 at 20:40, wrote: > Calculate a domains x86_vendor early in recalculate_cpuid_policy(); subsequent > patches need to make other recalculation decisions based on it. > > Signed-off-by: Andrew Cooper Reviewed-by: Jan Beulich

[Xen-devel] [PATCH 0/2] ocaml: Start on fixing brokenness when no ocamlopt

2017-01-20 Thread Ian Jackson
Debian jessie arm64 has Ocaml (in the package `ocaml-nox') but the package lacks ocamlopt. This causes xen.git builds to fail. Our configure script detects that ocaml is available. Our version of m4/ocaml.m4 sort of tries to figure out what the best compiler is but only does have the job. And

[Xen-devel] [PATCH 1/2] ocaml: Copy up-to-date m4/ocaml.m4 from forge.ocamlcore

2017-01-20 Thread Ian Jackson
From https://forge.ocamlcore.org/anonscm/git/ocaml-autoconf/ocaml-autoconf.git commit id a20a590beb523fb64bf087b52a883cf628256d2f Copy ocaml.m4, and also LICENSE to m4/ocaml.m4.LICENSE. (This is a permissive licence and thus compatible with the licence of Xen.) We mostly want this because

[Xen-devel] [PATCH 2/2] ocaml: Substitute tools/ocaml/common.make with configure

2017-01-20 Thread Ian Jackson
Rename tools/ocaml/common.make to .in, and add it to the list of files to generate. Add it to .gitignore. Signed-off-by: Ian Jackson --- .gitignore | 1 + tools/configure.ac | 1 + tools/ocaml/common.make| 22 --

Re: [Xen-devel] [PATCH 2/6] x86/cpuid: Remove BUG_ON() condition from guest_cpuid()

2017-01-20 Thread Jan Beulich
>>> On 18.01.17 at 20:40, wrote: > Include a min() against the appropriate ARRAY_SIZE(), and ASSERT() that > max_subleaf is within ARRAY_SIZE(). > > This is more robust to unexpected problems in a release build of Xen. > > Signed-off-by: Andrew Cooper

Re: [Xen-devel] [PATCH 1/6] x86/cpuid: Hide VT-x/SVM from HVM-based control domains

2017-01-20 Thread Jan Beulich
>>> On 18.01.17 at 20:40, wrote: > The VT-x/SVM features are hidden from PV dom0 by the pv_featureset[] upper > mask, but nothing thusfar has prevented the features being visible in > HVM-based control domains (where there is no toolstack decision to hide the >

[Xen-devel] [PATCH] x86/emul: Fix clang build following BMI1/BMI2/TBM instruction support

2017-01-20 Thread Andrew Cooper
Travis reports that Clang objects to integer truncation during assignments to a bitfield: ./x86_emulate/x86_emulate.c:6150:19: error: implicit truncation from 'int' to bitfield changes value from -1 to 15 [-Werror,-Wbitfield-constant-conversion] pxop->reg = ~0; /* rAX */

[Xen-devel] [xen-unstable-smoke test] 104349: tolerable all pass - PUSHED

2017-01-20 Thread osstest service owner
flight 104349 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/104349/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-amd64-amd64-libvirt 12 migrate-support-checkfail never pass test-armhf-armhf-xl

Re: [Xen-devel] [PATCH v12 05/10] x86: add multiboot2 protocol support for EFI platforms

2017-01-20 Thread Jan Beulich
>>> On 20.01.17 at 15:43, wrote: > On Fri, Jan 20, 2017 at 07:10:32AM -0700, Jan Beulich wrote: >> >>> On 20.01.17 at 14:46, wrote: >> > On Fri, Jan 20, 2017 at 05:40:55AM -0700, Jan Beulich wrote: >> >> >>> On 20.01.17 at 12:43,

Re: [Xen-devel] [PATCH v4 2/8] dm_op: convert HVMOP_*ioreq_server*

2017-01-20 Thread Andrew Cooper
On 17/01/17 17:29, Paul Durrant wrote: > The definitions of HVM_IOREQSRV_BUFIOREQ_* have to persist as they are > already in use by callers of the libxc interface. > > Suggested-by: Jan Beulich > Signed-off-by: Paul Durrant Reviewed-by: Andrew Cooper

Re: [Xen-devel] [PATCH v4 1/8] public / x86: Introduce __HYPERCALL_dm_op...

2017-01-20 Thread Paul Durrant
> -Original Message- > From: Andrew Cooper > Sent: 20 January 2017 14:35 > To: Paul Durrant ; xen-de...@lists.xenproject.org > Cc: Ian Jackson ; Jennifer Herbert > ; Daniel De Graaf ; > Wei

Re: [Xen-devel] [PATCH 1/1] kexec: ensure kexec_status() return bit value of 0 or 1

2017-01-20 Thread Andrew Cooper
On 19/01/17 17:10, Eric DeVolder wrote: > When checking kexec_flags bit corresponding to the > requested image, ensure that 0 or 1 is returned. > > Signed-off-by: Eric DeVolder Reviewed-by: Andrew Cooper and committed. I dropped the commit

[Xen-devel] [libvirt test] 104327: tolerable all pass - PUSHED

2017-01-20 Thread osstest service owner
flight 104327 libvirt real [real] http://logs.test-lab.xenproject.org/osstest/logs/104327/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-armhf-armhf-libvirt 13 saverestore-support-checkfail like 104283 test-armhf-armhf-libvirt-qcow2

Re: [Xen-devel] [PATCH v12 05/10] x86: add multiboot2 protocol support for EFI platforms

2017-01-20 Thread Daniel Kiper
On Fri, Jan 20, 2017 at 07:10:32AM -0700, Jan Beulich wrote: > >>> On 20.01.17 at 14:46, wrote: > > On Fri, Jan 20, 2017 at 05:40:55AM -0700, Jan Beulich wrote: > >> >>> On 20.01.17 at 12:43, wrote: > >> > On Fri, Jan 20, 2017 at 02:46:47AM

Re: [Xen-devel] memory hotplug for domUs

2017-01-20 Thread Boris Ostrovsky
On 01/20/2017 09:27 AM, Roger Pau Monné wrote: > On Fri, Jan 20, 2017 at 09:19:48AM -0500, Boris Ostrovsky wrote: For HVM/PVH guests however, ACPI Memory Hotplug. That way it will work for all operating systems, not just Linux. (Lets please not waste time building a

Re: [Xen-devel] [PATCH v4 1/8] public / x86: Introduce __HYPERCALL_dm_op...

2017-01-20 Thread Andrew Cooper
On 17/01/17 17:29, Paul Durrant wrote: > ...as a set of hypercalls to be used by a device model. > > As stated in the new docs/designs/dm_op.markdown: > > "The aim of DMOP is to prevent a compromised device model from > compromising domains other then the one it is associated with. (And is >

Re: [Xen-devel] memory hotplug for domUs

2017-01-20 Thread Juergen Gross
On 20/01/17 15:27, Roger Pau Monné wrote: > On Fri, Jan 20, 2017 at 09:19:48AM -0500, Boris Ostrovsky wrote: >> For HVM/PVH guests however, ACPI Memory Hotplug. That way it will work for all operating systems, not just Linux. (Lets please not waste time building a somewhat-square

Re: [Xen-devel] memory hotplug for domUs

2017-01-20 Thread Roger Pau Monné
On Fri, Jan 20, 2017 at 09:19:48AM -0500, Boris Ostrovsky wrote: > > >> For HVM/PVH guests however, ACPI Memory Hotplug. That way it will work > >> for all operating systems, not just Linux. (Lets please not waste time > >> building a somewhat-square wheel which only fits on a Linux shaped axle

Re: [Xen-devel] [RFC] Device memory mappings for Dom0 on ARM64 ACPI systems

2017-01-20 Thread Roger Pau Monné
On Fri, Jan 20, 2017 at 02:10:33PM +0100, Julien Grall wrote: > Hi Royger, > > On 20/01/2017 12:01, Roger Pau Monné wrote: > > On Thu, Jan 19, 2017 at 09:14:03PM +0100, Julien Grall wrote: > In case of ARM, Xen does not use any PCI devices (no PCI UART) itself so > scanning before hand is not

Re: [Xen-devel] memory hotplug for domUs

2017-01-20 Thread Boris Ostrovsky
>> For HVM/PVH guests however, ACPI Memory Hotplug. That way it will work >> for all operating systems, not just Linux. (Lets please not waste time >> building a somewhat-square wheel which only fits on a Linux shaped axle :) ) What about PVH dom0? I suspect we will have the same problem with

Re: [Xen-devel] [PATCH v12 05/10] x86: add multiboot2 protocol support for EFI platforms

2017-01-20 Thread Jan Beulich
>>> On 20.01.17 at 14:46, wrote: > On Fri, Jan 20, 2017 at 05:40:55AM -0700, Jan Beulich wrote: >> >>> On 20.01.17 at 12:43, wrote: >> > On Fri, Jan 20, 2017 at 02:46:47AM -0700, Jan Beulich wrote: >> >> >>> On 20.01.17 at 02:34,

Re: [Xen-devel] memory hotplug for domUs

2017-01-20 Thread Konrad Rzeszutek Wilk
On Fri, Jan 20, 2017 at 11:08:54AM +, Andrew Cooper wrote: > On 20/01/17 09:26, Juergen Gross wrote: Cc-ing Daniel as he wrote the Xen Balloon memory hotplug code. > > Recently Jim asked me why he can use "xl mem-max" to raise the allowed > > memory size of a domain in the hypervisor above

Re: [Xen-devel] [PATCH] docs: clarify xl mem-max semantics

2017-01-20 Thread Konrad Rzeszutek Wilk
On Fri, Jan 20, 2017 at 10:54:18AM +0100, Juergen Gross wrote: > The information given in the xl man page for the mem-max command is > rather brief. Expand it in order to let the reader understand what it > is really doing. > > As the related libxl function libxl_domain_setmaxmem() isn't much >

Re: [Xen-devel] [PATCH] swiotlb-xen: update dev_addr after swapping pages

2017-01-20 Thread Konrad Rzeszutek Wilk
On Thu, Jan 19, 2017 at 04:29:19PM -0800, Stefano Stabellini wrote: > On Thu, 19 Jan 2017, Konrad Rzeszutek Wilk wrote: > > On Thu, Jan 19, 2017 at 06:58:46PM -0500, Boris Ostrovsky wrote: > > > On 01/19/2017 01:39 PM, Stefano Stabellini wrote: > > > > In xen_swiotlb_map_page and

Re: [Xen-devel] [PATCH v12 05/10] x86: add multiboot2 protocol support for EFI platforms

2017-01-20 Thread Daniel Kiper
On Fri, Jan 20, 2017 at 05:40:55AM -0700, Jan Beulich wrote: > >>> On 20.01.17 at 12:43, wrote: > > On Fri, Jan 20, 2017 at 02:46:47AM -0700, Jan Beulich wrote: > >> >>> On 20.01.17 at 02:34, wrote: [...] > >> > +.Lefi_multiboot2_proto: > >> >

[Xen-devel] [linux-3.18 baseline-only test] 68400: trouble: blocked/broken

2017-01-20 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 68400 linux-3.18 real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/68400/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: build-armhf-xsm

Re: [Xen-devel] [RFC] Device memory mappings for Dom0 on ARM64 ACPI systems

2017-01-20 Thread Julien Grall
Hi Royger, On 20/01/2017 12:01, Roger Pau Monné wrote: On Thu, Jan 19, 2017 at 09:14:03PM +0100, Julien Grall wrote: On 19/01/2017 19:22, Stefano Stabellini wrote: On Thu, 19 Jan 2017, Roger Pau Monné wrote: On Wed, Jan 18, 2017 at 07:13:23PM +, Julien Grall wrote: Hi, On 18/01/17

Re: [Xen-devel] [PATCH v2] x86: segment attribute handling adjustments

2017-01-20 Thread Andrew Cooper
On 20/01/17 08:26, Jan Beulich wrote: > Null selector loads into SS (possible in 64-bit mode only, and only in > rings other than ring 3) must not alter SS.DPL. (This was found to be > an issue on KVM, and fixed in Linux commit 33ab91103b.) > > Further arch_set_info_hvm_guest() didn't make sure

Re: [Xen-devel] [PATCH v12 05/10] x86: add multiboot2 protocol support for EFI platforms

2017-01-20 Thread Jan Beulich
>>> On 20.01.17 at 12:43, wrote: > On Fri, Jan 20, 2017 at 02:46:47AM -0700, Jan Beulich wrote: >> >>> On 20.01.17 at 02:34, wrote: >> > @@ -100,20 +107,48 @@ multiboot2_header_start: >> > gdt_boot_descr: >> > .word 6*8-1 >> >

[Xen-devel] [OSSTEST PATCH 27/35] netboot: UEFI grub: Do it more like pxe

2017-01-20 Thread Ian Jackson
* Abolish setup_grub_efi_bootcfg and use setup_netboot_bootcfg instead. The latter function now expects to be given a config file for the host's bootloader. For a UEFI host, that's a grub config file, which it will write to the configured NetGrub location for the specific host. *

[Xen-devel] [OSSTEST PATCH 28/35] netboot: mg-netgrub-loader-update: Print sample dhcp config fragment

2017-01-20 Thread Ian Jackson
Signed-off-by: Ian Jackson --- mg-netgrub-loader-update | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/mg-netgrub-loader-update b/mg-netgrub-loader-update index 21157a2..73deb19 100755 --- a/mg-netgrub-loader-update +++

[Xen-devel] [OSSTEST PATCH 35/35] ARM64: Create jobs, and provide the installer (with backports kernel)

2017-01-20 Thread Ian Jackson
Signed-off-by: Ian Jackson --- make-flight| 9 ++--- mfi-common | 27 +++ mg-debian-installer-update | 1 + 3 files changed, 30 insertions(+), 7 deletions(-) diff --git a/make-flight b/make-flight index

[Xen-devel] [OSSTEST PATCH 32/35] d-i: backports: When using backports kernel for a host, install linux-base

2017-01-20 Thread Ian Jackson
Newer kernels may need a newer linux-base. This call has the happy side effect of adding backports to its sources.list, which is also generally needed. Signed-off-by: Ian Jackson --- Osstest/Debian.pm | 5 + 1 file changed, 5 insertions(+) diff --git

[Xen-devel] [OSSTEST PATCH 24/35] netboot: Change stash file name to not be pxelinux-specific

2017-01-20 Thread Ian Jackson
This will be wrong when this is used for netgrub too. Only functional change is to log outputs. 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

[Xen-devel] [OSSTEST PATCH 34/35] d-i: Add comment about interrupting the install

2017-01-20 Thread Ian Jackson
Signed-off-by: Ian Jackson --- Osstest/Debian.pm | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Osstest/Debian.pm b/Osstest/Debian.pm index 06918ef..acc4618 100644 --- a/Osstest/Debian.pm +++ b/Osstest/Debian.pm @@ -890,6 +890,8 @@ d-i grub-installer/only_debian

[Xen-devel] [OSSTEST PATCH 21/35] netboot: Rename ./mg-hosts mknetbootdir subcommand

2017-01-20 Thread Ian Jackson
Was ./mg-hosts mkpxedir, but it is more general. Signed-off-by: Ian Jackson --- README | 4 ++-- README.dev | 2 +- mg-hosts | 4 ++-- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/README b/README index 01644b0..91d0328 100644 --- a/README +++

[Xen-devel] [OSSTEST PATCH 26/35] mg-netgrub-loader-update: Introduce $img

2017-01-20 Thread Ian Jackson
No functional change. Signed-off-by: Ian Jackson --- mg-netgrub-loader-update | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/mg-netgrub-loader-update b/mg-netgrub-loader-update index 9eb20d5..ec6dfd3 100755 --- a/mg-netgrub-loader-update +++

[Xen-devel] [OSSTEST PATCH 19/35] netboot: Change pxe to netboot in variable names

2017-01-20 Thread Ian Jackson
No functional change. Signed-off-by: Ian Jackson --- Osstest/TestSupport.pm | 6 +++--- mg-hosts | 14 +++--- 2 files changed, 10 insertions(+), 10 deletions(-) diff --git a/Osstest/TestSupport.pm b/Osstest/TestSupport.pm index fde796a..bb72fe5

[Xen-devel] [OSSTEST PATCH 31/35] d-i: preseed_create_guest: Break out preseed_backports_packages

2017-01-20 Thread Ian Jackson
No functional change. Signed-off-by: Ian Jackson --- Osstest/Debian.pm | 39 +++ 1 file changed, 27 insertions(+), 12 deletions(-) diff --git a/Osstest/Debian.pm b/Osstest/Debian.pm index 4c73318..f496354 100644 ---

[Xen-devel] [OSSTEST PATCH 33/35] d-i: When using backports kernel, suppress kernel modules warning

2017-01-20 Thread Ian Jackson
Signed-off-by: Ian Jackson --- Osstest/Debian.pm | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/Osstest/Debian.pm b/Osstest/Debian.pm index 0137fc4..06918ef 100644 --- a/Osstest/Debian.pm +++ b/Osstest/Debian.pm @@ -1124,12 +1124,15 @@

[Xen-devel] [OSSTEST PATCH 25/35] mg-netgrub-loader-update: Introduce $grubbase

2017-01-20 Thread Ian Jackson
No functional change. Signed-off-by: Ian Jackson --- mg-netgrub-loader-update | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/mg-netgrub-loader-update b/mg-netgrub-loader-update index ca8abcb..9eb20d5 100755 --- a/mg-netgrub-loader-update +++

[Xen-devel] [OSSTEST PATCH 29/35] netboot: Rename TftpNetbootGroup (from TftpPxeGroup)

2017-01-20 Thread Ian Jackson
This applies to NetGrub too. Honour the old setting for compatibility. Signed-off-by: Ian Jackson --- Osstest/TestSupport.pm | 6 -- README | 2 +- mg-hosts| 2 +- production-config | 2 +-

[Xen-devel] [OSSTEST PATCH 30/35] netboot: mg-netgrub-loader-update: Add some more docs

2017-01-20 Thread Ian Jackson
Signed-off-by: Ian Jackson --- mg-netgrub-loader-update | 12 1 file changed, 12 insertions(+) diff --git a/mg-netgrub-loader-update b/mg-netgrub-loader-update index 73deb19..22ef67c 100755 --- a/mg-netgrub-loader-update +++ b/mg-netgrub-loader-update @@

[Xen-devel] [OSSTEST PATCH 23/35] netboot: Introduce TftpNetGrub*

2017-01-20 Thread Ian Jackson
* When expanding netboot filenames, look for NetGrub config keys rather than Pxe filenames. Currently this only affects mg-hosts because the files are created by setup_grub_efi_bootcfg, and the files are of course referred to only by the actual netboot grub image (which may be made by

[Xen-devel] [OSSTEST PATCH 22/35] netboot: Break out subst_netboot_template

2017-01-20 Thread Ian Jackson
No functional change. Signed-off-by: Ian Jackson --- Osstest/TestSupport.pm | 31 +++ 1 file changed, 19 insertions(+), 12 deletions(-) diff --git a/Osstest/TestSupport.pm b/Osstest/TestSupport.pm index bb72fe5..990ddbc 100644 ---

[Xen-devel] [OSSTEST PATCH 20/35] netboot: Rename mg-netgrub-loader-update

2017-01-20 Thread Ian Jackson
This used to be mg-pxe-loader-update, but it updates grub and not anything to do with PXE. Signed-off-by: Ian Jackson --- README | 2 +- mg-netgrub-loader-update | 84 mg-pxe-loader-update | 84

[Xen-devel] [OSSTEST PATCH 18/35] netboot: Change pxe to netboot in comments, docs and messages

2017-01-20 Thread Ian Jackson
UEFI grub netbooting is not PXE. Indeed PXE is a BIOS thing only. No functional change other than to messages. Signed-off-by: Ian Jackson --- INSTALL.production | 10 +- Osstest/Debian.pm | 2 +- Osstest/TestSupport.pm | 6 +++--- README

[Xen-devel] [OSSTEST PATCH 13/35] d-i: Cope if .ssh already exists (in /target/root/ and also /home/...)

2017-01-20 Thread Ian Jackson
Use mkdir -p in both cases. Otherwise these late_commands may fail. Signed-off-by: Ian Jackson --- Osstest/Debian.pm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Osstest/Debian.pm b/Osstest/Debian.pm index dacefb3..987c7c6 100644 ---

[Xen-devel] [OSSTEST PATCH 16/35] netboot: Rework interface to host_netboot_file (was host_pxefile)

2017-01-20 Thread Ian Jackson
* Now it takes the tail of the template key (currently, normally `Templates') rather than the whole of it (`PxeTemplates'); the `Pxe' part is implied. * Now it normally returns the whole path, including PxeDir, which the caller is no longer expected to prepend. In any case, it supplies

[Xen-devel] [OSSTEST PATCH 17/35] netboot: Rename pxeboot functions

2017-01-20 Thread Ian Jackson
git-ls-files | xargs perl -i~ -pe 's/\bsetup_pxeboot_/setup_netboot_/g' git-ls-files | xargs perl -i~ -pe 's/\bsetup_pxelinux_/setup_netboot_/g' No functional change. Signed-off-by: Ian Jackson --- Osstest/TestSupport.pm | 28 ++--

[Xen-devel] [OSSTEST PATCH 15/35] pxe templates: Do defaulting outside host_pxefile

2017-01-20 Thread Ian Jackson
host_pxefile now returns undef if the specified $templatekey is not configured, rather than defaulting to the plain PxeTemplates. This is going to useful because we want host_pxefile not to hardcode the default: with netgrub, the settings are all different. If $templatekey is 'PxeTemplates' then

[Xen-devel] [OSSTEST PATCH 14/35] d-i grub: Extend UEFI workaround to stretch

2017-01-20 Thread Ian Jackson
Debian bug #789798 remains open. I have pinged the blocking bug in grub2, #792547. Signed-off-by: Ian Jackson --- Osstest/Debian.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Osstest/Debian.pm b/Osstest/Debian.pm index 987c7c6..5cd6433 100644

[Xen-devel] [OSSTEST PATCH 12/35] backports handling: Cope with dtbs in subdirs

2017-01-20 Thread Ian Jackson
Some of the arm64 kernels have backports in subdirectories. So use a find rune instead of a plain mv. Signed-off-by: Ian Jackson --- mg-debian-installer-update | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/mg-debian-installer-update

[Xen-devel] [OSSTEST PATCH 10/35] backports handling: Refactor kernel deb handling

2017-01-20 Thread Ian Jackson
Introduce private function $kdeb->(). No functional change. Signed-off-by: Ian Jackson --- Osstest/Debian.pm | 45 + 1 file changed, 29 insertions(+), 16 deletions(-) diff --git a/Osstest/Debian.pm b/Osstest/Debian.pm

[Xen-devel] [OSSTEST PATCH 11/35] backports handling: Make backports initramfs-tools optional

2017-01-20 Thread Ian Jackson
If mg-debian-installer-update decided not to provide an initramfs.deb, just skip that. No functional change with earlier d-i fetches produced by existing versions of mg-debian-installer-update. Signed-off-by: Ian Jackson --- Osstest/Debian.pm | 6 +- 1 file

[Xen-devel] [OSSTEST PATCH 07/35] mg-debian-installer-update: backports: Provide pps and ptp

2017-01-20 Thread Ian Jackson
Our ARM64 softiron boxes need these to find their network and disks. Signed-off-by: Ian Jackson --- mg-debian-installer-update | 2 ++ 1 file changed, 2 insertions(+) diff --git a/mg-debian-installer-update b/mg-debian-installer-update index 444f80b..12586f8 100755

[Xen-devel] [OSSTEST PATCH 09/35] mg-debian-installer-update: backports: Provide crypto

2017-01-20 Thread Ian Jackson
Filesystem drivers now refer to this. Signed-off-by: Ian Jackson --- mg-debian-installer-update | 1 + 1 file changed, 1 insertion(+) diff --git a/mg-debian-installer-update b/mg-debian-installer-update index 079d3e8..82a1804 100755 --- a/mg-debian-installer-update

[Xen-devel] [OSSTEST PATCH 08/35] mg-debian-installer-update: backports: Add modules for EFI

2017-01-20 Thread Ian Jackson
UEFI uses vfat for the boot partiton. This also means we need some charset modules. Signed-off-by: Ian Jackson --- mg-debian-installer-update | 2 ++ 1 file changed, 2 insertions(+) diff --git a/mg-debian-installer-update b/mg-debian-installer-update index

[Xen-devel] [OSSTEST PATCH 05/35] mg-debian-installer-update: Print working directory to stderr

2017-01-20 Thread Ian Jackson
No other change Signed-off-by: Ian Jackson --- mg-debian-installer-update | 1 + 1 file changed, 1 insertion(+) diff --git a/mg-debian-installer-update b/mg-debian-installer-update index 00cc735..d0aabb5 100755 --- a/mg-debian-installer-update +++

[Xen-devel] [OSSTEST PATCH 04/35] mg-debian-installer-update: Break out default setting of src=

2017-01-20 Thread Ian Jackson
No functional change. Signed-off-by: Ian Jackson --- mg-debian-installer-update | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/mg-debian-installer-update b/mg-debian-installer-update index bb1ae62..00cc735 100755 ---

[Xen-devel] [OSSTEST PATCH 06/35] mg-debian-installer-update: Log our version into our output

2017-01-20 Thread Ian Jackson
Signed-off-by: Ian Jackson --- mg-debian-installer-update | 4 1 file changed, 4 insertions(+) diff --git a/mg-debian-installer-update b/mg-debian-installer-update index d0aabb5..444f80b 100755 --- a/mg-debian-installer-update +++ b/mg-debian-installer-update @@

[Xen-devel] [OSSTEST PATCH 03/35] ts-host-install: NTP not honoured bug remains

2017-01-20 Thread Ian Jackson
Debian #778564 remains open. Signed-off-by: Ian Jackson --- ts-host-install | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ts-host-install b/ts-host-install index 99ba130..770c51b 100755 --- a/ts-host-install +++ b/ts-host-install @@ -135,7 +135,7

[Xen-devel] [OSSTEST PATCH 00/35] ARM64 softiron support

2017-01-20 Thread Ian Jackson
I currently have this working well enough to run d-i and install the host. I'm running a commissioning flight on both softiron boxes (laxton0, laxton1). If this works well enough I will push this seris to osstest pretest and, assuming that it passes, put laxton* into production. Note that I

[Xen-devel] [OSSTEST PATCH 02/35] ts-host-install: Expect "pool" in ntp.conf, not only "server"

2017-01-20 Thread Ian Jackson
This is helpful for playing around with stretch, which seems to use this option by default. Signed-off-by: Ian Jackson --- ts-host-install | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ts-host-install b/ts-host-install index ecf5f0b..99ba130

[Xen-devel] [OSSTEST PATCH 01/35] Serial console: honour LinuxSerialConsole in bootloader setup

2017-01-20 Thread Ian Jackson
Signed-off-by: Ian Jackson --- Osstest/Debian.pm | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/Osstest/Debian.pm b/Osstest/Debian.pm index 5eecf92..52527bf 100644 --- a/Osstest/Debian.pm +++ b/Osstest/Debian.pm @@ -67,8 +67,9 @@ sub

Re: [Xen-devel] [RFC PATCH v2 03/26] ARM: GICv3 ITS: allocate device and collection table

2017-01-20 Thread Julien Grall
Hi Andre, On 20/01/2017 12:27, Andre Przywara wrote: On 20/01/17 11:12, Julien Grall wrote: Hello, On 04/01/2017 22:47, Stefano Stabellini wrote: On Thu, 22 Dec 2016, Andre Przywara wrote: Each ITS maps a pair of a DeviceID (usually the PCI b/d/f triplet) and an EventID (the MSI payload or

[Xen-devel] [PATCH 0/4] fuzz: basic AFL support

2017-01-20 Thread Wei Liu
Provide simple stub programs for AFL fuzzer. For usage, please see README.afl. This series doesn't aim to improve fuzzing targets. It is just demonstration for how we could use the same fuzzing code for both oss-fuzz and AFL. The stub programs are identical in libefl and x86 emulator at the

[Xen-devel] [PATCH 3/4] tools/fuzz: add AFL stub program for libefl fuzzer

2017-01-20 Thread Wei Liu
And hook it up into build system. Signed-off-by: Wei Liu --- Cc: Andrew Cooper Cc: George Dunlap Cc: Ian Jackson Cc: Jan Beulich --- .gitignore

[Xen-devel] [PATCH 4/4] tools/fuzz: add README.afl

2017-01-20 Thread Wei Liu
And rename README to README.oss-fuzz. Signed-off-by: Wei Liu --- Cc: Andrew Cooper Cc: George Dunlap Cc: Ian Jackson Cc: Jan Beulich --- tools/fuzz/README.afl

[Xen-devel] [PATCH 1/4] tools/fuzz: add missing dependency in x86 insn fuzzer build rule

2017-01-20 Thread Wei Liu
The said file needs the two header files. Signed-off-by: Wei Liu --- Cc: Ian Jackson --- tools/fuzz/x86_instruction_emulator/Makefile | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tools/fuzz/x86_instruction_emulator/Makefile

[Xen-devel] [PATCH 2/4] tools/fuzz: add AFL stub program for x86 insn emulator fuzzer

2017-01-20 Thread Wei Liu
This is a basic program to call into the unified fuzzing function. Hook it up into build system so that we can always build test it. Signed-off-by: Wei Liu --- Cc: Andrew Cooper Cc: George Dunlap Cc: Ian Jackson

Re: [Xen-devel] [linux-linus test] 104237: regressions - FAIL

2017-01-20 Thread Ian Jackson
Julien Grall writes ("Re: [Xen-devel] [linux-linus test] 104237: regressions - FAIL"): > I would not be surprised that it takes 2.5 hours for a make allmodules > on both the Cubietruck and Arndale. ... > So I can see 2 solutions: > 1) Increase the timeout > 2) Only build the kernel

[Xen-devel] [OSSTEST PATCH 2/3] timeouts: Call target_adjust_timeout for all build commands

2017-01-20 Thread Ian Jackson
Currently this has no real effect, because the only place where a timeout factor is specified is nested L1s and we do not run build commands on those. Signed-off-by: Ian Jackson --- Osstest/TestSupport.pm | 2 ++ 1 file changed, 2 insertions(+) diff --git

[Xen-devel] [OSSTEST PATCH 3/3] timeouts: Honour TimeoutFactor host property

2017-01-20 Thread Ian Jackson
This gives us a single place to say that a host is particularly slow. 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 b8946da..1108dbd 100644 ---

[Xen-devel] [OSSTEST PATCH 1/3] timeouts: Refactor target_adjust_timeout to have $adjust->(), and log

2017-01-20 Thread Ian Jackson
If a timeout is adjusted, print a log message. We are going to want to reuse this logic in a moment. Do the defaulting in $adjust->() rather than passing 1 as a default to the lookup functions, which is slightly neater. Signed-off-by: Ian Jackson ---

Re: [Xen-devel] [RFC PATCH v2 07/26] ARM: GICv3 ITS: introduce host LPI array

2017-01-20 Thread Julien Grall
Hi Stefano, Sorry for the late answer, still going through my e-mail backlog. On 06/01/2017 21:20, Stefano Stabellini wrote: On Fri, 6 Jan 2017, Andre Przywara wrote: It is also possible to end up calling mapti with an inexistent eventid for host_devid. Could that be a problem? Not at all.

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

2017-01-20 Thread Jan Beulich
>>> On 18.01.17 at 11:23, wrote: >> From: Jan Beulich [mailto:jbeul...@suse.com] >> Sent: Wednesday, January 18, 2017 5:38 PM >> >> >>> On 18.01.17 at 05:57, wrote: >> > Attached was my earlier comment: >> > >> > -- >> >> >>> On 20.12.16 at 06:37,

Re: [Xen-devel] [PATCH v12 05/10] x86: add multiboot2 protocol support for EFI platforms

2017-01-20 Thread Daniel Kiper
On Fri, Jan 20, 2017 at 02:46:47AM -0700, Jan Beulich wrote: > >>> On 20.01.17 at 02:34, wrote: > > @@ -100,20 +107,48 @@ multiboot2_header_start: > > gdt_boot_descr: > > .word 6*8-1 > > .long sym_phys(trampoline_gdt) > > +.long 0 /*

Re: [Xen-devel] [RFC PATCH v2 03/26] ARM: GICv3 ITS: allocate device and collection table

2017-01-20 Thread Andre Przywara
Hi, On 20/01/17 11:12, Julien Grall wrote: > Hello, > > On 04/01/2017 22:47, Stefano Stabellini wrote: >> On Thu, 22 Dec 2016, Andre Przywara wrote: >>> Each ITS maps a pair of a DeviceID (usually the PCI b/d/f triplet) and >>> an EventID (the MSI payload or interrupt ID) to a pair of LPI number

Re: [Xen-devel] [RFC PATCH v2 03/26] ARM: GICv3 ITS: allocate device and collection table

2017-01-20 Thread Julien Grall
Hello, On 04/01/2017 22:47, Stefano Stabellini wrote: On Thu, 22 Dec 2016, Andre Przywara wrote: Each ITS maps a pair of a DeviceID (usually the PCI b/d/f triplet) and an EventID (the MSI payload or interrupt ID) to a pair of LPI number and collection ID, which points to the target CPU. This

Re: [Xen-devel] [linux-linus bisection] complete test-amd64-amd64-xl-qemut-win7-amd64 [and 1 more messages]

2017-01-20 Thread Ian Jackson
Boris Ostrovsky writes ("Re: [linux-linus bisection] complete test-amd64-amd64-xl-qemut-win7-amd64 [and 1 more messages]"): > On 01/19/2017 01:05 PM, Ian Jackson wrote: > > This means that the bug is in commits which diverged before the last > > pass of this test. > > > > (Did your filters get

Re: [Xen-devel] memory hotplug for domUs

2017-01-20 Thread Andrew Cooper
On 20/01/17 09:26, Juergen Gross wrote: > Recently Jim asked me why he can use "xl mem-max" to raise the allowed > memory size of a domain in the hypervisor above the configured maxmem > limit of the domain, but not use "xl mem-set" to balloon the domain up > to this value later. > > I thought

Re: [Xen-devel] [PATCH] x86/emul: Corrections to cmpxchg{8, 16}b emulation (to fix 32bit PV guests)

2017-01-20 Thread Jan Beulich
>>> On 20.01.17 at 09:52, wrote: Commenting on just the parts not replaced by the other patch. > @@ -461,6 +484,53 @@ int main(int argc, char **argv) > goto fail; > printf("okay\n"); > > +#ifdef __x86_64__ > +memset(, 0, sizeof(state)); > +

Re: [Xen-devel] [RFC] Device memory mappings for Dom0 on ARM64 ACPI systems

2017-01-20 Thread Roger Pau Monné
On Thu, Jan 19, 2017 at 09:14:03PM +0100, Julien Grall wrote: > Hello, > > On 19/01/2017 19:22, Stefano Stabellini wrote: > > On Thu, 19 Jan 2017, Roger Pau Monné wrote: > > > On Wed, Jan 18, 2017 at 07:13:23PM +, Julien Grall wrote: > > > > Hi, > > > > > > > > On 18/01/17 19:05, Stefano

Re: [Xen-devel] [PATCH] x86/emul: Corrections to cmpxchg{8, 16}b emulation (to fix 32bit PV guests)

2017-01-20 Thread Jan Beulich
>>> On 20.01.17 at 11:39, wrote: > On 20/01/17 09:54, Jan Beulich wrote: > On 20.01.17 at 09:52, wrote: >>> @@ -2852,6 +2852,11 @@ x86_emulate( >>> else if ( !(d & Mov) ) /* optimisation - avoid slow emulated read >>> */ >>>

Re: [Xen-devel] [PATCH] x86emul: LOCK check adjustments

2017-01-20 Thread Andrew Cooper
On 20/01/17 10:10, Jan Beulich wrote: > BT, being encoded as DstBitBase just like BT{C,R,S}, nevertheless does > not write its (register or memory) operand and hence also doesn't allow > a LOCK prefix to be used. > > At the same time CLAC/STAC have no need to explicitly check lock_prefix > - this

Re: [Xen-devel] [PATCH] x86emul: CMPXCHG{8,16}B are memory writes

2017-01-20 Thread Andrew Cooper
On 20/01/17 08:23, Jan Beulich wrote: > This fixes a regression introduced by commit ff913f68c9 ("x86/PV: > restrict permitted instructions during memory write emulation") > breaking namely 32-bit PV guests (which commonly use CMPXCHG8B for > certain page table updates). > > Reported-by: Andrew

Re: [Xen-devel] [PATCH] x86/emul: Corrections to cmpxchg{8, 16}b emulation (to fix 32bit PV guests)

2017-01-20 Thread Andrew Cooper
On 20/01/17 09:54, Jan Beulich wrote: On 20.01.17 at 09:52, wrote: >> @@ -2852,6 +2852,11 @@ x86_emulate( >> else if ( !(d & Mov) ) /* optimisation - avoid slow emulated read */ >> { >> fail_if(lock_prefix ? !ops->cmpxchg :

Re: [Xen-devel] [PATCH v12 06/10] x86: change default load address from 1 MiB to 2 MiB

2017-01-20 Thread Daniel Kiper
On Fri, Jan 20, 2017 at 01:49:46AM -0700, Jan Beulich wrote: > >>> On 20.01.17 at 05:06, wrote: > > On 1/19/17 8:34 PM, Daniel Kiper wrote: > >> Subsequent patches introducing relocatable early boot code play with > >> page tables using 2 MiB huge pages. If load address is not

[Xen-devel] [PATCH] tools/fuzz: remove redundant rule in x86 insn fuzzer

2017-01-20 Thread Wei Liu
The predefined pattern rule works. Signed-off-by: Wei Liu --- tools/fuzz/x86_instruction_emulator/Makefile | 2 -- 1 file changed, 2 deletions(-) diff --git a/tools/fuzz/x86_instruction_emulator/Makefile b/tools/fuzz/x86_instruction_emulator/Makefile index

Re: [Xen-devel] [PATCH v5 2/9] x86/iommu: add IOMMU entries for p2m_mmio_direct pages

2017-01-20 Thread Roger Pau Monne
On Fri, Jan 20, 2017 at 06:41:06AM +, Tian, Kevin wrote: > > From: Roger Pau Monne [mailto:roger@citrix.com] > > Sent: Friday, January 20, 2017 1:30 AM > > > > There's nothing wrong with allowing the domain to perform DMA transfers to > > MMIO areas that it already can access from the

[Xen-devel] [linux-linus bisection] complete test-amd64-i386-qemuu-rhel6hvm-amd

2017-01-20 Thread osstest service owner
branch xen-unstable xenbranch xen-unstable job test-amd64-i386-qemuu-rhel6hvm-amd testid xen-boot Tree: linux git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux-2.6.git Tree: linuxfirmware git://xenbits.xen.org/osstest/linux-firmware.git Tree: qemu

<    1   2   3   >