Re: [Xen-devel] [PATCH] xen/efi: Avoid EFI stub using absolute symbols

2018-01-17 Thread Jan Beulich
>>> On 16.01.18 at 18:43, wrote: > On 12/01/18 13:13, Jan Beulich wrote: > On 09.01.18 at 20:43, wrote: >>> When I compiled the snippet on x86 and Arm, no relocation is available >>> for the pointers to string in the array in the final binary. Yet they >>> are available in the object. >> >>

[Xen-devel] [xen-unstable-smoke test] 118146: regressions - FAIL

2018-01-17 Thread osstest service owner
flight 118146 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/118146/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-arm64-xsm 6 xen-buildfail REGR. vs. 118105 build-armhf

Re: [Xen-devel] [PATCH v8 11/17] x86: Protect unaware domains from meddling hyperthreads

2018-01-17 Thread Jan Beulich
>>> On 16.01.18 at 22:11, wrote: > An alternative to the current levelling logic would be to treat STIBP as > a Special Bit (in CPUID terms, like OSXSAVE/etc) and unconditionally set > it equal to IBRS, irrespective of the toolstack setting. That way, > migration between HT and non-HT hardware is

Re: [Xen-devel] [PATCH v8 11/17] x86: Protect unaware domains from meddling hyperthreads

2018-01-17 Thread Andrew Cooper
On 17/01/2018 08:40, Jan Beulich wrote: On 16.01.18 at 22:11, wrote: >> An alternative to the current levelling logic would be to treat STIBP as >> a Special Bit (in CPUID terms, like OSXSAVE/etc) and unconditionally set >> it equal to IBRS, irrespective of the toolstack setting. That way, >

Re: [Xen-devel] [PATCH v8 12/17] x86/entry: Organise the use of MSR_SPEC_CTRL at each entry/exit point

2018-01-17 Thread Jan Beulich
>>> On 16.01.18 at 22:24, wrote: > On 15/01/18 12:09, Jan Beulich wrote: > On 12.01.18 at 19:01, wrote: >>> @@ -586,6 +611,10 @@ ENTRY(double_fault) >>> movl $TRAP_double_fault,4(%rsp) >>> /* Set AC to reduce chance of further SMAP faults */ >>> SAVE_ALL STAC >>> +

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

2018-01-17 Thread Jan Beulich
>>> On 16.01.18 at 18:30, wrote: >> From: Xen-devel [mailto:xen-devel-boun...@lists.xenproject.org] On Behalf >> Of Paul Durrant >> Sent: 16 January 2018 09:27 >> > From: Jan Beulich [mailto:jbeul...@suse.com] >> > Sent: 16 January 2018 08:58 >> > >>> On 16.01.18 at 09:43, wrote: >> > > flight 11

Re: [Xen-devel] [PATCH v8 08/17] x86/msr: Emulation of MSR_{SPEC_CTRL, PRED_CMD} for guests

