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

2015-05-05 Thread Stefano Stabellini
On Tue, 5 May 2015, Julien Grall wrote: On 05/05/15 11:39, Stefano Stabellini wrote: On Mon, 4 May 2015, Vijay Kilari wrote: On Thu, Apr 30, 2015 at 7:59 PM, Julien Grall julien.gr...@citrix.com wrote: Hi, On 30/04/15 14:47, Stefano Stabellini wrote: If the devid is not within

Re: [Xen-devel] [libvirt] [PATCH] libxl: support soundhw for hvm domains

2015-05-05 Thread Michal Privoznik
On 04.05.2015 22:35, Jim Fehlig wrote: The xend driver and the parsing/formating code in src/xenconfig have long supported soundhw. Add support in the libxl driver too. Signed-off-by: Jim Fehlig jfeh...@suse.com --- This patch continues the tradition of silently ignoring unsupported

Re: [Xen-devel] [PATCH] tools/libxc: Set HVM_PARAM_CONSOLE_EVTCHN during restore

2015-05-05 Thread Ian Campbell
On Thu, 2015-04-23 at 10:02 -0400, Konrad Rzeszutek Wilk wrote: On Thu, Apr 23, 2015 at 10:24:04AM +0100, Andrew Cooper wrote: On 23/04/15 03:49, Boris Ostrovsky wrote: When resuming, the guest needs to check whether the port has changed. HVM guests use this parameter to get the port

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

2015-05-05 Thread Ian Campbell
On Wed, 2015-04-29 at 15:08 +0530, 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

Re: [Xen-devel] [PATCH] libxl: convert strings and ints to xenbus_state

2015-05-05 Thread Ian Campbell
On Fri, 2015-04-24 at 10:17 +0100, Wei Liu wrote: On Fri, Apr 24, 2015 at 09:07:14AM +, Olaf Hering wrote: Convert all plain ints and strings which are used for xenbus state files to xenbus_state. This makes it easier to find code which deals with backend/frontend state changes.

Re: [Xen-devel] [PATCH] xen/arm: p2m: Add an ASSERT to check that p2m lock is taken in __p2m_lookup

2015-05-05 Thread Ian Campbell
On Mon, 2015-04-27 at 15:58 +0100, Julien Grall wrote: __p2m_lookup should be called with the p2m lock taken. Add an ASSERT in order to catch wrong caller in debug build. Signed-off-by: Julien Grall julien.gr...@citrix.com Acked + applied, thanks.

Re: [Xen-devel] [PATCH v6 12/14] tools/libxl: introduce some socket helpers

2015-05-05 Thread Ian Campbell
On Thu, 2015-04-23 at 17:55 +0800, Chao Peng wrote: Add libxl_socket_bitmap_alloc() to allow allocating a socket specific libxl_bitmap (as it is for cpu/node bitmap). Internal function libxl__count_physical_sockets() is introduced together to get the socket count when the size of bitmap is

Re: [Xen-devel] [PATCH] config: generate Paths-$SUBSYSTEM.mk

2015-05-05 Thread Ian Campbell
On Thu, 2015-04-23 at 11:21 +0100, Wei Liu wrote: On Thu, Apr 23, 2015 at 11:15:03AM +0100, Ian Campbell wrote: On Thu, 2015-04-23 at 10:10 +0100, Wei Liu wrote: ... so that we can run ./configure in each subsystem. Make inclusion of Paths-*.mk mandatory. Having done that does

Re: [Xen-devel] [PATCH] xen/arm: Make local_events_need_delivery working with idle VPCU

2015-05-05 Thread Stefano Stabellini
On Tue, 5 May 2015, Ian Campbell wrote: On Mon, 2015-04-27 at 17:32 +0100, Julien Grall wrote: diff --git a/xen/include/asm-arm/event.h b/xen/include/asm-arm/event.h index 5330dfe..0149d06 100644 --- a/xen/include/asm-arm/event.h +++ b/xen/include/asm-arm/event.h @@ -39,7 +39,12 @@

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

2015-05-05 Thread Vijay Kilari
On Tue, May 5, 2015 at 4:38 PM, Julien Grall julien.gr...@citrix.com wrote: On 05/05/15 11:28, Stefano Stabellini wrote: On Mon, 4 May 2015, Julien Grall wrote: Hi Vijay, On 04/05/2015 16:19, Vijay Kilari wrote: How did you implement the interrupt mode? Could it be improve? 1) In

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

2015-05-05 Thread Vijay Kilari
On Tue, May 5, 2015 at 5:17 PM, Julien Grall julien.gr...@citrix.com wrote: On 05/05/15 12:06, Vijay Kilari wrote: One MSI per domain is always consumed because each domain during ITS initialization creates Virtual ITS and sends basic initialization commands (MAPC) AFAICT, MAPC won't be

[Xen-devel] Xen/arm: Virtual ITS command queue handling

