[Xen-devel] [xen-unstable-smoke test] 139051: regressions - FAIL

[osstest service owner <osstest-admin@xxxxxxxxxxxxxx>]

flight 139051 xen-unstable-smoke real [real]
http://logs.test-lab.xenproject.org/osstest/logs/139051/

Regressions :-(

Tests which did not succeed and are blocking,
including tests which could not be run:
 test-amd64-amd64-xl-qemuu-debianhvm-amd64 10 debian-hvm-install fail REGR. vs. 
138907

Tests which did not succeed, but are not blocking:
 test-amd64-amd64-libvirt     13 migrate-support-check        fail   never pass
 test-arm64-arm64-xl-xsm      13 migrate-support-check        fail   never pass
 test-arm64-arm64-xl-xsm      14 saverestore-support-check    fail   never pass
 test-armhf-armhf-xl          13 migrate-support-check        fail   never pass
 test-armhf-armhf-xl          14 saverestore-support-check    fail   never pass

version targeted for testing:
 xen                  ec83f8256276c6f99dfe13bb2b56d168c080f098
baseline version:
 xen                  38eeb3864de40aa568c48f9f26271c141c62b50b

Last test of basis   138907  2019-07-11 15:01:02 Z    4 days
Testing same since   139043  2019-07-16 08:00:43 Z    0 days    2 attempts

------------------------------------------------------------
People who touched revisions under test:
  Jan Beulich <jbeulich@xxxxxxxx>
  Paul Durrant <paul.durrant@xxxxxxxxxx>
  Roger Pau Monné <roger.pau@xxxxxxxxxx>

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                                      pass    
 test-amd64-amd64-xl-qemuu-debianhvm-amd64                    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 ec83f8256276c6f99dfe13bb2b56d168c080f098
Author: Paul Durrant <paul.durrant@xxxxxxxxxx>
Date:   Tue Jul 16 09:10:36 2019 +0200

    mm.h: add helper function to test-and-clear _PGC_allocated
    
    The _PGC_allocated flag is set on a page when it is assigned to a domain
    along with an initial reference count of at least 1. To clear this
    'allocation' reference it is necessary to test-and-clear _PGC_allocated and
    then only drop the reference if the test-and-clear succeeds. This is open-
    coded in many places. It is also unsafe to test-and-clear _PGC_allocated
    unless the caller holds an additional reference.
    
    This patch adds a helper function, put_page_alloc_ref(), to replace all the
    open-coded test-and-clear/put_page occurrences. That helper function
    incorporates a check that an additional page reference is held and will
    BUG() if it is not.
    
    Signed-off-by: Paul Durrant <paul.durrant@xxxxxxxxxx>
    Acked-by: Jan Beulich <jbeulich@xxxxxxxx>

commit 3bfe102c16b0590d4f885250e82a2ef623867cf4
Author: Jan Beulich <jbeulich@xxxxxxxx>
Date:   Tue Jul 16 09:09:44 2019 +0200

    x86/hvm: make hvmemul_virtual_to_linear()'s reps parameter optional
    
    A majority of callers wants just a single iteration handled. Allow to
    express this by passing in a NULL pointer, instead of setting up a local
    variable just to hold the "1" to pass in here.
    
    Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
    Reviewed-by: Alexandru Isaila <aisaila@xxxxxxxxxxxxxxx>
    Reviewed-by: Paul Durrant <paul.durrant@xxxxxxxxxx>

commit ec2ab491b52815c1daedfdf3d95d13cfe25fb38e
Author: Roger Pau Monné <roger.pau@xxxxxxxxxx>
Date:   Tue Jul 16 09:05:28 2019 +0200

    x86/ept: pass correct level to p2m_entry_modify
    
    EPT differs from NPT and shadow when translating page orders to levels
    in the physmap page tables. EPT page tables level for order 0 pages is
    0, while NPT and shadow instead use 1, ie: EPT page tables levels
    starts at 0 while NPT and shadow starts at 1.
    
    Fix the p2m_entry_modify call in atomic_write_ept_entry to always add
    one to the level, in order to match NPT and shadow usage.
    
    While there also add a check to ensure p2m_entry_modify is never
    called with level == 0. That should allow to catch future errors
    related to the level parameter.
    
    Fixes: c7a4c088ad1c ('x86/mm: split p2m ioreq server pages special handling 
into helper')
    Signed-off-by: Roger Pau Monné <roger.pau@xxxxxxxxxx>
    Reviewed-by: Jan Beulich <jbeulich@xxxxxxxx>
    Reviewed-by: Kevin Tian <kevin.tian@xxxxxxxxx>
(qemu changes not included)

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel