Re: [Xen-devel] [PATCH v7 03/28] build: build Kconfig and config rules

2015-12-14 Thread Jan Beulich
>>> On 14.12.15 at 18:52, wrote: > On 12/14/15 10:35 AM, Jan Beulich wrote: > On 10.12.15 at 17:48, wrote: >>> --- /dev/null >>> +++ b/xen/arch/x86/Kconfig >>> @@ -0,0 +1,17 @@ >>> +config X86_64 >>> + def_bool y >>> + >>> +config X86 >>> + def_bool y >>> + >>> +config ARCH_DEFCONFIG >>>

Re: [Xen-devel] [PATCH v11 1/09] xen/x86: set the vPMU interface based on the presence of a lapic

2015-12-14 Thread Tian, Kevin
> From: Roger Pau Monne [mailto:roger@citrix.com] > Sent: Friday, December 11, 2015 6:17 PM > > Instead of choosing the interface to expose to guests based on the guest > type, do it based on whether the guest has an emulated local apic or not. > > Signed-off-by: Roger Pau Monné > Signed-off

[Xen-devel] [linux-3.10 bisection] complete test-amd64-i386-rumpuserxen-i386

2015-12-14 Thread osstest service owner
branch xen-unstable xenbranch xen-unstable job test-amd64-i386-rumpuserxen-i386 testid guest-start Tree: linux git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git Tree: linuxfirmware git://xenbits.xen.org/osstest/linux-firmware.git Tree: qemu git://xenbits.xen.org/qemu-xen-tradi

Re: [Xen-devel] [PATCH] x86/time: Don't use EFI's GetTime call by default

2015-12-14 Thread Jan Beulich
>>> On 08.12.15 at 12:02, wrote: > On Wed, 2015-12-02 at 02:33 -0700, Jan Beulich wrote: >> Then we should see about adding support for "efi=no-time". > > And based on what I'm reading in this thread about the reliability of the > time RS in the field it seems to me we should make it the default

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

2015-12-14 Thread Jim Fehlig
On 12/14/2015 03:41 AM, Stefano Stabellini wrote: > On Fri, 11 Dec 2015, Ian Jackson wrote: >> Ian Campbell writes ("Re: [libvirt test] 65654: regressions - FAIL"): >>> On Fri, 2015-12-11 at 15:18 +, osstest service owner wrote: flight 65654 libvirt real [real] http://logs.test-lab.x

[Xen-devel] [PATCH v2 3/4] libxc: stop migration in case of p2m list structural changes

2015-12-14 Thread Juergen Gross
With support of the virtual mapped linear p2m list for migration it is now possible to detect structural changes of the p2m list which before would either lead to a crashing or otherwise wrong behaving domU. A guest supporting the linear p2m list will increment the p2m_generation counter located i

[Xen-devel] [PATCH v2 2/4] libxc: support of linear p2m list for migration of pv-domains

2015-12-14 Thread Juergen Gross
In order to be able to migrate pv-domains with more than 512 GB of RAM the p2m information can be specified by the guest kernel via a virtual mapped linear p2m list instead of a 3 level tree. Add support for this new p2m format in libxc. As the sanity checking of the virtual p2m address needs def

[Xen-devel] [PATCH v2 0/4] support linear p2m list in migrate stream v2

2015-12-14 Thread Juergen Gross
Add support for the virtual mapped linear p2m list of pv-domains in the v2 migrate stream. This will allow to migrate domains larger than 512 GB. Tested with 32- and 64-bit pv-domains both with and without linear p2m list and with a hvm domain. Changes in V2: - Added some sanity tests in patch 2

[Xen-devel] [PATCH v2 1/4] libxc: split mapping p2m leaves into a separate function

2015-12-14 Thread Juergen Gross
In order to prepare using the virtual mapped linear p2m list for migration split mapping of the p2m leaf pages into a separate function. Signed-off-by: Juergen Gross Reviewed-by: Andrew Cooper --- tools/libxc/xc_sr_save_x86_pv.c | 77 - 1 file changed, 45

[Xen-devel] [PATCH v2 4/4] libxc: set flag for support of linear p2m list in domain builder

2015-12-14 Thread Juergen Gross
Set the SIF_VIRT_P2M_4TOOLS flag for pv-domUs in the domain builder to indicate the Xen tools have full support for the virtual mapped linear p2m list. This will enable pv-domUs to drop support of the 3 level p2m tree and use the linear list only. Without setting this flag some kernels might limit

[Xen-devel] [PATCH V11 2/5] libxl_utils: add internal function to read sysfs file contents

2015-12-14 Thread Chunyan Liu
Add a new function libxl_read_sysfs_file_contents to handle sysfs file specially. It would be used in later pvusb work. Signed-off-by: Chunyan Liu --- tools/libxl/libxl_internal.h | 4 +++ tools/libxl/libxl_utils.c| 77 2 files changed, 81 insert

[Xen-devel] [PATCH V11 5/5] domcreate: support pvusb in configuration file

2015-12-14 Thread Chunyan Liu
Add code to support pvusb in domain config file. One could specify usbctrl and usb in domain's configuration file and create domain, then usb controllers will be created and usb device would be attached to guest automatically. One could specify usb controllers and usb devices in config file like t

[Xen-devel] [PATCH V11 1/5] libxl: export some functions for pvusb use

2015-12-14 Thread Chunyan Liu
Signed-off-by: Chunyan Liu Signed-off-by: Simon Cao Reviewed-by: Wei Liu --- tools/libxl/libxl.c | 5 ++--- tools/libxl/libxl_internal.h | 3 +++ 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/tools/libxl/libxl.c b/tools/libxl/libxl.c index 712ea5a..e10242d 100644 --- a

[Xen-devel] [PATCH V11 4/5] xl: add pvusb commands

