Mailing List Archive

[xen-unstable-smoke test] 158488: regressions - FAIL
flight 158488 xen-unstable-smoke real [real]
flight 158490 xen-unstable-smoke real-retest [real]
http://logs.test-lab.xenproject.org/osstest/logs/158488/
http://logs.test-lab.xenproject.org/osstest/logs/158490/

Regressions :-(

Tests which did not succeed and are blocking,
including tests which could not be run:
test-arm64-arm64-xl-xsm 8 xen-boot fail REGR. vs. 158453

Tests which did not succeed, but are not blocking:
test-amd64-amd64-libvirt 15 migrate-support-check fail never pass
test-armhf-armhf-xl 15 migrate-support-check fail never pass
test-armhf-armhf-xl 16 saverestore-support-check fail never pass

version targeted for testing:
xen b066d57183548ae199acdf6a0022e012e324a7b2
baseline version:
xen db9c4ad1b1abaef3c38027b9b2700d9250d13125

Last test of basis 158453 2021-01-15 23:00:25 Z 2 days
Testing same since 158488 2021-01-18 12:02:57 Z 0 days 1 attempts

------------------------------------------------------------
People who touched revisions under test:
Andrew Cooper <andrew.cooper3@citrix.com>
Jan Beulich <jbeulich@suse.com>

jobs:
build-arm64-xsm pass
build-amd64 pass
build-armhf pass
build-amd64-libvirt pass
test-armhf-armhf-xl pass
test-arm64-arm64-xl-xsm fail
test-amd64-amd64-xl-qemuu-debianhvm-amd64 pass
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 b066d57183548ae199acdf6a0022e012e324a7b2
Author: Jan Beulich <jbeulich@suse.com>
Date: Mon Jan 18 12:14:19 2021 +0100

gnttab: consolidate pin-to-status syncing

Forever since the fix for XSA-230 the 2nd of the comments ahead of
fixup_status_for_copy_pin() has been stale - there's nothing specific to
transitive grants there anymore.

Move the function up, drop the "copy" part from its name again, add a
"readonly" parameter, and use it also on other paths having decremented
one (or not having got to increment any) of the pin counts.

Signed-off-by: Jan Beulich <jbeulich@suse.com>
Reviewed-by: Andrew Cooper <andrew.cooper3@citrix.com>

commit 8cec5d065c243e54cae97aaa14b0de447883c278
Author: Jan Beulich <jbeulich@suse.com>
Date: Mon Jan 18 12:13:42 2021 +0100

gnttab: adjust pin count overflow checks

It's at least odd to check counters which aren't going to be
incremented, resulting in failure just because prior operations may have
reached the refcount limit. And it's also not helpful to use open-coded
literal numbers in these checks.

Calculate the increment values first and derive from them the mask to
use in the checks.

Also move the pin count checks ahead of the calculation of the status
(and for copy also sha2) pointers: They're not needed in the failure
cases, and this way the compiler may also have an easier time keeping
the variables at least transiently in registers for the subsequent uses.

Signed-off-by: Jan Beulich <jbeulich@suse.com>
Reviewed-by: Andrew Cooper <andrew.cooper3@citrix.com>

commit d6627cf1b63ce57a6a7e2c1800dbc50eed742c32
Author: Jan Beulich <jbeulich@suse.com>
Date: Mon Jan 18 12:12:23 2021 +0100

x86/Dom0: support zstd compressed kernels

Taken from Linux at commit 1c4dd334df3a ("lib: decompress_unzstd: Limit
output size") for unzstd.c (renamed from decompress_unzstd.c) and
36f9ff9e03de ("lib: Fix fall-through warnings for Clang") for zstd/,
with bits from linux/zstd.h merged into suitable other headers.

To limit the editing necessary, introduce ptrdiff_t.

Signed-off-by: Jan Beulich <jbeulich@suse.com>
Acked-by: Andrew Cooper <andrew.cooper3@citrix.com>

commit 35d2960ae65f28106fdc5c2130f5f08fadca0e4c
Author: Jan Beulich <jbeulich@suse.com>
Date: Mon Jan 18 12:10:34 2021 +0100

lib: introduce xxhash

Taken from Linux at commit d89775fc929c ("lib/: replace HTTP links with
HTTPS ones"), but split into separate 32-bit and 64-bit sources, since
the immediate consumer (zstd) will need only the latter.

Note that the building of this code is restricted to x86 for now because
of the need to sort asm/unaligned.h for Arm.

Signed-off-by: Jan Beulich <jbeulich@suse.com>
Acked-by: Andrew Cooper <andrew.cooper3@citrix.com>

commit 7c9f81687ad611515474b1c17afc2f79f19faef5
Author: Jan Beulich <jbeulich@suse.com>
Date: Mon Jan 18 12:09:13 2021 +0100

introduce unaligned.h

Rather than open-coding commonly used constructs in yet more places when
pulling in zstd decompression support (and its xxhash prereq), pull out
the custom bits into a commonly used header (for the hypervisor build;
the tool stack and stubdom builds of libxenguest will still remain in
need of similarly taking care of). For now this is limited to x86, where
custom logic isn't needed (considering this is going to be used in init
code only, even using alternatives patching to use MOVBE doesn't seem
worthwhile).

For Arm64 with CONFIG_ACPI=y (due to efi-dom0.c's re-use of xz/crc32.c)
drop the not really necessary inclusion of xz's private.h.

No change in generated code.

Signed-off-by: Jan Beulich <jbeulich@suse.com>
Acked-by: Andrew Cooper <andrew.cooper3@citrix.com>
(qemu changes not included)
Re: [xen-unstable-smoke test] 158488: regressions - FAIL [ In reply to ]
On 18.01.2021 16:43, osstest service owner wrote:
> flight 158488 xen-unstable-smoke real [real]
> flight 158490 xen-unstable-smoke real-retest [real]
> http://logs.test-lab.xenproject.org/osstest/logs/158488/
> http://logs.test-lab.xenproject.org/osstest/logs/158490/
>
> Regressions :-(
>
> Tests which did not succeed and are blocking,
> including tests which could not be run:
> test-arm64-arm64-xl-xsm 8 xen-boot fail REGR. vs. 158453

Hmm, I've looked some at the log, but not being used to looking
at Arm logs I'm not sure whether some of the messages are
anomalies. What likely is indicative of the problem are

systemd-udevd[452]: e0010000.kcs: Worker [464] processing SEQNUM=3352 is taking a long time

rcu: INFO: rcu_preempt detected stalls on CPUs/tasks:

I'm having trouble though connecting such to the changes under
test. If anyone has any helpful thoughts, please share.

Jan