2015-05-05 Thread Vijay Kilari
Hi, As discussed, here is the design doc/txt. ARM GICv3 provides ITS (Interrupt Translation Service) feature to handle MSIx interrupts. Below are various mechanisms to handle ITS commands. ITS command completion detection mechanism:

Re: [Xen-devel] [PATCH] xen/arm: Make local_events_need_delivery working with idle VPCU

2015-05-05 Thread Ian Campbell
On Mon, 2015-04-27 at 17:32 +0100, Julien Grall wrote: diff --git a/xen/include/asm-arm/event.h b/xen/include/asm-arm/event.h index 5330dfe..0149d06 100644 --- a/xen/include/asm-arm/event.h +++ b/xen/include/asm-arm/event.h @@ -39,7 +39,12 @@ static inline int

Re: [Xen-devel] [PATCH v10 14/15] docs: libxc migration stream specification

2015-05-05 Thread Ian Campbell
On Thu, 2015-04-23 at 12:48 +0100, Andrew Cooper wrote: From: David Vrabel david.vra...@citrix.com Add the specification for a new migration stream format. The document includes all the details but to summarize: The existing (legacy) format is dependant on the word size of the toolstack.

Re: [Xen-devel] [PATCH v10 14/15] docs: libxc migration stream specification

2015-05-05 Thread Ian Campbell
On Tue, 2015-05-05 at 14:04 +0100, Andrew Cooper wrote: On 05/05/15 14:03, Ian Campbell wrote: On Thu, 2015-04-23 at 12:48 +0100, Andrew Cooper wrote: From: David Vrabel david.vra...@citrix.com Add the specification for a new migration stream format. The document includes all the

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

2015-05-05 Thread Ian Campbell
On Fri, 2015-05-01 at 11:40 +1000, 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. Test that the SMMU supports the P2M IPA bit size, use it if supported or bail

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

2015-05-05 Thread Julien Grall
On 05/05/15 12:06, Vijay Kilari wrote: One MSI per domain is always consumed because each domain during ITS initialization creates Virtual ITS and sends basic initialization commands (MAPC) AFAICT, MAPC won't be translated to a physical command. So not interrupt completion is necessary.

Re: [Xen-devel] [PATCH v10 09/15] tools/libxc: x86 PV restore code

2015-05-05 Thread Ian Campbell
On Thu, 2015-04-23 at 12:48 +0100, Andrew Cooper wrote: + * Set a pfn as populated, expanding the tracking structures if needed. To + * avoid realloc()ing too excessivly, the size increased to the nearest power excessively. Otherwise: Acked-by: Ian Campbell ian.campb...@citrix.com

Re: [Xen-devel] [PATCH v10 14/15] docs: libxc migration stream specification

2015-05-05 Thread Andrew Cooper
On 05/05/15 14:03, Ian Campbell wrote: On Thu, 2015-04-23 at 12:48 +0100, Andrew Cooper wrote: From: David Vrabel david.vra...@citrix.com Add the specification for a new migration stream format. The document includes all the details but to summarize: The existing (legacy) format is

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

2015-05-05 Thread Stefano Stabellini
On Tue, 5 May 2015, Julien Grall wrote: On 05/05/15 11:28, Stefano Stabellini wrote: On Mon, 4 May 2015, Julien Grall wrote: Hi Vijay, On 04/05/2015 16:19, Vijay Kilari wrote: How did you implement the interrupt mode? Could it be improve? 1) In physical ITS driver its_device

Re: [Xen-devel] [PATCH] xen/arm: Make local_events_need_delivery working with idle VPCU

2015-05-05 Thread Ian Campbell
On Tue, 2015-05-05 at 13:00 +0100, Julien Grall wrote: I'm inclined to just go with this patch for now, unless Stefano is nacking it. This patch seem to turn into a workaround, would it be better to move check idle_check in apply_p2m_check? I will prepare a follow-up to avoid properly

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

2015-05-05 Thread Jan Beulich
On 30.04.15 at 15:28, david.vra...@citrix.com 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 v10 02/15] libxc/progress: Extend the progress interface

2015-05-05 Thread Ian Campbell
On Thu, 2015-04-23 at 12:48 +0100, Andrew Cooper wrote: Progress information is logged via a different logger to regular libxc log messages, and currently can only express a range. However, not everything which needs reporting as progress comes with a range. Extend the interface to allow

Re: [Xen-devel] [PATCH v2 0/4] raisin: introduce tests

2015-05-05 Thread Ian Campbell
On Fri, 2015-05-01 at 16:48 +0100, Stefano Stabellini wrote: Hi all, this patch series introduces a framework to execute simple unit and functional tests in raisin. It can be used by developers to validate their changes before submitting a patch series to xen-devel. It can also be used by

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

2015-05-05 Thread Julien Grall
On 05/05/15 13:00, Vijay Kilari wrote: On Tue, May 5, 2015 at 5:17 PM, Julien Grall julien.gr...@citrix.com wrote: On 05/05/15 12:06, Vijay Kilari wrote: One MSI per domain is always consumed because each domain during ITS initialization creates Virtual ITS and sends basic initialization

