Re: [Xen-devel] [PATCH v3] PCI: Add guard to avoid mapping a invalid msix base address

2015-02-02 Thread Bjorn Helgaas
[+cc Jan] On Thu, Jan 29, 2015 at 11:54:43AM +0800, Yijing Wang wrote: Sometimes, a pci bridge device BAR was not assigned properly. After we call pci_bus_assign_resources(), the resource of the BAR would be reseted. So if we try to enable msix for this device, it will map a invalid resource

[Xen-devel] [qemu-upstream-unstable bisection] complete test-amd64-i386-freebsd10-i386

2015-02-02 Thread xen . org
branch xen-unstable xen branch xen-unstable job test-amd64-i386-freebsd10-i386 test guest-localmigrate Tree: linux git://xenbits.xen.org/linux-pvops.git Tree: linuxfirmware git://xenbits.xen.org/osstest/linux-firmware.git Tree: qemu git://xenbits.xen.org/staging/qemu-xen-unstable.git Tree: qemuu

Re: [Xen-devel] [PATCH 0/6] xen/arm: Move in/out code to/from init section

2015-02-02 Thread Julien Grall
On 02/02/15 11:15, Jan Beulich wrote: On 02.02.15 at 11:58, ian.campb...@citrix.com wrote: On Fri, 2015-01-30 at 11:33 +, Julien Grall wrote: On 30/01/15 11:30, Ian Campbell wrote: On Thu, 2015-01-29 at 18:32 +, Julien Grall wrote: Hello, Ping? Any more review for this version of

[Xen-devel] [PATCH 3/7] stubdom: don't look for mini-os source file during configure

2015-02-02 Thread Wei Liu
Mini-os source code will be fetched during build. Please rerun autogen.sh after applying this patch. Signed-off-by: Wei Liu wei.l...@citrix.com Cc: Ian Campbell ian.campb...@citrix.com Cc: Ian Jackson ian.jack...@eu.citrix.com --- stubdom/configure.ac | 1 - 1 file changed, 1 deletion(-) diff

[Xen-devel] [PATCH 5/7] Mini-OS: standalone build

2015-02-02 Thread Wei Liu
In order to keep the tree bisectable all the changes are done in one single commit. Things done in this commit: 1. Import necessary .mk files from Xen. 2. Move all XEN_ related variables to MINIOS_ namespace. 3. Import Xen public header files. 4. Import BSD's list.h and helper script. Mini-OS's

[Xen-devel] [PATCH 4/7] git-checkout.sh: use mkdir -p

2015-02-02 Thread Wei Liu
Otherwise mkdir extras/mini-os fails because extras doesn't exist. Signed-off-by: Wei Liu wei.l...@citrix.com Cc: Ian Campbell ian.campb...@citrix.com Cc: Ian Jackson ian.jack...@eu.citrix.com --- scripts/git-checkout.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[Xen-devel] [PATCH 6/7] build system: stubdom targets now depends on mini-os target

2015-02-02 Thread Wei Liu
Provide mini-os url and revision in Config.mk Introduce Makefile.mini-os which contains mini-os specific targets. Target mini-os-dir clones mini-os tree from upstream. Make stubdom targets depend on mini-os-dir target. Make subtree-force-update{,-all} depend on mini-os-dir-force-update. Also

[Xen-devel] [PATCH v2] xen/evtchn: Alter the alloc/free xen event channel functions to take a domain

2015-02-02 Thread Andrew Cooper
The resource behind an event channel is domain centric rather than vcpu centric. This change allows mem_event_disable() to avoid arbitrarily referencing d-vcpu[0] just to pass the domain. No functional change. Signed-off-by: Andrew Cooper andrew.coop...@citrix.com CC: Keir Fraser k...@xen.org

Re: [Xen-devel] [OSSTEST PATCH v4 8/9] make-flight: factor out do_pv_debian_tests

2015-02-02 Thread Ian Campbell
On Wed, 2014-12-10 at 22:21 +, Wei Liu wrote: Signed-off-by: Wei Liu wei.l...@citrix.com --- make-flight | 21 ++--- 1 file changed, 14 insertions(+), 7 deletions(-) diff --git a/make-flight b/make-flight index 9963a46..35904be 100755 --- a/make-flight +++

Re: [Xen-devel] [PATCH] docs: create reproducible html

2015-02-02 Thread Ian Jackson
Olaf Hering writes (Re: [PATCH] docs: create reproducible html): On Mon, Feb 02, Ian Jackson wrote: What do you think ? Thats overkill. The maintainers of make removed the internal sort with 3.82, likely for performance reasons. Sorted lists are required only in a few places. The Xen

Re: [Xen-devel] [PATCH OSSTEST] ts-xen-build-prep: install nasm

2015-02-02 Thread Ian Campbell
On Mon, 2015-02-02 at 10:38 +, Wei Liu wrote: Ian Campbell ian.campb...@citrix.com writes: On Sat, 2015-01-31 at 22:51 +, Wei Liu wrote: OVMF requires nasm to build. I suppose this is a new requirement added by the mainline branch we just added? Yes. The log in the latest

Re: [Xen-devel] support for more than 32 VCPUs when migrating PVHVM guest

2015-02-02 Thread Konrad Rzeszutek Wilk
On Mon, Feb 02, 2015 at 12:03:28PM +0100, Vitaly Kuznetsov wrote: Andrew Cooper andrew.coop...@citrix.com writes: On 02/02/15 10:47, Vitaly Kuznetsov wrote: Hi Konrad, I just hit an issue with PVHVM guests after save/restore (or migration), if a PVHVM guest has 32 VCPUs it hangs.

Re: [Xen-devel] [PATCH v2 4/5] IOMMU/x86: correct page_list_first() use

