[Xen-devel] [PATCH 12/32] cmdline: Parse -pciemulation before trying to use it

2015-04-22 Thread Andrew Cooper
From: Kaifeng Zhu kaifeng@citrix.com Signed-off-by: Kaifeng Zhu kaifeng@citrix.com Reviewed-by: Andrew Cooper andrew.coop...@citrix.com --- vl.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/vl.c b/vl.c index d21c3aa..67d9d86 100644 --- a/vl.c

[Xen-devel] [PATCH 31/32] block-raw-posix: Fix memory leak in posix_aio_init()

2015-04-22 Thread Andrew Cooper
From: Yunlei Ding yunlei.d...@citrix.com Free allocated memory s before return. Signed-off-by: Yunlei Ding yunlei.d...@citrix.com Coverity-ID: 1055915 Reviewed-by: Andrew Cooper andrew.coop...@citrix.com --- block-raw-posix.c |1 + 1 file changed, 1 insertion(+) diff --git

[Xen-devel] [PATCH 32/32] block-vvfat: fix resource leaks in read_directory()

2015-04-22 Thread Andrew Cooper
From: Yunlei Ding yunlei.d...@citrix.com Signed-off-by: Yunlei Ding yunlei.d...@citrix.com Coverity-IDs: 1055920 1055921 Reviewed-by: Andrew Cooper andrew.coop...@citrix.com --- block-vvfat.c |3 +++ 1 file changed, 3 insertions(+) diff --git a/block-vvfat.c b/block-vvfat.c index

[Xen-devel] [PATCH 22/32] hw/ide: fix memory leak from qemu_allocate_irqs()

2015-04-22 Thread Andrew Cooper
From: Kaifeng Zhu kaifeng@citrix.com qemu_allocate_irqs would return an array of irqs, not store the allocated array pointer, and subsequently leak it. Signed-off-by: Kaifeng Zhu kaifeng@citrix.com (defects not identified by Coverity Scan) Reviewed-by: Andrew Cooper

[Xen-devel] [PATCH 26/32] block-vvfat: fix memory leak in check_directory_consistency()

2015-04-22 Thread Andrew Cooper
From: Kaifeng Zhu kaifeng@citrix.com Memory pointed by cluster leaks in error handling code. Signed-off-by: Kaifeng Zhu kaifeng@citrix.com Coverity-ID: 1055917 Reviewed-by: Andrew Cooper andrew.coop...@citrix.com --- block-vvfat.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[Xen-devel] [PATCH 24/32] net: Fix memory/handle leaks in net_socket_listen_init()

2015-04-22 Thread Andrew Cooper
From: Kaifeng Zhu kaifeng@citrix.com fd and s could be leaked in case bind/listen failed. Signed-off-by: Kaifeng Zhu kaifeng@citrix.com Coverity-IDs: 1055923 1055924 Reviewed-by: Andrew Cooper andrew.coop...@citrix.com --- net.c |5 + 1 file changed, 5 insertions(+) diff --git

[Xen-devel] [PATCH 19/32] console: Avoid overrunning the dmask arrays

2015-04-22 Thread Andrew Cooper
From: Kaifeng Zhu kaifeng@citrix.com The valide range of font_data should be [0, 0xFF]. Signed-off-by: Kaifeng Zhu kaifeng@citrix.com (defects not identified by Coverity Scan) Reviewed-by: Andrew Cooper andrew.coop...@citrix.com --- console.c |9 + 1 file changed, 5

[Xen-devel] [PATCH 30/32] block-nbd: close sock in nbd_open() error path

2015-04-22 Thread Andrew Cooper
From: Yunlei Ding yunlei.d...@citrix.com Close sock handle before return. Signed-off-by: Yunlei Ding yunlei.d...@citrix.com Coverity-ID: 1055914 Reviewed-by: Andrew Cooper andrew.coop...@citrix.com --- block-nbd.c |3 +++ 1 file changed, 3 insertions(+) diff --git a/block-nbd.c

[Xen-devel] [PATCHv6 2/5] gnttab: introduce per-active entry locks

2015-04-22 Thread David Vrabel
From: Matt Wilson m...@amazon.com Instead of protecting the state of a grant table active entry with the grant table lock, use per active entry locks. Active entries must be acquired with active_entry_acquire() and released with active_entry_release() which lock and unlock the entry's spinlock.

[Xen-devel] [PATCHv6 4/5] gnttab: remove unnecessary grant table locks

2015-04-22 Thread David Vrabel
From: Malcolm Crossley malcolm.cross...@citrix.com The grant table lock is not required to protect reads of the table version, the active entry array, or the map track array. This is safe because: a) the grant table version only changes once from 0 to 1 or 2; b) the active entry array only

