Re: [Xen-devel] [PATCH 2/5] allow domain heap allocations to specify more than one NUMA node

2015-03-03 Thread Jan Beulich
>>> On 02.03.15 at 18:12, wrote: > On Thu, 2015-02-26 at 13:53 +, Jan Beulich wrote: >> ... using struct domain as a container for passing the respective >> affinity mask: Quite a number of allocations are domain specific, yet >> not to be accounted for that domain. Introduce a flag suppressin

Re: [Xen-devel] [PATCH v10 4/4] tools, docs: add total/local memory bandwith monitoring

2015-03-03 Thread Chao Peng
On Mon, Mar 02, 2015 at 01:48:43PM +, Ian Campbell wrote: > On Thu, 2015-02-26 at 16:45 +0800, Chao Peng wrote: > > Add Memory Bandwidth Monitoring(MBM) for VMs. Two types of monitoring > > are supported: total and local memory bandwidth monitoring. To use it, > > CMT should be enabled in hyper

Re: [Xen-devel] [PATCH 17/18] x86/efi: create new early memory allocator

2015-03-03 Thread Jan Beulich
>>> On 02.03.15 at 21:25, wrote: > On Mon, Mar 2, 2015 at 9:23 AM, Jan Beulich wrote: > On 30.01.15 at 18:54, wrote: >>> @@ -192,12 +218,7 @@ static void __init >>> efi_arch_process_memory_map(EFI_SYSTEM_TABLE *SystemTable, >>> >>> static void *__init efi_arch_allocate_mmap_buffer(UINTN *ma

[Xen-devel] [PATCH v5] modify the IO_TLB_SEGSIZE and IO_TLB_DEFAULT_SIZE configurable as flexible requirement about SW-IOMMU.

2015-03-03 Thread Wang Xiaoming
The maximum of SW-IOMMU is limited to 2^11*128 = 256K. And the size of IO_TLB_DEFAULT_SIZE is limited to (64UL<<20) 64M now. While in different platform and different requirement this seems improper. So modifing the IO_TLB_SEGSIZE to io_tlb_segsize and IO_TLB_DEFAULT_SIZE to io_tlb_default_size whi

[Xen-devel] [linux-3.16 test] 35762: regressions - FAIL

2015-03-03 Thread xen . org
flight 35762 linux-3.16 real [real] http://www.chiark.greenend.org.uk/~xensrcts/logs/35762/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-credit2 15 guest-localmigrate/x10fail REGR. vs. 34167 test-amd64-i386-pair

[Xen-devel] xen 4.5 non-debug error logging misleading when xz support is unavailable

2015-03-03 Thread svenvan svenvan
Hi, xen 4.5 building from source on debian wheezy kernel 3.16 amd64 $ xl create wheezy-template-amd64.cfg Parsing config from wheezy-template64.mens-net.private.cfg xc: error: panic: xc_dom_bzimageloader.c:730: xc_dom_probe_bzimage_kernel unable to XZ decompress kernel: Invalid kernel xc: error: p

Re: [Xen-devel] [PATCH 05/18] efi: split efi_enabled to efi_platform and efi_loader

2015-03-03 Thread Jan Beulich
>>> On 03.03.15 at 00:40, wrote: > Reviewing the #ifndef CONFIG_ARM in EFI code, and the efi_enabled > usage elsewhere, > the remaining EFI tasks on ARM look like: > * Support for SetVirtualAddressMap > * Runtime service support - looks like just time function used by x86 > in get_cmos_time() * P

Re: [Xen-devel] [PATCH v6 02/23] xen: move NUMA_NO_NODE to public memory.h as XEN_NUMA_NO_NODE

2015-03-03 Thread Jan Beulich
>>> On 03.03.15 at 04:42, wrote: > On Mon, 2015-03-02 at 18:19 +, Andrew Cooper wrote: >> In this case, a pnode of XEN_NO_NUMA_NODE seems like precisely the >> correct value to report. >> > Indeed. It tells Xen: < could not come up with any sane vnode-to-pnode mapping, please figure > that ou

Re: [Xen-devel] Strange memory balloning problem even if balloning is disabled

2015-03-03 Thread Fabio Fantoni
Il 02/03/2015 17:31, David Vrabel ha scritto: On 02/03/15 16:23, Jan Beulich wrote: On 02.03.15 at 16:52, wrote: On systems with wheezy as dom0 and kernel 3.2 it seems there are no visible problems. After updating the kernel to 3.16 (from backports repository), after starting the domUs, I get

Re: [Xen-devel] [PATCH v2] sched: credit2: respect per-vcpu hard affinity

2015-03-03 Thread Jan Beulich
>>> On 03.03.15 at 04:15, wrote: > On Sun, 2015-02-08 at 17:45 -1000, Justin T. Weaver wrote: >> --- a/xen/common/sched_credit2.c >> +++ b/xen/common/sched_credit2.c >> @@ -194,6 +194,12 @@ int opt_overload_balance_tolerance=-3; >> integer_param("credit2_balance_over", opt_overload_balance_tolera

Re: [Xen-devel] [PATCH v2 1/3] xen/arm: Add ThunderX platform support

2015-03-03 Thread Ian Campbell
On Tue, 2015-03-03 at 10:39 +0530, Vijay Kilari wrote: > On Mon, Feb 23, 2015 at 11:21 PM, Ian Campbell > wrote: > > On Wed, 2015-02-18 at 12:38 +, Julien Grall wrote: > >> > +uint64_t addr, size; > > > > Please use paddr_t and PRIpaddr etc. > > > >> > +res = map_mmio_regions(d, > >>

Re: [Xen-devel] RFC: [PATCH 1/3] Enhance platform support for PCI

2015-03-03 Thread Manish Jaggi
On Monday 02 March 2015 05:18 PM, Ian Campbell wrote: On Fri, 2015-02-27 at 17:15 +, Stefano Stabellini wrote: On Fri, 27 Feb 2015, Ian Campbell wrote: On Fri, 2015-02-27 at 16:35 +, Jan Beulich wrote: On 27.02.15 at 16:24, wrote: On Fri, 2015-02-27 at 14:54 +, Stefano Stabellin

Re: [Xen-devel] [PATCH v2 1/3] xen/arm: Add ThunderX platform support

2015-03-03 Thread Vijay Kilari
On Tue, Mar 3, 2015 at 2:33 PM, Ian Campbell wrote: > On Tue, 2015-03-03 at 10:39 +0530, Vijay Kilari wrote: >> On Mon, Feb 23, 2015 at 11:21 PM, Ian Campbell >> wrote: >> > On Wed, 2015-02-18 at 12:38 +, Julien Grall wrote: >> >> > +uint64_t addr, size; >> > >> > Please use paddr_t and

[Xen-devel] [PATCH V2] Add flag to start info regarding virtual mapped p2m list

2015-03-03 Thread Juergen Gross
In order to indicate the Xen tools capability to support the virtual mapped linear p2m list instead the 3 level mfn tree add a flag to the start_info page. Signed-off-by: Juergen Gross --- xen/include/public/xen.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/xen/include/public/xen.h b/x

[Xen-devel] [PATCH] xen-scsiback: use DRV_PFX in the pr macros

2015-03-03 Thread Tao Chen
Replace the string of {xen-pvscsi:} in the pr sentences with DRV_PFX, it makes the code easier to read. Signed-off-by: Tao Chen --- drivers/xen/xen-scsiback.c | 67 +++--- 1 file changed, 34 insertions(+), 33 deletions(-) diff --git a/drivers/xen/xen-scsi

Re: [Xen-devel] [PATCH 17/18] x86/efi: create new early memory allocator

2015-03-03 Thread Daniel Kiper
On Tue, Mar 03, 2015 at 08:04:09AM +, Jan Beulich wrote: > >>> On 02.03.15 at 21:25, wrote: > > On Mon, Mar 2, 2015 at 9:23 AM, Jan Beulich wrote: > > On 30.01.15 at 18:54, wrote: > >>> @@ -192,12 +218,7 @@ static void __init > >>> efi_arch_process_memory_map(EFI_SYSTEM_TABLE *SystemTabl

[Xen-devel] kasan_map_early_shadow() on Xen

2015-03-03 Thread Luis R. Rodriguez
Andrey, I believe that on Xen we should disable kasan, would like confirmation from someone on xen-devel though. Here's the thing though -- if true -- I'd like to do it *properly*, where *properly* means addressing a bit of architecture. A simple Kconfig slap seems rather reactive. I'd like to add

Re: [Xen-devel] [PATCH] xen-scsiback: use DRV_PFX in the pr macros

2015-03-03 Thread Juergen Gross
On 03/03/2015 09:37 AM, Tao Chen wrote: Replace the string of {xen-pvscsi:} in the pr sentences with DRV_PFX, it makes the code easier to read. I'm not really convinced this is worth a patch. OTOH I'm not completely against it. If nobody rejects this and all my further comments are addressed yo

Re: [Xen-devel] [PATCH 1/5] x86: allow specifying the NUMA nodes Dom0 should run on

2015-03-03 Thread Ian Campbell
On Thu, 2015-02-26 at 13:52 +, Jan Beulich wrote: > ... by introducing a "dom0_nodes" option augmenting the "dom0_mem" and > "dom0_max_vcpus" ones. > > Note that this gives meaning to MEMF_exact_node specified alone (i.e. > implicitly combined with NUMA_NO_NODE): In such a case any node inside

Re: [Xen-devel] [PATCH V2] Add flag to start info regarding virtual mapped p2m list

2015-03-03 Thread Andrew Cooper
On 03/03/15 09:29, Juergen Gross wrote: > In order to indicate the Xen tools capability to support the virtual > mapped linear p2m list instead the 3 level mfn tree add a flag to the > start_info page. > > Signed-off-by: Juergen Gross Reviewed-by: Andrew Cooper Much more simple! > --- > xen/i

Re: [Xen-devel] freemem-slack and large memory environments

2015-03-03 Thread Ian Campbell
On Mon, 2015-03-02 at 15:49 -0700, Mike Latimer wrote: > On Monday, March 02, 2015 04:15:41 PM Stefano Stabellini wrote: > > On Mon, 2 Mar 2015, Ian Campbell wrote: > > > ? "Continue as long as progress is being made" is exactly what > > > 2563bca1154 "libxl: Wait for ballooning if free memory is i

Re: [Xen-devel] [PATCH net-next 06/14] xen: Use eth__addr instead of memset

2015-03-03 Thread Ian Campbell
On Mon, 2015-03-02 at 19:54 -0800, Joe Perches wrote: > Use the built-in function instead of memset. > > Signed-off-by: Joe Perches Acked-by: Ian Campbell ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

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

2015-03-03 Thread Chen, Tiejun
Campbell, Are you free to look at my reply? Thanks Tiejun On 2015/3/2 9:20, Chen, Tiejun wrote: Here I just mean when Qemu realizes IGD is passed through but without that appropriate option set, Qemu can post something to explicitly notify user that this option is needed in his case. But it ma

Re: [Xen-devel] kasan_map_early_shadow() on Xen

2015-03-03 Thread David Vrabel
On 03/03/15 09:40, Luis R. Rodriguez wrote: > Andrey, > > I believe that on Xen we should disable kasan, would like confirmation Why? This is the first of heard of this. > from someone on xen-devel though. Here's the thing though -- if true > -- I'd like to do it *properly*, where *properly* me

Re: [Xen-devel] [RFC PATCH 00/19] xen/arm: Add ITS support

2015-03-03 Thread Ian Campbell
On Tue, 2015-03-03 at 09:32 +0530, Vijay Kilari wrote: > Hi Ian, > > On Mon, Mar 2, 2015 at 11:09 PM, Ian Campbell wrote: > > Forgot one series (albeit of a single patch): > > > > On Mon, 2015-03-02 at 14:53 +, Ian Campbell wrote: > >> There's quite a few series out there right now from you g

Re: [Xen-devel] [PATCH v10 4/4] tools, docs: add total/local memory bandwith monitoring

2015-03-03 Thread Ian Campbell
On Tue, 2015-03-03 at 16:00 +0800, Chao Peng wrote: > On Mon, Mar 02, 2015 at 01:48:43PM +, Ian Campbell wrote: > > On Thu, 2015-02-26 at 16:45 +0800, Chao Peng wrote: > > > Add Memory Bandwidth Monitoring(MBM) for VMs. Two types of monitoring > > > are supported: total and local memory bandwid

Re: [Xen-devel] kasan_map_early_shadow() on Xen

2015-03-03 Thread Jan Beulich
>>> On 03.03.15 at 10:40, wrote: > Let's go down the rabbit hole for a bit. HAVE_ARCH_KASAN will be > selected on x86 when: > > if X86_64 && SPARSEMEM_VMEMMAP > > Now Xen should not have SPARSEMEM_VMEMMAP Why would that be? Jan ___ Xen-devel mailin

Re: [Xen-devel] [PATCH] xen-scsiback: use DRV_PFX in the pr macros

2015-03-03 Thread David Vrabel
On 03/03/15 08:37, Tao Chen wrote: > Replace the string of {xen-pvscsi:} in the pr sentences with DRV_PFX, > it makes the code easier to read. You need to #define pr_fmt(fmt) to add a prefix like this. > struct ids_tuple v; /* translate from */ > - struct scsiback_tpg *tpg;

Re: [Xen-devel] [PATCH net-next 06/14] xen: Use eth__addr instead of memset

2015-03-03 Thread Wei Liu
On Mon, Mar 02, 2015 at 07:54:51PM -0800, Joe Perches wrote: > Use the built-in function instead of memset. > > Signed-off-by: Joe Perches Acked-by: Wei Liu > --- > drivers/net/xen-netback/interface.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/net/xen-ne

Re: [Xen-devel] [PATCH v2 1/3] xen/arm: Add ThunderX platform support

2015-03-03 Thread Ian Campbell
On Tue, 2015-03-03 at 14:59 +0530, Vijay Kilari wrote: > On Tue, Mar 3, 2015 at 2:33 PM, Ian Campbell wrote: > > On Tue, 2015-03-03 at 10:39 +0530, Vijay Kilari wrote: > >> On Mon, Feb 23, 2015 at 11:21 PM, Ian Campbell > >> wrote: > >> > On Wed, 2015-02-18 at 12:38 +, Julien Grall wrote: >

[Xen-devel] [PATCH v3] ts-xen-install: Enable debug level logging in libvirt

2015-03-03 Thread Ian Campbell
This should make diagnosing libvirt issues a bit easier. Refactor the existing loop to edit the xencommons kvp style config file editing code so it can be reused for libvirtd.conf. Signed-off-by: Ian Campbell Cc: Jim Fehlig Acked-by: Ian Jackson --- v3: Only apply if toolstack==libvirt, else w

Re: [Xen-devel] [PATCH v1] xen/arm: Do not allocate pte entries for MAP_SMALL_PAGES

2015-03-03 Thread Ian Campbell
On Tue, 2015-03-03 at 13:28 +0530, Vijay Kilari wrote: > On Tue, Feb 24, 2015 at 6:29 PM, Julien Grall wrote: > > On 24/02/15 10:26, Ian Campbell wrote: > >> On Tue, 2015-02-24 at 09:38 +, Julien Grall wrote: > >>> Hi Ian, > >>> > >>> On 24/02/2015 09:31, Ian Campbell wrote: > On Wed, 201

Re: [Xen-devel] [PATCH V2] Add flag to start info regarding virtual mapped p2m list

2015-03-03 Thread Jan Beulich
>>> On 03.03.15 at 10:29, <"jgr...@suse.com".non-mime.internet> wrote: > In order to indicate the Xen tools capability to support the virtual > mapped linear p2m list instead the 3 level mfn tree add a flag to the > start_info page. > > Signed-off-by: Juergen Gross > --- > xen/include/public/xen

Re: [Xen-devel] [PATCH V2] Add flag to start info regarding virtual mapped p2m list

2015-03-03 Thread Juergen Gross
On 03/03/2015 11:27 AM, Jan Beulich wrote: On 03.03.15 at 10:29, <"jgr...@suse.com".non-mime.internet> wrote: In order to indicate the Xen tools capability to support the virtual mapped linear p2m list instead the 3 level mfn tree add a flag to the start_info page. Signed-off-by: Juergen Gross

Re: [Xen-devel] freemem-slack and large memory environments

2015-03-03 Thread Stefano Stabellini
On Tue, 3 Mar 2015, Ian Campbell wrote: > On Mon, 2015-03-02 at 15:49 -0700, Mike Latimer wrote: > > On Monday, March 02, 2015 04:15:41 PM Stefano Stabellini wrote: > > > On Mon, 2 Mar 2015, Ian Campbell wrote: > > > > ? "Continue as long as progress is being made" is exactly what > > > > 2563bca11

Re: [Xen-devel] [xen-unstable test] 35257: regressions - FAIL

2015-03-03 Thread Ian Campbell
On Mon, 2015-03-02 at 10:59 -0700, Jim Fehlig wrote: > Ian Campbell wrote: > > On Mon, 2015-03-02 at 11:14 +, Ian Jackson wrote: > > > >> Jim Fehlig writes ("Re: [Xen-devel] [xen-unstable test] 35257: regressions > >> - FAIL"): > >> ... > >> > >>> In my experience, if ERROR is insuffic

Re: [Xen-devel] freemem-slack and large memory environments

2015-03-03 Thread Stefano Stabellini
On Mon, 2 Mar 2015, Mike Latimer wrote: > On Monday, March 02, 2015 04:15:41 PM Stefano Stabellini wrote: > > On Mon, 2 Mar 2015, Ian Campbell wrote: > > > ? "Continue as long as progress is being made" is exactly what > > > 2563bca1154 "libxl: Wait for ballooning if free memory is increasing" > >

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

2015-03-03 Thread Wei Liu
On Sat, Feb 28, 2015 at 10:45:02AM +0800, openlui wrote: > > > >Persistent grant is not silver bullet. There is email thread on the > >list discussing whether it should be removed in block driver. > > I have tried to look for the thread but no detailed info. Could you give me > some keyword to f

Re: [Xen-devel] Regression due to d9581c7dcac15c02ad4d47c60c60f4d8f197db55 en/fb: allow xenfb initialization for hvm guest

2015-03-03 Thread Stefano Stabellini
On Mon, 2 Mar 2015, Konrad Rzeszutek Wilk wrote: > On Mon, Mar 02, 2015 at 02:38:50PM +, David Vrabel wrote: > > On 02/03/15 14:25, Konrad Rzeszutek Wilk wrote: > > > On Mon, Mar 02, 2015 at 10:47:12AM +, David Vrabel wrote: > > >> On 27/02/15 21:24, Konrad Rzeszutek Wilk wrote: > > >>> Thi

Re: [Xen-devel] [PATCH 1/5] x86: allow specifying the NUMA nodes Dom0 should run on

2015-03-03 Thread Jan Beulich
>>> On 27.02.15 at 15:54, wrote: > On Fri, 2015-02-27 at 10:50 +, Jan Beulich wrote: >> >>> On 27.02.15 at 11:04, wrote: >> > On Fri, 2015-02-27 at 08:46 +, Jan Beulich wrote: >> >> I'm simply adjusting what sched_init_vcpu() did, which is alter >> >> hard affinity conditionally on is_pin

Re: [Xen-devel] [PATCH V2] Add flag to start info regarding virtual mapped p2m list

2015-03-03 Thread Jan Beulich
>>> On 03.03.15 at 11:32, wrote: > On 03/03/2015 11:27 AM, Jan Beulich wrote: >> Also I'm not particularly happy with the name, as it suggests to be >> a statement about the initial P2M the guest gets handed - yet that >> is always virtually mapped. SIF_PERMANENT_VIRT_P2M is getting a >> little lo

Re: [Xen-devel] Regression due to d9581c7dcac15c02ad4d47c60c60f4d8f197db55 en/fb: allow xenfb initialization for hvm guest

2015-03-03 Thread Ian Campbell
On Tue, 2015-03-03 at 10:44 +, Stefano Stabellini wrote: > On Mon, 2 Mar 2015, Konrad Rzeszutek Wilk wrote: > > On Mon, Mar 02, 2015 at 02:38:50PM +, David Vrabel wrote: > > > On 02/03/15 14:25, Konrad Rzeszutek Wilk wrote: > > > > On Mon, Mar 02, 2015 at 10:47:12AM +, David Vrabel wrot

Re: [Xen-devel] [PATCH V2] Add flag to start info regarding virtual mapped p2m list

2015-03-03 Thread Juergen Gross
On 03/03/2015 11:52 AM, Jan Beulich wrote: On 03.03.15 at 11:32, wrote: On 03/03/2015 11:27 AM, Jan Beulich wrote: Also I'm not particularly happy with the name, as it suggests to be a statement about the initial P2M the guest gets handed - yet that is always virtually mapped. SIF_PERMANENT_VI

Re: [Xen-devel] [PATCH v6 02/23] xen: move NUMA_NO_NODE to public memory.h as XEN_NUMA_NO_NODE

2015-03-03 Thread Wei Liu
On Tue, Mar 03, 2015 at 07:44:25AM +, Jan Beulich wrote: > >>> On 02.03.15 at 18:00, wrote: > > There are two issues here irrespective of this series: > > > > 1. should we expose that sentinel in ABI? > > 2. if so, what should the sentinel be? > > > > I think Andrew and you disagree on the f

Re: [Xen-devel] Strange memory balloning problem even if balloning is disabled

2015-03-03 Thread David Vrabel
On 03/03/15 09:05, Fabio Fantoni wrote: > Il 02/03/2015 17:31, David Vrabel ha scritto: >> On 02/03/15 16:23, Jan Beulich wrote: >> On 02.03.15 at 16:52, wrote: On systems with wheezy as dom0 and kernel 3.2 it seems there are no visible problems. After updating the kernel to 3.

[Xen-devel] [PATCH 3/4] freemem: remove call to libxl_wait_for_free_memory

2015-03-03 Thread Stefano Stabellini
Now that libxl_wait_for_memory_target is capable of waiting until dom0 reaches its target, we can remove the other wait function call: libxl_wait_for_free_memory. No need to wait twice. Once dom0 has met its target, simply loop again and recalculate free_memkb. Signed-off-by: Stefano Stabellini -

[Xen-devel] [PATCH 0/4] fix freemem loop

2015-03-03 Thread Stefano Stabellini
Hi all, this patch series fixes the freemem loop on machines with very large amount of memory, where the current wait time is not enough. In order to be able to handle arbitrarly large amount of ram, we implement in libxl_wait_for_memory_target a policy of waiting until dom0 is making progress.

[Xen-devel] [PATCH 4/4] libxl_wait_for_memory_target: wait for 2 sec at a time

2015-03-03 Thread Stefano Stabellini
Use a 2 sec sleep time in the loop to allow the guest to release a decent amount of memory in an iteration (empirical tests show ballooning speed to be 512MB/sec or recent boxes). Signed-off-by: Stefano Stabellini --- tools/libxl/libxl.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(

[Xen-devel] [PATCH 2/4] libxl_wait_for_memory_target: wait as long as dom0 is making progress

2015-03-03 Thread Stefano Stabellini
Decrement wait_secs only if dom0 is making no progress toward reaching the balloon target, otherwise loop again for free. Signed-off-by: Stefano Stabellini Tested-by: Mike Latimer --- tools/libxl/libxl.c | 29 ++--- tools/libxl/xl_cmdimpl.c |4 ++-- 2 files ch

[Xen-devel] [PATCH 1/4] Revert "libxl: Wait for ballooning if free memory is increasing"

2015-03-03 Thread Stefano Stabellini
This reverts commit 2563bca11544361dc2afa5e20b5663e10a0715cb. --- tools/libxl/xl_cmdimpl.c | 17 +++-- 1 file changed, 3 insertions(+), 14 deletions(-) diff --git a/tools/libxl/xl_cmdimpl.c b/tools/libxl/xl_cmdimpl.c index e41f633..f4c4122 100644 --- a/tools/libxl/xl_cmdimpl.c +++ b

Re: [Xen-devel] [PATCH RFC V2 1/5] libxl: add pvusb definitions

2015-03-03 Thread Ian Campbell
On Mon, 2015-01-19 at 16:28 +0800, Chunyan Liu wrote: Sorry for the long delay in replying. > To attach a usb device, a virtual usb controller should be created first. > This patch defines usbctrl and usbdevice related structs. Per <54ca17df026600095...@relay2.provo.novell.com> please could

Re: [Xen-devel] [PATCH RFC V2 2/5] libxl: export some functions for pvusb use

2015-03-03 Thread Ian Campbell
On Mon, 2015-01-19 at 16:28 +0800, Chunyan Liu wrote: > Signed-off-by: Chunyan Liu > Signed-off-by: Simon Cao Acked-by: Ian Campbell ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v1] xen/arm: Do not allocate pte entries for MAP_SMALL_PAGES

2015-03-03 Thread Julien Grall
Hi Ian, On 03/03/2015 10:27, Ian Campbell wrote: On Tue, 2015-03-03 at 13:28 +0530, Vijay Kilari wrote: On Tue, Feb 24, 2015 at 6:29 PM, Julien Grall wrote: On 24/02/15 10:26, Ian Campbell wrote: On Tue, 2015-02-24 at 09:38 +, Julien Grall wrote: Hi Ian, On 24/02/2015 09:31, Ian Campbe

[Xen-devel] [PATCH v8 2/4] xen/arm: Check for interrupt controller directly

2015-03-03 Thread Frediano Ziglio
This check allow to detect mail interrupt controller even if it does not match one of the standard ones. This allow boards with non standard controllers to be handled correctly without having to manually edit the global list every time. Signed-off-by: Frediano Ziglio --- xen/arch/arm/domain_buil

[Xen-devel] [PATCH v8 0/4] xen/arm: Add support for Huawei hip04-d01 platform

2015-03-03 Thread Frediano Ziglio
This set of patches add Xen support for hip04-d01 platform (see https://wiki.linaro.org/Boards/D01 for details). Changes from v7: - updated patches; - moved interrupt controller check in a separate patch (Ian Campbell); - changed conditional include of gic-hip04.c in Makefile (Ian Campbell); - add

[Xen-devel] [PATCH v8 4/4] xen/arm: Force dom0 to use normal GICv2 driver on Hip04 platform

2015-03-03 Thread Frediano Ziglio
Until vGIC support is not implemented and tested, this will prevent guest kernels to use their Hip04 driver, or crash when they don't have any. Signed-off-by: Frediano Ziglio --- xen/arch/arm/gic-hip04.c | 18 +++--- xen/include/asm-arm/gic.h | 4 +++- 2 files changed, 14 insertion

[Xen-devel] [PATCH v8 3/4] xen/arm: Make gic-v2 code handle hip04-d01 platform

2015-03-03 Thread Frediano Ziglio
The GIC in this platform is mainly compatible with the standard GICv2 beside: - ITARGET is extended to 16 bit to support 16 CPUs; - SGI mask is extended to support 16 CPUs; - maximum supported interrupt is 510; - GICH APR and LR register offsets. Signed-off-by: Frediano Ziglio Signed-off-by: Zolt

[Xen-devel] [PATCH v8 1/4] xen/arm: Duplicate gic-v2.c file to support hip04 platform version

2015-03-03 Thread Frediano Ziglio
HiSilison Hip04 platform use a slightly different version. This is just a verbatim copy of the file to workaround git not fully supporting copy operation. Signed-off-by: Frediano Ziglio --- xen/arch/arm/gic-hip04.c | 801 +++ 1 file changed, 801 insert

Re: [Xen-devel] [PATCH v3] ts-xen-install: Enable debug level logging in libvirt

2015-03-03 Thread Ian Campbell
On Tue, 2015-03-03 at 10:22 +, Ian Campbell wrote: > This should make diagnosing libvirt issues a bit easier. > > Refactor the existing loop to edit the xencommons kvp style config > file editing code so it can be reused for libvirtd.conf. > > Signed-off-by: Ian Campbell > Cc: Jim Fehlig >

Re: [Xen-devel] [PATCH DOCDAY v2] netif.h: describe request/response structures in terms of binary layout

2015-03-03 Thread Ian Campbell
On Wed, 2015-02-25 at 13:39 +, Ian Campbell wrote: > In RFC style, rather than relying on the implicit assumptions of a > particular C ABI. > > I have also confirmed, using the Python gdb extension technique in > [0], that the struct offsets (in a Linux binary at least) are the same > as descr

Re: [Xen-devel] [PATCH V2] Add flag to start info regarding virtual mapped p2m list

2015-03-03 Thread Jan Beulich
>>> On 03.03.15 at 12:00, wrote: > On 03/03/2015 11:52 AM, Jan Beulich wrote: > On 03.03.15 at 11:32, wrote: >>> On 03/03/2015 11:27 AM, Jan Beulich wrote: Also I'm not particularly happy with the name, as it suggests to be a statement about the initial P2M the guest gets handed - y

Re: [Xen-devel] [PATCH RFC V2 3/5] libxl: add pvusb API

2015-03-03 Thread Ian Campbell
On Mon, 2015-01-19 at 16:28 +0800, Chunyan Liu wrote: > Add pvusb APIs, including: > - attach/detach (create/destroy) virtual usb controller. > - attach/detach usb device > - list assignable usb devices in host > - some other helper functions > > Signed-off-by: Chunyan Liu > Signed-off-by: Si

Re: [Xen-devel] [PATCH RFC] xen: arm: context switch vtimer PPI state.

2015-03-03 Thread Stefano Stabellini
On Mon, 26 Jan 2015, Ian Campbell wrote: > ... instead of artificially masking the timer interrupt in the timer > state and relying on the guest to unmask (which it isn't required to > do per the h/w spec, although Linux does) > > To do this introduce the concept of routing a PPI to the currently

Re: [Xen-devel] [PATCH RFC V2 4/5] xl: add pvusb commands

2015-03-03 Thread Ian Campbell
On Mon, 2015-01-19 at 16:28 +0800, Chunyan Liu wrote: > Add pvusb commands. > > To attach a usb device to guest through pvusb, one could follow > following example: > > #xl usb-ctrl-attach test_vm version=1 num_ports=8 > > #xl usb-list test_vm > will show the usb controllers and port usage un

Re: [Xen-devel] [RFC PATCH 00/19] xen/arm: Add ITS support

2015-03-03 Thread Julien Grall
Hello Vijay, On 03/03/2015 03:55, Vijay Kilari wrote: On Mon, Mar 2, 2015 at 6:49 PM, Julien Grall wrote: On 02/03/15 12:30, vijay.kil...@gmail.com wrote: From: Vijaya Kumar K Add ITS support for arm. Following major features are supported - GICv3 ITS support for arm64 platform - Suppor

Re: [Xen-devel] [PATCH RFC V2 5/5] domcreate: support pvusb in configuration file

2015-03-03 Thread Ian Campbell
On Mon, 2015-01-19 at 16:28 +0800, Chunyan Liu wrote: > Add code to support pvusb from domain create. One could specify > usb in domain's configuration file and create domain, then usb > device would be attached to guest automatically. > > One could specify usb device in config file like this: > u

Re: [Xen-devel] Strange memory balloning problem even if balloning is disabled

2015-03-03 Thread Fabio Fantoni
Il 03/03/2015 12:09, David Vrabel ha scritto: On 03/03/15 09:05, Fabio Fantoni wrote: Il 02/03/2015 17:31, David Vrabel ha scritto: On 02/03/15 16:23, Jan Beulich wrote: On 02.03.15 at 16:52, wrote: On systems with wheezy as dom0 and kernel 3.2 it seems there are no visible problems. After u

Re: [Xen-devel] [PATCH v1] xen/arm: Do not allocate pte entries for MAP_SMALL_PAGES

2015-03-03 Thread Ian Campbell
On Tue, 2015-03-03 at 11:17 +, Julien Grall wrote: > Hi Ian, > > On 03/03/2015 10:27, Ian Campbell wrote: > > On Tue, 2015-03-03 at 13:28 +0530, Vijay Kilari wrote: > >> On Tue, Feb 24, 2015 at 6:29 PM, Julien Grall > >> wrote: > >>> On 24/02/15 10:26, Ian Campbell wrote: > On Tue, 2015

Re: [Xen-devel] [PATCH RFC] xen: arm: context switch vtimer PPI state.

2015-03-03 Thread Ian Campbell
On Tue, 2015-03-03 at 11:38 +, Stefano Stabellini wrote: > > gic_set_irq_properties(desc, cpumask_of(smp_processor_id()), > > GIC_PRI_IRQ); > > > > -/* Use vcpu0 to retrieve the pending_irq struct. Given that we only > > - * route SPIs to guests, it doesn't make any difference.

Re: [Xen-devel] [PATCH v1] xen/arm: Do not allocate pte entries for MAP_SMALL_PAGES

2015-03-03 Thread Julien Grall
On 03/03/2015 11:47, Ian Campbell wrote: I'm not sure to follow here. Do you think MAP_SMALL_PAGES should be defined as WRITE_ALLOC? Sorry I just meant that: +#define PAGE_HYPERVISOR_WC ((DEV_WC) | PTE_PRESENT ) should be just: +#define PAGE_HYPERVISOR_WC (DEV_WC |

Re: [Xen-devel] [PATCH RFC] xen: arm: context switch vtimer PPI state.

2015-03-03 Thread Ian Campbell
On Mon, 2015-03-02 at 18:42 +, Stefano Stabellini wrote: > On Tue, 10 Feb 2015, Ian Campbell wrote: > > Stefano, > > > > do you have any comments on the viability of the general approach here > > before I go off and start addressing the review comments? > > I think that the general approach i

Re: [Xen-devel] [PATCH v1] xen/arm: Do not allocate pte entries for MAP_SMALL_PAGES

2015-03-03 Thread Ian Campbell
On Tue, 2015-03-03 at 11:51 +, Julien Grall wrote: > > On 03/03/2015 11:47, Ian Campbell wrote: > >> I'm not sure to follow here. Do you think MAP_SMALL_PAGES should be > >> defined as WRITE_ALLOC? > > > > Sorry I just meant that: > > +#define PAGE_HYPERVISOR_WC ((DEV_WC) | PTE_P

Re: [Xen-devel] [PATCH RFC] xen: arm: context switch vtimer PPI state.

2015-03-03 Thread Stefano Stabellini
On Tue, 3 Mar 2015, Ian Campbell wrote: > On Mon, 2015-03-02 at 18:42 +, Stefano Stabellini wrote: > > On Tue, 10 Feb 2015, Ian Campbell wrote: > > > Stefano, > > > > > > do you have any comments on the viability of the general approach here > > > before I go off and start addressing the revie

Re: [Xen-devel] [PATCH RFC] xen: arm: context switch vtimer PPI state.

2015-03-03 Thread Stefano Stabellini
On Tue, 3 Mar 2015, Ian Campbell wrote: > On Tue, 2015-03-03 at 11:38 +, Stefano Stabellini wrote: > > > gic_set_irq_properties(desc, cpumask_of(smp_processor_id()), > > > GIC_PRI_IRQ); > > > > > > -/* Use vcpu0 to retrieve the pending_irq struct. Given that we only > > > - * ro

Re: [Xen-devel] [RFC PATCH v2 00/29] libxl: Cancelling asynchronous operations

2015-03-03 Thread Ian Campbell
On Tue, 2015-02-10 at 20:09 +, Ian Jackson wrote: > This is v2 of my work-in-progress series to support cancellation of > long-running libxl operations. [...] > I wouldn't recommend testing it yet until I've at least smoke tested > it to see that things still work if you don't cancel them. Wou

Re: [Xen-devel] [PATCH 00/18] x86: multiboot2 protocol support

2015-03-03 Thread Ian Campbell
On Fri, 2015-01-30 at 18:54 +0100, Daniel Kiper wrote: > xen/arch/x86/Makefile | 17 ++-- > xen/arch/x86/boot/Makefile|3 +- > xen/arch/x86/boot/head.S | 291 > ++ > xen/arch/x86/boot/reloc.c | 219 >

Re: [Xen-devel] [PATCH RFC] xen: arm: context switch vtimer PPI state.

2015-03-03 Thread Ian Campbell
On Tue, 2015-03-03 at 12:00 +, Stefano Stabellini wrote: > > > > > +/* An edge triggered interrupt should now be pending. */ > > > > > +t->ppi_state.pending = true; > > > > > +vcpu_unblock(t->v); > > > > > > I was going to say that this is trouble because > > > local_ev

Re: [Xen-devel] [PATCH v5] xen/iommu: smmu: Advertise when the SMMU support coherent table walk

2015-03-03 Thread Julien Grall
On 02/03/2015 17:04, Ian Campbell wrote: On Mon, 2015-03-02 at 15:42 +, Julien Grall wrote: I've just noticed that the support on the previous driver (i.e in Xen 4.5) may incorrectly expose this feature when all the SMMUs is not supporting coherent table walk. I'm not

Re: [Xen-devel] [PATCH 1/2] xen: arm: log warning for interrupt configuration mismatch

2015-03-03 Thread Julien Grall
Hi Ian, On 02/03/2015 17:01, Ian Campbell wrote: On Mon, 2015-03-02 at 12:56 +, Julien Grall wrote: From 852f6e3fe49f7fab801f2857b8b505922556d746 Mon Sep 17 00:00:00 2001 From: Ian Campbell Date: Mon, 2 Mar 2015 11:09:35 + Subject: [PATCH] xen: arm: Assume level triggered means high,

Re: [Xen-devel] [PATCH RFC] xen: arm: context switch vtimer PPI state.

2015-03-03 Thread Ian Campbell
On Tue, 2015-03-03 at 12:18 +, Ian Campbell wrote: > > > > > > +/* Xen s/w state */ > > > > > > +unsigned long inprogress:1; > > > > > > +}; > > > > > > > > Using a uint32_t bitmask would be more in line the rest of the code > > > > style, but it is just a matter of taste. > > > > > >

Re: [Xen-devel] [PATCH 00/18] x86: multiboot2 protocol support

2015-03-03 Thread Daniel Kiper
On Tue, Mar 03, 2015 at 12:10:35PM +, Ian Campbell wrote: > On Fri, 2015-01-30 at 18:54 +0100, Daniel Kiper wrote: > > xen/arch/x86/Makefile | 17 ++-- > > xen/arch/x86/boot/Makefile|3 +- > > xen/arch/x86/boot/head.S | 291 > > +

Re: [Xen-devel] [PATCH 00/18] x86: multiboot2 protocol support

2015-03-03 Thread Ian Campbell
On Tue, 2015-03-03 at 13:36 +0100, Daniel Kiper wrote: > On Tue, Mar 03, 2015 at 12:10:35PM +, Ian Campbell wrote: > > On Fri, 2015-01-30 at 18:54 +0100, Daniel Kiper wrote: > > > xen/arch/x86/Makefile | 17 ++-- > > > xen/arch/x86/boot/Makefile|3 +- > > > xen/arch/x

[Xen-devel] [PATCH] xsm/policy: remove gawk-ism line in Makefile

2015-03-03 Thread Wei Liu
Translate gawk regex to mawk regex to allow using mawk. The new regex works on both gawk and mawk. Signed-off-by: Wei Liu Cc: Ian Campbell Cc: Ian Jackson Cc: Daniel De Graaf --- tools/flask/policy/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/flask/policy

Re: [Xen-devel] [xen-unstable test] 35257: regressions - FAIL

2015-03-03 Thread Ian Campbell
On Thu, 2015-02-26 at 20:14 +, xen.org wrote: > flight 35257 xen-unstable real [real] > http://www.chiark.greenend.org.uk/~xensrcts/logs/35257/ > > Regressions :-( > > Tests which did not succeed and are blocking, > including tests which could not be run: > test-armhf-armhf-libvirt 12 gu

Re: [Xen-devel] [xen-unstable test] 35257: regressions - FAIL

2015-03-03 Thread Ian Campbell
On Tue, 2015-03-03 at 12:44 +, Ian Campbell wrote: > On Thu, 2015-02-26 at 20:14 +, xen.org wrote: > > version targeted for testing: > > xen 24b2b8dea180098a3acc809a91cde6c0cc4c8607 > > I have force pushed this revision. [...] > + echo 'branch 24b2b8dea180098a3acc809a91cd

Re: [Xen-devel] [PATCH v3 12/24] xen/arm: Release IRQ routed to a domain when it's destroying

2015-03-03 Thread Julien Grall
Hi Ian, On 20/02/2015 17:41, Julien Grall wrote: >>> +/* Only SPIs are supported */ +if ( virq < 32 || virq >= vgic_num_irqs(d) ) +return -EINVAL; + +p = irq_to_pending(d->vcpu[0], virq); +if ( !p->desc ) +return -EINVAL; Are we seeing this pattern a lot? It se

Re: [Xen-devel] [PATCH 00/18] x86: multiboot2 protocol support

2015-03-03 Thread Daniel Kiper
On Tue, Mar 03, 2015 at 12:39:45PM +, Ian Campbell wrote: > On Tue, 2015-03-03 at 13:36 +0100, Daniel Kiper wrote: > > On Tue, Mar 03, 2015 at 12:10:35PM +, Ian Campbell wrote: > > > On Fri, 2015-01-30 at 18:54 +0100, Daniel Kiper wrote: > > > > xen/arch/x86/Makefile | 17 ++-

[Xen-devel] [rumpuserxen test] 35788: regressions - FAIL

2015-03-03 Thread xen . org
flight 35788 rumpuserxen real [real] http://www.chiark.greenend.org.uk/~xensrcts/logs/35788/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-i386-rumpuserxen6 xen-build fail REGR. vs. 33866 build-amd64-rumpuserx

Re: [Xen-devel] kasan_map_early_shadow() on Xen

2015-03-03 Thread Andrey Ryabinin
On 03/03/2015 12:40 PM, Luis R. Rodriguez wrote: > Andrey, > > I believe that on Xen we should disable kasan, would like confirmation I guess Xen guests won't work with kasan because Xen guests doesn't setup shadow (kasan_map_early_shadow() is not called in xen guests). Disabling kasan for Xen i

[Xen-devel] [OSSTEST PATCH 2/3] allow files: Support negation syntax

2015-03-03 Thread Ian Jackson
We now support !, which undoes the effect of a specified previously (perhaps in another file). Note that the logic is a bit poor: !... only undoes the effect of an identical (or equivalent) earlier pattern; it doesn't provide a specific override. Improving that would involve a more invasive chan

[Xen-devel] [OSSTEST PATCH 1/3] allow files: Pass branch-specific allow file last to sg-report-flight

2015-03-03 Thread Ian Jackson
We are going to make it possible for a branch-specific allow file to override general allow information from allow.all. No functional change yet. Signed-off-by: Ian Jackson --- cri-args-hostlists |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cri-args-hostlists b/cri-arg

[Xen-devel] [OSSTEST PATCH 3/3] allow files: Allow all libvirt test failures on other branches

2015-03-03 Thread Ian Jackson
We have a problem where virtd crashes, and this is causing problems for other branches. So ignore libvirt test failures. Use the new ability to override allowances to cause libvirt failures to still block libvirt pushes. Signed-off-by: Ian Jackson --- allow.all |2 +- allow.libvirt |

Re: [Xen-devel] [PATCH] xsm/policy: remove gawk-ism line in Makefile

2015-03-03 Thread Ian Campbell
On Tue, 2015-03-03 at 12:44 +, Wei Liu wrote: > Translate gawk regex to mawk regex to allow using mawk. The new regex > works on both gawk and mawk. I think [[:alpha:]] might include A-Z too, but I presume that doesn't affect our inputs or your tests would have failed. So for my part: > Signe

Re: [Xen-devel] [PATCH v9 01/13] hvm: Move MAX_INST_LEN into x86_emulate.h

2015-03-03 Thread George Dunlap
On 02/16/2015 11:05 PM, Don Slutz wrote: > Change some hard coded 15 into MAX_INST_LEN > > Signed-off-by: Don Slutz Since it looks like you'll be re-sending the series anyway, it might be nice to include "No functional changes" in the description, just to make it clear. -George _

Re: [Xen-devel] [PATCH v9 01/13] hvm: Move MAX_INST_LEN into x86_emulate.h

2015-03-03 Thread Andrew Cooper
On 03/03/15 14:02, George Dunlap wrote: > On 02/16/2015 11:05 PM, Don Slutz wrote: >> Change some hard coded 15 into MAX_INST_LEN >> >> Signed-off-by: Don Slutz > Since it looks like you'll be re-sending the series anyway, it might be > nice to include "No functional changes" in the description, j

Re: [Xen-devel] [PATCH v9 01/13] hvm: Move MAX_INST_LEN into x86_emulate.h

2015-03-03 Thread George Dunlap
On 03/03/2015 02:08 PM, Andrew Cooper wrote: > On 03/03/15 14:02, George Dunlap wrote: >> On 02/16/2015 11:05 PM, Don Slutz wrote: >>> Change some hard coded 15 into MAX_INST_LEN >>> >>> Signed-off-by: Don Slutz >> Since it looks like you'll be re-sending the series anyway, it might be >> nice to

Re: [Xen-devel] [PATCH v9 03/13] tools: Add vmware_hwver support

2015-03-03 Thread Ian Campbell
On Mon, 2015-02-16 at 18:05 -0500, Don Slutz wrote: > diff --git a/tools/libxl/libxl.h b/tools/libxl/libxl.h > index c219f59..0c27e5c 100644 > --- a/tools/libxl/libxl.h > +++ b/tools/libxl/libxl.h > @@ -163,6 +163,16 @@ > #define LIBXL_HAVE_BUILDINFO_HVM_MMIO_HOLE_MEMKB 1 > > /* > + * The libxl

Re: [Xen-devel] kasan_map_early_shadow() on Xen

2015-03-03 Thread Konrad Rzeszutek Wilk
On Tue, Mar 03, 2015 at 04:15:06PM +0300, Andrey Ryabinin wrote: > On 03/03/2015 12:40 PM, Luis R. Rodriguez wrote: > > Andrey, > > > > I believe that on Xen we should disable kasan, would like confirmation > > I guess Xen guests won't work with kasan because Xen guests doesn't setup > shadow >

Re: [Xen-devel] [PATCH v9 07/13] tools: Add vmware_port support

2015-03-03 Thread Ian Campbell
On Mon, 2015-02-16 at 18:05 -0500, Don Slutz wrote: > +=item B > + > +Turns on or off the exposure of VMware port. This is known as > +vmport in QEMU. Also called VMware Backdoor I/O Port. Not all > +defined VMware backdoor commands are implemented. All of the > +ones that Linux kernel uses ar

Re: [Xen-devel] [PATCH v9 08/13] Add IOREQ_TYPE_VMWARE_PORT

2015-03-03 Thread Ian Campbell
On Mon, 2015-02-16 at 18:05 -0500, Don Slutz wrote: > tools/libxc/xc_hvm_build_x86.c | 5 +- > tools/libxl/libxl_dm.c | 4 + I've only looked at these bits. > diff --git a/tools/libxl/libxl_dm.c b/tools/libxl/libxl_dm.c > index c27f9a4..620013c 100644 > --- a/tools/li

  1   2   3   >