[Xen-devel] [libvirt test] 111209: tolerable all pass - PUSHED

2017-06-30 Thread osstest service owner
flight 111209 libvirt real [real] http://logs.test-lab.xenproject.org/osstest/logs/111209/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-armhf-armhf-libvirt 14 saverestore-support-checkfail like 111061 test-armhf-armhf-libvirt-xsm 14 saveresto

[Xen-devel] [linux-4.1 baseline-only test] 71621: tolerable trouble: blocked/broken/fail/pass

2017-06-30 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 71621 linux-4.1 real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/71621/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-amd64-xl-qemuu-win7-amd64 17 guest-start/win.rep

[Xen-devel] [xtf test] 111251: regressions - trouble: broken/pass

2017-06-30 Thread osstest service owner
flight 111251 xtf real [real] http://logs.test-lab.xenproject.org/osstest/logs/111251/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-xtf-amd64-amd64-4 59 leak-check/check fail REGR. vs. 111074 test-xtf-amd64-amd64-2

[Xen-devel] [linux-3.18 test] 111215: regressions - trouble: blocked/broken/fail/pass

2017-06-30 Thread osstest service owner
flight 111215 linux-3.18 real [real] http://logs.test-lab.xenproject.org/osstest/logs/111215/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-qemut-rhel6hvm-amd 12 guest-start/redhat.repeat fail REGR. vs. 110441 Tests which

[Xen-devel] [xtf test] 111247: regressions - trouble: broken/pass

2017-06-30 Thread osstest service owner
flight 111247 xtf real [real] http://logs.test-lab.xenproject.org/osstest/logs/111247/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-xtf-amd64-amd64-4 59 leak-check/check fail REGR. vs. 111074 test-xtf-amd64-amd64-2

[Xen-devel] [xen-unstable test] 111201: regressions - FAIL

2017-06-30 Thread osstest service owner
flight 111201 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/111201/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-xl 6 xen-install fail REGR. vs. 110465 test-amd64-i386-xl

[Xen-devel] [xen-unstable-smoke test] 111244: tolerable trouble: broken/pass - PUSHED

2017-06-30 Thread osstest service owner
flight 111244 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/111244/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-arm64-arm64-xl-xsm 1 build-check(1) blocked n/a test-amd64-amd64-libvirt 13 mig

[Xen-devel] [xtf test] 111240: regressions - trouble: broken/pass

2017-06-30 Thread osstest service owner
flight 111240 xtf real [real] http://logs.test-lab.xenproject.org/osstest/logs/111240/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-xtf-amd64-amd64-4 59 leak-check/check fail REGR. vs. 111074 test-xtf-amd64-amd64-5

Re: [Xen-devel] [PATCH 4/4] xen: don't use xenstore to save/restore physmap anymore

2017-06-30 Thread Stefano Stabellini
On Fri, 30 Jun 2017, Igor Druzhinin wrote: > If we have a system with xenforeignmemory_map2() implemented > we don't need to save/restore physmap on suspend/restore > anymore. In case we resume a VM without physmap - try to > recreate the physmap during memory region restore phase and > remap map c

Re: [Xen-devel] [PATCH 3/4] xen/mapcache: introduce xen_remap_cache_entry()

2017-06-30 Thread Stefano Stabellini
On Fri, 30 Jun 2017, Igor Druzhinin wrote: > This new call is trying to update a requested map cache entry > according to the changes in the physmap. The call is searching > for the entry, unmaps it, tries to translate the address and > maps again at the same place. If the mapping is dummy this cal

Re: [Xen-devel] [PATCH 2/4] xen/mapcache: add an ability to create dummy mappings

2017-06-30 Thread Stefano Stabellini
On Fri, 30 Jun 2017, Igor Druzhinin wrote: > Dummys are simple anonymous mappings that are placed instead > of regular foreign mappings in certain situations when we need > to postpone the actual mapping but still have to give a > memory region to QEMU to play with. > > This is planned to be used

Re: [Xen-devel] [PATCH 1/4] xen: move physmap saving into a separate function

2017-06-30 Thread Stefano Stabellini
On Fri, 30 Jun 2017, Igor Druzhinin wrote: > Non-functional change. > > Signed-off-by: Igor Druzhinin Reviewed-by: Stefano Stabellini > --- > hw/i386/xen/xen-hvm.c | 57 > --- > 1 file changed, 31 insertions(+), 26 deletions(-) > > diff --git

[Xen-devel] [xen-unstable-smoke test] 111241: tolerable trouble: broken/pass - PUSHED

2017-06-30 Thread osstest service owner
flight 111241 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/111241/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-arm64-arm64-xl-xsm 1 build-check(1) blocked n/a test-amd64-amd64-libvirt 13 mig

Re: [Xen-devel] [RFC] DOMCTL_memattrs_op : a new DOMCTL to play with stage-2 page attributes

2017-06-30 Thread Stefano Stabellini
On Sat, 1 Jul 2017, Zhongze Liu wrote: > >DOMCTL_memattrs_op : a new DOMCTL to play with stage-2 page attributes > > Zhongze Liu > > > *

[Xen-devel] [qemu-mainline baseline-only test] 71617: tolerable trouble: blocked/broken/fail/pass

2017-06-30 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 71617 qemu-mainline real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/71617/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-i386-xl-qemuu-winxpsp3-vcpus1 17 guest-start

Re: [Xen-devel] [PATCH v3 16/16] xen/arm: lpae: Switch from bool_t to bool

2017-06-30 Thread Stefano Stabellini
On Fri, 30 Jun 2017, Julien Grall wrote: > On Fri, 30 Jun 2017, 21:12 Stefano Stabellini, wrote: > On Fri, 30 Jun 2017, Julien Grall wrote: > > We are phasing out the use of bool_t in the hypervisor code. > > This is missing your signed-off-by. > > Reviewed-by: Stefano St

Re: [Xen-devel] [PATCH v2 4/4] vsmc: psci: remove 64 bit mode check

2017-06-30 Thread Stefano Stabellini
On Thu, 22 Jun 2017, Volodymyr Babchuk wrote: > PSCI handling code had helper routine that checked calling convention. > It does not needed anymore, because: > > - Generic handler checks that 64 bit calls can be made only by >64 bit guests. > > - SMCCC requires that 64-bit handler should su

Re: [Xen-devel] [PATCH v2 3/4] arm: traps: handle PSCI calls inside `vsmc.c`

2017-06-30 Thread Stefano Stabellini
On Thu, 22 Jun 2017, Volodymyr Babchuk wrote: > PSCI is part of HVC/SMC interface, so it should be handled in > appropriate place: `vsmc.c`. This patch just moves PSCI > handler calls from `traps.c` to `vsmc.c`. > > PSCI is considered as two different "services" in terms of SMCCC. > Older PSCI 1.0

[Xen-devel] [xtf test] 111230: regressions - trouble: broken/pass

2017-06-30 Thread osstest service owner
flight 111230 xtf real [real] http://logs.test-lab.xenproject.org/osstest/logs/111230/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-xtf-amd64-amd64-1 59 leak-check/check fail REGR. vs. 111074 test-xtf-amd64-amd64-4

[Xen-devel] [linux-4.1 test] 111195: tolerable FAIL - PUSHED

2017-06-30 Thread osstest service owner
flight 95 linux-4.1 real [real] http://logs.test-lab.xenproject.org/osstest/logs/95/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-armhf-armhf-xl-rtds16 guest-start/debian.repeat fail REGR. vs. 110472 Tests which did not succeed, b

Re: [Xen-devel] [PATCH v3 16/16] xen/arm: lpae: Switch from bool_t to bool

2017-06-30 Thread Julien Grall
On Fri, 30 Jun 2017, 21:12 Stefano Stabellini, wrote: > On Fri, 30 Jun 2017, Julien Grall wrote: > > We are phasing out the use of bool_t in the hypervisor code. > > This is missing your signed-off-by. > > Reviewed-by: Stefano Stabellini > Doh. Signed-off-by: Julien Grall Shall I resend the

Re: [Xen-devel] [PATCH v3 00/16] xen/arm: Clean-up memory subsystems

2017-06-30 Thread Stefano Stabellini
I committed the first 15 patches (all but the last) On Fri, 30 Jun 2017, Julien Grall wrote: > Hi all, > > This series, clean-up a bit the memory subsystems to ease further changes. > > Cheers, > > Julien Grall (16): > xen/arm: setup: Remove bogus xenheap_mfn_end in setup_mm for arm64 > xen

[Xen-devel] [RFC] DOMCTL_memattrs_op : a new DOMCTL to play with stage-2 page attributes

2017-06-30 Thread Zhongze Liu
DOMCTL_memattrs_op : a new DOMCTL to play with stage-2 page attributes Zhongze Liu

Re: [Xen-devel] [PATCH v3 16/16] xen/arm: lpae: Switch from bool_t to bool

2017-06-30 Thread Stefano Stabellini
On Fri, 30 Jun 2017, Julien Grall wrote: > We are phasing out the use of bool_t in the hypervisor code. This is missing your signed-off-by. Reviewed-by: Stefano Stabellini > --- > Changes in v3: > - Patch added > --- > xen/include/asm-arm/lpae.h | 6 +++--- > 1 file changed, 3 ins

Re: [Xen-devel] [PATCH v3 03/16] xen/arm: p2m: Redefine mfn_to_page and page_to_mfn to use typesafe

2017-06-30 Thread Stefano Stabellini
On Fri, 30 Jun 2017, Julien Grall wrote: > The file xen/arch/arm/p2m.c is using typesafe MFN in most of the place. > This requires caller to mfn_to_page and page_to_mfn to use _mfn/mfn_x. > > To avoid extra _mfn/mfn_x, re-define mfn_to_page and page_to_mfn within > xen/arch/arm/p2m.c to handle typ

Re: [Xen-devel] [PATCH v3 01/16] xen/arm: setup: Remove bogus xenheap_mfn_end in setup_mm for arm64

2017-06-30 Thread Stefano Stabellini
On Fri, 30 Jun 2017, Julien Grall wrote: > xenheap_mfn_end is storing an MFN and not a physical address. The value will > be reset after the loop. So drop this bogus xenheap_mfn_end. > > Signed-off-by: Julien Grall > --- > Changes in v2: > - Update commit message > > Changes in v

[Xen-devel] [xen-unstable-smoke test] 111237: tolerable trouble: broken/pass - PUSHED

2017-06-30 Thread osstest service owner
flight 111237 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/111237/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-arm64-arm64-xl-xsm 1 build-check(1) blocked n/a test-amd64-amd64-libvirt 13 mig

Re: [Xen-devel] [PATCH 10/18] x86/monitor.c: use plain bool

2017-06-30 Thread Razvan Cojocaru
On 06/30/2017 08:45 PM, Andrew Cooper wrote: > On 30/06/17 18:01, Wei Liu wrote: >> Signed-off-by: Wei Liu > > This file falls under introspection maintainership, so CC'ing them (not > that this change in controversial). > > Reviewed-by: Andrew Cooper Acked-by: Razvan Cojocaru Thanks, Razva

Re: [Xen-devel] [PATCH 00/18] x86: more bool_t to bool cleanup

2017-06-30 Thread Andrew Cooper
On 30/06/17 18:01, Wei Liu wrote: > Seeing that bool_t keeps creeping back in new patches I think the only > solution > is to get rid of bool_t once and for all, as soon as possible. Patches: 4-13, 16-18 Reviewed-by: Andrew Cooper Patch 1 still hasn't arrived in my mailbox yet, so review in sh

Re: [Xen-devel] [OSSTEST PATCH v11 16/20] ts-openstack-tempest: Update list of skipped tests

2017-06-30 Thread Anthony PERARD
On Fri, Jun 30, 2017 at 05:32:56PM +0100, Ian Jackson wrote: > Anthony PERARD writes ("Re: [OSSTEST PATCH v11 16/20] ts-openstack-tempest: > Update list of skipped tests"): > > But it can be told to print subunit v2, which can be pipped to a different > > program to transform into want we want, in

[Xen-devel] [xen-unstable-smoke test] 111234: tolerable trouble: broken/pass - PUSHED

2017-06-30 Thread osstest service owner
flight 111234 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/111234/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-arm64-arm64-xl-xsm 1 build-check(1) blocked n/a test-amd64-amd64-libvirt 13 mig

Re: [Xen-devel] [PATCH 15/18] x86/mpparse.c: use plain bool

2017-06-30 Thread Andrew Cooper
On 30/06/17 18:01, Wei Liu wrote: > @@ -781,10 +781,7 @@ void __init mp_register_lapic_address ( > } > > > -int mp_register_lapic ( > - u32 id, > - bool_t enabled, > - bool_t hotplug) > +int mp_register_lapic(u32 id, bool enable

Re: [Xen-devel] [PATCH 14/18] x86/io_apic.c: use plain bool

2017-06-30 Thread Andrew Cooper
On 30/06/17 18:01, Wei Liu wrote: > @@ -229,7 +229,7 @@ union entry_union { > }; > > struct IO_APIC_route_entry __ioapic_read_entry( > -unsigned int apic, unsigned int pin, bool_t raw) > +unsigned int apic, unsigned int pin, bool raw) Prototype. > { > unsigned int (*read)(unsign

[Xen-devel] [PATCH 01/18] x86/acpi: use plain bool

2017-06-30 Thread Wei Liu
Signed-off-by: Wei Liu --- xen/arch/x86/acpi/boot.c| 26 +- xen/arch/x86/acpi/cpu_idle.c| 19 ++- xen/arch/x86/acpi/cpufreq/cpufreq.c | 2 +- xen/include/asm-x86/acpi.h | 5 ++--- xen/include/asm-x86/cpuidle.h | 2 +- x

Re: [Xen-devel] [PATCH 10/18] x86/monitor.c: use plain bool

2017-06-30 Thread Andrew Cooper
On 30/06/17 18:01, Wei Liu wrote: > Signed-off-by: Wei Liu This file falls under introspection maintainership, so CC'ing them (not that this change in controversial). Reviewed-by: Andrew Cooper ___ Xen-devel mailing list Xen-devel@lists.xen.org https

Re: [Xen-devel] [PATCH 03/18] x86/debug.c: use plain bool

2017-06-30 Thread Andrew Cooper
On 30/06/17 18:01, Wei Liu wrote: > Signed-off-by: Wei Liu > --- > xen/arch/x86/debug.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/xen/arch/x86/debug.c b/xen/arch/x86/debug.c > index 2070077ec5..a5d85a92f9 100644 > --- a/xen/arch/x86/debug.c > +++ b/xen/arch/x86/

Re: [Xen-devel] [PATCH 02/18] x86/apic.c: use plain bool

2017-06-30 Thread Andrew Cooper
On 30/06/17 18:01, Wei Liu wrote: > Signed-off-by: Wei Liu > --- > xen/arch/x86/apic.c| 30 +++--- > xen/include/asm-x86/apic.h | 4 ++-- > 2 files changed, 17 insertions(+), 17 deletions(-) > > diff --git a/xen/arch/x86/apic.c b/xen/arch/x86/apic.c > index 8c6c2f

Re: [Xen-devel] offtopic: handling patches

2017-06-30 Thread Andrew Cooper
On 30/06/17 17:57, Marek Marczykowski-Górecki wrote: > Hi, > > How you guys handle patches with emails? I know git am and git > format-patch/send-email, but those tools are quite limited, especially > when handling patch series, subsequent versions etc. > What I miss there: > - patch versioning (g

[Xen-devel] [PATCH 10/18] x86/monitor.c: use plain bool

2017-06-30 Thread Wei Liu
Signed-off-by: Wei Liu --- xen/arch/x86/monitor.c| 20 ++-- xen/include/asm-x86/monitor.h | 2 +- 2 files changed, 11 insertions(+), 11 deletions(-) diff --git a/xen/arch/x86/monitor.c b/xen/arch/x86/monitor.c index 764195a7d1..706454f23c 100644 --- a/xen/arch/x86/monito

[Xen-devel] [PATCH 17/18] x86/msi.c: use plain bool

2017-06-30 Thread Wei Liu
Signed-off-by: Wei Liu --- xen/arch/x86/msi.c| 32 xen/include/asm-x86/msi.h | 6 +++--- 2 files changed, 19 insertions(+), 19 deletions(-) diff --git a/xen/arch/x86/msi.c b/xen/arch/x86/msi.c index d98f400699..77998f4fb3 100644 --- a/xen/arch/x86/msi.c

[Xen-devel] [PATCH 13/18] x86/smpboot.c: use plain bool

2017-06-30 Thread Wei Liu
Signed-off-by: Wei Liu --- xen/arch/x86/smpboot.c | 5 +++-- xen/arch/x86/time.c| 2 +- xen/include/asm-x86/time.h | 2 +- 3 files changed, 5 insertions(+), 4 deletions(-) diff --git a/xen/arch/x86/smpboot.c b/xen/arch/x86/smpboot.c index f375eb682d..8d91f6ca01 100644 --- a/xen/arch/

[Xen-devel] [PATCH 14/18] x86/io_apic.c: use plain bool

2017-06-30 Thread Wei Liu
Signed-off-by: Wei Liu --- xen/arch/x86/apic.c | 8 xen/arch/x86/io_apic.c| 34 +- xen/include/asm-x86/io_apic.h | 6 +++--- 3 files changed, 24 insertions(+), 24 deletions(-) diff --git a/xen/arch/x86/apic.c b/xen/arch/x86/apic.c inde

[Xen-devel] [PATCH 16/18] x86/numa.c: use plain bool

2017-06-30 Thread Wei Liu
Signed-off-by: Wei Liu --- xen/arch/x86/numa.c| 10 +- xen/include/asm-x86/numa.h | 2 +- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/xen/arch/x86/numa.c b/xen/arch/x86/numa.c index 6f4d438690..d45196fafc 100644 --- a/xen/arch/x86/numa.c +++ b/xen/arch/x86/numa

[Xen-devel] [PATCH 12/18] x86/srat.c: use plain bool

2017-06-30 Thread Wei Liu
Signed-off-by: Wei Liu --- xen/arch/x86/srat.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/xen/arch/x86/srat.c b/xen/arch/x86/srat.c index d86783eddd..cd1283e58c 100644 --- a/xen/arch/x86/srat.c +++ b/xen/arch/x86/srat.c @@ -41,7 +41,7 @@ static struct node node

[Xen-devel] [PATCH 18/18] x86/psr.c: use plain bool

2017-06-30 Thread Wei Liu
Signed-off-by: Wei Liu --- xen/arch/x86/psr.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/xen/arch/x86/psr.c b/xen/arch/x86/psr.c index 0b5073c41f..381f9a046d 100644 --- a/xen/arch/x86/psr.c +++ b/xen/arch/x86/psr.c @@ -270,7 +270,7 @@ static struct psr_cat_

[Xen-devel] [PATCH 15/18] x86/mpparse.c: use plain bool

2017-06-30 Thread Wei Liu
Signed-off-by: Wei Liu --- xen/arch/x86/acpi/boot.c | 4 ++-- xen/arch/x86/genapic/bigsmp.c | 4 ++-- xen/arch/x86/io_apic.c| 2 +- xen/arch/x86/mpparse.c| 25 +++-- xen/include/asm-x86/mpspec.h | 6 +++--- 5 files changed, 19 insertions(+), 22 deleti

[Xen-devel] [PATCH 11/18] x86/xstate.c: use plain bool

2017-06-30 Thread Wei Liu
Signed-off-by: Wei Liu --- xen/arch/x86/xstate.c| 30 +++--- xen/include/asm-x86/xstate.h | 6 +++--- 2 files changed, 18 insertions(+), 18 deletions(-) diff --git a/xen/arch/x86/xstate.c b/xen/arch/x86/xstate.c index c2a722c60e..845208c189 100644 --- a/xen/arch/

Re: [Xen-devel] [PATCH v3 4/8] osstest: add a FreeBSD host install recipe

2017-06-30 Thread Roger Pau Monne
On Fri, Jun 23, 2017 at 03:45:45PM +0100, Ian Jackson wrote: > Roger Pau Monne writes ("[PATCH v3 4/8] osstest: add a FreeBSD host install > recipe"): > > +sub setup_netboot_installer () { > > +my $image = "$path_prefix/install.img"; > > +my $pxeimg = target_tftp_prefix($ho) . "--freebsd.i

Re: [Xen-devel] [PATCH] mini-os: link to libgcc.a to fix build with gcc7

2017-06-30 Thread Olaf Hering
On Fri, Jun 30, Olaf Hering wrote: > On Fri, Jun 30, Wei Liu wrote: > > > On second thought I don't think we want to make this worse. So the > > change in this patch should be conditional on gcc. > > How whould I check for gcc in the Makefile? > In xen.git I see a conditional for clang. I dont h

[Xen-devel] [PATCH 00/18] x86: more bool_t to bool cleanup

2017-06-30 Thread Wei Liu
Seeing that bool_t keeps creeping back in new patches I think the only solution is to get rid of bool_t once and for all, as soon as possible. Wei Liu (18): x86/acpi: use plain bool x86/apic.c: use plain bool x86/debug.c: use plain bool x86/dmi.c: use plain bool x86/domctl: use plain boo

[Xen-devel] [PATCH 07/18] x86/e820.c: use plan bool

2017-06-30 Thread Wei Liu
Note that e820_mtrr_clip remains s8 although the command line parameter is bool, because it is a tristate variable. Signed-off-by: Wei Liu --- xen/arch/x86/e820.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/xen/arch/x86/e820.c b/xen/arch/x86/e820.c index bc1544a514..7c572

[Xen-devel] [PATCH 06/18] x86/hpet.c: use plain bool

2017-06-30 Thread Wei Liu
Signed-off-by: Wei Liu --- xen/arch/x86/hpet.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/xen/arch/x86/hpet.c b/xen/arch/x86/hpet.c index 7e8b438c0f..46f4c428a1 100644 --- a/xen/arch/x86/hpet.c +++ b/xen/arch/x86/hpet.c @@ -58,7 +58,7 @@ u8 __initdata hpet_flags; * if R

[Xen-devel] [PATCH 03/18] x86/debug.c: use plain bool

2017-06-30 Thread Wei Liu
Signed-off-by: Wei Liu --- xen/arch/x86/debug.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/xen/arch/x86/debug.c b/xen/arch/x86/debug.c index 2070077ec5..a5d85a92f9 100644 --- a/xen/arch/x86/debug.c +++ b/xen/arch/x86/debug.c @@ -156,7 +156,7 @@ dbg_pv_va2mfn(dbgva_t v

[Xen-devel] [PATCH 04/18] x86/dmi.c: use plain bool

2017-06-30 Thread Wei Liu
Signed-off-by: Wei Liu --- xen/arch/x86/dmi_scan.c | 8 xen/include/xen/dmi.h | 2 +- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/xen/arch/x86/dmi_scan.c b/xen/arch/x86/dmi_scan.c index 8564a77ea0..fcdf2d3952 100644 --- a/xen/arch/x86/dmi_scan.c +++ b/xen/arch/x86/dm

[Xen-devel] [PATCH 08/18] x86/i387.c: use plain bool

2017-06-30 Thread Wei Liu
Signed-off-by: Wei Liu --- xen/arch/x86/i387.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/xen/arch/x86/i387.c b/xen/arch/x86/i387.c index 9a172db5dc..88452522ad 100644 --- a/xen/arch/x86/i387.c +++ b/xen/arch/x86/i387.c @@ -22,7 +22,7 @@ /* Restore x87 extende

[Xen-devel] [PATCH 05/18] x86/domctl: use plain bool

2017-06-30 Thread Wei Liu
Signed-off-by: Wei Liu --- xen/arch/x86/domctl.c | 38 +++--- 1 file changed, 19 insertions(+), 19 deletions(-) diff --git a/xen/arch/x86/domctl.c b/xen/arch/x86/domctl.c index f40e989fd8..7fa58b49af 100644 --- a/xen/arch/x86/domctl.c +++ b/xen/arch/x86/domctl.c @

[Xen-devel] [PATCH 09/18] x86/i8259.c: use plain bool

2017-06-30 Thread Wei Liu
Signed-off-by: Wei Liu --- xen/arch/x86/i8259.c | 10 +- xen/include/asm-x86/irq.h | 2 +- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/xen/arch/x86/i8259.c b/xen/arch/x86/i8259.c index 07465aace8..166ca1a3e1 100644 --- a/xen/arch/x86/i8259.c +++ b/xen/arch/x86/i82

[Xen-devel] [xen-4.9-testing test] 111196: tolerable FAIL - PUSHED

2017-06-30 Thread osstest service owner
flight 96 xen-4.9-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/96/ Failures :-/ but no regressions. Tests which are failing intermittently (not blocking): test-amd64-amd64-xl-qemuu-win7-amd64 16 guest-localmigrate/x10 fail in 62 pass in 96 test-armhf-arm

[Xen-devel] [PATCH 02/18] x86/apic.c: use plain bool

2017-06-30 Thread Wei Liu
Signed-off-by: Wei Liu --- xen/arch/x86/apic.c| 30 +++--- xen/include/asm-x86/apic.h | 4 ++-- 2 files changed, 17 insertions(+), 17 deletions(-) diff --git a/xen/arch/x86/apic.c b/xen/arch/x86/apic.c index 8c6c2f5819..28a1ef8a58 100644 --- a/xen/arch/x86/apic.c

Re: [Xen-devel] [PATCH] mini-os: link to libgcc.a to fix build with gcc7

2017-06-30 Thread Olaf Hering
On Fri, Jun 30, Wei Liu wrote: > On second thought I don't think we want to make this worse. So the > change in this patch should be conditional on gcc. How whould I check for gcc in the Makefile? In xen.git I see a conditional for clang. I dont have a clang at hand, perhaps it knows about "-prin

[Xen-devel] offtopic: handling patches

2017-06-30 Thread Marek Marczykowski-Górecki
Hi, How you guys handle patches with emails? I know git am and git format-patch/send-email, but those tools are quite limited, especially when handling patch series, subsequent versions etc. What I miss there: - patch versioning (git notes could be used, but it doesn't survive git commit --ame

Re: [Xen-devel] [OSSTEST PATCH v11 20/20] Introduce flight for stable branches of OpenStack

2017-06-30 Thread Anthony PERARD
On Fri, Jun 23, 2017 at 06:00:29PM +0100, Ian Jackson wrote: > Anthony PERARD writes ("[OSSTEST PATCH v11 20/20] Introduce flight for stable > branches of OpenStack"): > > OpenStack have many different repo which should be in sync, so this > > patch should grab the revisions of the stable branch o

Re: [Xen-devel] [OSSTEST PATCH v11 16/20] ts-openstack-tempest: Update list of skipped tests

2017-06-30 Thread Ian Jackson
Anthony PERARD writes ("Re: [OSSTEST PATCH v11 16/20] ts-openstack-tempest: Update list of skipped tests"): > On Fri, Jun 23, 2017 at 04:38:05PM +0100, Ian Jackson wrote: > > Aha. Well, if the stable branch is stable then the set of tests to > > skip there is probably stable too ? And on master

[Xen-devel] [OSSTEST PATCH 08/17] step logfiles: Standalone mode: Do not print uninformative $script value

2017-06-30 Thread Ian Jackson
Right now, this is sometimes wrong; even if it were right, it's not going to be interesting in the transcript since the actual script name is obvious from context. Now this function does not care if $script is undef. While we're here, fix the prototype (which is mostly commentary here, as it is n

[Xen-devel] [OSSTEST PATCH 06/17] db updates: Make all INSERTs (except into steps) name columns explicitly

2017-06-30 Thread Ian Jackson
This makes the code more robust against schema updates. Specifically, if any columns were to be deleted, these runes would break. (Also if any column was inserted other than after the existing columns, although this is not something Postgres can currently do AIUI.) Affected tables are: flights

[Xen-devel] [OSSTEST PATCH 02/17] schema updates: Fix typo in developer doc

2017-06-30 Thread Ian Jackson
Signed-off-by: Ian Jackson --- schema/README.updates | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/schema/README.updates b/schema/README.updates index 25bc11a..c5abc36 100644 --- a/schema/README.updates +++ b/schema/README.updates @@ -109,7 +109,7 @@ on it. There are typica

[Xen-devel] [OSSTEST PATCH 15/17] step logfiles: sg-run-job: Break out step-log-leafname

2017-06-30 Thread Ian Jackson
No functional change. Signed-off-by: Ian Jackson --- tcl/JobDB-Executive.tcl | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/tcl/JobDB-Executive.tcl b/tcl/JobDB-Executive.tcl index 414a77e..ed636e4 100644 --- a/tcl/JobDB-Executive.tcl +++ b/tcl/JobDB-Executive.tcl @@ -43

[Xen-devel] [OSSTEST PATCH 13/17] step logfiles: Honour new logfile column

2017-06-30 Thread Ian Jackson
There are the following places in the tree where we construct the logfile name from stepno and step: * tcl/JobDB-Executivemain decision about logfile name } OK * JobDB::Executive final fallback for new steps } keep * steps-logfile-constraint.sql default for old

[Xen-devel] [OSSTEST PATCH 07/17] step logfiles: Break out $logfilename in two places

2017-06-30 Thread Ian Jackson
No functional change. Signed-off-by: Ian Jackson --- determine-failure-reasons | 4 ++-- sg-report-flight | 3 ++- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/determine-failure-reasons b/determine-failure-reasons index ecff332..a3b2100 100755 --- a/determine-failure-r

[Xen-devel] [OSSTEST PATCH 14/17] step logfiles: Make substep_* take $logfile, not $script

2017-06-30 Thread Ian Jackson
The $script parameter was used by Standalone to print a message, but this is no longer done. It is used by Executive as the value for the "step" column in the steps table, which shows up as "script" in the output from sg-report-flight. But, when running under sg-run-job, JobDB::Executive already

[Xen-devel] [OSSTEST PATCH 17/17] step logfiles: Rely on new logfile column

2017-06-30 Thread Ian Jackson
These are the three places which read logfile from the db, but had a fallback to a computation involving stepno and step. But, after steps-logfile-constraint, the logfile column is NOT NULL (and populated with computed default data if need be) so this is not needed any more This is part of a "Pop

[Xen-devel] [OSSTEST PATCH 11/17] substep logfiles: Populate new logfile column, at all, for substeps

2017-06-30 Thread Ian Jackson
For now we populate it with the sometimes-wrong value based on $script. In detail: * Add a parameter $logfile to $mjobdb->step_start, which currently always gets passed undef. * In JobDB::Executive::step_start, try to use it as the value for the new logfile column. But, if it is not set (wh

[Xen-devel] [OSSTEST PATCH 05/17] schema updates: Be willing to apply Ready changes

2017-06-30 Thread Ian Jackson
If the oldest code declares itself Ready for this change, or (implausibly) Needed, then it is OK to push. Signed-off-by: Ian Jackson --- mg-schema-update | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/mg-schema-update b/mg-schema-update index 43f313a..44e4283 100755 --- a/mg

[Xen-devel] [OSSTEST PATCH 16/17] step logfiles: Populate logfile column with logfile name

2017-06-30 Thread Ian Jackson
This is part of a "Populate-then-rely" schema change, as described in schema/README.updates. Specifically, this is the final part of the step: 5. Commit: code to populate new column; changing `add' to status Needed and `constraint' to status Ready. Signed-off-by: Ian Jackson --- schema/step

[Xen-devel] [OSSTEST PATCH 10/17] step logfiles: Provide new column

2017-06-30 Thread Ian Jackson
We will write things into it, and use it, soon. This is the start of a "Populate-then-rely" schema change, as described in schema/README.updates. Specifically, the steps: 1. Commit: new schema update `add', status Preparatory 2. Commit: new schema update `constraint', status Unfinished Signed-of

[Xen-devel] [OSSTEST PATCH 00/17] Fix substep (eg XTF) logfile links

2017-06-30 Thread Ian Jackson
It kept irritating me that the links to the substeps of xtf-run do not work. To fix this, it is necessary for the database to contain the right information - ie, a schema change. After this series, substep logfile names are recorded and reported properly, and you can click on them. At some point

Re: [Xen-devel] [PATCH 2/2] tools: utility to dump guest grant table info

2017-06-30 Thread Konrad Rzeszutek Wilk
On Fri, Jun 30, 2017 at 03:21:31PM +0800, Dongli Zhang wrote: > As both xen-netfront and xen-blkfront support multi-queue, they would > consume a lot of grant table references when there are many paravirtual > devices and vcpus assigned to guest. Guest domU might panic or hang due to > grant alloca

[Xen-devel] [OSSTEST PATCH 09/17] substep logfiles: Move defaulting of $script into Executive

2017-06-30 Thread Ian Jackson
We are going to want to default this here in a more sophisticated way, but we can't do that if it's already been defaulted from $0 earlier. The other consumer of this default was in JobDB/Standalone, but we have just changed that to ignore it. So, no functional change. Signed-off-by: Ian Jackson

[Xen-devel] [OSSTEST PATCH 12/17] substep logfiles: Set logfile and step columns correctly for substeps

2017-06-30 Thread Ian Jackson
If we are running under sg-run-job, OSSTEST_TESTID will be set to the testid of the parent step (ie, the step created by sg-run-job which relates to the execution of the whole script). Unless the caller has specified otherwise, the log and script are the same as for the parent step. Signed-off-by

[Xen-devel] [OSSTEST PATCH 01/17] Developer docs: A rune for flight comparision

2017-06-30 Thread Ian Jackson
Something like this rune can be useful for comparing testids of different flights. Signed-off-by: Ian Jackson --- NOTES.flight-comparison | 22 ++ 1 file changed, 22 insertions(+) create mode 100644 NOTES.flight-comparison diff --git a/NOTES.flight-comparison b/NOTES.flight

[Xen-devel] [OSSTEST PATCH 03/17] schema updates: Document how to remove a used column

2017-06-30 Thread Ian Jackson
Signed-off-by: Ian Jackson --- schema/README.updates | 26 ++ 1 file changed, 26 insertions(+) diff --git a/schema/README.updates b/schema/README.updates index c5abc36..ca1d37e 100644 --- a/schema/README.updates +++ b/schema/README.updates @@ -147,6 +147,32 @@ finish is n

[Xen-devel] [OSSTEST PATCH 04/17] schema updates: mg-schema-test-database: Cope with applied updates

2017-06-30 Thread Ian Jackson
The schema_updates table contents are written by the call to mg-schema-update. Trying to copy the contents from the source db correctly fails with duplicate key errors. So don't do that. Signed-off-by: Ian Jackson --- mg-schema-test-database | 3 +++ 1 file changed, 3 insertions(+) diff --git

Re: [Xen-devel] [OSSTEST PATCH v11 16/20] ts-openstack-tempest: Update list of skipped tests

2017-06-30 Thread Anthony PERARD
On Fri, Jun 23, 2017 at 04:38:05PM +0100, Ian Jackson wrote: > Anthony PERARD writes ("Re: [OSSTEST PATCH v11 16/20] ts-openstack-tempest: > Update list of skipped tests"): > > On Fri, Jun 23, 2017 at 02:47:37PM +0100, Ian Jackson wrote: > > > Again, does this not mean we're going to suffer a main

[Xen-devel] [PATCH 2/4] xen/mapcache: add an ability to create dummy mappings

2017-06-30 Thread Igor Druzhinin
Dummys are simple anonymous mappings that are placed instead of regular foreign mappings in certain situations when we need to postpone the actual mapping but still have to give a memory region to QEMU to play with. This is planned to be used for restore on Xen. Signed-off-by: Igor Druzhinin ---

[Xen-devel] [PATCH 4/4] xen: don't use xenstore to save/restore physmap anymore

2017-06-30 Thread Igor Druzhinin
If we have a system with xenforeignmemory_map2() implemented we don't need to save/restore physmap on suspend/restore anymore. In case we resume a VM without physmap - try to recreate the physmap during memory region restore phase and remap map cache entries accordingly. The old code is left for co

[Xen-devel] [PATCH 3/4] xen/mapcache: introduce xen_remap_cache_entry()

2017-06-30 Thread Igor Druzhinin
This new call is trying to update a requested map cache entry according to the changes in the physmap. The call is searching for the entry, unmaps it, tries to translate the address and maps again at the same place. If the mapping is dummy this call will make it real. This function makes use of a

[Xen-devel] [PATCH 1/4] xen: move physmap saving into a separate function

2017-06-30 Thread Igor Druzhinin
Non-functional change. Signed-off-by: Igor Druzhinin --- hw/i386/xen/xen-hvm.c | 57 --- 1 file changed, 31 insertions(+), 26 deletions(-) diff --git a/hw/i386/xen/xen-hvm.c b/hw/i386/xen/xen-hvm.c index cffa7e2..d259cf7 100644 --- a/hw/i386/xen/x

[Xen-devel] [PATCH 0/4] xen: don't save/restore the physmap on VM save/restore

2017-06-30 Thread Igor Druzhinin
Saving/restoring the physmap to/from xenstore was introduced to QEMU majorly in order to cover up the VRAM region restore issue. The sequence of restore operations implies that we should know the effective guest VRAM address *before* we have the VRAM region restored (which happens later). Unfortuna

[Xen-devel] [PATCH v3 01/16] xen/arm: setup: Remove bogus xenheap_mfn_end in setup_mm for arm64

2017-06-30 Thread Julien Grall
xenheap_mfn_end is storing an MFN and not a physical address. The value will be reset after the loop. So drop this bogus xenheap_mfn_end. Signed-off-by: Julien Grall --- Changes in v2: - Update commit message Changes in v3: - Don't mention the fact that Xen is not current

[Xen-devel] [PATCH v3 16/16] xen/arm: lpae: Switch from bool_t to bool

2017-06-30 Thread Julien Grall
We are phasing out the use of bool_t in the hypervisor code. --- Changes in v3: - Patch added --- xen/include/asm-arm/lpae.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/xen/include/asm-arm/lpae.h b/xen/include/asm-arm/lpae.h index 6fbf7c606c..a62b118630 1

[Xen-devel] [PATCH v3 08/16] xen/arm: create_xen_entries: Use typesafe MFN

2017-06-30 Thread Julien Grall
Add a bit more safety when using create_xen_entries. Also when destroying/modifying mapping, the MFN is currently not used. Rather than passing _mfn(0) use INVALID_MFN to stay consistent with the other usage. Signed-off-by: Julien Grall Reviewed-by: Stefano Stabellini --- Changes in v2:

[Xen-devel] [PATCH v3 10/16] xen/arm: lpae: Fix comments coding style

2017-06-30 Thread Julien Grall
Also adding one missing full stop + fix description Signed-off-by: Julien Grall Reviewed-by: Stefano Stabellini --- Cc: prosku...@sec.in.tum.de I haven't retained Stefano's reviewed-by because of the description update. Changes in v2: - Fix description regarding x86 page-ta

[Xen-devel] [PATCH v3 03/16] xen/arm: p2m: Redefine mfn_to_page and page_to_mfn to use typesafe

2017-06-30 Thread Julien Grall
The file xen/arch/arm/p2m.c is using typesafe MFN in most of the place. This requires caller to mfn_to_page and page_to_mfn to use _mfn/mfn_x. To avoid extra _mfn/mfn_x, re-define mfn_to_page and page_to_mfn within xen/arch/arm/p2m.c to handle typesafe MFN. Signed-off-by: Julien Grall --- Th

[Xen-devel] [PATCH v3 00/16] xen/arm: Clean-up memory subsystems

2017-06-30 Thread Julien Grall
Hi all, This series, clean-up a bit the memory subsystems to ease further changes. Cheers, Julien Grall (16): xen/arm: setup: Remove bogus xenheap_mfn_end in setup_mm for arm64 xen/arm: mm: Use typesafe mfn for xenheap_mfn_* xen/arm: p2m: Redefine mfn_to_page and page_to_mfn to use typesaf

[Xen-devel] [PATCH v3 13/16] xen/arm: mm: Use lpae_valid and lpae_table in create_xen_entries

2017-06-30 Thread Julien Grall
This newly introduced lpae_valid and lpae_table helpers will recude the code and make more readable. Signed-off-by: Julien Grall Reviewed-by: Stefano Stabellini --- Changes in v2: - Patch added Changes in v3: - Add Stefano's reviewed-by --- xen/arch/arm/mm.c | 9 --

[Xen-devel] [PATCH v3 12/16] xen/arm: p2m: Move lpae_* helpers in lpae.h

2017-06-30 Thread Julien Grall
lpae_* helpers can work on any LPAE translation tables. Move them in lpae.h to allow other part of Xen to use them. Signed-off-by: Julien Grall Reviewed-by: Stefano Stabellini --- Cc: prosku...@sec.in.tum.de Changes in v2: - Patch added Changes in v3: - Add Stefano's r

[Xen-devel] [PATCH v3 09/16] xen/arm: Move LPAE definition in a separate header

2017-06-30 Thread Julien Grall
page.h is getting bigger. Move out every LPAE definitions in a separate header. There is no functional changes. Signed-off-by: Julien Grall Reviewed-by: Stefano Stabellini --- Cc: prosku...@sec.in.tum.de Changes in v2: - Move comment after the #endif rather than before - A

[Xen-devel] [PATCH v3 06/16] xen/arm: alternative: Redefine virt_to_mfn to support typesafe

2017-06-30 Thread Julien Grall
The file xen/arch/arm/alternative.c is using typesafe MFN in most of the place. The only caller to virt_to_mfn is using with _mfn(...). To avoid extra _mfn(...), re-define virt_to_mfn within xen/arch/arm/alternative.c to handle typesafe MFN. Signed-off-by: Julien Grall Reviewed-by: Stefano Stabe

[Xen-devel] [PATCH v3 14/16] xen/arm: mm: Introduce temporary variable in create_xen_entries

2017-06-30 Thread Julien Grall
This is improving the code readability and avoid to dereference the table every single time we need to access the entry. Signed-off-by: Julien Grall Reviewed-by: Stefano Stabellini --- Changes in v2: - Patch added Changes in v3: - Add Stefano's reviewed-by --- xen/arch/

[Xen-devel] [PATCH v3 11/16] xen/arm: p2m: Rename p2m_valid, p2m_table, p2m_mapping and p2m_is_superpage

2017-06-30 Thread Julien Grall
The helpers p2m_valid, p2m_table, p2m_mapping and p2m_is_superpage are not specific to the stage-2 translation tables. They can also work on any LPAE translation tables. So rename then to lpae_* and use pte.walk to look for the value of the field. Signed-off-by: Julien Grall Reviewed-by: Stefano

  1   2   3   >