[Xen-devel] [PATCHv6 3/5] gnttab: split grant table lock into table and maptrack locks

2015-04-22 Thread David Vrabel
From: Matt Wilson m...@amazon.com The maptrack lock protects the maptrack state only. Signed-off-by: Matt Wilson m...@amazon.com Signed-off-by: David Vrabel david.vra...@citrix.com --- Subsequent changes make both these locks uncontented. Is this patch really necessary? -- dvrabel ---

[Xen-devel] [PATCHv6 0/6] gnttab: Improve scaleability

2015-04-22 Thread David Vrabel
The series makes the grant table locking for fine-grained and add per-VCPU maptrack free lists, which greatly improves scalability. The series builds on the original series by Matt Wilson and Christoph Egger from Amazon. The per-VCPU maptrack free lists makes one of our aggregate intrahost

Re: [Xen-devel] [PATCH v4 2/2] IB/qib: use arch_phys_wc_add()

2015-04-22 Thread Doug Ledford
On Wed, 2015-04-22 at 17:33 +0200, Luis R. Rodriguez wrote: On Wed, Apr 22, 2015 at 09:54:38AM -0400, Doug Ledford wrote: On Tue, 2015-04-21 at 14:50 -0700, Luis R. Rodriguez wrote: This: + /* MTRR was used if this is non-zero */ + if (!dd-wc_cookie) vma-vm_page_prot =

Re: [Xen-devel] [PATCH] libxl: fix xl mem-set regression from 0c029c4da2

2015-04-22 Thread Ian Campbell
On Wed, 2015-04-22 at 17:33 +0100, Jan Beulich wrote: On 22.04.15 at 17:36, ian.campb...@citrix.com wrote: On Wed, 2015-04-22 at 15:41 +0100, Jan Beulich wrote: On 22.04.15 at 16:01, ian.campb...@citrix.com wrote: On Wed, 2015-04-22 at 13:02 +0100, Jan Beulich wrote: Said commit

Re: [Xen-devel] [PATCH v4 2/2] IB/qib: use arch_phys_wc_add()

