[Xen-devel] [distros-debian-snapshot test] 66613: tolerable trouble: blocked/broken

2016-07-19 Thread Platform Team regression test user
flight 66613 distros-debian-snapshot real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/66613/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): build-armhf-pvops 3 host-install(3) broken like 66558 build-armhf

Re: [Xen-devel] [PATCH 2/3] xen: Have schedulers revise initial placement

2016-07-19 Thread Dario Faggioli
On Mon, 2016-07-18 at 22:36 +0100, Andrew Cooper wrote: > On 18/07/2016 19:55, Dario Faggioli wrote: > > > > On Mon, 2016-07-18 at 19:10 +0100, Andrew Cooper wrote: > > > > > > On 16/07/16 15:12, Dario Faggioli wrote: > > > > > > > > On Fri, 2016-07-15 at 19:07 +0100, Andrew Cooper wrote: > > >

[Xen-devel] [PATCH] XSM-docs: Flask operates on domain types and not on

2016-07-19 Thread Anshul Makkar
Signed-off-by: Anshul Makkar --- docs/misc/xsm-flask.txt | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/docs/misc/xsm-flask.txt b/docs/misc/xsm-flask.txt index 62f15dd..bf8bb6e 100644 --- a/docs/misc/xsm-flask.txt +++ b/docs/misc/xsm-flask.txt @@ -9,8 +9,8 @@ controls

Re: [Xen-devel] [PATCH v7] x86/mem-sharing: mem-sharing a range of memory