Re: [Xen-devel] [PATCH] xen/arm: Make local_events_need_delivery working with idle VPCU

2015-05-05 Thread Julien Grall
Hi Ian, On 05/05/15 12:40, Ian Campbell wrote: On Mon, 2015-04-27 at 17:32 +0100, Julien Grall wrote: diff --git a/xen/include/asm-arm/event.h b/xen/include/asm-arm/event.h index 5330dfe..0149d06 100644 --- a/xen/include/asm-arm/event.h +++ b/xen/include/asm-arm/event.h @@ -39,7 +39,12 @@

Re: [Xen-devel] [PATCH] libxl: initialize vfb defbools in libxlMakeVfb

2015-05-05 Thread Ian Campbell
On Fri, 2015-05-01 at 09:10 -0600, Jim Fehlig wrote: Ian Campbell wrote: On Fri, 2015-04-17 at 13:57 -0600, Jim Fehlig wrote: On 04/17/2015 11:59 AM, Olaf Hering wrote: On Fri, Apr 17, Olaf Hering wrote: If the domU configu has sdl enabled libvirtd crashes:

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

2015-05-05 Thread Yu, Zhang
Thank you, Wei. On 5/5/2015 5:12 PM, Wei Liu wrote: On Mon, May 04, 2015 at 08:51:56PM +0800, Yu, Zhang wrote: Hi Wei, Thanks for your reply. On 5/4/2015 5:44 PM, Wei Liu wrote: (Thanks for trimming the CC list before hand) On Mon, May 04, 2015 at 02:05:49PM +0800, Yu, Zhang wrote: Hi

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

2015-05-05 Thread Ian Campbell
On Fri, 2015-05-01 at 11:40 +1000, Edgar E. Iglesias wrote: 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:

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

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

Re: [Xen-devel] [PATCH v6 12/14] tools/libxl: introduce some socket helpers

2015-05-05 Thread Ian Campbell
On Tue, 2015-05-05 at 13:43 +0100, Ian Campbell wrote: On Thu, 2015-04-23 at 17:55 +0800, Chao Peng wrote: Add libxl_socket_bitmap_alloc() to allow allocating a socket specific libxl_bitmap (as it is for cpu/node bitmap). Internal function libxl__count_physical_sockets() is introduced

Re: [Xen-devel] [PATCH v6 14/14] docs: add xl-psr.markdown

2015-05-05 Thread Ian Campbell
On Thu, 2015-04-23 at 17:55 +0800, Chao Peng wrote: +Maximum RMID and supported monitor types in the system can be obtained by: + +`xl psr-cmt-hwinfo` I think this is now `xl psr-hwinfo --cmt` ? +For example, assuming a system with 8 portions and 3 domains: + + * A CBM of 0xff for every

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

2015-05-05 Thread Ian Campbell
On Fri, 2015-05-01 at 11:40 +1000, Edgar E. Iglesias wrote: 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 +

Re: [Xen-devel] [PATCH v6 1/6] Qemu-Xen-vTPM: Support for Xen stubdom vTPM command line options

2015-05-05 Thread Xu, Quan
-Original Message- From: Eric Blake [mailto:ebl...@redhat.com] Sent: Tuesday, May 05, 2015 10:29 PM To: Xu, Quan; stefano.stabell...@eu.citrix.com; stef...@linux.vnet.ibm.com Cc: qemu-de...@nongnu.org; wei.l...@citrix.com; dgde...@tycho.nsa.gov; xen-devel@lists.xen.org Subject:

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

2015-05-05 Thread Edgar E. Iglesias
On Tue, May 05, 2015 at 02:17:47PM +0100, Ian Campbell wrote: On Fri, 2015-05-01 at 11:40 +1000, Edgar E. Iglesias wrote: 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

Re: [Xen-devel] [Embedded-pv-devel] Renaming Xen Project Hackathons

2015-05-05 Thread Dario Faggioli
On Tue, 2015-05-05 at 17:38 +0100, Lars Kurth wrote: The current proposal is to Rename the Xen Project Hackathons to either a) Xen Project Architecture Design Summit, b) or Xen Project Design Summit A number of people voted already for b) on

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

2015-05-05 Thread Edgar E. Iglesias
On Tue, May 05, 2015 at 02:23:23PM +0100, Ian Campbell wrote: On Fri, 2015-05-01 at 11:40 +1000, Edgar E. Iglesias wrote: 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

[Xen-devel] [PATCH v2] config: generate Paths-$SUBSYSTEM.mk

2015-05-05 Thread Wei Liu
... so that we can run ./configure in each subsystem. Make inclusion of Paths-*.mk mandatory. Skip generating a global Paths.mk since it's of no use anymore. Signed-off-by: Wei Liu wei.l...@citrix.com Cc: Ian Campbell ian.campb...@citrix.com Cc: Ian Jackson ian.jack...@eu.citrix.com Acked-by:

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