2015-02-02 Thread Andrew Cooper
On 02/02/15 11:20, Jan Beulich wrote: Comparing its result against NULL is unsafe when page lists use normal list entries for linking together - page_list_empty() needs to be used instead. Signed-off-by: Jan Beulich jbeul...@suse.com Reviewed-by: Andrew Cooper andrew.coop...@citrix.com

Re: [Xen-devel] [PATCH 0/6] xen/arm: Move in/out code to/from init section

2015-02-02 Thread Jan Beulich
On 02.02.15 at 13:52, julien.gr...@linaro.org wrote: On 02/02/15 11:15, Jan Beulich wrote: On 02.02.15 at 11:58, ian.campb...@citrix.com wrote: On Fri, 2015-01-30 at 11:33 +, Julien Grall wrote: On 30/01/15 11:30, Ian Campbell wrote: On Thu, 2015-01-29 at 18:32 +, Julien Grall wrote:

Re: [Xen-devel] [PATCH v2 17/29] Ovmf/Xen: refactor XenBusDxe hypercall implementation

2015-02-02 Thread Laszlo Ersek
On 01/27/15 14:10, Ard Biesheuvel wrote: On 27 January 2015 at 12:46, Ard Biesheuvel ard.biesheu...@linaro.org wrote: On 27 January 2015 at 12:43, Stefano Stabellini stefano.stabell...@eu.citrix.com wrote: On Mon, 26 Jan 2015, Ard Biesheuvel wrote: This refactors the Xen hypercall

Re: [Xen-devel] apic-v reduce network performance in my test case

2015-02-02 Thread Liuqiming (John)
Hi Jan, Thanks for the reply. On 2015/2/2 18:12, Jan Beulich wrote: On 31.01.15 at 11:29, john.liuqim...@huawei.com wrote: Recently I met an odd performance problem: when I turn on APIC Virtualization feature (apicv=1), the network performance of a windows guest become worse.

[Xen-devel] [PATCH] docs: create reproducible html

2015-02-02 Thread Olaf Hering
The Makefile uses wildcard to collect a list of files. The resulting list of files is in directory order, which is random. As a result the generated html files will differ when build on different hosts. Use the built-in sort function to get a stable list of files. Signed-off-by: Olaf Hering

Re: [Xen-devel] [OSSTEST PATCH v4 8/9] make-flight: factor out do_pv_debian_tests

2015-02-02 Thread Ian Campbell
On Mon, 2015-02-02 at 13:18 +, Ian Campbell wrote: On Wed, 2014-12-10 at 22:21 +, Wei Liu wrote: Signed-off-by: Wei Liu wei.l...@citrix.com --- make-flight | 21 ++--- 1 file changed, 14 insertions(+), 7 deletions(-) diff --git a/make-flight b/make-flight

Re: [Xen-devel] [OSSTEST PATCH v4 0/9] XSM test case for OSSTest

2015-02-02 Thread Ian Campbell
On Wed, 2014-12-10 at 22:21 +, Wei Liu wrote: Hi all This patch series attempts to duplicate some Debian test cases for XSM. This is version 4 of this series. Apart from any rebasing issues relating to its age I think this series looks good.

Re: [Xen-devel] rcu_sched self-detect stall when disable vif device

2015-02-02 Thread Julien Grall
Hi David, On 30/01/15 16:04, David Vrabel wrote: How about this? This is working for me. Thanks! 8-- xen-netback: stop the guest rx thread after a fatal error After commit e9d8b2c2968499c1f96563e6522c56958d5a1d0d (xen-netback: disable rogue vif in

[Xen-devel] [PATCH OSSTEST] README.dev: Updates to commissioning process

2015-02-02 Thread Ian Campbell
- mkpxedir requires the resource to be allocated - mention the need to bless the hosts for comissioning run, and to add proper blessings afterwards. - run mg-execute-flight in screen Signed-off-by: Ian Campbell ian.campb...@citrix.com --- README.dev | 17 - mg-hosts |3

Re: [Xen-devel] [PATCH 1/2] sched: credit2: respect per-vcpu hard affinity

2015-02-02 Thread Dario Faggioli
On Sat, 2015-01-31 at 20:51 -1000, Justin Weaver wrote: On Mon, Jan 19, 2015 at 9:21 PM, Justin Weaver jtwea...@hawaii.edu wrote: On Mon, Jan 12, 2015 at 8:05 AM, Dario Faggioli For example... I start a guest with one vcpu with hard affinity 8 - 15 and xl vcpu-list says it's running on pcpu

Re: [Xen-devel] [PATCH OSSTEST] README.dev: Updates to commissioning process

2015-02-02 Thread Ian Campbell
On Mon, 2015-02-02 at 14:22 +, Ian Jackson wrote: Ian Campbell writes ([PATCH OSSTEST] README.dev: Updates to commissioning process): - mkpxedir requires the resource to be allocated - mention the need to bless the hosts for comissioning run, and to add proper blessings afterwards.

Re: [Xen-devel] [OSSTEST PATCH v4 4/9] mfi-common: create build-$arch-xsm job

2015-02-02 Thread Ian Campbell
On Wed, 2014-12-10 at 22:21 +, Wei Liu wrote: Signed-off-by: Wei Liu wei.l...@citrix.com Acked-by:Ian Campbell ian.campb...@citrix.com ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [OSSTEST PATCH v4 7/9] ts-xen-install: install Xen with XSM support if requested

2015-02-02 Thread Ian Campbell
On Wed, 2014-12-10 at 22:21 +, Wei Liu wrote: Signed-off-by: Wei Liu wei.l...@citrix.com Acked-by: Ian Campbell ian.campb...@citrix.com ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH RFC] libxl: set disk defaults in remove/destroy functions