2018-01-17 Thread Jan Beulich
>>> On 16.01.18 at 17:58, wrote: > On 16/01/18 11:10, David Woodhouse wrote: >> On Fri, 2018-01-12 at 18:00 +, Andrew Cooper wrote: >>> @@ -152,14 +163,38 @@ int guest_wrmsr(struct vcpu *v, uint32_t msr, >>> uint64_t val) >>> { >>> const struct vcpu *curr = current; >>> struct domai

Re: [Xen-devel] [PATCH v8 12/17] x86/entry: Organise the use of MSR_SPEC_CTRL at each entry/exit point

2018-01-17 Thread Andrew Cooper
No, not really. Omitting it on the grounds of "we don't expect to take a double fault" don't beat uniformally altering all the entrypoints in a consistent manor. The only thing which can go wrong is that we forget to do it when it is needed. ~Andrew Fr

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

2018-01-17 Thread Paul Durrant
> -Original Message- > From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: 17 January 2018 08:52 > To: Paul Durrant > Cc: xen-devel ; osstest- > ad...@xenproject.org > Subject: RE: [Xen-devel] [xen-unstable test] 118078: regressions - FAIL > > >>> On 16.01.18 at 18:30, wrote: > >> From:

Re: [Xen-devel] [PATCH v8 08/17] x86/msr: Emulation of MSR_{SPEC_CTRL, PRED_CMD} for guests

2018-01-17 Thread Andrew Cooper
From: Jan Beulich [jbeul...@suse.com] Sent: 17 January 2018 09:11 To: Andrew Cooper Cc: David Woodhouse; Xen-devel Subject: Re: [Xen-devel] [PATCH v8 08/17] x86/msr: Emulation of MSR_{SPEC_CTRL, PRED_CMD} for guests >>> On 16.01.18 at 17:58, wrote: > On

[Xen-devel] [PATCH 1/6] xen/pvshim: map vcpu_info earlier for APs

2018-01-17 Thread Roger Pau Monne
Or else init_percpu_time is going to dereference a NULL pointer when trying to access vcpu_info. Signed-off-by: Roger Pau Monné --- Cc: Jan Beulich Cc: Andrew Cooper Cc: Ian Jackson Cc: Wei Liu --- Should be backported to the 4.10.0-shim-comet branch. --- xen/arch/x86/smpboot.c | 6 +++--- 1

[Xen-devel] [PATCH 4/6] xen/pvshim: simplify replace_va_mapping code

2018-01-17 Thread Roger Pau Monne
No functional change. Signed-off-by: Roger Pau Monné --- Cc: Jan Beulich Cc: Andrew Cooper --- xen/arch/x86/pv/shim.c | 21 ++--- 1 file changed, 6 insertions(+), 15 deletions(-) diff --git a/xen/arch/x86/pv/shim.c b/xen/arch/x86/pv/shim.c index 702249719e..4f94047695 100644 -

[Xen-devel] [PATCH 5/6] xen/pvshim: fix coding style issues

2018-01-17 Thread Roger Pau Monne
Fix a couple of coding style issues. No code or functional change. Signed-off-by: Roger Pau Monné --- Cc: Jan Beulich Cc: Andrew Cooper --- xen/arch/x86/pv/shim.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/xen/arch/x86/pv/shim.c b/xen/arch/x86/pv/sh

[Xen-devel] [PATCH 2/6] xen/pvh: place the trampoline at page 0x1

2018-01-17 Thread Roger Pau Monne
Since PVH guest jump straight into trampoline_setup trampoline_phys is not initialized, thus the trampoline is relocated to address 0. This works, but has the undesirable effect of having VA 0 mapped to MFN 0, which means NULL pointed dereferences no longer trigger a page fault. In order to solve

[Xen-devel] [PATCH 0/6] xen/pvshim: fix for staging

2018-01-17 Thread Roger Pau Monne
Hello, The following series contain some code and style fixes for the pvshim (comet) code that has been merged into staging. IMHO patches 1-3 should be backported to the stable comet branch. A branch with the series is also available at: git://xenbits.xen.org/people/royger/xen.git pvshim_fixes_v

[Xen-devel] [PATCH 6/6] firmware/shim: fix build process to use POSIX find options

2018-01-17 Thread Roger Pau Monne
The -printf find option is not POSIX compatible, so replace it with another rune. Signed-off-by: Roger Pau Monné --- Cc: Ian Jackson Cc: Wei Liu --- tools/firmware/xen-dir/Makefile | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tools/firmware/xen-dir/Makefile b/tools/fir

[Xen-devel] [PATCH 3/6] xen/pvshim: identity pin shim vCPUs to pCPUs

2018-01-17 Thread Roger Pau Monne
Since VCPUOP_{up/down} already identity pins vCPU hotplug to pCPU hotplug also pin the vCPUs to the pCPUs in the scheduler. This prevent vCPU migration and should improve performance. While there also use __cpumask_set_cpu instead of cpumask_set_cpu, there's no need to use the locked variant. Sig

[Xen-devel] [linux-linus test] 118112: tolerable FAIL - PUSHED

2018-01-17 Thread osstest service owner
flight 118112 linux-linus real [real] http://logs.test-lab.xenproject.org/osstest/logs/118112/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-amd64-amd64-xl-qemut-win7-amd64 17 guest-stopfail like 117996 test-armhf-armhf-libvirt 14 saver

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

2018-01-17 Thread Paul Durrant
> -Original Message- [snip] > What I meant was that I'd expect the guest to have interrupts disabled whilst > poking the MSR to enable APIC assist on that CPU, since enabling APIC assist > is clearly going to modify the way in which interrupts are handled. If that's > not the case though th

[Xen-devel] [xen-unstable-coverity test] 118152: all pass - PUSHED

2018-01-17 Thread osstest service owner
flight 118152 xen-unstable-coverity real [real] http://logs.test-lab.xenproject.org/osstest/logs/118152/ Perfect :-) All tests in this flight passed as required version targeted for testing: xen e871e80c38547d9faefc6604532ba3e985e65873 baseline version: xen 2d70

Re: [Xen-devel] [PATCH v4] docs: note default for timer_mode in xl.cfg man

2018-01-17 Thread Roger Pau Monné
On Tue, Jan 16, 2018 at 10:28:56AM -0600, Doug Goldstein wrote: > There was no default documented but inspecting > libxl__domain_build_info_setdefault() shows the default to be > LIBXL_TIMER_MODE_NO_DELAY_FOR_MISSED_TICKS. > > Signed-off-by: Doug Goldstein Reviewed-by: Roger Pau Monné Thanks,

[Xen-devel] [xen-unstable-smoke test] 118150: regressions - FAIL

2018-01-17 Thread osstest service owner
flight 118150 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/118150/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-arm64-xsm 6 xen-buildfail REGR. vs. 118105 build-armhf

Re: [Xen-devel] [PATCH 1/5] xen/arm: Introduce enable callback to enable a capabilities on each online CPU

2018-01-17 Thread Julien Grall
Hi Stefano, On 16/01/18 23:55, Stefano Stabellini wrote: On Tue, 16 Jan 2018, Julien Grall wrote: Once Xen knows what features/workarounds present on the platform, it might be necessary to configure each online CPU. Introduce a new callback "enable" that will be called on each online CPU to co

Re: [Xen-devel] [Xen-users] DomU not starting under pvhv2

2018-01-17 Thread Roger Pau Monné
On Mon, Jan 08, 2018 at 06:01:57PM +, George Dunlap wrote: > Moving to xen-devel, and cc'ing Roger and Boris (who developed PVH) > > On Mon, Jan 8, 2018 at 5:24 AM, Peter wrote: > > Hi. > > > > Running Xen 4.10.0 > > What version of Linux are you using? > > > > > A VM is not starting with t

[Xen-devel] [PATCH] libxc: fix arm build after bdf693ee61b48

2018-01-17 Thread Wei Liu
The ramdisk fields were removed. We should use modules[0] instead. Signed-off-by: Wei Liu --- Cc: Stefano Stabellini Cc: Julien Grall Cc: Ian Jackson Verified the build with a qemu-chroot. --- tools/libxc/xc_dom_arm.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --g

Re: [Xen-devel] [PATCH 5/6] xen/pvshim: fix coding style issues

2018-01-17 Thread Wei Liu
On Wed, Jan 17, 2018 at 09:48:13AM +, Roger Pau Monne wrote: > Fix a couple of coding style issues. > > No code or functional change. > > Signed-off-by: Roger Pau Monné Reviewed-by: Wei Liu ___ Xen-devel mailing list Xen-devel@lists.xenproject.o

Re: [Xen-devel] [PATCH 4/6] xen/pvshim: simplify replace_va_mapping code

2018-01-17 Thread Wei Liu
On Wed, Jan 17, 2018 at 09:48:12AM +, Roger Pau Monne wrote: > No functional change. > > Signed-off-by: Roger Pau Monné Reviewed-by: Wei Liu ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/x

Re: [Xen-devel] [PATCH 1/6] xen/pvshim: map vcpu_info earlier for APs

2018-01-17 Thread Wei Liu
On Wed, Jan 17, 2018 at 09:48:09AM +, Roger Pau Monne wrote: > Or else init_percpu_time is going to dereference a NULL pointer when > trying to access vcpu_info. > > Signed-off-by: Roger Pau Monné Reviewed-by: Wei Liu ___ Xen-devel mailing list X

Re: [Xen-devel] [PATCH 5/5] xen/arm64: Implement branch predictor hardening for affected Cortex-A CPUs

2018-01-17 Thread Julien Grall
Hi Stefano, On 17/01/18 00:42, Stefano Stabellini wrote: On Tue, 16 Jan 2018, Julien Grall wrote: #define MIDR_RANGE(model, min, max) \ @@ -205,6 +232,28 @@ static const struct arm_cpu_capabilities arm_errata[] = { (1 << MIDR_VARIANT_SHIFT) | 2), }, #endif +#i

Re: [Xen-devel] [PATCH 2/6] xen/pvh: place the trampoline at page 0x1

2018-01-17 Thread Wei Liu
On Wed, Jan 17, 2018 at 09:48:10AM +, Roger Pau Monne wrote: > Since PVH guest jump straight into trampoline_setup trampoline_phys is > not initialized, thus the trampoline is relocated to address 0. > > This works, but has the undesirable effect of having VA 0 mapped to > MFN 0, which means N

Re: [Xen-devel] [PATCH 6/6] firmware/shim: fix build process to use POSIX find options

2018-01-17 Thread Wei Liu
On Wed, Jan 17, 2018 at 09:48:14AM +, Roger Pau Monne wrote: > The -printf find option is not POSIX compatible, so replace it with > another rune. > > Signed-off-by: Roger Pau Monné Acked-by: Wei Liu ___ Xen-devel mailing list Xen-devel@lists.xen

[Xen-devel] [PATCH for-4.10.0-shim-comet] x86/guest: use the vcpu_info area from shared_info

2018-01-17 Thread Roger Pau Monne
If using less than 32 vCPUs (XEN_LEGACY_MAX_VCPUS). This is a workaround that should allow to boot the shim on hypervisors without commit "x86/upcall: inject a spurious event after setting upcall vector" as long as less than 32 vCPUs are assigned to the shim. Signed-off-by: Roger Pau Monné --- C

Re: [Xen-devel] [PATCH for-4.10.0-shim-comet] x86/guest: use the vcpu_info area from shared_info

2018-01-17 Thread George Dunlap
On 01/17/2018 10:57 AM, Roger Pau Monne wrote: > If using less than 32 vCPUs (XEN_LEGACY_MAX_VCPUS). > > This is a workaround that should allow to boot the shim on hypervisors > without commit "x86/upcall: inject a spurious event after setting > upcall vector" as long as less than 32 vCPUs are ass

[Xen-devel] [qemu-mainline test] 118117: regressions - trouble: broken/fail/pass

2018-01-17 Thread osstest service owner
flight 118117 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/118117/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-pygrub broken test-amd64-amd64-xl-qcow2

Re: [Xen-devel] [Xen-users] DomU not starting under pvhv2

2018-01-17 Thread M A Young
On Wed, 17 Jan 2018, Roger Pau Monné wrote: > On Mon, Jan 08, 2018 at 06:01:57PM +, George Dunlap wrote: > > Moving to xen-devel, and cc'ing Roger and Boris (who developed PVH) > > > > On Mon, Jan 8, 2018 at 5:24 AM, Peter wrote: > > > Hi. > > > > > > Running Xen 4.10.0 > > > > What version

Re: [Xen-devel] [PATCH] libxc: fix arm build after bdf693ee61b48

2018-01-17 Thread Ian Jackson
Wei Liu writes ("[PATCH] libxc: fix arm build after bdf693ee61b48"): > The ramdisk fields were removed. We should use modules[0] instead. Acked-by: Ian Jackson ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailm

Re: [Xen-devel] [PATCH v4] docs: note default for timer_mode in xl.cfg man

2018-01-17 Thread Wei Liu
On Tue, Jan 16, 2018 at 10:28:56AM -0600, Doug Goldstein wrote: > There was no default documented but inspecting > libxl__domain_build_info_setdefault() shows the default to be > LIBXL_TIMER_MODE_NO_DELAY_FOR_MISSED_TICKS. > > Signed-off-by: Doug Goldstein Acked-by: Wei Liu ___

Re: [Xen-devel] [PATCH 3/6] xen/pvshim: identity pin shim vCPUs to pCPUs

2018-01-17 Thread Wei Liu
On Wed, Jan 17, 2018 at 09:48:11AM +, Roger Pau Monne wrote: > Since VCPUOP_{up/down} already identity pins vCPU hotplug to pCPU > hotplug also pin the vCPUs to the pCPUs in the scheduler. This prevent The description is a bit ambiguous. I read it as "the shim hotplug code pins vcpu to pcpu wh

Re: [Xen-devel] [PATCH for-4.10.0-shim-comet] x86/guest: use the vcpu_info area from shared_info

2018-01-17 Thread Ian Jackson
George Dunlap writes ("Re: [PATCH for-4.10.0-shim-comet] x86/guest: use the vcpu_info area from shared_info"): > Without this patch, people need to reboot their L0 hypervisor in order > to use Comet. People running 4.10. On 4.8 this presumably has no benefit. > The risk of checking it into the

Re: [Xen-devel] [PATCH for-4.10.0-shim-comet] x86/guest: use the vcpu_info area from shared_info

2018-01-17 Thread George Dunlap
On 01/17/2018 11:16 AM, Ian Jackson wrote: > George Dunlap writes ("Re: [PATCH for-4.10.0-shim-comet] x86/guest: use the > vcpu_info area from shared_info"): >> Without this patch, people need to reboot their L0 hypervisor in order >> to use Comet. > > People running 4.10. On 4.8 this presumably

Re: [Xen-devel] [PATCH] libxc: fix arm build after bdf693ee61b48

2018-01-17 Thread Wei Liu
On Wed, Jan 17, 2018 at 10:43:40AM +, Wei Liu wrote: > The ramdisk fields were removed. We should use modules[0] instead. > > Signed-off-by: Wei Liu > --- > Cc: Stefano Stabellini > Cc: Julien Grall > Cc: Ian Jackson > > Verified the build with a qemu-chroot. I will send out an updated v

Re: [Xen-devel] [PATCH] x86/efi: fix build with linkers that support both coff-x86-64 and pe-x86-64

2018-01-17 Thread Roger Pau Monné
On Mon, Jan 08, 2018 at 03:11:21AM -0700, Jan Beulich wrote: > >>> On 05.01.18 at 17:43, wrote: > > When using a linker that supports both formats the following error > > will be triggered: > > > > efi/buildid.o: file not recognized: File format is ambiguous > > efi/buildid.o: matching formats: c

[Xen-devel] [PATCH v2] tools: fix arm build after bdf693ee61b48

2018-01-17 Thread Wei Liu
The ramdisk fields were removed. We should use modules[0] instead. Signed-off-by: Wei Liu --- Cc: Stefano Stabellini Cc: Julien Grall Cc: Ian Jackson Verified the build with a qemu-chroot. --- tools/libxc/xc_dom_arm.c | 10 +- tools/libxl/libxl_arm.c | 6 +++--- 2 files changed, 8

Re: [Xen-devel] [PATCH 2/6] xen/pvh: place the trampoline at page 0x1

2018-01-17 Thread Roger Pau Monné
On Wed, Jan 17, 2018 at 10:55:22AM +, Wei Liu wrote: > On Wed, Jan 17, 2018 at 09:48:10AM +, Roger Pau Monne wrote: > > diff --git a/xen/arch/x86/boot/head.S b/xen/arch/x86/boot/head.S > > index 4fe5a776b1..7829e3f07c 100644 > > --- a/xen/arch/x86/boot/head.S > > +++ b/xen/arch/x86/boot/hea

Re: [Xen-devel] [PATCH 2/6] xen/pvh: place the trampoline at page 0x1

2018-01-17 Thread Wei Liu
On Wed, Jan 17, 2018 at 11:35:15AM +, Roger Pau Monné wrote: > > > > +movw$0x1000, sym_esi(trampoline_phys) > > > jmp trampoline_setup > > > > > > #endif /* CONFIG_PVH_GUEST */ > > > diff --git a/xen/arch/x86/mm.c b/xen/arch/x86/mm.c > > > index 1147a1afb1..586af2ba

[Xen-devel] [PATCH v2 2/6] xen/pvh: place the trampoline at page 0x1

2018-01-17 Thread Roger Pau Monne
Since PVH guest jump straight into trampoline_setup trampoline_phys is not initialized, thus the trampoline is relocated to address 0. This works, but has the undesirable effect of having VA 0 mapped to MFN 0, which means NULL pointed dereferences no longer trigger a page fault. In order to solve

Re: [Xen-devel] [PATCH v2 2/6] xen/pvh: place the trampoline at page 0x1

2018-01-17 Thread Wei Liu
On Wed, Jan 17, 2018 at 12:00:41PM +, Roger Pau Monne wrote: > Since PVH guest jump straight into trampoline_setup trampoline_phys is > not initialized, thus the trampoline is relocated to address 0. > > This works, but has the undesirable effect of having VA 0 mapped to > MFN 0, which means N

Re: [Xen-devel] [PATCH 1/5] xen/arm: Introduce enable callback to enable a capabilities on each online CPU

2018-01-17 Thread Lars Kurth
> On 17 Jan 2018, at 10:31, Julien Grall wrote: > > Hi Stefano, > > On 16/01/18 23:55, Stefano Stabellini wrote: >> On Tue, 16 Jan 2018, Julien Grall wrote: >>> Once Xen knows what features/workarounds present on the platform, it >>> might be necessary to configure each online CPU. >>> >>> In

Re: [Xen-devel] [PATCH] x86/efi: fix build with linkers that support both coff-x86-64 and pe-x86-64

2018-01-17 Thread Jan Beulich
>>> On 17.01.18 at 12:21, wrote: > On Mon, Jan 08, 2018 at 03:11:21AM -0700, Jan Beulich wrote: >> >>> On 05.01.18 at 17:43, wrote: >> > --- a/xen/arch/x86/Makefile >> > +++ b/xen/arch/x86/Makefile >> > @@ -188,20 +188,20 @@ endif >> > $(TARGET).efi: prelink-efi.o $(note_file) efi.lds efi/relocs

Re: [Xen-devel] [PATCH 1/5] xen/arm: Introduce enable callback to enable a capabilities on each online CPU

2018-01-17 Thread Julien Grall
Hi Lars, On 17/01/18 12:23, Lars Kurth wrote: On 17 Jan 2018, at 10:31, Julien Grall > wrote: Hi Stefano, On 16/01/18 23:55, Stefano Stabellini wrote: On Tue, 16 Jan 2018, Julien Grall wrote: Once Xen knows what features/workarounds present on the platform,

[Xen-devel] [xen-unstable-smoke test] 118158: regressions - FAIL

2018-01-17 Thread osstest service owner
flight 118158 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/118158/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-arm64-xsm 6 xen-buildfail REGR. vs. 118105 build-armhf

Re: [Xen-devel] [PATCH for-4.10.0-shim-comet] x86/guest: use the vcpu_info area from shared_info

2018-01-17 Thread Jan Beulich
>>> On 17.01.18 at 12:04, wrote: > On 01/17/2018 10:57 AM, Roger Pau Monne wrote: >> If using less than 32 vCPUs (XEN_LEGACY_MAX_VCPUS). >> >> This is a workaround that should allow to boot the shim on hypervisors >> without commit "x86/upcall: inject a spurious event after setting >> upcall vect

[Xen-devel] [PATCH] x86/hvm: more sure APIC assist is aborted if guest EOIs APIC

2018-01-17 Thread Paul Durrant
It appears there is a case where Windows enables the APIC assist enlightenment (see Microsoft Hypervisor Top Level Functional Spec. 5.0b, section 10.3.5) but does not use it. This scenario is perfectly valid according to the documentation, but causes the state machine in Xen to become confused lead

Re: [Xen-devel] [PATCH] x86/hvm: more sure APIC assist is aborted if guest EOIs APIC

2018-01-17 Thread Paul Durrant
> -Original Message- > From: Paul Durrant [mailto:paul.durr...@citrix.com] > Sent: 17 January 2018 12:54 > To: xen-devel@lists.xenproject.org > Cc: Paul Durrant ; Jan Beulich > ; Andrew Cooper > Subject: [PATCH] x86/hvm: more sure APIC assist is aborted if guest EOIs > APIC Sorry, I typo-

[Xen-devel] [PATCH v2] x86/efi: fix build with linkers that support both coff-x86-64 and pe-x86-64

2018-01-17 Thread Roger Pau Monne
When using a linker that supports both formats the following error will be triggered: efi/buildid.o: file not recognized: File format is ambiguous efi/buildid.o: matching formats: coff-x86-64 pe-x86-64 Solve this by specifying the efi/buildid.o format to pe-x86-64. Signed-off-by: Roger Pau Monné

[Xen-devel] xl: Unable to boot Xen Arm guest when using latest staging

2018-01-17 Thread Julien Grall
Hi all, Just a quick heads-up before I go into debugging. I have built and install the latest staging (db3ae8becc) for Arm. When I boot a guest, I get the following error message: libxl: debug: libxl_create.c:1664:do_domain_create: Domain 0:ao 0x386e44d0: create: how=(nil) callback=(nil) poller

Re: [Xen-devel] [Embedded-pv-devel] Xen on RCarH3 Starter Kit

2018-01-17 Thread Ganesh H
Dear Andrii, Sorry for the confusion. I am having an RCarH3 starter kit. Is there a Xen port available for RCarH3 starter kit right now? If not what are the steps to create one? Best Regards Ganesh.H -Original Message- From: Andrii Anisov [mailto:andrii.ani...@gmail.com] Sent: Tuesday

Re: [Xen-devel] Access I2C bus from guest/DomU on ARM board

2018-01-17 Thread Saumya Rajesh
On Tue, Jan 16, 2018 at 9:22 PM, Andrii Anisov wrote: > Dear Rajesh, > > > You can try to get an I2C bus controller in DomU in PIO mode following > [1], keeping in mind [2]. > > If you want it DMA capable you need Renesas IPMMU support in XEN [3], [4] > to be incorporated. > > > [1] - https://xen

Re: [Xen-devel] [PATCH 1/5] xen/arm: Introduce enable callback to enable a capabilities on each online CPU

2018-01-17 Thread Lars Kurth
Hi Julien, > On 17 Jan 2018, at 12:31, Julien Grall wrote: > > If you took the code from Linux, you need to add the original Signed-off-by from the Linux commit. Aside from that: >>> >>> There are multiple commits touching this function. So I followed what we >>> did in similar

[Xen-devel] [PATCH] Revert "xl: Default guest mode changed from PV to PVH with PV shim"

2018-01-17 Thread Ian Jackson
This breaks ARM. It should be protected by some x86 #if. For now, revert it, as it's not critical (and it isn't included in the comet/vixen security patch branches published via XSA-254). This reverts commit 63080b704351022cb7badb73339d47646fb465bd. Signed-off-by: Ian Jackson CC: Wei Liu CC:

Re: [Xen-devel] [PATCH 2/5] x86/pv: Rename invalidate_shadow_ldt() to pv_destroy_ldt()

2018-01-17 Thread Wei Liu
On Fri, Jan 12, 2018 at 06:37:21PM +, Andrew Cooper wrote: > and move it into pv/descriptor-tables.c beside its GDT counterpart. Reduce > the !in_irq() check from a BUG_ON() to ASSERT(). > > Signed-off-by: Andrew Cooper Reviewed-by: Wei Liu ___

Re: [Xen-devel] [PATCH 1/5] x86/idt: Factor out enabling and disabling of ISTs

2018-01-17 Thread Wei Liu
On Fri, Jan 12, 2018 at 06:37:20PM +, Andrew Cooper wrote: > All alteration of IST settings (other than the crash path) happen in an > identical triple. Introduce helpers to keep the triple in sync, and reduce > the risk of opencoded mistakes. > > Signed-off-by: Andrew Cooper Reviewed-by: W

[Xen-devel] [xen-unstable-smoke test] 118160: regressions - FAIL

2018-01-17 Thread osstest service owner
flight 118160 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/118160/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-arm64-arm64-xl-xsm 12 guest-start fail REGR. vs. 118105 build-armhf

Re: [Xen-devel] [Embedded-pv-devel] Xen on RCarH3 Starter Kit

2018-01-17 Thread Andrii Anisov
Hello Ganesh On 17.01.18 15:33, Ganesh H wrote: Dear Andrii, Sorry for the confusion. I am having an RCarH3 starter kit. Is there a Xen port available for RCarH3 starter kit right now? We did not run XEN on h3ulcb. But it can be done with minimal efforts IMHO. If not what are the steps to c

Re: [Xen-devel] Access I2C bus from guest/DomU on ARM board

2018-01-17 Thread Andrii Anisov
Rajesh, On 17.01.18 16:03, Saumya Rajesh wrote: Just out of curiosity, is it possible to split the Driver for the Renesas RCar I2C unit [1] into frontend and backend to use the i2c bus from guest? Or to do something similar to PCI passthrough? Please forgive me if I sound illogical. I'm just c

Re: [Xen-devel] [PATCH 6/6] firmware/shim: fix build process to use POSIX find options

2018-01-17 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH 6/6] firmware/shim: fix build process to use POSIX find options"): > The -printf find option is not POSIX compatible, so replace it with > another rune. ... > cd $(D)/$(d); \ > - find $(XEN_ROOT)/$(d)/ -type d -printf "./%P\n" | xargs

Re: [Xen-devel] [PATCH 6/6] firmware/shim: fix build process to use POSIX find options

2018-01-17 Thread Wei Liu
On Wed, Jan 17, 2018 at 04:24:27PM +, Ian Jackson wrote: > Roger Pau Monne writes ("[PATCH 6/6] firmware/shim: fix build process to use > POSIX find options"): > > The -printf find option is not POSIX compatible, so replace it with > > another rune. > ... > > cd $(D)/$(d); \ > >

[Xen-devel] [PATCH v2 3/6] xen/pvshim: identity pin shim vCPUs to pCPUs

2018-01-17 Thread Roger Pau Monne
Since VCPUOP_{up/down} already identity maps vCPU hotplug to pCPU hotplug also identity pin the vCPUs to the pCPUs in the scheduler. This prevents vCPU migration and should improve performance. While there also use __cpumask_set_cpu instead of cpumask_set_cpu, there's no need to use the locked var

[Xen-devel] [PATCH] ocaml: fix arm build

2018-01-17 Thread Wei Liu
ARM doesn't have emulation_flags in the arch_domainconfig. Signed-off-by: Wei Liu --- Cc: Ian Jackson Cc: Julien Grall Cc: Andrew Cooper --- tools/ocaml/libs/xc/xenctrl_stubs.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/tools/ocaml/libs/xc/xenctrl_stubs.c b/tools/ocaml/libs/xc

Re: [Xen-devel] [PATCH v3] kexec-tools: Perform run-time linking of libxenctrl.so

2018-01-17 Thread Eric DeVolder
Responses are inlined below. Eric On 01/16/2018 03:39 PM, Daniel Kiper wrote: On Fri, Jan 12, 2018 at 03:21:13PM -0600, Eric DeVolder wrote: When kexec is utilized in a Xen environment, it has an explicit run-time dependency on libxenctrl.so. This dependency occurs during the configure stage an

[Xen-devel] [PATCH v4] kexec-tools: Perform run-time linking of libxenctrl.so

2018-01-17 Thread Eric DeVolder
When kexec is utilized in a Xen environment, it has an explicit run-time dependency on libxenctrl.so. This dependency occurs during the configure stage and when building kexec-tools. When kexec is utilized in a non-Xen environment (either bare metal or KVM), the configure and build of kexec-tools

[Xen-devel] [xen-unstable test] 118136: regressions - trouble: broken/fail/pass

2018-01-17 Thread osstest service owner
flight 118136 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/118136/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-i386-pvgrub broken test-amd64-amd64-i386-pvgrub 4 host-inst

[Xen-devel] [xen-unstable-smoke test] 118162: regressions - FAIL

2018-01-17 Thread osstest service owner
flight 118162 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/118162/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-armhf 6 xen-buildfail REGR. vs. 118105 Tests which

Re: [Xen-devel] [PATCH] ocaml: fix arm build

2018-01-17 Thread Julien Grall
Hi Wei, On 17/01/18 16:43, Wei Liu wrote: ARM doesn't have emulation_flags in the arch_domainconfig. Signed-off-by: Wei Liu Reviewed-by: Julien Grall Cheers, --- Cc: Ian Jackson Cc: Julien Grall Cc: Andrew Cooper --- tools/ocaml/libs/xc/xenctrl_stubs.c | 6 ++ 1 file changed, 6

Re: [Xen-devel] [PATCH 6/6] firmware/shim: fix build process to use POSIX find options

2018-01-17 Thread Ian Jackson
Wei Liu writes ("Re: [PATCH 6/6] firmware/shim: fix build process to use POSIX find options"): > On Wed, Jan 17, 2018 at 04:24:27PM +, Ian Jackson wrote: > > And finally, I don't really understand why it is necessary to strip > > $(XEN_ROOT)/$(d)/ out at all. > > We want the path without $(XE

Re: [Xen-devel] [PATCH 6/6] firmware/shim: fix build process to use POSIX find options

2018-01-17 Thread Wei Liu
On Wed, Jan 17, 2018 at 04:55:01PM +, Ian Jackson wrote: > Wei Liu writes ("Re: [PATCH 6/6] firmware/shim: fix build process to use > POSIX find options"): > > On Wed, Jan 17, 2018 at 04:24:27PM +, Ian Jackson wrote: > > > And finally, I don't really understand why it is necessary to strip