2015-04-22 Thread Luis R. Rodriguez
On Wed, Apr 22, 2015 at 12:57:18PM -0400, Doug Ledford wrote: On Wed, 2015-04-22 at 17:33 +0200, Luis R. Rodriguez wrote: On Wed, Apr 22, 2015 at 09:54:38AM -0400, Doug Ledford wrote: On Tue, 2015-04-21 at 14:50 -0700, Luis R. Rodriguez wrote: This: + /* MTRR was used if

Re: [Xen-devel] [PATCH v4 2/2] IB/qib: use arch_phys_wc_add()

2015-04-22 Thread Doug Ledford
On Wed, 2015-04-22 at 19:37 +0200, Luis R. Rodriguez wrote: On Wed, Apr 22, 2015 at 12:57:18PM -0400, Doug Ledford wrote: On Wed, 2015-04-22 at 17:33 +0200, Luis R. Rodriguez wrote: On Wed, Apr 22, 2015 at 09:54:38AM -0400, Doug Ledford wrote: On Tue, 2015-04-21 at 14:50 -0700, Luis R.

Re: [Xen-devel] [PATCH v4 2/2] IB/qib: use arch_phys_wc_add()

2015-04-22 Thread Luis R. Rodriguez
On Wed, Apr 22, 2015 at 01:48:27PM -0400, Doug Ledford wrote: On Wed, 2015-04-22 at 19:37 +0200, Luis R. Rodriguez wrote: On Wed, Apr 22, 2015 at 12:57:18PM -0400, Doug Ledford wrote: On Wed, 2015-04-22 at 17:33 +0200, Luis R. Rodriguez wrote: On Wed, Apr 22, 2015 at 09:54:38AM -0400,

[Xen-devel] [PATCH v5 2/2] IB/qib: use arch_phys_wc_add()

2015-04-22 Thread Luis R. Rodriguez
From: Luis R. Rodriguez mcg...@suse.com This driver already makes use of ioremap_wc() on PIO buffers, so convert it to use arch_phys_wc_add(). The qib driver uses a mmap() special case for when PAT is not used, this behaviour used to be determined with a module parameter but since we have been

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

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

[Xen-devel] [PATCH] [media] ivtv: use arch_phys_wc_add() and require PAT disabled

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

[Xen-devel] Possible bug/question in xen-hptool?

2015-04-22 Thread Meng Xu
Hi, I was looking at using xen-hptool (tool/misc/xen-hptool.c) to make one page of a guest domain offline. I created a guest domain on Xen unstable:​ # xen-mfndump dump-p2m 1 I have dom1's mfn of pfn (0x1d): pfn=0x1d == mfn=0x14ee17 (type 0x0) ​Run `lookup-pte` to find the mfn of the pte of mfn

Re: [Xen-devel] [OSSTEST Nested PATCH v8 3/7] Edit some APIs in TestSupport.pm for nested test

2015-04-22 Thread Hu, Robert
-Original Message- From: Ian Jackson [mailto:ian.jack...@eu.citrix.com] Sent: Wednesday, April 22, 2015 8:50 PM To: Ian Campbell Cc: Pang, LongtaoX; xen-devel@lists.xen.org; wei.l...@citrix.com; Hu, Robert Subject: Re: [OSSTEST Nested PATCH v8 3/7] Edit some APIs in TestSupport.pm

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

2015-04-22 Thread Boris Ostrovsky
When resuming, the guest needs to check whether the port has changed. HVM guests use this parameter to get the port number. (We can't always use xenstore where this value is also written: for example on Linux the console is resumed very early, before the store is up). Signed-off-by: Boris

Re: [Xen-devel] [PATCH] xen: Add __GFP_DMA flag when xen_swiotlb_init gets free pages.

2015-04-22 Thread Chen Baozi
On Tue, Apr 21, 2015 at 12:11:01PM +0100, Stefano Stabellini wrote: Chen, could you please try the patch below in your repro scenario? I have only build tested it. --- xen: Add __GFP_DMA flag when xen_swiotlb_init gets free pages on ARM From: Chen Baozi baoz...@gmail.com Make sure

Re: [Xen-devel] [PATCH] libxl: document foreground '-F' option of create command

2015-04-22 Thread Giuseppe Mazzotta
On 04/22/2015 04:40 PM, Giuseppe Mazzotta wrote: On 04/22/2015 04:21 PM, Ian Campbell wrote: On Fri, 2015-04-17 at 17:36 +0200, Giuseppe Mazzotta wrote: Do you fancy also adding -F to docs/man/xl.pod.1? Yes, I will follow-up in a few hours with another patch for that. Please disregard my

[Xen-devel] [PATCH] docs/xl: document -F options of create subcommand

2015-04-22 Thread Giuseppe Mazzotta
From: Giuseppe g.mazzo...@iragan.com Document '-F' option. Other options are still missing and not in this patch. Signed-off-by: Giuseppe Mazzotta g.mazzo...@iragan.com --- docs/man/xl.pod.1 | 7 +++ 1 file changed, 7 insertions(+) diff --git a/docs/man/xl.pod.1 b/docs/man/xl.pod.1 index

[Xen-devel] [PATCH 1/2] xen/arm: gic: Refactor the code for creating gic node

2015-04-22 Thread Suravee Suthikulpanit
Since fdt_begin_node() is called by all gicXX_make_dt_node() to create the interrupt-controller devicetree node, this patch refactors the call and moves it inside make_gic_node(). This also matches the fdt_end_node() call at the end of make_gic_node(). This patch also move the call to

[Xen-devel] [PATCH 2/2] xen/arm: gicv2: Adding support for GICv2m in Dom0

2015-04-22 Thread Suravee Suthikulpanit
This patch detect and propagate the gic-v2m-frame devicetree sub-node. This allows Dom0 kernel to setup and intialize GICv2m MSI frame. Signed-off-by: Suravee Suthikulpanit suravee.suthikulpa...@amd.com --- xen/arch/arm/gic-v2.c | 169 ++ 1 file

[Xen-devel] [PATCH 0/2] Introducing GICv2m Supports

2015-04-22 Thread Suravee Suthikulpanit
This patch series introduce GICv2m supports in Xen Dom0. This patch series depend on: [PATCH v3 0/5] xen: arm: Parse PCI DT nodes' ranges and interrupt-map http://lists.xen.org/archives/html/xen-devel/2015-04/msg02200.html This has been tested on AMD Seattle platform with the following

Re: [Xen-devel] [OSSTEST Nested PATCH v8 4/7] Changes on test step of Debian hvm guest install

2015-04-22 Thread Robert Hu
On Tue, 2015-04-21 at 11:28 +0100, Ian Campbell wrote: On Mon, 2015-04-13 at 17:19 -0400, longtao.pang wrote: 1. Increase disk size to accommodate to nested test requirement. 2. Since 'Debain-xxx-.iso' image will be stored in rootfs of L1 guest, therefore needs more disk capacity, increase

[Xen-devel] [PATCH 2/2] docs/xl: document -e and -F options of create subcommand

2015-04-22 Thread Giuseppe Mazzotta
From: Giuseppe g.mazzo...@iragan.com While documenting '-F' option, I discovered that not only '-e' was missing (and hereby added with this patch) but others too. I will eventually submit other patches for the remaining missing options, as this one focuses only on '-e' and '-F'. Signed-off-by:

[Xen-devel] [PATCH 1/2] libxl: reword command-line help text for -e and -F options of create subcommand

2015-04-22 Thread Giuseppe Mazzotta
Use a better wording about what the -e and -F options do; this is evident by looking at main_create(), as the only difference is the setting of 'monitor' variable. The help text for '-e' was possibly a copy-paste artifact. Signed-off-by: Giuseppe Mazzotta g.mazzo...@iragan.com ---

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

2015-04-22 Thread Olaf Hering
On Tue, Apr 21, Konrad Rzeszutek Wilk wrote: On Fri, Apr 17, 2015 at 08:30:58AM +, Olaf Hering wrote: +=item Bvscsi=[ VSCSI_SPEC_STRING, VSCSI_SPEC_STRING, ...] + +Specifies the PVSCSI devices to be provided to the guest. PVSCSI passes +dom0 SCSI devices as-is to the guest.

Re: [Xen-devel] [OSSTEST Nested PATCH v8 3/7] Edit some APIs in TestSupport.pm for nested test

2015-04-22 Thread Pang, LongtaoX
-Original Message- From: Ian Campbell [mailto:ian.campb...@citrix.com] Sent: Tuesday, April 21, 2015 10:44 PM To: Ian Jackson Cc: Pang, LongtaoX; xen-devel@lists.xen.org; wei.l...@citrix.com; Hu, Robert Subject: Re: [OSSTEST Nested PATCH v8 3/7] Edit some APIs in TestSupport.pm

[Xen-devel] my maile to post to xen-devel

2015-04-22 Thread Esmail Asyabi
esmail.asy...@gmail.com ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [OSSTEST Nested PATCH v8 5/7] Add new script to customize nested test configuration

2015-04-22 Thread Pang, LongtaoX
-Original Message- From: Ian Campbell [mailto:ian.campb...@citrix.com] Sent: Tuesday, April 21, 2015 6:40 PM To: Pang, LongtaoX Cc: xen-devel@lists.xen.org; ian.jack...@eu.citrix.com; wei.l...@citrix.com; Hu, Robert Subject: Re: [OSSTEST Nested PATCH v8 5/7] Add new script to

Re: [Xen-devel] [OSSTEST PATCH] target_cmd_build: Delete build-ok-stamp before starting

2015-04-22 Thread Ian Campbell
On Tue, 2015-04-21 at 17:40 +0100, Ian Jackson wrote: Many of the callers of target_cmd_build use a build-ok-stamp idiom to detect failed builds. This idiom does not work if the stamp file exists already, so delete it. In the future we may move more of the test build-ok-stamp, echo ok,

Re: [Xen-devel] [PATCH v2 4/6] raisin: pass --with-system-seabios with seabios was built

2015-04-22 Thread Ian Campbell
On Tue, 2015-04-21 at 17:48 +0100, Stefano Stabellini wrote: Detect whether we have built seabios and only pass the relative command line argument to Xen if we actually did. For this and the following ovmf if we didn't build seabios/ovmf here then you pass nothing, which will result in xen.git

Re: [Xen-devel] [OSSTEST Nested PATCH v8 3/7] Edit some APIs in TestSupport.pm for nested test

2015-04-22 Thread Ian Campbell
On Wed, 2015-04-22 at 08:25 +, Pang, LongtaoX wrote: -Original Message- From: Ian Campbell [mailto:ian.campb...@citrix.com] Sent: Tuesday, April 21, 2015 10:44 PM To: Ian Jackson Cc: Pang, LongtaoX; xen-devel@lists.xen.org; wei.l...@citrix.com; Hu, Robert Subject: Re:

Re: [Xen-devel] [OSSTEST Nested PATCH v8 5/7] Add new script to customize nested test configuration

2015-04-22 Thread Ian Campbell
On Wed, 2015-04-22 at 08:35 +, Pang, LongtaoX wrote: -Original Message- From: Ian Campbell [mailto:ian.campb...@citrix.com] Sent: Tuesday, April 21, 2015 6:40 PM To: Pang, LongtaoX Cc: xen-devel@lists.xen.org; ian.jack...@eu.citrix.com; wei.l...@citrix.com; Hu,

Re: [Xen-devel] OpenStack - Libvirt+Xen CI overview : anyone interested in a walk/through or presentation on how it works

2015-04-22 Thread Lars Kurth
Hi folks, last week we were focussed on getting http://jenkins.openstack.xenproject.org/view/Dashboard/builds into a decent state, so we decided to defer this. Now most of the tests have been in green status for a few days, so it may make sense to arrange a day and time. With the timezones

Re: [Xen-devel] Archiving Xen on ARM and PVOPS subprojects

2015-04-22 Thread Lars Kurth
Any other votes by committers (in the TO list) before I tally the votes? Regards Lars On 17 Apr 2015, at 14:26, Stefano Stabellini stefano.stabell...@eu.citrix.com wrote: On Wed, 15 Apr 2015, Lars Kurth wrote: [snip] We would probably also put together a blog post, once all of this has

Re: [Xen-devel] [OSSTEST Nested PATCH v8 6/7] Compose the main recipe of nested test job

2015-04-22 Thread Ian Campbell
On Wed, 2015-04-22 at 08:38 +, Pang, LongtaoX wrote: -Original Message- From: Ian Campbell [mailto:ian.campb...@citrix.com] Sent: Tuesday, April 21, 2015 6:49 PM To: Pang, LongtaoX Cc: xen-devel@lists.xen.org; ian.jack...@eu.citrix.com; wei.l...@citrix.com; Hu,

Re: [Xen-devel] Archiving Xen on ARM and PVOPS subprojects

2015-04-22 Thread Tim Deegan
At 17:23 +0100 on 15 Apr (1429118591), Lars Kurth wrote: Hi all, I wanted to make the proposal to archive the following two subproject on the grounds that they completed their goals a) http://xenproject.org/developers/teams/pvops.html http://xenproject.org/developers/teams/pvops.html

Re: [Xen-devel] [OSSTEST Nested PATCH v8 6/7] Compose the main recipe of nested test job

2015-04-22 Thread Ian Campbell
On Wed, 2015-04-22 at 12:04 +0100, Ian Campbell wrote: On Wed, 2015-04-22 at 08:38 +, Pang, LongtaoX wrote: -Original Message- From: Ian Campbell [mailto:ian.campb...@citrix.com] Sent: Tuesday, April 21, 2015 6:49 PM To: Pang, LongtaoX Cc: xen-devel@lists.xen.org;

[Xen-devel] [PATCH] domctl: don't truncate XEN_DOMCTL_max_mem requests

2015-04-22 Thread Jan Beulich
Instead saturate the value if the input can't be represented in the respective struct domain field. Signed-off-by: Jan Beulich jbeul...@suse.com --- a/xen/common/domctl.c +++ b/xen/common/domctl.c @@ -944,7 +944,7 @@ long do_domctl(XEN_GUEST_HANDLE_PARAM(xe case XEN_DOMCTL_max_mem: {

Re: [Xen-devel] [OSSTEST Nested PATCH v8 6/7] Compose the main recipe of nested test job

2015-04-22 Thread Pang, LongtaoX
-Original Message- From: Ian Campbell [mailto:ian.campb...@citrix.com] Sent: Tuesday, April 21, 2015 6:49 PM To: Pang, LongtaoX Cc: xen-devel@lists.xen.org; ian.jack...@eu.citrix.com; wei.l...@citrix.com; Hu, Robert Subject: Re: [OSSTEST Nested PATCH v8 6/7] Compose the main

[Xen-devel] [PATCH] libxl: fix xl mem-set regression from 0c029c4da2

2015-04-22 Thread Jan Beulich
Said commit (libxl_set_memory_target: retain the same maxmem offset on top of the current target) caused a regression for xl mem-set against Dom0: While prior to creation of the first domain this works, the first domain creation involving ballooning breaks. Due to enforce not being set in the

Re: [Xen-devel] [PATCH] domctl: don't truncate XEN_DOMCTL_max_mem requests

2015-04-22 Thread Andrew Cooper
On 22/04/15 12:56, Jan Beulich wrote: Instead saturate the value if the input can't be represented in the respective struct domain field. Signed-off-by: Jan Beulich jbeul...@suse.com Reviewed-by: Andrew Cooper andrew.coop...@citrix.com Do I take it you have a 16TB guest to hand? ---

Re: [Xen-devel] [PATCH] domctl: don't truncate XEN_DOMCTL_max_mem requests

2015-04-22 Thread Jan Beulich
On 22.04.15 at 14:03, andrew.coop...@citrix.com wrote: On 22/04/15 12:56, Jan Beulich wrote: Instead saturate the value if the input can't be represented in the respective struct domain field. Signed-off-by: Jan Beulich jbeul...@suse.com Reviewed-by: Andrew Cooper

<    1   2