[Xen-devel] [OSSTEST PATCH v3 05/25] step status skip: Ignore in report_run_getinfo

2016-09-06 Thread Wei Liu
From: Ian Jackson report_run_getinfo is trying to generate some HTML to describe a job's (current) status. It sometimes looks at the steps to find `interesting' information to report. Completely ignore steps with status `skip' for this purpose, just like we ignore ones with status `pass'. Sign

[Xen-devel] [OSSTEST PATCH v3 11/25] DO NOT APPLY make-flight-substep-test

2016-09-06 Thread Wei Liu
From: Ian Jackson This is a simple test case for substeps. Signed-off-by: Ian Jackson CC: Wei Liu --- make-flight-substep-test | 14 ++ sg-run-job | 7 +++ 2 files changed, 21 insertions(+) create mode 100755 make-flight-substep-test diff --git a/make-flight-s

[Xen-devel] [OSSTEST PATCH v3 23/25] sg-run-job: test-xtf recipe

2016-09-06 Thread Wei Liu
Install XTF, run FEP test and then run all available tests. Signed-off-by: Wei Liu Acked-by: Ian Jackson --- sg-run-job | 10 ++ 1 file changed, 10 insertions(+) diff --git a/sg-run-job b/sg-run-job index 95ee89e..13203ef 100755 --- a/sg-run-job +++ b/sg-run-job @@ -411,6 +411,16 @@ pr

[Xen-devel] [OSSTEST PATCH v3 16/25] BuildSupport: move buildcmd_stamped_logged here

2016-09-06 Thread Wei Liu
... so that other build scripts can use it, too. It now accepts one more parameter called "component" to be useful in other build scripts. No functional change. Signed-off-by: Wei Liu Acked-by: Ian Jackson --- Osstest/BuildSupport.pm | 15 +++ ts-xen-build| 20 +++-

[Xen-devel] [OSSTEST PATCH v3 21/25] Introduce ts-xtf-fep

2016-09-06 Thread Wei Liu
Test the availability of FEP during runtime. Signed-off-by: Wei Liu --- v3: use logm to log result v2: 1. use target_cmd_output_root to get output 2. die if the output is not in expected format 3. use fep test result as exit code of the script 4. remove the use of xtfdir runvar --- ts-xtf-fep

[Xen-devel] [OSSTEST PATCH v3 10/25] Executive: Support substeps