2015-02-02 Thread Ian Campbell
On Mon, 2015-01-26 at 16:14 -0700, Jim Fehlig wrote: Cc-ing the other toolstack maintainers, both of whom have more familiarity with this part of libxl than I. The attached patch is a hack I cooked up to fix one of the libvirt-TCK Xen failures. The test (200-disk-hotplug.t) attempts to hot

Re: [Xen-devel] [PATCH] docs: create reproducible html

2015-02-02 Thread Ian Campbell
On Mon, 2015-02-02 at 14:40 +0100, Olaf Hering wrote: On Mon, Feb 02, Ian Jackson wrote: What do you think ? Thats overkill. The maintainers of make removed the internal sort with 3.82, likely for performance reasons. Aren't you adding uses of it here? Sorted lists are required only

Re: [Xen-devel] [PATCH 3/5] hvmemul_do_io: If the send to the ioreq server failed do not retry.

2015-02-02 Thread Don Slutz
On 02/02/15 05:14, Jan Beulich wrote: On 02.02.15 at 11:06, paul.durr...@citrix.com wrote: -Original Message- From: Jan Beulich [mailto:jbeul...@suse.com] Sent: 02 February 2015 10:04 To: Paul Durrant; Don Slutz Cc: Andrew Cooper; George Dunlap; Ian Campbell; Ian Jackson; Stefano

Re: [Xen-devel] [PATCH 0/6] xen/arm: Move in/out code to/from init section

2015-02-02 Thread Ian Campbell
On Mon, 2015-02-02 at 12:48 +, Julien Grall wrote: On 02/02/15 10:58, Ian Campbell wrote: On Fri, 2015-01-30 at 11:33 +, Julien Grall wrote: Hi Ian, On 30/01/15 11:30, Ian Campbell wrote: On Thu, 2015-01-29 at 18:32 +, Julien Grall wrote: Hello, Ping? Any more review

Re: [Xen-devel] [OSSTEST PATCH v4 9/9] mfi-common, make-flight: create XSM test jobs

2015-02-02 Thread Ian Campbell
On Wed, 2014-12-10 at 22:21 +, Wei Liu wrote: Duplicate Debian PV and HVM test jobs for XSM testing. Signed-off-by: Wei Liu wei.l...@citrix.com Acked-by: Ian Campbell ian.campb...@citrix.com Ian. ___ Xen-devel mailing list

Re: [Xen-devel] [PATCH 3/5] hvmemul_do_io: If the send to the ioreq server failed do not retry.

2015-02-02 Thread Don Slutz
On 02/02/15 03:36, Jan Beulich wrote: On 30.01.15 at 20:19, dsl...@verizon.com wrote: Soon after sending this, I came up with a 2nd way for fix. Change hvm_has_dm() to use hvm_select_ioreq_server(). Then the correct answer will be found, and so will not retry. To avoid 2 calls to

Re: [Xen-devel] [PATCH OSSTEST] README.dev: Updates to commissioning process

2015-02-02 Thread Ian Jackson
Ian Campbell writes ([PATCH OSSTEST] README.dev: Updates to commissioning process): - mkpxedir requires the resource to be allocated - mention the need to bless the hosts for comissioning run, and to add proper blessings afterwards. - run mg-execute-flight in screen ... +$ mf-hosts

Re: [Xen-devel] [PATCH 3/5] hvmemul_do_io: If the send to the ioreq server failed do not retry.

2015-02-02 Thread Jan Beulich
On 02.02.15 at 11:06, paul.durr...@citrix.com wrote: -Original Message- From: Jan Beulich [mailto:jbeul...@suse.com] Sent: 02 February 2015 10:04 To: Paul Durrant; Don Slutz Cc: Andrew Cooper; George Dunlap; Ian Campbell; Ian Jackson; Stefano Stabellini; Wei Liu;

Re: [Xen-devel] [PATCH] xen/arm: Fix rtds scheduler for arm

2015-02-02 Thread Denys Drozdov
Hi Ian, The issue observed on credit2 scheduler is similar to the rt scheduler on arm platform. The root cause is that interrupts are disabled in the beginning of arm context_switch, thus spin_lock_irq is failing in ASSERT(local_irq_is_enabled()). I propose to change both credit2 and rt scheduler

[Xen-devel] [PATCH] xen-blkfront: fix accounting of reqs when migrating

2015-02-02 Thread Roger Pau Monne
Current migration code uses blk_put_request in order to finish a request before requeuing it. This function doesn't update the statistics of the queue, which completely screws accounting. Use blk_end_request_all instead which properly updates the statistics of the queue. Signed-off-by: Roger Pau

[Xen-devel] [PATCH v2 3/5] x86/mm: allow for building without shadow mode support

2015-02-02 Thread Jan Beulich
Considering the complexity of the code, it seems to be a reasonable thing to allow people to disable that code entirely even outside the immediate need for this by the next patch. Signed-off-by: Jan Beulich jbeul...@suse.com --- v2: Set mode table in shadow_vcpu_init() stub. Convert

Re: [Xen-devel] [PATCH linux-2.6.18] xen: mark pvscsi frontend request consumed only after last read

2015-02-02 Thread Juergen Gross
On 02/02/2015 08:52 AM, Jan Beulich wrote: On 30.01.15 at 14:51, jgr...@suse.com.non-mime.internet wrote: A request in the ring buffer mustn't be read after it has been marked as consumed. Otherwise it might already have been reused by the frontend without violating the ring protocol. This is

Re: [Xen-devel] [RFC V10 0/4] domain snapshot document

2015-02-02 Thread Ian Campbell
On Sun, 2015-02-01 at 20:55 +, Dave Scott wrote: On 29 Jan 2015, at 16:36, Ian Campbell ian.campb...@citrix.com wrote: On Mon, 2015-01-26 at 11:25 +0800, Chunyan Liu wrote: Changes to V9: This looks good to me, thanks. Ian/Wei do you have any comments? xapi folks? The

