[Xen-devel] [seabios baseline-only test] 38601: tolerable FAIL

[Platform Team regression test user <citrix-osstest@xxxxxxxxxxxxxx>]

This run is configured for baseline tests only.

flight 38601 seabios real [real]
http://osstest.xs.citrite.net/~osstest/testlogs/logs/38601/

Failures :-/ but no regressions.

Regressions which are regarded as allowable (not blocking):
 test-amd64-amd64-qemuu-nested-intel 13 xen-boot/l1             fail like 38597
 test-amd64-amd64-xl-qemuu-winxpsp3  9 windows-install          fail like 38597

Tests which did not succeed, but are not blocking:
 test-amd64-amd64-qemuu-nested-amd 16 debian-hvm-install/l1/l2  fail never pass
 test-amd64-amd64-libvirt-qemuu-debianhvm-amd64-xsm 10 migrate-support-check 
fail never pass
 test-amd64-i386-libvirt-qemuu-debianhvm-amd64-xsm 10 migrate-support-check 
fail never pass
 test-amd64-i386-xl-qemuu-win7-amd64 16 guest-stop              fail never pass
 test-amd64-amd64-xl-qemuu-win7-amd64 16 guest-stop             fail never pass

version targeted for testing:
 seabios              16a9e7926a6c6845a98df2a6eac509c23c6206ba
baseline version:
 seabios              71479612401b794e6cb5025f61e5352c51f35877

Last test of basis    38597  2016-01-06 17:24:58 Z    1 days
Testing same since    38601  2016-01-07 20:56:04 Z    0 days    1 attempts

------------------------------------------------------------
People who touched revisions under test:
  Kevin O'Connor <kevin@xxxxxxxxxxxx>

jobs:
 build-amd64-xsm                                              pass    
 build-i386-xsm                                               pass    
 build-amd64                                                  pass    
 build-i386                                                   pass    
 build-amd64-libvirt                                          pass    
 build-i386-libvirt                                           pass    
 build-amd64-pvops                                            pass    
 build-i386-pvops                                             pass    
 test-amd64-amd64-libvirt-qemuu-debianhvm-amd64-xsm           pass    
 test-amd64-i386-libvirt-qemuu-debianhvm-amd64-xsm            pass    
 test-amd64-amd64-xl-qemuu-debianhvm-amd64-xsm                pass    
 test-amd64-i386-xl-qemuu-debianhvm-amd64-xsm                 pass    
 test-amd64-amd64-qemuu-nested-amd                            fail    
 test-amd64-i386-qemuu-rhel6hvm-amd                           pass    
 test-amd64-amd64-xl-qemuu-debianhvm-amd64                    pass    
 test-amd64-i386-xl-qemuu-debianhvm-amd64                     pass    
 test-amd64-amd64-xl-qemuu-ovmf-amd64                         pass    
 test-amd64-i386-xl-qemuu-ovmf-amd64                          pass    
 test-amd64-amd64-xl-qemuu-win7-amd64                         fail    
 test-amd64-i386-xl-qemuu-win7-amd64                          fail    
 test-amd64-amd64-qemuu-nested-intel                          fail    
 test-amd64-i386-qemuu-rhel6hvm-intel                         pass    
 test-amd64-i386-xl-qemuu-winxpsp3-vcpus1                     pass    
 test-amd64-amd64-xl-qemuu-winxpsp3                           fail    
 test-amd64-i386-xl-qemuu-winxpsp3                            pass    


------------------------------------------------------------
sg-report-flight on osstest.xs.citrite.net
logs: /home/osstest/logs
images: /home/osstest/images

Logs, config files, etc. are available at
    http://osstest.xs.citrite.net/~osstest/testlogs/logs

Test harness code can be found at
    http://xenbits.xensource.com/gitweb?p=osstest.git;a=summary


Push not applicable.