Re: [Xen-devel] [PATCH] ocaml: fix arm build

2018-01-17 Thread Wei Liu
CC Ocaml experts On Wed, Jan 17, 2018 at 04:43:54PM +, Wei Liu wrote: > ARM doesn't have emulation_flags in the arch_domainconfig. > > Signed-off-by: Wei Liu > --- > Cc: Ian Jackson > Cc: Julien Grall > Cc: Andrew Cooper > --- > tools/ocaml/libs/xc/xenctrl_stubs.c | 6 ++ > 1 file ch

Re: [Xen-devel] [PATCH 6/6] firmware/shim: fix build process to use POSIX find options

2018-01-17 Thread Ian Jackson
Wei Liu writes ("Re: [PATCH 6/6] firmware/shim: fix build process to use POSIX find options"): > On Wed, Jan 17, 2018 at 04:55:01PM +, Ian Jackson wrote: > > FYI I am going to ship the patch as is to "comet" users via an XSA-254 > > update. I'm just quibbling about this for #staging. > > Yea

Re: [Xen-devel] [PATCH 5/5] xen/arm64: Implement branch predictor hardening for affected Cortex-A CPUs

2018-01-17 Thread Stefano Stabellini
On Wed, 17 Jan 2018, Julien Grall wrote: > Hi Stefano, > > On 17/01/18 00:42, Stefano Stabellini wrote: > > On Tue, 16 Jan 2018, Julien Grall wrote: > > > #define MIDR_RANGE(model, min, max) \ > > > @@ -205,6 +232,28 @@ static const struct arm_cpu_capabilities arm_errata[] > > > = { > > >