Re: [Xen-devel] PCI-Passthrough Error: transmit queue timed out

2015-02-02 Thread Jan Beulich
On 31.01.15 at 09:49, open...@126.com wrote: 2015-01-31T04:53:45.005161-05:00 linux-gbze kernel: [63331.820079] [ cut here ] 2015-01-31T04:53:45.005187-05:00 linux-gbze kernel: [63331.820163] WARNING: CPU: 0 PID: 0 at net/sched/sch_generic.c:264

Re: [Xen-devel] apic-v reduce network performance in my test case

2015-02-02 Thread Jan Beulich
On 31.01.15 at 11:29, john.liuqim...@huawei.com wrote: Recently I met an odd performance problem: when I turn on APIC Virtualization feature (apicv=1), the network performance of a windows guest become worse. My test case like this: host only have one windows 2008 R2 HVM guest

[Xen-devel] [PATCH v2 1/5] x86/shadow: tidy up fragmentary page lists in multi‑page shadows

2015-02-02 Thread Jan Beulich
From: Tim Deegan t...@xen.org Multi-page shadows are linked together using the 'list' field. When those shadows are in the pinned list, the list fragments are spliced into the pinned list; otherwise they have no associated list head. Rework the code that handles these fragments to use the

[Xen-devel] [PATCH v2 2/5] x86/shadow: don't needlessly expose internal functions

