branch xen-unstable
xenbranch xen-unstable
job build-i386
testid xen-build

Tree: ovmf https://github.com/tianocore/edk2.git
Tree: qemu git://xenbits.xen.org/qemu-xen-traditional.git
Tree: qemuu git://xenbits.xen.org/qemu-xen.git
Tree: xen git://xenbits.xen.org/xen.git

*** Found and reproduced problem changeset ***

  Bug is in tree:  ovmf https://github.com/tianocore/edk2.git
  Bug introduced:  0c9f2cb10b7ddec56a3440e77219fd3ab1725e5c
  Bug not present: d22c995a481485d8240453a9e5ea9af1d87a4c80
  Last fail repro: http://logs.test-lab.xenproject.org/osstest/logs/122098/


  commit 0c9f2cb10b7ddec56a3440e77219fd3ab1725e5c
  Author: Kinney, Michael D <michael.d.kin...@intel.com>
  Date:   Tue Apr 3 06:40:24 2018 +0800
  
      MdeModulePkg/Gcd: Filter gCpu->SetMemoryAttributes() calls
      
      This patch fixes an issue with VlvTbltDevicePkg introduced
      by commit 5b91bf82c67b586b9588cbe4bbffa1588f6b5926.
      
      The history is as below.
      To support heap guard feature, 14dde9e903bb9a719ebb8f3381da72b19509bc36
      added support for SetMemorySpaceAttributes() to handle page attributes,
      but after that, a combination of CPU arch attributes and other attributes
      was not allowed anymore, for example, UC + RUNTIME. It is a regression.
      Then 5b91bf82c67b586b9588cbe4bbffa1588f6b5926 was to fix the regression,
      and we thought 0 CPU arch attributes may be used to clear CPU arch
      attributes, so 0 CPU arch attributes was allowed to be sent to
      gCpu->SetMemoryAttributes().
      
      But some implementation of CPU driver may return error for 0 CPU arch
      attributes. That fails the case that caller just calls
      SetMemorySpaceAttributes() with none CPU arch attributes (for example,
      RUNTIME), and the purpose of the case is not to clear CPU arch attributes.
      
      This patch filters the call to gCpu->SetMemoryAttributes()
      if the requested attributes is 0.  It also removes the #define
      INVALID_CPU_ARCH_ATTRIBUTES that is no longer used.
      
      Cc: Heyi Guo <heyi....@linaro.org>
      Cc: Yi Li <phoenix.l...@huawei.com>
      Cc: Renhao Liang <liangren...@huawei.com>
      Cc: Star Zeng <star.z...@intel.com>
      Cc: Eric Dong <eric.d...@intel.com>
      Cc: Liming Gao <liming....@intel.com>
      Cc: Jian J Wang <jian.j.w...@intel.com>
      Cc: Ruiyu Ni <ruiyu...@intel.com>
      Signed-off-by: Michael D Kinney <michael.d.kin...@intel.com>
      Signed-off-by: Star Zeng <star.z...@intel.com>
      Contributed-under: TianoCore Contribution Agreement 1.1
      Reviewed-by: Jiewen Yao <jiewen....@intel.com>
      Reviewed-by: Michael D Kinney <michael.d.kin...@intel.com>


For bisection revision-tuple graph see:
   
http://logs.test-lab.xenproject.org/osstest/results/bisect/ovmf/build-i386.xen-build.html
Revision IDs in each graph node refer, respectively, to the Trees above.

----------------------------------------
Running cs-bisection-step 
--graph-out=/home/logs/results/bisect/ovmf/build-i386.xen-build 
--summary-out=tmp/122098.bisection-summary --basis-template=121769 
--blessings=real,real-bisect ovmf build-i386 xen-build
Searching for failure / basis pass:
 122084 fail [host=chardonnay1] / 121769 ok.
