[Xen-devel] [ovmf test] 129268: all pass - PUSHED

2018-10-31 Thread osstest service owner
flight 129268 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/129268/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf fd950156738ce878c0ba29af99da2ffcac5168e1 baseline version: ovmf

Re: [Xen-devel] Does XEN ARM support RTC in domu?

2018-10-31 Thread Peng Fan
Hi Julien, > -Original Message- > From: Julien Grall [mailto:julien.gr...@arm.com] > Sent: 2018年11月1日 2:52 > To: Peng Fan ; xen-devel@lists.xenproject.org; Stefano > Stabellini > Subject: Re: Does XEN ARM support RTC in domu? > > > > On 10/31/18 1:57 PM, Peng Fan wrote: > > Hi All, >

Re: [Xen-devel] XSM SILO boot time spew

2018-10-31 Thread Xin Li (Talons)
In patchset v4, we call register_xsm() to setup silo module. This debug log is to check if some ops not overrided by the module. I thought this is OK, since the log level is debug. I think calling register_xsm() is good, if we do want to suppress this debug log explicitly, we can check if ops

Re: [Xen-devel] [PATCH v1 5/6] x86/vvmx: correctly report vvmcs size

2018-10-31 Thread Tian, Kevin
> From: Sergey Dyasli [mailto:sergey.dya...@citrix.com] > Sent: Tuesday, October 30, 2018 8:36 PM > > On 30/10/2018 08:06, Tian, Kevin wrote: > >> From: Sergey Dyasli [mailto:sergey.dya...@citrix.com] > >> Sent: Friday, October 12, 2018 11:28 PM > >> > >> The size of Xen's virtual vmcs region is

Re: [Xen-devel] [PATCH v1 1/6] x86/vvmx: introduce vvmcx_valid()

2018-10-31 Thread Tian, Kevin
> From: Sergey Dyasli [mailto:sergey.dya...@citrix.com] > Sent: Tuesday, October 30, 2018 8:41 PM > > On 30/10/2018 07:41, Tian, Kevin wrote: > >> From: Sergey Dyasli [mailto:sergey.dya...@citrix.com] > >> Sent: Friday, October 12, 2018 11:28 PM > >> > >> As a convenient helper function and

[Xen-devel] [qemu-mainline test] 129176: tolerable FAIL - PUSHED

2018-10-31 Thread osstest service owner
flight 129176 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/129176/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-amd64-amd64-xl-qemuu-win7-amd64 17 guest-stopfail like 129131 test-armhf-armhf-libvirt 14

Re: [Xen-devel] Interrupt injection with ISR set on Intel hardware

2018-10-31 Thread Tian, Kevin
> From: Tian, Kevin > Sent: Tuesday, October 30, 2018 3:00 PM > > > From: Jan Beulich [mailto:jbeul...@suse.com] > > Sent: Thursday, October 25, 2018 9:58 PM > > > > >>> On 25.10.18 at 15:02, wrote: > > > On 25/10/18 13:51, Jan Beulich wrote: > > > On 15.10.18 at 14:06, wrote: > > >>> From

[Xen-devel] [linux-linus bisection] complete test-amd64-i386-xl-raw

2018-10-31 Thread osstest service owner
branch xen-unstable xenbranch xen-unstable job test-amd64-i386-xl-raw 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 git://xenbits.xen.org/qemu-xen-traditional.git

[Xen-devel] [PATCH v3] CONFIG_XEN_PV breaks xen_create_contiguous_region on ARM