2015-05-05 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| 3 +++ xen/include/asm-arm/p2m.h | 3 +++ 2 files changed, 6 insertions(+) diff

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

2015-05-05 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. Reviewed-by: Julien Grall julien.gr...@citrix.com

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

2015-05-05 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 implements a global p2m_ipa_bits cap for S2 input-size as discussed in the previous RFC. Best regards,

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

2015-05-05 Thread Edgar E. Iglesias
On Tue, May 05, 2015 at 03:30:10PM +0100, Julien Grall wrote: On 05/05/15 14:59, Ian Campbell wrote: On Tue, 2015-05-05 at 14:48 +0100, Julien Grall wrote: Do we need to also check that we are configuring the same number of levels of PT etc, or is that already handled? The SMMU only

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

2015-05-05 Thread Julien Grall
Hi Pranav, On 29/04/15 10:38, 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

Re: [Xen-devel] [PATCH v4 4/5] vscsiif.h: add some notes about xenstore layout

2015-05-05 Thread Wei Liu
On Fri, Apr 17, 2015 at 08:30:59AM +, Olaf Hering wrote: Signed-off-by: Olaf Hering o...@aepfle.de Cc: Ian Campbell ian.campb...@citrix.com Cc: Ian Jackson ian.jack...@eu.citrix.com Cc: Jan Beulich jbeul...@suse.com Cc: Keir Fraser k...@xen.org Cc: Tim Deegan t...@xen.org ---

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

2015-05-05 Thread Ian Campbell
On Tue, 2015-05-05 at 14:48 +0100, Julien Grall wrote: Do we need to also check that we are configuring the same number of levels of PT etc, or is that already handled? The SMMU only care about the number of IPA bits. What ensures that the starting level of the SMMU matches the starting

Re: [Xen-devel] [PATCH v10 00/15] Migration v2 (libxc)

2015-05-05 Thread Ian Campbell
On Thu, 2015-04-23 at 12:48 +0100, Andrew Cooper wrote: Presented here is v10 of the Migration v2 series (libxc subset), which is able to function when transparently inserted under an unmodified xl/libxl. I've applied the v11 branch with my minor comments addressed which you provided privately.

Re: [Xen-devel] Next Document Day Moved to Wednesday, May 6

2015-05-05 Thread Russ Pavlicek
Reminder: Our Document Day is tomorrow! Theme for this month: No-Can-Do Without a HowTo We've got a lot of good information in the documentation, but it isn't always in a form that will help someone use a particular feature for the first time. Or, the information is present, but it might need to

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

2015-05-05 Thread Julien Grall
On 05/05/15 14:59, Ian Campbell wrote: On Tue, 2015-05-05 at 14:48 +0100, Julien Grall wrote: Do we need to also check that we are configuring the same number of levels of PT etc, or is that already handled? The SMMU only care about the number of IPA bits. What ensures that the starting

Re: [Xen-devel] [PATCH] OSSTEST: introduce a raisin build test

2015-05-05 Thread Ian Jackson
Ian Campbell writes (Re: [PATCH] OSSTEST: introduce a raisin build test): On Fri, 2015-04-24 at 16:46 +0100, Stefano Stabellini wrote: Signed-off-by: Stefano Stabellini stefano.stabell...@eu.citrix.com This looks like a good start, a few comments below. ... You should also call

Re: [Xen-devel] [libvirt] [PATCH] libxl: support soundhw for hvm domains

2015-05-05 Thread Jim Fehlig
Michal Privoznik wrote: On 04.05.2015 22:35, Jim Fehlig wrote: The xend driver and the parsing/formating code in src/xenconfig have long supported soundhw. Add support in the libxl driver too. Signed-off-by: Jim Fehlig jfeh...@suse.com --- This patch continues the tradition of

Re: [Xen-devel] [PATCH] xen/arm: p2m: Restrict preemption check in apply_p2m_changes

2015-05-05 Thread Ian Campbell
On Tue, 2015-05-05 at 16:02 +0100, Julien Grall wrote: The commit 569fb6c xen/arm: Data abort exception (R/W) mem_access events makes apply_p2m_changes to call hypercall_preempt_check for any operation rather than for relinquish. The function hypercall_preempt_check call

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

2015-05-05 Thread Tim Deegan
At 14:49 +0100 on 05 May (1430837355), Ian Campbell wrote: On Thu, 2015-04-30 at 16:33 +0100, David Vrabel wrote: 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. atomic.h is an odd place for it

Re: [Xen-devel] [PATCH v3 3/3] x86/hvm: Use white-lists for HVM param guest accessibility checks

2015-05-05 Thread Andrew Cooper
On 05/05/15 15:29, Jan Beulich wrote: On 05.05.15 at 16:09, paul.durr...@citrix.com wrote: From: Andrew Cooper [mailto:andrew.coop...@citrix.com] Sent: 05 May 2015 11:54 On 05/05/15 11:25, Paul Durrant wrote: @@ -5657,21 +5664,11 @@ static int hvm_allow_set_param(struct domain *d, case

Re: [Xen-devel] [PATCH v2 1/2] Introduce configure option --with-system-qemu-traditional

2015-05-05 Thread Ian Campbell
On Thu, 2015-04-16 at 12:24 +0100, Stefano Stabellini wrote: Introduce a configure option to disable the in-tree qemu-traditional build and use an externally provided qemu-dm binary. The option is very similar to the existing --with-system-qemu-xen. This overlaps a bit with the existing

Re: [Xen-devel] [PATCH v3 3/3] x86/hvm: Use white-lists for HVM param guest accessibility checks

2015-05-05 Thread Jan Beulich
On 05.05.15 at 16:43, andrew.coop...@citrix.com wrote: On 05/05/15 15:29, Jan Beulich wrote: On 05.05.15 at 16:09, paul.durr...@citrix.com wrote: From: Andrew Cooper [mailto:andrew.coop...@citrix.com] Sent: 05 May 2015 11:54 On 05/05/15 11:25, Paul Durrant wrote: @@ -5657,21 +5664,11 @@

Re: [Xen-devel] [PATCH OSSTEST v5 04/24] Debian: refactor code to add preseed commands to the preseed file

2015-05-05 Thread Ian Campbell
On Tue, 2015-05-05 at 10:57 +0100, Ian Campbell wrote: I did some testing of v6 over the weekend and fixed a couple of minor regressions which I caused to test-*-debianhvm-*. A final test is running now so I hope to post v6 either this afternoon or tomorrow AM. Unfortunately that test showed

Re: [Xen-devel] [PATCH 1/3] xen/vt-d: need barriers to workaround CLFLUSH

2015-05-05 Thread Boris Ostrovsky
On 05/04/2015 05:14 AM, Andrew Cooper wrote: On 04/05/2015 09:52, Jan Beulich wrote: On 04.05.15 at 04:16, tiejun.c...@intel.com wrote: --- a/xen/drivers/passthrough/vtd/x86/vtd.c +++ b/xen/drivers/passthrough/vtd/x86/vtd.c @@ -56,7 +56,9 @@ unsigned int get_cache_line_size(void) void

Re: [Xen-devel] stable release cadence

2015-05-05 Thread Lars Kurth
On 5 May 2015, at 16:44, Jan Beulich jbeul...@suse.com wrote: All, on the hackathon we discussed this topic. With the rate of changes going into the stable trees not being as high anymore as a few years ago, we settled on trying to stick to a 4 month release cycle going forward. +1

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

2015-05-05 Thread Ian Campbell
On Thu, 2015-04-30 at 16:33 +0100, David Vrabel wrote: 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

Re: [Xen-devel] Xen-unstable-staging: Xen BUG at iommu_map.c:455

2015-05-05 Thread Jan Beulich
On 05.05.15 at 17:17, t...@xen.org wrote: At 16:10 +0100 on 05 May (1430842206), Jan Beulich wrote: From what I can tell (and assuming other code works correctly) the fact that arch_iommu_populate_page_table() sets d-need_iommu to -1 first thing should make sure that any subsequent changes

[Xen-devel] [PATCH] xen/arm: p2m: Restrict preemption check in apply_p2m_changes

2015-05-05 Thread Julien Grall
The commit 569fb6c xen/arm: Data abort exception (R/W) mem_access events makes apply_p2m_changes to call hypercall_preempt_check for any operation rather than for relinquish. The function hypercall_preempt_check call local_events_need_delivery which rely on the current VCPU is not an idle VCPU.

Re: [Xen-devel] [PATCH V2] xen/arm64: Use virtual address when setting up early_printk fixmap

2015-05-05 Thread Julien Grall
Hi Ian, On 07/04/15 12:24, Chen Baozi wrote: We have already had the boot pagetable when reaching the point of early_printk fixmap setup. Thus there is no longer necessary to calculate physical address of xen_fixmap. Signed-off-by: Chen Baozi baoz...@gmail.com This patch has been fully ack

Re: [Xen-devel] Xen/arm: Virtual ITS command queue handling

2015-05-05 Thread Julien Grall
On 05/05/15 14:51, Stefano Stabellini wrote: - With this irq/LPI descriptor we can identify the domain/vITS. - Info of all the ongoing ITS requests(put in pITS Queue) of this domain is stored in ITS command status array (called its_requests). This is managed per vITS. 1) Trap of CWRITER

Re: [Xen-devel] [PATCH v3 3/3] x86/hvm: Use white-lists for HVM param guest accessibility checks

2015-05-05 Thread Paul Durrant
-Original Message- From: Andrew Cooper [mailto:andrew.coop...@citrix.com] Sent: 05 May 2015 11:54 To: Paul Durrant; xen-de...@lists.xenproject.org Cc: Keir (Xen.org); Jan Beulich Subject: Re: [PATCH v3 3/3] x86/hvm: Use white-lists for HVM param guest accessibility checks On

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

