[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

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 edgar.igles...@xilinx.com 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 edgar.igles...@xilinx.com

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 -stable

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 ian.campb...@citrix.com

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 vijay.kil...@gmail.com wrote: On Wed, Apr 29, 2015

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 and won't

[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 jgr...@suse.com --- arch/x86/include/asm/spinlock.h | 3

[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

[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

[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 jgr...@suse.com --- arch/x86/include/asm/spinlock.h | 3 --- arch/x86/kernel/kvm.c| 14 --

[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

[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

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 call

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 =

[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

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 david.vra...@citrix.com 04/29/15 5:28 PM On 29/04/15 00:15, Jan Beulich wrote: David Vrabel david.vra...@citrix.com 04/28/15 6:16 PM Are there any structures whose size you're particularly concerned about? No

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 vijay.kil...@gmail.com wrote: On Wed, Apr 29, 2015 at 7:05 PM, Julien Grall julien.gr...@citrix.com wrote:

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

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 vijay.kil...@gmail.com wrote: On Wed, Apr 29, 2015 at 7:05 PM, Julien Grall julien.gr...@citrix.com wrote: On 29/04/15 12:56, Julien Grall wrote: As the 2

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 for

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

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),

[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é roger@citrix.com Cc: Jan Beulich jbeul...@suse.com Cc:

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 __init

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 david.vra...@citrix.com 04/29/15 5:28 PM On 29/04/15 00:15, Jan Beulich wrote: David Vrabel david.vra...@citrix.com 04/28/15 6:16 PM Are there any structures whose size you'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
From: Edgar E. Iglesias edgar.igles...@xilinx.com 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 edgar.igles...@xilinx.com ---

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

2015-04-30 Thread David Vrabel
From: Christoph Egger cheg...@amazon.de 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

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

2015-04-30 Thread David Vrabel
From: Malcolm Crossley malcolm.cross...@citrix.com 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

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

2015-04-30 Thread David Vrabel
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

[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

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 stefano.stabell...@eu.citrix.com 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

[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 edgar.igles...@xilinx.com 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 julien.gr...@citrix.com Signed-off-by: Edgar E. Iglesias edgar.igles...@xilinx.com ---

[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 edgar.igles...@xilinx.com 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,

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

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

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 stefano.stabell...@eu.citrix.com 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

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_TYPER.Devbits. Vijay, what is

[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

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.

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 rather

[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

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 mcg...@suse.com 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

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 mcg...@suse.com This allows drivers to take advantage of write-combining when possible. Ideally we'd

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

2015-04-30 Thread Don Slutz
Signed-off-by: Don Slutz dsl...@verizon.com (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 ---

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

2015-04-30 Thread Luis R. Rodriguez
From: Luis R. Rodriguez mcg...@suse.com 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

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 mcg...@do-not-panic.com 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

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 mcg...@suse.com 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 mcg...@suse.com This allows

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 mcg...@suse.com 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.

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 mcg...@suse.com Now that we have pci_iomap_wc() add the respective devres helpers. I guess I'm still

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 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 to

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 can be

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 qxl [

[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

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 Zijlstrapet...@infradead.org 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

[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 mcg...@suse.com 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.

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 boris.ostrov...@oracle.com 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

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.

[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 mcg...@suse.com 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.

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 boris.ostrov...@oracle.com wrote: On 04/30/2015 03:17 PM, Andy Lutomirski wrote: On Thu, Apr 30, 2015 at 12:08 PM, Boris Ostrovsky boris.ostrov...@oracle.com wrote: Commit 61f01dd941ba (x86_64, asm: Work around AMD SYSRET SS descriptor

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 boris.ostrov...@oracle.com 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

[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

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 miss

[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 edgar.igles...@xilinx.com 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-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

[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 edgar.igles...@xilinx.com 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,

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 edgar.igles...@xilinx.com The Stage2 input-size must match what the CPU uses because the SMMU and the CPU share page-tables. Assert that the

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

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

[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 edgar.igles...@xilinx.com 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 julien.gr...@citrix.com Signed-off-by: Edgar E. Iglesias edgar.igles...@xilinx.com ---

[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

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, and

[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

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 boris.ostrov...@oracle.com wrote: On 04/30/2015 03:17 PM, Andy Lutomirski wrote: On Thu, Apr 30, 2015 at 12:08 PM, Boris Ostrovsky boris.ostrov...@oracle.com wrote: Commit 61f01dd941ba (x86_64,

[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

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é roger@citrix.com

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 mcg...@suse.com 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

[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

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 malcolm.cross...@citrix.com 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

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 yum

[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 david.vra...@citrix.com --- xen/include/asm-x86/domain.h |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/xen/include/asm-x86/domain.h

[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 david.vra...@citrix.com --- xen/include/asm-x86/system.h | 48 ++ 1 file changed, 48 insertions(+) diff

[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

[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-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

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 edgar.igles...@xilinx.com 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

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.

[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.

[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 david.vra...@citrix.com Reviewed-by: Tim Deegan t...@xen.org Acked-by: Jan Beulich jbeul...@suse.com ---

[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-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 david.vra...@citrix.com --- xen/include/asm-x86/hvm/domain.h | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git

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

2015-04-30 Thread Edgar E. Iglesias
From: Edgar E. Iglesias edgar.igles...@xilinx.com 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_pfn_bits cap for S2 input-size as discussed in the previous RFC. Best regards,

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

2015-04-30 Thread Edgar E. Iglesias
From: Edgar E. Iglesias edgar.igles...@xilinx.com 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 given pfn bit size is supported by the SMMU and use it. Signed-off-by: Edgar E. Iglesias

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

2015-04-30 Thread Edgar E. Iglesias
From: Edgar E. Iglesias edgar.igles...@xilinx.com Export p2m_pfn_bits holding the bit size of pfn addresses used in p2m tables. Signed-off-by: Edgar E. Iglesias edgar.igles...@xilinx.com --- xen/arch/arm/p2m.c| 5 + xen/include/asm-arm/p2m.h | 3 +++ 2 files changed, 8 insertions(+)

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

2015-04-30 Thread Edgar E. Iglesias
From: Edgar E. Iglesias edgar.igles...@xilinx.com 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 edgar.igles...@xilinx.com --- xen/arch/arm/setup.c | 4 ++-- 1 file changed, 2 insertions(+), 2

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 edgar.igles...@xilinx.com 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.