[Xen-devel] [ovmf test] 52776: regressions - FAIL

2015-04-30 Thread osstest service user
flight 52776 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/52776/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-freebsd10-amd64 15 guest-localmigrate.2 fail REGR. vs. 36764 test-amd64-i386-freebsd10-i3

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

2015-04-30 Thread osstest service user
flight 52702 linux-linus real [real] http://logs.test-lab.xenproject.org/osstest/logs/52702/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-credit2 13 guest-saverestore fail REGR. vs. 50329 test-amd64-amd64-xl-m

[Xen-devel] [xen-4.3-testing test] 52661: regressions - FAIL

2015-04-30 Thread osstest service user
flight 52661 xen-4.3-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/52661/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-freebsd10-amd64 16 guest-localmigrate/x10 fail REGR. vs. 36755 test-amd64-amd64-

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

2015-04-30 Thread osstest service user
flight 52652 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/52652/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-freebsd10-amd64 13 guest-localmigrate fail REGR. vs. 50391 test-amd64-i386-xl-

[Xen-devel] [PATCH v4 3/3] xen/iommu: arm: Use p2m_ipa_bits as stage2 input size

2015-04-30 Thread Edgar E. Iglesias
From: "Edgar E. Iglesias" The Stage2 input-size must match what the CPU uses because the SMMU and the CPU share page-tables. Test that the SMMU supports the P2M IPA bit size, use it if supported or bail out if not. Signed-off-by: Edgar E. Iglesias --- xen/drivers/passthrough/arm/smmu.c | 10 +

[Xen-devel] [PATCH v4 2/3] xen/arm: Add p2m_ipa_bits

2015-04-30 Thread Edgar E. Iglesias
From: "Edgar E. Iglesias" Export p2m_ipa_bits holding the bit size of IPAs used in p2m tables. Signed-off-by: Edgar E. Iglesias --- xen/arch/arm/p2m.c| 5 + xen/include/asm-arm/p2m.h | 3 +++ 2 files changed, 8 insertions(+) diff --git a/xen/arch/arm/p2m.c b/xen/arch/arm/p2m.c ind

[Xen-devel] [PATCH v4 1/3] xen/arm: Re-order iommu_setup to after setup_virt_paging

2015-04-30 Thread Edgar E. Iglesias
From: "Edgar E. Iglesias" This is needed to allow the paging setup to probe for IPA bit sizes to be used in p2m tables prior to iommu setup. Reviewed-by: Julien Grall Signed-off-by: Edgar E. Iglesias --- xen/arch/arm/setup.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --gi

Re: [Xen-devel] [PATCH v3 3/3] xen/iommu: arm: Use p2m_ipa_bits as stage2 input size

2015-04-30 Thread Edgar E. Iglesias
On Thu, Apr 30, 2015 at 04:07:27PM +0100, Julien Grall wrote: > Hi Edgar, > > On 30/04/15 12:55, Edgar E. Iglesias wrote: > > From: "Edgar E. Iglesias" > > > > The Stage2 input-size must match what the CPU uses because > > the SMMU and the CPU share page-tables. > > > > Assert that the SMMU sup

[Xen-devel] [PATCH v4 0/3] Set SMMU s2 input-size based on p2m tables

2015-04-30 Thread Edgar E. Iglesias
From: "Edgar E. Iglesias" Hi, This is a fix for the issue I'm seeing on ZynqMP with missmatched setup of the SMMU and the shared p2m page-tables with the CPU. This implementes a global p2m_ipa_bits cap for S2 input-size as discussed in the previous RFC. Best regards, Edgar Changelog: v3 -> v4

Re: [Xen-devel] [PATCH v4 2/5] lib: devres: add pcim_iomap_wc() variants

2015-04-30 Thread Luis R. Rodriguez
On Thu, Apr 30, 2015 at 04:46:38PM -0500, Bjorn Helgaas wrote: > On Thu, Apr 30, 2015 at 07:27:23PM +0200, Luis R. Rodriguez wrote: > > On Thu, Apr 30, 2015 at 11:26:47AM -0500, Bjorn Helgaas wrote: > > > > I don't see users of either pcim_iomap_wc() or pcim_iomap_wc_regions() so > > > far. Did I

[Xen-devel] [xen-4.2-testing test] 52651: regressions - FAIL

2015-04-30 Thread osstest service user
flight 52651 xen-4.2-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/52651/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-qemuu-debianhvm-amd64 5 xen-install fail REGR. vs. 36512 test-amd64-i386-q

[Xen-devel] [xen-4.5-testing test] 52649: regressions - FAIL

2015-04-30 Thread osstest service user
flight 52649 xen-4.5-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/52649/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-xl-credit2 6 xen-boot fail REGR. vs. 50408 test-amd64-amd64-

Re: [Xen-devel] [PATCH] hypervisor/x86/xen: Unset X86_BUG_SYSRET_SS_ATTRS on Xen PV guests

