Re: [Xen-devel] [v8][PATCH 02/17] introduce XEN_DOMCTL_set_rdm

2014-12-02 Thread Tian, Kevin
From: Chen, Tiejun Sent: Monday, December 01, 2014 5:24 PM This should be based on a new parameter globally, 'pci_rdmforce'. pci_rdmforce = 1 = Of course this should be 0 by default. '1' means we should force check to reserve all ranges. If failed VM wouldn't be created successfully.

Re: [Xen-devel] [v8][PATCH 04/17] update the existing hypercall to support XEN_DOMCTL_set_rdm

2014-12-02 Thread Tian, Kevin
From: Chen, Tiejun Sent: Monday, December 01, 2014 5:24 PM After we intend to expost that hypercall explicitly based on XEN_DOMCTL_set_rdm, we need this rebase. I hope we can squash this into that previous patch once Jan Ack this. better to merge together, since it's the right thing to do

Re: [Xen-devel] [v8][PATCH 06/17] tools/libxc: check if modules space is overlapping with reserved device memory

2014-12-02 Thread Tian, Kevin
From: Chen, Tiejun Sent: Monday, December 01, 2014 5:24 PM In case of reserved device memory overlapping with ram, it also probably overlap with modules space so we need to check these reserved device memory as well. Signed-off-by: Tiejun Chen tiejun.c...@intel.com Reviewed-by: Kevin

Re: [Xen-devel] [PATCH v4] x86: add p2m_mmio_write_dm

2014-12-02 Thread Jan Beulich
On 02.12.14 at 08:48, yu.c.zh...@linux.intel.com wrote: On 12/1/2014 8:31 PM, Jan Beulich wrote: On 01.12.14 at 13:13, t...@xen.org wrote: At 11:17 + on 01 Dec (1417429027), Jan Beulich wrote: On 01.12.14 at 11:30, t...@xen.org wrote: At 09:32 + on 01 Dec (1417422746), Jan Beulich

[Xen-devel] PVHVM drivers in upstream linux kernel

2014-12-02 Thread Juergen Gross
Hi, looking into the upstream linux sources I realized that the PVHVM drivers of XEN are only available with the pvops kernel. Is this on purpose? Shouldn't the frontend drivers, xen/platform-pci.c etc. be configurable without having to enable CONFIG_PARAVIRT? Juergen

Re: [Xen-devel] [v8][PATCH 09/17] hvmloader/ram: check if guest memory is out of reserved device memory maps

2014-12-02 Thread Tian, Kevin
From: Chen, Tiejun Sent: Monday, December 01, 2014 5:24 PM We need to check to reserve all reserved device memory maps in e820 to avoid any potential guest memory conflict. Currently, if we can't insert RDM entries directly, we may need to handle several ranges as follows: a. Fixed

Re: [Xen-devel] [v8][PATCH 10/17] hvmloader/mem_hole_alloc: skip any overlap with reserved device memory

2014-12-02 Thread Tian, Kevin
From: Chen, Tiejun Sent: Monday, December 01, 2014 5:24 PM In some cases like igd_opregion_pgbase, guest will use mem_hole_alloc to allocate some memory to use in runtime cycle, so we alsoe need to make sure all reserved device memory don't overlap such a region. OK, seems you meant to use

Re: [Xen-devel] [PATCH for-4.5] call xc_domain_irq_permission before xc_domain_irq_permission

2014-12-02 Thread Jan Beulich
On 01.12.14 at 20:22, stefano.stabell...@eu.citrix.com wrote: xc_physdev_unmap_pirq might revoke the permission to map the irq from the domain causing the following xc_domain_irq_permission call to fail and return error (domain_pirq_to_irq returns 0). Apart from the patch title being rather

Re: [Xen-devel] [v8][PATCH 11/17] xen/x86/p2m: reject populating for reserved device memory mapping

2014-12-02 Thread Tian, Kevin
From: Chen, Tiejun Sent: Monday, December 01, 2014 5:24 PM We need to reject to populate reserved device memory mapping, and then make sure all reserved device memory can't be accessed by any !iommu approach. Signed-off-by: Tiejun Chen tiejun.c...@intel.com --- xen/arch/x86/mm/p2m.c

Re: [Xen-devel] [v8][PATCH 12/17] xen/x86/ept: handle reserved device memory in ept_handle_violation

2014-12-02 Thread Tian, Kevin
From: Chen, Tiejun Sent: Monday, December 01, 2014 5:25 PM We always reserve these ranges since we never allow any stuff to poke them. But in theory some untrusted VM can maliciously access them. So we need to intercept this approach. But we just don't want to leak anything or introduce

Re: [Xen-devel] [v8][PATCH 14/17] xen/x86/p2m: introduce set_identity_p2m_entry

2014-12-02 Thread Tian, Kevin
From: Chen, Tiejun Sent: Monday, December 01, 2014 5:25 PM We will create RMRR mapping as follows: If gfn space unoccupied, we just set that. If space already occupy by 1:1 RMRR mapping do thing. Others should be failed. Signed-off-by: Tiejun Chen tiejun.c...@intel.com Reviewed-by:

Re: [Xen-devel] [v8][PATCH 15/17] xen:vtd: create RMRR mapping

2014-12-02 Thread Tian, Kevin
From: Chen, Tiejun Sent: Monday, December 01, 2014 5:25 PM intel_iommu_map_page() does nothing if VT-d shares EPT page table. So rmrr_identity_mapping() never create RMRR mapping but in some cases like some GFX drivers it still need to access RMRR. Here we will create those RMRR mappings

[Xen-devel] [PATCH 1/2] gnttab: Introduce rwlock to protect updates to grant table state