[Xen-devel] Xen Security Advisory 254 (CVE-2017-5753, CVE-2017-5715, CVE-2017-5754) - Information leak via side effects of speculative execution

2018-01-17 Thread Xen . org security team
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 Xen Security Advisory CVE-2017-5753,CVE-2017-5715,CVE-2017-5754 / XSA-254 version 9 Information leak via side effects of speculative execution UPDATES IN VERSION 9 "Stage 1" pagetable

Re: [Xen-devel] [PATCH 1/5] xen/arm: Introduce enable callback to enable a capabilities on each online CPU

2018-01-17 Thread Stefano Stabellini
On Wed, 17 Jan 2018, Lars Kurth wrote: > Regarding README.source, this is covering file and contain the same > mention as in the commit message. As this is a single function. Isn't the > commit message > enough? > > > From a legal viewpoint it is enough. If that is enough from a le

Re: [Xen-devel] [PATCH v8 15/17] x86/ctxt: Issue a speculation barrier between vcpu contexts

2018-01-17 Thread David Woodhouse
On Mon, 2018-01-15 at 22:39 +0100, David Woodhouse wrote: > On Mon, 2018-01-15 at 14:23 +0100, David Woodhouse wrote: > > > > > > > > > > > > > > >   > > > > Also... if you're doing that in context_switch() does it do the right > > > > thing with idle? If a CPU switches to the idle domain and t

Re: [Xen-devel] [PATCH 6/6] firmware/shim: fix build process to use POSIX find options

2018-01-17 Thread Roger Pau Monné
On Wed, Jan 17, 2018 at 04:24:27PM +, Ian Jackson wrote: > Roger Pau Monne writes ("[PATCH 6/6] firmware/shim: fix build process to use > POSIX find options"): > > The -printf find option is not POSIX compatible, so replace it with > > another rune. > ... > > cd $(D)/$(d); \ > >

[Xen-devel] [seabios test] 118140: regressions - FAIL

2018-01-17 Thread osstest service owner
flight 118140 seabios real [real] http://logs.test-lab.xenproject.org/osstest/logs/118140/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemuu-ws16-amd64 17 guest-stop fail REGR. vs. 115539 Tests which did not suc

Re: [Xen-devel] [PATCH 6/6] firmware/shim: fix build process to use POSIX find options