2015-12-14 Thread Chunyan Liu
Add pvusb commands: usbctrl-attach, usbctrl-detach, usb-list, usbdev-attach and usbdev-detach. To attach a usb device to guest through pvusb, one could follow following example: #xl usbctrl-attach test_vm version=1 ports=8 #xl usb-list test_vm will show the usb controllers and port usage unde

[Xen-devel] [PATCH V11 3/5] libxl: add pvusb API

2015-12-14 Thread Chunyan Liu
Add pvusb APIs, including: - attach/detach (create/destroy) virtual usb controller. - attach/detach usb device - list usb controller and usb devices - some other helper functions Signed-off-by: Chunyan Liu Signed-off-by: Simon Cao Signed-off-by: George Dunlap --- changes: * format fix: ext

[Xen-devel] [PATCH V11 0/5] xen pvusb toolstack work

2015-12-14 Thread Chunyan Liu
This patch series is to add pvusb toolstack work, supporting hot add|remove USB device to|from guest and specify USB device in domain configuration file. Changes to V10: * some changes in libxl pvusb API (patch 3/7) V10: http://lists.xen.org/archives/html/xen-devel/2015-12/msg01172.html V9: http

[Xen-devel] [PATCH 00/11] Support System RAM resource type and EINJ to NVDIMM

2015-12-14 Thread Toshi Kani
This patch-set introduces a new I/O resource type, IORESOURCE_SYSTEM_RAM, for System RAM while keeping the current IORESOURCE_MEM type bit set for all memory-mapped ranges (including System RAM) for backward compatibility. With the new System RAM type, walking through the iomem resource table no lo

[Xen-devel] [PATCH 05/11] xen: Set IORESOURCE_SYSTEM_RAM to System RAM

2015-12-14 Thread Toshi Kani
Set IORESOURCE_SYSTEM_RAM to the flags of memory hotplug resource ranges with "System RAM". Cc: Konrad Rzeszutek Wilk Cc: xen-de...@lists.xenproject.org Signed-off-by: Toshi Kani --- drivers/xen/balloon.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/xen/balloon

[Xen-devel] [libvirt bisection] complete build-i386-libvirt

2015-12-14 Thread osstest service owner
branch xen-unstable xenbranch xen-unstable job build-i386-libvirt testid libvirt-build Tree: libvirt git://libvirt.org/libvirt.git Tree: libvirt_gnulib git://git.sv.gnu.org/gnulib.git Tree: qemu git://xenbits.xen.org/qemu-xen-traditional.git Tree: qemuu git://xenbits.xen.org/qemu-xen.git Tree: xen

[Xen-devel] [linux-3.14 test] 66306: regressions - FAIL

2015-12-14 Thread osstest service owner
flight 66306 linux-3.14 real [real] http://logs.test-lab.xenproject.org/osstest/logs/66306/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-i3865 xen-build fail REGR. vs. 64562 Regressions which are

Re: [Xen-devel] [PATCH v10 3/5] libxl: add pvusb API

2015-12-14 Thread Chun Yan Liu
>>> On 12/14/2015 at 07:37 PM, in message , George Dunlap wrote: > On Mon, Dec 14, 2015 at 7:25 AM, Chun Yan Liu wrote: > > > > > On 12/10/2015 at 08:08 PM, in message <56696b4b.7060...@citrix.com>, > George > > Dunlap wrote: > >> On 10/12/15 12:05, George Dunlap wrote: > >>

[Xen-devel] [V9 3/3] Differentiate IO/mem resources tracked by ioreq server

2015-12-14 Thread Shuai Ruan
From: Yu Zhang Currently in ioreq server, guest write-protected ram pages are tracked in the same rangeset with device mmio resources. Yet unlike device mmio, which can be in big chunks, the guest write- protected pages may be discrete ranges with 4K bytes each. This patch uses a seperate rangese

Re: [Xen-devel] [PATCH v4 --for 4.6 COLOPre 11/25] tools/libxc: support to resume uncooperative HVM guests

2015-12-14 Thread Wen Congyang
On 07/17/2015 12:27 AM, Ian Jackson wrote: > Yang Hongyang writes ("Re: [Xen-devel] [PATCH v4 --for 4.6 COLOPre 11/25] > tools/libxc: support to resume uncooperative HVM guests"): >> On 07/16/2015 11:40 PM, Ian Jackson wrote: >>>what this patch is doing >>> >>> That is, what the change in

[Xen-devel] [V9 1/3] Remove identical relationship between ioreq type and rangeset type.

2015-12-14 Thread Shuai Ruan
From: Yu Zhang This patch uses HVMOP_IO_RANGE_XXX values rather than the raw ioreq type to select the ioreq server, therefore the identical relationship between ioreq type and rangeset type is no longer necessary. Signed-off-by: Yu Zhang Reviewed-by: Paul Durrant Signed-off-by: Shuai Ruan Ack

[Xen-devel] [V9 2/3] Refactor rangeset structure for better performance.

2015-12-14 Thread Shuai Ruan
From: Yu Zhang This patch refactors struct rangeset to base it on the red-black tree structure, instead of on the current doubly linked list. By now, ioreq leverages rangeset to keep track of the IO/memory resources to be emulated. Yet when number of ranges inside one ioreq server is very high, t

[Xen-devel] [V9 0/3] Refactor ioreq server for better performance.

2015-12-14 Thread Shuai Ruan
From: Yu Zhang XenGT leverages ioreq server to track and forward the accesses to GPU I/O resources, e.g. the PPGTT(per-process graphic translation tables). Currently, ioreq server uses rangeset to track the BDF/ PIO/MMIO ranges to be emulated. To select an ioreq server, the rangeset is searched

[Xen-devel] [distros-debian-sid test] 38509: tolerable FAIL

2015-12-14 Thread Platform Team regression test user
flight 38509 distros-debian-sid real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/38509/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-i386-amd64-sid-netboot-pygrub 9 debian-di-install fail blocked in 38457 test-amd64-i38