------------------------------------------------------------
commit 16a9e7926a6c6845a98df2a6eac509c23c6206ba
Author: Kevin O'Connor <kevin@xxxxxxxxxxxx>
Date:   Tue Dec 29 23:04:15 2015 -0500

    tpm: Don't use 16bit BIOS return codes in TPM menu functions
    
    Don't use the return codes from the 16bit BIOS spec in the internal
    menu functions.  Only the 16bit BIOS interface code should need to
    handle the details of that spec.  For functions that need to return
    the TIS command status, return those codes directly instead of via a
    pointer parameter.
    
    Signed-off-by: Kevin O'Connor <kevin@xxxxxxxxxxxx>

commit b8631eaeb3b2216371de028c76abf17186ac84ab
Author: Kevin O'Connor <kevin@xxxxxxxxxxxx>
Date:   Wed Dec 30 12:51:27 2015 -0500

    tpm: Don't use 16bit BIOS return codes in tpmhw_* functions
    
    Don't use the return codes from the 16bit BIOS spec in the internal
    tpmhw functions.  Only the 16bit BIOS interface code should need to
    handle the details of that spec.
    
    Signed-off-by: Kevin O'Connor <kevin@xxxxxxxxxxxx>

commit 9ddea3b018fcb2e0d8d49a7e6c3c36763d4e93e0
Author: Kevin O'Connor <kevin@xxxxxxxxxxxx>
Date:   Wed Dec 30 12:40:11 2015 -0500

    tpm: Don't use 16bit BIOS return codes in tpm_log_event()
    
    Don't use the return codes from the 16bit BIOS spec in the internal
    tpm_log_event() and tpm_log_extend_event() functions.  Only the 16bit
    BIOS interface code should need to handle the details of that spec.
    
    Signed-off-by: Kevin O'Connor <kevin@xxxxxxxxxxxx>

commit cac29f2504450a97e05a95fa6e76cdb9199eb879
Author: Kevin O'Connor <kevin@xxxxxxxxxxxx>
Date:   Tue Dec 29 17:54:37 2015 -0500

    tpm: Don't use 16bit BIOS return codes in build_and_send_cmd()
    
    Don't use the return codes from the 16bit BIOS spec in the internal
    function build_and_send_cmd().  Instead, return the TIS command status
    code of the command or -1 if there was a command transmission failure.
    This eliminates the need for a returnCode pointer parameter.
    
    Also, implement debugging dprintf() in build_and_send_cmd() instead of
    in every caller.  This replaces the command name with the integer
    command id, but it does make the debugging more consistent.
    
    Signed-off-by: Kevin O'Connor <kevin@xxxxxxxxxxxx>

commit 5ffcb2c32b3decc463890c5a2355c410b3fca80a
Author: Kevin O'Connor <kevin@xxxxxxxxxxxx>
Date:   Wed Dec 30 00:48:57 2015 -0500

    tpm: Don't call tpm_set_failure() from tpm_log_extend_event()
    
    The 16bit BIOS interface shouldn't be able to shutdown the TPM.  Move
    the check for tpm_is_working() and tpm_set_failure() to the only
    caller of tpm_log_extend_event() that may shutdown the TPM.
    
    Signed-off-by: Kevin O'Connor <kevin@xxxxxxxxxxxx>

commit ed8d55d87b395c147cfe4689f536ebf6760c2a45
Author: Kevin O'Connor <kevin@xxxxxxxxxxxx>
Date:   Wed Dec 30 00:15:10 2015 -0500

    tpm: No need to check the return status of measurements
    
    The low-level measurement functions already handle error conditions,
    there is no need to check for the errors in the high level measurement
    functions.
    
    Signed-off-by: Kevin O'Connor <kevin@xxxxxxxxxxxx>

commit e98fafaa5e68d4fa65f872e522d2f8da10f68363
Author: Kevin O'Connor <kevin@xxxxxxxxxxxx>
Date:   Tue Dec 29 23:14:22 2015 -0500

    tpm: Don't return a status from external bios measurement functions
    
    The callers of the measurements don't care what happens, so no need to
    return a status.
    
    Signed-off-by: Kevin O'Connor <kevin@xxxxxxxxxxxx>

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel