Re: [Xen-devel] [PATCH v8 02/13] x86: detect and initialize Intel CAT feature

2015-05-29 Thread Jan Beulich
On 29.05.15 at 04:40, chao.p.p...@linux.intel.com wrote: On Thu, May 28, 2015 at 01:54:39PM +0100, Jan Beulich wrote: On 21.05.15 at 10:41, chao.p.p...@linux.intel.com wrote: + +if ( !cpu_has(c, X86_FEATURE_CAT) ) +return; + +socket = cpu_to_socket(cpu); +if (

Re: [Xen-devel] [PATCH v8 01/13] x86: add socket_cpumask

2015-05-29 Thread Chao Peng
On Fri, May 29, 2015 at 09:01:53AM +0100, Jan Beulich wrote: On 29.05.15 at 04:35, chao.p.p...@linux.intel.com wrote: On Thu, May 28, 2015 at 01:38:05PM +0100, Jan Beulich wrote: On 21.05.15 at 10:41, chao.p.p...@linux.intel.com wrote: --- a/xen/arch/x86/mpparse.c +++

[Xen-devel] [xen-4.3-testing test] 57458: regressions - trouble: blocked/broken/fail/pass

2015-05-29 Thread osstest service user
flight 57458 xen-4.3-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/57458/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xend-qemut-winxpsp3 16 guest-stop fail REGR. vs. 53768 Tests which are

[Xen-devel] [PATCH] efi: Reallocate memory map if ExitBootServices() fails

2015-05-29 Thread Ross Lagerwall
If calling ExitBootServices() fails, the memory map size may have increased, so determine the new size and reallocate the memory map before calling GetMemoryMap() again. This was seen on the following machine when using the iscsidxe UEFI driver. The machine would consistently fail the first call

Re: [Xen-devel] [RFC][v2][PATCH 00/14] Fix RMRR

2015-05-29 Thread Chen, Tiejun
On 2015/5/28 15:55, Jan Beulich wrote: On 28.05.15 at 07:48, tiejun.c...@intel.com wrote: On 2015/5/22 17:46, Jan Beulich wrote: On 22.05.15 at 11:35, tiejun.c...@intel.com wrote: As you know all devices are owned by Dom0 firstly before we create any DomU, right? Do we allow Dom0 still own a

Re: [Xen-devel] [PATCH v2 5/9] x86/intel_pstate: relocate the driver register/unregister function

2015-05-29 Thread Jan Beulich
On 29.05.15 at 04:47, wei.w.w...@intel.com wrote: On 26/05/2015 21:06, Jan Beulich wrote On 13.05.16 at 09:50, wei.w.w...@intel.com wrote: Register/unregister the CPU hotplug notifier when the driver is registered, and move the driver register/unregister function to the cpufreq.c.

[Xen-devel] [PATCH] libxl: Don't insert PCI device into xenstore for HVM guests

2015-05-29 Thread Ross Lagerwall
When doing passthrough of a PCI device for an HVM guest, don't insert the device into xenstore, otherwise pciback attempts to use it which conflicts with QEMU. This manifests itself such that the first time a device is passed to a domain, it succeeds. Subsequent attempts fail unless the device is

Re: [Xen-devel] [PATCH v8 01/13] x86: add socket_cpumask

2015-05-29 Thread Jan Beulich
On 29.05.15 at 04:35, chao.p.p...@linux.intel.com wrote: On Thu, May 28, 2015 at 01:38:05PM +0100, Jan Beulich wrote: On 21.05.15 at 10:41, chao.p.p...@linux.intel.com wrote: --- a/xen/arch/x86/mpparse.c +++ b/xen/arch/x86/mpparse.c @@ -87,6 +87,18 @@ void __init set_nr_cpu_ids(unsigned

Re: [Xen-devel] [PATCH v2 6/9] x86/intel_pstate: the main boby of the intel_pstate driver

2015-05-29 Thread Wang, Wei W
On 26/05/2015 21:58, Jan Beulich wrote On 13.05.16 at 09:50, wei.w.w...@intel.com wrote: +static inline int ceiling_fp(int32_t x) { +int mask, ret; Please here and below, consider whether types really need to be signed. One exception: If you intend to import the Linux source file

Re: [Xen-devel] [PATCH v8 03/13] x86: maintain COS to CBM mapping for each socket

2015-05-29 Thread Chao Peng
On Fri, May 29, 2015 at 09:06:46AM +0100, Jan Beulich wrote: On 29.05.15 at 04:43, chao.p.p...@linux.intel.com wrote: On Thu, May 28, 2015 at 02:17:54PM +0100, Jan Beulich wrote: On 21.05.15 at 10:41, chao.p.p...@linux.intel.com wrote: +static int cat_cpu_init(unsigned int cpu) +{

Re: [Xen-devel] [PATCH v2 6/9] x86/intel_pstate: the main boby of the intel_pstate driver

2015-05-29 Thread Jan Beulich
On 29.05.15 at 10:19, wei.w.w...@intel.com wrote: On 26/05/2015 21:58, Jan Beulich wrote On 13.05.16 at 09:50, wei.w.w...@intel.com wrote: +static int intel_pstate_verify_policy(struct cpufreq_policy *policy) +{ +cpufreq_verify_within_limits(policy, policy-cpuinfo.min_freq, +

[Xen-devel] [PATCH v2] x86: Don't crash when mapping a page using EFI runtime page tables

2015-05-29 Thread Ross Lagerwall
When an interrupt is received during an EFI runtime service call, Xen may call map_domain_page() while using the EFI runtime page tables. This fails because, although the EFI runtime page tables are a copy of the idle domain's page tables, current points at a different domain's vCPU. To fix this,

Re: [Xen-devel] [PATCH v8 03/13] x86: maintain COS to CBM mapping for each socket

2015-05-29 Thread Jan Beulich
On 29.05.15 at 04:43, chao.p.p...@linux.intel.com wrote: On Thu, May 28, 2015 at 02:17:54PM +0100, Jan Beulich wrote: On 21.05.15 at 10:41, chao.p.p...@linux.intel.com wrote: +static int cat_cpu_init(unsigned int cpu) +{ +int rc; +const struct cpuinfo_x86 *c = cpu_data + cpu;

Re: [Xen-devel] [PATCH v8 05/13] x86: expose CBM length and COS number information

2015-05-29 Thread Jan Beulich
On 29.05.15 at 04:47, chao.p.p...@linux.intel.com wrote: On Thu, May 28, 2015 at 02:26:03PM +0100, Jan Beulich wrote: --- a/xen/include/public/sysctl.h +++ b/xen/include/public/sysctl.h @@ -694,6 +694,20 @@ struct xen_sysctl_pcitopoinfo { typedef struct xen_sysctl_pcitopoinfo

Re: [Xen-devel] [PATCHv10 3/4] gnttab: make the grant table lock a read-write lock

2015-05-29 Thread Jan Beulich
On 28.05.15 at 18:09, dvra...@cantab.net wrote: On 28/05/15 15:55, Jan Beulich wrote: On 26.05.15 at 20:00, david.vra...@citrix.com wrote: @@ -254,23 +254,23 @@ double_gt_lock(struct grant_table *lgt, struct grant_table *rgt) { if ( lgt rgt ) { -spin_lock(lgt-lock);

Re: [Xen-devel] [PATCHv10 4/4] gnttab: use per-VCPU maptrack free lists

2015-05-29 Thread Jan Beulich
On 28.05.15 at 18:10, dvra...@cantab.net wrote: On 28/05/15 16:39, Jan Beulich wrote: On 26.05.15 at 20:00, david.vra...@citrix.com wrote: --- a/xen/common/grant_table.c +++ b/xen/common/grant_table.c @@ -57,7 +57,7 @@ integer_param(gnttab_max_frames, max_grant_frames); * New options

Re: [Xen-devel] [PATCH v8 01/13] x86: add socket_cpumask

2015-05-29 Thread Jan Beulich
On 29.05.15 at 10:28, chao.p.p...@linux.intel.com wrote: On Fri, May 29, 2015 at 09:01:53AM +0100, Jan Beulich wrote: On 29.05.15 at 04:35, chao.p.p...@linux.intel.com wrote: On Thu, May 28, 2015 at 01:38:05PM +0100, Jan Beulich wrote: On 21.05.15 at 10:41, chao.p.p...@linux.intel.com

[Xen-devel] [PATCH] x86/paging: remove pointless current domain checks

2015-05-29 Thread Jan Beulich
Checking that the subject domain is not the current one is pointless when already having paused that domain: domain_pause() already ASSERT()s this to be the case. Signed-off-by: Jan Beulich jbeul...@suse.com --- a/xen/arch/x86/mm/hap/hap.c +++ b/xen/arch/x86/mm/hap/hap.c @@ -464,13 +464,6 @@ int

Re: [Xen-devel] [RFC PATCH 00/13] Persistent grant maps for xen net drivers

2015-05-29 Thread Yuzhou (C)
Hi, About rx zerocopy, I have a question: If some application make a socket, then listen and accept, the client sends packets to it, but it doesn't recv from this socket right now, all persistent grant page would be in used. So other application cannot receive any packets. Is

Re: [Xen-devel] Xen BUG at page_alloc.c:1738 (Xen 4.5)

2015-05-29 Thread Jason Fritcher
On Wed, 20 May 2015, Major Hayden wrote: On 05/20/2015 05:41 AM, Jan Beulich wrote: Considering that no-one else is seeing this - is this perhaps connected to you building Xen with pre-release gcc 5.0.1? This is also because in order for the above to indeed occur, mmio_ro_do_page_fault()'s

[Xen-devel] [PATCH v2 2/4] libxc: print more error messages when failed

2015-05-29 Thread Wei Liu
No functional changes introduced. Signed-off-by: Wei Liu wei.l...@citrix.com Cc: Ian Campbell ian.campb...@citrix.com Cc: Ian Jackson ian.jack...@eu.citrix.com --- tools/libxc/xc_hvm_build_x86.c | 30 +++--- 1 file changed, 27 insertions(+), 3 deletions(-) diff --git

[Xen-devel] [PATCH v2 3/4] libxc: rework vnuma bits in setup_guest

2015-05-29 Thread Wei Liu
Make the setup process similar to PV counterpart. That is, to allocate a P2M array that covers the whole memory range and start from there. This is clearer than using an array with no holes in it. Also the dummy layout should take MMIO hole into consideration. We might end up having two

[Xen-devel] [PATCH v2 4/4] libxl: fix HVM vNUMA

2015-05-29 Thread Wei Liu
This patch does two thing: The original code erroneously fills in xc_hvm_build_args before generating vmemranges. The effect is that guest memory is populated without vNUMA information. Move the hunk to right place to fix this. Move the subtraction of video ram to

[Xen-devel] [PATCH v2 0/4] Fix HVM vNUMA

2015-05-29 Thread Wei Liu
Boris discovered that HVM vNUMA didn't actually work. This patch series fixes that. The first patch is a prerequisite patch for the actual fixes. The second patch is to help debugging. The fixes are in the last two patches, which can be squashed into one if necessary. Patch 3 is updated in this

[Xen-devel] [PATCH v2 1/4] libxc/libxl: fill xc_hvm_build_args in libxl

2015-05-29 Thread Wei Liu
When building HVM guests, originally some fields of xc_hvm_build_args are filled in xc_hvm_build (and buried in the wrong function), some are set in libxl__build_hvm before passing xc_hvm_build_args to xc_hvm_build. This is fragile. After examining the code in xc_hvm_build that sets those fields,

Re: [Xen-devel] [PATCH Remus v7 0/3] Remus support for Migration-v2

2015-05-29 Thread Ian Campbell
On Mon, 2015-05-25 at 17:06 +0800, Yang Hongyang wrote: Ping... Acked + Applied. Sorry for the delay, for some reason I thought I was waiting for Andrew to comment even though they all already had Reviewed-by. Ian. On 05/18/2015 03:03 PM, Yang Hongyang wrote: This patchset implement the

Re: [Xen-devel] [PATCH] efi: Reallocate memory map if ExitBootServices() fails

2015-05-29 Thread Andrew Cooper
On 29/05/15 08:48, Ross Lagerwall wrote: If calling ExitBootServices() fails, the memory map size may have increased, so determine the new size and reallocate the memory map before calling GetMemoryMap() again. This was seen on the following machine when using the iscsidxe UEFI driver. The

[Xen-devel] [RFC][PATCH] x86: remove vmalloc.h from asm/io.h

2015-05-29 Thread Stephen Rothwell
Nothing in asm/io.h uses anything from vmalloc.h, so remove the include and fix up the build problems in an allmodconfig (64 bit and 32 bit) build. This may be the place where x86 builds get vmalloc.h implicitly included and that tends to hide places where vmalloc() et al are added to files but

Re: [Xen-devel] [PATCH] libxl: Don't insert PCI device into xenstore for HVM guests

2015-05-29 Thread Ross Lagerwall
On 05/29/2015 10:41 AM, Wei Liu wrote: On Fri, May 29, 2015 at 08:59:45AM +0100, Ross Lagerwall wrote: When doing passthrough of a PCI device for an HVM guest, don't insert the device into xenstore, otherwise pciback attempts to use it which conflicts with QEMU. This manifests itself such that

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

2015-05-29 Thread Andrew Cooper
On 28/05/15 11:10, Jan Beulich wrote: On 28.05.15 at 11:26, ian.campb...@citrix.com wrote: On Thu, 2015-05-28 at 09:50 +0100, Jan Beulich wrote: On 27.05.15 at 18:04, ian.campb...@citrix.com wrote: On Tue, 2015-05-26 at 14:29 +0100, Ian Campbell wrote: I've now managed to reproduce using the

Re: [Xen-devel] [PATCH] efi: Reallocate memory map if ExitBootServices() fails

2015-05-29 Thread Ross Lagerwall
On 05/29/2015 10:45 AM, Jan Beulich wrote: On 29.05.15 at 09:48, ross.lagerw...@citrix.com wrote: --- a/xen/common/efi/boot.c +++ b/xen/common/efi/boot.c @@ -1053,14 +1053,14 @@ efi_start(EFI_HANDLE ImageHandle, EFI_SYSTEM_TABLE *SystemTable) efi_arch_video_init(gop, info_size,

Re: [Xen-devel] Xen BUG at page_alloc.c:1738 (Xen 4.5)

2015-05-29 Thread Andrew Cooper
On 29/05/15 07:24, Jason Fritcher wrote: On Wed, 20 May 2015, Major Hayden wrote: / On 05/20/2015 05:41 AM, Jan Beulich wrote:/ / Considering that no-one else is seeing this - is this perhaps connected/ / to you building Xen with pre-release gcc 5.0.1? This is also because in/ / order for

Re: [Xen-devel] [PATCH] libxl: Don't insert PCI device into xenstore for HVM guests

2015-05-29 Thread Wei Liu
On Fri, May 29, 2015 at 10:54:09AM +0100, Ross Lagerwall wrote: On 05/29/2015 10:50 AM, Wei Liu wrote: On Fri, May 29, 2015 at 10:43:08AM +0100, Ross Lagerwall wrote: On 05/29/2015 10:41 AM, Wei Liu wrote: On Fri, May 29, 2015 at 08:59:45AM +0100, Ross Lagerwall wrote: When doing passthrough

Re: [Xen-devel] [PATCH v2 for Xen 4.6 0/4] Enabling XL to set per-VCPU parameters of a domain for RTDS scheduler

2015-05-29 Thread Dario Faggioli
On Tue, 2015-05-26 at 09:59 +0100, Ian Campbell wrote: On Mon, 2015-05-25 at 18:59 -0500, Chong Li wrote: This series arrived in my mailbox as 5 distinct mails. Please use git send-email such that the mails arrive as a single email thread (i.e. each mail as a reply to the previous or to

Re: [Xen-devel] [PATCH v2 1/4] libxc/libxl: fill xc_hvm_build_args in libxl

2015-05-29 Thread Andrew Cooper
On 29/05/15 12:37, Wei Liu wrote: When building HVM guests, originally some fields of xc_hvm_build_args are filled in xc_hvm_build (and buried in the wrong function), some are set in libxl__build_hvm before passing xc_hvm_build_args to xc_hvm_build. This is fragile. After examining the code

Re: [Xen-devel] [PATCH] efi: Reallocate memory map if ExitBootServices() fails

2015-05-29 Thread Jan Beulich
On 29.05.15 at 09:48, ross.lagerw...@citrix.com wrote: --- a/xen/common/efi/boot.c +++ b/xen/common/efi/boot.c @@ -1053,14 +1053,14 @@ efi_start(EFI_HANDLE ImageHandle, EFI_SYSTEM_TABLE *SystemTable) efi_arch_video_init(gop, info_size, mode_info); } -

Re: [Xen-devel] Question on QEMU restore process

2015-05-29 Thread Lengyel, Tamas
On Tue, May 26, 2015 at 5:49 PM, Ian Campbell ian.campb...@citrix.com wrote: On Tue, 2015-05-26 at 17:38 +0200, Lengyel, Tamas wrote: Hi all, I'm wondering if someone can point me in the right direction. I'm trying to understand the process around domain save/restore using XL. I can

Re: [Xen-devel] [PATCH v8 05/13] x86: expose CBM length and COS number information

2015-05-29 Thread Dario Faggioli
On Thu, 2015-05-28 at 14:26 +0100, Jan Beulich wrote: On 21.05.15 at 10:41, chao.p.p...@linux.intel.com wrote: --- a/xen/include/public/sysctl.h +++ b/xen/include/public/sysctl.h @@ -694,6 +694,20 @@ struct xen_sysctl_pcitopoinfo { typedef struct xen_sysctl_pcitopoinfo

Re: [Xen-devel] [PATCH V7] xen/vm_event: Clean up control-register-write vm_events and add XCR0 event

2015-05-29 Thread Jan Beulich
On 28.05.15 at 18:32, rcojoc...@bitdefender.com wrote: On 05/28/2015 07:06 PM, Lengyel, Tamas wrote: That seems a bit odd to me that we now have both a macro and a function with the same name. I think making the macro at least all capital would avoid confusion and make the code a lot more

Re: [Xen-devel] [RFC][PATCH] x86: remove vmalloc.h from asm/io.h

2015-05-29 Thread Ingo Molnar
* Stephen Rothwell s...@canb.auug.org.au wrote: Nothing in asm/io.h uses anything from vmalloc.h, so remove the include and fix up the build problems in an allmodconfig (64 bit and 32 bit) build. This may be the place where x86 builds get vmalloc.h implicitly included and that tends to

Re: [Xen-devel] [PATCH v8 05/13] x86: expose CBM length and COS number information

2015-05-29 Thread Jan Beulich
On 29.05.15 at 11:23, dario.faggi...@citrix.com wrote: On Fri, 2015-05-29 at 09:07 +0100, Jan Beulich wrote: On 29.05.15 at 04:47, chao.p.p...@linux.intel.com wrote: On Thu, May 28, 2015 at 02:26:03PM +0100, Jan Beulich wrote: --- a/xen/include/public/sysctl.h +++

[Xen-devel] [PATCH] x86/mem_sharing: Relax sanity check for memops

2015-05-29 Thread Tamas K Lengyel
The sharing vm_event ring being enabled is not necessary for mem_sharing memops. Signed-off-by: Tamas K Lengyel tkleng...@sec.in.tum.de --- xen/arch/x86/mm/mem_sharing.c | 4 1 file changed, 4 deletions(-) diff --git a/xen/arch/x86/mm/mem_sharing.c b/xen/arch/x86/mm/mem_sharing.c index

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

2015-05-29 Thread Jan Beulich
On 29.05.15 at 11:56, andrew.coop...@citrix.com wrote: On 28/05/15 11:10, Jan Beulich wrote: On 28.05.15 at 11:26, ian.campb...@citrix.com wrote: On Thu, 2015-05-28 at 09:50 +0100, Jan Beulich wrote: On 27.05.15 at 18:04, ian.campb...@citrix.com wrote: On Tue, 2015-05-26 at 14:29 +0100, Ian

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

2015-05-29 Thread Ian Campbell
On Fri, 2015-05-29 at 10:56 +0100, Andrew Cooper wrote: On 28/05/15 11:10, Jan Beulich wrote: On 28.05.15 at 11:26, ian.campb...@citrix.com wrote: On Thu, 2015-05-28 at 09:50 +0100, Jan Beulich wrote: On 27.05.15 at 18:04, ian.campb...@citrix.com wrote: On Tue, 2015-05-26 at 14:29 +0100,

Re: [Xen-devel] xen-netfront sets partial checksum at wrong offset

2015-05-29 Thread Jan Beulich
On 29.05.15 at 12:39, wei.l...@citrix.com wrote: On Fri, May 29, 2015 at 11:34:07AM +0100, Jan Beulich wrote: On 11.05.15 at 19:25, venkat.x.venkatsu...@oracle.com wrote: Please CC the maintainers of the driver. You can get that from 'scripts/get_maintainer.pl' I've done that for

Re: [Xen-devel] Xen BUG at page_alloc.c:1738 (Xen 4.5)

2015-05-29 Thread Andrew Cooper
On 29/05/15 11:50, M A Young wrote: On Fri, 29 May 2015, Andrew Cooper wrote: Are you in a position to compile identical Xen 4.5 source with two different versions of gcc? (current staging-4.5 staging even has the gcc5 build fix in) If it is a gcc compiler bug, we would expect the version

Re: [Xen-devel] [PATCH] x86/paging: remove pointless current domain checks

2015-05-29 Thread Andrew Cooper
On 29/05/15 10:09, Jan Beulich wrote: Checking that the subject domain is not the current one is pointless when already having paused that domain: domain_pause() already ASSERT()s this to be the case. Signed-off-by: Jan Beulich jbeul...@suse.com Reviewed-by: Andrew Cooper

Re: [Xen-devel] [PATCH] libxl: Don't insert PCI device into xenstore for HVM guests

2015-05-29 Thread Wei Liu
On Fri, May 29, 2015 at 08:59:45AM +0100, Ross Lagerwall wrote: When doing passthrough of a PCI device for an HVM guest, don't insert the device into xenstore, otherwise pciback attempts to use it which conflicts with QEMU. This manifests itself such that the first time a device is passed to

Re: [Xen-devel] [PATCH] x86/mem_sharing: Relax sanity check for memops

2015-05-29 Thread Jan Beulich
On 29.05.15 at 11:37, tkleng...@sec.in.tum.de wrote: The sharing vm_event ring being enabled is not necessary for mem_sharing memops. If indeed so, why would the same not apply to mem_paging memops? Jan ___ Xen-devel mailing list

Re: [Xen-devel] [PATCH] libxl: Don't insert PCI device into xenstore for HVM guests

2015-05-29 Thread Ross Lagerwall
On 05/29/2015 10:50 AM, Wei Liu wrote: On Fri, May 29, 2015 at 10:43:08AM +0100, Ross Lagerwall wrote: On 05/29/2015 10:41 AM, Wei Liu wrote: On Fri, May 29, 2015 at 08:59:45AM +0100, Ross Lagerwall wrote: When doing passthrough of a PCI device for an HVM guest, don't insert the device into

Re: [Xen-devel] [PATCH v2 26/41] arm : acpi add xen environment table

2015-05-29 Thread Stefano Stabellini
On Thu, 28 May 2015, Jan Beulich wrote: On 28.05.15 at 14:12, stefano.stabell...@eu.citrix.com wrote: On Thu, 28 May 2015, Jan Beulich wrote: On 28.05.15 at 12:58, stefano.stabell...@eu.citrix.com wrote: Let's take a closer look at this table. After the boilierplate, these are the

Re: [Xen-devel] xen-netfront sets partial checksum at wrong offset

2015-05-29 Thread Wei Liu
On Fri, May 29, 2015 at 11:34:07AM +0100, Jan Beulich wrote: On 11.05.15 at 19:25, venkat.x.venkatsu...@oracle.com wrote: Please CC the maintainers of the driver. You can get that from 'scripts/get_maintainer.pl' I've done that for you. Thanks, Konrad. I am copying Wei too

[Xen-devel] [PATCH] osstest: reduce FreeBSD install timeouts

2015-05-29 Thread Roger Pau Monne
Only the first block is expected to take longer (because it decompresses the image and writes it to a LVM volume), the remaining commands should execute much faster, so reduce the timeout. Signed-off-by: Roger Pau Monné roger@citrix.com Cc: Ian Jackson ian.jack...@eu.citrix.com Cc: Ian

Re: [Xen-devel] [PATCH 0/4] Fix HVM vNUMA

2015-05-29 Thread Wei Liu
Ping... On Mon, May 18, 2015 at 04:34:48PM +0100, Wei Liu wrote: Boris discovered that HVM vNUMA didn't actually work. This patch series fixes that. The first patch is a prerequisite patch for the actual fixes. The second patch is to help debugging. The fixes are in the last two patches,

Re: [Xen-devel] Xen BUG at page_alloc.c:1738 (Xen 4.5)

2015-05-29 Thread M A Young
On Fri, 29 May 2015, Andrew Cooper wrote: Are you in a position to compile identical Xen 4.5 source with two different versions of gcc?  (current staging-4.5 staging even has the gcc5 build fix in) If it is a gcc compiler bug, we would expect the version compiled with gcc 4.9 to work fine,

Re: [Xen-devel] Xen BUG at page_alloc.c:1738 (Xen 4.5)

2015-05-29 Thread M A Young
On Fri, 29 May 2015, Andrew Cooper wrote: On 29/05/15 11:50, M A Young wrote: On Fri, 29 May 2015, Andrew Cooper wrote: Are you in a position to compile identical Xen 4.5 source with two different versions of gcc? (current staging-4.5 staging even has the gcc5 build fix in) If

Re: [Xen-devel] [PATCH v8 05/13] x86: expose CBM length and COS number information

2015-05-29 Thread Dario Faggioli
On Fri, 2015-05-29 at 09:07 +0100, Jan Beulich wrote: On 29.05.15 at 04:47, chao.p.p...@linux.intel.com wrote: On Thu, May 28, 2015 at 02:26:03PM +0100, Jan Beulich wrote: --- a/xen/include/public/sysctl.h +++ b/xen/include/public/sysctl.h @@ -694,6 +694,20 @@ struct

Re: [Xen-devel] [PATCH] libxl: Don't insert PCI device into xenstore for HVM guests

2015-05-29 Thread Wei Liu
On Fri, May 29, 2015 at 10:43:08AM +0100, Ross Lagerwall wrote: On 05/29/2015 10:41 AM, Wei Liu wrote: On Fri, May 29, 2015 at 08:59:45AM +0100, Ross Lagerwall wrote: When doing passthrough of a PCI device for an HVM guest, don't insert the device into xenstore, otherwise pciback attempts to

Re: [Xen-devel] xen-netfront sets partial checksum at wrong offset

2015-05-29 Thread Jan Beulich
On 11.05.15 at 19:25, venkat.x.venkatsu...@oracle.com wrote: Please CC the maintainers of the driver. You can get that from 'scripts/get_maintainer.pl' I've done that for you. Thanks, Konrad. I am copying Wei too who had fixed the below problem earlier. It fixed the incorrect

Re: [Xen-devel] [PATCH] x86/mem_sharing: Relax sanity check for memops

2015-05-29 Thread Tamas K Lengyel
On Fri, May 29, 2015 at 11:54 AM, Jan Beulich jbeul...@suse.com wrote: On 29.05.15 at 11:37, tkleng...@sec.in.tum.de wrote: The sharing vm_event ring being enabled is not necessary for mem_sharing memops. If indeed so, why would the same not apply to mem_paging memops? Jan The ring

Re: [Xen-devel] [PATCH v2 26/41] arm : acpi add xen environment table

2015-05-29 Thread Jan Beulich
On 29.05.15 at 12:31, stefano.stabell...@eu.citrix.com wrote: On Thu, 28 May 2015, Jan Beulich wrote: On 28.05.15 at 14:12, stefano.stabell...@eu.citrix.com wrote: Could you please make a concrete suggestion with table names and fields? I already pointed you at 6.0's new FADT field

Re: [Xen-devel] [PATCH] efi: Reallocate memory map if ExitBootServices() fails

2015-05-29 Thread Jan Beulich
On 29.05.15 at 11:57, ross.lagerw...@citrix.com wrote: On 05/29/2015 10:45 AM, Jan Beulich wrote: On 29.05.15 at 09:48, ross.lagerw...@citrix.com wrote: --- a/xen/common/efi/boot.c +++ b/xen/common/efi/boot.c @@ -1053,14 +1053,14 @@ efi_start(EFI_HANDLE ImageHandle, EFI_SYSTEM_TABLE

Re: [Xen-devel] [PATCH 0/4] Fix HVM vNUMA

2015-05-29 Thread Dario Faggioli
On Fri, 2015-05-29 at 11:46 +0100, Wei Liu wrote: Ping... FWIW, I had a look at the patches and they seemed fine, sorry to not having said that before. TBH, that was partly because I was expecting a v2 to show up, fixing the (trivial) issue you identified together with Boris, and was planning

Re: [Xen-devel] [RFC][PATCH] x86: remove vmalloc.h from asm/io.h

2015-05-29 Thread Stephen Rothwell
Hi Ingo, On Fri, 29 May 2015 11:21:05 +0200 Ingo Molnar mi...@kernel.org wrote: Good idea. Acked-by: Ingo Molnar mi...@kernel.org Thanks. Please also test x86 allnoconfig and defconfig 32/64, that tends to unearth the remaining places. People doing randconfig testing will find the

Re: [Xen-devel] [RFC][PATCH] x86: remove vmalloc.h from asm/io.h

2015-05-29 Thread Takashi Iwai
At Fri, 29 May 2015 19:18:47 +1000, Stephen Rothwell wrote: Nothing in asm/io.h uses anything from vmalloc.h, so remove the include and fix up the build problems in an allmodconfig (64 bit and 32 bit) build. This may be the place where x86 builds get vmalloc.h implicitly included and that

Re: [Xen-devel] [PATCH v2 1/4] libxc/libxl: fill xc_hvm_build_args in libxl

2015-05-29 Thread Wei Liu
On Fri, May 29, 2015 at 01:29:18PM +0100, Andrew Cooper wrote: On 29/05/15 12:37, Wei Liu wrote: When building HVM guests, originally some fields of xc_hvm_build_args are filled in xc_hvm_build (and buried in the wrong function), some are set in libxl__build_hvm before passing

Re: [Xen-devel] ACPI shutdown unreliable with win7?

2015-05-29 Thread Ian Campbell
On Fri, 2015-05-22 at 10:08 +0100, Ian Campbell wrote: If win7 doesn't shutdown given a power button request I'd be more inclined to remove the setting in osstest for those flights and let guest-stop go back to being never pass than to start making such changes to the VM config which I think

Re: [Xen-devel] [PATCH v2 for Xen 4.6 1/4] xen: enabling XL to set per-VCPU parameters of a domain for RTDS scheduler

2015-05-29 Thread Dario Faggioli
On Wed, 2015-05-27 at 12:41 +0100, Jan Beulich wrote: On 26.05.15 at 19:18, lichong...@gmail.com wrote: On Tue, May 26, 2015 at 4:08 AM, Jan Beulich jbeul...@suse.com wrote: On 26.05.15 at 02:05, lichong...@gmail.com wrote: Add two hypercalls(XEN_DOMCTL_SCHEDOP_getvcpuinfo/putvcpuinfo) to

Re: [Xen-devel] ACPI shutdown unreliable with win7?

2015-05-29 Thread Ian Campbell
On Fri, 2015-05-29 at 14:14 +0100, Andrew Cooper wrote: On 29/05/15 14:04, Jan Beulich wrote: On 29.05.15 at 14:54, ian.campb...@citrix.com wrote: On Fri, 2015-05-22 at 10:08 +0100, Ian Campbell wrote: If win7 doesn't shutdown given a power button request I'd be more inclined to remove

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

2015-05-29 Thread osstest service user
flight 57518 rumpuserxen real [real] http://logs.test-lab.xenproject.org/osstest/logs/57518/ 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

Re: [Xen-devel] ACPI shutdown unreliable with win7?

2015-05-29 Thread Jan Beulich
On 29.05.15 at 14:54, ian.campb...@citrix.com wrote: On Fri, 2015-05-22 at 10:08 +0100, Ian Campbell wrote: If win7 doesn't shutdown given a power button request I'd be more inclined to remove the setting in osstest for those flights and let guest-stop go back to being never pass than to

Re: [Xen-devel] ACPI shutdown unreliable with win7?

2015-05-29 Thread Ian Campbell
On Fri, 2015-05-29 at 14:04 +0100, Jan Beulich wrote: On 29.05.15 at 14:54, ian.campb...@citrix.com wrote: On Fri, 2015-05-22 at 10:08 +0100, Ian Campbell wrote: If win7 doesn't shutdown given a power button request I'd be more inclined to remove the setting in osstest for those flights

Re: [Xen-devel] [PATCH v2 for Xen 4.6 1/4] xen: enabling XL to set per-VCPU parameters of a domain for RTDS scheduler

2015-05-29 Thread Jan Beulich
On 29.05.15 at 15:01, dario.faggi...@citrix.com wrote: On Wed, 2015-05-27 at 12:41 +0100, Jan Beulich wrote: On 26.05.15 at 19:18, lichong...@gmail.com wrote: On Tue, May 26, 2015 at 4:08 AM, Jan Beulich jbeul...@suse.com wrote: On 26.05.15 at 02:05, lichong...@gmail.com wrote: + +

Re: [Xen-devel] ACPI shutdown unreliable with win7?

2015-05-29 Thread Jan Beulich
On 29.05.15 at 15:14, andrew.coop...@citrix.com wrote: On 29/05/15 14:04, Jan Beulich wrote: On 29.05.15 at 14:54, ian.campb...@citrix.com wrote: On Fri, 2015-05-22 at 10:08 +0100, Ian Campbell wrote: If win7 doesn't shutdown given a power button request I'd be more inclined to remove the

[Xen-devel] [PATCH v2] efi: Reallocate memory map if ExitBootServices() fails

2015-05-29 Thread Ross Lagerwall
If calling ExitBootServices() fails, the memory map size may increase. Allocate an extra page the first time around so that we hopefully shouldn't have to reallocate the memory map. Allocate a new memory map if needed, although this only a fallback, because some arches may use boot services to

Re: [Xen-devel] [osstest test] 57401: tolerable FAIL - PUSHED

2015-05-29 Thread Robert Hu
On Thu, 2015-05-28 at 11:26 +0100, Ian Campbell wrote: On Thu, 2015-05-28 at 04:49 +, osstest service user wrote: flight 57401 osstest real [real] http://logs.test-lab.xenproject.org/osstest/logs/57401/ Failures :-/ but no regressions. Lontao, Robert, The grub related bits of

Re: [Xen-devel] [PATCH v2 2/4] libxc: print more error messages when failed

2015-05-29 Thread Andrew Cooper
On 29/05/15 12:37, Wei Liu wrote: No functional changes introduced. Signed-off-by: Wei Liu wei.l...@citrix.com Cc: Ian Campbell ian.campb...@citrix.com Cc: Ian Jackson ian.jack...@eu.citrix.com This patch reminds me of a todo item I have wanted to do for a while using setvcpucontext rather

Re: [Xen-devel] ACPI shutdown unreliable with win7?

2015-05-29 Thread Andrew Cooper
On 29/05/15 14:04, Jan Beulich wrote: On 29.05.15 at 14:54, ian.campb...@citrix.com wrote: On Fri, 2015-05-22 at 10:08 +0100, Ian Campbell wrote: If win7 doesn't shutdown given a power button request I'd be more inclined to remove the setting in osstest for those flights and let guest-stop go

Re: [Xen-devel] [Qemu-devel] [PATCH 1/1] Fix device listener interface for PCI to PCI bridges

2015-05-29 Thread Don Slutz
On 05/28/15 17:05, Michael S. Tsirkin wrote: On Thu, May 28, 2015 at 11:03:07PM +0200, Michael S. Tsirkin wrote: On Thu, May 28, 2015 at 03:09:48PM -0400, Don Slutz wrote: On 05/28/15 08:28, Michael S. Tsirkin wrote: On Thu, May 28, 2015 at 07:25:50AM -0400, Don Slutz wrote: On 05/28/15

Re: [Xen-devel] [Draft C] Xen on ARM vITS Handling

2015-05-29 Thread Julien Grall
Hi Ian, NIT: You used my Linaro email which I think is de-activated now :). On 27/05/2015 13:48, Ian Campbell wrote: Here follows draft C based on previous feedback. Also at: http://xenbits.xen.org/people/ianc/vits/draftC.{pdf,html} I think I've captured most of the previous discussion,

[Xen-devel] [PATCH V8] xen/vm_event: Clean up control-register-write vm_events and add XCR0 event

2015-05-29 Thread Razvan Cojocaru
As suggested by Andrew Cooper, this patch attempts to remove some redundancy and allow for an easier time when adding vm_events for new control registers in the future, by having a single VM_EVENT_REASON_WRITE_CTRLREG vm_event type, meant to serve CR0, CR3, CR4 and (newly introduced) XCR0. The

Re: [Xen-devel] [PATCH v2] efi: Reallocate memory map if ExitBootServices() fails

2015-05-29 Thread Jan Beulich
On 29.05.15 at 15:30, ross.lagerw...@citrix.com wrote: @@ -1053,14 +1053,23 @@ efi_start(EFI_HANDLE ImageHandle, EFI_SYSTEM_TABLE *SystemTable) efi_arch_video_init(gop, info_size, mode_info); } -efi_bs-GetMemoryMap(efi_memmap_size, NULL, map_key, -

Re: [Xen-devel] [PATCH v2 for Xen 4.6 1/4] xen: enabling XL to set per-VCPU parameters of a domain for RTDS scheduler

2015-05-29 Thread Dario Faggioli
On Mon, 2015-05-25 at 19:05 -0500, Chong Li wrote: diff --git a/xen/common/domctl.c b/xen/common/domctl.c index 28aea55..8143c44 100644 --- a/xen/common/domctl.c +++ b/xen/common/domctl.c @@ -841,6 +841,11 @@ long do_domctl(XEN_GUEST_HANDLE_PARAM(xen_domctl_t) u_domctl) copyback

Re: [Xen-devel] [PATCH V7] xen/vm_event: Clean up control-register-write vm_events and add XCR0 event

2015-05-29 Thread Razvan Cojocaru
On 05/29/2015 12:12 PM, Jan Beulich wrote: On 28.05.15 at 18:32, rcojoc...@bitdefender.com wrote: the macro will probably go out the window (or the first parameter will need to be changed to VM_EVENT_##what instead of VM_EVENT_X86_##what) as soon as ARM control register write events will come

Re: [Xen-devel] ACPI shutdown unreliable with win7?

2015-05-29 Thread Ian Campbell
On Fri, 2015-05-29 at 14:28 +0100, Jan Beulich wrote: On 29.05.15 at 15:14, andrew.coop...@citrix.com wrote: On 29/05/15 14:04, Jan Beulich wrote: On 29.05.15 at 14:54, ian.campb...@citrix.com wrote: On Fri, 2015-05-22 at 10:08 +0100, Ian Campbell wrote: If win7 doesn't shutdown given a

[Xen-devel] [PATCH v3] run QEMU as non-root

2015-05-29 Thread Stefano Stabellini
Try to use xen-qemudepriv-$domname first, then xen-qemudepriv-base + domid, finally xen-qemudepriv-shared and root if everything else fails. The uids need to be manually created by the user or, more likely, by the xen package maintainer. To actually secure QEMU when running in Dom0, we need at

Re: [Xen-devel] [Draft C] Xen on ARM vITS Handling

2015-05-29 Thread Julien Grall
Hi Vijay, On 27/05/15 17:44, Vijay Kilari wrote: ## Command Translation Of the existing GICv3 ITS commands, `MAPC`, `MAPD`, `MAPVI`/`MAPI` are potentially time consuming commands as these commands creates entry in the Xen ITS structures, which are used to validate other ITS commands.

Re: [Xen-devel] [PATCH V4 7/8] xen/arm64: increase MAX_VIRT_CPUS to 128 on arm64

2015-05-29 Thread Julien Grall
Hi Chen, On 28/05/15 11:15, Chen Baozi wrote: From: Chen Baozi baoz...@gmail.com GIC-500 supports up to 128 cores in a single SoC. Increase MAX_VIRT_CPUS to 128 on arm64. This should be the last patch. You still have to modify domain_max_vcpus before being able to increase the number of

Re: [Xen-devel] ACPI shutdown unreliable with win7?

2015-05-29 Thread Ian Campbell
On Fri, 2015-05-29 at 15:31 +0100, Andrew Cooper wrote: On 29/05/15 15:00, Ian Campbell wrote: On Fri, 2015-05-29 at 14:28 +0100, Jan Beulich wrote: On 29.05.15 at 15:14, andrew.coop...@citrix.com wrote: On 29/05/15 14:04, Jan Beulich wrote: On 29.05.15 at 14:54, ian.campb...@citrix.com

Re: [Xen-devel] [PATCH v7 05/10] xsm: add XEN_DOMCTL_soft_reset support

2015-05-29 Thread Jan Beulich
On 27.05.15 at 17:25, vkuzn...@redhat.com wrote: +static XSM_INLINE int xsm_soft_reset(XSM_DEFAULT_ARG struct domain *d1, + struct domain *d2) Hard tabs used here ... +static inline int xsm_soft_reset (xsm_default_t def, struct domain *d1, +

Re: [Xen-devel] [PATCH v2 for Xen 4.6 0/4] Enabling XL to set per-VCPU parameters of a domain for RTDS scheduler

2015-05-29 Thread Meng Xu
2015-05-29 4:15 GMT-07:00 Dario Faggioli dario.faggi...@citrix.com: On Tue, 2015-05-26 at 09:59 +0100, Ian Campbell wrote: On Mon, 2015-05-25 at 18:59 -0500, Chong Li wrote: This series arrived in my mailbox as 5 distinct mails. Please use git send-email such that the mails arrive as a

Re: [Xen-devel] [PATCH v3] run QEMU as non-root

2015-05-29 Thread Ian Campbell
On Fri, 2015-05-29 at 14:47 +0100, Stefano Stabellini wrote: Try to use xen-qemudepriv-$domname first, then xen-qemudepriv-base + domid, finally xen-qemudepriv-shared and root if everything else fails. The uids need to be manually created by the user or, more likely, by the xen package

Re: [Xen-devel] ACPI shutdown unreliable with win7?

2015-05-29 Thread Paul Durrant
-Original Message- From: Paul Durrant Sent: 29 May 2015 16:07 To: Ian Campbell Cc: Jan Beulich; Andrew Cooper; Ian Jackson; xen-devel Subject: RE: [Xen-devel] ACPI shutdown unreliable with win7? -Original Message- From: Ian Campbell [mailto:ian.campb...@citrix.com]

Re: [Xen-devel] ACPI shutdown unreliable with win7?

2015-05-29 Thread Paul Durrant
-Original Message- From: Ross Philipson [mailto:ross.philip...@gmail.com] Sent: 29 May 2015 16:35 To: Ian Campbell; Paul Durrant Cc: Andrew Cooper; xen-devel; Jan Beulich; Ian Jackson Subject: Re: [Xen-devel] ACPI shutdown unreliable with win7? On 05/29/2015 11:11 AM, Ian Campbell

Re: [Xen-devel] ACPI shutdown unreliable with win7?

2015-05-29 Thread Ross Philipson
On 05/29/2015 12:00 PM, Paul Durrant wrote: -Original Message- From: Ross Philipson [mailto:ross.philip...@gmail.com] Sent: 29 May 2015 16:35 To: Ian Campbell; Paul Durrant Cc: Andrew Cooper; xen-devel; Jan Beulich; Ian Jackson Subject: Re: [Xen-devel] ACPI shutdown unreliable with win7?

Re: [Xen-devel] [PATCH V4 7/8] xen/arm64: increase MAX_VIRT_CPUS to 128 on arm64

2015-05-29 Thread Andrew Cooper
On 29/05/15 17:45, Julien Grall wrote: On 29/05/15 17:41, Andrew Cooper wrote: On 29/05/15 17:20, Julien Grall wrote: On 29/05/15 16:51, Julien Grall wrote: diff --git a/xen/include/asm-arm/config.h b/xen/include/asm-arm/config.h index 3b23e05..817c216 100644 ---

Re: [Xen-devel] [PATCH V4 5/8] tools/libxl: Set 'reg' of cpu node equal to MPIDR affinity for domU

2015-05-29 Thread Julien Grall
On 29/05/15 16:55, Ian Campbell wrote: On Fri, 2015-05-29 at 16:44 +0100, Julien Grall wrote: +name = GCSPRINTF(cpu@%lx, mpidr_aff); It's not necessary to change the cpu@. AIUI it is conventional in DT for this to match the first reg entry. Well, it's conventional when the reg

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

2015-05-29 Thread Ian Campbell
On Wed, 2015-05-27 at 17:04 +0100, Ian Campbell wrote: Looking at the netback side though it seems like netback_remove is switching to state=Closed _before_ it calls kobject_uevent(..., KOBJ_OFFLINE) and it is this which generates the call to netback_uevent which tries and fails to read script

Re: [Xen-devel] ACPI shutdown unreliable with win7?

2015-05-29 Thread Ross Philipson
On 05/29/2015 11:11 AM, Ian Campbell wrote: On Fri, 2015-05-29 at 16:06 +0100, Paul Durrant wrote: FWIW We appear to wait 200s, if we were seeing failures due to windows update then I'd be inclined to extend that, but I think right now that would be premature, unless WU happens with no status

[Xen-devel] [PATCH] xen: netback: fix error printf format string.

2015-05-29 Thread Ian Campbell
drivers/net/xen-netback/netback.c: In function ‘xenvif_tx_build_gops’: drivers/net/xen-netback/netback.c:1253:8: warning: format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 5 has type ‘int’ [-Wformat=] (txreq.offset~PAGE_MASK) + txreq.size); ^ txreq.offset

  1   2   >