flight 180321 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/180321/
Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: build-armhf <job status> broken build-armhf 2 hosts-allocate broken REGR. vs. 180302 Tests which did not succeed, but are not blocking: test-armhf-armhf-xl 1 build-check(1) blocked n/a test-amd64-amd64-libvirt 15 migrate-support-check fail never pass test-arm64-arm64-xl-xsm 15 migrate-support-check fail never pass test-arm64-arm64-xl-xsm 16 saverestore-support-check fail never pass version targeted for testing: xen e74360e4ba4a6b6827a44f8b1b22a0ec4311694a baseline version: xen 8676092a0f16ca6ad188d3fb270784a2caecf542 Last test of basis 180302 2023-04-18 20:01:55 Z 1 days Failing since 180314 2023-04-19 10:00:24 Z 0 days 4 attempts Testing same since 180321 2023-04-19 18:01:59 Z 0 days 1 attempts ------------------------------------------------------------ People who touched revisions under test: Andrew Cooper <[email protected]> Henry Wang <[email protected]> Jan Beulich <[email protected]> Roger Pau Monné <[email protected]> jobs: build-arm64-xsm pass build-amd64 pass build-armhf broken build-amd64-libvirt pass test-armhf-armhf-xl blocked test-arm64-arm64-xl-xsm pass 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 broken-job build-armhf broken broken-step build-armhf hosts-allocate Not pushing. ------------------------------------------------------------ commit e74360e4ba4a6b6827a44f8b1b22a0ec4311694a Author: Andrew Cooper <[email protected]> Date: Thu Apr 13 20:56:15 2023 +0100 xen/livepatch: Fix .altinstructions safety checks The prior check has && vs || mixups, making it tautologically false and thus providing no safety at all. There are boundary errors too. First start with a comment describing how the .altinstructions and .altinstr_replacement sections interact, and perform suitable cross-checking. Second, rewrite the alt_instr loop entirely from scratch. Origin sites have non-zero size, and must be fully contained within the livepatches .text section(s). Any non-zero sized replacements must be fully contained within the .altinstr_replacement section. Fixes: f8a10174e8b1 ("xsplice: Add support for alternatives") Signed-off-by: Andrew Cooper <[email protected]> Reviewed-by: Ross Lagerwall <[email protected]> commit 418cf59c4e29451010d7efb3835b900690d19866 Author: Andrew Cooper <[email protected]> Date: Sun Apr 16 01:10:43 2023 +0100 arm/alternatives: Rename alt_instr fields which are used in common code Alternatives auditing for livepatches is currently broken. To fix it, the livepatch code needs to inspect more fields of alt_instr. Rename ARM's fields to match x86's, because: * ARM already exposes alt_offset under the repl name via ALT_REPL_PTR(). * "alt" is ambiguous in a structure entirely about alternatives already. * "repl", being the same width as orig leads to slightly neater code. No functional change. Signed-off-by: Andrew Cooper <[email protected]> Reviewed-by: Stefano Stabellini <[email protected]> commit cfa2bb82c01f0c656804cedd8f44eb2a99a2b5bc Author: Andrew Cooper <[email protected]> Date: Sun Apr 16 01:29:09 2023 +0100 xen/ELF: Fix ELF32 PRI formatters It is rude to hide width formatting inside a PRI* macro, doubly so when it's only in one bitness of the macro. However its fully buggy when all the users use %#"PRI because then it expands to the common trap of %#08x which does not do what the author intends. Switch the 32bit ELF PRI formatters to use plain integer PRI's, just like on the 64bit side already. No practical change. Fixes: 7597fabca76e ("livepatch: Include sizes when an mismatch occurs") Fixes: 380b229634f8 ("xsplice: Implement payload loading") Signed-off-by: Andrew Cooper <[email protected]> Acked-by: Jan Beulich <[email protected]> Reviewed-by: Ross Lagerwall <[email protected]> commit 569632a5832c02bd84790e0411940b8d3150fa17 Author: Roger Pau Monné <[email protected]> Date: Wed Apr 19 11:03:30 2023 +0200 CHANGELOG: add gnttab_max_{maptrack_,}frames option changes Note in the changelog that the purpose of gnttab_max_{maptrack_,}frames command line options has been changed. Fixes: b2ea81d2b935 ('xen/grants: repurpose command line max options') Signed-off-by: Roger Pau Monné <[email protected]> Acked-by: Henry Wang <[email protected]> commit 768846690d64bc730c1a1123e8de3af731bb2eb3 Author: Jan Beulich <[email protected]> Date: Wed Apr 19 11:02:47 2023 +0200 x86: fix build with old gcc after CPU policy changes Old gcc won't cope with initializers involving unnamed struct/union fields. Fixes: 441b1b2a50ea ("x86/emul: Switch x86_emulate_ctxt to cpu_policy") Signed-off-by: Jan Beulich <[email protected]> Acked-by: Andrew Cooper <[email protected]> commit 741599fa521fbbb4cf71a98d7ec22ba5f4671cfa Author: Jan Beulich <[email protected]> Date: Wed Apr 19 11:01:29 2023 +0200 x86: cpu{id,}_policy_updated() can be static The function merely needs moving earlier in the file to avoid the need for a forward declaration. While moving it, also rename it following the recent folding of CPUID and MSR policies. Signed-off-by: Jan Beulich <[email protected]> Acked-by: Roger Pau Monné <[email protected]> commit 224211c55bdded74c5a65f5a7e34281a8c5c56f2 Author: Jan Beulich <[email protected]> Date: Wed Apr 19 11:00:19 2023 +0200 tests/cpu-policy: fix "run" goal An earlier change converted TARGET-y to TARGETS, but failed to replace all references. Convert run's dependency, but use $< in the command to avoid the leading blank that += inserts. Fixes: 6a9f5477637a ("tests/cpu-policy: Rework Makefile") Signed-off-by: Jan Beulich <[email protected]> Acked-by: Roger Pau Monné <[email protected]> (qemu changes not included)
