[Xen-devel] [ovmf test] 94594: regressions - FAIL

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

flight 94594 ovmf real [real]
http://logs.test-lab.xenproject.org/osstest/logs/94594/

Regressions :-(

Tests which did not succeed and are blocking,
including tests which could not be run:
 build-i386-xsm                5 xen-build                 fail REGR. vs. 94588
 build-amd64                   5 xen-build                 fail REGR. vs. 94588
 build-amd64-xsm               5 xen-build                 fail REGR. vs. 94588
 build-i386                    5 xen-build                 fail REGR. vs. 94588

Tests which did not succeed, but are not blocking:
 build-amd64-libvirt           1 build-check(1)               blocked  n/a
 build-i386-libvirt            1 build-check(1)               blocked  n/a
 test-amd64-i386-xl-qemuu-ovmf-amd64  1 build-check(1)              blocked n/a
 test-amd64-amd64-xl-qemuu-ovmf-amd64  1 build-check(1)             blocked n/a

version targeted for testing:
 ovmf                 6340f2b8587e85fd094261065831546c84b27926
baseline version:
 ovmf                 1f1ec99dea4d6d04fed96fa8a2e299212f6bc8cb

Last test of basis    94588  2016-05-20 01:14:27 Z    0 days
Failing since         94590  2016-05-20 03:46:31 Z    0 days    2 attempts
Testing same since    94594  2016-05-20 05:43:46 Z    0 days    1 attempts

------------------------------------------------------------
People who touched revisions under test:
  Fu Siyuan <siyuan.fu@xxxxxxxxx>
  Gary Lin <glin@xxxxxxxx>

jobs:
 build-amd64-xsm                                              fail    
 build-i386-xsm                                               fail    
 build-amd64                                                  fail    
 build-i386                                                   fail    
 build-amd64-libvirt                                          blocked 
 build-i386-libvirt                                           blocked 
 build-amd64-pvops                                            pass    
 build-i386-pvops                                             pass    
 test-amd64-amd64-xl-qemuu-ovmf-amd64                         blocked 
 test-amd64-i386-xl-qemuu-ovmf-amd64                          blocked 


------------------------------------------------------------
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 6340f2b8587e85fd094261065831546c84b27926
Author: Gary Lin <glin@xxxxxxxx>
Date:   Fri May 20 11:18:16 2016 +0800

    NetworkPkg/HttpDxe: Don't free Wrap in HttpTcpReceiveNotifyDpc
    
    The HTTP Token Wrap is created in EfiHttpResponse() and then passed
    to the deferred Receive event callback, HttpTcpReceiveNotifyDpc.
    HttpTcpReceiveHeader and HttpTcpReceiveBody use a Tcp polling loop to
    monitor the socket status and trigger the Receive event when a new
    packet arrives. The Receive event brings up HttpTcpReceiveNotifyDpc
    to process the HTTP message and the function will set Wrap->TcpWrap.IsRxDone
    to TRUE to break the Tcp polling loop.
    
    However, HttpTcpReceiveNotifyDpc mistakenly freed Wrap, so the Tcp
    polling loop was actually checking a dead variable, and this led the
    system into an unstable status.
    
    Given the fact that the HTTP Token Wrap will be freed in EfiHttpResponse
    or HttpResponseWorker, this commit removes every "FreePool (Wrap)" in
    HttpTcpReceiveNotifyDpc.
    
    v2:
    * Free Wrap after HttpTcpReceiveBody returns normally.
    
    Cc: "Wu, Jiaxin" <jiaxin.wu@xxxxxxxxx>
    Cc: "Siyuan Fu" <siyuan.fu@xxxxxxxxx>
    Cc: "El-Haj-Mahmoud, Samer" <samer.el-haj-mahmoud@xxxxxxx>
    Cc: "Laszlo Ersek" <lersek@xxxxxxxxxx>
    Cc: "Hegde, Nagaraj P" <nagaraj-p.hegde@xxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.0
    Signed-off-by: Gary Lin <glin@xxxxxxxx>
    Reviewed-by: Samer El-Haj-Mahmoud <elhaj@xxxxxxx>
    Reviewed-by: Jiaxin Wu <jiaxin.wu@xxxxxxxxx>
    Reviewed-by: Siyuan Fu <siyuan.fu@xxxxxxxxx>

commit 72a4f34eab072132652565dd6bed24ec8d553aac
Author: Gary Lin <glin@xxxxxxxx>
Date:   Thu May 19 11:49:18 2016 +0800

    NetworkPkg/TcpDxe: Remove the status check of SockProcessRcvToken
    
    SockProcessRcvToken only returns the number of the received bytes, not
    an EFI Status.
    
    Cc: "Siyuan Fu" <siyuan.fu@xxxxxxxxx>
    Cc: "Jiaxin Wu" <jiaxin.wu@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.0
    Signed-off-by: Gary Lin <glin@xxxxxxxx>
    Reviewed-by: Samer El-Haj-Mahmoud <elhaj@xxxxxxx>
    Reviewed-by: Jiaxin Wu <jiaxin.wu@xxxxxxxxx>
    Reviewed-by: Siyuan Fu <siyuan.fu@xxxxxxxxx>

commit 2be45bfe2779043bc3566e879e7ec279412012dc
Author: Fu Siyuan <siyuan.fu@xxxxxxxxx>
Date:   Fri May 6 10:30:09 2016 +0800

    ShellPkg: Add argument to set block size for tftp command.
    
    TFTP block size has a big impact on the transmit performance, this patch is 
to
    add new argument [-s <block size>] for shell "tftp" command to configure the
    block size for file download.
    
    Contributed-under: TianoCore Contribution Agreement 1.0
    Signed-off-by: Fu Siyuan <siyuan.fu@xxxxxxxxx>
    Reviewed-by: Ye Ting <ting.ye@xxxxxxxxx>
    Reviewed-by: Qiu Shumin <shumin.qiu@xxxxxxxxx>
    Reviewed-by: Jaben Carsey <jaben.carsey@xxxxxxxxx>

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