2014-12-02 Thread Christoph Egger
Rename lock to maptrack_lock and use it to protect maptrack state. The new rwlock is used to prevent readers from accessing inconsistent grant table state such as current version, partially initialized active table pages, etc. Signed-off-by: Matt Wilson m...@amazon.com [chegger: ported to

[Xen-devel] [PATCH 0/2] gnttab: Improve scaleability

2014-12-02 Thread Christoph Egger
This patch series changes the grant table locking to a more fain grained locking protocol. The result is a performance boost measured with blkfront/blkback. Document the locking protocol. [PATCH 1/2] gnttab: Introduce rwlock to protect updates to grant [PATCH 2/2] gnttab: refactor locking for

[Xen-devel] [PATCH 2/2] gnttab: refactor locking for scalability

2014-12-02 Thread Christoph Egger
From: Matt Wilson m...@amazon.com This patch refactors grant table locking. It splits the previous single spinlock per grant table into multiple locks. The heavily modified components of the grant table (the maptrack state and the active entries) are now protected by their own spinlocks. The

Re: [Xen-devel] [PATCH v4] Fixes for PCI backend for 3.19

2014-12-02 Thread Jan Beulich
On 21.11.14 at 23:17, konrad.w...@oracle.com wrote: Konrad Rzeszutek Wilk (7): xen/pciback: Don't deadlock when unbinding. driver core: Provide an wrapper around the mutex to do lockdep warnings xen/pciback: Include the domain id if removing the device whilst still in use

Re: [Xen-devel] [PATCH v4] x86: add p2m_mmio_write_dm

2014-12-02 Thread Tim Deegan
At 12:31 + on 01 Dec (1417433464), Jan Beulich wrote: On 01.12.14 at 13:13, t...@xen.org wrote: At 11:17 + on 01 Dec (1417429027), Jan Beulich wrote: On 01.12.14 at 11:30, t...@xen.org wrote: At 09:32 + on 01 Dec (1417422746), Jan Beulich wrote: On 01.12.14 at 09:49,

Re: [Xen-devel] [PATCH for-4.5] call xc_domain_irq_permission before xc_domain_irq_permission

2014-12-02 Thread Stefano Stabellini
On Tue, 2 Dec 2014, Jan Beulich wrote: On 01.12.14 at 20:22, stefano.stabell...@eu.citrix.com wrote: xc_physdev_unmap_pirq might revoke the permission to map the irq from the domain causing the following xc_domain_irq_permission call to fail and return error (domain_pirq_to_irq returns 0).

Re: [Xen-devel] [PATCH 0/2] gnttab: Improve scaleability

2014-12-02 Thread Andrew Cooper
On 02/12/14 10:06, Christoph Egger wrote: This patch series changes the grant table locking to a more fain grained locking protocol. The result is a performance boost measured with blkfront/blkback. Document the locking protocol. [PATCH 1/2] gnttab: Introduce rwlock to protect updates to

Re: [Xen-devel] Poor network performance between DomU with multiqueue support

2014-12-02 Thread Wei Liu
On Tue, Dec 02, 2014 at 04:30:49PM +0800, zhangleiqiang wrote: Hi, all I am testing the performance of xen netfront-netback driver that with multi-queues support. The throughput from domU to remote dom0 is 9.2Gb/s, but the throughput from domU to remote domU is only 3.6Gb/s, I think the

Re: [Xen-devel] PVHVM drivers in upstream linux kernel

2014-12-02 Thread Juergen Gross
On 12/02/2014 11:54 AM, David Vrabel wrote: On 02/12/14 09:39, Juergen Gross wrote: Hi, looking into the upstream linux sources I realized that the PVHVM drivers of XEN are only available with the pvops kernel. Is this on purpose? Shouldn't the frontend drivers, xen/platform-pci.c etc. be

Re: [Xen-devel] [PATCH v4] x86: add p2m_mmio_write_dm

2014-12-02 Thread Jan Beulich
On 02.12.14 at 11:37, t...@xen.org wrote: At 12:31 + on 01 Dec (1417433464), Jan Beulich wrote: On 01.12.14 at 13:13, t...@xen.org wrote: At 11:17 + on 01 Dec (1417429027), Jan Beulich wrote: On 01.12.14 at 11:30, t...@xen.org wrote: At 09:32 + on 01 Dec (1417422746), Jan

Re: [Xen-devel] PVHVM drivers in upstream linux kernel

2014-12-02 Thread Ian Campbell
On Tue, 2014-12-02 at 10:54 +, David Vrabel wrote: On 02/12/14 09:39, Juergen Gross wrote: Hi, looking into the upstream linux sources I realized that the PVHVM drivers of XEN are only available with the pvops kernel. Is this on purpose? Shouldn't the frontend drivers,

Re: [Xen-devel] [PATCH 0/2] gnttab: Improve scaleability

2014-12-02 Thread Egger, Christoph
On 2014/12/02 12:00, Andrew Cooper wrote: On 02/12/14 10:06, Christoph Egger wrote: This patch series changes the grant table locking to a more fain grained locking protocol. The result is a performance boost measured with blkfront/blkback. Document the locking protocol. [PATCH 1/2] gnttab:

Re: [Xen-devel] [PATCH for-4.5] call xc_domain_irq_permission before xc_domain_irq_permission

2014-12-02 Thread Jan Beulich
On 02.12.14 at 11:55, stefano.stabell...@eu.citrix.com wrote: On Tue, 2 Dec 2014, Jan Beulich wrote: On 01.12.14 at 20:22, stefano.stabell...@eu.citrix.com wrote: xc_physdev_unmap_pirq might revoke the permission to map the irq from the domain causing the following xc_domain_irq_permission

Re: [Xen-devel] [PATCH 1/2] gnttab: Introduce rwlock to protect updates to grant table state

2014-12-02 Thread Andrew Cooper
On 02/12/14 10:06, Christoph Egger wrote: Rename lock to maptrack_lock and use it to protect maptrack state. The new rwlock is used to prevent readers from accessing inconsistent grant table state such as current version, partially initialized active table pages, etc. I would suggest

Re: [Xen-devel] PVHVM drivers in upstream linux kernel

2014-12-02 Thread Juergen Gross
On 12/02/2014 12:05 PM, Ian Campbell wrote: On Tue, 2014-12-02 at 10:54 +, David Vrabel wrote: On 02/12/14 09:39, Juergen Gross wrote: Hi, looking into the upstream linux sources I realized that the PVHVM drivers of XEN are only available with the pvops kernel. Is this on purpose?

Re: [Xen-devel] PVHVM drivers in upstream linux kernel

2014-12-02 Thread Ian Campbell
On Tue, 2014-12-02 at 12:33 +0100, Juergen Gross wrote: On 12/02/2014 12:05 PM, Ian Campbell wrote: On Tue, 2014-12-02 at 10:54 +, David Vrabel wrote: On 02/12/14 09:39, Juergen Gross wrote: Hi, looking into the upstream linux sources I realized that the PVHVM drivers of XEN are

Re: [Xen-devel] PVHVM drivers in upstream linux kernel

2014-12-02 Thread Juergen Gross
On 12/02/2014 12:36 PM, Ian Campbell wrote: On Tue, 2014-12-02 at 12:33 +0100, Juergen Gross wrote: On 12/02/2014 12:05 PM, Ian Campbell wrote: On Tue, 2014-12-02 at 10:54 +, David Vrabel wrote: On 02/12/14 09:39, Juergen Gross wrote: Hi, looking into the upstream linux sources I

Re: [Xen-devel] [PATCH v4] x86: add p2m_mmio_write_dm

2014-12-02 Thread Tim Deegan
At 15:38 +0800 on 02 Dec (1417531126), Yu, Zhang wrote: On 12/1/2014 8:13 PM, Tim Deegan wrote: At 11:17 + on 01 Dec (1417429027), Jan Beulich wrote: On 01.12.14 at 11:30, t...@xen.org wrote: During this bit of archaeology I realised that either this new type should _not_ be made part

Re: [Xen-devel] [PATCH] libxl_set_memory_target: retain the same maxmem offset on top of the current target

2014-12-02 Thread Stefano Stabellini
On Mon, 1 Dec 2014, Don Slutz wrote: On 11/25/14 12:54, Stefano Stabellini wrote: In libxl_set_memory_target when setting the new maxmem, retain the same offset on top of the current target. The offset includes memory allocated by QEMU for rom files. Signed-off-by: Stefano Stabellini

Re: [Xen-devel] [PATCH v4] x86: add p2m_mmio_write_dm

2014-12-02 Thread Jan Beulich
On 02.12.14 at 12:40, t...@xen.org wrote: At 15:38 +0800 on 02 Dec (1417531126), Yu, Zhang wrote: 2 p2m_grant_map_ro is also supposed to be discarded? Will handling of this type of pages goes into __hvm_copy()/__hvm_clear(), or should? I think so, yes. At the moment we inject #GP when the

Re: [Xen-devel] Poor network performance between DomU with multiqueue support

2014-12-02 Thread Zhangleiqiang (Trump)
-Original Message- From: xen-devel-boun...@lists.xen.org [mailto:xen-devel-boun...@lists.xen.org] On Behalf Of Wei Liu Sent: Tuesday, December 02, 2014 7:02 PM To: zhangleiqiang Cc: wei.l...@citrix.com; xen-devel@lists.xen.org Subject: Re: [Xen-devel] Poor network performance

Re: [Xen-devel] Poor network performance between DomU with multiqueue support

2014-12-02 Thread Zhangleiqiang (Trump)
-Original Message- From: xen-devel-boun...@lists.xen.org [mailto:xen-devel-boun...@lists.xen.org] On Behalf Of David Vrabel Sent: Tuesday, December 02, 2014 6:57 PM To: zhangleiqiang; xen-devel@lists.xen.org Subject: Re: [Xen-devel] Poor network performance between DomU with

[Xen-devel] [PATCH v2 for-4.6] libxl_set_memory_target: retain the same maxmem offset on top of the current target

2014-12-02 Thread Stefano Stabellini
In libxl_set_memory_target when setting the new maxmem, retain the same offset on top of the current target. The offset includes memory allocated by QEMU for rom files. Signed-off-by: Stefano Stabellini stefano.stabell...@eu.citrix.com --- Changes in v2: - call libxl_domain_info instead of

Re: [Xen-devel] Poor network performance between DomU with multiqueue support

2014-12-02 Thread Wei Liu
On Tue, Dec 02, 2014 at 11:50:59AM +, Zhangleiqiang (Trump) wrote: -Original Message- From: xen-devel-boun...@lists.xen.org [mailto:xen-devel-boun...@lists.xen.org] On Behalf Of Wei Liu Sent: Tuesday, December 02, 2014 7:02 PM To: zhangleiqiang Cc: wei.l...@citrix.com;

[Xen-devel] [qemu-mainline test] 31983: regressions - FAIL

2014-12-02 Thread xen . org
flight 31983 qemu-mainline real [real] http://www.chiark.greenend.org.uk/~xensrcts/logs/31983/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-qemut-debianhvm-amd64 5 xen-boot fail REGR. vs. 31947 Regressions which

Re: [Xen-devel] [Qemu-devel] [PATCH] increase maxmem before calling xc_domain_populate_physmap

2014-12-02 Thread Stefano Stabellini
On Mon, 1 Dec 2014, Don Slutz wrote: On 12/01/14 10:37, Stefano Stabellini wrote: On Mon, 1 Dec 2014, Don Slutz wrote: On 11/27/14 05:48, Stefano Stabellini wrote: [...] Works fine in both claim modes and with PoD used (maxmem memory). Do not know how to test with tmem. I

Re: [Xen-devel] [PATCH 1/2] gnttab: Introduce rwlock to protect updates to grant table state

2014-12-02 Thread Julien Grall
Hi Christoph, On 02/12/14 10:06, Christoph Egger wrote: diff --git a/xen/arch/x86/mm.c b/xen/arch/x86/mm.c index 522c43d..37c13b1 100644 --- a/xen/arch/x86/mm.c +++ b/xen/arch/x86/mm.c @@ -4565,7 +4565,7 @@ int xenmem_add_to_physmap_one( mfn = virt_to_mfn(d-shared_info);

Re: [Xen-devel] PVHVM drivers in upstream linux kernel

2014-12-02 Thread Juergen Gross
On 12/02/2014 12:59 PM, David Vrabel wrote: On 02/12/14 11:33, Juergen Gross wrote: I think we have customers wanting to run a default kernel as domU. So it isn't always the distro refusing paravirt, it might be the user... I don't think this is a sensible use case but I'm not adverse to

Re: [Xen-devel] [PATCH 1/2] gnttab: Introduce rwlock to protect updates to grant table state

2014-12-02 Thread Egger, Christoph
On 2014/12/02 13:35, Julien Grall wrote: Hi Christoph, On 02/12/14 10:06, Christoph Egger wrote: diff --git a/xen/arch/x86/mm.c b/xen/arch/x86/mm.c index 522c43d..37c13b1 100644 --- a/xen/arch/x86/mm.c +++ b/xen/arch/x86/mm.c @@ -4565,7 +4565,7 @@ int xenmem_add_to_physmap_one(

Re: [Xen-devel] [PATCH 1/2] gnttab: Introduce rwlock to protect updates to grant table state

2014-12-02 Thread Julien Grall
Hi, On 02/12/14 13:03, Egger, Christoph wrote: On 2014/12/02 13:35, Julien Grall wrote: Hi Christoph, On 02/12/14 10:06, Christoph Egger wrote: diff --git a/xen/arch/x86/mm.c b/xen/arch/x86/mm.c index 522c43d..37c13b1 100644 --- a/xen/arch/x86/mm.c +++ b/xen/arch/x86/mm.c @@ -4565,7

Re: [Xen-devel] [PATCH 1/2] gnttab: Introduce rwlock to protect updates to grant table state

2014-12-02 Thread Egger, Christoph
On 2014/12/02 14:08, Julien Grall wrote: Hi, On 02/12/14 13:03, Egger, Christoph wrote: On 2014/12/02 13:35, Julien Grall wrote: Hi Christoph, On 02/12/14 10:06, Christoph Egger wrote: diff --git a/xen/arch/x86/mm.c b/xen/arch/x86/mm.c index 522c43d..37c13b1 100644 ---

Re: [Xen-devel] [PATCH] libxl: Fix building libxlu_cfg_y.y with bison 3.0

2014-12-02 Thread Ian Campbell
On Mon, 2014-12-01 at 12:19 +, Wei Liu wrote: On Mon, Dec 01, 2014 at 09:42:13AM +, Ian Campbell wrote: On Sat, 2014-11-29 at 21:23 -0800, Ed Swierk wrote: - Use %lex-param instead of obsolete YYLEX_PARAM to override lex scanner parameter - Change deprecated %name-prefix= to

Re: [Xen-devel] [PATCH] xen/arm: Handle platforms with edge-triggered virtual timer

2014-12-02 Thread Ian Campbell
On Fri, 2014-11-28 at 15:17 +, Julien Grall wrote: Some platforms (such as Xgene and ARMv8 models) use an edge-triggered interrupt for the virtual timer. Even if the timer output signal is masked in the context switch, the GIC will keep track that of any interrupts raised while IRQs are

Re: [Xen-devel] [PATCH] libxl: Fix building libxlu_cfg_y.y with bison 3.0

2014-12-02 Thread Andrew Cooper
On 02/12/14 13:47, Ian Campbell wrote: On Mon, 2014-12-01 at 12:19 +, Wei Liu wrote: On Mon, Dec 01, 2014 at 09:42:13AM +, Ian Campbell wrote: On Sat, 2014-11-29 at 21:23 -0800, Ed Swierk wrote: - Use %lex-param instead of obsolete YYLEX_PARAM to override lex scanner parameter -

Re: [Xen-devel] [PATCH] xen/arm: Handle platforms with edge-triggered virtual timer

2014-12-02 Thread Julien Grall
Hi Ian, On 02/12/14 13:54, Ian Campbell wrote: On Fri, 2014-11-28 at 15:17 +, Julien Grall wrote: Some platforms (such as Xgene and ARMv8 models) use an edge-triggered interrupt for the virtual timer. Even if the timer output signal is masked in the context switch, the GIC will keep

Re: [Xen-devel] PVHVM drivers in upstream linux kernel

2014-12-02 Thread David Vrabel
On 02/12/14 13:00, Juergen Gross wrote: I'd see: - XEN_PV (selects PARAVIRT, XEN_FRONTEND): be able to run as pv-domain (x86 only) Depends on PARAVIRT perhaps? - XEN_PVHVM (selects XEN_FRONTEND): be able to run as hvm-domain with pv-drivers - XEN_BACKEND (selects PARAVIRT if x86): be

Re: [Xen-devel] [PATCH v2 for-4.5 2/2] xl: fix two memory leaks

2014-12-02 Thread Ian Campbell
On Mon, 2014-12-01 at 11:31 +, Wei Liu wrote: Free strings returned by libxl_basename after used. 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/libxl/xl_cmdimpl.c |8 ++-- 1 file

Re: [Xen-devel] [PATCH v2 for-4.5 2/2] xl: fix two memory leaks

2014-12-02 Thread Wei Liu
On Tue, Dec 02, 2014 at 02:19:49PM +, Ian Campbell wrote: On Mon, 2014-12-01 at 11:31 +, Wei Liu wrote: Free strings returned by libxl_basename after used. Signed-off-by: Wei Liu wei.l...@citrix.com Cc: Ian Campbell ian.campb...@citrix.com Cc: Ian Jackson

Re: [Xen-devel] [PATCH v2 for-4.6] libxl_set_memory_target: retain the same maxmem offset on top of the current target

2014-12-02 Thread Stefano Stabellini
On Tue, 2 Dec 2014, Don Slutz wrote: On 12/02/14 06:53, Stefano Stabellini wrote: In libxl_set_memory_target when setting the new maxmem, retain the same offset on top of the current target. The offset includes memory allocated by QEMU for rom files. Signed-off-by: Stefano

Re: [Xen-devel] [PATCH] xen/arm: Handle platforms with edge-triggered virtual timer

2014-12-02 Thread Julien Grall
On 02/12/14 14:21, Ian Campbell wrote: On Tue, 2014-12-02 at 14:08 +, Julien Grall wrote: Hi Ian, On 02/12/14 13:54, Ian Campbell wrote: On Fri, 2014-11-28 at 15:17 +, Julien Grall wrote: Some platforms (such as Xgene and ARMv8 models) use an edge-triggered interrupt for the

Re: [Xen-devel] [PATCH] xen/arm: Handle platforms with edge-triggered virtual timer

2014-12-02 Thread Ian Campbell
On Tue, 2014-12-02 at 14:32 +, Julien Grall wrote: On 02/12/14 14:21, Ian Campbell wrote: On Tue, 2014-12-02 at 14:08 +, Julien Grall wrote: Hi Ian, On 02/12/14 13:54, Ian Campbell wrote: On Fri, 2014-11-28 at 15:17 +, Julien Grall wrote: Some platforms (such as Xgene and

Re: [Xen-devel] [v8][PATCH 13/17] xen/mem_access: don't allow accessing reserved device memory

2014-12-02 Thread Julien Grall
Hi, CC Tamas as he did some work on memaccess for ARM. On 01/12/14 09:24, Tiejun Chen wrote: We can't expost those reserved device memory in case of mem_access s/expost/expose/ since any access may corrupt device usage. Signed-off-by: Tiejun Chen tiejun.c...@intel.com ---

[Xen-devel] [PATCH v2 1/2] gnttab: Introduce rwlock to protect updates to grant table state

2014-12-02 Thread Christoph Egger
Split grant table lock into two separate locks. One to protect maptrack state and change the other into a rwlock. The rwlock is used to prevent readers from accessing inconsistent grant table state such as current version, partially initialized active table pages, etc. Signed-off-by: Matt Wilson

Re: [Xen-devel] [PATCH for-xen-4.5] console: increase initial conring size

2014-12-02 Thread Andrew Cooper
On 02/12/14 14:57, Daniel Kiper wrote: In general initial conring size is sufficient. However, if log level is increased on platforms which have e.g. huge number of memory regions (I have an IBM System x3550 M2 with 8 GiB RAM which has more than 200 entries in EFI memory map) then some of

Re: [Xen-devel] [PATCH v2 for-4.6] libxl_set_memory_target: retain the same maxmem offset on top of the current target

2014-12-02 Thread Stefano Stabellini
On Tue, 2 Dec 2014, Don Slutz wrote: On 12/02/14 09:26, Stefano Stabellini wrote: On Tue, 2 Dec 2014, Don Slutz wrote: On 12/02/14 06:53, Stefano Stabellini wrote: In libxl_set_memory_target when setting the new maxmem, retain the same offset on top of the current target. The offset

Re: [Xen-devel] [PATCH for-4.5] pygrub: Fix regression from c/s d1b93ea, attempt 2

2014-12-02 Thread Konrad Rzeszutek Wilk
On Tue, Dec 02, 2014 at 02:02:58PM +, Ian Campbell wrote: On Tue, 2014-12-02 at 02:43 +, Andrew Cooper wrote: On 01/12/2014 20:30, Konrad Rzeszutek Wilk wrote: On Fri, Nov 28, 2014 at 11:31:24AM +, Ian Campbell wrote: On Tue, 2014-11-25 at 11:11 -0500, Boris Ostrovsky wrote:

Re: [Xen-devel] PVHVM drivers in upstream linux kernel

2014-12-02 Thread Konrad Rzeszutek Wilk
On Tue, Dec 02, 2014 at 11:05:14AM +, Ian Campbell wrote: On Tue, 2014-12-02 at 10:54 +, David Vrabel wrote: On 02/12/14 09:39, Juergen Gross wrote: Hi, looking into the upstream linux sources I realized that the PVHVM drivers of XEN are only available with the pvops

Re: [Xen-devel] [Xen-users] 4.5 git: regression in xen systemd shutdown hangs the OS

2014-12-02 Thread Olaf Hering
On Tue, Dec 02, Ian Campbell wrote: On Mon, 2014-12-01 at 23:41 +, Mark Pryor wrote: list, Thanks. If you've identified a buggy changeset then it is fine to post to the devel lists. I've added a CC. I've also CCd everyone listed in the commit which you've fingered. Olaf, does this

[Xen-devel] [PATCH for-xen-4.5 2/3] gitignore: ignore some files generated by configure

2014-12-02 Thread Daniel Kiper
Signed-off-by: Daniel Kiper daniel.ki...@oracle.com --- .gitignore |3 +++ 1 file changed, 3 insertions(+) diff --git a/.gitignore b/.gitignore index b24e905..2b51d5f 100644 --- a/.gitignore +++ b/.gitignore @@ -141,12 +141,15 @@ tools/flask/utils/flask-set-bool

[Xen-devel] [PATCH for-xen-4.5 1/3] tools/hotplug: distclean target should remove files generated by configure

2014-12-02 Thread Daniel Kiper
Signed-off-by: Daniel Kiper daniel.ki...@oracle.com --- tools/Makefile |3 +++ tools/hotplug/Makefile |5 - 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/tools/Makefile b/tools/Makefile index af9798a..19b24f3 100644 --- a/tools/Makefile +++ b/tools/Makefile @@

[Xen-devel] [PATCH] libxc: check in xc_get_tot_pages() that the proper domain is reported

2014-12-02 Thread Vitaly Kuznetsov
XEN_DOMCTL_getdomaininfo, which is being used by xc_domain_getinfo(), has strange interface: it reports first domain which has domid = requested domid so all callers are supposed to check that the proper domain(s) was queried by checking domid. xc_get_tot_pages() doesn't do that. In case the

[Xen-devel] [PATCH for-xen-4.5 3/3] gitignore: group tools/hotplug files in one place

2014-12-02 Thread Daniel Kiper
Signed-off-by: Daniel Kiper daniel.ki...@oracle.com --- .gitignore |9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/.gitignore b/.gitignore index 2b51d5f..8c8c06f 100644 --- a/.gitignore +++ b/.gitignore @@ -145,6 +145,10 @@

[Xen-devel] [PATCH for-xen-4.5 0/3] tools: build system fixes and cleanups

2014-12-02 Thread Daniel Kiper
Hi, Here are three tools build system patches: - tools/hotplug: distclean target should remove files generated by configure, - gitignore: ignore some files generated by configure, - gitignore: group tools/hotplug files in one place. First two are real fixes. Last one is a cleanup which is

Re: [Xen-devel] [PATCH v2 for-4.6] libxl_set_memory_target: retain the same maxmem offset on top of the current target

2014-12-02 Thread Don Slutz
On 12/02/14 09:59, Don Slutz wrote: On 12/02/14 09:26, Stefano Stabellini wrote: On Tue, 2 Dec 2014, Don Slutz wrote: On 12/02/14 06:53, Stefano Stabellini wrote: In libxl_set_memory_target when setting the new maxmem, retain the same offset on top of the current target. The offset includes

[Xen-devel] [PATCH for-4.5] systemd: use pkg-config to determine systemd library availability

2014-12-02 Thread Wei Liu
AC_CHECK_LIB fails on Debian Jessie since the ld flag it generates is incorrect, even in the event systemd library is available. Use PKG_CHECK_MODULES instead. Tested on Debian Jessie and Arch Linux. Please rerun autogen.sh after applying this patch. Reported-by: Mark Pryor tlvie...@yahoo.com

Re: [Xen-devel] [RFC v2] Add support for Xen ARM guest on FreeBSD

2014-12-02 Thread Julien Grall
Hello Andrew, On 28/11/2014 13:57, Andrew Turner wrote: On Sun, 23 Nov 2014 22:35:36 + Julien Grall julien.gr...@linaro.org wrote: Major changes in this new version: * Add Device Tree support via Linux Boot ABI * Add zImage support * Netfront support *

Re: [Xen-devel] [PATCH] libxc: check in xc_get_tot_pages() that the proper domain is reported

2014-12-02 Thread Don Slutz
On 12/02/14 10:18, Vitaly Kuznetsov wrote: XEN_DOMCTL_getdomaininfo, which is being used by xc_domain_getinfo(), has strange interface: it reports first domain which has domid = requested domid so all callers are supposed to check that the proper domain(s) was queried by checking domid.

Re: [Xen-devel] [COVERITY ACCESS] Request for access to Coverity

2014-12-02 Thread Ian Campbell
On Tue, 2014-12-02 at 15:10 +, George Dunlap wrote: On Thu, Nov 27, 2014 at 3:11 PM, Konrad Rzeszutek Wilk konrad.w...@oracle.com wrote: On Nov 27, 2014 6:59 AM, Tim Deegan t...@xen.org wrote: At 11:39 + on 27 Nov (1417084797), George Dunlap wrote: -BEGIN PGP SIGNED

Re: [Xen-devel] [PATCH for-4.5] systemd: use pkg-config to determine systemd library availability

2014-12-02 Thread Ian Campbell
On Tue, 2014-12-02 at 15:11 +, Wei Liu wrote: AC_CHECK_LIB fails on Debian Jessie since the ld flag it generates is incorrect, even in the event systemd library is available. Use PKG_CHECK_MODULES instead. Tested on Debian Jessie and Arch Linux. Please rerun autogen.sh after applying

Re: [Xen-devel] xl pci-attach silently fails the first time

2014-12-02 Thread Olaf Hering
On Mon, Dec 01, Sander Eikelenboom wrote: Monday, December 1, 2014, 3:34:09 PM, you wrote: actually be a workaround for the double pci-attach bug. Don't know about that bug. You just replied to it. ;-) Olaf ___ Xen-devel mailing list

Re: [Xen-devel] xl pci-attach silently fails the first time

2014-12-02 Thread Olaf Hering
On Mon, Dec 01, Konrad Rzeszutek Wilk wrote: That is odd - I see any device 'hot-plugged' being added at 00:05 and further. Does this by any chance depend on the guest?! I mean, how is the guest notified that a PCI device is gone (by unplug)? Maybe the pvops case just happens to work because

Re: [Xen-devel] [PATCH v2 for-4.5 0/2] xl/libxl: fix API and two memory leaks

2014-12-02 Thread Ian Campbell
On Mon, 2014-12-01 at 16:54 -0500, Konrad Rzeszutek Wilk wrote: On Mon, Dec 01, 2014 at 11:31:11AM +, Wei Liu wrote: Return value of libxl_basename was erroneously marked as const. This series removes that const and fixes two memory leaks in xl. I think these fixes should be included

Re: [Xen-devel] [PATCH] xenstore: Clarify xs_open() semantics

2014-12-02 Thread Ian Campbell
On Mon, 2014-12-01 at 16:16 -0500, Konrad Rzeszutek Wilk wrote: On Fri, Nov 28, 2014 at 12:31:34PM +, Ian Jackson wrote: Razvan Cojocaru writes ([PATCH] xenstore: Clarify xs_open() semantics): Added to the xs_open() comments in xenstore.h. The text has been taken almost verbatim from

Re: [Xen-devel] 答复: Re: [PATCH] missing chunk of HVM direct kernel boot patch

2014-12-02 Thread Ian Campbell
On Mon, 2014-12-01 at 16:51 -0500, Konrad Rzeszutek Wilk wrote: On Mon, Dec 01, 2014 at 09:21:24AM +, Ian Campbell wrote: On Mon, 2014-12-01 at 01:21 -0700, Chun Yan Liu wrote: Ian Campbell ian.campb...@citrix.com 2014-11-28 下午 20:01 On Fri, 2014-11-28 at 13:55 +0800, Chunyan

Re: [Xen-devel] [PATCH v2] fix migration failure with xl migrate --debug

2014-12-02 Thread Ian Campbell
On Mon, 2014-12-01 at 15:25 -0500, Konrad Rzeszutek Wilk wrote: On Fri, Nov 28, 2014 at 11:52:26AM +, Ian Jackson wrote: Ian Campbell writes (Re: [PATCH v2] fix migration failure with xl migrate --debug): Acked-by: Ian Campbell ian.campb...@citrix.com Thanks for reviewing it :-).

Re: [Xen-devel] [Xen-users] 4.5 git: regression in xen systemd shutdown hangs the OS

2014-12-02 Thread Ian Campbell
On Tue, 2014-12-02 at 16:17 +0100, Olaf Hering wrote: On Tue, Dec 02, Ian Campbell wrote: On Mon, 2014-12-01 at 23:41 +, Mark Pryor wrote: list, Thanks. If you've identified a buggy changeset then it is fine to post to the devel lists. I've added a CC. I've also CCd everyone

Re: [Xen-devel] [PATCH] libxl: Don't derefence null new_name pointer in libxl_domain_rename()

2014-12-02 Thread Ian Campbell
On Mon, 2014-12-01 at 17:02 -0500, Konrad Rzeszutek Wilk wrote: On Mon, Dec 01, 2014 at 02:50:56PM +, Ian Campbell wrote: On Mon, 2014-12-01 at 14:27 +, Euan Harris wrote: libxl__domain_rename() unconditionally dereferences its new_name parameter, to check whether it is an empty

Re: [Xen-devel] [PATCH] libxl: libxl_domain_info: fix typo in error message

2014-12-02 Thread Ian Campbell
On Mon, 2014-12-01 at 11:15 +, Ian Campbell wrote: On Mon, 2014-12-01 at 10:47 +, Euan Harris wrote: Signed-off-by: Euan Harris euan.har...@citrix.com Acked-by: Ian Campbell ian.campb...@citrix.com This is so trivial as to not need a release ack IMHO, I'll apply next time I'm

Re: [Xen-devel] [PATCH] tools/hotplug: update systemd dependency to use service instead of socket

2014-12-02 Thread Ian Campbell
On Tue, 2014-12-02 at 16:39 +0100, Olaf Hering wrote: Since commit 4542ae340d75bd6319e3fcd94e6c9336e210aeef (tools/hotplug: systemd xenstored dependencies) all service files use the .socket unit as startup dependency. While this happens to work for boot it fails for shutdown because a .socket

[Xen-devel] [PATCH OSSTEST v2 03/18] Pass host to toolstack()

2014-12-02 Thread Ian Campbell
This will be needed in a future patch. Everywhere already has a $ho in hand. Also cache the answer as $ho-{Toolstack}. I scanned the source with: find -name \*.pm -exec perl -c {} \; for i in ts-* ; do perl -c $i; done which reported Not enough arguments for

[Xen-devel] [PATCH OSSTEST v2 10/18] Toolstack: Refactor shutdown support

2014-12-02 Thread Ian Campbell
Signed-off-by: Ian Campbell ian.campb...@citrix.com --- Osstest/Toolstack/libvirt.pm | 6 ++ Osstest/Toolstack/xend.pm| 1 + Osstest/Toolstack/xl.pm | 7 +++ ts-guest-stop| 5 + 4 files changed, 15 insertions(+), 4 deletions(-) diff --git

[Xen-devel] [PATCH OSSTEST v2 02/18] ts-logs-capture: Collect some libvirt logs and capabilities

2014-12-02 Thread Ian Campbell
Signed-off-by: Ian Campbell ian.campb...@citrix.com --- ts-logs-capture | 4 1 file changed, 4 insertions(+) diff --git a/ts-logs-capture b/ts-logs-capture index 21974a9..6cf51c1 100755 --- a/ts-logs-capture +++ b/ts-logs-capture @@ -117,6 +117,9 @@ sub fetch_logs_host_guests () {

Re: [Xen-devel] [PATCH] libxc: check in xc_get_tot_pages() that the proper domain is reported

2014-12-02 Thread Wei Liu
On Tue, Dec 02, 2014 at 04:18:08PM +0100, Vitaly Kuznetsov wrote: XEN_DOMCTL_getdomaininfo, which is being used by xc_domain_getinfo(), has strange interface: it reports first domain which has domid = requested domid so all callers are supposed to check that the proper domain(s) was queried by

[Xen-devel] [PATCH OSSTEST v2 04/18] ts-rumpuserxen-demo-xenstorels: Use standard functions for things

2014-12-02 Thread Ian Campbell
Specifically guest_create and guest_find_domid. Signed-off-by: Ian Campbell ian.campb...@citrix.com --- ts-rumpuserxen-demo-xenstorels | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/ts-rumpuserxen-demo-xenstorels b/ts-rumpuserxen-demo-xenstorels index a2a6a77..6698848

[Xen-devel] [PATCH OSSTEST v2 11/18] Toolstack: Refactor migration support check.

2014-12-02 Thread Ian Campbell
Not implemented for libvirt (the check itself that is, the hook is present). Signed-off-by: Ian Campbell ian.campb...@citrix.com --- Osstest/Toolstack/libvirt.pm | 5 + Osstest/Toolstack/xend.pm| 3 +++ Osstest/Toolstack/xl.pm | 9 + ts-migrate-support-check | 10

Re: [Xen-devel] [PATCH] libxc: check in xc_get_tot_pages() that the proper domain is reported

2014-12-02 Thread Andrew Cooper
On 02/12/14 15:18, Vitaly Kuznetsov wrote: XEN_DOMCTL_getdomaininfo, which is being used by xc_domain_getinfo(), has strange interface: it reports first domain which has domid = requested domid so all callers are supposed to check that the proper domain(s) was queried by checking domid.

Re: [Xen-devel] [Xen-users] 4.5 git: regression in xen systemd shutdown hangs the OS

2014-12-02 Thread Olaf Hering
On Tue, Dec 02, Ian Campbell wrote: On Tue, 2014-12-02 at 16:17 +0100, Olaf Hering wrote: On Tue, Dec 02, Ian Campbell wrote: On Mon, 2014-12-01 at 23:41 +, Mark Pryor wrote: list, Thanks. If you've identified a buggy changeset then it is fine to post to the devel

[Xen-devel] [PATCH OSSTEST v2 09/18] Toolstack: Refactor consolecmd handling

2014-12-02 Thread Ian Campbell
Signed-off-by: Ian Campbell ian.campb...@citrix.com --- Osstest/Toolstack/libvirt.pm | 6 ++ Osstest/Toolstack/xend.pm| 1 + Osstest/Toolstack/xl.pm | 6 ++ ts-logs-capture | 2 +- 4 files changed, 14 insertions(+), 1 deletion(-) diff --git

[Xen-devel] [PATCH OSSTEST v2 06/18] TestSupport: always use xl for generic operations.

2014-12-02 Thread Ian Campbell
Unless the toolstack is xend (for compatibility with pre-xl Xen versions), when we use xm. For several operations in TestSupport.pm the actual toolstack isn't really relevant, since we want info straight from Xen. For simplicity just use xl (or xm) in these cases, to avoid needing to implement

[Xen-devel] [PATCH OSSTEST v2 08/18] Toolstack: Refactor guest lifecycle.

2014-12-02 Thread Ian Campbell
Implement destory/create as per toolstack methods, including implementing the libvirt version which previously didn't work. To do this we use the virsh capability to convert an xl/xm style config file into the correct XML. xend basically calls into the xl helper since they are compatible. xl/x,

Re: [Xen-devel] Poor network performance between DomU with multiqueue support

2014-12-02 Thread Wei Liu
On Tue, Dec 02, 2014 at 02:46:36PM +, Zhangleiqiang (Trump) wrote: Thanks for your reply, Wei. I do the following testing just now and found the results as follows: There are three DomUs (4U4G) are running on Host A (6U6G) and one DomU (4U4G) is running on Host B (6U6G), I send

[Xen-devel] [PATCH OSSTEST v2 17/18] ts-guest-start: Use guest_create

2014-12-02 Thread Ian Campbell
This allows us to abolish CfgPathVar which was inconsistently used, appears redundant with $gho-{CfgPath} and in any case never set to anything other than 'cfgpath'. I suppose it was intended to deal with toolstacks with a cfg format completely dissimilar to xm/xl's. I think if this arises in a

[Xen-devel] [PATCH OSSTEST v2 14/18] libvirt: Implement initscript restart which has some hope of working.

2014-12-02 Thread Ian Campbell
Signed-off-by: Ian Campbell ian.campb...@citrix.com --- ts-libvirt-build | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/ts-libvirt-build b/ts-libvirt-build index 940c034..878d4e2 100755 --- a/ts-libvirt-build +++ b/ts-libvirt-build @@ -140,8 +140,10 @@ case $1 in

[Xen-devel] [PATCH OSSTEST v2 07/18] TestSupport: guest_create takes a $ho.

2014-12-02 Thread Ian Campbell
And looks up the toolstack from it. This is now consistent with guest_destroy. Signed-off-by: Ian Campbell ian.campb...@citrix.com --- Osstest/TestSupport.pm | 5 +++-- ts-debian-hvm-install | 7 ++- ts-redhat-install | 7 ++- ts-rumpuserxen-demo-xenstorels

[Xen-devel] [PATCH OSSTEST v2 12/18] Toolstack: Refactor migration support.

2014-12-02 Thread Ian Campbell
Note that since the previous patch arranges for ts-migration-support-check to continue to fail for libvirt the libvirt code is not actually called yet (and will die if it is). This patch is mainly included to reduce the number of users of toolstack()-{Command} closer to zero. Signed-off-by: Ian

[Xen-devel] [PATCH v2] introduce grant copy for user land

2014-12-02 Thread Thanos Makatos
This patch introduces the interface to allow user-space applications execute grant-copy operations. This is done by sending an ioctl to the grant device. Signed-off-by: Thanos Makatos thanos.maka...@citrix.com --- drivers/xen/gntdev.c | 171 +

[Xen-devel] [PATCH OSSTEST v2 16/18] Toolstack: Remove Command field for all toolstacks.

2014-12-02 Thread Ian Campbell
Nothing in generic code uses this now, so remove. xl+xend retain as _Command for internal use only. Signed-off-by: Ian Campbell ian.campb...@citrix.com --- Osstest/Toolstack/libvirt.pm | 1 - Osstest/Toolstack/xend.pm| 2 +- Osstest/Toolstack/xl.pm | 18 +- 3 files

[Xen-devel] [PATCH OSSTEST v2 01/18] apt: lock osstest's usages of apt-get against each other

2014-12-02 Thread Ian Campbell
Currently we rely on all apt-get invocations being in a single ts-xen-build-prep job which can't run on a shared host. That is a bit inflexible so instead use our own lock. We wait indefinitely and rely on osstest's existing command timeout infrastructure to catch problems.

  1   2   >