[Xen-devel] [linux-mingo-tip-master test] 66316: regressions - FAIL

2015-12-14 Thread osstest service owner
flight 66316 linux-mingo-tip-master real [real] http://logs.test-lab.xenproject.org/osstest/logs/66316/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-i386-pvops 5 kernel-build fail REGR. vs. 60684 build-amd6

Re: [Xen-devel] Taking on a Xen development project

2015-12-14 Thread Joshua Otto
On Mon, Dec 14, 2015 at 11:08:43AM +, Ian Campbell wrote: > On Sat, 2015-12-12 at 17:07 -0500, Joshua Otto wrote: > > On Fri, Dec 11, 2015 at 01:52:41PM +, Ian Campbell wrote: > > > Cool! Just to be clear, you are looking for one project for the 3 of > > > you to > > > work on as a group (v

Re: [Xen-devel] Taking on a Xen development project

2015-12-14 Thread Joshua Otto
On Sat, Dec 12, 2015 at 11:02:50PM +, Andrew Cooper wrote: > On 12/12/2015 22:30, Joshua Otto wrote: > > On Sat, Dec 12, 2015 at 10:19:31AM +0800, Yang Hongyang wrote: > >> On 2015年12月11日 01:23, Andrew Cooper wrote: > >>> Hello - thankyou for your interest. > >>> > >>> One area to look at might

[Xen-devel] [3.19.y-ckt stable] Patch "x86/cpu: Fix SMAP check in PVOPS environments" has been added to staging queue

2015-12-14 Thread Kamal Mostafa
This is a note to let you know that I have just added a patch titled x86/cpu: Fix SMAP check in PVOPS environments to the linux-3.19.y-queue branch of the 3.19.y-ckt extended stable tree which can be found at: http://kernel.ubuntu.com/git/ubuntu/linux.git/log/?h=linux-3.19.y-queue This

Re: [Xen-devel] [PATCH] x86/HVM: Merge HVM and PVH hypercall tables

2015-12-14 Thread Boris Ostrovsky
On 12/14/2015 02:51 AM, Jan Beulich wrote: On 11.12.15 at 17:50, wrote: On 12/10/2015 11:53 AM, Boris Ostrovsky wrote: Especially given that PVH dom0 is not booting for me, as I just found out: ... (XEN) d0v0 EPT violation 0x1aa (-w-/r-x) gpa 0x00c0008116 mfn 0xc0008 type 5 (XEN) d0v0 Wal

Re: [Xen-devel] linux 4.4 Regression: 100% cpu usage on idle pv guest under Xen with single vcpu

2015-12-14 Thread Sander Eikelenboom
On 2015-12-14 20:48, Eric Shelton wrote: Please note that the same issue appears to have been introduced in the recent 4.2.7 kernel. It perhaps has to do with b4ff8389ed14b849354b59ce9b360bdefcdbf99c having a matching commit e8d097151d309eb71f750bbf34e6a7ef6256da7e in linux-stable.git. The be

Re: [Xen-devel] [PATCH] xen-pciback: fix up cleanup path when alloc fails

2015-12-14 Thread Konrad Rzeszutek Wilk
On Mon, Dec 14, 2015 at 04:08:13PM +, David Vrabel wrote: > On 02/12/15 14:56, Doug Goldstein wrote: > > On 12/2/15 4:35 AM, David Vrabel wrote: > >> On 26/11/15 20:32, Doug Goldstein wrote: > >>> When allocating a pciback device fails, avoid the possibility of a > >>> use after free. > >> > >>

Re: [Xen-devel] linux 4.4 Regression: 100% cpu usage on idle pv guest under Xen with single vcpu

2015-12-14 Thread Eric Shelton
Please note that the same issue appears to have been introduced in the recent 4.2.7 kernel. It perhaps has to do with b4ff8389ed14b849354b59ce9b360bdefcdbf99c having a matching commit e8d097151d309eb71f750bbf34e6a7ef6256da7e in linux-stable.git. The below patch to arch/x86/kernel/rtc.c was also e

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

2015-12-14 Thread osstest service owner
flight 65797 linux-linus real [real] http://logs.test-lab.xenproject.org/osstest/logs/65797/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-freebsd10-i386 6 xen-bootfail REGR. vs. 59254 test-amd64-i386-xl-qe

[Xen-devel] [PATCH] squash into 'build: build Kconfig and config rules'

2015-12-14 Thread Doug Goldstein
Signed-off-by: Doug Goldstein --- xen/Makefile | 3 +-- xen/tools/kconfig/Makefile.kconfig | 20 +--- 2 files changed, 6 insertions(+), 17 deletions(-) diff --git a/xen/Makefile b/xen/Makefile index 1d2c814..fd16650 100644 --- a/xen/Makefile +++ b/xen/Makef

Re: [Xen-devel] [PATCH] squash into 'build: build Kconfig and config rules'

2015-12-14 Thread Doug Goldstein
On 12/14/15 11:57 AM, Doug Goldstein wrote: > Signed-off-by: Doug Goldstein > --- > xen/Makefile | 3 +-- > xen/tools/kconfig/Makefile.kconfig | 28 ++-- > 2 files changed, 15 insertions(+), 16 deletions(-) > > diff --git a/xen/Makefile b/xen/Makefi

Re: [Xen-devel] [PATCH OSSTEST] Debian: Support runvar to set flask mode

2015-12-14 Thread Ian Jackson
Ian Campbell writes ("[PATCH OSSTEST] Debian: Support runvar to set flask mode"): > Currently it is hardcoding to enforcing but it might be useful to > run with e.g. permissive or even disabled (aka dummy mode) ... > --- a/overlay/etc/grub.d/20_linux_xen > +++ b/overlay/etc/grub.d/20_linux_xen >

Re: [Xen-devel] [PATCH RFC 0/3] Xen on Virtio

2015-12-14 Thread Andy Lutomirski
On Mon, Dec 14, 2015 at 6:12 AM, Michael S. Tsirkin wrote: > On Mon, Dec 14, 2015 at 02:00:05PM +, David Vrabel wrote: >> On 07/12/15 16:19, Stefano Stabellini wrote: >> > Hi all, >> > >> > this patch series introduces support for running Linux on top of Xen >> > inside a virtual machine with

Re: [Xen-devel] [PATCH] xen/tools/get-fields.sh: Use printf for POSIX compat

2015-12-14 Thread Alex Xu
On Mon, 14 Dec 2015 13:22:28 -0500 Alex Xu wrote: > xen/tools/get-fields.sh used echo -n which is not POSIX compatible and > breaks building with dash (shell). Change it to use printf "%s" which > is usable everywhere. > > Signed-off-by: Alex Xu > --- > xen/tools/get-fields.sh | 30 +++

[Xen-devel] [PATCH] xen/tools/get-fields.sh: Use printf for POSIX compat

2015-12-14 Thread Alex Xu
xen/tools/get-fields.sh used echo -n which is not POSIX compatible and breaks building with dash (shell). Change it to use printf "%s" which is usable everywhere. Signed-off-by: Alex Xu --- xen/tools/get-fields.sh | 30 +++--- 1 file changed, 15 insertions(+), 15 deletion

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

2015-12-14 Thread osstest service owner
flight 65799 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/65799/ Failures :-/ but no regressions. Tests which are failing intermittently (not blocking): test-armhf-armhf-xl-arndale 3 host-install(3) broken in 65766 pass in 65799 test-armhf-armhf-xl-rtds 11 g

[Xen-devel] [PATCH] squash into 'build: use generated Kconfig option for Xen'

2015-12-14 Thread Doug Goldstein
Signed-off-by: Doug Goldstein --- xen/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/xen/Makefile b/xen/Makefile index 771f10e..487eeaa 100644 --- a/xen/Makefile +++ b/xen/Makefile @@ -237,7 +237,7 @@ kconfig := silentoldconfig oldconfig config menuconfig defconfig

[Xen-devel] [PATCH] squash into 'build: build Kconfig and config rules'

2015-12-14 Thread Doug Goldstein
Signed-off-by: Doug Goldstein --- xen/Makefile | 3 +-- xen/tools/kconfig/Makefile.kconfig | 28 ++-- 2 files changed, 15 insertions(+), 16 deletions(-) diff --git a/xen/Makefile b/xen/Makefile index 1d2c814..fd16650 100644 --- a/xen/Makefile +++ b/

Re: [Xen-devel] [PATCH] xen: building with perfc=y was broken

2015-12-14 Thread Paul Durrant
> -Original Message- > From: Dario Faggioli [mailto:raistlin...@gmail.com] On Behalf Of Dario > Faggioli > Sent: 14 December 2015 17:45 > To: xen-de...@lists.xenproject.org > Cc: Andrew Cooper; Paul Durrant; Jan Beulich > Subject: [PATCH] xen: building with perfc=y was broken > > because

Re: [Xen-devel] [PATCH v7 03/28] build: build Kconfig and config rules

2015-12-14 Thread Doug Goldstein
On 12/14/15 10:35 AM, Jan Beulich wrote: On 10.12.15 at 17:48, wrote: >> --- /dev/null >> +++ b/xen/Kconfig >> @@ -0,0 +1,24 @@ >> +# >> +# For a description of the syntax of this configuration file, >> +# see docs/misc/kconfig-language.txt >> +# >> +mainmenu "Xen/$SRCARCH $XEN_FULLVERSION Co

Re: [Xen-devel] [PATCH] xen: building with perfc=y was broken

2015-12-14 Thread Andrew Cooper
On 14/12/15 17:45, Dario Faggioli wrote: > because of b38d426ad09 ("x86/viridian: flush remote tlbs > by hypercall") which was defining mshv_call_flush, but using > mshv_flush. > > Signed-off-by: Dario Faggioli > Cc: Jan Beulich > Cc: Andrew Cooper > Cc: Paul Durrant Reviewed-by: Andrew Cooper

[Xen-devel] [PATCH] xen: building with perfc=y was broken

2015-12-14 Thread Dario Faggioli
because of b38d426ad09 ("x86/viridian: flush remote tlbs by hypercall") which was defining mshv_call_flush, but using mshv_flush. Signed-off-by: Dario Faggioli Cc: Jan Beulich Cc: Andrew Cooper Cc: Paul Durrant --- xen/arch/x86/hvm/viridian.c |2 +- 1 file changed, 1 insertion(+), 1 delet

[Xen-devel] [PATCH OSSTEST] Debian: Support runvar to set flask mode

2015-12-14 Thread Ian Campbell
Currently it is hardcoding to enforcing but it might be useful to run with e.g. permissive or even disabled (aka dummy mode) Signed-off-by: Ian Campbell --- Osstest/Debian.pm | 10 -- overlay/etc/grub.d/20_linux_xen | 4 +++- 2 files changed, 11 insertions(+), 3 deletions(

Re: [Xen-devel] [PATCH] x86/time: Don't use EFI's GetTime call by default

2015-12-14 Thread Jan Beulich
>>> On 14.12.15 at 17:53, wrote: > On Tue, Dec 08, 2015 at 11:02:16AM +, Ian Campbell wrote: >> In reality only Microsoft (and perhaps to a lesser extent Apple) have any >> sort of ability to force things in this way (even so it is alleged in this >> thread that even Windows avoids the GetTime

Re: [Xen-devel] [PATCH] x86/time: Don't use EFI's GetTime call by default

2015-12-14 Thread Konrad Rzeszutek Wilk
On Tue, Dec 08, 2015 at 11:02:16AM +, Ian Campbell wrote: > On Wed, 2015-12-02 at 02:33 -0700, Jan Beulich wrote: > > > > > On 01.12.15 at 20:26, wrote: > > > On 01/12/15 17:26, Jan Beulich wrote: > > > > > > > On 01.12.15 at 17:57, wrote: > > > > > When EFI is used, don't use EFI's GetTime()

[Xen-devel] blkback name greater than 16 characters issue

2015-12-14 Thread Konrad Rzeszutek Wilk
Hey, Way back in the past Jan pointed out this issue http://lists.xenproject.org/archives/html/xen-devel/2015-05/msg03505.html which is that the device name is "blkback.." for exmaple: blkback.8.xvda, blkback.11.hda With the multiqueue block backend we add "-%d" for the queue. But sadly this i

Re: [Xen-devel] [PATCH] xen: arm: Drop trailing ; from DEFINE_XEN_GUEST_HANDLE

2015-12-14 Thread Jan Beulich
>>> On 14.12.15 at 17:21, wrote: > This is always present at the point of use, which with -pedantic > provokes: > > error: ISO C does not allow extra ';' outside of a function [-Werror=edantic] > > Signed-off-by: Ian Campbell FWIW Acked-by: Jan Beulich __

Re: [Xen-devel] [PATCH] hvmloader: Load proper ACPI tables with OVMF

2015-12-14 Thread Jan Beulich
>>> On 14.12.15 at 17:23, wrote: > On 14/12/15 16:17, Ian Campbell wrote: >> On Mon, 2015-12-14 at 16:08 +, Anthony PERARD wrote: >>> This patch loads the ACPI tables associated with QEMU instead of the one >>> for qemu-traditional. >> I'd add "... we only support OVMF with qemu-xen" or someth

Re: [Xen-devel] [PATCH v7 04/28] build: use generated Kconfig options for Xen

2015-12-14 Thread Jan Beulich
>>> On 10.12.15 at 17:48, wrote: > --- a/xen/Makefile > +++ b/xen/Makefile > @@ -34,6 +34,8 @@ default: build > .PHONY: dist > dist: install > > +build:: include/config/auto.conf > + > .PHONY: build install uninstall clean distclean cscope TAGS tags MAP gtags > build install uninstall debug

Re: [Xen-devel] [PATCH v7 03/28] build: build Kconfig and config rules

2015-12-14 Thread Jan Beulich
>>> On 10.12.15 at 17:48, wrote: > --- /dev/null > +++ b/xen/Kconfig > @@ -0,0 +1,24 @@ > +# > +# For a description of the syntax of this configuration file, > +# see docs/misc/kconfig-language.txt > +# > +mainmenu "Xen/$SRCARCH $XEN_FULLVERSION Configuration" > + > +config SRCARCH > + string

Re: [Xen-devel] [PATCH RFC v2 4/4] xen/MSI: re-expose masking capability

2015-12-14 Thread Stefano Stabellini
On Mon, 14 Dec 2015, Jan Beulich wrote: > >>> On 11.12.15 at 17:56, wrote: > > For the original issue here, could the flag be exposed as a > > XEN_SYSCTL_PHYSCAP_ > > Yes, I think it could, albeit calling this a "capability" or "feature" > seems odd (since really the original behavior was bog

Re: [Xen-devel] [PATCH] xen: arm: Drop trailing ; from DEFINE_XEN_GUEST_HANDLE

2015-12-14 Thread Stefano Stabellini
On Mon, 14 Dec 2015, Ian Campbell wrote: > This is always present at the point of use, which with -pedantic > provokes: > > error: ISO C does not allow extra ';' outside of a function [-Werror=edantic] > > Signed-off-by: Ian Campbell Acked-by: Stefano Stabellini > xen/include/public/arch-ar

Re: [Xen-devel] [PATCH] hvmloader: Load proper ACPI tables with OVMF

2015-12-14 Thread Andrew Cooper
On 14/12/15 16:17, Ian Campbell wrote: > On Mon, 2015-12-14 at 16:08 +, Anthony PERARD wrote: >> This patch loads the ACPI tables associated with QEMU instead of the one >> for qemu-traditional. > I'd add "... we only support OVMF with qemu-xen" or something, just to make > it clear why this is

[Xen-devel] Pre-domain released event?

2015-12-14 Thread Razvan Cojocaru
Hello, Currently, all the in-tree examples that figure out that a domain has been shut down subscribe to xenstore's @releaseDomain combined with an xs_is_domain_introduced() check. That works fine if you're only interested that the domain no longer exists. However, I'd like to be able to do some c

[Xen-devel] [PATCH] xen: arm: Drop trailing ; from DEFINE_XEN_GUEST_HANDLE

2015-12-14 Thread Ian Campbell
This is always present at the point of use, which with -pedantic provokes: error: ISO C does not allow extra ';' outside of a function [-Werror=edantic] Signed-off-by: Ian Campbell --- xen/include/public/arch-arm.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/xen/include/

Re: [Xen-devel] [PATCH] hvmloader: Load proper ACPI tables with OVMF

2015-12-14 Thread Ian Campbell
On Mon, 2015-12-14 at 16:08 +, Anthony PERARD wrote: > This patch loads the ACPI tables associated with QEMU instead of the one > for qemu-traditional. I'd add "... we only support OVMF with qemu-xen" or something, just to make it clear why this is correct. > Signed-off-by: Anthony PERARD A

Re: [Xen-devel] [PATCH] xen/tools/get-fields.sh: Use printf for POSIX compat

2015-12-14 Thread Jan Beulich
>>> On 14.12.15 at 16:56, wrote: > On Mon, 14 Dec 2015 02:12:16 -0700 > "Jan Beulich" wrote: > >> >>> On 12.12.15 at 19:18, wrote: >> > xen/tools/get-fields.sh used echo -n which is not POSIX compatible >> > and breaks with dash. Change it to use printf "%s" which is usable >> > everywhere.

[Xen-devel] [PATCH] hvmloader: Load proper ACPI tables with OVMF

2015-12-14 Thread Anthony PERARD
This patch loads the ACPI tables associated with QEMU instead of the one for qemu-traditional. Signed-off-by: Anthony PERARD --- tools/firmware/hvmloader/ovmf.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/tools/firmware/hvmloader/ovmf.c b/tools/firmware/hvmloader/

[Xen-devel] [PATCH] squash into 'build: convert HAS_EHCI use to Kconfig'

2015-12-14 Thread Doug Goldstein
Signed-off-by: Doug Goldstein --- xen/drivers/char/Kconfig | 1 - 1 file changed, 1 deletion(-) diff --git a/xen/drivers/char/Kconfig b/xen/drivers/char/Kconfig index 36a742b..08973cf 100644 --- a/xen/drivers/char/Kconfig +++ b/xen/drivers/char/Kconfig @@ -53,7 +53,6 @@ config HAS_SCIF # USB EH

[Xen-devel] [PATCH] squash into 'build: convert HAS_VGA use to Kconfig'

2015-12-14 Thread Doug Goldstein
Signed-off-by: Doug Goldstein --- xen/arch/x86/Kconfig | 1 - xen/drivers/video/Kconfig | 3 ++- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/xen/arch/x86/Kconfig b/xen/arch/x86/Kconfig index b03d228..a42c149 100644 --- a/xen/arch/x86/Kconfig +++ b/xen/arch/x86/Kconfig @@ -

Re: [Xen-devel] [PATCH] xen-pciback: fix up cleanup path when alloc fails

2015-12-14 Thread David Vrabel
On 02/12/15 14:56, Doug Goldstein wrote: > On 12/2/15 4:35 AM, David Vrabel wrote: >> On 26/11/15 20:32, Doug Goldstein wrote: >>> When allocating a pciback device fails, avoid the possibility of a >>> use after free. >> >> We should not require clearing drvdata for correctness. We should >> ensur

Re: [Xen-devel] [PATCH] xen/x86/pvh: Use HVM's flush_tlb_others op

2015-12-14 Thread Boris Ostrovsky
On 12/14/2015 10:35 AM, Roger Pau Monné wrote: El 14/12/15 a les 16.27, Konrad Rzeszutek Wilk ha escrit: On Sat, Dec 12, 2015 at 07:25:55PM -0500, Boris Ostrovsky wrote: Using MMUEXT_TLB_FLUSH_MULTI doesn't buy us much since the hypervisor will likely perform same IPIs as would have the guest.

Re: [Xen-devel] [PATCH RFC v2 4/4] xen/MSI: re-expose masking capability

2015-12-14 Thread Jan Beulich
>>> On 11.12.15 at 17:56, wrote: > For the original issue here, could the flag be exposed as a > XEN_SYSCTL_PHYSCAP_ Yes, I think it could, albeit calling this a "capability" or "feature" seems odd (since really the original behavior was bogus/buggy). But - with sysctl not being a stable inte

Re: [Xen-devel] [PATCH] xen/tools/get-fields.sh: Use printf for POSIX compat

2015-12-14 Thread Alex Xu
On Mon, 14 Dec 2015 02:12:16 -0700 "Jan Beulich" wrote: > >>> On 12.12.15 at 19:18, wrote: > > xen/tools/get-fields.sh used echo -n which is not POSIX compatible > > and breaks with dash. Change it to use printf "%s" which is usable > > everywhere. > > Looks okay, but a couple of remarks: >

[Xen-devel] [PATCH] squash into 'build: convert HAS_PCI use to Kconfig'

2015-12-14 Thread Doug Goldstein
Signed-off-by: Doug Goldstein --- xen/include/xen/iommu.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/xen/include/xen/iommu.h b/xen/include/xen/iommu.h index 47f3180..8217cb7 100644 --- a/xen/include/xen/iommu.h +++ b/xen/include/xen/iommu.h @@ -107,7 +107,7 @@ int iom

Re: [Xen-devel] [PATCHv4 1/2] x86/ept: invalidate guest physical mappings on VMENTER

2015-12-14 Thread George Dunlap
On 14/12/15 15:09, Andrew Cooper wrote: > On 14/12/15 15:00, David Vrabel wrote: >> On 14/12/15 14:52, Andrew Cooper wrote: >>> On 14/12/15 14:39, David Vrabel wrote: diff --git a/xen/arch/x86/mm/p2m-ept.c b/xen/arch/x86/mm/p2m-ept.c index eef0372..6e0cf89 100644 --- a/xen/arch/x86/m

Re: [Xen-devel] [PATCH] xen/x86/pvh: Use HVM's flush_tlb_others op

2015-12-14 Thread Roger Pau Monné
El 14/12/15 a les 16.27, Konrad Rzeszutek Wilk ha escrit: > On Sat, Dec 12, 2015 at 07:25:55PM -0500, Boris Ostrovsky wrote: >> Using MMUEXT_TLB_FLUSH_MULTI doesn't buy us much since the hypervisor >> will likely perform same IPIs as would have the guest. >> > > But if the VCPU is asleep, doing it

Re: [Xen-devel] [PATCHv4] xen/gntdev: add ioctl for grant copy

2015-12-14 Thread Boris Ostrovsky
On 12/14/2015 09:21 AM, David Vrabel wrote: On 01/12/15 16:43, David Vrabel wrote: Add IOCTL_GNTDEV_GRANT_COPY to allow applications to copy between user space buffers and grant references. This interface is similar to the GNTTABOP_copy hypercall ABI except the local buffers are provided using

Re: [Xen-devel] [PATCH] xen/x86/pvh: Use HVM's flush_tlb_others op

2015-12-14 Thread Konrad Rzeszutek Wilk
On Sat, Dec 12, 2015 at 07:25:55PM -0500, Boris Ostrovsky wrote: > Using MMUEXT_TLB_FLUSH_MULTI doesn't buy us much since the hypervisor > will likely perform same IPIs as would have the guest. > But if the VCPU is asleep, doing it via the hypervisor will save us waking up the guest VCPU, sending

Re: [Xen-devel] [PATCHv4 1/2] x86/ept: invalidate guest physical mappings on VMENTER

2015-12-14 Thread Andrew Cooper
On 14/12/15 15:00, David Vrabel wrote: > On 14/12/15 14:52, Andrew Cooper wrote: >> On 14/12/15 14:39, David Vrabel wrote: >>> diff --git a/xen/arch/x86/mm/p2m-ept.c b/xen/arch/x86/mm/p2m-ept.c >>> index eef0372..6e0cf89 100644 >>> --- a/xen/arch/x86/mm/p2m-ept.c >>> +++ b/xen/arch/x86/mm/p2m-ept.c

Re: [Xen-devel] [PATCHv4 1/2] x86/ept: invalidate guest physical mappings on VMENTER

2015-12-14 Thread David Vrabel
On 14/12/15 14:52, Andrew Cooper wrote: > On 14/12/15 14:39, David Vrabel wrote: >> diff --git a/xen/arch/x86/mm/p2m-ept.c b/xen/arch/x86/mm/p2m-ept.c >> index eef0372..6e0cf89 100644 >> --- a/xen/arch/x86/mm/p2m-ept.c >> +++ b/xen/arch/x86/mm/p2m-ept.c [...] >> +on_selected_cpus(d->domain_dirt

Re: [Xen-devel] [PATCHv4 1/2] x86/ept: invalidate guest physical mappings on VMENTER

2015-12-14 Thread Andrew Cooper
On 14/12/15 14:39, David Vrabel wrote: > diff --git a/xen/arch/x86/mm/p2m-ept.c b/xen/arch/x86/mm/p2m-ept.c > index eef0372..6e0cf89 100644 > --- a/xen/arch/x86/mm/p2m-ept.c > +++ b/xen/arch/x86/mm/p2m-ept.c > @@ -1089,9 +1089,10 @@ static void ept_memory_type_changed(struct p2m_domain > *p2m) >

Re: [Xen-devel] [PATCHv3 1/2] x86/ept: invalidate guest physical mappings on VMENTER

2015-12-14 Thread David Vrabel
On 07/12/15 10:25, George Dunlap wrote: > > I took the past tense ("synced") to mean, "These CPUs have been > brought into sync (or are no longer out of sync)". So they start out > not-synced, so you initialize the bit to be clear; when an INVEPT is > executed, they become synced, so you set the

[Xen-devel] [PATCHv4 2/2] x86/ept: defer the invalidation until the p2m lock is released

2015-12-14 Thread David Vrabel
Holding the p2m lock while calling ept_sync_domain() is very expensive since it does a on_selected_cpus() call. IPIs on many socket machines can be very slows and on_selected_cpus() is serialized. Defer the invalidate until the p2m lock is released. Since the processor may cache partial translat

[Xen-devel] [PATCHv4 1/2] x86/ept: invalidate guest physical mappings on VMENTER

2015-12-14 Thread David Vrabel
If a guest allocates a page and the tlbflush_timestamp on the page indicates that a TLB flush of the previous owner is required, only the linear and combined mappings are invalidated. The guest-physical mappings are not invalidated. This is currently safe because the EPT code ensures that the gue

[Xen-devel] [PATCHv4 0/2] x86/ept: reduce translation invalidation impact

2015-12-14 Thread David Vrabel
This series improves the performance of EPT by further reducing the impact of the translation invalidations (ept_sync_domain()). By: a) Deferring invalidations until the p2m write lock is released. Prior to this change a 16 VCPU guest could not be successfully migrated on an (admittedly slow) 160

Re: [Xen-devel] [PATCHv4] xen/gntdev: add ioctl for grant copy

2015-12-14 Thread David Vrabel
On 01/12/15 16:43, David Vrabel wrote: > Add IOCTL_GNTDEV_GRANT_COPY to allow applications to copy between user > space buffers and grant references. > > This interface is similar to the GNTTABOP_copy hypercall ABI except > the local buffers are provided using a virtual address (instead of a > GFN

Re: [Xen-devel] [PATCH] squash into 'build: convert HAS_MEM_ACCESS to use Kconfig'

2015-12-14 Thread Doug Goldstein
On 12/14/15 3:01 AM, Jan Beulich wrote: On 11.12.15 at 17:15, wrote: >> I've submitted 'tools: always enable HAS_MEM_ACCESS' and once that lands >> this can be squashed into 'build: convert HAS_MEM_ACCESS to use Kconfig'. > > Well, for some particular definition of "squash" only. I can't see

Re: [Xen-devel] [PATCH RFC 1/3] xen: export xen_phys_to_bus, xen_bus_to_phys and xen_virt_to_bus

2015-12-14 Thread David Vrabel
On 07/12/15 16:19, Stefano Stabellini wrote: > Signed-off-by: Stefano Stabellini Can you add a brief description about why these are being moved? Then, assuming this is needed in the end: Acked-by: David Vrabel David ___ Xen-devel mailing list Xen-

Re: [Xen-devel] [PATCH RFC 0/3] Xen on Virtio

2015-12-14 Thread Michael S. Tsirkin
On Mon, Dec 14, 2015 at 02:00:05PM +, David Vrabel wrote: > On 07/12/15 16:19, Stefano Stabellini wrote: > > Hi all, > > > > this patch series introduces support for running Linux on top of Xen > > inside a virtual machine with virtio devices (nested virt scenario). > > The problem is that Lin

Re: [Xen-devel] [PATCH XEN v6 10/32] tools: Refactor /dev/xen/gnt{dev, shr} wrappers into libxengnttab.

2015-12-14 Thread Ian Campbell
On Thu, 2015-12-03 at 11:22 +, Ian Campbell wrote: > [...] > +void *xengnttab_map_grant_ref(xengnttab_handle *xgt, > +  uint32_t domid, > +  uint32_t ref, > +  int prot); > [...] > +int xengnttab_munmap(xengntta

Re: [Xen-devel] [PATCH] xen/x86/pvh: Use HVM's flush_tlb_others op

2015-12-14 Thread Boris Ostrovsky
On 12/14/2015 08:58 AM, David Vrabel wrote: On 13/12/15 00:25, Boris Ostrovsky wrote: Using MMUEXT_TLB_FLUSH_MULTI doesn't buy us much since the hypervisor will likely perform same IPIs as would have the guest. More importantly, using MMUEXT_INVLPG_MULTI may not to invalidate the guest's addres

Re: [Xen-devel] [PATCH RFC 0/3] Xen on Virtio

2015-12-14 Thread David Vrabel
On 07/12/15 16:19, Stefano Stabellini wrote: > Hi all, > > this patch series introduces support for running Linux on top of Xen > inside a virtual machine with virtio devices (nested virt scenario). > The problem is that Linux virtio drivers use virt_to_phys to get the > guest pseudo-physical addr

Re: [Xen-devel] [PATCH] xen/x86/pvh: Use HVM's flush_tlb_others op

2015-12-14 Thread David Vrabel
On 13/12/15 00:25, Boris Ostrovsky wrote: > Using MMUEXT_TLB_FLUSH_MULTI doesn't buy us much since the hypervisor > will likely perform same IPIs as would have the guest. > > More importantly, using MMUEXT_INVLPG_MULTI may not to invalidate the > guest's address on remote CPU (when, for example, V

Re: [Xen-devel] [PATCH] scripts: Add a script to build and submit to coverity.

2015-12-14 Thread Ian Campbell
On Thu, 2015-12-10 at 18:06 +, Ian Jackson wrote: > Ian Jackson writes ("Re: [PATCH] scripts: Add a script to build and > submit to coverity."): > > If curl can do that then fine.  Given > > > > > > > > +declare -a curl_args > > > > > > +curl_args+=("--form" "token=$COV_TOKEN") > > > >

Re: [Xen-devel] [PATCH v7 23/28] build: convert HAS_EHCI use to Kconfig

2015-12-14 Thread Jan Beulich
>>> On 10.12.15 at 17:48, wrote: > +# USB EHCI debug port support > +config HAS_EHCI > + bool > + depends on X86 As said before, dependencies on prompt-less options are bogus and potentially confusing (and really wrong here - there's nothing precluding ARM to also use that code as soon as

Re: [Xen-devel] [PATCH v7 00/28] Kconfig conversion

2015-12-14 Thread Jan Beulich
>>> On 10.12.15 at 17:48, wrote: > The following series is a follow on to the Kconfig conversion patch series. > There are still more components to convert however this is the bare minimal > to get everything working and get the options out of the existing makefiles. > > The CONFIG_HAS_ variables

Re: [Xen-devel] [PATCH v7 12/28] build: convert HAS_VGA use to Kconfig

2015-12-14 Thread Jan Beulich
>>> On 10.12.15 at 17:48, wrote: > --- a/xen/drivers/video/Kconfig > +++ b/xen/drivers/video/Kconfig > @@ -2,3 +2,7 @@ > # Select HAS_VIDEO if video is supported > config HAS_VIDEO > bool > + > +# Select HAS_VGA if VGA is supported > +config HAS_VGA > + bool I think HAS_VGA should s

[Xen-devel] [linux-mingo-tip-master test] 65794: regressions - FAIL

2015-12-14 Thread osstest service owner
flight 65794 linux-mingo-tip-master real [real] http://logs.test-lab.xenproject.org/osstest/logs/65794/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-rumpuserxen-i386 10 guest-start fail REGR. vs. 60684 test-amd64

Re: [Xen-devel] [PATCH v13 7/8] Add IOREQ_TYPE_VMWARE_PORT

2015-12-14 Thread Don Slutz
On 12/07/15 08:36, Paul Durrant wrote: >> -Original Message- > [snip] >> >> if ( rc ) >> -hvm_unmap_ioreq_page(s, 0); >> +{ >> +hvm_unmap_ioreq_page(s, IOREQ_PAGE_TYPE_IOREQ); >> +return rc; >> +} >> + >> +rc = hvm

Re: [Xen-devel] [PATCH] flask: Allow device model to raise PCI interrupts (pcilevel capability)

2015-12-14 Thread Ian Jackson
Ian Campbell writes ("[PATCH] flask: Allow device model to raise PCI interrupts (pcilevel capability)"): ... > - allow $1 $2_target:hvm { getparam setparam trackdirtyvram hvmctl > irqlevel pciroute cacheattr send_irq }; > + allow $1 $2_target:hvm { getparam setparam trackdirtyvram hvmctl

Re: [Xen-devel] stubdom migration failure on merlot* XSM related (Was: [adhoc test] 65682: tolerable FAIL])

2015-12-14 Thread Ian Campbell
On Mon, 2015-12-14 at 10:14 +, Ian Campbell wrote: >  > I've running a test with the following patch. I'm reasonably hopeful. and it did indeed pass: http://logs.test-lab.xenproject.org/osstest/logs/66273/ I'll resubmit as a proper patch. Ian. > > Ian. > > From 3f14c5afedc0df360952364b93c

[Xen-devel] [PATCH] flask: Allow device model to raise PCI interrupts (pcilevel capability)

2015-12-14 Thread Ian Campbell
Allows: (XEN) avc: denied { pcilevel } for domid=2 target=1 scontext=system_u:system_r:dm_dom_t tcontext=system_u:system_r:domU_t_target tclass=hvm Which otherwise leads to the following on resume after migrate (comparing non-XSM to XSM): ata2.00: configured for MWDMA2 usb 1-2: reset full-

  1   2   >