[Xen-devel] how to locate the hypercall address in memory?

2015-07-14 Thread big strong
As syscalls can be located with the help of symbol files, is it possible to do it to hypercalls too? ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v4 for Xen 4.6 0/4] Enable per-VCPU parameter settings for RTDS scheduler

2015-07-14 Thread Dario Faggioli
On Mon, 2015-07-13 at 22:45 -0700, Meng Xu wrote: > Hi Dario, > Hi, > - not enough benchmarks/performance figures: I'd like to have > the >latency numbers, e.g., from cyclictest, we've spoke many > times with >Meng, give our official blessing at usi

[Xen-devel] [rumpuserxen test] 59532: regressions - FAIL

2015-07-14 Thread osstest service owner
flight 59532 rumpuserxen real [real] http://logs.test-lab.xenproject.org/osstest/logs/59532/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-amd64-rumpuserxen 5 rumpuserxen-build fail REGR. vs. 33866 build-i386-rumpuserxe

Re: [Xen-devel] [PATCH 1/9] libxl: fix libxl__abs_path

2015-07-14 Thread Ian Campbell
On Mon, 2015-07-13 at 18:12 +0100, Ian Jackson wrote: > Ian Campbell writes ("Re: [PATCH 1/9] libxl: fix libxl__abs_path"): > > I rather dislike subjects of the form "fix $function", since it gives > > very little clue to someone reading the shortlog what is going on. > > Yes. > > > In this case

Re: [Xen-devel] [v7][PATCH 13/16] libxl: construct e820 map with RDM information for HVM guest

2015-07-14 Thread Ian Campbell
On Tue, 2015-07-14 at 13:44 +0800, Chen, Tiejun wrote: > On 2015/7/13 18:15, Ian Campbell wrote: > > On Mon, 2015-07-13 at 17:47 +0800, Chen, Tiejun wrote: > >>> This approach looks like it should work, and I think given the point in > >>> the release it would be acceptable for 4.6. > >>> > >>> How

Re: [Xen-devel] Requesting for freeze exception for ARM/ITS patches

2015-07-14 Thread Ian Campbell
On Mon, 2015-07-13 at 18:24 +0100, Stefano Stabellini wrote: > On Mon, 13 Jul 2015, Wei Liu wrote: > > On Fri, Jul 10, 2015 at 04:16:07PM +0530, Vijay Kilari wrote: > > > Hi Wei, > > > > > > I would like to have freeze exception for ITS feature on ARM64. > > > Design got freeze few weeks back

Re: [Xen-devel] [PATCH V3] libxl: Increase device model startup timeout to 1min.

2015-07-14 Thread Ian Campbell
On Tue, 2015-07-14 at 07:17 +0100, Jan Beulich wrote: > >>> On 07.07.15 at 17:41, wrote: > > On Tue, 2015-07-07 at 16:14 +0100, Ian Jackson wrote: > >> Anthony PERARD writes ("[PATCH V3] libxl: Increase device model startup > > timeout to 1min."): > >> > On a busy host, QEMU may take more than 10

[Xen-devel] [xen-4.4-testing test] 59510: regressions - FAIL

2015-07-14 Thread osstest service owner
flight 59510 xen-4.4-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/59510/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemuu-winxpsp3 15 guest-localmigrate/x10 fail REGR. vs. 59289 Regressions w

Re: [Xen-devel] [v7][PATCH 13/16] libxl: construct e820 map with RDM information for HVM guest

2015-07-14 Thread Chen, Tiejun
However long term I think it might make sense to try and reuse one of the existing libxl__arch hooks, i.e. libxl__arch_domain_init_hw_description or libxl__arch_domain_finalise_hw_description. On ARM these are to do with setting the Device Tree Blob, which included the memory map, so it is somewha

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

2015-07-14 Thread osstest service owner
flight 59511 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/59511/ Perfect :-) All tests in this flight passed version targeted for testing: ovmf 2ad9cf37a492e69a4e1b7624d92d9a35fce083fc baseline version: ovmf 6bc4e42f9d043bcda03f03a74e6dec0aa5c

Re: [Xen-devel] [GIT-PULL OSSTEST] ap-fetch-version: Arrange for osstest merges from upstream to be stable

2015-07-14 Thread Ian Campbell
On Mon, 2015-07-13 at 18:03 +0100, Ian Jackson wrote: > Ian Campbell writes ("[GIT-PULL OSSTEST] ap-fetch-version: Arrange for > osstest merges from upstream to be stable"): > > "ap-fetch-version: Arrange for osstest merges from upstream to be > > stable" has now passed the Cambridge push gate and

Re: [Xen-devel] [PATCH V3 1/1] libxl: set stub domain size based on VRAM size

2015-07-14 Thread George Dunlap
On Sat, Jul 11, 2015 at 10:30 PM, Eric Shelton wrote: > Allocate additional memory to the stub domain for qemu-traditional if > more than 4 MB is assigned to the video adapter to avoid out of memory > condition for QEMU. > > Signed-off-by: Eric Shelton This seems like a good fix for now, thanks.

Re: [Xen-devel] [PATCH v3 05/13] x86/altp2m: basic data structures and support routines.

2015-07-14 Thread Jan Beulich
>>> On 14.07.15 at 02:01, wrote: >>From: Jan Beulich [mailto:jbeul...@suse.com] >>Sent: Monday, July 13, 2015 1:01 AM > On 10.07.15 at 23:48, wrote: From: Jan Beulich [mailto:jbeul...@suse.com] Sent: Thursday, July 09, 2015 6:30 AM >>> On 01.07.15 at 20:09, wrote: > +

Re: [Xen-devel] [PATCH v4 11/15] x86/altp2m: define and implement alternate p2m HVMOP types.