2016-07-19 Thread Julien Grall
Hello Tamas, On 18/07/2016 22:14, Tamas K Lengyel wrote: diff --git a/tools/libxc/include/xenctrl.h b/tools/libxc/include/xenctrl.h index e904bd5..0ca94cd 100644 --- a/tools/libxc/include/xenctrl.h +++ b/tools/libxc/include/xenctrl.h @@ -2334,6 +2334,21 @@ int xc_memshr_add_to_physmap(xc_interfa

Re: [Xen-devel] [PATCH] XSM-docs: Flask operates on domain types and not on

2016-07-19 Thread Wei Liu
Hi Anshul Your email subject line seems to be incomplete. Wei. On Tue, Jul 19, 2016 at 08:33:01AM +0100, Anshul Makkar wrote: > Signed-off-by: Anshul Makkar > --- > docs/misc/xsm-flask.txt | 8 > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/docs/misc/xsm-flask.txt

Re: [Xen-devel] unable to compile android-hikey-linaro-4.1

2016-07-19 Thread Wei Liu
On Mon, Jul 18, 2016 at 02:23:54PM -0400, Kamenee Arumugam wrote: > Hi All, > > I am referring to this wiki page (http://wiki.xen.org/wiki/HiKey) to setup > xen in hikey. I am currently facing problem in building linux ( > android-hikey-linaro-4.1 >

Re: [Xen-devel] Is: Revert c5ad33184354260be6d05de57e46a5498692f6d6 "mm/swap.c: flush lru pvecs on compound page arrival" from stable tree? Was:[osstest-ad...@xenproject.org: [linux-4.1 bisection] com

2016-07-19 Thread Michal Hocko
[CCing Sasha] On Mon 18-07-16 11:30:46, Konrad Rzeszutek Wilk wrote: > Hey Lukasz, > > We found that your patch in the automated Xen test-case ends up > OOMing the box when trying to install guests. This worked prior > to your patch. > > See serial log: > http://logs.test-lab.xenproject.org/osst

Re: [Xen-devel] unable to compile android-hikey-linaro-4.1

2016-07-19 Thread Julien Grall
On 18/07/2016 19:23, Kamenee Arumugam wrote: Hi All, Hello, I am referring to this wiki page (http://wiki.xen.org/wiki/HiKey) to setup xen in hikey. I am currently facing problem in building linux ( android-hikey-linaro-4.1

[Xen-devel] [PATCH 0/9] mini-os: prepare for support of ballooning

2016-07-19 Thread Juergen Gross
Do some cleanups, a small correction and add some basic features to lay groundwork for support of ballooning in Mini-OS. The main visible change is the virtual memory layout: to be able to add memory to the running Mini-OS we need to have some spare areas especially after the 1:1 mapping of physic

[Xen-devel] [PATCH 1/9] mini-os: correct first free pfn

2016-07-19 Thread Juergen Gross
The first free pfn available for allocation is calculated by adding the number of page table frames to the pfn of the first page table and then the magic number 3 to account for start info page et al. As the start info page, xenstore page and console page are allocated _before_ the page tables lea

[Xen-devel] [PATCH 2/9] mini-os: remove unused alloc_contig_pages() function

2016-07-19 Thread Juergen Gross
alloc_contig_pages() is never used anywhere in mini-os. Remove it. Signed-off-by: Juergen Gross --- arch/x86/mm.c | 142 -- include/mm.h | 1 - 2 files changed, 143 deletions(-) diff --git a/arch/x86/mm.c b/arch/x86/mm.c index ae1036e..

[Xen-devel] [PATCH 7/9] mini-os: add ballooning config item

2016-07-19 Thread Juergen Gross
Add CONFIG_BALLOON defaulting to 'n' as a config item to Mini-OS. Signed-off-by: Juergen Gross --- Makefile | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Makefile b/Makefile index 2e4bdba..61d960c 100644 --- a/Makefile +++ b/Makefile @@ -33,6 +33,7 @@ CONFIG_CONSFRONT ?= y CONFIG_XENBUS

[Xen-devel] [PATCH 4/9] mini-os: add description of x86 memory usage

2016-07-19 Thread Juergen Gross
Add a brief description how the physical and virtual address usage looks like on x86 to include/x86/arch_mm.h Signed-off-by: Juergen Gross --- include/x86/arch_mm.h | 20 1 file changed, 20 insertions(+) diff --git a/include/x86/arch_mm.h b/include/x86/arch_mm.h index 58f29

[Xen-devel] [PATCH 3/9] mini-os: remove MM_DEBUG code

2016-07-19 Thread Juergen Gross
mm.c contains unused code inside #ifdef MM_DEBUG areas. Its usability is rather questionable and some parts are even wrong (e.g. print_chunks() called with nr_pages > 1000 will clobber an arbitrary stack content with a 0 byte). Remove this code. Signed-off-by: Juergen Gross --- mm.c | 60 --

[Xen-devel] [PATCH 6/9] mini-os: let memory allocation fail if no free page available

2016-07-19 Thread Juergen Gross
Instead of panicing when no page can be allocated try to fail the memory allocation by returning NULL instead. Signed-off-by: Juergen Gross --- mm.c | 5 + 1 file changed, 5 insertions(+) diff --git a/mm.c b/mm.c index 263a356..185a8a5 100644 --- a/mm.c +++ b/mm.c @@ -335,6 +335,11 @@ void

[Xen-devel] [PATCH 8/9] mini-os: get maximum memory size from hypervisor

2016-07-19 Thread Juergen Gross
Add support for obtaining the maximum memory size from the hypervisor. This will make it possible to support ballooning. Signed-off-by: Juergen Gross --- include/mm.h | 1 + mm.c | 22 ++ 2 files changed, 23 insertions(+) diff --git a/include/mm.h b/include/mm.h ind

[Xen-devel] [PATCH 9/9] mini-os: modify virtual memory layout for support of ballooning

2016-07-19 Thread Juergen Gross
In order to be able to support ballooning the virtual memory layout of Mini-OS has to be modified: instead of a (nearly) consecutive area used for physical memory mapping, on demand mappings, and heap we need enough spare place for adding new memory. So instead of dynamically place the different r

[Xen-devel] [PATCH 5/9] mini-os: add nr_free_pages counter

2016-07-19 Thread Juergen Gross
Add a variable holding the number of available memory pages. This will aid auto-ballooning later. Signed-off-by: Juergen Gross --- include/mm.h | 1 + mm.c | 6 ++ 2 files changed, 7 insertions(+) diff --git a/include/mm.h b/include/mm.h index a48f485..b97b43e 100644 --- a/include/m

Re: [Xen-devel] Is: Revert c5ad33184354260be6d05de57e46a5498692f6d6 "mm/swap.c: flush lru pvecs on compound page arrival" from stable tree? Was:[osstest-ad...@xenproject.org: [linux-4.1 bisection] com

2016-07-19 Thread Sebastian Gottschall
No such Message-ID known. Am 19.07.2016 um 10:32 schrieb Michal Hocko: [CCing Sasha] On Mon 18-07-16 11:30:46, Konrad Rzeszutek Wilk wrote: Hey Lukasz, We found that your patch in the automated Xen test-case ends up OOMing the box when trying to install guests. This worked prior to your pat

[Xen-devel] [PATCH v2] xl: add option to leave domain paused after migration

2016-07-19 Thread Roger Pau Monne
This is useful for debugging domains that crash on resume from migration. Signed-off-by: Roger Pau Monné --- Cc: ian.jack...@eu.citrix.com Cc: wei.l...@citrix.com --- Changes since v1: - Document the newly added option in the xl man page. --- docs/man/xl.pod.1 | 4 tools/libxl/xl_

Re: [Xen-devel] [PATCH v2] xl: add option to leave domain paused after migration

2016-07-19 Thread Wei Liu
On Tue, Jul 19, 2016 at 10:58:15AM +0200, Roger Pau Monne wrote: > This is useful for debugging domains that crash on resume from migration. > > Signed-off-by: Roger Pau Monné Acked-by: Wei Liu ___ Xen-devel mailing list Xen-devel@lists.xen.org https

[Xen-devel] [PATCH] XSM-docs: Flask operates on domain types and not on individual domain. Updated the documentation to reflect this.

2016-07-19 Thread Anshul Makkar
Signed-off-by: Anshul Makkar --- * Resending the patch due to incomplete subject in the previous patch. docs/misc/xsm-flask.txt | 8 1 file changed, 4 insertions(+), 4 deletions(-) --- diff --git a/docs/misc/xsm-flask.txt b/docs/misc/xsm-flask.txt index 62f15dd..bf8bb6e 100644 --- a/do

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

2016-07-19 Thread osstest service owner
flight 97622 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/97622/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-qemuu-ovmf-amd64 17 guest-start/debianhvm.repeat fail REGR. vs. 94748 test-amd64-amd64-

Re: [Xen-devel] [PATCH v1 10/20] acpi/hvmloader: Replace mem_alloc() and virt_to_phys() with memory ops

2016-07-19 Thread Jan Beulich
>>> Boris Ostrovsky 07/08/16 6:20 PM >>> >On 07/08/2016 11:35 AM, Jan Beulich wrote: > On 08.07.16 at 17:23, wrote: >>> Is it up to the builder to decide which tables are important and which >>> are not? >> I'm afraid that's not so easy to tell. If for example we can't fit the >> HPET table,

Re: [Xen-devel] [PATCH v3 2/2] qdisk - hw/block/xen_disk: grant copy implementation

2016-07-19 Thread Roger Pau Monné
On Wed, Jun 22, 2016 at 10:38:53AM +0200, Paulina Szubarczyk wrote: > Copy data operated on during request from/to local buffers to/from > the grant references. > > Before grant copy operation local buffers must be allocated what is > done by calling ioreq_init_copy_buffers. For the 'read' operati

Re: [Xen-devel] [PATCH] docs/misc/hvmlite: Sync up hvm_start_info data structure

2016-07-19 Thread Anthony PERARD
On Mon, Jul 18, 2016 at 06:49:33PM +0100, Andrew Cooper wrote: > On 18/07/16 17:15, Anthony PERARD wrote: > > It as been modified by: > > 3c8d890 x86/PVHv2: update the start info structure layout > > 247d38c xen: change the sizes of memory fields in the HVM start info to be > > 64bits > > > > Sign

Re: [Xen-devel] Is: Revert c5ad33184354260be6d05de57e46a5498692f6d6 "mm/swap.c: flush lru pvecs on compound page arrival" from stable tree? Was:[osstest-ad...@xenproject.org: [linux-4.1 bisection] com

2016-07-19 Thread Michal Hocko
On Tue 19-07-16 10:35:09, Sebastian Gottschall wrote: > No such Message-ID known. Ups, sorry about that. I didn't know that the stable tree is not archived via lkml.kernel.org. Here is the link http://www.spinics.net/lists/stable/msg138760.html > Am 19.07.2016 um 10:32 schrieb Michal Hocko: > >

Re: [Xen-devel] [PATCH 03/16] x86/monitor: mechanical renames

2016-07-19 Thread Corneliu ZUZU
On 7/18/2016 9:07 PM, Andrew Cooper wrote: On 15/07/16 08:18, Corneliu ZUZU wrote: On 7/12/2016 9:10 AM, Corneliu ZUZU wrote: On 7/11/2016 7:43 PM, Tamas K Lengyel wrote: On Sat, Jul 9, 2016 at 12:46 PM, Corneliu ZUZU wrote: On 7/9/2016 9:10 PM, Tamas K Lengyel wrote: On Fri, Jul 8, 2016 at

Re: [Xen-devel] [PATCH v2 11/11] xen: credit2: implement true SMT support

2016-07-19 Thread George Dunlap
On Mon, Jul 18, 2016 at 6:24 PM, Dario Faggioli wrote: > On Mon, 2016-07-18 at 17:48 +0100, George Dunlap wrote: >> On 15/07/16 15:50, Dario Faggioli wrote: >> > >> > +/* >> > + * If all the siblings of cpu (including cpu itself) are in >> > idlers, >> > + * set all their bits in mask. >> > + * >>

Re: [Xen-devel] [PATCH v2 11/11] xen: credit2: implement true SMT support

2016-07-19 Thread Dario Faggioli
On Tue, 2016-07-19 at 10:39 +0100, George Dunlap wrote: > On Mon, Jul 18, 2016 at 6:24 PM, Dario Faggioli > wrote: > >  > > If you're saying that this discrepancy between rqd->idle's and > > rqd->smt_idle's semantic is, at minimum, unideal, I do agree... but > > I > > think, for now at least, it's

[Xen-devel] [qemu-mainline bisection] complete test-amd64-amd64-libvirt-xsm

2016-07-19 Thread osstest service owner
branch xen-unstable xenbranch xen-unstable job test-amd64-amd64-libvirt-xsm testid guest-start Tree: libvirt git://xenbits.xen.org/libvirt.git Tree: libvirt_gnulib git://git.sv.gnu.org/gnulib.git Tree: linux git://xenbits.xen.org/linux-pvops.git Tree: linuxfirmware git://xenbits.xen.org/osstest/li

Re: [Xen-devel] [PATCH v2 1/5] xenstore: call each xenstored command function with temporary context

2016-07-19 Thread Wei Liu
On Mon, Jul 18, 2016 at 09:31:25AM +0200, Juergen Gross wrote: > In order to be able to avoid leaving temporary memory allocated after > processing of a command in xenstored call all command functions with > the temporary "in" context. Each function can then make use of that > temporary context for

Re: [Xen-devel] [PATCH v2 2/5] xenstore: add explicit memory context parameter to get_parent()

2016-07-19 Thread Wei Liu
On Mon, Jul 18, 2016 at 09:31:26AM +0200, Juergen Gross wrote: > Add a parameter to xenstored get_parent() function to explicitly > specify the memory context to be used for allocations. This will make > it easier to avoid memory leaks by using a context which is freed > soon. > > When available u

Re: [Xen-devel] [PATCH v2 3/5] xenstore: add explicit memory context parameter to read_node()

2016-07-19 Thread Wei Liu
On Mon, Jul 18, 2016 at 09:31:27AM +0200, Juergen Gross wrote: > Add a parameter to xenstored read_node() function to explicitly > specify the memory context to be used for allocations. This will make > it easier to avoid memory leaks by using a context which is freed > soon. > > When calling read

Re: [Xen-devel] [PATCH v2 4/5] xenstore: add explicit memory context parameter to get_node()

2016-07-19 Thread Wei Liu
On Mon, Jul 18, 2016 at 09:31:28AM +0200, Juergen Gross wrote: > Add a parameter to xenstored get_node() function to explicitly > specify the memory context to be used for allocations. This will make > it easier to avoid memory leaks by using a context which is freed > soon. > > This requires addi

Re: [Xen-devel] [PATCH v2 11/11] xen: credit2: implement true SMT support

2016-07-19 Thread George Dunlap
On 19/07/16 10:57, Dario Faggioli wrote: > On Tue, 2016-07-19 at 10:39 +0100, George Dunlap wrote: >> On Mon, Jul 18, 2016 at 6:24 PM, Dario Faggioli >> wrote: >>> >>> If you're saying that this discrepancy between rqd->idle's and >>> rqd->smt_idle's semantic is, at minimum, unideal, I do agree.

Re: [Xen-devel] [PATCH v3 2/2] qdisk - hw/block/xen_disk: grant copy implementation

2016-07-19 Thread Paulina Szubarczyk
On 07/19/2016 11:12 AM, Roger Pau Monné wrote: On Wed, Jun 22, 2016 at 10:38:53AM +0200, Paulina Szubarczyk wrote: Copy data operated on during request from/to local buffers to/from the grant references. Before grant copy operation local buffers must be allocated what is done by calling ioreq

Re: [Xen-devel] [PATCH v3 2/2] qdisk - hw/block/xen_disk: grant copy implementation

2016-07-19 Thread Paulina Szubarczyk
On 07/15/2016 07:11 PM, Anthony PERARD wrote: On Fri, Jul 15, 2016 at 12:15:45PM +0100, Wei Liu wrote: On Fri, Jul 15, 2016 at 12:28:48PM +0200, Paulina Szubarczyk wrote: On 07/14/2016 12:37 PM, Wei Liu wrote: On Wed, Jun 22, 2016 at 10:38:53AM +0200, Paulina Szubarczyk wrote: diff --git

Re: [Xen-devel] [PATCH v2 11/11] xen: credit2: implement true SMT support

2016-07-19 Thread Dario Faggioli
On Tue, 2016-07-19 at 11:05 +0100, George Dunlap wrote: > On 19/07/16 10:57, Dario Faggioli wrote: > >  > > > What about folding in something like the attached patch? > > > > > I'd be totally fine with this. > Do you mean you ack me folding in that particular patch (so that the > resulting commit

Re: [Xen-devel] [PATCH v2 5/5] xenstore: use temporary memory context for firing watches

2016-07-19 Thread Wei Liu
On Mon, Jul 18, 2016 at 09:31:29AM +0200, Juergen Gross wrote: > static void add_event(struct connection *conn, > + void *tmp, tmp -> ctx or context. Reviewed-by: Wei Liu ___ Xen-devel mailing list Xen-devel@lists.xen.org https://li

[Xen-devel] [PATCH v2 1/5] xenstore: call each xenstored command function with temporary context

2016-07-19 Thread Ian Jackson
Juergen Gross writes ("[PATCH v2 1/5] xenstore: call each xenstored command function with temporary context"): > In order to be able to avoid leaving temporary memory allocated after > processing of a command in xenstored call all command functions with > the temporary "in" context. Each function

Re: [Xen-devel] [PATCH v2 2/5] xenstore: add explicit memory context parameter to get_parent()

2016-07-19 Thread Ian Jackson
Wei Liu writes ("Re: [PATCH v2 2/5] xenstore: add explicit memory context parameter to get_parent()"): > I would name mem ctx or context instead. And maybe document this > function a bit saying that memory allocation is done with the first > parameter. > > With those cosmetic issues fixed: > > R

[Xen-devel] [PATCH v2 2/5] xenstore: add explicit memory context parameter to get_parent()

2016-07-19 Thread Ian Jackson
Juergen Gross writes ("[PATCH v2 2/5] xenstore: add explicit memory context parameter to get_parent()"): > Add a parameter to xenstored get_parent() function to explicitly > specify the memory context to be used for allocations. This will make > it easier to avoid memory leaks by using a context w

Re: [Xen-devel] [PATCH v2 3/5] xenstore: add explicit memory context parameter to read_node()

2016-07-19 Thread Ian Jackson
Wei Liu writes ("Re: [PATCH v2 3/5] xenstore: add explicit memory context parameter to read_node()"): > On Mon, Jul 18, 2016 at 09:31:27AM +0200, Juergen Gross wrote: > > /* If it fails, returns NULL and sets errno. */ > > -static struct node *read_node(struct connection *conn, const char *name)

Re: [Xen-devel] [PATCH v2 16/17] libxc/xc_dom_arm: Copy ACPI tables to guest space

2016-07-19 Thread Wei Liu
On Fri, Jul 15, 2016 at 05:39:32PM +0800, Shannon Zhao wrote: [...] > > > > It would be trivial to have another option in xl.cfg to allow MB > > granularity. But I don't think that's a good idea. Asking for more > > memory when you don't really know how much is enough is not very useful. > > If an

Re: [Xen-devel] [PATCH v2 4/5] xenstore: add explicit memory context parameter to get_node()

2016-07-19 Thread Ian Jackson
Wei Liu writes ("Re: [PATCH v2 4/5] xenstore: add explicit memory context parameter to get_node()"): > On Mon, Jul 18, 2016 at 09:31:28AM +0200, Juergen Gross wrote: > > Add a parameter to xenstored get_node() function to explicitly > > specify the memory context to be used for allocations. This w

Re: [Xen-devel] [PATCH v2 03/17] libxl/arm: Add a configuration option for ARM DomU ACPI

2016-07-19 Thread Wei Liu
On Mon, Jul 18, 2016 at 12:40:43PM -0700, Stefano Stabellini wrote: [...] > > #if defined(__arm__) || defined(__aarch64__) > > ? > > I am not a Libxl maintainer anymore, but I think that should be OK or at > least it would be a step in the right direction. Yes, I think that's the correct ifdefs t

Re: [Xen-devel] [PATCH v2 5/5] xenstore: use temporary memory context for firing watches

2016-07-19 Thread Ian Jackson
Wei Liu writes ("Re: [PATCH v2 5/5] xenstore: use temporary memory context for firing watches"): > On Mon, Jul 18, 2016 at 09:31:29AM +0200, Juergen Gross wrote: > > static void add_event(struct connection *conn, > > + void *tmp, > > tmp -> ctx or context. Once again, Acked-by:

Re: [Xen-devel] [PATCH v2 03/17] libxl/arm: Add a configuration option for ARM DomU ACPI

2016-07-19 Thread Ian Jackson
Stefano Stabellini writes ("Re: [Xen-devel] [PATCH v2 03/17] libxl/arm: Add a configuration option for ARM DomU ACPI"): ... > > >>> I know but here we want to unify the acpi option for x86 and ARM while > > >>> on x86 it's true by default. What I want to ask is that how to > > >>> distinguish x86

Re: [Xen-devel] [PATCH v3 2/2] qdisk - hw/block/xen_disk: grant copy implementation

2016-07-19 Thread Paulina Szubarczyk
On 07/15/2016 06:55 PM, Anthony PERARD wrote: On Wed, Jun 22, 2016 at 10:38:53AM +0200, Paulina Szubarczyk wrote: Copy data operated on during request from/to local buffers to/from the grant references. Before grant copy operation local buffers must be allocated what is done by calling ioreq_

[Xen-devel] [PATCH v3 4/5] xenstore: add explicit memory context parameter to get_node()

2016-07-19 Thread Juergen Gross
Add a parameter to xenstored get_node() function to explicitly specify the memory context to be used for allocations. This will make it easier to avoid memory leaks by using a context which is freed soon. This requires adding the temporary context to errno_from_parents() and ask_parents(), too. W

[Xen-devel] [PATCH v3 3/5] xenstore: add explicit memory context parameter to read_node()

2016-07-19 Thread Juergen Gross
Add a parameter to xenstored read_node() function to explicitly specify the memory context to be used for allocations. This will make it easier to avoid memory leaks by using a context which is freed soon. When calling read_node() select a sensible memory context for the new parameter by preferrin

[Xen-devel] [PATCH v3 1/5] xenstore: call each xenstored command function with temporary context

2016-07-19 Thread Juergen Gross
In order to be able to avoid leaving temporary memory allocated after processing of a command in xenstored call all command functions with the temporary "in" context. Each function can then make use of that temporary context for allocating temporary memory instead of either leaving that memory allo

[Xen-devel] [PATCH v3 2/5] xenstore: add explicit memory context parameter to get_parent()

2016-07-19 Thread Juergen Gross
Add a parameter to xenstored get_parent() function to explicitly specify the memory context to be used for allocations. This will make it easier to avoid memory leaks by using a context which is freed soon. When available use a temporary context when calling get_parent(), otherwise mimic the old b

[Xen-devel] [PATCH v3 0/5] xenstore: fix memory leak of xenstored

2016-07-19 Thread Juergen Gross
xenstored has a memory leak when setting watches: a no longer active watch which fired in the past will still use some memory. This is critical for long running connections to xenstored like the qemu process serving as a qdisk backend for dom0. It will use some few kB in xenstored for each domain c

[Xen-devel] [PATCH v3 5/5] xenstore: use temporary memory context for firing watches

2016-07-19 Thread Juergen Gross
Use a temporary memory context for memory allocations when firing watches. This will avoid leaking memory in case of long living connections and/or xenstore entries. This requires adding a new parameter to fire_watches() and add_event() to specify the memory context to use for allocations. Signed

Re: [Xen-devel] [PATCH v4] altp2m: Allow the hostp2m entries to be of type p2m_ram_shared

2016-07-19 Thread George Dunlap
On 18/07/16 18:06, Tamas K Lengyel wrote: >>> Anyhow, at this point I'm >>> going to start carrying out-of-tree patches for Xen in my project and >>> just resign from my mem_sharing maintainership as I feel like it's >>> pretty pointless. >> >> I'm sorry that you're discouraged; all I can say is th

Re: [Xen-devel] [PATCH] xenstore: add memory allocation debugging capability

2016-07-19 Thread Wei Liu
On Fri, Jul 15, 2016 at 07:28:26AM +0200, Juergen Gross wrote: > Add support for debugging memory allocation statistics to xenstored. > Specifying "-M " on the command line will enable the feature. > Whenever xenstored receives SIGUSR1 it will dump out a full talloc > report to . This helps finding

[Xen-devel] [PATCH 0/2] xen: Credit2: fix two issues from recently committed series.

2016-07-19 Thread Dario Faggioli
Series committed yesterday, to be precise, and two (not too big, at least :-)) issues, have been found already (thanks Andrew!). Thanks and Regards, Dario --- Dario Faggioli (2): xen: credit2: fix two s_time_t handling issues in load balancing xen: credit2: fix potential issues in csch

[Xen-devel] [PATCH 2/2] xen: credit2: fix potential issues in csched2_cpu_pick with tracing enabled

2016-07-19 Thread Dario Faggioli
In fact, when not finding a suitable runqueue where to place a vCPU, and hence using a fallback, we either: - don't issue any trace record (while we should), - risk underruning when accessing the runqueues array, while preparing the trace record. Fix both issues and, while there, also a coupl

[Xen-devel] [PATCH 1/2] xen: credit2: fix two s_time_t handling issues in load balancing

2016-07-19 Thread Dario Faggioli
both introduced in d205f8a7f48e2ec ("xen: credit2: rework load tracking logic"). First, in __update_runq_load(), the ASSERT() was actually useless. Let's instead check that the computed value of the load has not overflowed (and hence gone negative). While there, do that in __update_svc_load() as

[Xen-devel] [PATCH v2] xenstore: add memory allocation debugging capability

2016-07-19 Thread Juergen Gross
Add support for debugging memory allocation statistics to xenstored. Specifying "-M " on the command line will enable the feature. Whenever xenstored receives SIGUSR1 it will dump out a full talloc report to . This helps finding e.g. memory leaks in xenstored. Signed-off-by: Juergen Gross --- To

Re: [Xen-devel] [PATCH 0/2] xen: Credit2: fix two issues from recently committed series.

2016-07-19 Thread Andrew Cooper
On 19/07/16 13:06, Dario Faggioli wrote: > Series committed yesterday, to be precise, and two (not too big, at least :-)) > issues, have been found already (thanks Andrew!). I tend to put "Spotted by Coverity." in the commit message of these, even when we can't provide CID numbers. Its not like I

Re: [Xen-devel] [PATCH 0/2] xen: Credit2: fix two issues from recently committed series.

2016-07-19 Thread Dario Faggioli
On Tue, 2016-07-19 at 13:20 +0100, Andrew Cooper wrote: > On 19/07/16 13:06, Dario Faggioli wrote: > > > > Series committed yesterday, to be precise, and two (not too big, at > > least :-)) > > issues, have been found already (thanks Andrew!). > I tend to put "Spotted by Coverity." in the commit m

Re: [Xen-devel] [PATCH 0/2] xen: Credit2: fix two issues from recently committed series.

2016-07-19 Thread Andrew Cooper
On 19/07/16 13:26, Dario Faggioli wrote: > On Tue, 2016-07-19 at 13:20 +0100, Andrew Cooper wrote: >> On 19/07/16 13:06, Dario Faggioli wrote: >>> Series committed yesterday, to be precise, and two (not too big, at >>> least :-)) >>> issues, have been found already (thanks Andrew!). >> I tend to pu

Re: [Xen-devel] [PATCH v2] xl: add option to leave domain paused after migration

2016-07-19 Thread Wei Liu
On Tue, Jul 19, 2016 at 10:58:15AM +0200, Roger Pau Monne wrote: > This is useful for debugging domains that crash on resume from migration. > > Signed-off-by: Roger Pau Monné > --- > Cc: ian.jack...@eu.citrix.com > Cc: wei.l...@citrix.com > --- > Changes since v1: > - Document the newly added o

Re: [Xen-devel] [PATCH v2] xenstore: add memory allocation debugging capability

2016-07-19 Thread Wei Liu
On Tue, Jul 19, 2016 at 02:08:18PM +0200, Juergen Gross wrote: > Add support for debugging memory allocation statistics to xenstored. > Specifying "-M " on the command line will enable the feature. > Whenever xenstored receives SIGUSR1 it will dump out a full talloc > report to . This helps finding

Re: [Xen-devel] [PATCH v3 0/5] xenstore: fix memory leak of xenstored

2016-07-19 Thread Wei Liu
Queued. Thanks. ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v3] libxl: trigger attach events for devices attached before xl devd startup

2016-07-19 Thread Wei Liu
On Mon, Jul 18, 2016 at 04:31:30PM +0100, Wei Liu wrote: > On Sat, Jul 16, 2016 at 01:47:56AM +0200, Marek Marczykowski-Górecki wrote: > > When this daemon is started after creating backend device, that device > > will not be configured. > > > > Racy situation: > > 1. driver domain is started > >

Re: [Xen-devel] [PATCH] XSM-docs: Flask operates on domain types and not on individual domain. Updated the documentation to reflect this.

2016-07-19 Thread Doug Goldstein
On 7/19/16 4:05 AM, Anshul Makkar wrote: > Signed-off-by: Anshul Makkar > --- > * Resending the patch due to incomplete subject in the previous patch. > > docs/misc/xsm-flask.txt | 8 > 1 file changed, 4 insertions(+), 4 deletions(-) > --- > diff --git a/docs/misc/xsm-flask.txt b/docs/

Re: [Xen-devel] [PATCH 1/2] xen: credit2: fix two s_time_t handling issues in load balancing

2016-07-19 Thread Dario Faggioli
On Tue, 2016-07-19 at 14:07 +0200, Dario Faggioli wrote: > --- a/xen/common/sched_credit2.c > +++ b/xen/common/sched_credit2.c > @@ -656,7 +656,8 @@ __update_runq_load(const struct scheduler *ops, >  rqd->load += change; >  rqd->load_last_update = now; >   > -ASSERT(rqd->avgload <= STIM

Re: [Xen-devel] [PATCH v4] altp2m: Allow the hostp2m entries to be of type p2m_ram_shared

2016-07-19 Thread Ian Jackson
Tamas: George brought this thread to my attention. I'm sorry that you feel blocked and/or overruled. The hypervisor MM code is not my area of expertise, but I have a keen interest in seeing a good, productive and friendly Xen community. I definitely don't want to see you pushed away, and driven

[Xen-devel] Xen 4.8 Development Update

2016-07-19 Thread Wei Liu
This email only tracks big items for xen.git tree. Please reply for items you woulk like to see in 4.8 so that people have an idea what is going on and prioritise accordingly. You're welcome to provide description and use cases of the feature you're working on. = Timeline = We now adopt a fixed

Re: [Xen-devel] [PATCH v1 10/20] acpi/hvmloader: Replace mem_alloc() and virt_to_phys() with memory ops

2016-07-19 Thread Boris Ostrovsky
On 07/19/2016 05:11 AM, Jan Beulich wrote: Boris Ostrovsky 07/08/16 6:20 PM >>> >> On 07/08/2016 11:35 AM, Jan Beulich wrote: >> On 08.07.16 at 17:23, wrote: Is it up to the builder to decide which tables are important and which are not? >>> I'm afraid that's not so easy to tel

Re: [Xen-devel] [PATCH 2/3] xen-scsiback: One function call less in scsiback_device_action() after error detection

2016-07-19 Thread SF Markus Elfring
>> @@ -606,7 +606,7 @@ static void scsiback_device_action(struct vscsibk_pend >> *pending_req, >> tmr = kzalloc(sizeof(struct scsiback_tmr), GFP_KERNEL); >> if (!tmr) { >> target_put_sess_cmd(se_cmd); >> -goto err; >> +goto do_resp; >> } > > Hmm

Re: [Xen-devel] [PATCH] acpi: Re-license ACPI builder files from GPLv2 to LGPLv2.1

2016-07-19 Thread Daniel Kiper
On Mon, Jul 18, 2016 at 10:01:27AM -0400, Boris Ostrovsky wrote: > ACPI builder is currently distributed under GPLv2 license. > > We plan to make the builder available to components other > than the hvmloader (which is also GPLv2). Some of these > components (such as libxl) may be distributed under

[Xen-devel] [xen-unstable test] 97623: tolerable FAIL - PUSHED

2016-07-19 Thread osstest service owner
flight 97623 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/97623/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): build-amd64-rumpuserxen 6 xen-buildfail like 97562 build-i386-rumpuserxen

Re: [Xen-devel] [PATCH v4] altp2m: Allow the hostp2m entries to be of type p2m_ram_shared

2016-07-19 Thread Tamas K Lengyel
On Tue, Jul 19, 2016 at 5:39 AM, George Dunlap wrote: > On 18/07/16 18:06, Tamas K Lengyel wrote: Anyhow, at this point I'm going to start carrying out-of-tree patches for Xen in my project and just resign from my mem_sharing maintainership as I feel like it's pretty pointless.

Re: [Xen-devel] [PATCH] XSM-docs: Flask operates on domain types and not on individual domain. Updated the documentation to reflect this.

2016-07-19 Thread anshul makkar
On 19/07/16 14:30, Doug Goldstein wrote: On 7/19/16 4:05 AM, Anshul Makkar wrote: Signed-off-by: Anshul Makkar --- * Resending the patch due to incomplete subject in the previous patch. docs/misc/xsm-flask.txt | 8 1 file changed, 4 insertions(+), 4 deletions(-) --- diff --git a/

Re: [Xen-devel] [PATCH] XSM-docs: Flask operates on domain types and not on individual domain. Updated the documentation to reflect this.

2016-07-19 Thread Daniel De Graaf
On 07/19/2016 11:21 AM, anshul makkar wrote: On 19/07/16 14:30, Doug Goldstein wrote: On 7/19/16 4:05 AM, Anshul Makkar wrote: Signed-off-by: Anshul Makkar --- * Resending the patch due to incomplete subject in the previous patch. docs/misc/xsm-flask.txt | 8 1 file changed, 4 i

Re: [Xen-devel] [PATCH v4] altp2m: Allow the hostp2m entries to be of type p2m_ram_shared

2016-07-19 Thread Tamas K Lengyel
On Tue, Jul 19, 2016 at 7:36 AM, Ian Jackson wrote: > Tamas: George brought this thread to my attention. I'm sorry that you > feel blocked and/or overruled. The hypervisor MM code is not my area > of expertise, but I have a keen interest in seeing a good, productive > and friendly Xen community.

[Xen-devel] [PATCH v2 0/2] xen: Credit2: fix two issues from recently committed series.

2016-07-19 Thread Dario Faggioli
v2 of <146892985892.30642.2392453881110942183.st...@solace.fritz.box>, as v1 was making things worse! In fact, there was a bug in patch 1 which turned the ASSERT() from being useless to being wrong, and it was actually triggering. Sorry for the noise. Regards, Dario --- Dario Faggioli (2):

[Xen-devel] [PATCH v2 1/2] xen: credit2: fix two s_time_t handling issues in load balancing

2016-07-19 Thread Dario Faggioli
both introduced in d205f8a7f48e2ec ("xen: credit2: rework load tracking logic"). First, in __update_runq_load(), the ASSERT() was actually useless. Let's instead check that the computed value of the load has not overflowed (and hence gone negative). While there, do that in __update_svc_load() as

[Xen-devel] [PATCH v2 2/2] xen: credit2: fix potential issues in csched2_cpu_pick with tracing enabled

2016-07-19 Thread Dario Faggioli
In fact, when not finding a suitable runqueue where to place a vCPU, and hence using a fallback, we either: - don't issue any trace record (while we should), - risk underruning when accessing the runqueues array, while preparing the trace record. Fix both issues and, while there, also a coupl

Re: [Xen-devel] [PATCH v2] xl: add option to leave domain paused after migration

2016-07-19 Thread Roger Pau Monne
On Tue, Jul 19, 2016 at 02:15:28PM +0100, Wei Liu wrote: > On Tue, Jul 19, 2016 at 10:58:15AM +0200, Roger Pau Monne wrote: > > This is useful for debugging domains that crash on resume from migration. > > > > Signed-off-by: Roger Pau Monné > > --- > > Cc: ian.jack...@eu.citrix.com > > Cc: wei.l.

Re: [Xen-devel] [PATCH v2] xl: add option to leave domain paused after migration

2016-07-19 Thread Wei Liu
On Tue, Jul 19, 2016 at 05:50:32PM +0200, Roger Pau Monne wrote: > On Tue, Jul 19, 2016 at 02:15:28PM +0100, Wei Liu wrote: > > On Tue, Jul 19, 2016 at 10:58:15AM +0200, Roger Pau Monne wrote: > > > This is useful for debugging domains that crash on resume from migration. > > > > > > Signed-off-by

Re: [Xen-devel] Xen 4.8 Development Update

2016-07-19 Thread Tamas K Lengyel
On Tue, Jul 19, 2016 at 7:48 AM, Wei Liu wrote: > This email only tracks big items for xen.git tree. Please reply for items you > woulk like to see in 4.8 so that people have an idea what is going on and > prioritise accordingly. > > You're welcome to provide description and use cases of the featu

Re: [Xen-devel] Xen 4.8 Development Update

2016-07-19 Thread Wei Liu
On Tue, Jul 19, 2016 at 10:06:57AM -0600, Tamas K Lengyel wrote: > On Tue, Jul 19, 2016 at 7:48 AM, Wei Liu wrote: > > This email only tracks big items for xen.git tree. Please reply for items > > you > > woulk like to see in 4.8 so that people have an idea what is going on and > > prioritise acc

Re: [Xen-devel] [PATCH v7] x86/mem-sharing: mem-sharing a range of memory

2016-07-19 Thread Tamas K Lengyel
On Mon, Jul 18, 2016 at 3:47 PM, Andrew Cooper wrote: > On 18/07/2016 22:14, Tamas K Lengyel wrote: >> Currently mem-sharing can be performed on a page-by-page basis from the >> control >> domain. However, this process is quite wasteful when a range of pages have to >> be deduplicated. >> >> This

Re: [Xen-devel] [PATCH v7] x86/mem-sharing: mem-sharing a range of memory

2016-07-19 Thread Tamas K Lengyel
On Tue, Jul 19, 2016 at 1:54 AM, Julien Grall wrote: > Hello Tamas, > > On 18/07/2016 22:14, Tamas K Lengyel wrote: >> >> diff --git a/tools/libxc/include/xenctrl.h b/tools/libxc/include/xenctrl.h >> index e904bd5..0ca94cd 100644 >> --- a/tools/libxc/include/xenctrl.h >> +++ b/tools/libxc/include/

Re: [Xen-devel] [PATCH v7] x86/mem-sharing: mem-sharing a range of memory

2016-07-19 Thread Andrew Cooper
On 19/07/16 17:27, Tamas K Lengyel wrote: > >>> +{ >>> +int rc = 0; >>> +shr_handle_t sh, ch; >>> +unsigned long start = >>> +range->_scratchspace ? range->_scratchspace : range->start; >> This can be shortened to "unsigned long start = range->_scratchspace ?: >> range->start;"

Re: [Xen-devel] [PATCH v7] x86/mem-sharing: mem-sharing a range of memory

2016-07-19 Thread Tamas K Lengyel
On Tue, Jul 19, 2016 at 10:49 AM, Andrew Cooper wrote: > On 19/07/16 17:27, Tamas K Lengyel wrote: >> +{ +int rc = 0; +shr_handle_t sh, ch; +unsigned long start = +range->_scratchspace ? range->_scratchspace : range->start; >>> This can be shortened to

Re: [Xen-devel] [PATCH v7] x86/mem-sharing: mem-sharing a range of memory

2016-07-19 Thread Andrew Cooper
On 19/07/16 17:54, Tamas K Lengyel wrote: > On Tue, Jul 19, 2016 at 10:49 AM, Andrew Cooper > wrote: >> On 19/07/16 17:27, Tamas K Lengyel wrote: > +{ > +int rc = 0; > +shr_handle_t sh, ch; > +unsigned long start = > +range->_scratchspace ? range->_scratchsp

Re: [Xen-devel] [PATCH v7] x86/mem-sharing: mem-sharing a range of memory

2016-07-19 Thread Tamas K Lengyel
On Tue, Jul 19, 2016 at 10:55 AM, Andrew Cooper wrote: > On 19/07/16 17:54, Tamas K Lengyel wrote: >> On Tue, Jul 19, 2016 at 10:49 AM, Andrew Cooper >> wrote: >>> On 19/07/16 17:27, Tamas K Lengyel wrote: >> +{ >> +int rc = 0; >> +shr_handle_t sh, ch; >> +unsigned lon

[Xen-devel] [qemu-mainline test] 97627: regressions - trouble: blocked/broken/fail/pass

2016-07-19 Thread osstest service owner
flight 97627 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/97627/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-libvirt-xsm 11 guest-start fail REGR. vs. 96791 test-amd64-amd64-li

Re: [Xen-devel] [PATCH v4] altp2m: Allow the hostp2m entries to be of type p2m_ram_shared

2016-07-19 Thread George Dunlap
On 18/07/16 18:06, Tamas K Lengyel wrote: >> Incremental improvements are welcome; but they must not cause >> regressions in existing functionality. > > Existing functionality does not get impaired by this as what happens > right now is a hypervisor crash. I don't see how things can get any > wors

[Xen-devel] [PATCH 2/2] arm/traps: fix bug in dump_guest_s1_walk handling of level 2 page tables

2016-07-19 Thread Jonathan Daugherty
dump_guest_s1_walk intends to walk to level 2 page table entries but was failing to do so because of a check that caused level 2 page table descriptors to be ignored. This change fixes the check so that level 2 page table walks occur as intended by ignoring descriptors unless their low two bits mat

[Xen-devel] [PATCH 1/2] arm/traps: fix bug in dump_guest_s1_walk L1 page table offset computation

2016-07-19 Thread Jonathan Daugherty
The dump_guest_s1_walk function was incorrectly using the top 10 bits of the virtual address to select the L1 page table index. The correct amount is 12 bits, resulting in a shift of 20 bits rather than 22. For more details, see the ARMv7-A ARM, section B3.5, "Short-descriptor translation table f

[Xen-devel] [xen-unstable-smoke test] 97661: tolerable all pass - PUSHED

2016-07-19 Thread osstest service owner
flight 97661 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/97661/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-amd64-amd64-libvirt 12 migrate-support-checkfail never pass test-armhf-armhf-xl 12

[Xen-devel] [libvirt test] 97638: tolerable FAIL - PUSHED

2016-07-19 Thread osstest service owner
flight 97638 libvirt real [real] http://logs.test-lab.xenproject.org/osstest/logs/97638/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-armhf-armhf-libvirt 14 guest-saverestorefail never pass test-armhf-armhf-libvirt 12 migrate-sup

  1   2   >