2015-04-30 Thread Boris Ostrovsky
On 04/30/2015 03:35 PM, Andy Lutomirski wrote: On Thu, Apr 30, 2015 at 12:30 PM, Boris Ostrovsky wrote: On 04/30/2015 03:17 PM, Andy Lutomirski wrote: On Thu, Apr 30, 2015 at 12:08 PM, Boris Ostrovsky wrote: Commit 61f01dd941ba ("x86_64, asm: Work around AMD SYSRET SS descriptor attribute is

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

2015-04-30 Thread osstest service user
flight 52703 libvirt real [real] http://logs.test-lab.xenproject.org/osstest/logs/52703/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-libvirt 11 guest-start fail REGR. vs. 50368 Tests which did not succe

Re: [Xen-devel] [PATCH v4 2/5] lib: devres: add pcim_iomap_wc() variants

2015-04-30 Thread Bjorn Helgaas
On Thu, Apr 30, 2015 at 07:27:23PM +0200, Luis R. Rodriguez wrote: > On Thu, Apr 30, 2015 at 11:26:47AM -0500, Bjorn Helgaas wrote: > > I don't see users of either pcim_iomap_wc() or pcim_iomap_wc_regions() so > > far. Did I miss them, or do you just expect them in the near future? > > The later

[Xen-devel] [PATCH v5 6/6] IB/ipath: use arch_phys_wc_add() and require PAT disabled

2015-04-30 Thread Luis R. Rodriguez
From: "Luis R. Rodriguez" We are burrying direct access to MTRR code support on x86 in order to take advantage of PAT. In the future we also want to make the default behaviour of ioremap_nocache() to use strong UC, use of mtrr_add() on those systems would make write-combining void. In order to h

[Xen-devel] [PATCH v5 4/6] ivtv: use arch_phys_wc_add() and require PAT disabled

2015-04-30 Thread Luis R. Rodriguez
From: "Luis R. Rodriguez" We are burrying direct access to MTRR code support on x86 in order to take advantage of PAT. In the future we also want to make the default behaviour of ioremap_nocache() to use strong UC, use of mtrr_add() on those systems would make write-combining void. In order to h

Re: [Xen-devel] [PATCH] hypervisor/x86/xen: Unset X86_BUG_SYSRET_SS_ATTRS on Xen PV guests

2015-04-30 Thread Andy Lutomirski
On Thu, Apr 30, 2015 at 12:30 PM, Boris Ostrovsky wrote: > On 04/30/2015 03:17 PM, Andy Lutomirski wrote: >> >> On Thu, Apr 30, 2015 at 12:08 PM, Boris Ostrovsky >> wrote: >>> >>> Commit 61f01dd941ba ("x86_64, asm: Work around AMD SYSRET SS descriptor >>> attribute issue") makes AMD processors se

Re: [Xen-devel] [PATCH] hypervisor/x86/xen: Unset X86_BUG_SYSRET_SS_ATTRS on Xen PV guests

2015-04-30 Thread Boris Ostrovsky
On 04/30/2015 03:17 PM, Andy Lutomirski wrote: On Thu, Apr 30, 2015 at 12:08 PM, Boris Ostrovsky wrote: Commit 61f01dd941ba ("x86_64, asm: Work around AMD SYSRET SS descriptor attribute issue") makes AMD processors set SS to __KERNEL_DS in __switch_to() to deal with cases when SS is NULL. This

Re: [Xen-devel] [PATCH] hypervisor/x86/xen: Unset X86_BUG_SYSRET_SS_ATTRS on Xen PV guests

2015-04-30 Thread Andy Lutomirski
On Thu, Apr 30, 2015 at 12:08 PM, Boris Ostrovsky wrote: > Commit 61f01dd941ba ("x86_64, asm: Work around AMD SYSRET SS descriptor > attribute issue") makes AMD processors set SS to __KERNEL_DS in > __switch_to() to deal with cases when SS is NULL. > > This breaks Xen PV guests who do not want to

[Xen-devel] [PATCH] hypervisor/x86/xen: Unset X86_BUG_SYSRET_SS_ATTRS on Xen PV guests

2015-04-30 Thread Boris Ostrovsky
Commit 61f01dd941ba ("x86_64, asm: Work around AMD SYSRET SS descriptor attribute issue") makes AMD processors set SS to __KERNEL_DS in __switch_to() to deal with cases when SS is NULL. This breaks Xen PV guests who do not want to load SS with__KERNEL_DS. Since the problem that the commit is tryi

Re: [Xen-devel] [PATCH v16 13/14] pvqspinlock: Improve slowpath performance by avoiding cmpxchg

2015-04-30 Thread Waiman Long
On 04/29/2015 02:27 PM, Linus Torvalds wrote: On Wed, Apr 29, 2015 at 11:11 AM, Peter Zijlstra wrote: On Fri, Apr 24, 2015 at 02:56:42PM -0400, Waiman Long wrote: In the pv_scan_next() function, the slow cmpxchg atomic operation is performed even if the other CPU is not even close to being hal

Re: [Xen-devel] [PATCH v16 13/14] pvqspinlock: Improve slowpath performance by avoiding cmpxchg

2015-04-30 Thread Waiman Long
On 04/29/2015 02:11 PM, Peter Zijlstra wrote: On Fri, Apr 24, 2015 at 02:56:42PM -0400, Waiman Long wrote: In the pv_scan_next() function, the slow cmpxchg atomic operation is performed even if the other CPU is not even close to being halted. This extra cmpxchg can harm slowpath performance. Th

[Xen-devel] [PATCH 1/1] xen-hvm: Add trace to ioreq

2015-04-30 Thread Don Slutz
Signed-off-by: Don Slutz (cherry picked from commit b72adbe7510d0a30053d32334665ee887bec9e43) --- trace-events | 7 +++ xen-hvm.c| 21 + 2 files changed, 28 insertions(+) diff --git a/trace-events b/trace-events index 30eba92..4666dad 100644 --- a/trace-events +++ b/

Re: [Xen-devel] [PATCH v1 23/47] staging: xgifb: use arch_phys_wc_add() and ioremap_wc()

2015-04-30 Thread Luis R. Rodriguez
On Fri, Mar 20, 2015 at 4:18 PM, Luis R. Rodriguez wrote: > The same area used for ioremap() is used for the MTRR area. > Convert the driver from using the x86 specific MTRR code to > the architecture agnostic arch_phys_wc_add(). arch_phys_wc_add() > will avoid MTRR if write-combining is available

[Xen-devel] [PATCH v5 1/5] pci: add pci_iomap_wc() variants

2015-04-30 Thread Luis R. Rodriguez
From: "Luis R. Rodriguez" This allows drivers to take advantage of write-combining when possible. The PCI specification does not allow for us to automatically identify a memory region which needs write-combining so drivers have to identify these areas on their own. There is IORESOURCE_PREFETCH bu

Re: [Xen-devel] [PATCH v4 2/5] lib: devres: add pcim_iomap_wc() variants

2015-04-30 Thread Luis R. Rodriguez
On Thu, Apr 30, 2015 at 11:26:47AM -0500, Bjorn Helgaas wrote: > [+cc linux-pci] > > Hi Luis, > > On Wed, Apr 29, 2015 at 02:36:09PM -0700, Luis R. Rodriguez wrote: > > From: "Luis R. Rodriguez" > > > > Now that we have pci_iomap_wc() add the respective devres helpers. > > I guess I'm still co

Re: [Xen-devel] [PATCH v4 1/5] pci: add pci_iomap_wc() variants

2015-04-30 Thread Luis R. Rodriguez
On Thu, Apr 30, 2015 at 10:03:18AM -0700, Andy Lutomirski wrote: > On Thu, Apr 30, 2015 at 9:52 AM, Luis R. Rodriguez wrote: > > On Thu, Apr 30, 2015 at 10:59:17AM -0500, Bjorn Helgaas wrote: > >> [+cc linux-pci] > >> > >> Hi Luis, > >> > >> On Wed, Apr 29, 2015 at 02:36:08PM -0700, Luis R. Rodrig

Re: [Xen-devel] [PATCH v4 1/5] pci: add pci_iomap_wc() variants

2015-04-30 Thread Andy Lutomirski
On Thu, Apr 30, 2015 at 9:52 AM, Luis R. Rodriguez wrote: > On Thu, Apr 30, 2015 at 10:59:17AM -0500, Bjorn Helgaas wrote: >> [+cc linux-pci] >> >> Hi Luis, >> >> On Wed, Apr 29, 2015 at 02:36:08PM -0700, Luis R. Rodriguez wrote: >> > From: "Luis R. Rodriguez" >> > >> > This allows drivers to tak

Re: [Xen-devel] [PATCH 1/3] raisin: introduce tests

2015-04-30 Thread Stefano Stabellini
On Thu, 30 Apr 2015, Anthony PERARD wrote: > On Wed, Apr 29, 2015 at 06:00:54PM +0100, Stefano Stabellini wrote: > > Introduce a new command to run functional tests and unit tests. > > Introduce a generic infrastrucutre to run tests on the local machine. > > Add a library of common functions that c

Re: [Xen-devel] Xen 4.6 Development Update (three months reminder)

2015-04-30 Thread Jim Fehlig
Fabio Fantoni wrote: > Il 29/04/2015 18:39, Jim Fehlig ha scritto: >> dom0 and domU are both SLES12. Xen is fairly recent xen-unstable plus a >> few libxl patches. >> >> Regards, >> Jim >> > > But the domU was without qxl driver installed? No, it was installed: linux-tyb8:~/ # dmesg | grep -i qx

Re: [Xen-devel] [PATCH v4 1/5] pci: add pci_iomap_wc() variants

2015-04-30 Thread Luis R. Rodriguez
On Thu, Apr 30, 2015 at 10:59:17AM -0500, Bjorn Helgaas wrote: > [+cc linux-pci] > > Hi Luis, > > On Wed, Apr 29, 2015 at 02:36:08PM -0700, Luis R. Rodriguez wrote: > > From: "Luis R. Rodriguez" > > > > This allows drivers to take advantage of write-combining > > when possible. Ideally we'd hav

Re: [Xen-devel] [PATCH 3/3] raisin: improve output

2015-04-30 Thread Stefano Stabellini
On Thu, 30 Apr 2015, Anthony PERARD wrote: > On Wed, Apr 29, 2015 at 06:00:56PM +0100, Stefano Stabellini wrote: > > Introduce an error_echo function that prints to stderr. > > Call error_echo or verbose_echo, instead of echo, when possible and > > appropriate. > > Redirect build and tests output t

Re: [Xen-devel] [PATCH 0/6] x86: reduce paravirtualized spinlock overhead

2015-04-30 Thread Jeremy Fitzhardinge
On 04/30/2015 03:53 AM, Juergen Gross wrote: > Paravirtualized spinlocks produce some overhead even if the kernel is > running on bare metal. The main reason are the more complex locking > and unlocking functions. Especially unlocking is no longer just one > instruction but so complex that it is no

Re: [Xen-devel] [PATCH v4 2/5] lib: devres: add pcim_iomap_wc() variants

2015-04-30 Thread Bjorn Helgaas
[+cc linux-pci] Hi Luis, On Wed, Apr 29, 2015 at 02:36:09PM -0700, Luis R. Rodriguez wrote: > From: "Luis R. Rodriguez" > > Now that we have pci_iomap_wc() add the respective devres helpers. I guess I'm still confused about the relationship between pci_iomap_wc() and arch_phys_wc_add(). Do yo

Re: [Xen-devel] [PATCH v4 1/5] pci: add pci_iomap_wc() variants

2015-04-30 Thread Bjorn Helgaas
[+cc linux-pci] Hi Luis, On Wed, Apr 29, 2015 at 02:36:08PM -0700, Luis R. Rodriguez wrote: > From: "Luis R. Rodriguez" > > This allows drivers to take advantage of write-combining > when possible. Ideally we'd have pci_read_bases() just > peg an IORESOURCE_WC flag for us This makes it sound

Re: [Xen-devel] [PATCH v4] xen/pvh: use a custom IO bitmap for PVH hardware domains

2015-04-30 Thread Andrew Cooper
On 30/04/15 11:42, Roger Pau Monne wrote: Since a PVH hardware domain has access to the physical hardware create a custom more permissive IO bitmap. The permissions set on the bitmap are populated based on the contents of the ioports rangeset. Signed-off-by: Roger Pau Monné Cc: Jan Beulich Cc:

Re: [Xen-devel] [PATCHv3 0/4] Use ticket locks for spinlocks

2015-04-30 Thread David Vrabel
On 23/04/15 13:42, David Vrabel wrote: > On 21/04/15 11:22, Jan Beulich wrote: > On 21.04.15 at 12:11, wrote: >>> We have analysed the affect of this series on interrupt latency (by >>> measuring the duration of irq disable/enable regions) and there is no >>> signficant impact. >>> >>> http://

[Xen-devel] [PATCHv4 4/8] arm: provide add_sized()

2015-04-30 Thread David Vrabel
add_sized(ptr, inc) adds inc to the value at ptr using only the correct size of loads and stores for the type of *ptr. The add is /not/ atomic. This is needed for ticket locks to ensure the increment of the head ticket does not affect the tail ticket. Signed-off-by: David Vrabel --- xen/includ

[Xen-devel] [PATCHv4 8/8] x86: reduce struct hvm_domain size

2015-04-30 Thread David Vrabel
Pack struct hvm_domain to reduce it by 8 bytes. Thus reducing the size of struct domain by 8 bytes. Signed-off-by: David Vrabel --- xen/include/asm-x86/hvm/domain.h | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/xen/include/asm-x86/hvm/domain.h b/xen/include/

[Xen-devel] [PATCHv4 6/8] x86, arm: remove asm/spinlock.h from all architectures

2015-04-30 Thread David Vrabel
Now that all architecture use a common ticket lock implementation for spinlocks, remove the architecture specific byte lock implementations. Signed-off-by: David Vrabel Reviewed-by: Tim Deegan Acked-by: Jan Beulich --- xen/arch/arm/README.LinuxPrimitives | 28 --- xen/include/as

[Xen-devel] [PATCHv4 2/8] arm: provide arch_fetch_and_add()

2015-04-30 Thread David Vrabel
arch_fetch_and_add() atomically adds a value and returns the previous value. This generic arm implementation uses the GCC __sync_fetch_and_add() builtin. This builtin resulted in suitable inlined asm for GCC 4.8.3 (arm64) and GCC 4.6.3 (arm32). This is needed to implement ticket locks. Signed-o

[Xen-devel] [PATCHv4 3/8] x86: provide add_sized()

2015-04-30 Thread David Vrabel
add_sized(ptr, inc) adds inc to the value at ptr using only the correct size of loads and stores for the type of *ptr. The add is /not/ atomic. This is needed for ticket locks to ensure the increment of the head ticket does not affect the tail ticket. Signed-off-by: David Vrabel --- xen/includ

[Xen-devel] [PATCHv4 5/8] xen: use ticket locks for spin locks

2015-04-30 Thread David Vrabel
Replace the byte locks with ticket locks. Ticket locks are: a) fair; and b) peform better when contented since they spin without an atomic operation. The lock is split into two ticket values: head and tail. A locker acquires a ticket by (atomically) increasing tail and using the previous tail va