2016-09-06 Thread Wei Liu
From: Ian Jackson ts-* scripts can now create `substeps'. For the purposes of archaeology etc., a substep is just like a step. But it does correspond to a single specific ts-* invocation. Instead, it is started and finished explicitly as required. The whole job implementation code needs to ex

[Xen-devel] [OSSTEST PATCH v3 15/25] DO NOT APPLY point xtf to my personal tree

2016-09-06 Thread Wei Liu
The master branch of this tree is xtf.git master + one test case that always fails. --- ap-common | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/ap-common b/ap-common index 6d25025..9cdb1c1 100644 --- a/ap-common +++ b/ap-common @@ -33,9 +33,9 @@ : ${TREEVCS_LINUX:=gi

[Xen-devel] [OSSTEST PATCH v3 25/25] Create XTF branch

2016-09-06 Thread Wei Liu
This branch contains Xen and Linux build jobs and all jobs which contain xtf in their name. Signed-off-by: Wei Liu --- I'm not sure if everything is ok because I can't test all changes. --- ap-fetch-version | 4 ap-print-url | 3 +++ ap-push | 5 + cr-daily-branch | 7

[Xen-devel] [OSSTEST PATCH v3 14/25] ap-common: add xtf tree

2016-09-06 Thread Wei Liu
Signed-off-by: Wei Liu Acked-by: Ian Jackson --- ap-common | 4 1 file changed, 4 insertions(+) diff --git a/ap-common b/ap-common index 62d2e4f..6d25025 100644 --- a/ap-common +++ b/ap-common @@ -33,6 +33,10 @@ : ${TREEVCS_LINUX:=git} +: ${TREE_XTF:=git://xenbits.xen.org/xtf.git} +:

[Xen-devel] [OSSTEST PATCH v3 12/25] DO NOT APPLY provide ts-substep-test

2016-09-06 Thread Wei Liu
From: Ian Jackson --- ts-substep-test | 38 ++ 1 file changed, 38 insertions(+) create mode 100755 ts-substep-test diff --git a/ts-substep-test b/ts-substep-test new file mode 100755 index 000..cea8a6d --- /dev/null +++ b/ts-substep-test @@ -0,0 +1,38 @@

[Xen-devel] [OSSTEST PATCH v3 13/25] ts-xen-build: always compile in FEP support

2016-09-06 Thread Wei Liu
By default FEP depends on debug flag. When we are near release the debug flag will be turned off. In order to test a release build, we explicitly enable FEP in build configuration. Since we target Xen versions that already have Kconfig support, only a Kconfig option is created for now. We can eas

[Xen-devel] [OSSTEST PATCH v3 20/25] mfi-common: create xtf build job for 4.4 onwards

2016-09-06 Thread Wei Liu
Xen 4.4 is the oldest one that we still provide security support at this point in time. Signed-off-by: Wei Liu --- mfi-common | 25 + 1 file changed, 25 insertions(+) diff --git a/mfi-common b/mfi-common index 8f27923..0864266 100644 --- a/mfi-common +++ b/mfi-common @@

[Xen-devel] [OSSTEST PATCH v3 22/25] Introduce ts-xtf-run

2016-09-06 Thread Wei Liu
This is the main script for running XTF. It will first perform selftest, and then run each XTF test case as a substep. It does the following things: 1. Run self tests for individual environment and record the result. 2. Collect tests according to available environments. 3. Run the collected test

[Xen-devel] [OSSTEST PATCH v3 24/25] make-flight: create 5 xtf jobs

2016-09-06 Thread Wei Liu
Create jobs only for x86 and set host flag diverse-xtf-x86. Signed-off-by: Wei Liu Acked-by: Ian Jackson --- make-flight | 22 ++ 1 file changed, 22 insertions(+) diff --git a/make-flight b/make-flight index c94dfa4..0aada37 100755 --- a/make-flight +++ b/make-flight @@ -42

[Xen-devel] [OSSTEST PATCH v3 18/25] sg-run-job: create xtf build recipe

2016-09-06 Thread Wei Liu
Signed-off-by: Wei Liu Acked-by: Ian Jackson --- sg-run-job | 5 + 1 file changed, 5 insertions(+) diff --git a/sg-run-job b/sg-run-job index 314019c..95ee89e 100755 --- a/sg-run-job +++ b/sg-run-job @@ -476,6 +476,7 @@ proc need-hosts/build {} { return BUILD } proc need-hosts/build-kern {

[Xen-devel] [OSSTEST PATCH v3 17/25] Introduce ts-xtf-build

2016-09-06 Thread Wei Liu
Clone, build and package XTF for later use. Signed-off-by: Wei Liu Acked-by: Ian Jackson --- ts-xtf-build | 54 ++ 1 file changed, 54 insertions(+) create mode 100755 ts-xtf-build diff --git a/ts-xtf-build b/ts-xtf-build new file mode 100755

[Xen-devel] [OSSTEST PATCH v3 19/25] Introduce ts-xtf-install

2016-09-06 Thread Wei Liu
Extract XTF to the desired location. Signed-off-by: Wei Liu --- v3: fix typo in commit message v2: remove reference of runvar xtfdir --- ts-xtf-install | 37 + 1 file changed, 37 insertions(+) create mode 100755 ts-xtf-install diff --git a/ts-xtf-install b/t

Re: [Xen-devel] [RFC PATCH] x86/apicv: fix RTC periodic timer and apicv issue

2016-09-06 Thread Jan Beulich
>>> On 06.09.16 at 13:22, wrote: > On September 06, 2016 5:05 PM, Jan Beulich > On 06.09.16 at 10:53, wrote: >>> to be honest, my fix is also a tradeoff.. w/ my patch, multiple >>> pending pt interrupt instances may be combined as one injection of >>> guest timer interrupt.. >>> but as Yang

Re: [Xen-devel] xen/xen vs xen/kvm nesting with pv drivers

2016-09-06 Thread Anthony Wright
On 06/09/2016 14:05, Andrew Cooper wrote: > On 06/09/16 13:47, Anthony Wright wrote: >> I tried to install Xen (4.7.0 with linux 4.7.2 Dom0) on an AWS virtual >> machine and it failed because while AWS uses Xen it requires that you use >> the PVHVM network driver. I then tried to install Xen on a

Re: [Xen-devel] [PATCH v2 1/4] xen: indicate gcov in log messages

2016-09-06 Thread Jan Beulich
>>> On 06.09.16 at 14:47, wrote: > On 06/09/16 13:41, Wei Liu wrote: >> --- a/xen/drivers/char/console.c >> +++ b/xen/drivers/char/console.c >> @@ -735,10 +735,11 @@ void __init console_init_preirq(void) >> spin_lock(&console_lock); >> __putstr(xen_banner()); >> spin_unlock(&console

Re: [Xen-devel] [PATCH v2 2/4] gcov: collect more sections to constructor list

2016-09-06 Thread Jan Beulich
>>> On 06.09.16 at 14:41, wrote: > The version of gcc (4.9.2) I use put constructors into .init_array* > section(s). Collect those sections into constructor list as well. > > Modify both arm and x86 scripts to keep them in sync. > > Signed-off-by: Wei Liu > Acked-by: Julien Grall Acked-by: Ja

Re: [Xen-devel] Bug in hash changes to netback in 4.7.2 kernel

2016-09-06 Thread Anthony Wright
On 06/09/2016 13:57, Paul Durrant wrote: >> -Original Message- >> From: Anthony Wright [mailto:anth...@overnetdata.com] >> Sent: 06 September 2016 13:23 >> To: Xen-devel >> Cc: Paul Durrant >> Subject: Bug in hash changes to netback in 4.7.2 kernel >> >> When I run Xen (4.7.0) nested in V

Re: [Xen-devel] [PATCH v2 4/4] xen: make clear gcov support limitation in Kconfig

2016-09-06 Thread Jan Beulich
>>> On 06.09.16 at 14:41, wrote: > Signed-off-by: Wei Liu Acked-by: Jan Beulich with one possible further request: > --- a/xen/Kconfig.debug > +++ b/xen/Kconfig.debug > @@ -33,6 +33,12 @@ config GCOV > ---help--- > Enable gcov (a test coverage program in GCC) support. > >

Re: [Xen-devel] [PATCH v2 4/4] xen: make clear gcov support limitation in Kconfig

2016-09-06 Thread Andrew Cooper
On 06/09/16 14:37, Jan Beulich wrote: On 06.09.16 at 14:41, wrote: >> Signed-off-by: Wei Liu > Acked-by: Jan Beulich > > with one possible further request: > >> --- a/xen/Kconfig.debug >> +++ b/xen/Kconfig.debug >> @@ -33,6 +33,12 @@ config GCOV >> ---help--- >> Enable gco

[Xen-devel] Xen Code Review Dashboard

2016-09-06 Thread Anubha Aggarwal
Hello i am Anubha . I am interested to do work in this project . I am good at Java, Xml and Sql. So can you guide me how i can make my first contribution to it. Tnks ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

Re: [Xen-devel] Bug in hash changes to netback in 4.7.2 kernel

2016-09-06 Thread Paul Durrant
> -Original Message- > From: Anthony Wright [mailto:anth...@overnetdata.com] > Sent: 06 September 2016 14:37 > To: Paul Durrant ; Xen-devel de...@lists.xen.org> > Subject: Re: Bug in hash changes to netback in 4.7.2 kernel > > On 06/09/2016 13:57, Paul Durrant wrote: > >> -Original Me

Re: [Xen-devel] [OSSTEST PATCH v3 19/25] Introduce ts-xtf-install

2016-09-06 Thread Ian Jackson
Wei Liu writes ("[OSSTEST PATCH v3 19/25] Introduce ts-xtf-install"): > Extract XTF to the desired location. > > Signed-off-by: Wei Liu Acked-by: Ian Jackson ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

Re: [Xen-devel] [OSSTEST PATCH v3 25/25] Create XTF branch

2016-09-06 Thread Ian Jackson
Wei Liu writes ("[OSSTEST PATCH v3 25/25] Create XTF branch"): > This branch contains Xen and Linux build jobs and all jobs which contain > xtf in their name. This looks plausible but can you please provide the output from standalone-generate-dump-flight-runvars ? Ian. __

Re: [Xen-devel] [OSSTEST PATCH v3 20/25] mfi-common: create xtf build job for 4.4 onwards

2016-09-06 Thread Ian Jackson
Wei Liu writes ("[OSSTEST PATCH v3 20/25] mfi-common: create xtf build job for 4.4 onwards"): > Xen 4.4 is the oldest one that we still provide security support at this > point in time. > > Signed-off-by: Wei Liu This ought to come with a diff, showing the change to the output of standalone-gen

Re: [Xen-devel] [PATCH v2 1/4] xen: indicate gcov in log messages

2016-09-06 Thread Wei Liu
On Tue, Sep 06, 2016 at 07:34:26AM -0600, Jan Beulich wrote: > >>> On 06.09.16 at 14:47, wrote: > > On 06/09/16 13:41, Wei Liu wrote: > >> --- a/xen/drivers/char/console.c > >> +++ b/xen/drivers/char/console.c > >> @@ -735,10 +735,11 @@ void __init console_init_preirq(void) > >> spin_lock(&co

Re: [Xen-devel] [PATCH V3] tools/libxc, xen/x86: Added xc_set_mem_access_multi()

2016-09-06 Thread Jan Beulich
>>> On 06.09.16 at 12:00, wrote: > --- a/xen/arch/arm/p2m.c > +++ b/xen/arch/arm/p2m.c > @@ -1836,6 +1836,15 @@ long p2m_set_mem_access(struct domain *d, gfn_t gfn, > uint32_t nr, > return 0; > } > > +long p2m_set_mem_access_multi(struct domain *d, > + const X

Re: [Xen-devel] [OSSTEST PATCH v3 21/25] Introduce ts-xtf-fep

2016-09-06 Thread Ian Jackson
Wei Liu writes ("[OSSTEST PATCH v3 21/25] Introduce ts-xtf-fep"): > Test the availability of FEP during runtime. > > Signed-off-by: Wei Liu Acked-by: Ian Jackson ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH V3] tools/libxc, xen/x86: Added xc_set_mem_access_multi()

2016-09-06 Thread Ian Jackson
Razvan Cojocaru writes ("Re: [PATCH V3] tools/libxc, xen/x86: Added xc_set_mem_access_multi()"): > But before that, since Tamas has expressed interest in this feature and > is already working with ARM projects, maybe he'd be interested in a > follow-up patch implementing this for ARM? IMHO this wo

Re: [Xen-devel] [PATCH v2 1/4] xen: indicate gcov in log messages

2016-09-06 Thread Jan Beulich
>>> On 06.09.16 at 16:06, wrote: > On Tue, Sep 06, 2016 at 07:34:26AM -0600, Jan Beulich wrote: >> >>> On 06.09.16 at 14:47, wrote: >> > On 06/09/16 13:41, Wei Liu wrote: >> >> --- a/xen/drivers/char/console.c >> >> +++ b/xen/drivers/char/console.c >> >> @@ -735,10 +735,11 @@ void __init console_

Re: [Xen-devel] [PATCH v2 1/4] xen: indicate gcov in log messages

2016-09-06 Thread Andrew Cooper
On 06/09/16 15:06, Wei Liu wrote: > On Tue, Sep 06, 2016 at 07:34:26AM -0600, Jan Beulich wrote: > On 06.09.16 at 14:47, wrote: >>> On 06/09/16 13:41, Wei Liu wrote: --- a/xen/drivers/char/console.c +++ b/xen/drivers/char/console.c @@ -735,10 +735,11 @@ void __init console_init_

Re: [Xen-devel] [PATCH V3] tools/libxc, xen/x86: Added xc_set_mem_access_multi()

2016-09-06 Thread Razvan Cojocaru
On 09/06/2016 05:07 PM, Jan Beulich wrote: On 06.09.16 at 12:00, wrote: >> --- a/xen/arch/arm/p2m.c >> +++ b/xen/arch/arm/p2m.c >> @@ -1836,6 +1836,15 @@ long p2m_set_mem_access(struct domain *d, gfn_t gfn, >> uint32_t nr, >> return 0; >> } >> >> +long p2m_set_mem_access_multi(struct

Re: [Xen-devel] [PATCH V3] tools/libxc, xen/x86: Added xc_set_mem_access_multi()

2016-09-06 Thread Razvan Cojocaru
On 09/06/2016 05:08 PM, Ian Jackson wrote: > Razvan Cojocaru writes ("Re: [PATCH V3] tools/libxc, xen/x86: Added > xc_set_mem_access_multi()"): >> But before that, since Tamas has expressed interest in this feature and >> is already working with ARM projects, maybe he'd be interested in a >> follo

Re: [Xen-devel] [PATCH V3] tools/libxc, xen/x86: Added xc_set_mem_access_multi()

2016-09-06 Thread Jan Beulich
>>> On 06.09.16 at 16:21, wrote: > On 09/06/2016 05:07 PM, Jan Beulich wrote: > On 06.09.16 at 12:00, wrote: >>> --- a/xen/arch/arm/p2m.c >>> +++ b/xen/arch/arm/p2m.c >>> @@ -1836,6 +1836,15 @@ long p2m_set_mem_access(struct domain *d, gfn_t gfn, >>> uint32_t nr, >>> return 0; >>> } >>

Re: [Xen-devel] xen/xen vs xen/kvm nesting with pv drivers

2016-09-06 Thread Andrew Cooper
On 06/09/16 14:32, Anthony Wright wrote: > On 06/09/2016 14:05, Andrew Cooper wrote: >> On 06/09/16 13:47, Anthony Wright wrote: >>> I tried to install Xen (4.7.0 with linux 4.7.2 Dom0) on an AWS virtual >>> machine and it failed because while AWS uses Xen it requires that you use >>> the PVHVM n

Re: [Xen-devel] [RFC 01/22] xen/arm: do_trap_instr_abort_guest: Move the IPA computation out of the switch

2016-09-06 Thread Julien Grall
On 31/08/16 20:43, Stefano Stabellini wrote: On Tue, 16 Aug 2016, Julien Grall wrote: Hi Stefano, On 16/08/2016 01:21, Stefano Stabellini wrote: On Thu, 28 Jul 2016, Julien Grall wrote: A follow-up patch will add more case to the switch that will require the IPA. So move the computation out

Re: [Xen-devel] [RFC 14/22] xen/arm: p2m: Re-implement p2m_cache_flush using p2m_get_entry

2016-09-06 Thread Julien Grall
Hi Stefano, On 05/09/16 22:13, Stefano Stabellini wrote: On Thu, 28 Jul 2016, Julien Grall wrote: The function p2m_cache_flush can be re-implemented using the generic function p2m_get_entry by iterating over the range and using the mapping order given by the callee. As the current implementati

Re: [Xen-devel] [RFC 15/22] xen/arm: p2m: Re-implement relinquish_p2m_mapping using p2m_get_entry

2016-09-06 Thread Julien Grall
Hi Stefano, On 05/09/16 22:58, Stefano Stabellini wrote: On Thu, 28 Jul 2016, Julien Grall wrote: The current implementation of relinquish_p2m_mapping is modifying the page table to erase the entry one by one. However, this is not necessary because the domain is not running anymore and therefor

[Xen-devel] [PATCH] libelf: drop pointless uses of __FUNCTION__

2016-09-06 Thread Jan Beulich
Non-debugging message text should be (and is in the cases here, albeit often only with the addition of an ELF: prefix) distinguishable without also logging function names. In the messages touched at once use %#x (or variants thereof) in favor of 0x%x. Signed-off-by: Jan Beulich --- a/xen/common

Re: [Xen-devel] [PATCH] libelf: drop pointless uses of __FUNCTION__

2016-09-06 Thread Andrew Cooper
On 06/09/16 16:10, Jan Beulich wrote: > --- a/xen/common/libelf/libelf-loader.c > +++ b/xen/common/libelf/libelf-loader.c > @@ -33,7 +33,7 @@ elf_errorstatus elf_init(struct elf_bina > > if ( !elf_is_elfbinary(image_input, size) ) > { > -elf_err(elf, "%s: not an ELF binary\n", _

Re: [Xen-devel] [PATCH] libelf: drop pointless uses of __FUNCTION__

2016-09-06 Thread Ian Jackson
Jan Beulich writes ("[PATCH] libelf: drop pointless uses of __FUNCTION__"): > Non-debugging message text should be (and is in the cases here, albeit > often only with the addition of an ELF: prefix) distinguishable without > also logging function names. > > In the messages touched at once use %#x

Re: [Xen-devel] Xen Code Review Dashboard

2016-09-06 Thread Jesus M. Gonzalez-Barahona
Hi, Anubha, Are you commenting about this in the context of Outreacy or anything else? Jesus. On Tue, 2016-09-06 at 19:11 +0530, Anubha Aggarwal wrote: > Hello i am Anubha . I am interested to do work in this project . I am > good at Java, Xml and Sql. > So can you guide me how i can mak

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

2016-09-06 Thread osstest service owner
flight 100775 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/100775/ 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 1

Re: [Xen-devel] Xen Code Review Dashboard

2016-09-06 Thread Anubha Aggarwal
Yes in the context of outreachy. On 6 Sep 2016 20:43, "Jesus M. Gonzalez-Barahona" wrote: > Hi, Anubha, > > Are you commenting about this in the context of Outreacy or anything > else? > > Jesus. > > On Tue, 2016-09-06 at 19:11 +0530, Anubha Aggarwal wrote: > > Hello i am Anubha . I am in

Re: [Xen-devel] [PATCH v5 06/16] x86/boot/reloc: create generic alloc and copy functions

2016-09-06 Thread Doug Goldstein
On 8/30/16 9:32 AM, Daniel Kiper wrote: > On Thu, Aug 25, 2016 at 05:34:31AM -0600, Jan Beulich wrote: > On 20.08.16 at 00:43, wrote: >>> Create generic alloc and copy functions. We need >>> separate tools for memory allocation and copy to >>> provide multiboot2 protocol support. >>> >>> Signe

Re: [Xen-devel] xen/xen vs xen/kvm nesting with pv drivers

2016-09-06 Thread Konrad Rzeszutek Wilk
On Tue, Sep 06, 2016 at 03:36:28PM +0100, Andrew Cooper wrote: > On 06/09/16 14:32, Anthony Wright wrote: > > On 06/09/2016 14:05, Andrew Cooper wrote: > >> On 06/09/16 13:47, Anthony Wright wrote: > >>> I tried to install Xen (4.7.0 with linux 4.7.2 Dom0) on an AWS virtual > >>> machine and it fa

Re: [Xen-devel] [PATCH] docs: document old SUSE/Novell unplug for HVM

2016-09-06 Thread Konrad Rzeszutek Wilk
On Fri, Sep 02, 2016 at 11:32:55AM +0200, Olaf Hering wrote: > Signed-off-by: Olaf Hering Reviewed-by: Konrad Rzeszutek Wilk And if nobody complains I will commit it in 'staging' in a day or so. P.S. I couldn't actually find it - but is there an spec for what the BARs in the Xen Platform PCI d

Re: [Xen-devel] Test Meetup at Developer Summit

2016-09-06 Thread Doug Goldstein
On 9/5/16 6:19 AM, Lars Kurth wrote: > == Attendees == > Lars Kurth > George Dunlap > Doug Goldstein > Andrew Cooper > Paul Durrant > > There were a few others, which I may have missed I know Jonathan Creekmore was involved as well. > > I tried to transcribe from a recording we had at lunch, bu

Re: [Xen-devel] [MINIOS PATCH] Add travis.yml and travis-build script

2016-09-06 Thread Doug Goldstein
On 9/5/16 9:43 AM, Wei Liu wrote: > Signed-off-by: Wei Liu > --- > See: > https://travis-ci.org/liuw/mini-os/builds/157653746 > > Cc: Samuel Thibault > Cc: Juergen Gross > Cc: Doug Goldstein > > IRC notification is not yet set up. > > Doug, can we mirror mini-os.git to github/xen-project as

Re: [Xen-devel] missing unplug of SCSI devices in HVM guest

2016-09-06 Thread George Dunlap
On Wed, Aug 24, 2016 at 10:24 AM, Olaf Hering wrote: > Does anyone remember why the the vbd frontend drivers also claim the > SCSI disks, but the vbd backend in qemu has no unplug support for SCSI? > > The current situation for qemu-xen and qemu-xen-traditional is that both > will create an emulat

Re: [Xen-devel] [PATCH v4 1/9] livepatch: Clear .bss when payload is reverted

2016-09-06 Thread Konrad Rzeszutek Wilk
On Wed, Aug 24, 2016 at 02:55:21AM -0600, Jan Beulich wrote: > >>> On 24.08.16 at 04:22, wrote: > > --- a/xen/common/livepatch.c > > +++ b/xen/common/livepatch.c > > @@ -70,6 +70,9 @@ struct payload { > > unsigned int nsyms; /* Nr of entries in .strtab and > > symbols. */ >

Re: [Xen-devel] [PATCH v4 1/9] livepatch: Clear .bss when payload is reverted

2016-09-06 Thread Konrad Rzeszutek Wilk
On Thu, Aug 25, 2016 at 05:08:16PM +0100, Andrew Cooper wrote: > On 24/08/16 09:55, Jan Beulich wrote: > > > > > On 24.08.16 at 04:22, wrote: > > > --- a/xen/common/livepatch.c > > > +++ b/xen/common/livepatch.c > > > @@ -70,6 +70,9 @@ struct payload { > > > unsigned int nsyms;

Re: [Xen-devel] [PATCH v4 4/9] version: Print build-id at bootup.

2016-09-06 Thread Konrad Rzeszutek Wilk
On Wed, Aug 24, 2016 at 02:58:48AM -0600, Jan Beulich wrote: > >>> On 24.08.16 at 04:22, wrote: > > Livepatch expected at some point to be able to print the > > build-id during bootup, which it did not. The reason is > > that xen_build_init and livepatch_init are both __initcall > > type routines

Re: [Xen-devel] [PATCH v2 1/2] x86/altp2m: use __get_gfn_type_access to avoid lock conflicts

2016-09-06 Thread George Dunlap
On Wed, Aug 10, 2016 at 4:00 PM, Tamas K Lengyel wrote: > From: Tamas K Lengyel > > Use __get_gfn_type_access instead of get_gfn_type_access when checking > the hostp2m entries during altp2m mem_access setting and gfn remapping > to avoid a lock conflict which can make dom0 freeze. During mem_acc

Re: [Xen-devel] [RFC 18/22] xen/arm: p2m: Introduce p2m_set_entry and __p2m_set_entry

2016-09-06 Thread Julien Grall
Hi Stefano, On 06/09/16 02:08, Stefano Stabellini wrote: On Thu, 28 Jul 2016, Julien Grall wrote: The ARM architecture mandates to use of a break-before-make sequence when changing translation entries if the page table is shared between multiple CPUs whenever a valid entry is replaced by anothe

Re: [Xen-devel] [PATCH v2 01/20] livepatch: Bubble up sanity checks on Elf relocs

2016-09-06 Thread Konrad Rzeszutek Wilk
On Thu, Aug 25, 2016 at 09:37:16AM -0400, Konrad Rzeszutek Wilk wrote: > The checks for SHT_REL[,A] ELF sanity checks does not need to > be in the platform specific file and can be bubbled up > in the platform agnostic file. > > This makes the ARM 32/64 implementation easier as the > duplicate che

Re: [Xen-devel] [PATCH v2 15/20] livepatch: Move test-cases to common

2016-09-06 Thread Konrad Rzeszutek Wilk
On Thu, Aug 25, 2016 at 09:05:27AM -0600, Jan Beulich wrote: > >>> On 25.08.16 at 15:37, wrote: > > So they can be shared with ARM64 (but not yet, so they > > are only built on x86). > > > > No functional change. > > > > We also need to tweak the MAINTAINERS and .gitignore file > > > > Signed-o

Re: [Xen-devel] [PATCH v2 15/20] livepatch: Move test-cases to common

2016-09-06 Thread Konrad Rzeszutek Wilk
On Thu, Aug 25, 2016 at 09:37:30AM -0400, Konrad Rzeszutek Wilk wrote: > So they can be shared with ARM64 (but not yet, so they > are only built on x86). > > No functional change. > > We also need to tweak the MAINTAINERS and .gitignore file > > Signed-off-by: Konrad Rzeszutek Wilk Ross, could

Re: [Xen-devel] [PATCH v4 9/9] livepach: Add .livepatch.hooks functions and test-case

2016-09-06 Thread Konrad Rzeszutek Wilk
On Tue, Aug 23, 2016 at 10:22:12PM -0400, Konrad Rzeszutek Wilk wrote: > From: Ross Lagerwall > > Add hook functions which run during patch apply and patch revert. > Hook functions are used by livepatch payloads to manipulate data > structures during patching, etc. > > One use case is the XSA91.

Re: [Xen-devel] xen/xen vs xen/kvm nesting with pv drivers

2016-09-06 Thread Konrad Rzeszutek Wilk
. snip.. > > >> It would be lovely if someone would work on this, but it is a very large > > >> swamp. > > >> > > >> ~Andrew > > > Does the L1's Dom0 have to issue the hypercalls directly? Would it be > > > possible to get the L1's Dom0 to issue the request to the L1 hypervisor > > > and that to ca

[Xen-devel] [distros-debian-snapshot test] 67647: tolerable FAIL

2016-09-06 Thread Platform Team regression test user
flight 67647 distros-debian-snapshot real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/67647/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-i386-amd64-daily-netboot-pygrub 9 debian-di-install fail like 67607 test-armhf-arm

Re: [Xen-devel] [PATCH v2 4/9] x86/pv: Implement pv_hypercall() in C

2016-09-06 Thread Andrew Cooper
On 06/09/16 18:44, George Dunlap wrote: > On Tue, Sep 6, 2016 at 11:12 AM, Andrew Cooper > wrote: >> In a similar style to hvm_do_hypercall(). The C version is far easier to >> understand and edit than the assembly versions. >> >> There are a few small differences however. The register clobberin

Re: [Xen-devel] [PATCH v2 4/9] x86/pv: Implement pv_hypercall() in C

2016-09-06 Thread George Dunlap
On Tue, Sep 6, 2016 at 11:12 AM, Andrew Cooper wrote: > In a similar style to hvm_do_hypercall(). The C version is far easier to > understand and edit than the assembly versions. > > There are a few small differences however. The register clobbering values > have changed (to match the HVM side),

Re: [Xen-devel] [PATCH linux v3 3/9] xen: introduce xen_vcpu_id mapping

2016-09-06 Thread Stefano Stabellini
On Tue, 6 Sep 2016, Vitaly Kuznetsov wrote: > Stefano Stabellini writes: > > > On Mon, 5 Sep 2016, Vitaly Kuznetsov wrote: > >> Julien Grall writes: > >> > >> > Hi Vitaly, > >> > > >> > On 26/07/16 13:30, Vitaly Kuznetsov wrote: > >> >> It may happen that Xen's and Linux's ideas of vCPU id dive

Re: [Xen-devel] [PATCH] docs: document old SUSE/Novell unplug for HVM

2016-09-06 Thread Stefano Stabellini
On Tue, 6 Sep 2016, Konrad Rzeszutek Wilk wrote: > On Fri, Sep 02, 2016 at 11:32:55AM +0200, Olaf Hering wrote: > > Signed-off-by: Olaf Hering > > Reviewed-by: Konrad Rzeszutek Wilk > > And if nobody complains I will commit it in 'staging' in a day or so. Looks good to me too. > P.S. > I cou

Re: [Xen-devel] [RFC 15/22] xen/arm: p2m: Re-implement relinquish_p2m_mapping using p2m_get_entry

2016-09-06 Thread Stefano Stabellini
On Tue, 6 Sep 2016, Julien Grall wrote: > Hi Stefano, > > On 05/09/16 22:58, Stefano Stabellini wrote: > > On Thu, 28 Jul 2016, Julien Grall wrote: > > > The current implementation of relinquish_p2m_mapping is modifying the > > > page table to erase the entry one by one. However, this is not neces

Re: [Xen-devel] [PATCH v4 9/9] livepach: Add .livepatch.hooks functions and test-case

2016-09-06 Thread Andrew Cooper
On 06/09/16 18:22, Konrad Rzeszutek Wilk wrote: > On Tue, Aug 23, 2016 at 10:22:12PM -0400, Konrad Rzeszutek Wilk wrote: >> From: Ross Lagerwall >> >> Add hook functions which run during patch apply and patch revert. >> Hook functions are used by livepatch payloads to manipulate data >> structures

Re: [Xen-devel] [RFC 18/22] xen/arm: p2m: Introduce p2m_set_entry and __p2m_set_entry

2016-09-06 Thread Stefano Stabellini
On Tue, 6 Sep 2016, Julien Grall wrote: > > > +if ( !p2m_valid(entry) || p2m_is_superpage(entry, level) ) > > > +return; > > > + > > > +if ( level == 3 ) > > > +{ > > > +p2m_put_l3_page(_mfn(entry.p2m.base), entry.p2m.type); > > > +return; > > > +} > > > + >

Re: [Xen-devel] [RFC 20/22] xen/arm: p2m: Re-implement p2m_insert_mapping using p2m_set_entry

2016-09-06 Thread Stefano Stabellini
On Thu, 28 Jul 2016, Julien Grall wrote: > The function p2m_insert_mapping can be re-implemented using the generic > function p2m_set_entry. > > Note that the mapping is not reverted anymore if Xen fails to insert a > mapping. This was added to ensure the MMIO are not kept half-mapped > in case of

Re: [Xen-devel] [PATCH v2 14/20] livepatch: ARM 32|64: Ignore mapping symbols: $[d, a, x, t]

2016-09-06 Thread Konrad Rzeszutek Wilk
.snip.. > > diff --git a/xen/arch/arm/livepatch.c b/xen/arch/arm/livepatch.c > > index f49e347..c290602 100644 > > --- a/xen/arch/arm/livepatch.c > > +++ b/xen/arch/arm/livepatch.c > > @@ -82,6 +82,38 @@ void arch_livepatch_unmask(void) > > local_abort_enable(); > > } > > > > +int arch_is_pa

Re: [Xen-devel] [RFC 22/22] xen/arm: p2m: Do not handle shattering in p2m_create_table

2016-09-06 Thread Stefano Stabellini
On Thu, 28 Jul 2016, Julien Grall wrote: > The helper p2m_create_table is only called to create a brand new table. > > Signed-off-by: Julien Grall Reviewed-by: Stefano Stabellini > xen/arch/arm/p2m.c | 51 ++- > 1 file changed, 6 insertions(+),

Re: [Xen-devel] [RFC 19/22] xen/arm: p2m: Re-implement p2m_remove_using using p2m_set_entry

2016-09-06 Thread Stefano Stabellini
On Thu, 28 Jul 2016, Julien Grall wrote: > The function p2m_insert_mapping can be re-implemented using the generic > function p2m_set_entry. > > Also drop the operation REMOVE in apply_* as nobody is using it anymore. > Note that the functions could have been dropped in one go at the end, > howeve

Re: [Xen-devel] [RFC 21/22] xen/arm: p2m: Re-implement p2m_set_mem_access using p2m_{set, get}_entry

2016-09-06 Thread Stefano Stabellini
On Thu, 28 Jul 2016, Julien Grall wrote: > The function p2m_set_mem_access can be re-implemented using the generic > functions p2m_get_entry and __p2m_set_entry. > > Note that because of the implementation of p2m_get_entry, a TLB > invalidation instruction will be issued for each 4KB page. Therefo

Re: [Xen-devel] [OSSTEST PATCH v3 25/25] Create XTF branch

2016-09-06 Thread Wei Liu
There is one minor rebase error. The patch below fixes it. ---8<--- From f950c2a8a23fcfbdce738a6c8af55f9a2941854c Mon Sep 17 00:00:00 2001 From: Wei Liu Date: Tue, 6 Sep 2016 20:13:11 +0100 Subject: [OSSTEST PATCH] fixup! Create XTF branch --- cr-daily-branch | 1 + 1 file changed, 1 insertion

Re: [Xen-devel] [RFC 21/22] xen/arm: p2m: Re-implement p2m_set_mem_access using p2m_{set, get}_entry

2016-09-06 Thread Razvan Cojocaru
On 09/06/16 22:06, Stefano Stabellini wrote: > On Thu, 28 Jul 2016, Julien Grall wrote: >> > The function p2m_set_mem_access can be re-implemented using the generic >> > functions p2m_get_entry and __p2m_set_entry. >> > >> > Note that because of the implementation of p2m_get_entry, a TLB >> > inva

Re: [Xen-devel] [OSSTEST PATCH v3 25/25] Create XTF branch

2016-09-06 Thread Wei Liu
On Tue, Sep 06, 2016 at 03:03:50PM +0100, Ian Jackson wrote: > Wei Liu writes ("[OSSTEST PATCH v3 25/25] Create XTF branch"): > > This branch contains Xen and Linux build jobs and all jobs which contain > > xtf in their name. > > This looks plausible but can you please provide the output from > st

Re: [Xen-devel] [RFC 21/22] xen/arm: p2m: Re-implement p2m_set_mem_access using p2m_{set, get}_entry

2016-09-06 Thread Stefano Stabellini
On Tue, 6 Sep 2016, Razvan Cojocaru wrote: > On 09/06/16 22:06, Stefano Stabellini wrote: > > On Thu, 28 Jul 2016, Julien Grall wrote: > >> > The function p2m_set_mem_access can be re-implemented using the generic > >> > functions p2m_get_entry and __p2m_set_entry. > >> > > >> > Note that because

Re: [Xen-devel] [PATCH] xen/arm: alternative: Make it possible to patch outside of the hypervisor

2016-09-06 Thread Konrad Rzeszutek Wilk
On Thu, Sep 01, 2016 at 08:56:39AM +0100, Julien Grall wrote: > With livepatch the alternatives that should be patched are outside of > the Xen hypervisor _start -> _end. The current code is assuming that > only Xen could be patched and therefore will explode when a payload > contains alternatives.

[Xen-devel] [xen-4.7-testing test] 100770: regressions - trouble: blocked/broken/fail/pass

2016-09-06 Thread osstest service owner
flight 100770 xen-4.7-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/100770/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-libvirt 3 host-install(3)broken REGR. vs. 100635 test-amd64-amd6

Re: [Xen-devel] [PATCH v2 18/20] xen/arm32/livepatch: Add BPICALLIS to helper to invalidate all instruction caches

2016-09-06 Thread Konrad Rzeszutek Wilk
On Thu, Sep 01, 2016 at 04:13:30PM +0100, Julien Grall wrote: > Hi Konrad, > > On 25/08/16 14:37, Konrad Rzeszutek Wilk wrote: > > When we are flushing the cache we are most likley also want > > to flush the branch predictor too. Hence add this. > > I think it makes more sense to fold this patch

Re: [Xen-devel] [PATCH v4 6/9] livepatch: Add parsing for the symbol+0x

2016-09-06 Thread Konrad Rzeszutek Wilk
On Wed, Aug 24, 2016 at 03:08:01AM -0600, Jan Beulich wrote: > >>> On 24.08.16 at 04:22, wrote: > > --- a/xen/common/livepatch.c > > +++ b/xen/common/livepatch.c > > @@ -237,13 +237,34 @@ static const char *livepatch_symbols_lookup(unsigned > > long addr, > > static int resolve_old_address(struc

Re: [Xen-devel] [PATCH v4 7/9] livepatch: NOP if func->new_[addr] is zero.

2016-09-06 Thread Konrad Rzeszutek Wilk
On Wed, Aug 24, 2016 at 03:13:18AM -0600, Jan Beulich wrote: > >>> On 24.08.16 at 04:22, wrote: > > The NOP functionality will NOP any of the code at > > the 'old_addr' or at 'name' if the 'new_addr' is zero. > > The purpose of this is to NOP out calls, such as: > > > > e8 <4-bytes-offset> > >

Re: [Xen-devel] [PATCH v2 07/20] arm/x86: Add ALTERNATIVE and HAS_EX_TABLE

2016-09-06 Thread Konrad Rzeszutek Wilk
On Thu, Aug 25, 2016 at 08:54:51AM -0600, Jan Beulich wrote: > >>> On 25.08.16 at 15:37, wrote: > > @@ -22,6 +24,12 @@ config X86 > > select NUMA > > select VGA > > > > +config ALTERNATIVE > > + bool > > + > > +config HAS_EX_TABLE > > + bool > > These need to move out of arch/x86/,

Re: [Xen-devel] [PATCH v2 4/9] x86/pv: Implement pv_hypercall() in C

2016-09-06 Thread George Dunlap
On Tue, Sep 6, 2016 at 6:49 PM, Andrew Cooper wrote: > On 06/09/16 18:44, George Dunlap wrote: > > On Tue, Sep 6, 2016 at 11:12 AM, Andrew Cooper > wrote: > > In a similar style to hvm_do_hypercall(). The C version is far easier to > understand and edit than the assembly versions. > > There are

Re: [Xen-devel] [PATCH v2 07/20] arm/x86: Add ALTERNATIVE and HAS_EX_TABLE

2016-09-06 Thread Konrad Rzeszutek Wilk
On Thu, Aug 25, 2016 at 08:56:07AM -0600, Jan Beulich wrote: > >>> On 25.08.16 at 15:58, wrote: > > On 25/08/16 14:37, Konrad Rzeszutek Wilk wrote: > > > >> x86 implements all of them by default - and we just > >> add two extra CONFIG_ variables to be declared in autoconf.h. > >> > >> ARM 64 only

Re: [Xen-devel] [PATCH v2 07/20] arm/x86: Add ALTERNATIVE and HAS_EX_TABLE

2016-09-06 Thread Konrad Rzeszutek Wilk
On Tue, Sep 06, 2016 at 04:36:40PM -0400, Konrad Rzeszutek Wilk wrote: > On Thu, Aug 25, 2016 at 08:56:07AM -0600, Jan Beulich wrote: > > >>> On 25.08.16 at 15:58, wrote: > > > On 25/08/16 14:37, Konrad Rzeszutek Wilk wrote: > > > > > >> x86 implements all of them by default - and we just > > >>

Re: [Xen-devel] [PATCH v2 19/20] livepatch/elf: Adjust section aligment to word

2016-09-06 Thread Konrad Rzeszutek Wilk
On Thu, Aug 25, 2016 at 09:11:15AM -0600, Jan Beulich wrote: > >>> On 25.08.16 at 15:37, wrote: > > On most architectures it does not matter what the aligment is. > > > > On ARM32 it is paramount that the aligment is word-size (4) > > otherwise we get a Data Abort when trying to perform ELF > > r

[Xen-devel] [xen-4.5-testing test] 100769: tolerable FAIL - PUSHED

2016-09-06 Thread osstest service owner
flight 100769 xen-4.5-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/100769/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-armhf-armhf-xl-rtds 11 guest-start fail REGR. vs. 100637 test-amd64-amd64-xl-qemu

Re: [Xen-devel] [PATCH v2 1/4] xen: indicate gcov in log messages

2016-09-06 Thread Stefano Stabellini
On Tue, 6 Sep 2016, Wei Liu wrote: > Signed-off-by: Wei Liu Acked-by: Stefano Stabellini > Cc: Stefano Stabellini > Cc: Julien Grall > Cc: Jan Beulich > Cc: Andrew Cooper > --- > xen/arch/arm/traps.c| 5 +++-- > xen/arch/x86/x86_64/traps.c | 5 +++-- > xen/drivers/char/console.c

Re: [Xen-devel] [PATCH v3 3/6] VMX: Cleanup PI per-cpu blocking list when vcpu is destroyed

2016-09-06 Thread Wu, Feng
> -Original Message- > From: Dario Faggioli [mailto:dario.faggi...@citrix.com] > Sent: Tuesday, September 6, 2016 5:22 PM > To: Wu, Feng ; xen-devel@lists.xen.org > Cc: Tian, Kevin ; george.dun...@eu.citrix.com; > andrew.coop...@citrix.com; jbeul...@suse.com > Subject: Re: [Xen-devel] [PA

Re: [Xen-devel] Failed vm entry with heavy use of emulator

2016-09-06 Thread Tamas K Lengyel
On Tue, Jan 5, 2016 at 7:37 AM, Razvan Cojocaru wrote: > On 01/05/2016 04:16 PM, Tamas K Lengyel wrote: >> >> >> On Tue, Jan 5, 2016 at 3:01 PM, Razvan Cojocaru >> mailto:rcojoc...@bitdefender.com>> wrote: >> >> On 01/05/2016 03:49 PM, Andrew Cooper wrote: >> > On 05/01/16 13:39, Razvan Co

Re: [Xen-devel] [PATCH v2 13/20] livepatch: Initial ARM64 support.

2016-09-06 Thread Konrad Rzeszutek Wilk
> > +void arch_livepatch_apply_jmp(struct livepatch_func *func) > > +{ > > +uint32_t insn; > > +uint32_t *old_ptr; > > +uint32_t *new_ptr; > > + > > +BUILD_BUG_ON(PATCH_INSN_SIZE > sizeof(func->opaque)); > > +BUILD_BUG_ON(PATCH_INSN_SIZE != sizeof(insn)); > > + > > +ASSERT(v

Re: [Xen-devel] [RFC PATCH] x86/apicv: fix RTC periodic timer and apicv issue

2016-09-06 Thread Xuquan (Euler)
On September 06, 2016 9:21 PM, Jan Beulich wrote: On 06.09.16 at 13:22, wrote: >> On September 06, 2016 5:05 PM, Jan Beulich >> On 06.09.16 at 10:53, wrote: to be honest, my fix is also a tradeoff.. w/ my patch, multiple pending pt interrupt instances may be combined as one i

[Xen-devel] [xen-4.6-testing test] 100771: regressions - FAIL

2016-09-06 Thread osstest service owner
flight 100771 xen-4.6-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/100771/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-qemuu-winxpsp3 17 guest-start/win.repeat fail REGR. vs. 100630 Regressions

Re: [Xen-devel] [PATCH v2 13/20] livepatch: Initial ARM64 support.

2016-09-06 Thread Konrad Rzeszutek Wilk
On Thu, Aug 25, 2016 at 09:02:48AM -0600, Jan Beulich wrote: > >>> On 25.08.16 at 15:37, wrote: > > --- a/xen/include/xen/types.h > > +++ b/xen/include/xen/types.h > > @@ -14,6 +14,12 @@ > > #define NULL ((void*)0) > > #endif > > > > +#define U16_MAX ((u16)~0U) > > +#define S16_MAX

Re: [Xen-devel] [PATCH] xen/arm: alternative: Make it possible to patch outside of the hypervisor

2016-09-06 Thread Konrad Rzeszutek Wilk
> > --- > > xen/arch/arm/alternative.c | 58 > > +- > > 1 file changed, 31 insertions(+), 27 deletions(-) > > > > diff --git a/xen/arch/arm/alternative.c b/xen/arch/arm/alternative.c > > index 8ee5a11..db4bd0f 100644 > > --- a/xen/arch/arm/alternative.

<    1   2   3   >