[xen-unstable-smoke test] 181948: regressions - FAIL

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

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

Regressions :-(

Tests which did not succeed and are blocking,
including tests which could not be run:
 build-arm64-xsm               6 xen-build                fail REGR. vs. 181941

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     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                  0c53c638e16278078371ce028c74693841d7738a
baseline version:
 xen                  1ab2ae1610d99423af5b810829959431e43de12d

Last test of basis   181941  2023-07-20 23:03:34 Z    0 days
Testing same since   181948  2023-07-21 07:00:24 Z    0 days    1 attempts

------------------------------------------------------------
People who touched revisions under test:
  Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
  Federico Serafini <federico.serafini@xxxxxxxxxxx>
  Jan Beulich <jbeulich@xxxxxxxx>
  Juergen Gross <jgross@xxxxxxxx>
  Julien Grall <jgrall@xxxxxxxxxx>
  Nicola Vetrini <nicola.vetrini@xxxxxxxxxxx>
  Roger Pau Monné <roger.pau@xxxxxxxxxx>

jobs:
 build-arm64-xsm                                              fail    
 build-amd64                                                  pass    
 build-armhf                                                  pass    
 build-amd64-libvirt                                          pass    
 test-armhf-armhf-xl                                          pass    
 test-arm64-arm64-xl-xsm                                      blocked 
 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 0c53c638e16278078371ce028c74693841d7738a
Author: Juergen Gross <jgross@xxxxxxxx>
Date:   Fri Jul 21 08:32:43 2023 +0200

    tools/xenstore: fix XSA-417 patch
    
    The fix for XSA-417 had a bug: domain_alloc_permrefs() will not return
    a negative value in case of an error, but a plain errno value.
    
    Note this is not considered to be a security issue, as the only case
    where domain_alloc_permrefs() will return an error is a failed memory
    allocation. As a guest should not be able to drive Xenstore out of
    memory, this is NOT a problem a guest can trigger at will.
    
    Fixes: ab128218225d ("tools/xenstore: fix checking node permissions")
    Signed-off-by: Juergen Gross <jgross@xxxxxxxx>
    Acked-by: Julien Grall <jgrall@xxxxxxxxxx>

commit 65fc6d8c92bc339d8ed874c5821e0e65101f6c52
Author: Federico Serafini <federico.serafini@xxxxxxxxxxx>
Date:   Fri Jul 21 08:32:15 2023 +0200

    x86/mtrr: address violations of MISRA C:2012 Rule 8.3 on parameter types
    
    Change parameter types of function declarations to be consistent with
    the ones used in the corresponding definitions,
    thus addressing violations of MISRA C:2012 Rule 8.3 ("All declarations
    of an object or function shall use the same names and type qualifiers").
    
    No functional changes.
    
    Signed-off-by: Federico Serafini <federico.serafini@xxxxxxxxxxx>
    Acked-by: Roger Pau Monné <roger.pau@xxxxxxxxxx>

commit 652d8fbb8182f0b9a57ba371c24d21e0a981ee07
Author: Federico Serafini <federico.serafini@xxxxxxxxxxx>
Date:   Fri Jul 21 08:31:42 2023 +0200

    x86/HVM: address violations of MISRA C:2012 Rules 8.2 and 8.3
    
    Give a name to unnamed parameters thus addressing violations of
    MISRA C:2012 Rule 8.2 ("Function types shall be in prototype form with
    named parameters").
    Keep consistency between parameter names and types used in function
    declarations and the ones used in the corresponding function
    definitions, thus addressing violations of MISRA C:2012 Rule 8.3
    ("All declarations of an object or function shall use the same names
    and type qualifiers").
    
    No functional changes.
    
    Signed-off-by: Federico Serafini <federico.serafini@xxxxxxxxxxx>
    Acked-by: Jan Beulich <jbeulich@xxxxxxxx>

commit 60576722d0523122ce34c81b91fd1eafac8f7f5f
Author: Jan Beulich <jbeulich@xxxxxxxx>
Date:   Fri Jul 21 08:31:09 2023 +0200

    x86/vRTC: move and tidy convert_hour() and {to,from}_bcd()
    
    This is to avoid the need for forward declarations, which in turn
    addresses a violation of MISRA C:2012 Rule 8.3 ("All declarations of an
    object or function shall use the same names and type qualifiers").
    
    While doing so,
    - drop inline (leaving the decision to the compiler),
    - add const,
    - add unsigned,
    - correct style.
    
    Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
    Acked-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
    Reviewed-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>

commit cd5048353725f37746da231202feaf5b88ceb46a
Author: Nicola Vetrini <nicola.vetrini@xxxxxxxxxxx>
Date:   Fri Jul 21 08:30:29 2023 +0200

    xen: address MISRA C:2012 Rule 4.1
    
    MISRA C:2012 Rule 4.1 has the following headline:
    "Octal and hexadecimal escape sequences shall be terminated."
    
    The string literals modified by this patch contain octal or
    hexadecimal escape sequences that are neither terminated by the
    end of the literal, nor by the beginning of another escape sequence.
    
    Therefore, such unterminated sequences have been split into a
    separate literal as a way to comply with the rule and preserve the
    semantics of the code.
    
    No functional changes.
    
    Signed-off-by: Nicola Vetrini <nicola.vetrini@xxxxxxxxxxx>
    Reviewed-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>
    Acked-by: Jan Beulich <jbeulich@xxxxxxxx>
(qemu changes not included)