[Xen-devel] [PATCHv4 1/8] x86: provide arch_fetch_and_add()

2015-04-30 Thread David Vrabel
arch_fetch_and_add() atomically adds a value and returns the previous value. This is needed to implement ticket locks. Signed-off-by: David Vrabel --- xen/include/asm-x86/system.h | 48 ++ 1 file changed, 48 insertions(+) diff --git a/xen/include/asm-x

[Xen-devel] [PATCHv4 7/8] x86: reduce struct paging_domain size

2015-04-30 Thread David Vrabel
Pack struct paging_domain to reduce it by 8 bytes. Thus reducing the size of struct domain by 8 bytes. Signed-off-by: David Vrabel --- xen/include/asm-x86/domain.h |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/xen/include/asm-x86/domain.h b/xen/include/asm-x86/domai

[Xen-devel] [PATCHv4 0/8] Use ticket locks for spinlocks

2015-04-30 Thread David Vrabel
Use ticket locks for spin locks instead of the current byte locks. Ticket locks are fair. This is an important property for hypervisor locks. Note that spin_lock_irq() and spin_lock_irqsave() now spin with irqs disabled (previously, they would spin with irqs enabled if possible). This is required

Re: [Xen-devel] [PATCH v10 3/6] Support for BIOS interrupt handler

