[Xen-devel] [PATCH v3 6/6] xen/arm: Map mmio-sram nodes as un-cached memory

2016-09-06 Thread Edgar E. Iglesias
From: "Edgar E. Iglesias" Map mmio-sram nodes as un-cached memory. If the node has set the no-memory-wc property, we map it as device. The DTS bindings for mmio-sram nodes can be found in the Linux tree under Documentation/devicetree/bindings/sram/sram.txt. Signed-off-by: Edgar E. Iglesias ---

[Xen-devel] [PATCH v3 0/6] xen/arm: Add support for mapping mmio-sram nodes into dom0

2016-09-06 Thread Edgar E. Iglesias
From: "Edgar E. Iglesias" This series adds support for mapping mmio-sram nodes into dom0 as normal uncached MEMORY with RWX perms. If the no-memory-wc prop is present in the DTB node, we create DEVICE RW mappings instead. Julien, I've scratched the map_regions_mattr name since we are now passin

[Xen-devel] [PATCH v3 4/6] xen/device-tree: Make dt_match_node match props

2016-09-06 Thread Edgar E. Iglesias
From: "Edgar E. Iglesias" Make dt_match_node match for existing properties. We only search for the existence of the properties, not for specific values. Signed-off-by: Edgar E. Iglesias --- xen/common/device_tree.c | 5 - xen/include/xen/device_tree.h | 7 +++ 2 files changed, 11

[Xen-devel] [PATCH v3 1/6] xen/arm: p2m: Add support for normal non-cacheable memory

2016-09-06 Thread Edgar E. Iglesias
From: "Edgar E. Iglesias" Add support for describing normal non-cacheable memory. Signed-off-by: Edgar E. Iglesias --- xen/arch/arm/p2m.c | 18 ++ xen/include/asm-arm/p2m.h | 1 + xen/include/asm-arm/page.h | 1 + 3 files changed, 20 insertions(+) diff --git a/xen/a

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

2016-09-06 Thread Julien Grall
Hi Ravzan, On 06/09/2016 20:16, 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 of the implementa

[Xen-devel] [PATCH v3 3/6] xen/device-tree: Add __DT_MATCH macros without braces

2016-09-06 Thread Edgar E. Iglesias
From: "Edgar E. Iglesias" Add __DT_MATCH macros without braces to allow the creation of match descriptors with multiple combined match options. Acked-by: Julien Grall Signed-off-by: Edgar E. Iglesias --- xen/include/xen/device_tree.h | 13 + 1 file changed, 9 insertions(+), 4 dele

[Xen-devel] [PATCH v3 5/6] xen/arm: domain_build: Plumb for different mapping attributes

2016-09-06 Thread Edgar E. Iglesias
From: "Edgar E. Iglesias" Add plumbing for passing around mapping attributes. This is in preparation to allow us to differentiate the attributes for specific device nodes. We still use the same DEVICE mappings for all nodes so this patch has no functional change. Signed-off-by: Edgar E. Iglesia

[Xen-devel] [PATCH v3 2/6] xen/arm: Rename and generalize un/map_regions_rw_cache

2016-09-06 Thread Edgar E. Iglesias
From: "Edgar E. Iglesias" Rename and generalize un/map_regions_rw_cache into un/map_regions_p2mt. The new functions take the mapping attributes as an argument. No functional change. Signed-off-by: Edgar E. Iglesias --- xen/arch/arm/domain_build.c | 18 ++ xen/arch/arm/p2m.c

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

2016-09-06 Thread Julien Grall
Hi Konrad, On 06/09/2016 20:25, Konrad Rzeszutek Wilk wrote: 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 the

[Xen-devel] Xen on the ARM Foundation Model documentation

2016-09-06 Thread Razvan Cojocaru
Hello, I'm trying to get Xen to run on the free ARM Foundation Model. I've found this document, which everyone else seems to copy and slightly modify: https://community.arm.com/groups/processors/blog/2014/03/28/virtualization-on-arm-with-xen However, the document, great as it is for starting out

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

2016-09-06 Thread Wei Liu
On Tue, Sep 06, 2016 at 10:54:32AM -0500, Doug Goldstein wrote: > 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 notifica

Re: [Xen-devel] [PATCH v2 0/4] gcov: more cleanup

2016-09-06 Thread Wei Liu
On Tue, Sep 06, 2016 at 01:41:03PM +0100, Wei Liu wrote: > Wei Liu (4): > xen: indicate gcov in log messages > gcov: collect more sections to constructor list > xen: replace TEST_COVERAGE with CONFIG_GCOV > xen: make clear gcov support limitation in Kconfig > I believe this series has got

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

2016-09-06 Thread Wei Liu
On Tue, Sep 06, 2016 at 11:47:20AM -0400, 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 > Pushed with your reviewed-by. > And if nobody complains I will commit it in 'staging' i

[Xen-devel] [xen-4.5-testing baseline-only test] 67663: regressions - FAIL

2016-09-06 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 67663 xen-4.5-testing real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/67663/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-libvirt 14 guest-savere

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

2016-09-06 Thread Razvan Cojocaru
On 09/07/16 02:31, Tamas K Lengyel wrote: > 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, Andre

Re: [Xen-devel] [PATCH 2/2] x86/HVM: adjust feature checking in MSR intercept handling

2016-09-06 Thread Tian, Kevin
> From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: Friday, September 02, 2016 6:22 PM > > Consistently consult hvm_cpuid(). With that, BNDCFGS gets better > handled outside of VMX specific code, just like XSS. Don't needlessly > check for MTRR support when the MSR being accessed clearly is not

Re: [Xen-devel] [PATCH 1/2] VMX: correct feature checks for MPX and XSAVES

2016-09-06 Thread Tian, Kevin
> From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: Friday, September 02, 2016 6:21 PM > > Their VMCS fields aren't tied to the respective base CPU feature flags > but instead to VMX specific ones. > > Note that while the VMCS GUEST_BNDCFGS field exists if either of the > two respective featur

[Xen-devel] [ovmf test] 100774: all pass - PUSHED

2016-09-06 Thread osstest service owner
flight 100774 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/100774/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf 96c13c011766a950247c743887705cc035a15497 baseline version: ovmf bc54e50e0fe03c570014f

Re: [Xen-devel] [PATCH v2] Livepatch for ARM 64 and 32.

2016-09-06 Thread Konrad Rzeszutek Wilk
On Wed, Aug 31, 2016 at 09:54:30AM -0600, Jan Beulich wrote: > >>> On 31.08.16 at 17:24, wrote: > > On 31/08/16 16:09, Julien Grall wrote: > >> On 31/08/16 16:06, Konrad Rzeszutek Wilk wrote: > >>> So you are thinking of exposing this 'xen feature bits' that Andrew > >>> mention > >>> to be on ARM

[Xen-devel] [qemu-mainline test] 100772: regressions - FAIL

2016-09-06 Thread osstest service owner
flight 100772 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/100772/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-libvirt-raw 5 xen-install fail REGR. vs. 100756 test-armhf-armhf-

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

2016-09-06 Thread Konrad Rzeszutek Wilk
> ...snip.. > > > +case R_AARCH64_ABS32: > > > +ovf = reloc_data(RELOC_OP_ABS, dest, val, 32); > > > +break; > > > > I have noticed that not all the relocations are implemented (e.g > > R_AARCH64_ABS16, R_AARCH64_MOVW_*...). I don't think there is anything > > preve

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.

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

[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] [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

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] 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 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] [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

[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 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

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 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 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: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 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 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 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

[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] 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.

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] [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 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
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 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] [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 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] [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 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] [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] [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 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] 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] [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 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 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] 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

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.

[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 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 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] [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 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] [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 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 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] 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] [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] 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] [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] 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 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 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

[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 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

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] [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", _

[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] [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

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 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] 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] [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] [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 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 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 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] [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 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] [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] [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] [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 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] 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

[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] [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

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] 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

  1   2   3   >