2018-10-31 Thread Stefano Stabellini
From: Stefano Stabellini xen_create_contiguous_region has now only an implementation if CONFIG_XEN_PV is defined. However, on ARM we never set CONFIG_XEN_PV but we do have an implementation of xen_create_contiguous_region which is required for swiotlb-xen to work correctly (although it just sets

Re: [Xen-devel] Xen optimization

2018-10-31 Thread Julien Grall
On 10/31/18 8:35 PM, Milan Boberic wrote: Hi, Interesting. Could you confirm the commit you were using (or the point release)? Stefano's number were based on commit "fuzz: update README.afl example" 55a04feaa1f8ab6ef7d723fbb1d39c6b96ad184a which is an unreleased version of Xen. All Xens I

Re: [Xen-devel] Xen optimization

2018-10-31 Thread Milan Boberic
Hi, > Interesting. Could you confirm the commit you were using (or the point > release)? > Stefano's number were based on commit "fuzz: update README.afl example" > 55a04feaa1f8ab6ef7d723fbb1d39c6b96ad184a which is an unreleased version > of Xen. All Xens I used are from Xilinx git repository

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

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

Re: [Xen-devel] [PATCH] libxl/arm: fix guest type conversion

2018-10-31 Thread Julien Grall
Hi Wei, On 10/31/18 2:25 PM, Wei Liu wrote: Commit 359970fd8b ("tools/libxl: Switch Arm guest type to PVH") missed changing the type field in c_info. This issue didn't surface until ef72c93df9 which made creating PV guest on Arm unusable. Signed-off-by: Wei Liu --- Cc: Stefano Stabellini Cc:

Re: [Xen-devel] Xen PV: Sample new PV driver for buffer sharing between domains

2018-10-31 Thread Julien Grall
On 10/17/18 1:24 PM, Omkar Bolla wrote: Hi, Hi Omkar, I have started finding which patch introduced Armv8 Secondary CPUs issue. I just want to start PV vdevb before domainU debian rootfs mount. Is it possible? May I ask why you need the dependency on the rootfs? Cheers, -- Julien

[Xen-devel] [linux-linus test] 129167: regressions - FAIL

2018-10-31 Thread osstest service owner
flight 129167 linux-linus real [real] http://logs.test-lab.xenproject.org/osstest/logs/129167/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-qemuu-dmrestrict-amd64-dmrestrict 7 xen-boot fail REGR. vs. 125898

Re: [Xen-devel] Xen optimization

2018-10-31 Thread Julien Grall
Hi Milan, On 10/29/18 12:29 PM, Milan Boberic wrote: Sorry for late reply, Don't worry, thank you for the testing and sending the .config. I am afraid no. .config is generated during building time. So can you paste here please. ".config" file is in attachment. I also tried Xen 4.9 and

Re: [Xen-devel] Does XEN ARM support RTC in domu?

2018-10-31 Thread Julien Grall
On 10/31/18 1:57 PM, Peng Fan wrote: Hi All, Hi, Just have a question, does XEN ARM support RTC in domu? To support Android in DomU, RTC is needed for alarm, but I did not find information about RTC on xen for domu. So this need a new RTC paravirtualization driver? Any suggestions? By

Re: [Xen-devel] [OSSTEST PATCH] README.hardware-acquisition

2018-10-31 Thread Stefano Stabellini
On Wed, 31 Oct 2018, George Dunlap wrote: > [CC'ing Stefano and Julien] > On Wed, Oct 31, 2018 at 5:49 PM George Dunlap wrote: > > > > On Tue, Oct 30, 2018 at 4:14 PM Ian Jackson > > wrote: > > > + + Baremetal boot from Debian stable or stable-backports: > > > + > > > + A suitable Linux

Re: [Xen-devel] [OSSTEST PATCH] README.hardware-acquisition [and 1 more messages]

2018-10-31 Thread Stefano Stabellini
> Julien Grall writes ("Re: [OSSTEST PATCH] README.hardware-acquisition"): > > Most of the time, the issue to boot Debian is related to the kernel. > > Yet, the price might be too high for some of the vendors to donate the > > hardware. [...] > > > > I still want to encourage vendor to work

Re: [Xen-devel] [OSSTEST PATCH] README.hardware-acquisition

2018-10-31 Thread Julien Grall
Hi Ian, On 10/31/18 2:44 PM, Ian Jackson wrote: I'm going to reply to this email twice. This email is about what seem to be the noncontentious issues: everything *except* the requirement in my document that the board is supported by Debian -backports kernels, at least. Julien Grall writes

[Xen-devel] [PATCH v2 18/21] xen/arm: Remove unnecessary includes in asm-arm/acpi.h

2018-10-31 Thread Julien Grall
Signed-off-by: Julien Grall Reviewed-by: Andrii Anisov --- Changes in v2: - Add Andrii's reviewed-by --- xen/include/asm-arm/acpi.h | 1 - 1 file changed, 1 deletion(-) diff --git a/xen/include/asm-arm/acpi.h b/xen/include/asm-arm/acpi.h index feec4fb0ac..50340281a9 100644 ---

[Xen-devel] [PATCH v2 03/21] xen/arm: Remove __init from prototype

2018-10-31 Thread Julien Grall
In Xen, it is common to add __init to the declaration and not the prototype. Remove the few __init on some prototypes which allows to avoid the inclusion of init.h in headers. With these changes, init.h is now required to be included on some c files. Also, add __init where it was missing in

[Xen-devel] [PATCH v2 06/21] xen/arm: Move VABORT_GEN_BY_GUEST to traps.h and turned into inline

2018-10-31 Thread Julien Grall
The macro VABORT_GEN_BY_GUEST is only used by the trap code. So move it to trap.h. While moving the code, convert is to a static inline to allow typecheck. Signed-off-by: Julien Grall Reviewed-by: Andrii Anisov --- Changes in v2: - Add Andrii's reviewed-by ---

[Xen-devel] [PATCH v2 17/21] xen/arm: Remove unnecessary includes in asm/p2m.h

2018-10-31 Thread Julien Grall
Signed-off-by: Julien Grall Reviewed-by: Andrii Anisov --- Changes in v2: - Add Andrii's reviewed-by --- xen/include/asm-arm/p2m.h | 3 --- 1 file changed, 3 deletions(-) diff --git a/xen/include/asm-arm/p2m.h b/xen/include/asm-arm/p2m.h index c03557544a..be58125fb7 100644 ---

[Xen-devel] [PATCH v2 14/21] xen/arm: Remove unnecessary includes in asm/vgic.h

2018-10-31 Thread Julien Grall
Signed-off-by: Julien Grall Reviewed-by: Andrii Anisov --- Changes in v2: - Add Andrii's reviewed-by --- xen/include/asm-arm/vgic.h | 3 --- 1 file changed, 3 deletions(-) diff --git a/xen/include/asm-arm/vgic.h b/xen/include/asm-arm/vgic.h index 760392f9ef..0316d87f66 100644 ---

[Xen-devel] [PATCH v2 19/21] xen/arm: Remove unnecessary includes in asm/current.h

2018-10-31 Thread Julien Grall
Signed-off-by: Julien Grall Reviewed-by: Andrii Anisov --- Changes in v2: - Add Andrii's reviewed-by --- xen/include/asm-arm/current.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/xen/include/asm-arm/current.h b/xen/include/asm-arm/current.h index f9819b34fc..c4af66fbb9

[Xen-devel] [PATCH v2 00/21] xen/arm: Bunch of clean-ups

2018-10-31 Thread Julien Grall
Hi all, This patch series is a bunch of clean-ups I have collected the past few weeks. The main goal is to reduce the scope of some includes and reduce the size of processor.h by moving some parts in new/current headers. All the patches but #1 and #3 have been reviewed by Andrii. Cheers,

[Xen-devel] [PATCH v2 04/21] xen/arm: bugs: Move do_bug_frame to traps.h

2018-10-31 Thread Julien Grall
do_bug_frame is only necessary when trapping. This allows to remove processor.h include. However, time.h was missing an include resulting to compilation error if processor.h is removed from bug.h. Signed-off-by: Julien Grall Reviewed-by: Andrii Anisov --- Changes in v2: - Rebase

[Xen-devel] [PATCH v2 01/21] xen/arm: traps: Constify show_*, do_unexpected_trap and do_bug_frame parameters

2018-10-31 Thread Julien Grall
Those helpers are not meant to modify most of the parameters. So constify them. Signed-off-by: Julien Grall --- Changes in v2: - Constify more parameters. --- xen/arch/arm/traps.c| 26 +- xen/include/asm-arm/bug.h | 2 +-

[Xen-devel] [PATCH v2 21/21] xen/arm: Move vgic_* helpers from gic.h to vgic.h

2018-10-31 Thread Julien Grall
Keep vgic_* helpers in a single place. At the same time remove gic.h from event.h since the helpers has now been moved to vgic.h (included by domain.h). Signed-off-by: Julien Grall Reviewed-by: Andrii Anisov --- Changes in v2: - Add Andrii's reviewed-by ---

[Xen-devel] [PATCH v2 05/21] xen/arm: Consolidate CPU identification in cpufeature.{c, h}

2018-10-31 Thread Julien Grall
At the moment, CPU Identification is spread accross cpu.c, cpufeature.c, processor.h, cpufeature.h. It would be better to keep everything together in a single place. Signed-off-by: Julien Grall Reviewed-by: Andrii Anisov --- Changes in v2: - Add Andrii's reviewed-by ---

[Xen-devel] [PATCH v2 07/21] xen/arm: gic-3: Remove unused includes

2018-10-31 Thread Julien Grall
Signed-off-by: Julien Grall Reviewed-by: Andrii Anisov --- Changes in v2: - Add Andrii's reviewed-by --- xen/arch/arm/gic-v3.c | 5 - 1 file changed, 5 deletions(-) diff --git a/xen/arch/arm/gic-v3.c b/xen/arch/arm/gic-v3.c index 2952335d05..8ff4e0f08e 100644 ---

[Xen-devel] [PATCH v2 20/21] xen/arm: platform: Don't include p2m.h in exynos5 and omap5

2018-10-31 Thread Julien Grall
None of the platforms are using the p2m helpers. Signed-off-by: Julien Grall Reviewed-by: Andrii Anisov --- Changes in v2: - Add Andrii's reviewed-by --- xen/arch/arm/platforms/exynos5.c | 1 - xen/arch/arm/platforms/omap5.c | 1 - 2 files changed, 2 deletions(-) diff --git

[Xen-devel] [PATCH v2 12/21] xen/arm: Only include stringify.h when necessary

2018-10-31 Thread Julien Grall
Signed-off-by: Julien Grall Reviewed-by: Andrii Anisov --- Changes in v2: - Add Andrii's reviewed-by --- xen/include/asm-arm/arm32/sysregs.h | 1 + xen/include/asm-arm/arm64/processor.h | 2 -- xen/include/asm-arm/cpregs.h | 2 -- 3 files changed, 1 insertion(+), 4

[Xen-devel] [PATCH v2 11/21] xen/arm: Move out of processor.h traps related variable/function

2018-10-31 Thread Julien Grall
do_unexpected_traps() is moved to traps.h while init_traps() and hyp_traps_vectors() are moved to setup.h. Signed-off-by: Julien Grall Reviewed-by: Andrii Anisov --- Changes in v2: - Rebase - Add Andrii's reviewed-by --- xen/include/asm-arm/processor.h | 7 ---

[Xen-devel] [PATCH v2 08/21] xen/arm: gic-v3: Re-order includes in alphabetical order

2018-10-31 Thread Julien Grall
Signed-off-by: Julien Grall Reviewed-by: Andrii Anisov --- Changes in v2: - Add Andrii's reviewed-by --- xen/arch/arm/gic-v3.c | 22 -- 1 file changed, 12 insertions(+), 10 deletions(-) diff --git a/xen/arch/arm/gic-v3.c b/xen/arch/arm/gic-v3.c index

[Xen-devel] [PATCH v2 16/21] xen/arm: Remove unnecessary includes in traps.c

2018-10-31 Thread Julien Grall
Also, include smccc.h instead of psci.h. Signed-off-by: Julien Grall Reviewed-by: Andrii Anisov --- Changes in v2: - Add Andrii's reviewed-by --- xen/arch/arm/traps.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/xen/arch/arm/traps.c b/xen/arch/arm/traps.c

[Xen-devel] [PATCH v2 09/21] xen/arm: Move HSR defines in a new header hsr.h

2018-10-31 Thread Julien Grall
The HSR defines are pretty much self-contained and not necessary to be included everywhere in Xen. So move them in a new header hsr.h. Signed-off-by: Julien Grall Reviewed-by: Andrii Anisov --- Changes in v2: - Add Andrii's reviewed-by --- xen/arch/arm/arm64/traps.c | 1

[Xen-devel] [PATCH v2 13/21] xen/arm: Only include vreg.h when necessary

2018-10-31 Thread Julien Grall
Signed-off-by: Julien Grall Reviewed-by: Andrii Anisov --- Changes in v2: - Add Andrii's reviewed-by --- xen/arch/arm/vgic-v2.c | 1 + xen/arch/arm/vgic-v3-its.c | 1 + xen/arch/arm/vpl011.c| 1 + xen/include/asm-arm/vgic.h | 1 - xen/include/asm-arm/vpl011.h | 1

[Xen-devel] [PATCH v2 15/21] xen/arm: Remove unnecessary includes in asm/mmio.h

2018-10-31 Thread Julien Grall
Signed-off-by: Julien Grall Reviewed-by: Andrii Anisov --- Changes in v2: - Add Andrii's reviewed-by --- xen/include/asm-arm/mmio.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/xen/include/asm-arm/mmio.h b/xen/include/asm-arm/mmio.h index 3ed3f82bf7..8dbfb27682 100644 ---

[Xen-devel] [PATCH v2 10/21] xen/arm: Move SYSREG accessors in sysregs.h

2018-10-31 Thread Julien Grall
System registers accessors are self-contained and should not be included everywhere in Xen. Move the accessors in sysregs.h and include the file when necessary. With that change, it is not necessary to include processor.h in time.h. Signed-off-by: Julien Grall Reviewed-by: Andrii Anisov ---

[Xen-devel] [PATCH v2 02/21] xen/arm: regs: Convert guest_mode to a static inline helper

2018-10-31 Thread Julien Grall
At the same time, switch the parameter guest_mode from int to bool Signed-off-by: Julien Grall Reviewed-by: Andrii Anisov --- Changes in v2: - Add andrii's reviewed-by --- xen/arch/arm/traps.c | 6 +++--- xen/include/asm-arm/regs.h | 22 -- 2 files

Re: [Xen-devel] [OSSTEST PATCH] README.hardware-acquisition

2018-10-31 Thread George Dunlap
[CC'ing Stefano and Julien] On Wed, Oct 31, 2018 at 5:49 PM George Dunlap wrote: > > On Tue, Oct 30, 2018 at 4:14 PM Ian Jackson wrote: > > + + Baremetal boot from Debian stable or stable-backports: > > + > > + A suitable Linux kernel binary which can boot baremetal on the > > +

Re: [Xen-devel] [OSSTEST PATCH] README.hardware-acquisition

2018-10-31 Thread George Dunlap
On Tue, Oct 30, 2018 at 4:14 PM Ian Jackson wrote: > + + Baremetal boot from Debian stable or stable-backports: > + > + A suitable Linux kernel binary which can boot baremetal on the > + proposed hardware must be available from Debian (at least > + `stable', or, if that is not

Re: [Xen-devel] [PATCH v3 7/7] amd/pvh: enable ACPI C1E disable quirk on PVH Dom0

2018-10-31 Thread Boris Ostrovsky
On 10/30/18 11:41 AM, Roger Pau Monne wrote: > PV Dom0 has a quirk for some AMD processors, where enabling ACPI can > also enable C1E mode. Apply the same workaround as done on PV for a > PVH Dom0, which consist on trapping accesses to the SMI command IO > port and disabling C1E if ACPI is

[Xen-devel] [libvirt test] 129168: regressions - FAIL

2018-10-31 Thread osstest service owner
flight 129168 libvirt real [real] http://logs.test-lab.xenproject.org/osstest/logs/129168/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-i386-libvirt6 libvirt-buildfail REGR. vs. 128963 build-armhf-pvops

Re: [Xen-devel] [PATCH v4 1/4] x86: use PDEP/PEXT for maddr/direct-map-offset conversion when available

2018-10-31 Thread Wei Liu
On Tue, Sep 18, 2018 at 06:36:21AM -0600, Jan Beulich wrote: > This allows to fold 6 instructions into a single one, reducing code size > quite a bit, especially when not considering the fallback functions > (which won't ever need to be brought into iCache or their mappings into > iTLB on systems

Re: [Xen-devel] [PATCH v2 1/1] tools/ocaml: Re-introduce Xenctrl.with_intf wrapper [and 1 more messages]

2018-10-31 Thread Ian Jackson
Christian Lindig writes ("Re: [PATCH v2 1/1] tools/ocaml: Re-introduce Xenctrl.with_intf wrapper [and 1 more messages]"): > On 31/10/18 16:01, Ian Jackson wrote: > > IMO the bug is that spong calls close_handle which has a distant > > destructive action, without being able to know that it is safe

Re: [Xen-devel] [PATCH v2 1/1] tools/ocaml: Re-introduce Xenctrl.with_intf wrapper [and 1 more messages]

2018-10-31 Thread Christian Lindig
On 31/10/18 16:01, Ian Jackson wrote: IMO the bug is that spong calls close_handle which has a distant destructive action, without being able to know that it is safe to do so. I will add add more words or caution. Thanks for pointing it out. -- C

Re: [Xen-devel] [PATCH V6] x86/altp2m: Add a subop for obtaining the mem access of a page

2018-10-31 Thread Wei Liu
On Wed, Oct 31, 2018 at 06:45:49PM +0200, Razvan Cojocaru wrote: > On 10/31/18 6:39 PM, Wei Liu wrote: > > On Thu, Sep 27, 2018 at 10:58:54AM +0300, Razvan Cojocaru wrote: > >> Currently there is a subop for setting the memaccess of a page, but not > >> for consulting it. The new

Re: [Xen-devel] [PATCH V6] x86/altp2m: Add a subop for obtaining the mem access of a page

2018-10-31 Thread Razvan Cojocaru
On 10/31/18 6:39 PM, Wei Liu wrote: > On Thu, Sep 27, 2018 at 10:58:54AM +0300, Razvan Cojocaru wrote: >> Currently there is a subop for setting the memaccess of a page, but not >> for consulting it. The new HVMOP_altp2m_get_mem_access adds this >> functionality. >> >> Both altp2m get/set mem

Re: [Xen-devel] [PATCH] x86: deal with firmware setting bogus TSC_ADJUST values

2018-10-31 Thread Wei Liu
On Thu, Oct 25, 2018 at 09:09:25AM -0600, Jan Beulich wrote: > >>> On 12.10.18 at 18:29, wrote: > > On Mon, Oct 01, 2018 at 07:42:12AM -0600, Jan Beulich wrote: > >> The system Intel have handed me for AVX512 emulator work ("Gigabyte > >> Technology Co., Ltd. X299 AORUS Gaming 3 Pro/X299 AORUS

Re: [Xen-devel] [PATCH V6] x86/altp2m: Add a subop for obtaining the mem access of a page

2018-10-31 Thread Wei Liu
On Thu, Sep 27, 2018 at 10:58:54AM +0300, Razvan Cojocaru wrote: > Currently there is a subop for setting the memaccess of a page, but not > for consulting it. The new HVMOP_altp2m_get_mem_access adds this > functionality. > > Both altp2m get/set mem access functions use the struct >

Re: [Xen-devel] [PATCH v3 2/7] x86/hvm: introduce a define for the debug output IO port

2018-10-31 Thread Wei Liu
On Tue, Oct 30, 2018 at 04:41:18PM +0100, Roger Pau Monne wrote: > No functional change intended. > > Signed-off-by: Roger Pau Monné > Reviewed-by: Jan Beulich Reviewed-by: Wei Liu ___ Xen-devel mailing list Xen-devel@lists.xenproject.org

[Xen-devel] [ovmf test] 129238: all pass - PUSHED

2018-10-31 Thread osstest service owner
flight 129238 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/129238/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf e23366d3c3eafae69cf0436f4212ed126a7534f1 baseline version: ovmf

[Xen-devel] [ovmf baseline-only test] 75550: tolerable FAIL

2018-10-31 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 75550 ovmf real [real] http://osstest.xensource.com/osstest/logs/75550/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: build-amd64-libvirt 6 libvirt-buildfail like 75548

[Xen-devel] [freebsd-master test] 129229: all pass - PUSHED

2018-10-31 Thread osstest service owner
flight 129229 freebsd-master real [real] http://logs.test-lab.xenproject.org/osstest/logs/129229/ Perfect :-) All tests in this flight passed as required version targeted for testing: freebsd 2f480ce108ecce5a6d0e78d011781a5e33bd4c67 baseline version: freebsd

Re: [Xen-devel] [PATCH v2 1/1] tools/ocaml: Re-introduce Xenctrl.with_intf wrapper [and 1 more messages]

2018-10-31 Thread Ian Jackson
Christian Lindig writes ("Re: [PATCH v2 1/1] tools/ocaml: Re-introduce Xenctrl.with_intf wrapper [and 1 more messages]"): > On 31/10/18 15:41, Ian Jackson wrote: > > this needs a stronger warning against anyone ever calling it, or a > > clearer explanation of the consquences (whose scope is very

Re: [Xen-devel] [PATCH v2] tools/misc/xenpm: fix getting info when some CPUs are offline

2018-10-31 Thread Wei Liu
On Wed, Oct 31, 2018 at 02:04:58PM +0100, Marek Marczykowski-Górecki wrote: > Use physinfo.max_cpu_id instead of physinfo.nr_cpus to get max CPU id. > This fixes for example 'xenpm get-cpufreq-para' with smt=off, which > otherwise would miss half of the cores. > > Signed-off-by: Marek

Re: [Xen-devel] [PATCH v2 9/9] viridian: introduce struct viridian_page

2018-10-31 Thread Wei Liu
On Wed, Oct 31, 2018 at 12:43:35PM +, Paul Durrant wrote: > The 'vp_assist' page is currently an example of a guest page which needs to > be kept mapped throughout the life-time of a guest, but there are other > such examples in the specifiction [1]. This patch therefore introduces a > generic

Re: [Xen-devel] [PATCH v2 1/1] tools/ocaml: Re-introduce Xenctrl.with_intf wrapper [and 1 more messages]

2018-10-31 Thread Christian Lindig
On 31/10/18 15:41, Ian Jackson wrote: this needs a stronger warning against anyone ever calling it, or a clearer explanation of the consquences (whose scope is very broad). The consequence is that the global handle is closed but it would be opened again if one called with_intf again. So I'm

[Xen-devel] dom0/pvh: Follow up discussion from xen-devel - Low hanging fruit

2018-10-31 Thread Alexandru Vasile
On Wed, Oct 31, 2018 at 05:07 PM +0200, Roger Pau Monné wrote: On Wed, Oct 31, 2018 at 04:38:08PM +0200, Alexandru Vasile wrote: > Hello, > > > > I am writing as a follow up from our discussion on the xen-devel mailing > > list [0]. > > > You mentioned some ‘low-hanging fruit, both in

Re: [Xen-devel] [PATCH v2 7/9] viridian: define type for the 'virtual VP assist page'

2018-10-31 Thread Wei Liu
On Wed, Oct 31, 2018 at 12:43:33PM +, Paul Durrant wrote: > The specification [1] defines a type so we should use it, rather than just > OR-ing and AND-ing magic bits. > > No functional change. > > NOTE: The type defined in the specification does include an anonymous > sub-struct in

Re: [Xen-devel] [PATCH v2 6/9] viridian: separate time related enlightenment implementations...

2018-10-31 Thread Wei Liu
On Wed, Oct 31, 2018 at 12:43:32PM +, Paul Durrant wrote: > ...into new 'time' module. > > This patch reduces the size of the main viridian source module by > moving time related enlightenments into their own source module. This is > done in anticipation of implementation of more such

Re: [Xen-devel] [PATCH v2 1/1] tools/ocaml: Re-introduce Xenctrl.with_intf wrapper [and 1 more messages]

2018-10-31 Thread Ian Jackson
Christian Lindig writes ("[PATCH 1/1] tools/ocaml: Re-introduce Xenctrl.with_intf wrapper"): > Commit 81946a73dc975a7dafe9017a8e61d1e64fdbedbf removed > Xenctrl.with_intf based on its undesirable behaviour of opening and > closing a Xenctrl connection with every invocation. This commit >

Re: [Xen-devel] [PATCH v2 5/9] viridian: separate interrupt related enlightenment implementations...

2018-10-31 Thread Wei Liu
On Wed, Oct 31, 2018 at 12:43:31PM +, Paul Durrant wrote: > ...into new 'synic' module. > > The SynIC (synthetic interrupt controller) is specified [1] to be a super- > set of a virtualized LAPIC, and its definition encompasses all > enlightenments related to virtual interrupt control. > >

Re: [Xen-devel] [OSSTEST PATCH] README.hardware-acquisition [and 1 more messages]

2018-10-31 Thread Ian Jackson
I'm going to reply to this email twice. This email is about the requirement in my document that the board is supported by Debian -backports kernels, at least. I'm replying to two mail@: first Julien's; then Stefano's. Julien Grall writes ("Re: [OSSTEST PATCH] README.hardware-acquisition"): >

Re: [Xen-devel] [PATCH v8 6/8] docs: documentation about static shared memory regions

2018-10-31 Thread Stefano Stabellini
On Wed, 31 Oct 2018, Ian Jackson wrote: > Julien Grall writes ("Re: [PATCH v8 6/8] docs: documentation about static > shared memory regions"): > > On 10/30/18 6:58 PM, Stefano Stabellini wrote: > > > I completely agree with you here, and getting rid of the master/slave > > > terminology would be

Re: [Xen-devel] [RFC PATCH v2] tools: libxl/xl: run NUMA placement even when an hard-affinity is set

2018-10-31 Thread Wei Liu
On Fri, Oct 19, 2018 at 05:54:41PM +0200, Dario Faggioli wrote: > Right now, if either an hard or soft-affinity are explicitly specified > in a domain's config file, automatic NUMA placement is skipped. However, > automatic NUMA placement affects only the soft-affinity of the domain > which is

[Xen-devel] [PATCH v3] iommu / p2m: add a page_order parameter to iommu_map/unmap_page()...

2018-10-31 Thread Paul Durrant
...and re-name them to iommu_map/unmap() since they no longer necessarily operate on a single page. The P2M code currently contains many loops to deal with the fact that, while it may be require to handle page orders greater than 0, the IOMMU map and unmap functions do not. This patch adds a

Re: [Xen-devel] [PATCH v7 2/8] libxl: introduce a new structure to represent static shared memory regions

2018-10-31 Thread Wei Liu
On Wed, Sep 12, 2018 at 11:29:32AM +0100, Julien Grall wrote: > > > On 09/12/2018 12:09 AM, Stefano Stabellini wrote: > > > > + > > > > +/* Constants for libxl_static_shm */ > > > > +#define LIBXL_SSHM_RANGE_UNKNOWN UINT64_MAX > > > > +#define LIBXL_SSHM_ID_MAXLEN128 > > > > > > Why do you

Re: [Xen-devel] [PATCH] libxl/arm: fix guest type conversion

2018-10-31 Thread Wei Liu
On Wed, Oct 31, 2018 at 02:25:45PM +, Wei Liu wrote: > Commit 359970fd8b ("tools/libxl: Switch Arm guest type to PVH") missed > changing the type field in c_info. This issue didn't surface until > ef72c93df9 which made creating PV guest on Arm unusable. > > Signed-off-by: Wei Liu > --- > Cc:

[Xen-devel] [linux-linus bisection] complete test-amd64-i386-xl-qemuu-debianhvm-amd64-xsm

2018-10-31 Thread osstest service owner
branch xen-unstable xenbranch xen-unstable job test-amd64-i386-xl-qemuu-debianhvm-amd64-xsm 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

Re: [Xen-devel] [OSSTEST PATCH] README.hardware-acquisition

2018-10-31 Thread Ian Jackson
I'm going to reply to this email twice. This email is about what seem to be the noncontentious issues: everything *except* the requirement in my document that the board is supported by Debian -backports kernels, at least. Julien Grall writes ("Re: [OSSTEST PATCH] README.hardware-acquisition"):

Re: [Xen-devel] [PATCH v2] iommu / p2m: add a page_order parameter to iommu_map/unmap_page()

2018-10-31 Thread Paul Durrant
> -Original Message- > From: Roger Pau Monne > Sent: 30 October 2018 14:44 > To: Paul Durrant > Cc: xen-devel@lists.xenproject.org; Kevin Tian ; > Stefano Stabellini ; Wei Liu > ; Jun Nakajima ; Konrad > Rzeszutek Wilk ; Andrew Cooper > ; Ian Jackson ; George > Dunlap ; Tim (Xen.org) ;

[Xen-devel] [PATCH] libxl/arm: fix guest type conversion

2018-10-31 Thread Wei Liu
Commit 359970fd8b ("tools/libxl: Switch Arm guest type to PVH") missed changing the type field in c_info. This issue didn't surface until ef72c93df9 which made creating PV guest on Arm unusable. Signed-off-by: Wei Liu --- Cc: Stefano Stabellini Cc: Julien Grall Julien, can you give this a

Re: [Xen-devel] [PATCH v2 8/9] tools/misc: fix hard tabs in xen-hvmctx.c

2018-10-31 Thread Wei Liu
On Wed, Oct 31, 2018 at 12:43:34PM +, Paul Durrant wrote: > Also add emacs boilerplate to avoid future problems. > > Purely cosmetic. No functional change. > > Signed-off-by: Paul Durrant Acked-by: Wei Liu ___ Xen-devel mailing list

[Xen-devel] Does XEN ARM support RTC in domu?

2018-10-31 Thread Peng Fan
Hi All, Just have a question, does XEN ARM support RTC in domu? To support Android in DomU, RTC is needed for alarm, but I did not find information about RTC on xen for domu. So this need a new RTC paravirtualization driver? Any suggestions? Thanks, Peng.

Re: [Xen-devel] [PATCH v5 25/25] xen/arm: split domain_build.c

2018-10-31 Thread Julien Grall
On 10/23/18 3:03 AM, Stefano Stabellini wrote: domain_build.c is too large. Move all the ACPI specific device tree generating functions from domain_build.c to acpi/domain_build.c. Signed-off-by: Stefano Stabellini --- Changes in v4: - rename acpi_dt_build to domain_build.c - add copyright

Re: [Xen-devel] [PATCH v5 24/25] xen/arm: move kernel.h to asm-arm/

2018-10-31 Thread Julien Grall
Hi Stefano, On 10/23/18 3:03 AM, Stefano Stabellini wrote: It will be #included by a file in a xen/arch/arm subdirectory. Signed-off-by: Stefano Stabellini Reviewed-by: Julien Grall Cheers, --- xen/arch/arm/domain_build.c | 2 +- xen/arch/arm/kernel.c| 3 +-

Re: [Xen-devel] [PATCH v5 23/25] xen/vpl011: buffer out chars when the backend is xen

2018-10-31 Thread Julien Grall
Hi Stefano, On 10/23/18 3:03 AM, Stefano Stabellini wrote: diff --git a/xen/include/asm-arm/vpl011.h b/xen/include/asm-arm/vpl011.h index 5eb6d25..ab6fd79 100644 --- a/xen/include/asm-arm/vpl011.h +++ b/xen/include/asm-arm/vpl011.h @@ -30,9 +30,13 @@ #define VPL011_UNLOCK(d,flags)

[Xen-devel] [PATCH v2] tools/misc/xenpm: fix getting info when some CPUs are offline

2018-10-31 Thread Marek Marczykowski-Górecki
Use physinfo.max_cpu_id instead of physinfo.nr_cpus to get max CPU id. This fixes for example 'xenpm get-cpufreq-para' with smt=off, which otherwise would miss half of the cores. Signed-off-by: Marek Marczykowski-Górecki --- Reported by @tfm1: https://github.com/QubesOS/qubes-issues/issues/4456

Re: [Xen-devel] [PATCH v3] devicetree, xen: add xen, shared-memory binding

2018-10-31 Thread Julien Grall
Hi, On 10/24/18 2:18 AM, Stefano Stabellini wrote: From: Stefano Stabellini Introduce a device tree binding for Xen reserved-memory regions. They are used to share memory across VMs from the VM config files. (See static_shm config option.) Signed-off-by: Stefano Stabellini Cc:

[Xen-devel] [PATCH v2 6/9] viridian: separate time related enlightenment implementations...

2018-10-31 Thread Paul Durrant
...into new 'time' module. This patch reduces the size of the main viridian source module by moving time related enlightenments into their own source module. This is done in anticipation of implementation of more such enightenments and a desire to not further lengthen the main source module when

[Xen-devel] [PATCH v2 3/9] viridian: remove comments referencing section number in the spec.

2018-10-31 Thread Paul Durrant
Microsoft has a habit of re-numbering sections in the spec. so avoid referring to section numbers in comments. Also remove the URL for the spec. from the boilerplate... Again, Microsoft has a habit of changing these too. This patch also cleans up some > 80 character lines. Purely cosmetic. No

[Xen-devel] [PATCH v2 2/9] viridian: remove MSR perf counters

2018-10-31 Thread Paul Durrant
They're not really useful so maintaining them is pointless. Signed-off-by: Paul Durrant Reviewed-by: Roger Pau Monne --- Cc: Jan Beulich Cc: Andrew Cooper Cc: Wei Liu --- xen/arch/x86/hvm/viridian/viridian.c | 21 - xen/include/asm-x86/perfc_defn.h | 26

[Xen-devel] [PATCH v2 8/9] tools/misc: fix hard tabs in xen-hvmctx.c

2018-10-31 Thread Paul Durrant
Also add emacs boilerplate to avoid future problems. Purely cosmetic. No functional change. Signed-off-by: Paul Durrant --- Cc: Ian Jackson Cc: Wei Liu v2: - New in v2 --- tools/misc/xen-hvmctx.c | 16 +--- 1 file changed, 13 insertions(+), 3 deletions(-) diff --git

[Xen-devel] [PATCH v2 9/9] viridian: introduce struct viridian_page

2018-10-31 Thread Paul Durrant
The 'vp_assist' page is currently an example of a guest page which needs to be kept mapped throughout the life-time of a guest, but there are other such examples in the specifiction [1]. This patch therefore introduces a generic 'viridian_page' type and converts the current vp_assist/apic_assist

[Xen-devel] [PATCH v2 1/9] viridian: move the code into its own sub-directory

2018-10-31 Thread Paul Durrant
Subsequent patches will introduce support for more viridian enlightenments which will make a single source module quite lengthy. This patch therefore creates a new arch/x86/hvm/viridian sub-directory and moves viridian.c into that. The patch also fixes some bad whitespace whilst moving the code

[Xen-devel] [PATCH v2 5/9] viridian: separate interrupt related enlightenment implementations...

2018-10-31 Thread Paul Durrant
...into new 'synic' module. The SynIC (synthetic interrupt controller) is specified [1] to be a super- set of a virtualized LAPIC, and its definition encompasses all enlightenments related to virtual interrupt control. This patch reduces the size of the main viridian source module by giving

[Xen-devel] [PATCH v2 0/9] viridian cleanup

2018-10-31 Thread Paul Durrant
I plan to add implementations for more viridian enlightenments in the near future. This series is just some cleanup I've been doing in preparation. Paul Durrant (9): viridian: move the code into its own sub-directory viridian: remove MSR perf counters viridian: remove comments referencing

[Xen-devel] [PATCH v2 7/9] viridian: define type for the 'virtual VP assist page'

2018-10-31 Thread Paul Durrant
The specification [1] defines a type so we should use it, rather than just OR-ing and AND-ing magic bits. No functional change. NOTE: The type defined in the specification does include an anonymous sub-struct in the page type but, as we currently use only the first element, the

[Xen-devel] [PATCH v2 4/9] viridian: remove duplicate union types

2018-10-31 Thread Paul Durrant
The 'viridian_vp_assist', 'viridian_hypercall_gpa' and 'viridian_reference_tsc' union types are identical in layout. The layout is also common throughout the specification [1]. This patch declares a common 'viridian_page_msr' type and converts the rest of the code to use that type for both the

[Xen-devel] XSM SILO boot time spew

2018-10-31 Thread Andrew Cooper
Hello, I've noticed that the SILO code causes the following debug spew: (XEN) XSM Framework v1.0.0 initialized (XEN) Initialising XSM SILO mode (XEN) dummy.c:31: Had to override the security_domaininfo security operation with the dummy one. (XEN) dummy.c:32: Had to override the domain_create

Re: [Xen-devel] [PATCH v8 6/8] docs: documentation about static shared memory regions

2018-10-31 Thread Ian Jackson
Julien Grall writes ("Re: [PATCH v8 6/8] docs: documentation about static shared memory regions"): > On 10/30/18 6:58 PM, Stefano Stabellini wrote: > > I completely agree with you here, and getting rid of the master/slave > > terminology would be nice, in retrospect, it was not a good choice. But

[Xen-devel] [PATCH v2 1/1] tools/ocaml: Re-introduce Xenctrl.with_intf wrapper

2018-10-31 Thread Christian Lindig
This patch i updated: * set the global handle to None * updated documentation ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel

[Xen-devel] [PATCH 1/1] tools/ocaml: Re-introduce Xenctrl.with_intf wrapper

2018-10-31 Thread Christian Lindig
Commit 81946a73dc975a7dafe9017a8e61d1e64fdbedbf removed Xenctrl.with_intf based on its undesirable behaviour of opening and closing a Xenctrl connection with every invocation. This commit re-introduces with_intf but with an updated behaviour: it maintains a global Xenctrl connection which is

Re: [Xen-devel] [PATCH v8 7/8] xen/arm: export shared memory regions as reserved-memory on device tree

2018-10-31 Thread Ian Jackson
Julien Grall writes ("Re: [PATCH v8 7/8] xen/arm: export shared memory regions as reserved-memory on device tree"): > On 10/30/18 3:58 PM, Ian Jackson wrote: > > But it's not documented. > > Device-Tree bindings are documented in > linux/Document/device-tree/bindings. Stefano sent it via the DT

Re: [Xen-devel] [PATCH v8 3/8] libxl: support mapping static shared memory areas during domain creation

2018-10-31 Thread Ian Jackson
Julien Grall writes ("Re: [PATCH v8 3/8] libxl: support mapping static shared memory areas during domain creation"): > On 10/30/18 3:36 PM, Ian Jackson wrote: > > This cache policy stuff is odd. I couldn't see it being used by the > > hypervisor. Why is it even there ? > > We decided to not

Re: [Xen-devel] [PATCH] tools/misc/xenpm: fix getting info when some CPUs are offline

2018-10-31 Thread Jan Beulich
>>> On 31.10.18 at 11:53, wrote: > On Wed, Oct 31, 2018 at 03:33:58AM -0600, Jan Beulich wrote: >> I'm also afraid there are further quirks here, with various constructs >> along the lines of (as bodies of aforementioned for()) >> >> if ( show_cxstat_by_cpuid(xc_handle, i) == -ENODEV )

  1   2   >