2015-04-30 Thread Stefan Berger
On 04/30/2015 09:21 AM, Xu, Quan wrote: Stefan, Sorry to reply so late to you. I try to make it compatible with Xen vTPM by below patch Against your serious of patch(https://github.com/KevinOConnor/seabios/tree/tcg-testing ). Also I have tested this patch, which is working. Thanks.

Re: [Xen-devel] [PATCH 3/3] raisin: improve output

2015-04-30 Thread Anthony PERARD
On Wed, Apr 29, 2015 at 06:00:56PM +0100, Stefano Stabellini wrote: > Introduce an error_echo function that prints to stderr. > Call error_echo or verbose_echo, instead of echo, when possible and > appropriate. > Redirect build and tests output to /dev/null unless VERBOSE==1. > Redirect apt-get and

Re: [Xen-devel] [PATCHv7 3/3] gnttab: use per-VCPU maptrack free lists

2015-04-30 Thread Tim Deegan
At 14:28 +0100 on 30 Apr (1430404125), David Vrabel wrote: > From: Malcolm Crossley > > Performance analysis of aggregate network throughput with many VMs > shows that performance is signficantly limited by contention on the > maptrack lock when obtaining/releasing maptrack handles from the free

Re: [Xen-devel] [PATCH v3 3/3] xen/iommu: arm: Use p2m_ipa_bits as stage2 input size

2015-04-30 Thread Julien Grall
Hi Edgar, On 30/04/15 12:55, Edgar E. Iglesias wrote: > From: "Edgar E. Iglesias" > > The Stage2 input-size must match what the CPU uses because > the SMMU and the CPU share page-tables. > > Assert that the SMMU supports the P2M IPA bit size and use it. > > Signed-off-by: Edgar E. Iglesias >

[Xen-devel] [xen-unstable test] 52633: regressions - FAIL

2015-04-30 Thread osstest service user
flight 52633 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/52633/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-multivcpu 11 guest-start fail REGR. vs. 50405 test-amd64-amd64-xl-

Re: [Xen-devel] [PATCH v3 2/3] xen/arm: Add p2m_ipa_bits

2015-04-30 Thread Julien Grall
Hi Edgar, On 30/04/15 12:55, Edgar E. Iglesias wrote: > From: "Edgar E. Iglesias" > > Export p2m_ipa_bits holding the bit size of IPA addresses used NIT: I think "addresses" is redundant here. IPA always has the word "address" in it > in p2m tables. > > Signed-off-by: Edgar E. Iglesias > ---

Re: [Xen-devel] [PATCH 1/3] raisin: introduce tests

2015-04-30 Thread Anthony PERARD
On Wed, Apr 29, 2015 at 06:00:54PM +0100, Stefano Stabellini wrote: > Introduce a new command to run functional tests and unit tests. > Introduce a generic infrastrucutre to run tests on the local machine. > Add a library of common functions that can be used by the test scripts > to setup guest VMs

[Xen-devel] gntdev/gntalloc and fork

2015-04-30 Thread Marek Marczykowski-Górecki
Hi, What is the proper way to handle shared pages (either side - using gntdev or gntalloc) regarding fork and possible exec later? The child process do not need to access those pages in any way, but will map different one(s), using newly opened FD to the gntdev/gntalloc device. Should it unmap the

Re: [Xen-devel] [PATCHv7 2/3] gnttab: refactor locking for scalability

2015-04-30 Thread Tim Deegan
At 14:28 +0100 on 30 Apr (1430404124), David Vrabel wrote: > +/* > + * N.B.: while taking the left side maptrack spinlock prevents > + * any mapping changes, the right side active entries could be > + * changing while we are counting. IIRC, the lX/rX variables are local/remote rat

Re: [Xen-devel] [RFC PATCH v2 13/22] xen/arm: its: Add virtual ITS command support

2015-04-30 Thread Julien Grall
Hi, On 30/04/15 14:47, Stefano Stabellini wrote: If the devid is not within this range, the ITS won't recognize the value and won't be able to send the interrupt. So this is clearly not the right value. >>> >>> Sure, in that case the maximum value allowed by GITS_TYP

[Xen-devel] [PATCH] tools: add conigure --with-dumpdir=DIR option

2015-04-30 Thread Olaf Hering
The current base directory /var/xen for domU dumps will be patched to /var/lib/xen by most distros. Provide a configure option to avoid patching the source. If the option is not specified the default remains /var/xen/dump. Please rerun autogen.sh after applying this patch. Signed-off-by: Olaf He

Re: [Xen-devel] [RFC PATCH v2 13/22] xen/arm: its: Add virtual ITS command support

2015-04-30 Thread Stefano Stabellini
On Thu, 30 Apr 2015, Vijay Kilari wrote: > On Thu, Apr 30, 2015 at 3:45 PM, Stefano Stabellini > wrote: > > On Thu, 30 Apr 2015, Julien Grall wrote: > >> Hi Stefano, > >> > >> On 30/04/2015 11:02, Stefano Stabellini wrote: > >> > On Wed, 29 Apr 2015, Julien Grall wrote: > >> > > On 29/04/15 17:30,

[Xen-devel] [PATCHv7 3/3] gnttab: use per-VCPU maptrack free lists

2015-04-30 Thread David Vrabel
From: Malcolm Crossley Performance analysis of aggregate network throughput with many VMs shows that performance is signficantly limited by contention on the maptrack lock when obtaining/releasing maptrack handles from the free list. Instead of a single free list use a per-VCPU list. This avoids

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

2015-04-30 Thread David Vrabel
From: 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. Sig

[Xen-devel] [PATCHv7 0/3] gnttab: Improve scaleability

2015-04-30 Thread David Vrabel
The series makes the grant table locking more fine-grained and adds per-VCPU maptrack free lists, which greatly improves scalability. The series builds on the original series by Matt Wilson and Christoph Egger from Amazon. Performance results for aggregate intrahost network throughput (between 20

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

2015-04-30 Thread David Vrabel
From: Matt Wilson 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 remaining elements o

Re: [Xen-devel] [PATCH v10 3/6] Support for BIOS interrupt handler

2015-04-30 Thread Xu, Quan
> -Original Message- > From: Stefan Berger [mailto:stef...@linux.vnet.ibm.com] > Sent: Wednesday, April 29, 2015 7:18 PM > To: Xu, Quan > Cc: Kevin O'Connor; seab...@seabios.org; stefano.stabell...@eu.citrix.com; > xen-devel@lists.xen.org; Daniel De Graaf; wei.l...@citrix.com > Subject: R

Re: [Xen-devel] [RFC PATCH v2 13/22] xen/arm: its: Add virtual ITS command support

2015-04-30 Thread Vijay Kilari
On Thu, Apr 30, 2015 at 3:45 PM, Stefano Stabellini wrote: > On Thu, 30 Apr 2015, Julien Grall wrote: >> Hi Stefano, >> >> On 30/04/2015 11:02, Stefano Stabellini wrote: >> > On Wed, 29 Apr 2015, Julien Grall wrote: >> > > On 29/04/15 17:30, Vijay Kilari wrote: >> > > > On Wed, Apr 29, 2015 at 9:5

[Xen-devel] [PATCH v3 3/3] xen/iommu: arm: Use p2m_ipa_bits as stage2 input size

2015-04-30 Thread Edgar E. Iglesias
From: "Edgar E. Iglesias" The Stage2 input-size must match what the CPU uses because the SMMU and the CPU share page-tables. Assert that the SMMU supports the P2M IPA bit size and use it. Signed-off-by: Edgar E. Iglesias --- xen/drivers/passthrough/arm/smmu.c | 5 +++-- 1 file changed, 3 inse

[Xen-devel] [PATCH v3 2/3] xen/arm: Add p2m_ipa_bits

2015-04-30 Thread Edgar E. Iglesias
From: "Edgar E. Iglesias" Export p2m_ipa_bits holding the bit size of IPA addresses used in p2m tables. Signed-off-by: Edgar E. Iglesias --- xen/arch/arm/p2m.c| 5 + xen/include/asm-arm/p2m.h | 3 +++ 2 files changed, 8 insertions(+) diff --git a/xen/arch/arm/p2m.c b/xen/arch/arm/

[Xen-devel] [PATCH v3 1/3] xen/arm: Re-order iommu_setup to after setup_virt_paging

2015-04-30 Thread Edgar E. Iglesias
From: "Edgar E. Iglesias" This is needed to allow the paging setup to probe for IPA bit sizes to be used in p2m tables prior to iommu setup. Reviewed-by: Julien Grall Signed-off-by: Edgar E. Iglesias --- xen/arch/arm/setup.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --gi

[Xen-devel] [PATCH v3 0/3] Set SMMU s2 input-size based on p2m tables

2015-04-30 Thread Edgar E. Iglesias
From: "Edgar E. Iglesias" Hi, This is a fix for the issue I'm seeing on ZynqMP with missmatched setup of the SMMU and the shared p2m page-tables with the CPU. This implementes a global p2m_ipa_bits cap for S2 input-size as discussed in the previous RFC. Best regards, Edgar Changelog: v2 -> v3

Re: [Xen-devel] [PATCHv3 3/4] xen: use ticket locks for spin locks

2015-04-30 Thread David Vrabel
On 30/04/15 11:09, Tim Deegan wrote: > At 00:56 +0100 on 30 Apr (1430355366), Jan Beulich wrote: > David Vrabel 04/29/15 5:28 PM >>> >>> On 29/04/15 00:15, Jan Beulich wrote: >>> David Vrabel 04/28/15 6:16 PM >>> > Are there any structures whose size you're particularly concerned abou

Re: [Xen-devel] [PATCH v2 2/3] xen/arm: Add p2m_pfn_bits

2015-04-30 Thread Edgar E. Iglesias
On Thu, Apr 30, 2015 at 10:39:19AM +0100, Julien Grall wrote: > Hi Edgar, > > On 30/04/2015 07:06, Edgar E. Iglesias wrote: > > val |= VTCR_T0SZ(0x18); /* 40 bit IPA */ > > val |= VTCR_SL0(0x1); /* P2M starts at first level */ > > #else /* CONFIG_ARM_64 */ > >@@ -1557,6 +1561,7 @@ void

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

2015-04-30 Thread osstest service user
flight 52629 xen-4.4-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/52629/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-armhf-libvirt 5 libvirt-build fail REGR. vs. 50429 test-amd64-amd64-

Re: [Xen-devel] [PATCH v3] x86: allow 64-bit PV guest kernels to suppress user mode exposure of M2P

2015-04-30 Thread Tim Deegan
Hi, At 15:31 +0100 on 24 Apr (1429889471), Jan Beulich wrote: > --- a/xen/arch/x86/mm/shadow/multi.c > +++ b/xen/arch/x86/mm/shadow/multi.c > @@ -1435,6 +1435,14 @@ void sh_install_xen_entries_in_l4(struct > shadow_l4e_from_mfn(page_to_mfn(d->arch.perdomain_l3_pg), >

Re: [Xen-devel] [v3 00/10] PML (Page Modification Logging) support

2015-04-30 Thread Tim Deegan
At 16:19 +0800 on 24 Apr (1429892368), Kai Huang wrote: > v2->v3: > > - Merged v2 patch 02 (document change) to patch 01 as a single patch, and > changed new parameter description as suggested by Andrew. > - changed vmx_vcpu_flush_pml_buffer to call mark_dirty for all logged GFNs, > and > cal

[Xen-devel] [PATCH 2/6] x86: move decision about clearing slowpath flag into arch_spin_lock()

2015-04-30 Thread Juergen Gross
The decision whether the slowpath flag is to be cleared for paravirtualized spinlocks is located in __ticket_check_and_clear_slowpath() today. Move that decision into arch_spin_lock() and add an unlikely attribute to it to avoid calling a function in case the compiler chooses not to inline __ticke

[Xen-devel] [PATCH 0/6] x86: reduce paravirtualized spinlock overhead

2015-04-30 Thread Juergen Gross
Paravirtualized spinlocks produce some overhead even if the kernel is running on bare metal. The main reason are the more complex locking and unlocking functions. Especially unlocking is no longer just one instruction but so complex that it is no longer inlined. This patch series addresses this is

[Xen-devel] [PATCH 5/6] x86: switch config from UNINLINE_SPIN_UNLOCK to INLINE_SPIN_UNLOCK

2015-04-30 Thread Juergen Gross
There is no need any more for a special treatment of _raw_spin_unlock() regarding inlining compared to the other spinlock functions. Just treat it like all the other spinlock functions. Remove selecting UNINLINE_SPIN_UNLOCK in case of PARAVIRT_SPINLOCKS. Signed-off-by: Juergen Gross --- arch/x8

[Xen-devel] [PATCH 4/6] x86: introduce new pvops function spin_unlock

2015-04-30 Thread Juergen Gross
To speed up paravirtualized spinlock handling when running on bare metal introduce a new pvops function "spin_unlock". This is a simple add instruction (possibly with lock prefix) when the kernel is running on bare metal. As the patched instruction includes a lock prefix in some configurations ann

[Xen-devel] [PATCH 6/6] x86: remove no longer needed paravirt_ticketlocks_enabled

2015-04-30 Thread Juergen Gross
With the paravirtualized spinlock unlock function being a pvops function paravirt_ticketlocks_enabled is no longer needed. Remove it. Signed-off-by: Juergen Gross --- arch/x86/include/asm/spinlock.h | 3 --- arch/x86/kernel/kvm.c| 14 -- arch/x86/kernel/paravirt

[Xen-devel] [PATCH 3/6] x86: introduce new pvops function clear_slowpath

2015-04-30 Thread Juergen Gross
To speed up paravirtualized spinlock handling when running on bare metal introduce a new pvops function "clear_slowpath". This is a nop when the kernel is running on bare metal. As the clear_slowpath function is common for all users add a new initialization function to set the pvops function point

[Xen-devel] [PATCH 1/6] x86: use macro instead of "0" for setting TICKET_SLOWPATH_FLAG

2015-04-30 Thread Juergen Gross
For paravirtualized spinlocks setting the "slowpath" flag in __ticket_enter_slowpath() is done via setting bit "0" in lock->tickets.head instead of using a macro. Change this by defining an appropriate macro. Signed-off-by: Juergen Gross --- arch/x86/include/asm/spinlock.h | 3 ++- arch/x

Re: [Xen-devel] [RFC PATCH v2 13/22] xen/arm: its: Add virtual ITS command support

2015-04-30 Thread Stefano Stabellini
On Thu, 30 Apr 2015, Julien Grall wrote: > On 30/04/2015 11:15, Stefano Stabellini wrote: > > > As said earlier, the number of DevBits implemented by the ITS can be > > > limited > > > (see GITS_TYPER.Devbits). > > > > > > If the devid is not within this range, the ITS won't recognize the value >

[Xen-devel] [PATCH v4] xen/pvh: use a custom IO bitmap for PVH hardware domains

2015-04-30 Thread Roger Pau Monne
Since a PVH hardware domain has access to the physical hardware create a custom more permissive IO bitmap. The permissions set on the bitmap are populated based on the contents of the ioports rangeset. Signed-off-by: Roger Pau Monné Cc: Jan Beulich Cc: Andrew Cooper Cc: Boris Ostrovsky Cc: Sur

Re: [Xen-devel] Xen 4.6 Development Update (three months reminder)

2015-04-30 Thread Fabio Fantoni
Il 29/04/2015 18:39, Jim Fehlig ha scritto: Fabio Fantoni wrote: Il 29/04/2015 02:21, Andrew Cooper ha scritto: On 28/04/15 23:17, Jim Fehlig wrote: Jim Fehlig wrote: wei.l...@citrix.com wrote: Hi all We are now three months into 4.6 development window. This is an email to keep track of all

Re: [Xen-devel] [RFC PATCH v2 13/22] xen/arm: its: Add virtual ITS command support

2015-04-30 Thread Julien Grall
On 30/04/2015 11:15, Stefano Stabellini wrote: As said earlier, the number of DevBits implemented by the ITS can be limited (see GITS_TYPER.Devbits). If the devid is not within this range, the ITS won't recognize the value and won't be able to send the interrupt. So this is clearly not the ri

Re: [Xen-devel] [RFC PATCH v2 13/22] xen/arm: its: Add virtual ITS command support

2015-04-30 Thread Stefano Stabellini
On Thu, 30 Apr 2015, Julien Grall wrote: > Hi Stefano, > > On 30/04/2015 11:02, Stefano Stabellini wrote: > > On Wed, 29 Apr 2015, Julien Grall wrote: > > > On 29/04/15 17:30, Vijay Kilari wrote: > > > > On Wed, Apr 29, 2015 at 9:56 PM, Vijay Kilari > > > > wrote: > > > > > On Wed, Apr 29, 2015 a

Re: [Xen-devel] [RFC PATCH v2 13/22] xen/arm: its: Add virtual ITS command support

2015-04-30 Thread Julien Grall
Hi Stefano, On 30/04/2015 11:02, Stefano Stabellini wrote: On Wed, 29 Apr 2015, Julien Grall wrote: On 29/04/15 17:30, Vijay Kilari wrote: On Wed, Apr 29, 2015 at 9:56 PM, Vijay Kilari wrote: On Wed, Apr 29, 2015 at 7:05 PM, Julien Grall wrote: On 29/04/15 12:56, Julien Grall wrote: As th

Re: [Xen-devel] [PATCHv3 3/4] xen: use ticket locks for spin locks

2015-04-30 Thread Tim Deegan
At 00:56 +0100 on 30 Apr (1430355366), Jan Beulich wrote: > >>> David Vrabel 04/29/15 5:28 PM >>> > >On 29/04/15 00:15, Jan Beulich wrote: > > David Vrabel 04/28/15 6:16 PM >>> > >>> Are there any structures whose size you're particularly concerned about? > >> > >> No specific ones (but of c

Re: [Xen-devel] [PATCH V3] xen: arm: X-Gene Storm check GIC DIST address for EOI quirk

2015-04-30 Thread Stefano Stabellini
On Wed, 29 Apr 2015, Pranavkumar Sawargaonkar wrote: > In old X-Gene Storm firmware and DT, secure mode addresses have been > mentioned in GICv2 node. In this case maintenance interrupt is used > instead of EOI HW method. > > This patch checks the GIC Distributor Base Address to enable EOI quirk >

Re: [Xen-devel] [RFC PATCH v2 13/22] xen/arm: its: Add virtual ITS command support

2015-04-30 Thread Stefano Stabellini
On Wed, 29 Apr 2015, Julien Grall wrote: > On 29/04/15 17:30, Vijay Kilari wrote: > > On Wed, Apr 29, 2015 at 9:56 PM, Vijay Kilari > > wrote: > >> On Wed, Apr 29, 2015 at 7:05 PM, Julien Grall > >> wrote: > >>> On 29/04/15 12:56, Julien Grall wrote: > As the 2 suggested approach don't see

Re: [Xen-devel] [PATCH v2 2/3] xen/arm: Add p2m_pfn_bits

2015-04-30 Thread Julien Grall
Hi Edgar, On 30/04/2015 07:06, Edgar E. Iglesias wrote: val |= VTCR_T0SZ(0x18); /* 40 bit IPA */ val |= VTCR_SL0(0x1); /* P2M starts at first level */ #else /* CONFIG_ARM_64 */ @@ -1557,6 +1561,7 @@ void __init setup_virt_paging(void) p2m_root_order = pa_range_info[pa_range]

Re: [Xen-devel] [PATCH v2 1/3] xen/arm: Re-order iommu_setup to after setup_virt_paging

2015-04-30 Thread Julien Grall
Hi Edgar, On 30/04/2015 07:06, Edgar E. Iglesias wrote: From: "Edgar E. Iglesias" This is needed to allow the paging setup to probe for pfn bit sizes to be used in p2m tables prior to iommu setup. Signed-off-by: Edgar E. Iglesias Reviewed-by: Julien Grall Regards, -- Julien Grall _

Re: [Xen-devel] [PATCH v3 2/5] xen: dt: add dt_for_each_range helper

2015-04-30 Thread Julien Grall
Hi Ian, On 20/04/2015 13:16, Ian Campbell wrote: This function iterates over a node's ranges property and calls a callback for each region. For now it only supplies the MMIO range (in terms of CPU addresses, i.e. already translated). Signed-off-by: Ian Campbell Reviewed-by: Julien Grall Re

Re: [Xen-devel] [PATCHv3 3/4] xen: use ticket locks for spin locks

2015-04-30 Thread Tim Deegan
At 18:00 +0100 on 29 Apr (1430330400), David Vrabel wrote: > On 29/04/15 17:56, Tim Deegan wrote: > > At 16:36 +0100 on 29 Apr (1430325362), David Vrabel wrote: > >> On 23/04/15 15:58, Jan Beulich wrote: > >> On 23.04.15 at 16:43, wrote: > At 14:54 +0100 on 23 Apr (1429800874), Jan Beulic

[Xen-devel] [seabios test] 52630: regressions - FAIL

2015-04-30 Thread osstest service user
flight 52630 seabios real [real] http://logs.test-lab.xenproject.org/osstest/logs/52630/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-pair 21 guest-migrate/src_host/dst_host fail REGR. vs. 36656 Tests which are failing i

Re: [Xen-devel] "don't bugger nd->seq" seems to break umount sometimes

2015-04-30 Thread Zefan Li
On 2015/4/29 20:21, Al Viro wrote: > On Wed, Apr 29, 2015 at 12:45:45PM +0100, Ian Jackson wrote: > >> The symptoms are that `umount' fails with EBUSY, > > [lizf: Backported to 3.4: > - remove the changes to follow_link() as it doesn't call set_root()] > > looks dubious - I don't have -