2015-02-02 Thread Jan Beulich
... and drop an unused one. Signed-off-by: Jan Beulich jbeul...@suse.com --- a/xen/arch/x86/mm/shadow/common.c +++ b/xen/arch/x86/mm/shadow/common.c @@ -2383,7 +2383,7 @@ int sh_remove_write_access_from_sl1p(str /* Remove all mappings of a guest frame from the shadow tables. * Returns

Re: [Xen-devel] [PATCH 3/5] hvmemul_do_io: If the send to the ioreq server failed do not retry.

2015-02-02 Thread Paul Durrant
-Original Message- From: Don Slutz [mailto:dsl...@verizon.com] Sent: 30 January 2015 19:19 To: Don Slutz; Jan Beulich; Paul Durrant Cc: Andrew Cooper; Ian Campbell; Wei Liu; George Dunlap; Ian Jackson; Stefano Stabellini; xen-devel@lists.xen.org; Keir (Xen.org) Subject: Re: [PATCH

Re: [Xen-devel] [v2][PATCH] libxl: add one machine property to support IGD GFX passthrough

2015-02-02 Thread Ian Campbell
On Mon, 2015-02-02 at 09:17 +0800, Tiejun Chen wrote: When we're working to support IGD GFX passthrough with qemu upstream, instead of -gfx_passthru we'd like to make that a machine option, -machine xxx,-igd-passthru=on. This need to bring a change on tool side. From which Qemu version is

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

2015-02-02 Thread Ian Campbell
On Thu, 2015-01-29 at 15:08 +, Stefano Stabellini wrote: @@ -4775,6 +4781,14 @@ retry_transaction: new_target_memkb = current_target_memkb + target_memkb; } else new_target_memkb = target_memkb - videoram; + +if (new_target_memkb = 0) { +

Re: [Xen-devel] [PATCH v3 1/2] xen/arm: vgic: Keep track of vIRQ used by a domain

2015-02-02 Thread Ian Campbell
On Thu, 2015-01-29 at 15:51 +, Julien Grall wrote: - Move the retry after looking for first/end. I keep the goto rather than a loop because it's more clear that we retry because we were unable to set the bit Then I think a do {} while (!successfully allocated) is

Re: [Xen-devel] [PATCH] time: widen wallclock seconds to 64 bits

2015-02-02 Thread Jan Beulich
On 02.02.15 at 15:40, ian.campb...@citrix.com wrote: On Thu, 2015-01-29 at 15:39 +, Jan Beulich wrote: [...] A note on the conditional suppressing the xen_wc_sec_hi helper macro definition in the ix86 case for hypervisor and tools: Neither of the two actually need this, and its presence

Re: [Xen-devel] [PATCH 6/7] build system: stubdom targets now depends on mini-os target

2015-02-02 Thread Wei Liu
On Mon, Feb 02, 2015 at 03:01:44PM +, Ian Campbell wrote: On Mon, 2015-02-02 at 12:59 +, Wei Liu wrote: Provide mini-os url and revision in Config.mk Introduce Makefile.mini-os which contains mini-os specific targets. I'm not sure it has enough content to warrant that, those two

Re: [Xen-devel] [PATCH v3 2/2] xen/arm: Automatically find a PPI for the DOM0 event channel interrupt

2015-02-02 Thread Ian Campbell
On Thu, 2015-01-29 at 15:51 +, Julien Grall wrote: Use the new vgic interface to know which virtual PPI is free and use it for the event channel code. At the DOM0 creation time, Xen doesn't know which vIRQ will be free. All the vIRQ will be reserved when we parse the device tree. So we

Re: [Xen-devel] [PATCH v2 18/29] Ovmf/Xen: move XenBusDxe hypercall code to separate library

2015-02-02 Thread Laszlo Ersek
one important comment On 01/26/15 20:03, Ard Biesheuvel wrote: This moves all of the Xen hypercall code that was private to XenBusDxe to a new library class XenHypercallLib. This will allow us to reimplement it for ARM, and to export the Xen hypercall functionality to other parts of the code,

Re: [Xen-devel] [PATCH] tools/libxc: Don't leave scratch_pfn uninitialised if the domain has no memory

2015-02-02 Thread Ian Campbell
On Wed, 2015-01-28 at 15:52 +, Andrew Cooper wrote: c/s 5b5c40c0d1 libxc: introduce a per architecture scratch pfn for temporary grant mapping accidentally an issue whereby there were two paths out of xc_core_arch_get_scratch_gpfn() which returned 0, but only one of which assigned a value

Re: [Xen-devel] [PATCH] ocaml/xenctrl: Make failwith_xc() thread safe

2015-02-02 Thread Ian Campbell
On Fri, 2015-01-30 at 14:24 +, Dave Scott wrote: On 30 Jan 2015, at 14:20, Wei Liu wei.l...@citrix.com wrote: On Fri, Jan 30, 2015 at 02:19:53PM +, Dave Scott wrote: Looks ok to me. Signed-off-by: David Scott dave.sc...@citrix.com I think this should be an

Re: [Xen-devel] [PATCH v2 05/15] xen/arm: vgic-v3: Correctly implement read into GICR_NSACR

2015-02-02 Thread Ian Campbell
On Thu, 2015-01-29 at 18:25 +, Julien Grall wrote: The 32-bit register GICR_NSACR is RAZ/WI on non-secure state. Therefore we should not inject a data abort to the guest. Signed-off-by: Julien Grall julien.gr...@linaro.org Acked-by: Ian Campbell ian.campb...@citrix.com

Re: [Xen-devel] [PATCH] time: widen wallclock seconds to 64 bits

2015-02-02 Thread Ian Campbell
On Thu, 2015-01-29 at 15:39 +, Jan Beulich wrote: Linux is in the process of converting their seconds representation to 64 bits, so in order to support it consistently we should follow suit (which at some point in quite a few years we'd have to do anyway). To represent this in struct

Re: [Xen-devel] [PATCH 3/7] stubdom: don't look for mini-os source file during configure

2015-02-02 Thread Wei Liu
On Mon, Feb 02, 2015 at 03:09:32PM +, Ian Jackson wrote: Wei Liu writes ([PATCH 3/7] stubdom: don't look for mini-os source file during configure): Mini-os source code will be fetched during build. ... -AC_CONFIG_SRCDIR([../extras/mini-os/kernel.c]) You should replace this with a

Re: [Xen-devel] [PATCH v2 04/15] xen/arm: vgic-v3: Correctly handle RAZ/WI registers

2015-02-02 Thread Ian Campbell
On Thu, 2015-01-29 at 18:25 +, Julien Grall wrote: Some of the registers are accessible via multiple size (see GICD_IPRIORITYR*). Thoses registers are misimplemented when they should be RAZ. Only Those and incorrectly implemented. word-access size are currently allowed for them. To

Re: [Xen-devel] [PATCH] tools/xenctrl: correct some function declarations

2015-02-02 Thread Ian Campbell
On Fri, 2015-01-30 at 12:40 +, Wei Liu wrote: On Fri, Jan 30, 2015 at 03:32:26PM +0800, Tiejun Chen wrote: When commit 6865e52b78f4, PCI multi-seg: adjust domctl interface, is introduced, we missed to sync that head file. Signed-off-by: Tiejun Chen tiejun.c...@intel.com Acked-by:

Re: [Xen-devel] apic-v reduce network performance in my test case

2015-02-02 Thread Konrad Rzeszutek Wilk
On Mon, Feb 02, 2015 at 09:58:57PM +0800, Liuqiming (John) wrote: Hi Jan, Thanks for the reply. On 2015/2/2 18:12, Jan Beulich wrote: On 31.01.15 at 11:29, john.liuqim...@huawei.com wrote: Recently I met an odd performance problem: when I turn on APIC Virtualization feature

[Xen-devel] [PATCH v2 2/3] hvm_complete_assist_req: We should not be able to get here on IOREQ_TYPE_PCI_CONFIG

2015-02-02 Thread Don Slutz
Suggested-by: Paul Durrant paul.durr...@citrix.com Signed-off-by: Don Slutz dsl...@verizon.com --- v2: Add Suggested-by xen/arch/x86/hvm/hvm.c | 1 + 1 file changed, 1 insertion(+) diff --git a/xen/arch/x86/hvm/hvm.c b/xen/arch/x86/hvm/hvm.c index c7984d1..6f7b407 100644 ---

[Xen-devel] [PATCH v2 0/3] Skip unneeded VMENTRY VMEXIT

2015-02-02 Thread Don Slutz
Changes v2: Paul Durrant: I think the two changes only make sense in combination. folded old #3 and old #5 into #3. Actually that comment is not right. The operation is not binned; it's just already been done. Comment has been dropped. I think you can ASSERT(s)

Re: [Xen-devel] [PATCH] ocaml/xenctrl: Fix stub_xc_readconsolering()

2015-02-02 Thread Ian Campbell
On Fri, 2015-01-30 at 14:30 +, Dave Scott wrote: On 30 Jan 2015, at 14:11, Andrew Cooper andrew.coop...@citrix.com wrote: The Ocaml stub to retrieve the hypervisor console ring had a few problems. * A single 32k buffer would truncate a large console ring. * The buffer was

Re: [Xen-devel] [PATCH v2 19/29] Ovmf/Xen: introduce XENIO_PROTOCOL

2015-02-02 Thread Laszlo Ersek
On 01/26/15 20:03, Ard Biesheuvel wrote: This introduces the abstract XENIO_PROTOCOL that will be used to communicate the Xen grant table address to drivers supporting this protocol. Primary purpose is allowing us to change the XenBusDxe implementation so that it can support non-PCI Xen

[Xen-devel] [PATCH v2 3/3] hvm_has_dm: Do a full check for backing DM

2015-02-02 Thread Don Slutz
This saves a VMENTRY and a VMEXIT since we not longer retry the ioport read on backing DM not handling a given ioreq. To only call on hvm_select_ioreq_server() once in this code path, return s. Also switch to hvm_send_assist_req_to_ioreq_server(). Signed-off-by: Don Slutz dsl...@verizon.com ---

[Xen-devel] [PATCH v2 1/3] xentrace: Adjust IOPORT MMIO format

2015-02-02 Thread Don Slutz
The 1st item is not data, but the port (address). The 2nd item is the data. Signed-off-by: Don Slutz dsl...@verizon.com --- tools/xentrace/formats | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/tools/xentrace/formats b/tools/xentrace/formats index da658bf..5d7b72a

Re: [Xen-devel] [PATCH 1/7] stubdom: fix make build

2015-02-02 Thread Ian Campbell
On Mon, 2015-02-02 at 12:58 +, Wei Liu wrote: Cross compiling libxc requires some symlinks to exist. Note that make -C tools/include requires running tools/configure. But at least now the error message is much better than just a file not found error. Signed-off-by: Wei Liu

Re: [Xen-devel] [PATCH 2/7] Makefile: refactor build/clean/distclean targets

2015-02-02 Thread Ian Campbell
On Mon, 2015-02-02 at 12:58 +, Wei Liu wrote: Factor out per-subsystem build/clean/distclean-% targets, so that we can build subsystems independently in top level directory. The motive behind this is after splitting out mini-os from Xen tree, stubdom is in effect a downstream of mini-os.

Re: [Xen-devel] [PATCH 5/7] Mini-OS: standalone build

2015-02-02 Thread Ian Campbell
On Mon, 2015-02-02 at 12:59 +, Wei Liu wrote: In order to keep the tree bisectable all the changes are done in one single commit. Things done in this commit: 1. Import necessary .mk files from Xen. 2. Move all XEN_ related variables to MINIOS_ namespace. 3. Import Xen public header

Re: [Xen-devel] [PATCH 4/7] git-checkout.sh: use mkdir -p

2015-02-02 Thread Ian Campbell
On Mon, 2015-02-02 at 12:59 +, Wei Liu wrote: Otherwise mkdir extras/mini-os fails because extras doesn't exist. Signed-off-by: Wei Liu wei.l...@citrix.com Acked-by: Ian Campbell ian.campb...@citrix.com As a future change perhaps we should clone mini-os under stubdom/* somewhere instead

Re: [Xen-devel] [PATCH 6/7] build system: stubdom targets now depends on mini-os target

2015-02-02 Thread Ian Jackson
Wei Liu writes (Re: [PATCH 6/7] build system: stubdom targets now depends on mini-os target): So I once had something like: $(XEN_ROOT)/Config.mk: mini-os-dir Mini-os got clone and the build proceeded. But stubdom build still broke miserably, because *FLAGS in stubdom/Makefile are not

Re: [Xen-devel] [PATCH v3] libxl: Wait for ballooning if free memory is increasing

2015-02-02 Thread Mike Latimer
On Monday, February 02, 2015 02:35:39 PM Ian Campbell wrote: On Fri, 2015-01-30 at 14:01 -0700, Mike Latimer wrote: During domain startup, all required memory ballooning must complete within a maximum window of 33 seconds (3 retries, 11 seconds of delay). If not, domain creation is aborted

Re: [Xen-devel] [RFC PATCH] dpci: Put the dpci back on the list if running on another CPU.

2015-02-02 Thread Jan Beulich
On 02.02.15 at 15:29, konrad.w...@oracle.com wrote: On Tue, Jan 13, 2015 at 10:20:00AM +, Jan Beulich wrote: On 12.01.15 at 17:45, konrad.w...@oracle.com wrote: There is race when we clear the STATE_SCHED in the softirq - which allows the 'raise_softirq_for' to progress and schedule

Re: [Xen-devel] [PATCH 6/7] build system: stubdom targets now depends on mini-os target

2015-02-02 Thread Wei Liu
On Mon, Feb 02, 2015 at 03:15:07PM +, Ian Jackson wrote: Wei Liu writes (Re: [PATCH 6/7] build system: stubdom targets now depends on mini-os target): So I once had something like: $(XEN_ROOT)/Config.mk: mini-os-dir Mini-os got clone and the build proceeded. But stubdom build

Re: [Xen-devel] [PATCH] libxl_set_memory_target: only remove videoram from absolute targets

2015-02-02 Thread Ian Campbell
On Wed, 2015-01-28 at 14:47 +, Stefano Stabellini wrote: On Wed, 28 Jan 2015, Ian Campbell wrote: On Wed, 2015-01-28 at 14:24 +, Stefano Stabellini wrote: On Wed, 28 Jan 2015, Ian Campbell wrote: On Mon, 2015-01-26 at 16:47 +, Stefano Stabellini wrote: If the new target

Re: [Xen-devel] [PATCH v2 07/15] xen/arm: vgic-v3: Use a struct to describe contiguous rdist regions

2015-02-02 Thread Ian Campbell
On Thu, 2015-01-29 at 18:25 +, Julien Grall wrote: Also update the different comment to make clear that we register one MMIO region per contiguous regions and not per re-distributor. Signed-off-by: Julien Grall julien.gr...@linaro.org Acked-by: Ian Campbell ian.campb...@citrix.com

Re: [Xen-devel] [RFC PATCH] dpci: Put the dpci back on the list if running on another CPU.

2015-02-02 Thread Konrad Rzeszutek Wilk
On Tue, Jan 13, 2015 at 10:20:00AM +, Jan Beulich wrote: On 12.01.15 at 17:45, konrad.w...@oracle.com wrote: There is race when we clear the STATE_SCHED in the softirq - which allows the 'raise_softirq_for' to progress and schedule another dpci. During that time the other CPU could

Re: [Xen-devel] [OSSTEST PATCH v4 3/9] ts-xen-build: build with XSM support if requested

2015-02-02 Thread Wei Liu
On Mon, Feb 02, 2015 at 01:14:34PM +, Ian Campbell wrote: On Wed, 2014-12-10 at 22:21 +, Wei Liu wrote: Signed-off-by: Wei Liu wei.l...@citrix.com Inconsistent whitespace in the final hunk, but: Acked-by: Ian Campbell ian.campb...@citrix.com This will override the default

Re: [Xen-devel] [PATCH 0/7] Split off mini-os to a separate tree

2015-02-02 Thread Ian Campbell
On Mon, 2015-02-02 at 12:58 +, Wei Liu wrote: As far as I can tell there is no incoming mini-os patches at the moment. So I suggest we commence once we get that last mini-os patch in staging pushed to master. Is the new xenbits tree (the proper one, not your people one) for mini-os all

Re: [Xen-devel] [OSSTEST PATCH v4 3/9] ts-xen-build: build with XSM support if requested

2015-02-02 Thread Ian Campbell
On Mon, 2015-02-02 at 14:30 +, Wei Liu wrote: On Mon, Feb 02, 2015 at 01:14:34PM +, Ian Campbell wrote: On Wed, 2014-12-10 at 22:21 +, Wei Liu wrote: Signed-off-by: Wei Liu wei.l...@citrix.com Inconsistent whitespace in the final hunk, but: Acked-by: Ian Campbell

Re: [Xen-devel] [PATCH 7/7] Remove in-tree mini-os directory

2015-02-02 Thread Ian Campbell
On Mon, 2015-02-02 at 12:59 +, Wei Liu wrote: Signed-off-by: Wei Liu wei.l...@citrix.com Cc: Ian Campbell ian.campb...@citrix.com Cc: Ian Jackson ian.jack...@eu.citrix.com [ output trimmed ] Acked-by: Ian Campbell ian.campb...@citrix.com

Re: [Xen-devel] [PATCH v2 04/15] xen/arm: vgic-v3: Correctly handle RAZ/WI registers

2015-02-02 Thread Ian Campbell
On Thu, 2015-01-29 at 18:25 +, Julien Grall wrote: Some of the registers are accessible via multiple size (see GICD_IPRIORITYR*). Thoses registers are misimplemented when they should be RAZ. Only Those and incorrectly implemented. word-access size are currently allowed for them. To

Re: [Xen-devel] [RFC PATCH] dpci: Put the dpci back on the list if running on another CPU.

2015-02-02 Thread Konrad Rzeszutek Wilk
On Mon, Feb 02, 2015 at 03:19:33PM +, Jan Beulich wrote: On 02.02.15 at 15:29, konrad.w...@oracle.com wrote: On Tue, Jan 13, 2015 at 10:20:00AM +, Jan Beulich wrote: On 12.01.15 at 17:45, konrad.w...@oracle.com wrote: There is race when we clear the STATE_SCHED in the softirq

Re: [Xen-devel] [PATCH v2 06/15] xen/arm: vgic-v3: Set stride during domain initialization

2015-02-02 Thread Ian Campbell
On Thu, 2015-01-29 at 18:25 +, Julien Grall wrote: The stride may not be set if the hardware GIC is using the default layout. It happens on the Foundation model. On GICv3, the default stride is 2 * 64K. Therefore it's possible to avoid checking at every redistributor MMIO access if the

Re: [Xen-devel] [PATCH v2 04/15] xen/arm: vgic-v3: Correctly handle RAZ/WI registers

2015-02-02 Thread Julien Grall
Hi Ian, On 02/02/15 15:24, Ian Campbell wrote: On Thu, 2015-01-29 at 18:25 +, Julien Grall wrote: Some of the registers are accessible via multiple size (see GICD_IPRIORITYR*). Thoses registers are misimplemented when they should be RAZ. Only Those and incorrectly implemented.

Re: [Xen-devel] [PATCH v2 08/15] xen/arm: vgic-v3: Emulate correctly the re-distributor

2015-02-02 Thread Ian Campbell
On Thu, 2015-01-29 at 18:25 +, Julien Grall wrote: There is a one-to-one mapping between each re-distributors and processors. Each re-distributors can be accessed by any processor at any time. For instance during the initialization of the GIC, the drivers will browse the re-distributor to

Re: [Xen-devel] [PATCH v2 11/15] xen/arm: vgic-v2: Correctly handle RAZ/WI registers

2015-02-02 Thread Ian Campbell
On Thu, 2015-01-29 at 18:25 +, Julien Grall wrote: Some of the registers are accessible via multiple size (see GICD_IPRIORITYR*). They are byte accessible, but are they half word accessible? I suspect not. Thoses registers are misimplemented when they should be RAZ. Only Same typoes as

Re: [Xen-devel] Did you get my e-mail about DRBD with Xen 4.4 on Jessie ?

2015-02-02 Thread Lars Kurth
Tomasz, you don't have to subscribe, but I think you used the wrong email address. If you don't subscript your mail gets moderated and it may take a few hours for a moderator to pick it up. From your mail it says you used xen-us...@list.xen.org, whereas the right email address is

Re: [Xen-devel] [PATCH v2 06/15] xen/arm: vgic-v3: Set stride during domain initialization

2015-02-02 Thread Julien Grall
On 02/02/15 15:40, Ian Campbell wrote: On Thu, 2015-01-29 at 18:25 +, Julien Grall wrote: The stride may not be set if the hardware GIC is using the default layout. It happens on the Foundation model. On GICv3, the default stride is 2 * 64K. Therefore it's possible to avoid checking at

Re: [Xen-devel] [PATCH RFC] libxl: set disk defaults in remove/destroy functions

2015-02-02 Thread Ian Jackson
Wei Liu writes (Re: [Xen-devel] [PATCH RFC] libxl: set disk defaults in remove/destroy functions): FWIW xl block-detach calls libxl_vdev_to_device_disk to convert a vdev to disk. That function reads xenstore to get the actual backend of that specific vdev. Don't know how useful it is to

Re: [Xen-devel] [PATCH v2 15/15] xen/arm: gic-v3: Update some comments in the code

2015-02-02 Thread Julien Grall
Hi Ian, On 02/02/15 16:05, Ian Campbell wrote: On Thu, 2015-01-29 at 18:25 +, Julien Grall wrote: - Drop wrong comment about the default stride. It's not always 2 * SZ_64K What other defaults are possible and under what circumstances? 128K, when the re-distributor supports VLPIs.

Re: [Xen-devel] [PATCHv1 net] xen-netback: stop the guest rx thread after a fatal error

2015-02-02 Thread Wei Liu
On Mon, Feb 02, 2015 at 04:57:51PM +, David Vrabel wrote: After commit e9d8b2c2968499c1f96563e6522c56958d5a1d0d (xen-netback: disable rogue vif in kthread context), a fatal (protocol) error would leave the guest Rx thread spinning, wasting CPU time. Commit

Re: [Xen-devel] [PATCH v2 04/15] xen/arm: vgic-v3: Correctly handle RAZ/WI registers

2015-02-02 Thread Ian Campbell
On Mon, 2015-02-02 at 15:59 +, Julien Grall wrote: Hi Ian, On 02/02/15 15:24, Ian Campbell wrote: On Thu, 2015-01-29 at 18:25 +, Julien Grall wrote: Some of the registers are accessible via multiple size (see GICD_IPRIORITYR*). Thoses registers are misimplemented when they

Re: [Xen-devel] [PATCH RFC] libxl: set disk defaults in remove/destroy functions

2015-02-02 Thread Wei Liu
On Mon, Feb 02, 2015 at 01:27:13PM +, Ian Campbell wrote: On Mon, 2015-01-26 at 16:14 -0700, Jim Fehlig wrote: Cc-ing the other toolstack maintainers, both of whom have more familiarity with this part of libxl than I. The attached patch is a hack I cooked up to fix one of the

Re: [Xen-devel] [PATCH v2 11/15] xen/arm: vgic-v2: Correctly handle RAZ/WI registers

2015-02-02 Thread Ian Campbell
On Mon, 2015-02-02 at 16:36 +, Julien Grall wrote: Hi Ian, On 02/02/15 16:02, Ian Campbell wrote: On Thu, 2015-01-29 at 18:25 +, Julien Grall wrote: Some of the registers are accessible via multiple size (see GICD_IPRIORITYR*). They are byte accessible, but are they half

[Xen-devel] PV USB

2015-02-02 Thread Vitaly Chernooky
Hi All! What is the current state of PV USB drivers and where can I get actual info about it? I already have read this wiki page http://wiki.xen.org/wiki/Xen_USB_Passthrough but instead of clearness I have got a bit confusion. Could anybody clarify this theme for me? With best regards, --

Re: [Xen-devel] [PATCH v2 09/15] xen/arm: vgic-v3: Clarify which distributor is used in the common emulation

2015-02-02 Thread Ian Campbell
On Thu, 2015-01-29 at 18:25 +, Julien Grall wrote: The messages in the common emulation doesn't specify which distributor (redistributor or distributor) is used. This make difficult to find the correct registers. Signed-off-by: Julien Grall julien.gr...@linaro.org Acked-by: Ian Campbell

[Xen-devel] WebCAM via PV-USB

2015-02-02 Thread Vitaly Chernooky
Hi All! Is there known any USB webcam to be working via PV USB? With best regards, -- *Vitaly Chernooky | Senior Developer - Product Engineering and Development* GlobalLogic P *+380.44.4929695 ext.1136* M *+380.98.7920568* S cvv_2k www.globallogic.com

Re: [Xen-devel] [PATCH v2 13/15] xen/arm: vgic-v2: GICD_I{S, C}PENDR* are only word-accessible

2015-02-02 Thread Ian Campbell
On Thu, 2015-01-29 at 18:25 +, Julien Grall wrote: Signed-off-by: Julien Grall julien.gr...@linaro.org Acked-by: Ian Campbell ian.campb...@citrix.com ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v2 14/15] xen/arm: vgic: Drop iactive, ipend, pendsgi field

2015-02-02 Thread Ian Campbell
On Thu, 2015-01-29 at 18:25 +, Julien Grall wrote: The current VGIC code doesn't support to change the pending and active status of an IRQ via the (re-)distributor. If we plan to support it in the future, it will unlikely require a specific bitfield as we already store the status per vIRQ.

Re: [Xen-devel] [PATCH v2 15/15] xen/arm: gic-v3: Update some comments in the code

2015-02-02 Thread Ian Campbell
On Thu, 2015-01-29 at 18:25 +, Julien Grall wrote: - Drop wrong comment about the default stride. It's not always 2 * SZ_64K What other defaults are possible and under what circumstances? - Explain why SZ_64K * 2 Signed-off-by: Julien Grall julien.gr...@linaro.org ---

Re: [Xen-devel] [PATCH v2 04/15] xen/arm: vgic-v3: Correctly handle RAZ/WI registers

2015-02-02 Thread Julien Grall
On 02/02/15 16:08, Ian Campbell wrote: On Mon, 2015-02-02 at 15:59 +, Julien Grall wrote: Hi Ian, On 02/02/15 15:24, Ian Campbell wrote: On Thu, 2015-01-29 at 18:25 +, Julien Grall wrote: Some of the registers are accessible via multiple size (see GICD_IPRIORITYR*). Thoses

  1   2   >