2018-01-17 Thread Andrew Cooper
On 17/01/18 17:32, Roger Pau Monné wrote: > On Wed, Jan 17, 2018 at 04:24:27PM +, Ian Jackson wrote: >> Roger Pau Monne writes ("[PATCH 6/6] firmware/shim: fix build process to use >> POSIX find options"): >>> The -printf find option is not POSIX compatible, so replace it with >>> another rune

Re: [Xen-devel] [PATCH v3 6/6] x86/domctl: Remove XEN_DOMCTL_pin_mem_cacheattr

2018-01-17 Thread Daniel De Graaf
On 01/12/2018 07:45 AM, Ross Lagerwall wrote: Remove the implementation of XEN_DOMCTL_pin_mem_cacheattr since it has been replaced by a dmop. Change xc_domain_pin_memory_cacheattr() so that it is only defined when XC_WANT_COMPAT_DEVICEMODEL_API is set and have it call the new dmop. Leave the def

Re: [Xen-devel] [PATCH 4/5] xen/arm64: Add skeleton to harden the branch predictor aliasing attacks

2018-01-17 Thread Stefano Stabellini
On Tue, 16 Jan 2018, Julien Grall wrote: > Aliasing attacked against CPU branch predictors can allow an attacker to > redirect speculative control flow on some CPUs and potentially divulge > information from one context to another. > > This patch adds initial skeleton code behind a new Kconfig opt

[Xen-devel] [xen-unstable-smoke test] 118173: regressions - FAIL

2018-01-17 Thread osstest service owner
flight 118173 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/118173/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-armhf 6 xen-buildfail REGR. vs. 118105 Tests which

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

2018-01-17 Thread osstest service owner
flight 118137 libvirt real [real] http://logs.test-lab.xenproject.org/osstest/logs/118137/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-armhf-armhf-libvirt-xsm 14 saverestore-support-checkfail like 117772 test-armhf-armhf-libvirt 14 saveresto

[Xen-devel] [linux-3.18 test] 118149: regressions - FAIL

2018-01-17 Thread osstest service owner
flight 118149 linux-3.18 real [real] http://logs.test-lab.xenproject.org/osstest/logs/118149/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-examine 6 xen-install fail REGR. vs. 117702 Tests which did not

[Xen-devel] [xen-unstable-smoke test] 118178: regressions - FAIL

2018-01-17 Thread osstest service owner
flight 118178 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/118178/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-armhf 6 xen-buildfail REGR. vs. 118105 Tests which

Re: [Xen-devel] [PATCH 1/5] xen/arm: Introduce enable callback to enable a capabilities on each online CPU

2018-01-17 Thread Stefano Stabellini
On Wed, 17 Jan 2018, Stefano Stabellini wrote: > On Wed, 17 Jan 2018, Lars Kurth wrote: > > Regarding README.source, this is covering file and contain the same > > mention as in the commit message. As this is a single function. Isn't the > > commit message > > enough? > > > > > > Fr

[Xen-devel] [xen-unstable-smoke test] 118187: regressions - FAIL

2018-01-17 Thread osstest service owner
flight 118187 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/118187/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-armhf 6 xen-buildfail REGR. vs. 118105 Tests which

[Xen-devel] [xen-unstable-smoke test] 118194: regressions - FAIL

2018-01-17 Thread osstest service owner
flight 118194 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/118194/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-armhf 6 xen-buildfail REGR. vs. 118105 Tests which

[Xen-devel] [PULL 10/19] xen: Add only xen-sysdev to dynamic sysbus device list

2018-01-17 Thread Eduardo Habkost
There's no need to make the machine allow every possible sysbus device. We can now just add xen-sysdev to the allowed list. Cc: Stefano Stabellini Cc: Anthony Perard Cc: xen-devel@lists.xenproject.org Cc: Juergen Gross Signed-off-by: Eduardo Habkost Message-Id: <20171125151610.20547-6-ehabk..

[Xen-devel] [PULL 06/19] machine: Replace has_dynamic_sysbus with list of allowed devices

2018-01-17 Thread Eduardo Habkost
The existing has_dynamic_sysbus flag makes the machine accept every user-creatable sysbus device type on the command-line. Replace it with a list of allowed device types, so machines can easily accept some sysbus devices while rejecting others. To keep exactly the same behavior as before, the exis

[Xen-devel] [xen-unstable-smoke test] 118197: regressions - FAIL

2018-01-17 Thread osstest service owner
flight 118197 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/118197/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-armhf 6 xen-buildfail REGR. vs. 118105 Tests which

[Xen-devel] [RFC PATCH] arm64: vgic-v3: Add ITS doorbell region in Dom0 stage-2

2018-01-17 Thread mjaggi
From: Manish Jaggi This patch introduces a function vgic_map_translation_space for mapping ITS 64K translater space for doorbells in dom0 stage-2. It is required as dom0 PCI devices behined SMMU wont be able to write MSIs. This function is called from vgic_v3_its_init_domain only for hardware d

[Xen-devel] [xen-unstable-smoke test] 118199: regressions - FAIL

2018-01-17 Thread osstest service owner
flight 118199 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/118199/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-armhf 6 xen-buildfail REGR. vs. 118105 Tests which

[Xen-devel] [linux-4.9 test] 118151: trouble: broken/fail/pass

2018-01-17 Thread osstest service owner
flight 118151 linux-4.9 real [real] http://logs.test-lab.xenproject.org/osstest/logs/118151/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-xl-rtds broken Regressions which are r

  1   2   >