Failure / basis pass flights: 122084 / 121769
(tree with no url: minios)
(tree with no url: seabios)
Tree: ovmf https://github.com/tianocore/edk2.git
Tree: qemu git://xenbits.xen.org/qemu-xen-traditional.git
Tree: qemuu git://xenbits.xen.org/qemu-xen.git
Tree: xen git://xenbits.xen.org/xen.git
Latest ab0ff0b0a633f5f566e434f7625009a92f02304c 
c8ea0457495342c417c3dc033bba25148b279f60 
5c3fdee026a204a59cb392e43a313ab558de9682 
451004603247205467ec34b366b4cfa3814a5d95
Basis pass aae02dccf5b0ad07e60d2738f350b3b39df389d7 
c8ea0457495342c417c3dc033bba25148b279f60 
5c3fdee026a204a59cb392e43a313ab558de9682 
eabb83121226d5a6a5a68da3a913ac0b5bb1e0cf
Generating revisions with ./adhoc-revtuple-generator  
https://github.com/tianocore/edk2.git#aae02dccf5b0ad07e60d2738f350b3b39df389d7-ab0ff0b0a633f5f566e434f7625009a92f02304c
 
git://xenbits.xen.org/qemu-xen-traditional.git#c8ea0457495342c417c3dc033bba25148b279f60-c8ea0457495342c417c3dc033bba25148b279f60
 
git://xenbits.xen.org/qemu-xen.git#5c3fdee026a204a59cb392e43a313ab558de9682-5c3fdee026a204a59cb392e43a313ab558de9682
 
git://xenbits.xen.org/xen.git#eabb83121226d5a6a5a68da3a913ac0b5bb1e0cf-451004603247205467ec34b366b4cfa3814a5d95
Loaded 2001 nodes in revision graph
Searching for test results:
 121769 pass aae02dccf5b0ad07e60d2738f350b3b39df389d7 
c8ea0457495342c417c3dc033bba25148b279f60 
5c3fdee026a204a59cb392e43a313ab558de9682 
eabb83121226d5a6a5a68da3a913ac0b5bb1e0cf
 121901 fail irrelevant
 121977 [host=chardonnay0]
 122070 fail 0c9f2cb10b7ddec56a3440e77219fd3ab1725e5c 
c8ea0457495342c417c3dc033bba25148b279f60 
5c3fdee026a204a59cb392e43a313ab558de9682 
451004603247205467ec34b366b4cfa3814a5d95
 122045 [host=chardonnay0]
 122007 [host=chardonnay0]
 122072 pass aae02dccf5b0ad07e60d2738f350b3b39df389d7 
c8ea0457495342c417c3dc033bba25148b279f60 
5c3fdee026a204a59cb392e43a313ab558de9682 
eabb83121226d5a6a5a68da3a913ac0b5bb1e0cf
 122051 fail 0c9f2cb10b7ddec56a3440e77219fd3ab1725e5c 
c8ea0457495342c417c3dc033bba25148b279f60 
5c3fdee026a204a59cb392e43a313ab558de9682 
451004603247205467ec34b366b4cfa3814a5d95
 122074 pass aae02dccf5b0ad07e60d2738f350b3b39df389d7 
c8ea0457495342c417c3dc033bba25148b279f60 
5c3fdee026a204a59cb392e43a313ab558de9682 
e5fe34fd23816601de17b0a428909c95acf01c93
 122057 fail 0c9f2cb10b7ddec56a3440e77219fd3ab1725e5c 
c8ea0457495342c417c3dc033bba25148b279f60 
5c3fdee026a204a59cb392e43a313ab558de9682 
451004603247205467ec34b366b4cfa3814a5d95
 122075 pass aae02dccf5b0ad07e60d2738f350b3b39df389d7 
c8ea0457495342c417c3dc033bba25148b279f60 
5c3fdee026a204a59cb392e43a313ab558de9682 
3f8f12281dd20da62366f555cf13be0e87d68a91
 122073 fail 0c9f2cb10b7ddec56a3440e77219fd3ab1725e5c 
c8ea0457495342c417c3dc033bba25148b279f60 
5c3fdee026a204a59cb392e43a313ab558de9682 
451004603247205467ec34b366b4cfa3814a5d95
 122076 fail 0c9f2cb10b7ddec56a3440e77219fd3ab1725e5c 
c8ea0457495342c417c3dc033bba25148b279f60 
5c3fdee026a204a59cb392e43a313ab558de9682 
451004603247205467ec34b366b4cfa3814a5d95
 122081 pass f0053e837a2141e3c09ad376efe27fd17c522b04 
c8ea0457495342c417c3dc033bba25148b279f60 
5c3fdee026a204a59cb392e43a313ab558de9682 
9383de210e747f15d0fd10ade89e35d543fbc4e8
 122078 pass fc504fdea7fe92bfa88e15f50e64b4d76d4f75fd 
c8ea0457495342c417c3dc033bba25148b279f60 
5c3fdee026a204a59cb392e43a313ab558de9682 
9383de210e747f15d0fd10ade89e35d543fbc4e8
 122085 pass d22c995a481485d8240453a9e5ea9af1d87a4c80 
c8ea0457495342c417c3dc033bba25148b279f60 
5c3fdee026a204a59cb392e43a313ab558de9682 
9383de210e747f15d0fd10ade89e35d543fbc4e8
 122082 pass 9686a4678de78e797e1368b9ff131890d3dee41b 
c8ea0457495342c417c3dc033bba25148b279f60 
5c3fdee026a204a59cb392e43a313ab558de9682 
9383de210e747f15d0fd10ade89e35d543fbc4e8
 122086 pass aae02dccf5b0ad07e60d2738f350b3b39df389d7 
c8ea0457495342c417c3dc033bba25148b279f60 
5c3fdee026a204a59cb392e43a313ab558de9682 
eabb83121226d5a6a5a68da3a913ac0b5bb1e0cf
 122084 fail ab0ff0b0a633f5f566e434f7625009a92f02304c 
c8ea0457495342c417c3dc033bba25148b279f60 
5c3fdee026a204a59cb392e43a313ab558de9682 
451004603247205467ec34b366b4cfa3814a5d95
 122089 fail ab0ff0b0a633f5f566e434f7625009a92f02304c 
c8ea0457495342c417c3dc033bba25148b279f60 
5c3fdee026a204a59cb392e43a313ab558de9682 
451004603247205467ec34b366b4cfa3814a5d95
 122090 fail 0c9f2cb10b7ddec56a3440e77219fd3ab1725e5c 
c8ea0457495342c417c3dc033bba25148b279f60 
5c3fdee026a204a59cb392e43a313ab558de9682 
9383de210e747f15d0fd10ade89e35d543fbc4e8
 122091 pass d22c995a481485d8240453a9e5ea9af1d87a4c80 
c8ea0457495342c417c3dc033bba25148b279f60 
5c3fdee026a204a59cb392e43a313ab558de9682 
9383de210e747f15d0fd10ade89e35d543fbc4e8
 122093 fail 0c9f2cb10b7ddec56a3440e77219fd3ab1725e5c 
c8ea0457495342c417c3dc033bba25148b279f60 
5c3fdee026a204a59cb392e43a313ab558de9682 
9383de210e747f15d0fd10ade89e35d543fbc4e8
 122095 pass d22c995a481485d8240453a9e5ea9af1d87a4c80 
c8ea0457495342c417c3dc033bba25148b279f60 
5c3fdee026a204a59cb392e43a313ab558de9682 
9383de210e747f15d0fd10ade89e35d543fbc4e8
 122098 fail 0c9f2cb10b7ddec56a3440e77219fd3ab1725e5c 
c8ea0457495342c417c3dc033bba25148b279f60 
5c3fdee026a204a59cb392e43a313ab558de9682 
9383de210e747f15d0fd10ade89e35d543fbc4e8
Searching for interesting versions
 Result found: flight 121769 (pass), for basis pass
 Result found: flight 122084 (fail), for basis failure
 Repro found: flight 122086 (pass), for basis pass
 Repro found: flight 122089 (fail), for basis failure
 0 revisions at d22c995a481485d8240453a9e5ea9af1d87a4c80 