2015-07-14 Thread Jan Beulich
>>> On 14.07.15 at 01:39, wrote: > >>-Original Message- >>From: Jan Beulich [mailto:jbeul...@suse.com] >>Sent: Monday, July 13, 2015 12:26 AM >>To: Sahita, Ravi >>Cc: Andrew Cooper; Wei Liu; George Dunlap; Ian Jackson; White, Edmund H; >>xen-devel@lists.xen.org; tleng...@novetta.com; Dan

Re: [Xen-devel] Requesting for freeze exception for RMRR

2015-07-14 Thread Jan Beulich
>>> On 14.07.15 at 02:26, wrote: >> > 1. clarify the state of patch series / feature. >>> >>> ReviewedAcked RMRR series v7 >>> Y Y [v7][PATCH 01/16] xen: introduce >>> XENMEM_reserved_device_memory_map >>> Y Y [v7][PATCH 02/16] xen/vtd: create RMRR mapping >>

Re: [Xen-devel] [v7][PATCH 15/16] xen/vtd: prevent from assign the device with shared rmrr

2015-07-14 Thread Jan Beulich
>>> On 14.07.15 at 03:42, wrote: >> > +{ >>> +printk(XENLOG_G_ERR VTDPREFIX >>> + " cannot assign %04x:%02x:%02x.%u" >>> + " with shared RMRR for Dom%d.\n", >>> + seg, bus, PCI_SLOT(devfn), PCI_FUNC(devfn

Re: [Xen-devel] Requesting for freeze exception for VT-d posted-interrupts

2015-07-14 Thread Wei Liu
On Tue, Jul 14, 2015 at 05:51:02AM +, Wu, Feng wrote: > > > > -Original Message- > > From: Wei Liu [mailto:wei.l...@citrix.com] > > Sent: Monday, July 13, 2015 7:01 PM > > To: Wu, Feng > > Cc: wei.l...@citrix.com; xen-devel@lists.xen.org; Jan Beulich > > (jbeul...@suse.com); andrew.co

Re: [Xen-devel] Requesting for freeze exception for ARM/ITS patches

2015-07-14 Thread Vijay Kilari
Hi Ian, On Sat, Jul 11, 2015 at 12:06 PM, Julien Grall wrote: > Hi, > > On 10/07/2015 17:52, Ian Campbell wrote: >> >> On Fri, 2015-07-10 at 12:01 +0100, Jan Beulich wrote: >> >> On 10.07.15 at 12:46, wrote: I would like to have freeze exception for ITS feature on ARM64. >

Re: [Xen-devel] Requesting for freeze exception for RMRR

2015-07-14 Thread Ian Campbell
On Tue, 2015-07-14 at 10:18 +0100, Jan Beulich wrote: > >>> Y Y [v7][PATCH 14/16] xen/vtd: enable USB device assignment diffstat: xen/drivers/passthrough/vtd/dmar.h | 1 - xen/drivers/passthrough/vtd/iommu.c | 11 ++- xen/drivers/passthrough/vtd/utils.c | 7 --- 3 fi

Re: [Xen-devel] [PATCH V3] libxl: Increase device model startup timeout to 1min.

2015-07-14 Thread Dario Faggioli
On Tue, 2015-07-14 at 08:55 +0100, Ian Campbell wrote: > On Tue, 2015-07-14 at 07:17 +0100, Jan Beulich wrote: > > >>> On 07.07.15 at 17:41, wrote: > > > On Tue, 2015-07-07 at 16:14 +0100, Ian Jackson wrote: > > >> Anthony PERARD writes ("[PATCH V3] libxl: Increase device model startup > > > time

Re: [Xen-devel] [v7][PATCH 06/16] hvmloader/pci: skip reserved ranges

2015-07-14 Thread Jan Beulich
>>> On 14.07.15 at 08:39, wrote: >> > -} *resource, mem_resource, high_mem_resource, io_resource; >>> +} *resource, mem_resource, high_mem_resource, io_resource, > exp_mem_resource; >> >> Despite having gone through description and the rest of the patch I >> can't seem to be able to guess

Re: [Xen-devel] Requesting for freeze exception for RMRR

2015-07-14 Thread Chen, Tiejun
Y Y [v7][PATCH 14/16] xen/vtd: enable USB device assignment Y Y [v7][PATCH 15/16] xen/vtd: prevent from assign the device with shared rmrr And yet again for these two. Please avoid giving a false impression But these two patches really won Kevin's Ack,

Re: [Xen-devel] [PATCH v2 0/3] arm64: Add multiboot support (via fdt) for Xen boot

2015-07-14 Thread Ian Campbell
On Mon, 2015-07-13 at 16:53 +0800, fu@linaro.org wrote: > From: Fu Wei > > - This adds support for the Xen boot on ARM specification for arm64. I have used this to PXE boot Xen on a mustang board. My (handcrafted) grub.cfg was: set default=0 set timeout=5 menuentry

Re: [Xen-devel] Requesting for freeze exception for RMRR

2015-07-14 Thread Wei Liu
On Tue, Jul 14, 2015 at 09:27:17AM +0800, Chen, Tiejun wrote: > >Please work with maintainers to get those hvmloader patches acked or > >reviewed. > > I will do. > > > > >> > >>Note Jackson and Campbell also raised some comments to improve current > >>codes. > >> > >>2. explain why it needs to be

Re: [Xen-devel] [v7][PATCH 07/16] hvmloader/e820: construct guest e820 table

2015-07-14 Thread Jan Beulich
>>> On 14.07.15 at 07:22, wrote: >>> +for ( i = 0; i < memory_map.nr_map; i++ ) >>> +{ >>> +uint64_t end = e820[i].addr + e820[i].size; >> >> Either loop index/boundary or used array are wrong here: In the >> earlier loop you copied memory_map[0...nr_map-1] to >> e820[n...n+nr_map-

Re: [Xen-devel] [PATCH V3] libxl: Increase device model startup timeout to 1min.

2015-07-14 Thread Ian Campbell
On Tue, 2015-07-14 at 11:25 +0200, Dario Faggioli wrote: > On Tue, 2015-07-14 at 08:55 +0100, Ian Campbell wrote: > > On Tue, 2015-07-14 at 07:17 +0100, Jan Beulich wrote: > > > >>> On 07.07.15 at 17:41, wrote: > > > > On Tue, 2015-07-07 at 16:14 +0100, Ian Jackson wrote: > > > >> Anthony PERARD w

Re: [Xen-devel] Requesting for freeze exception for RMRR

2015-07-14 Thread Jan Beulich
>>> On 14.07.15 at 11:25, wrote: > On Tue, 2015-07-14 at 10:18 +0100, Jan Beulich wrote: > >> >>> YY [v7][PATCH 14/16] xen/vtd: enable USB device >> >>> assignment > > diffstat: > xen/drivers/passthrough/vtd/dmar.h | 1 - > xen/drivers/passthrough/vtd/iommu.c | 11 ++---

Re: [Xen-devel] [PATCH v2 10/10] libxl: fix caller of libxl_cpupool functions

2015-07-14 Thread Dario Faggioli
On Mon, 2015-07-13 at 17:22 +0100, Wei Liu wrote: > Coverity complains cpupool_info leaks a string in failure path. Instead > of fixing that path, we rely on the callers (two public APIs at the > moment) of cpupool_info correctly call libxl_cpupoolinfo_dispose in > their failure path to dispose of

Re: [Xen-devel] Requesting for freeze exception for RMRR

2015-07-14 Thread Jan Beulich
>>> On 14.07.15 at 11:27, wrote: > Y Y [v7][PATCH 14/16] xen/vtd: enable USB device assignment > Y Y [v7][PATCH 15/16] xen/vtd: prevent from assign the > device with > shared rmrr And yet again for these two. Please avoid giving a false impres

Re: [Xen-devel] [PATCH] sysctl: adjust XEN_SYSCTL_cputopoinfo behavior

2015-07-14 Thread Jan Beulich
>>> On 13.07.15 at 19:43, wrote: > On 07/13/2015 11:53 AM, Jan Beulich wrote: >> The new function's implementation, other than the original one of >> XEN_SYSCTL_topologyinfo, didn't allow the caller to get what it needs >> (if e.g. it's after the data for just one specific CPU) with just one >> hy

Re: [Xen-devel] [PATCH v2 2/3] util/grub.d/20_linux_xen.in: Add arm64 support

2015-07-14 Thread Ian Campbell
On Tue, 2015-07-14 at 06:53 +0300, Andrei Borzenkov wrote: > > +if [ "x$machine" != xaarch64 ]; then > > + multiboot_cmd="multiboot" > > + module_cmd="module" > > +else > > + multiboot_cmd="xen_hypervisor" > > + module_cmd="xen_module" > > +fi > > + > > Strictly speaking, this is boot-time

Re: [Xen-devel] Requesting for freeze exception for ARM/ITS patches

2015-07-14 Thread Ian Campbell
On Tue, 2015-07-14 at 14:54 +0530, Vijay Kilari wrote: > I am trying to boot latest staging Xen branch on ThunderX with ITS patches. > > I face below issues with above [1] patch series > > 1) If pcie support only MSI, then INT mapping is not specified in DT. However > the below code returns er

[Xen-devel] [PATCH] sysctl: adjust XEN_SYSCTL_numainfo behavior

2015-07-14 Thread Jan Beulich
... to match XEN_SYSCTL_cputopoinfo, allowing the caller to get what it needs (if e.g. it's after the data for just one specific node) with just one hypercall, without caring about the total number of nodes in the system. Suggested-by: Boris Ostrovsky Signed-off-by: Jan Beulich --- a/xen/common

Re: [Xen-devel] [PATCH v2 1/3] xen/domain_page: Convert map_domain_page_global() to using mfn_t

2015-07-14 Thread Jan Beulich
>>> On 13.07.15 at 18:56, wrote: > Jan Beulich writes ("Re: [Xen-devel] [PATCH v2 1/3] xen/domain_page: Convert > map_domain_page_global() to using mfn_t"): >> On 07.07.15 at 12:07, wrote: >> > Just like free(), these functions are not performing a read-only >> > operation on the destination poi

Re: [Xen-devel] [PATCH v5 15/15] tools/xen-access: altp2m testcases

2015-07-14 Thread Wei Liu
On Mon, Jul 13, 2015 at 05:15:03PM -0700, Ed White wrote: > From: Tamas K Lengyel > > Working altp2m test-case. Extended the test tool to support singlestepping > to better highlight the core feature of altp2m view switching. > > Signed-off-by: Tamas K Lengyel > Signed-off-by: Ed White > > Re

Re: [Xen-devel] Requesting for freeze exception for ARM/ITS patches

2015-07-14 Thread Vijay Kilari
On Tue, Jul 14, 2015 at 3:20 PM, Ian Campbell wrote: > On Tue, 2015-07-14 at 14:54 +0530, Vijay Kilari wrote: >> I am trying to boot latest staging Xen branch on ThunderX with ITS patches. >> >> I face below issues with above [1] patch series >> >> 1) If pcie support only MSI, then INT mapping is

Re: [Xen-devel] Ping: [PATCH v6] dmar: device scope mem leak fix

2015-07-14 Thread Jan Beulich
>>> On 13.07.15 at 20:10, wrote: > On its way Jan! This wasn't a ping to you, but to the VT-d maintainers to finally ack this patch. Jan ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] Requesting for freeze exception for ARM/ITS patches

2015-07-14 Thread Vijay Kilari
On Sat, Jul 11, 2015 at 12:48 PM, Julien Grall wrote: > Hi Ian, > > On 10/07/2015 18:07, Ian Campbell wrote: >> >> On Fri, 2015-07-10 at 16:16 +0530, Vijay Kilari wrote: >>> >>> I would like to have freeze exception for ITS feature on ARM64. >>> Design got freeze few weeks back and I have sen

Re: [Xen-devel] [PATCH V3 1/1] libxl: set stub domain size based on VRAM size

2015-07-14 Thread Eric Shelton
On Jul 14, 2015 4:51 AM, "George Dunlap" wrote: > > On Sat, Jul 11, 2015 at 10:30 PM, Eric Shelton wrote: > > Allocate additional memory to the stub domain for qemu-traditional if > > more than 4 MB is assigned to the video adapter to avoid out of memory > > condition for QEMU. > > > > Signed-off

Re: [Xen-devel] Requesting for freeze exception for VT-d posted-interrupts

2015-07-14 Thread Jan Beulich
>>> On 14.07.15 at 11:21, wrote: > On Tue, Jul 14, 2015 at 05:51:02AM +, Wu, Feng wrote: >> Is it possible to get to 4.6 if making this feature default off? > > Note that I'm not the only one who makes the decision and I can't speak > for maintainers. The first thing you ought to do is to con

Re: [Xen-devel] [PATCH v10 0/5] iommu: add rmrr Xen command line option

2015-07-14 Thread Jan Beulich
>>> On 13.07.15 at 20:17, wrote: > From: Elena Ufimtseva > > Add Xen command line option rmrr to specify RMRR > > regions for devices that are not defined in ACPI thus > > causing IO Page Fault while booting dom0 in PVH mode.

Re: [Xen-devel] [v7][PATCH 07/16] hvmloader/e820: construct guest e820 table

2015-07-14 Thread Chen, Tiejun
On 2015/7/14 17:32, Jan Beulich wrote: On 14.07.15 at 07:22, wrote: +for ( i = 0; i < memory_map.nr_map; i++ ) +{ +uint64_t end = e820[i].addr + e820[i].size; Either loop index/boundary or used array are wrong here: In the earlier loop you copied memory_map[0...nr_map-1] to e82

Re: [Xen-devel] [PATCH 1/9] libxl: fix libxl__abs_path

2015-07-14 Thread Ian Jackson
Ian Campbell writes ("Re: [PATCH 1/9] libxl: fix libxl__abs_path"): > On Mon, 2015-07-13 at 18:12 +0100, Ian Jackson wrote: > > There are no call sites in libxl which can pass a NULL. > > > > I think that if we are to retain this feature, it ought to be > > documented, at least. > > Or add an ass

Re: [Xen-devel] [PATCH V3 1/1] libxl: set stub domain size based on VRAM size

2015-07-14 Thread George Dunlap
On 07/14/2015 11:02 AM, Eric Shelton wrote: > On Jul 14, 2015 4:51 AM, "George Dunlap" > wrote: >> >> On Sat, Jul 11, 2015 at 10:30 PM, Eric Shelton wrote: >>> Allocate additional memory to the stub domain for qemu-traditional if >>> more than 4 MB is assigned to the video adapter to avoid out of

Re: [Xen-devel] Request a freeze exception for Libxl Migration v2 in 4.6

2015-07-14 Thread Ian Jackson
Andrew Cooper writes ("Request a freeze exception for Libxl Migration v2 in 4.6"): > I would like to request a freeze exception for libxl migration v2. > > v3 of the series was posted this morning, and review seems to indicate > that it is mostly on track. I hope to have v4 ready to post tomorro

Re: [Xen-devel] [PATCH v3 26/28] tools/libxl: Handle checkpoint records in a libxl migration v2 stream

2015-07-14 Thread Yang Hongyang
On 07/13/2015 08:01 PM, Andrew Cooper wrote: [...] domcreate_rebuild_done(egc, dcs, rc); @@ -966,6 +989,7 @@ static void domcreate_bootloader_done(libxl__egc *egc, } /* Restore */ +callbacks->checkpoint = libxl__remus_domain_checkpoint_callback; This should be moved a

Re: [Xen-devel] [PATCH v3 26/28] tools/libxl: Handle checkpoint records in a libxl migration v2 stream

2015-07-14 Thread Andrew Cooper
On 14/07/15 11:33, Yang Hongyang wrote: > On 07/13/2015 08:01 PM, Andrew Cooper wrote: > [...] >> domcreate_rebuild_done(egc, dcs, rc); >> @@ -966,6 +989,7 @@ static void domcreate_bootloader_done(libxl__egc >> *egc, >> } >> >> /* Restore */ >> +callbacks->checkpoint = lib

Re: [Xen-devel] [PATCH v10 5/5] iommu: add rmrr Xen command line option for extra rmrrs

2015-07-14 Thread Jan Beulich
>>> On 13.07.15 at 20:18, wrote: > --- a/xen/drivers/passthrough/vtd/dmar.c > +++ b/xen/drivers/passthrough/vtd/dmar.c > @@ -867,6 +867,145 @@ out: > return ret; > } > > +#define MAX_EXTRA_RMRR_PAGES 16 > +#define MAX_EXTRA_RMRR 10 > + > +/* RMRR units derived from command line rmrr option

Re: [Xen-devel] [v7][PATCH 03/16] xen/passthrough: extend hypercall to support rdm reservation policy

2015-07-14 Thread George Dunlap
On 07/13/2015 07:47 AM, Chen, Tiejun wrote: >> Thanks for this; a few more comments... >> > > Thanks for your time. > >>> @@ -1577,9 +1578,15 @@ int iommu_do_pci_domctl( >>> seg = machine_sbdf >> 16; >>> bus = PCI_BUS(machine_sbdf); >>> devfn = PCI_DEVFN2(machine_sbd

Re: [Xen-devel] [v7][PATCH 07/16] hvmloader/e820: construct guest e820 table

2015-07-14 Thread Jan Beulich
>>> On 14.07.15 at 12:22, wrote: > On 2015/7/14 17:32, Jan Beulich wrote: > On 14.07.15 at 07:22, wrote: > +for ( i = 0; i < memory_map.nr_map; i++ ) > +{ > +uint64_t end = e820[i].addr + e820[i].size; Either loop index/boundary or used array are wrong he

Re: [Xen-devel] [PATCH V5 0/3] Vm_event memory introspection helpers

2015-07-14 Thread Jan Beulich
>>> On 13.07.15 at 19:14, wrote: > I've also moved x86 logic in patch 3/3 to x86 source files, this > seems to have gone unnoticed but would likely have not compiled > on ARM. Which leaves open whether this time you actually checked that ARM continues to build. Jan

Re: [Xen-devel] [PATCH V3] libxl: Increase device model startup timeout to 1min.

2015-07-14 Thread Dario Faggioli
On Tue, 2015-07-14 at 10:37 +0100, Ian Campbell wrote: > On Tue, 2015-07-14 at 11:25 +0200, Dario Faggioli wrote: > > On Tue, 2015-07-14 at 08:55 +0100, Ian Campbell wrote: > > > It'll be hard to say until this change gets through the Xen push gate > > > and that version gets used for other branch

Re: [Xen-devel] [v7][PATCH 03/16] xen/passthrough: extend hypercall to support rdm reservation policy

2015-07-14 Thread Chen, Tiejun
The way this sort of thing is defined in the rest of domctl.h is like this: #define _XEN_DOMCTL_CDF_hvm_guest 0 #define XEN_DOMCTL_CDF_hvm_guest (1U<<_XEN_DOMCTL_CDF_hvm_guest) So the above should be #define _XEN_DOMCTL_DEV_RDM_RELAXED 0 #define XEN_DOMCTL_DEV_RDM_RELAXED (1U<<_XEN_DOM

Re: [Xen-devel] Requesting for freeze exception for ARM/ITS patches

2015-07-14 Thread Stefano Stabellini
On Tue, 14 Jul 2015, Ian Campbell wrote: > On Mon, 2015-07-13 at 18:24 +0100, Stefano Stabellini wrote: > > On Mon, 13 Jul 2015, Wei Liu wrote: > > > On Fri, Jul 10, 2015 at 04:16:07PM +0530, Vijay Kilari wrote: > > > > Hi Wei, > > > > > > > > I would like to have freeze exception for ITS feat

Re: [Xen-devel] [PATCH v2 1/3] xen/domain_page: Convert map_domain_page_global() to using mfn_t

2015-07-14 Thread Ian Jackson
Jan Beulich writes ("Re: [Xen-devel] [PATCH v2 1/3] xen/domain_page: Convert map_domain_page_global() to using mfn_t"): > On 13.07.15 at 18:56, wrote: > > Surely xfree() ought to have the same prototype as free() ? > > Why? If it were to be a full match, why wouldn't we call it free() in > the f

Re: [Xen-devel] [v7][PATCH 06/16] hvmloader/pci: skip reserved ranges

2015-07-14 Thread Chen, Tiejun
Note here I don't address your comments above since I think we should achieve an agreement firstly. I think bitmap mechanism is a good idea but honestly, its not easy to cover all requirements here. And just like bootmem on Linux side, so its a little complicated to implement this entirely. S

[Xen-devel] [PATCH v4 06/29] tools/libxl: Split libxl__domain_create_state.restore_fd in two

2015-07-14 Thread Andrew Cooper
In a future patch, we shall support automatically converting a legacy stream to a v2 stream, in which case libxc needs to read from a different fd. Simply overwriting restore_fd does not work; the two fd's have different circumstances. The restore_fd needs to be returned to its original state bef

[Xen-devel] [PATCH v4 04/29] tools/libxl: Introduce libxl__kill()

2015-07-14 Thread Andrew Cooper
as a wrapper to kill(2), and use it in preference to sendsig in libxl_save_callout.c. Signed-off-by: Andrew Cooper Acked-by: Wei Liu Acked-by: Ian Campbell CC: Ian Jackson --- v3: Fix typo, add _hidden Logically new in v2 - split out from a v1 change which was itself a cherrypick-and-modify

[Xen-devel] [PATCH v4 03/29] tools/libxl: Introduce ROUNDUP()

2015-07-14 Thread Andrew Cooper
This is the same as is used by libxc. Signed-off-by: Andrew Cooper Acked-by: Ian Campbell CC: Ian Jackson CC: Wei Liu --- tools/libxl/libxl_internal.h |3 +++ 1 file changed, 3 insertions(+) diff --git a/tools/libxl/libxl_internal.h b/tools/libxl/libxl_internal.h index 5235d25..19fc425 1

[Xen-devel] [PATCH v4 09/29] tools/libxl: Fix libxl__carefd_opened() to be more useful with an invalid fd

2015-07-14 Thread Andrew Cooper
In the case that fd is -1, preserve errno and don't attempt to set CLOEXEC. Note that the implementation can still fail, as it ignores fcntl() errors and may not set CLOEXEC properly. Update the documentation accordingly until it is fixed. Signed-off-by: Andrew Cooper CC: Ian Campbell CC: Ian

[Xen-devel] [PATCH v4 07/29] tools/libxl: Extra management APIs for the save helper

2015-07-14 Thread Andrew Cooper
With migration v2, there are several moving parts needing to be juggled at once. This requires the error handling logic to be able to query the state of each moving part, possibly before they have been started, and be able to cancel them. Signed-off-by: Andrew Cooper Acked-by: Ian Campbell CC:

[Xen-devel] [PATCH v4 00/27] Libxl migration v2

2015-07-14 Thread Andrew Cooper
This series adds support for the libxl migration v2 stream, and untangles the existing layering violations of the toolstack and qemu records. It can be found on the branch "libxl-migv2-v4" git://xenbits.xen.org/people/andrewcoop/xen.git http://xenbits.xen.org/git-http/people/andrewcoop/xen.git

[Xen-devel] [PATCH v4 01/29] bsd-sys-queue-h-seddery: Massage `offsetof'

2015-07-14 Thread Andrew Cooper
From: Ian Jackson For some reason BSD's queue.h uses `__offsetof'. It expects it to work just like offsetof. So use offsetof. Reported-by: Andrew Cooper Signed-off-by: Ian Jackson Acked-by: Ian Campbell --- tools/include/xen-external/bsd-sys-queue-h-seddery |2 ++ 1 file changed, 2 ins

[Xen-devel] [PATCH v4 08/29] tools/libxl: Add save_helper_state pointers to libxl__xc_domain_{save, restore}()

2015-07-14 Thread Andrew Cooper
Currently, libxl__xc_domain_{save,restore}() have specific knowledge of where the libxl__save_helper_state lives inside a libxl__domain_{create,save}_state object. In later changes, the logical ownership of the libxl__save_helper_state will change and will no longer be d{c,s}s->shs. No functional

[Xen-devel] [PATCH v4 02/29] tools/libxc: Always compile the compat qemu variables into xc_sr_context

2015-07-14 Thread Andrew Cooper
This is safe (as the variables will simply be unused), and is required for correct compilation when midway through untangling the libxc/libxl interaction. The #define is left in place to highlight that the variables can be removed once the untangling is complete. Signed-off-by: Andrew Cooper Ack

[Xen-devel] [PATCH v4 05/29] tools/libxl: Stash all restore parameters in domain_create_state

2015-07-14 Thread Andrew Cooper
Shortly more parameters will appear, and this saves unboxing each one. libxl_domain_restore_params is mandatory for restore streams, and ignored for plain creation. The old 'checkpointed_stream' was incorrectly identified as a private parameter when it was infact public. No functional change. Si

[Xen-devel] [PATCH v4 27/29] tools/libxl: Handle checkpoint records in a libxl migration v2 stream

2015-07-14 Thread Andrew Cooper
This is the final bit of untangling for Remus. When libxc issues a checkpoint callback, start reading and buffering all libxl records from the stream. Once a CHECKPOINT_END record is encountered, start processing all records. Signed-off-by: Andrew Cooper Acked-by: Ian Campbell CC: Ian Jackson

[Xen-devel] [PATCH v4 25/29] tools/libxl: Write checkpoint records into the stream

2015-07-14 Thread Andrew Cooper
when signalled to do so by libxl__remus_domain_checkpoint_callback() Signed-off-by: Andrew Cooper Acked-by: Ian Campbell CC: Ian Jackson CC: Wei Liu --- v3: Corrections to comments This patch has changed substantially in v2 as a result of changes earlier in the series. No behavioural differ

[Xen-devel] [PATCH v4 24/29] docs/libxl: Introduce CHECKPOINT_END to support migration v2 remus streams

2015-07-14 Thread Andrew Cooper
In a remus scenario, libxc will write a CHECKPOINT record, then hand ownership of the fd to libxl. Libxl then writes any records required and finishes with a CHECKPOINT_END record, then hands ownership of the fd back to libxc. Signed-off-by: Andrew Cooper Acked-by: Ian Campbell CC: Ian Jackson

[Xen-devel] [PATCH v4 10/29] tools/xl: Mandatory flag indicating the format of the migration stream

2015-07-14 Thread Andrew Cooper
Introduced at this point so the python stream conversion code has a concrete ABI to use. Later when libxl itself starts supporting a v2 stream, it will be added to XL_MANDATORY_FLAG_ALL. Signed-off-by: Andrew Cooper Acked-by: Ian Campbell CC: Ian Jackson CC: Wei Liu --- v2: Expand commit mes

[Xen-devel] [PATCH v4 23/29] tools/libxc+libxl+xl: Save v2 streams

2015-07-14 Thread Andrew Cooper
This is a complicated set of changes which must be done together for bisectability. * libxl-save-helper is updated to unconditionally use libxc migration v2. * libxl compatibility workarounds in libxc are disabled for save operations. * libxl__stream_write_start() is logically spliced into the

[Xen-devel] [PATCH v4 22/29] tools/libxl: Infrastructure for writing a v2 stream

2015-07-14 Thread Andrew Cooper
From: Ross Lagerwall This contains the event machinery and state machines to write non-checkpointed migration v2 stream (with the exception of the xc_domain_save() handling which is spliced later in a bisectable way). Signed-off-by: Ross Lagerwall Signed-off-by: Andrew Cooper CC: Ian Campbell

[Xen-devel] [PATCH v4 28/29] tools/libxc: Drop all XG_LIBXL_HVM_COMPAT code from libxc

2015-07-14 Thread Andrew Cooper
Libxl has now been fully adjusted not to need it. Signed-off-by: Andrew Cooper Acked-by: Ian Campbell CC: Ian Jackson CC: Wei Liu --- tools/libxc/xc_sr_common.h |5 -- tools/libxc/xc_sr_restore.c | 18 - tools/libxc/xc_sr_restore_x86_hvm.c | 124 ---

[Xen-devel] [PATCH v4 17/29] tools/libxl: Migration v2 stream format

2015-07-14 Thread Andrew Cooper
From: Ross Lagerwall C structures describing the Libxl migration v2 stream format Signed-off-by: Ross Lagerwall Signed-off-by: Andrew Cooper Acked-by: Ian Campbell CC: Ian Jackson CC: Wei Liu --- v2: Move into libxl__ namespace --- tools/libxl/libxl_sr_stream_format.h | 57 +

[Xen-devel] [PATCH v4 11/29] docs: Libxl migration v2 stream specification

2015-07-14 Thread Andrew Cooper
Signed-off-by: Andrew Cooper Acked-by: Ian Campbell CC: Ian Jackson CC: Wei Liu --- v3: Spelling fixes --- docs/specs/libxl-migration-stream.pandoc | 205 ++ 1 file changed, 205 insertions(+) create mode 100644 docs/specs/libxl-migration-stream.pandoc diff --git

[Xen-devel] [PATCH v4 29/29] tools/libxl: Drop all knowledge of toolstack callbacks

2015-07-14 Thread Andrew Cooper
Libxl has now been fully adjusted not to need them. Signed-off-by: Andrew Cooper Acked-by: Ian Campbell CC: Ian Jackson CC: Wei Liu --- tools/libxl/libxl_dom.c|1 - tools/libxl/libxl_internal.h |2 -- tools/libxl/libxl_save_callout.c | 39 +---

[Xen-devel] [PATCH v4 13/29] tools/python: Libxl migration v2 infrastructure

2015-07-14 Thread Andrew Cooper
Contains: * Python implementation of the libxl migration v2 records * Verification code for spec compliance * Unit tests Signed-off-by: Andrew Cooper Acked-by: Ian Campbell CC: Ian Jackson CC: Wei Liu --- tools/python/xen/migration/libxl.py | 188 +++ tools

[Xen-devel] [PATCH v4 12/29] tools/python: Libxc migration v2 infrastructure

2015-07-14 Thread Andrew Cooper
Contains: * Python implementation of the libxc migration v2 records * Verification code for spec compliance * Unit tests Signed-off-by: Andrew Cooper Acked-by: Ian Campbell CC: Ian Jackson CC: Wei Liu --- tools/python/setup.py|1 + tools/python/xen/migration/libxc.py |

[Xen-devel] [PATCH v4 14/29] tools/python: Other migration infrastructure

2015-07-14 Thread Andrew Cooper
Contains: * Reverse-engineered notes of the legacy format from xg_save_restore.h * Python implementation of the legacy format * Public HVM Params used in the legacy stream * XL header format Signed-off-by: Andrew Cooper Acked-by: Ian Campbell CC: Ian Jackson CC: Wei Liu --- New in v2 - re

[Xen-devel] [PATCH v4 18/29] tools/libxl: Infrastructure for reading a libxl migration v2 stream

2015-07-14 Thread Andrew Cooper
From: Ross Lagerwall This contains the event machinery and state machines to read an act on a non-checkpointed migration v2 stream (with the exception of the xc_domain_restore() handling which is spliced later in a bisectable way). It also contains some boilerplate to help support checkpointed s

[Xen-devel] [PATCH v4 20/29] tools/libxl: Convert a legacy stream if needed

2015-07-14 Thread Andrew Cooper
For backwards compatibility, a legacy stream needs converting before it can be read by the v2 stream logic. This causes the v2 stream logic to need to juggle two parallel tasks. check_all_finished() is introduced for the purpose of joining the tasks in both success and error cases. Signed-off-by:

[Xen-devel] [PATCH v4 16/29] tools/python: Conversion utility for legacy migration streams

2015-07-14 Thread Andrew Cooper
This utility will take a legacy stream as in input, and produce a v2 stream as an output. It is exec()'d by libxl to provide backwards compatibility. Signed-off-by: Andrew Cooper Acked-by: Ian Campbell CC: Ian Jackson CC: Wei Liu --- tools/python/Makefile |4 + tools

[Xen-devel] [PATCH v4 26/29] tools/libx{c, l}: Introduce restore_callbacks.checkpoint()

2015-07-14 Thread Andrew Cooper
And call it when a checkpoint record is found in the libxc stream. Some parts of this patch have been based on patches from the COLO series. Signed-off-by: Wen Congyang Signed-off-by: Yang Hongyang Signed-off-by: Andrew Cooper Acked-by: Ian Campbell CC: Ian Jackson CC: Wei Liu --- v3: Name

[Xen-devel] [PATCH v4 19/29] tools/libxl: Infrastructure to convert a legacy stream

2015-07-14 Thread Andrew Cooper
Provide a thin wrapper around exec()ing the python conversion utility, and a stub implementation for cases where conversion is not wanted (i.e. not x86). One complication is that the caller of this interface needs to assume ownership of the output fd, to prevent it being closed while still in use

[Xen-devel] [PATCH v4 15/29] tools/python: Verification utility for v2 stream spec compliance

2015-07-14 Thread Andrew Cooper
Signed-off-by: Andrew Cooper Acked-by: Ian Campbell CC: Ian Jackson CC: Wei Liu --- This is exceedingly useful for development, but not of practical use being installed into a production dom0. --- tools/python/scripts/verify-stream-v2 | 174 + 1 file changed,

[Xen-devel] [PATCH v4 21/29] tools/libxc+libxl+xl: Restore v2 streams

2015-07-14 Thread Andrew Cooper
This is a complicated set of changes which must be done together for bisectability. * libxl-save-helper is updated to unconditionally use libxc migration v2. * libxl compatibility workarounds in libxc are disabled for restore operations. * libxl__stream_read_start() is logically spliced i

Re: [Xen-devel] [PATCH v3 03/13] VMX: implement suppress #VE.

2015-07-14 Thread George Dunlap
On 07/13/2015 08:40 AM, Jan Beulich wrote: On 10.07.15 at 21:30, wrote: >>> From: Jan Beulich [mailto:jbeul...@suse.com] >>> Sent: Thursday, July 09, 2015 6:01 AM >> On 01.07.15 at 20:09, wrote: @@ -232,6 +235,15 @@ static int ept_set_middle_entry(struct p2m_domain @@ -1134,6 +

Re: [Xen-devel] [v7][PATCH 03/16] xen/passthrough: extend hypercall to support rdm reservation policy

2015-07-14 Thread George Dunlap
On 07/14/2015 11:53 AM, Chen, Tiejun wrote: >> The way this sort of thing is defined in the rest of domctl.h is like >> this: >> >> #define _XEN_DOMCTL_CDF_hvm_guest 0 >> #define XEN_DOMCTL_CDF_hvm_guest (1U<<_XEN_DOMCTL_CDF_hvm_guest) >> >> So the above should be >> >> #define _XEN_DOMCTL

[Xen-devel] [PATCH] x86/HVM: drop now wrong ASSERT() from hvm_broadcast_ioreq()

2015-07-14 Thread Jan Beulich
The function is now also being used for IOREQ_TYPE_TIMEOFFSET. Signed-off-by: Jan Beulich --- a/xen/arch/x86/hvm/hvm.c +++ b/xen/arch/x86/hvm/hvm.c @@ -2717,8 +2717,6 @@ unsigned int hvm_broadcast_ioreq(ioreq struct hvm_ioreq_server *s; unsigned int failed = 0; -ASSERT(p->type ==

Re: [Xen-devel] [PATCH v3 05/13] x86/altp2m: basic data structures and support routines.

2015-07-14 Thread George Dunlap
On 07/14/2015 01:01 AM, Sahita, Ravi wrote: > > >> -Original Message- >> From: Jan Beulich [mailto:jbeul...@suse.com] >> Sent: Monday, July 13, 2015 1:01 AM >> > On 10.07.15 at 23:48, wrote: From: Jan Beulich [mailto:jbeul...@suse.com] Sent: Thursday, July 09, 2015 6:30 AM

Re: [Xen-devel] [PATCH v2 1/3] xen/domain_page: Convert map_domain_page_global() to using mfn_t

2015-07-14 Thread Jan Beulich
>>> On 14.07.15 at 12:54, wrote: > Jan Beulich writes ("Re: [Xen-devel] [PATCH v2 1/3] xen/domain_page: Convert > map_domain_page_global() to using mfn_t"): >> On 13.07.15 at 18:56, wrote: >> > Surely xfree() ought to have the same prototype as free() ? >> >> Why? If it were to be a full match,

Re: [Xen-devel] [PATCH RFC 1 6/8] xen/pt: Make xen_pt_unregister_device idempotent

2015-07-14 Thread Stefano Stabellini
On Thu, 2 Jul 2015, Konrad Rzeszutek Wilk wrote: > > > @@ -858,15 +863,20 @@ static void xen_pt_unregister_device(PCIDevice *d) > > > machine_irq, errno); > > > } > > > } > > > +s->machine_irq = 0; > > > } > > > > > > /* delete a

Re: [Xen-devel] [PATCH V5 0/3] Vm_event memory introspection helpers

2015-07-14 Thread Razvan Cojocaru
On 07/14/2015 01:50 PM, Jan Beulich wrote: On 13.07.15 at 19:14, wrote: >> I've also moved x86 logic in patch 3/3 to x86 source files, this >> seems to have gone unnoticed but would likely have not compiled >> on ARM. > > Which leaves open whether this time you actually checked that > ARM co

Re: [Xen-devel] [v7][PATCH 03/16] xen/passthrough: extend hypercall to support rdm reservation policy

2015-07-14 Thread Jan Beulich
>>> On 14.07.15 at 13:30, wrote: > On 07/14/2015 11:53 AM, Chen, Tiejun wrote: >>> The way this sort of thing is defined in the rest of domctl.h is like >>> this: >>> >>> #define _XEN_DOMCTL_CDF_hvm_guest 0 >>> #define XEN_DOMCTL_CDF_hvm_guest (1U<<_XEN_DOMCTL_CDF_hvm_guest) >>> >>> So th

Re: [Xen-devel] [v7][PATCH 06/16] hvmloader/pci: skip reserved ranges

2015-07-14 Thread Jan Beulich
>>> On 14.07.15 at 12:54, wrote: >>> I think bitmap mechanism is a good idea but honestly, its not easy to >>> cover all requirements here. And just like bootmem on Linux side, so its >>> a little complicated to implement this entirely. So I prefer not to >>> introduce this way in current phase. >

Re: [Xen-devel] [PATCH v4 12/15] x86/altp2m: Add altp2mhvm HVM domain parameter.

2015-07-14 Thread George Dunlap
On 07/10/2015 11:12 PM, Sahita, Ravi wrote: >> From: dunl...@gmail.com [mailto:dunl...@gmail.com] On Behalf Of George >> Dunlap >> Sent: Friday, July 10, 2015 10:32 AM >> >> On Fri, Jul 10, 2015 at 1:52 AM, Ed White >> wrote: >>> The altp2mhvm and nestedhvm parameters are mutually exclusive and >>

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

2015-07-14 Thread Jan Beulich
>>> On 14.07.15 at 08:23, wrote: > flight 59509 xen-unstable real [real] > http://logs.test-lab.xenproject.org/osstest/logs/59509/ > > Regressions :-( > > Tests which did not succeed and are blocking, > including tests which could not be run: > test-amd64-amd64-xl-qemut-stubdom-debianhvm-amd64

Re: [Xen-devel] [PATCH v5 15/15] tools/xen-access: altp2m testcases

2015-07-14 Thread Lengyel, Tamas
On Tue, Jul 14, 2015 at 5:56 AM, Wei Liu wrote: > On Mon, Jul 13, 2015 at 05:15:03PM -0700, Ed White wrote: > > From: Tamas K Lengyel > > > > Working altp2m test-case. Extended the test tool to support > singlestepping > > to better highlight the core feature of altp2m view switching. > > > > Si

Re: [Xen-devel] [PATCH V5 0/3] Vm_event memory introspection helpers

2015-07-14 Thread Jan Beulich
>>> On 14.07.15 at 13:45, wrote: > On 07/14/2015 01:50 PM, Jan Beulich wrote: > On 13.07.15 at 19:14, wrote: >>> I've also moved x86 logic in patch 3/3 to x86 source files, this >>> seems to have gone unnoticed but would likely have not compiled >>> on ARM. >> >> Which leaves open whether th

Re: [Xen-devel] [PATCH v2 0/3] arm64: Add multiboot support (via fdt) for Xen boot

2015-07-14 Thread Fu Wei
Hi Ian, Great thanks for your testing, will add your tested-by in my next patchset :-) On 14 July 2015 at 17:29, Ian Campbell wrote: > On Mon, 2015-07-13 at 16:53 +0800, fu@linaro.org wrote: >> From: Fu Wei >> >> - This adds support for the Xen boot on ARM specification for arm64. > > I ha

  1   2   3   >