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
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
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
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
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
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
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
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
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
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
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
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
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
On Sat, 1 Jul 2017, Zhongze Liu wrote:
>
>DOMCTL_memattrs_op : a new DOMCTL to play with stage-2 page attributes
>
> Zhongze Liu
>
>
> *
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
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
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
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
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
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
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
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
DOMCTL_memattrs_op : a new DOMCTL to play with stage-2 page attributes
Zhongze Liu
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
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
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
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
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
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
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
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
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
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
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
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
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/
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
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
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
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
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/
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
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
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
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_
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
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/
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
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
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
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
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
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
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
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
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
@
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
---
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
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
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
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
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
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
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:
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
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
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
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 --
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
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
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
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/
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 - 100 of 224 matches
Mail list logo