2015-05-05 Thread Boris Ostrovsky
On 05/05/2015 09:51 AM, Andy Lutomirski wrote: On Mon, May 4, 2015 at 8:02 AM, 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

Re: [Xen-devel] Xen/arm: Virtual ITS command queue handling

2015-05-05 Thread Julien Grall
On 05/05/15 13:14, Vijay Kilari wrote: Hi, Hi Vijay, As discussed, here is the design doc/txt. I will comment on the proposal 2 as it seems to be the preferred one assuming you are able to find why it's slow. Proposal 2: Here when guest writes command to vITS queue

Re: [Xen-devel] [PATCH v6 1/6] Qemu-Xen-vTPM: Support for Xen stubdom vTPM command line options

2015-05-05 Thread Eric Blake
On 05/04/2015 01:22 AM, Quan Xu wrote: Signed-off-by: Quan Xu quan...@intel.com --Changes in v6: -Remove stray insertion. --- configure| 14 ++ hmp.c| 2 ++ qapi-schema.json | 16 ++-- qemu-options.hx | 13 +++-- tpm.c|

Re: [Xen-devel] [PATCH] xen/arm: Make local_events_need_delivery working with idle VPCU

2015-05-05 Thread Julien Grall
On 05/05/15 13:29, Ian Campbell wrote: On Tue, 2015-05-05 at 13:00 +0100, Julien Grall wrote: I'm inclined to just go with this patch for now, unless Stefano is nacking it. This patch seem to turn into a workaround, would it be better to move check idle_check in apply_p2m_check? I will

Re: [Xen-devel] Xen-unstable-staging: Xen BUG at iommu_map.c:455

2015-05-05 Thread Jan Beulich
On 16.04.15 at 11:28, t...@xen.org wrote: At 22:35 +0100 on 11 Apr (1428791713), Andrew Cooper wrote: I am not certain that it is the correct way to fix the issue, nor that the ioreq server code is the only way to trigger it. There are several ways to shoot a gfn mapping from the guests

Re: [Xen-devel] Xen-unstable-staging: Xen BUG at iommu_map.c:455

2015-05-05 Thread Tim Deegan
At 16:10 +0100 on 05 May (1430842206), Jan Beulich wrote: On 16.04.15 at 11:28, t...@xen.org wrote: At 22:35 +0100 on 11 Apr (1428791713), Andrew Cooper wrote: I am not certain that it is the correct way to fix the issue, nor that the ioreq server code is the only way to trigger it. There

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

2015-05-05 Thread Ian Campbell
On Thu, 2015-04-30 at 16:33 +0100, David Vrabel wrote: 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. atomic.h is an odd place for it then ;-) But given you use the infrastructure I can't suggest a

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

2015-05-05 Thread Julien Grall
Hi Ian, On 05/05/15 14:24, Ian Campbell wrote: On Fri, 2015-05-01 at 11:40 +1000, 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. Test that the SMMU supports the

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

