* [xen-unstable-smoke test] 99722: regressions - FAIL
@ 2016-07-27 19:51 osstest service owner
2016-07-27 20:06 ` Andrew Cooper
0 siblings, 1 reply; 3+ messages in thread
From: osstest service owner @ 2016-07-27 19:51 UTC (permalink / raw)
To: xen-devel, osstest-admin
[-- Attachment #1: Type: text/plain, Size: 12716 bytes --]
flight 99722 xen-unstable-smoke real [real]
http://logs.test-lab.xenproject.org/osstest/logs/99722/
Regressions :-(
Tests which did not succeed and are blocking,
including tests which could not be run:
test-amd64-amd64-xl-qemuu-debianhvm-i386 12 guest-saverestore fail REGR. vs. 99707
Tests which did not succeed, but are not blocking:
test-amd64-amd64-libvirt 12 migrate-support-check fail never pass
test-armhf-armhf-xl 12 migrate-support-check fail never pass
test-armhf-armhf-xl 13 saverestore-support-check fail never pass
version targeted for testing:
xen 7179cd39efdb22ac847ae465d1aa11cd6263f19b
baseline version:
xen d5438accceecc8172db2d37d98b695eb8bc43afc
Last test of basis 99707 2016-07-26 10:01:43 Z 1 days
Testing same since 99722 2016-07-27 18:01:50 Z 0 days 1 attempts
------------------------------------------------------------
People who touched revisions under test:
Andrew Cooper <andrew.cooper3@citrix.com>
Juergen Gross <jgross@suse.com>
Julien Grall <julien.grall@arm.com>
Shanker Donthineni <shankerd@codeaurora.org>
Stefano Stabellini <sstabellini@kernel.org>
Tamas K Lengyel <tamas.lengyel@zentific.com>
Wei Liu <wei.liu2@citrix.com>
jobs:
build-amd64 pass
build-armhf pass
build-amd64-libvirt pass
test-armhf-armhf-xl pass
test-amd64-amd64-xl-qemuu-debianhvm-i386 fail
test-amd64-amd64-libvirt pass
------------------------------------------------------------
sg-report-flight on osstest.test-lab.xenproject.org
logs: /home/logs/logs
images: /home/logs/images
Logs, config files, etc. are available at
http://logs.test-lab.xenproject.org/osstest/logs
Explanation of these reports, and of osstest in general, is at
http://xenbits.xen.org/gitweb/?p=osstest.git;a=blob;f=README.email;hb=master
http://xenbits.xen.org/gitweb/?p=osstest.git;a=blob;f=README;hb=master
Test harness code can be found at
http://xenbits.xen.org/gitweb?p=osstest.git;a=summary
Not pushing.
------------------------------------------------------------
commit 7179cd39efdb22ac847ae465d1aa11cd6263f19b
Author: Andrew Cooper <andrew.cooper3@citrix.com>
Date: Fri Jul 15 16:43:48 2016 +0100
xen/domctl: Add DOMINFO_hap to xen_domctl_getdomaininfo
This allows a toolstack to identify whether a running domain is using hardware
assisted paging or not.
The appropriate tests differ by architecture, so introduce
arch_get_domain_info(). ARM unconditionally sets the new flag, while x86
checks with the paging subsystem first.
Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
Acked-by: Wei Liu <wei.liu2@citrix.com>
Reviewed-by: Julien Grall <julien.grall@arm.com>
Reviewed-by: George Dunlap <george.dunlap@citrix.com>
commit d2412fd63b14c6c21d0a3d4367afa448425dfb8a
Author: Juergen Gross <jgross@suse.com>
Date: Tue Jul 12 17:30:44 2016 +0200
libxl: move common nic stuff into one source
Put all nic related stuff of libxl form common files into a dedicated
source file.
Signed-off-by: Juergen Gross <jgross@suse.com>
Acked-by: Wei Liu <wei.liu2@citrix.com>
commit 9f3fbaff6cccdab9122e8c987fa5694f0930d7f4
Author: Juergen Gross <jgross@suse.com>
Date: Tue Jul 12 17:30:43 2016 +0200
libxl: add config update callback to device type framework
Some device types require a configuration update after resume of
domain. Add a callback for this purpose.
Signed-off-by: Juergen Gross <jgross@suse.com>
Acked-by: Wei Liu <wei.liu2@citrix.com>
commit 20609ad8ee914c6e040bd64dac11e61aa5186ceb
Author: Juergen Gross <jgross@suse.com>
Date: Tue Jul 12 17:30:42 2016 +0200
libxl: split libxl vtpm code into one source
Put all vtpm related stuff of libxl into a dedicated source file.
Signed-off-by: Juergen Gross <jgross@suse.com>
Acked-by: Wei Liu <wei.liu2@citrix.com>
commit 803d4d332243fc2aa87c63a0ed871197252d23d1
Author: Juergen Gross <jgross@suse.com>
Date: Tue Jul 12 17:30:41 2016 +0200
libxl: move library pvusb specific code into libxl_pvusb.c
Outside libxl_pvusb.c only libxl_util.c still contains some pvusb code.
Move it to libxl_pvusb.c.
Signed-off-by: Juergen Gross <jgross@suse.com>
Acked-by: Wei Liu <wei.liu2@citrix.com>
commit e320815ea5a4ba538210d91c8527b9eabeb15333
Author: Juergen Gross <jgross@suse.com>
Date: Tue Jul 12 17:30:40 2016 +0200
libxl: add "pv device mode needed" support to device type framework
Add another callback to the device type framework in order to aid
decision whether a pv domain needs a device model.
Signed-off-by: Juergen Gross <jgross@suse.com>
Acked-by: Wei Liu <wei.liu2@citrix.com>
commit de0b58c574b0bafb13585a94815049398a1fd9fe
Author: Juergen Gross <jgross@suse.com>
Date: Tue Jul 12 17:30:39 2016 +0200
libxl: add "merge" function to generic device type support
Instead of using a macro generating the code to merge xenstore and
json configuration data, use the generic device type support for
this purpose.
This requires to add some accessor functions to the framework and
a structure for disks (as disks are added separately they didn't need
such a structure up to now).
Signed-off-by: Juergen Gross <jgross@suse.com>
Acked-by: Wei Liu <wei.liu2@citrix.com>
commit 653948018a6f2e7c4fe314329a9886c6b8388f96
Author: Tamas K Lengyel <tamas.lengyel@zentific.com>
Date: Wed Jul 27 10:31:59 2016 +0100
altp2m: Allow shared entries to be copied to altp2m views during lazycopy
Move sharing locks above altp2m to avoid locking order violation and crashing
the hypervisor during unsharing operations when altp2m is active.
Applying mem_access settings or remapping gfns in altp2m views will
automatically unshare the page if it was shared previously. Also,
disallow nominating pages for which there are pre-existing altp2m
mem_access settings or remappings present. However, allow altp2m to
populate altp2m views with shared entries during lazycopy as unsharing
will automatically propagate the change to these entries in altp2m
views as well.
While we're here, switch to using the appropriate wrappers rather than
calling p2m->get_entry() directly.
Signed-off-by: Tamas K Lengyel <tamas.lengyel@zentific.com>
Reviewed-by: George Dunlap <george.dunlap@citrix.com>
commit a97bedb2120eb5b277eb5a9cca85f4e659c17cbe
Author: Julien Grall <julien.grall@arm.com>
Date: Wed Jul 20 17:10:50 2016 +0100
xen/arm: p2m: Simplify p2m type check by using bitmask
The resulting assembly code for the macros is much simpler and will
never contain more than one instruction branch.
The idea is taken from x86 (see include/asm-x86/p2m.h). Also move the
two helpers earlier to keep all the p2m type definitions together.
Signed-off-by: Julien Grall <julien.grall@arm.com>
Reviewed-by: Stefano Stabellini <sstabellini@kernel.org>
commit 8cfe8bcf33cf371f3f1cbf7029d0bc1e2fa12769
Author: Julien Grall <julien.grall@arm.com>
Date: Wed Jul 20 17:10:49 2016 +0100
xen/arm: p2m: Use p2m_is_foreign in get_page_from_gfn to avoid open coding
No functional change.
Signed-off-by: Julien Grall <julien.grall@arm.com>
Reviewed-by: Stefano Stabellini <sstabellini@kernel.org>
commit 4b11387906fdcc3ad66de1038699b9293144ab8b
Author: Julien Grall <julien.grall@arm.com>
Date: Wed Jul 20 17:10:47 2016 +0100
xen/arm: p2m: Clean-up mfn_to_p2m_entry
The physical address is computed from the machine frame number, so
checking if the physical address is page aligned is pointless.
Furthermore, directly assigned the MFN to the corresponding field in the
entry rather than converting to a physical address and orring the value.
It will avoid to rely on the field position and make the code clearer.
Signed-off-by: Julien Grall <julien.grall@arm.com>
Reviewed-by: Stefano Stabellini <sstabellini@kernel.org>
commit 96d1be3788805257690a58dbea65d4d6b7ac30dc
Author: Shanker Donthineni <shankerd@codeaurora.org>
Date: Wed Jul 20 09:00:56 2016 -0500
arm/vgic: Change fixed number of mmio handlers to variable number
Compute the number of mmio handlers that are required for vGICv3 and
vGICv2 emulation drivers in vgic_v3_init()/vgic_v2_init(). Augment
this variable number of mmio handlers to a fixed number MAX_IO_HANDLER
and pass it to domain_io_init() to allocate enough memory.
New code path:
domain_vgic_register(&count)
domain_io_init(count + MAX_IO_HANDLER)
domain_vgic_init()
Signed-off-by: Shanker Donthineni <shankerd@codeaurora.org>
Acked-by: Julien Grall <julien.grall@arm.com>
commit 8047e090f42a39bda911f6b2e81fdab12603b64a
Author: Shanker Donthineni <shankerd@codeaurora.org>
Date: Wed Jul 20 09:00:55 2016 -0500
xen/arm: io: Use binary search for mmio handler lookup
As the number of I/O handlers increase, the overhead associated with
linear lookup also increases. The system might have maximum of 144
(assuming CONFIG_NR_CPUS=128) mmio handlers. In worst case scenario,
it would require 144 iterations for finding a matching handler. Now
it is time for us to change from linear (complexity O(n)) to a binary
search (complexity O(log n) for reducing mmio handler lookup overhead.
Signed-off-by: Shanker Donthineni <shankerd@codeaurora.org>
Acked-by: Julien Grall <julien.grall@arm.com>
Acked-by: Stefano Stabellini <sstabellini@kernel.org>
commit e3eb84e33c365ce0af1a75ef25ec36cf646c958b
Author: Shanker Donthineni <shankerd@codeaurora.org>
Date: Wed Jul 20 09:00:54 2016 -0500
xen: Add generic implementation of binary search
This patch adds the generic implementation of binary search algorithm
which is copied from Linux kernel v4.7-rc7. No functional changes.
Signed-off-by: Shanker Donthineni <shankerd@codeaurora.org>
Reviewed-by: Andrew Cooper <andrew.cooper3@citrix.com>
Reviewed-by: Julien Grall <julien.grall@arm.com>
commit 96a1eeef79194b04877f1e3f4fcef3f155fbd393
Author: Shanker Donthineni <shankerd@codeaurora.org>
Date: Wed Jul 20 09:00:53 2016 -0500
arm/io: Use separate memory allocation for mmio handlers
The number of mmio handlers are limited to a compile time macro
MAX_IO_HANDLER which is 16. This number is not at all sufficient
to support per CPU distributor regions. Either it needs to be
increased to a bigger number, at least CONFIG_NR_CPUS+16, or
allocate a separate memory for mmio handlers dynamically during
domain build.
This patch uses the dynamic allocation strategy to reduce memory
footprint for 'struct domain' instead of static allocation.
Signed-off-by: Shanker Donthineni <shankerd@codeaurora.org>
Acked-by: Julien Grall <julien.grall@arm.com>
Acked-by: Stefano Stabellini <sstabellini@kernel.org>
commit 9f1441487aa215193a7c00fd9cb80b335542465e
Author: Andrew Cooper <andrew.cooper3@citrix.com>
Date: Wed Jun 15 18:32:14 2016 +0100
x86/entry: Avoid SMAP violation in compat_create_bounce_frame()
A 32bit guest kernel might be running on user mappings.
compat_create_bounce_frame() must whitelist its guest accesses to avoid
risking a SMAP violation.
For both variants of create_bounce_frame(), re-blacklist user accesses if
execution exits via an exception table redirection.
This is XSA-183 / CVE-2016-6259
Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
Reviewed-by: George Dunlap <george.dunlap@citrix.com>
Reviewed-by: Jan Beulich <jbeulich@suse.com>
commit e1bff4c2ea3b32464510ac00c320bba28a8dbcca
Author: Andrew Cooper <andrew.cooper3@citrix.com>
Date: Mon Jul 11 14:32:03 2016 +0100
x86/pv: Remove unsafe bits from the mod_l?_entry() fastpath
All changes in writeability and cacheability must go through full
re-validation.
Rework the logic as a whitelist, to make it clearer to follow.
This is XSA-182
Reported-by: Jérémie Boutoille <jboutoille@ext.quarkslab.com>
Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
Reviewed-by: Tim Deegan <tim@xen.org>
(qemu changes not included)
[-- Attachment #2: Type: text/plain, Size: 127 bytes --]
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [xen-unstable-smoke test] 99722: regressions - FAIL
2016-07-27 19:51 [xen-unstable-smoke test] 99722: regressions - FAIL osstest service owner
@ 2016-07-27 20:06 ` Andrew Cooper
2016-07-28 7:29 ` Juergen Gross
0 siblings, 1 reply; 3+ messages in thread
From: Andrew Cooper @ 2016-07-27 20:06 UTC (permalink / raw)
To: osstest service owner, xen-devel; +Cc: Juergen Gross, Ian Jackson, Wei Liu
On 27/07/2016 20:51, osstest service owner wrote:
> flight 99722 xen-unstable-smoke real [real]
> http://logs.test-lab.xenproject.org/osstest/logs/99722/
>
> Regressions :-(
>
> Tests which did not succeed and are blocking,
> including tests which could not be run:
> test-amd64-amd64-xl-qemuu-debianhvm-i386 12 guest-saverestore fail REGR. vs. 99707
This is :
*** Error in `xl': double free or corruption (fasttop): 0x000000000108e560 ***
from the commit range:
* 7179cd3 - xen/domctl: Add DOMINFO_hap to xen_domctl_getdomaininfo (5
hours ago) <Andrew Cooper>
* d2412fd - libxl: move common nic stuff into one source (8 hours ago)
<Juergen Gross>
* 9f3fbaf - libxl: add config update callback to device type framework
(8 hours ago) <Juergen Gross>
* 20609ad - libxl: split libxl vtpm code into one source (8 hours ago)
<Juergen Gross>
* 803d4d3 - libxl: move library pvusb specific code into libxl_pvusb.c
(8 hours ago) <Juergen Gross>
* e320815 - libxl: add "pv device mode needed" support to device type
framework (8 hours ago) <Juergen Gross>
* de0b58c - libxl: add "merge" function to generic device type support
(8 hours ago) <Juergen Gross>
* 6539480 - altp2m: Allow shared entries to be copied to altp2m views
during lazycopy (10 hours ago) <Tamas K Lengyel>
* a97bedb - xen/arm: p2m: Simplify p2m type check by using bitmask (21
hours ago) <Julien Grall>
* 8cfe8bc - xen/arm: p2m: Use p2m_is_foreign in get_page_from_gfn to
avoid open coding (21 hours ago) <Julien Grall>
* 4b11387 - xen/arm: p2m: Clean-up mfn_to_p2m_entry (21 hours ago)
<Julien Grall>
* 96d1be3 - arm/vgic: Change fixed number of mmio handlers to variable
number (22 hours ago) <Shanker Donthineni>
* 8047e09 - xen/arm: io: Use binary search for mmio handler lookup (22
hours ago) <Shanker Donthineni>
* e3eb84e - xen: Add generic implementation of binary search (22 hours
ago) <Shanker Donthineni>
* 96a1eee - arm/io: Use separate memory allocation for mmio handlers (22
hours ago) <Shanker Donthineni>
* 9f14414 - x86/entry: Avoid SMAP violation in
compat_create_bounce_frame() (31 hours ago) <Andrew Cooper>
* e1bff4c - x86/pv: Remove unsafe bits from the mod_l?_entry() fastpath
(31 hours ago) <Andrew Cooper>
Sorry Juergen, but your libxl series looks like the only plausible
candidate.
The XenServer Coverity instance will get around to looking at this code
presently, and might spot something, but I don't have any results yet.
~Andrew
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [xen-unstable-smoke test] 99722: regressions - FAIL
2016-07-27 20:06 ` Andrew Cooper
@ 2016-07-28 7:29 ` Juergen Gross
0 siblings, 0 replies; 3+ messages in thread
From: Juergen Gross @ 2016-07-28 7:29 UTC (permalink / raw)
To: Andrew Cooper, osstest service owner, xen-devel; +Cc: Ian Jackson, Wei Liu
On 27/07/16 22:06, Andrew Cooper wrote:
> On 27/07/2016 20:51, osstest service owner wrote:
>> flight 99722 xen-unstable-smoke real [real]
>> http://logs.test-lab.xenproject.org/osstest/logs/99722/
>>
>> Regressions :-(
>>
>> Tests which did not succeed and are blocking,
>> including tests which could not be run:
>> test-amd64-amd64-xl-qemuu-debianhvm-i386 12 guest-saverestore fail REGR. vs. 99707
>
> This is :
>
> *** Error in `xl': double free or corruption (fasttop): 0x000000000108e560 ***
>
> from the commit range:
>
> * 7179cd3 - xen/domctl: Add DOMINFO_hap to xen_domctl_getdomaininfo (5
> hours ago) <Andrew Cooper>
> * d2412fd - libxl: move common nic stuff into one source (8 hours ago)
> <Juergen Gross>
> * 9f3fbaf - libxl: add config update callback to device type framework
> (8 hours ago) <Juergen Gross>
> * 20609ad - libxl: split libxl vtpm code into one source (8 hours ago)
> <Juergen Gross>
> * 803d4d3 - libxl: move library pvusb specific code into libxl_pvusb.c
> (8 hours ago) <Juergen Gross>
> * e320815 - libxl: add "pv device mode needed" support to device type
> framework (8 hours ago) <Juergen Gross>
> * de0b58c - libxl: add "merge" function to generic device type support
> (8 hours ago) <Juergen Gross>
> * 6539480 - altp2m: Allow shared entries to be copied to altp2m views
> during lazycopy (10 hours ago) <Tamas K Lengyel>
> * a97bedb - xen/arm: p2m: Simplify p2m type check by using bitmask (21
> hours ago) <Julien Grall>
> * 8cfe8bc - xen/arm: p2m: Use p2m_is_foreign in get_page_from_gfn to
> avoid open coding (21 hours ago) <Julien Grall>
> * 4b11387 - xen/arm: p2m: Clean-up mfn_to_p2m_entry (21 hours ago)
> <Julien Grall>
> * 96d1be3 - arm/vgic: Change fixed number of mmio handlers to variable
> number (22 hours ago) <Shanker Donthineni>
> * 8047e09 - xen/arm: io: Use binary search for mmio handler lookup (22
> hours ago) <Shanker Donthineni>
> * e3eb84e - xen: Add generic implementation of binary search (22 hours
> ago) <Shanker Donthineni>
> * 96a1eee - arm/io: Use separate memory allocation for mmio handlers (22
> hours ago) <Shanker Donthineni>
> * 9f14414 - x86/entry: Avoid SMAP violation in
> compat_create_bounce_frame() (31 hours ago) <Andrew Cooper>
> * e1bff4c - x86/pv: Remove unsafe bits from the mod_l?_entry() fastpath
> (31 hours ago) <Andrew Cooper>
>
> Sorry Juergen, but your libxl series looks like the only plausible
> candidate.
Indeed. Correction patch sent out already.
Sorry for the bug...
Juergen
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-07-28 7:29 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-07-27 19:51 [xen-unstable-smoke test] 99722: regressions - FAIL osstest service owner
2016-07-27 20:06 ` Andrew Cooper
2016-07-28 7:29 ` Juergen Gross
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.