c8ea0457495342c417c3dc033bba25148b279f60 
5c3fdee026a204a59cb392e43a313ab558de9682 
9383de210e747f15d0fd10ade89e35d543fbc4e8
No revisions left to test, checking graph state.
 Result found: flight 122085 (pass), for last pass
 Result found: flight 122090 (fail), for first failure
 Repro found: flight 122091 (pass), for last pass
 Repro found: flight 122093 (fail), for first failure
 Repro found: flight 122095 (pass), for last pass
 Repro found: flight 122098 (fail), for first failure

*** Found and reproduced problem changeset ***

  Bug is in tree:  ovmf https://github.com/tianocore/edk2.git
  Bug introduced:  0c9f2cb10b7ddec56a3440e77219fd3ab1725e5c
  Bug not present: d22c995a481485d8240453a9e5ea9af1d87a4c80
  Last fail repro: http://logs.test-lab.xenproject.org/osstest/logs/122098/


  commit 0c9f2cb10b7ddec56a3440e77219fd3ab1725e5c
  Author: Kinney, Michael D <michael.d.kin...@intel.com>
  Date:   Tue Apr 3 06:40:24 2018 +0800
  
      MdeModulePkg/Gcd: Filter gCpu->SetMemoryAttributes() calls
      
      This patch fixes an issue with VlvTbltDevicePkg introduced
      by commit 5b91bf82c67b586b9588cbe4bbffa1588f6b5926.
      
      The history is as below.
      To support heap guard feature, 14dde9e903bb9a719ebb8f3381da72b19509bc36
      added support for SetMemorySpaceAttributes() to handle page attributes,
      but after that, a combination of CPU arch attributes and other attributes
      was not allowed anymore, for example, UC + RUNTIME. It is a regression.
      Then 5b91bf82c67b586b9588cbe4bbffa1588f6b5926 was to fix the regression,
      and we thought 0 CPU arch attributes may be used to clear CPU arch
      attributes, so 0 CPU arch attributes was allowed to be sent to
      gCpu->SetMemoryAttributes().
      
      But some implementation of CPU driver may return error for 0 CPU arch
      attributes. That fails the case that caller just calls
      SetMemorySpaceAttributes() with none CPU arch attributes (for example,
      RUNTIME), and the purpose of the case is not to clear CPU arch attributes.
      
      This patch filters the call to gCpu->SetMemoryAttributes()
      if the requested attributes is 0.  It also removes the #define
      INVALID_CPU_ARCH_ATTRIBUTES that is no longer used.
      
      Cc: Heyi Guo <heyi....@linaro.org>
      Cc: Yi Li <phoenix.l...@huawei.com>
      Cc: Renhao Liang <liangren...@huawei.com>
      Cc: Star Zeng <star.z...@intel.com>
      Cc: Eric Dong <eric.d...@intel.com>
      Cc: Liming Gao <liming....@intel.com>
      Cc: Jian J Wang <jian.j.w...@intel.com>
      Cc: Ruiyu Ni <ruiyu...@intel.com>
      Signed-off-by: Michael D Kinney <michael.d.kin...@intel.com>
      Signed-off-by: Star Zeng <star.z...@intel.com>
      Contributed-under: TianoCore Contribution Agreement 1.1
      Reviewed-by: Jiewen Yao <jiewen....@intel.com>
      Reviewed-by: Michael D Kinney <michael.d.kin...@intel.com>

Revision graph left in 
/home/logs/results/bisect/ovmf/build-i386.xen-build.{dot,ps,png,html,svg}.
----------------------------------------
122098: tolerable ALL FAIL

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

Failures :-/ but no regressions.

Tests which did not succeed,
including tests which could not be run:
 build-i386                    6 xen-build               fail baseline untested


jobs:
 build-i386                                                   fail    


------------------------------------------------------------
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


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

Reply via email to