2015-05-05 Thread David Vrabel
On 05/05/15 14:56, Ian Campbell wrote: On Thu, 2015-04-30 at 16:33 +0100, David Vrabel wrote: void _spin_lock_irq(spinlock_t *lock) { -LOCK_PROFILE_VAR; - ASSERT(local_irq_is_enabled()); local_irq_disable(); -check_lock(lock-debug); -while (

Re: [Xen-devel] [PATCH v3 3/3] x86/hvm: Use white-lists for HVM param guest accessibility checks

2015-05-05 Thread Jan Beulich
On 05.05.15 at 16:09, paul.durr...@citrix.com wrote: From: Andrew Cooper [mailto:andrew.coop...@citrix.com] Sent: 05 May 2015 11:54 On 05/05/15 11:25, Paul Durrant wrote: @@ -5657,21 +5664,11 @@ static int hvm_allow_set_param(struct domain *d, case HVM_PARAM_IDENT_PT: case

[Xen-devel] stable release cadence

2015-05-05 Thread Jan Beulich
All, on the hackathon we discussed this topic. With the rate of changes going into the stable trees not being as high anymore as a few years ago, we settled on trying to stick to a 4 month release cycle going forward. Of course, issues like the currently severely lagging osstest can always lead

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

2015-05-05 Thread Andy Lutomirski
On Mon, May 4, 2015 at 8:02 AM, 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

Re: [Xen-devel] Xen/arm: Virtual ITS command queue handling

2015-05-05 Thread Stefano Stabellini
On Tue, 5 May 2015, Vijay Kilari wrote: Proposal 2: Here when guest writes command to vITS queue and updates CWRITER registers, it is trapped in XEN and below steps are followed to process ITS command - Dom0 creates a ITS completion device with device id (00:00.1) and

Re: [Xen-devel] [PATCH v4 3/5] libxl: add support for vscsi

2015-05-05 Thread Wei Liu
On Fri, Apr 17, 2015 at 08:30:58AM +, Olaf Hering wrote: Port pvscsi support from xend to libxl: vscsi=['pdev,vdev{,options}'] xl scsi-attach xl scsi-detach xl scsi-list Signed-off-by: Olaf Hering o...@aepfle.de Cc: Ian Jackson ian.jack...@eu.citrix.com Cc: Stefano Stabellini

Re: [Xen-devel] [PATCH v2 0/4] raisin: introduce tests

2015-05-05 Thread Stefano Stabellini
On Tue, 5 May 2015, Ian Campbell wrote: On Fri, 2015-05-01 at 16:48 +0100, Stefano Stabellini wrote: Hi all, this patch series introduces a framework to execute simple unit and functional tests in raisin. It can be used by developers to validate their changes before submitting a patch

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

2015-05-05 Thread Ian Campbell
On Thu, 2015-04-30 at 16:33 +0100, David Vrabel wrote: void _spin_lock_irq(spinlock_t *lock) { -LOCK_PROFILE_VAR; - ASSERT(local_irq_is_enabled()); local_irq_disable(); -check_lock(lock-debug); -while ( unlikely(!_raw_spin_trylock(lock-raw)) ) -{ -

Re: [Xen-devel] [PATCH v3 3/3] x86/hvm: Use white-lists for HVM param guest accessibility checks

2015-05-05 Thread Andrew Cooper
On 05/05/15 15:09, Paul Durrant wrote: -Original Message- From: Andrew Cooper [mailto:andrew.coop...@citrix.com] Sent: 05 May 2015 11:54 To: Paul Durrant; xen-de...@lists.xenproject.org Cc: Keir (Xen.org); Jan Beulich Subject: Re: [PATCH v3 3/3] x86/hvm: Use white-lists for HVM param

Re: [Xen-devel] [PATCH] OSSTEST: introduce a raisin build test

2015-05-05 Thread Stefano Stabellini
On Tue, 5 May 2015, Ian Campbell wrote: On Fri, 2015-04-24 at 16:46 +0100, Stefano Stabellini wrote: Signed-off-by: Stefano Stabellini stefano.stabell...@eu.citrix.com This looks like a good start, a few comments below. diff --git a/ap-common b/ap-common index 64749e3..985eeec 100644

[Xen-devel] (release) versioning

2015-05-05 Thread Jan Beulich
All, on the hackathon we also discussed possibly changing the versioning of Xen. The main rationale for the proposal is that (just like in many other software projects) version numbers (in particular the major one) currently don't really convey much information. The proposal is to take gcc's new

Re: [Xen-devel] Xen/arm: Virtual ITS command queue handling

2015-05-05 Thread Vijay Kilari
On Tue, May 5, 2015 at 7:21 PM, Stefano Stabellini stefano.stabell...@eu.citrix.com wrote: On Tue, 5 May 2015, Vijay Kilari wrote: Proposal 2: Here when guest writes command to vITS queue and updates CWRITER registers, it is trapped in XEN and below steps are followed to

[Xen-devel] [PATCH v4 0/3] x86/hvm: HVMOP_get/set_param improvements

2015-05-05 Thread Paul Durrant
The following 3 patches re-structure the code implementing HVMOP_set_param and HVMOP_get_param. Patch #1 gives each operation its own function Patch #2 splits out checks for getting/setting non-reflexive params and setting params with change-once semantics, as well as the XSM check into separate

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

2015-05-05 Thread David Vrabel
On 04/05/15 16:02, Boris Ostrovsky wrote: --- a/arch/x86/xen/enlighten.c +++ b/arch/x86/xen/enlighten.c @@ -1760,6 +1760,9 @@ static struct notifier_block xen_hvm_cpu_notifier = { static void __init xen_hvm_guest_init(void) { + if (xen_pv_domain()) + return; + I

Re: [Xen-devel] [PATCH 1/3] xen/vt-d: need barriers to workaround CLFLUSH

2015-05-05 Thread Jan Beulich
On 05.05.15 at 17:46, boris.ostrov...@oracle.com wrote: On 05/04/2015 05:14 AM, Andrew Cooper wrote: On 04/05/2015 09:52, Jan Beulich wrote: On 04.05.15 at 04:16, tiejun.c...@intel.com wrote: --- a/xen/drivers/passthrough/vtd/x86/vtd.c +++ b/xen/drivers/passthrough/vtd/x86/vtd.c @@ -56,7

[Xen-devel] [PATCH v4 3/3] x86/hvm: Use white-lists for HVM param guest accessibility checks

2015-05-05 Thread Paul Durrant
There are actually very few HVM parameters that a guest needs to read and even fewer that a guest needs to write. Use white-lists to specify those parameters and also ensre that, by default, newly introduced parameters are not accessible. Signed-off-by: Paul Durrant paul.durr...@citrix.com Cc:

[Xen-devel] [PATCH v4 1/3] x86/hvm: give HVMOP_set_param and HVMOP_get_param their own functions

2015-05-05 Thread Paul Durrant
The level of switch nesting in those ops is getting unreadable. Giving them their own functions does introduce some code duplication in the the pre-op checks but the overall result is easier to follow. This patch is code movement (including style fixes). There is no functional change.

Re: [Xen-devel] [PATCH V2] xen/arm64: Use virtual address when setting up early_printk fixmap

2015-05-05 Thread Ian Campbell
On Fri, 2015-04-17 at 19:39 +0800, Chen Baozi wrote: Acked-by: Ian Campbell ian.campb...@citrix.com If it is confirmed that I got the correct meaning in the first one I can fix both on commit. Yes, please. Thanks. Done and pushed, thanks.

Re: [Xen-devel] [PATCH] xen/arm: p2m: Restrict preemption check in apply_p2m_changes

2015-05-05 Thread Konrad Rzeszutek Wilk
On Tue, May 05, 2015 at 04:02:09PM +0100, Julien Grall wrote: The commit 569fb6c xen/arm: Data abort exception (R/W) mem_access events makes apply_p2m_changes to call hypercall_preempt_check for any operation rather than for relinquish. The function hypercall_preempt_check call

[Xen-devel] [PATCH v4 2/3] x86/hvm: introduce functions for HVMOP_get/set_param allowance checks

2015-05-05 Thread Paul Durrant
Some parameters can only (validly) be set once. Some should not be set by a guest for its own domain, and others must not be set since they require the domain to be paused. Consolidate these checks, along with the XSM check, in a new hvm_allow_set_param() function for clarity. Also, introduce

Re: [Xen-devel] [PATCH v4 3/3] x86/hvm: Use white-lists for HVM param guest accessibility checks

2015-05-05 Thread Andrew Cooper
On 05/05/15 16:47, Paul Durrant wrote: There are actually very few HVM parameters that a guest needs to read and even fewer that a guest needs to write. Use white-lists to specify those parameters and also ensre that, by default, newly introduced parameters are not accessible. Signed-off-by:

Re: [Xen-devel] [PATCH] OSSTEST: introduce a raisin build test

2015-05-05 Thread Ian Jackson
Stefano Stabellini writes (Re: [PATCH] OSSTEST: introduce a raisin build test): I agree that revisions should be passed to ts-raisin-build by osstest, as Ian suggested. AFAICT that should be enough to meet all your criteria. You also need to call store_revision for all the trees involved, as

Re: [Xen-devel] (release) versioning

2015-05-05 Thread Andrew Cooper
On 05/05/15 16:54, Jan Beulich wrote: All, on the hackathon we also discussed possibly changing the versioning of Xen. The main rationale for the proposal is that (just like in many other software projects) version numbers (in particular the major one) currently don't really convey much

Re: [Xen-devel] [PATCH 1/3] xen/vt-d: need barriers to workaround CLFLUSH

2015-05-05 Thread Boris Ostrovsky
On 05/05/2015 11:58 AM, Jan Beulich wrote: On 05.05.15 at 17:46, boris.ostrov...@oracle.com wrote: On 05/04/2015 05:14 AM, Andrew Cooper wrote: On 04/05/2015 09:52, Jan Beulich wrote: On 04.05.15 at 04:16, tiejun.c...@intel.com wrote: --- a/xen/drivers/passthrough/vtd/x86/vtd.c +++

[Xen-devel] Renaming Xen Project Hackathons

2015-05-05 Thread Lars Kurth
Hi all, one piece of feedback we got at the last and this Hackathon, was that the name for the event is really misleading. Although some Hacking goes on, in reality the event is really more of an Architecture, Design and Deployment Summit. There was some discussion about this topic last year,

Re: [Xen-devel] [PATCH] xen/arm: p2m: Restrict preemption check in apply_p2m_changes

2015-05-05 Thread Julien Grall
Hi Konrad, On 05/05/15 16:50, Konrad Rzeszutek Wilk wrote: --- xen/arch/arm/p2m.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/xen/arch/arm/p2m.c b/xen/arch/arm/p2m.c index 65efa94..59dd23a 100644 --- a/xen/arch/arm/p2m.c +++ b/xen/arch/arm/p2m.c @@ -948,6

Re: [Xen-devel] Xen/arm: Virtual ITS command queue handling

2015-05-05 Thread Vijay Kilari
On Tue, May 5, 2015 at 7:39 PM, Julien Grall julien.gr...@citrix.com wrote: On 05/05/15 13:14, Vijay Kilari wrote: Hi, Hi Vijay, As discussed, here is the design doc/txt. I will comment on the proposal 2 as it seems to be the preferred one assuming you are able to find why it's slow.

Re: [Xen-devel] Xen/arm: Virtual ITS command queue handling

2015-05-05 Thread Julien Grall
On 05/05/15 17:09, Vijay Kilari wrote: On Tue, May 5, 2015 at 7:39 PM, Julien Grall julien.gr...@citrix.com wrote: On 05/05/15 13:14, Vijay Kilari wrote: Proposal 2: Here when guest writes command to vITS queue and updates CWRITER registers, it is trapped in XEN and below

  1   2   >