[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Xen-devel] [qemu-mainline bisection] complete build-armhf-xsm



branch xen-unstable
xenbranch xen-unstable
job build-armhf-xsm
testid xen-build

Tree: qemuu git://git.qemu.org/qemu.git
Tree: xen git://xenbits.xen.org/xen.git

*** Found and reproduced problem changeset ***

  Bug is in tree:  qemuu git://git.qemu.org/qemu.git
  Bug introduced:  b3e46a89147493d4474dafe983befca2d6500275
  Bug not present: a51568b78ea011e0f1e67664b8b0c6b693f8ee5a
  Last fail repro: http://logs.test-lab.xenproject.org/osstest/logs/112237/


  commit b3e46a89147493d4474dafe983befca2d6500275
  Merge: a51568b 331b518
  Author: Peter Maydell <peter.maydell@xxxxxxxxxx>
  Date:   Wed Jul 19 16:31:08 2017 +0100
  
      Merge remote-tracking branch 'remotes/sstabellini/tags/xen-20170718-tag' 
into staging
      
      Xen 2017/07/18
      
      # gpg: Signature made Tue 18 Jul 2017 23:18:16 BST
      # gpg:                using RSA key 0x894F8F4870E1AE90
      # gpg: Good signature from "Stefano Stabellini 
<stefano.stabellini@xxxxxxxxxxxxx>"
      # gpg:                 aka "Stefano Stabellini <sstabellini@xxxxxxxxxx>"
      # Primary key fingerprint: D04E 33AB A51F 67BA 07D3  0AEA 894F 8F48 70E1 
AE90
      
      * remotes/sstabellini/tags/xen-20170718-tag:
        xen: don't use xenstore to save/restore physmap anymore
        xen/mapcache: introduce xen_replace_cache_entry()
        xen/mapcache: add an ability to create dummy mappings
        xen: move physmap saving into a separate function
        xen-platform: separate unplugging of NVMe disks
        xen_pt_msi.c: Check for xen_host_pci_get_* failures in 
xen_pt_msix_init()
        hw/xen: Set emu_mask for igd_opregion register
      
      Signed-off-by: Peter Maydell <peter.maydell@xxxxxxxxxx>
  
  commit 331b5189d756d431b1d18ae7097527ba3d3ea809
  Author: Igor Druzhinin <igor.druzhinin@xxxxxxxxxx>
  Date:   Mon Jul 10 23:40:03 2017 +0100
  
      xen: don't use xenstore to save/restore physmap anymore
      
      If we have a system with xenforeignmemory_map2() implemented
      we don't need to save/restore physmap on suspend/restore
      anymore. In case we resume a VM without physmap - try to
      recreate the physmap during memory region restore phase and
      remap map cache entries accordingly. The old code is left
      for compatibility reasons.
      
      Signed-off-by: Igor Druzhinin <igor.druzhinin@xxxxxxxxxx>
      Reviewed-by: Paul Durrant <paul.durrant@xxxxxxxxxx>
      Reviewed-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>
      Signed-off-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>
  
  commit 5ba3d7564593c55292056ef5af84d50b55ebcf0e
  Author: Igor Druzhinin <igor.druzhinin@xxxxxxxxxx>
  Date:   Mon Jul 10 23:40:02 2017 +0100
  
      xen/mapcache: introduce xen_replace_cache_entry()
      
      This new call is trying to update a requested map cache entry
      according to the changes in the physmap. The call is searching
      for the entry, unmaps it and maps again at the same place using
      a new guest address. If the mapping is dummy this call will
      make it real.
      
      This function makes use of a new xenforeignmemory_map2() call
      with an extended interface that was recently introduced in
      libxenforeignmemory [1].
      
      [1] https://www.mail-archive.com/xen-devel@xxxxxxxxxxxxx/msg113007.html
      
      Signed-off-by: Igor Druzhinin <igor.druzhinin@xxxxxxxxxx>
      Reviewed-by: Paul Durrant <paul.durrant@xxxxxxxxxx>
      Reviewed-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>
      Signed-off-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>
  
  commit 759235653de427e4e7b62d8e6fb1ef9cb68bac7d
  Author: Igor Druzhinin <igor.druzhinin@xxxxxxxxxx>
  Date:   Mon Jul 10 23:40:01 2017 +0100
  
      xen/mapcache: add an ability to create dummy mappings
      
      Dummys are simple anonymous mappings that are placed instead
      of regular foreign mappings in certain situations when we need
      to postpone the actual mapping but still have to give a
      memory region to QEMU to play with.
      
      This is planned to be used for restore on Xen.
      
      Signed-off-by: Igor Druzhinin <igor.druzhinin@xxxxxxxxxx>
      Reviewed-by: Paul Durrant <paul.durrant@xxxxxxxxxx>
      Reviewed-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>
  
  commit 697b66d006676620a56fb5b79720ce457158204b
  Author: Igor Druzhinin <igor.druzhinin@xxxxxxxxxx>
  Date:   Mon Jul 10 23:40:00 2017 +0100
  
      xen: move physmap saving into a separate function
      
      Non-functional change.
      
      Signed-off-by: Igor Druzhinin <igor.druzhinin@xxxxxxxxxx>
      Reviewed-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>
      Reviewed-by: Paul Durrant <paul.durrant@xxxxxxxxxx>
  
  commit 04d6da4ff6084a3cb1b7a981769d9aa17e469348
  Author: Stefano Stabellini <sstabellini@xxxxxxxxxx>
  Date:   Tue Jul 18 13:28:12 2017 -0700
  
      xen-platform: separate unplugging of NVMe disks
      
      Commit 090fa1c8 "add support for unplugging NVMe disks..." extended the
      existing disk unplug flag to cover NVMe disks as well as IDE and SCSI.
      
      The recent thread on the xen-devel mailing list [1] has highlighted that
      this is not desirable behaviour: PV frontends should be able to 
distinguish
      NVMe disks from other types of disk and should have separate control over
      whether they are unplugged.
      
      This patch defines a new bit in the unplug mask for this purpose (see Xen
      commit [2]) and also tidies up the definitions of, and improves the
      comments regarding, the previously exiting bits in the protocol.
      
      [1] https://lists.xen.org/archives/html/xen-devel/2017-03/msg02924.html
      [2] http://xenbits.xen.org/gitweb/?p=xen.git;a=commit;h=1096aa02
      
      Signed-off-by: Paul Durrant <paul.durrant@xxxxxxxxxx>
      Reviewed-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>
      Signed-off-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>
  
  commit 64c7c1175b4e3e6fe005934cde63259d8adad392
  Author: Peter Maydell <peter.maydell@xxxxxxxxxx>
  Date:   Sun Jul 9 17:37:22 2017 +0100
  
      xen_pt_msi.c: Check for xen_host_pci_get_* failures in xen_pt_msix_init()
      
      Check the return status of the xen_host_pci_get_* functions we call in
      xen_pt_msix_init(), and fail device init if the reads failed rather than
      ploughing ahead. (Spotted by Coverity: CID 777338.)
      
      Signed-off-by: Peter Maydell <peter.maydell@xxxxxxxxxx>
      Reviewed-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>
      Signed-off-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>
  
  commit a19bae42e341a05f43f685fc9ff0e19ba6f129c3
  Author: Xiong Zhang <xiong.y.zhang@xxxxxxxxx>
  Date:   Fri Jul 7 12:07:58 2017 +0800
  
      hw/xen: Set emu_mask for igd_opregion register
      
      In igd passthrough environment, guest could only access opregion at the
      first bootup time. Once guest shutdown, later guest couldn't access
      opregion anymore.
      This is because qemu set emulated guest opregion base address to host
      register. Later guest get a wrong host opregion base address, and couldn't
      access it anymore.
      
      This patch set emu_mask for igd_opregion register, so guest won't set
      guest opregion base address to host.
      
      Signed-off-by: Xiong Zhang <xiong.y.zhang@xxxxxxxxx>
      Acked-by: Anthony PERARD <anthony.perard@xxxxxxxxxx>
      Signed-off-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>


For bisection revision-tuple graph see:
   
http://logs.test-lab.xenproject.org/osstest/results/bisect/qemu-mainline/build-armhf-xsm.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/qemu-mainline/build-armhf-xsm.xen-build 
--summary-out=tmp/112237.bisection-summary --basis-template=111765 
--blessings=real,real-bisect qemu-mainline build-armhf-xsm xen-build
Searching for failure / basis pass:
 112155 fail [host=arndale-bluewater] / 112011 ok.
Failure / basis pass flights: 112155 / 112011
Tree: qemuu git://git.qemu.org/qemu.git
Tree: xen git://xenbits.xen.org/xen.git
Latest 91939262ffcd3c85ea6a4793d3029326eea1d649 
d535d8922f571502252deaf607e82e7475cd1728
Basis pass 63cb55783c5e8f783b1dcebd3a2935941f872d44 
2b8a8a03f56e21381c7dd560b081002d357639e2
Generating revisions with ./adhoc-revtuple-generator  
git://git.qemu.org/qemu.git#63cb55783c5e8f783b1dcebd3a2935941f872d44-91939262ffcd3c85ea6a4793d3029326eea1d649
 
git://xenbits.xen.org/xen.git#2b8a8a03f56e21381c7dd560b081002d357639e2-d535d8922f571502252deaf607e82e7475cd1728
Loaded 8030 nodes in revision graph
Searching for test results:
 112011 pass 63cb55783c5e8f783b1dcebd3a2935941f872d44 
2b8a8a03f56e21381c7dd560b081002d357639e2
 112041 [host=arndale-lakeside]
 112072 [host=arndale-metrocentre]
 112100 [host=arndale-metrocentre]
 112216 [host=arndale-metrocentre]
 112198 [host=arndale-metrocentre]
 112202 [host=arndale-metrocentre]
 112196 [host=arndale-metrocentre]
 112197 [host=arndale-metrocentre]
 112200 [host=arndale-metrocentre]
 112203 [host=arndale-metrocentre]
 112206 [host=arndale-metrocentre]
 112215 [host=arndale-metrocentre]
 112208 [host=arndale-metrocentre]
 112209 [host=arndale-metrocentre]
 112155 fail 91939262ffcd3c85ea6a4793d3029326eea1d649 
d535d8922f571502252deaf607e82e7475cd1728
 112218 pass 63cb55783c5e8f783b1dcebd3a2935941f872d44 
2b8a8a03f56e21381c7dd560b081002d357639e2
 112219 fail 91939262ffcd3c85ea6a4793d3029326eea1d649 
d535d8922f571502252deaf607e82e7475cd1728
 112221 fail 14e167530adfc33f083299fb9f33499d18f3e729 
d535d8922f571502252deaf607e82e7475cd1728
 112223 fail 87a60ee84f8abfc26f200ceac86ef32a690b9e21 
d535d8922f571502252deaf607e82e7475cd1728
 112225 fail 810d5cad4087236236e00fd3046a16adf26e9060 
d535d8922f571502252deaf607e82e7475cd1728
 112226 pass 6d60e295ef020759a03b90724d0342012c189ba2 
d535d8922f571502252deaf607e82e7475cd1728
 112227 fail 824dbfb45d4834c6538f6f70737014b1f19e2e55 
d535d8922f571502252deaf607e82e7475cd1728
 112229 pass a51568b78ea011e0f1e67664b8b0c6b693f8ee5a 
d535d8922f571502252deaf607e82e7475cd1728
 112230 fail b3e46a89147493d4474dafe983befca2d6500275 
d535d8922f571502252deaf607e82e7475cd1728
 112231 pass a51568b78ea011e0f1e67664b8b0c6b693f8ee5a 
d535d8922f571502252deaf607e82e7475cd1728
 112234 fail b3e46a89147493d4474dafe983befca2d6500275 
d535d8922f571502252deaf607e82e7475cd1728
 112236 pass a51568b78ea011e0f1e67664b8b0c6b693f8ee5a 
d535d8922f571502252deaf607e82e7475cd1728
 112237 fail b3e46a89147493d4474dafe983befca2d6500275 
d535d8922f571502252deaf607e82e7475cd1728
Searching for interesting versions
 Result found: flight 112011 (pass), for basis pass
 Result found: flight 112155 (fail), for basis failure
 Repro found: flight 112218 (pass), for basis pass
 Repro found: flight 112219 (fail), for basis failure
 0 revisions at a51568b78ea011e0f1e67664b8b0c6b693f8ee5a 
d535d8922f571502252deaf607e82e7475cd1728
No revisions left to test, checking graph state.
 Result found: flight 112229 (pass), for last pass
 Result found: flight 112230 (fail), for first failure
 Repro found: flight 112231 (pass), for last pass
 Repro found: flight 112234 (fail), for first failure
 Repro found: flight 112236 (pass), for last pass
 Repro found: flight 112237 (fail), for first failure

*** Found and reproduced problem changeset ***

  Bug is in tree:  qemuu git://git.qemu.org/qemu.git
  Bug introduced:  b3e46a89147493d4474dafe983befca2d6500275
  Bug not present: a51568b78ea011e0f1e67664b8b0c6b693f8ee5a
  Last fail repro: http://logs.test-lab.xenproject.org/osstest/logs/112237/


  commit b3e46a89147493d4474dafe983befca2d6500275
  Merge: a51568b 331b518
  Author: Peter Maydell <peter.maydell@xxxxxxxxxx>
  Date:   Wed Jul 19 16:31:08 2017 +0100
  
      Merge remote-tracking branch 'remotes/sstabellini/tags/xen-20170718-tag' 
into staging
      
      Xen 2017/07/18
      
      # gpg: Signature made Tue 18 Jul 2017 23:18:16 BST
      # gpg:                using RSA key 0x894F8F4870E1AE90
      # gpg: Good signature from "Stefano Stabellini 
<stefano.stabellini@xxxxxxxxxxxxx>"
      # gpg:                 aka "Stefano Stabellini <sstabellini@xxxxxxxxxx>"
      # Primary key fingerprint: D04E 33AB A51F 67BA 07D3  0AEA 894F 8F48 70E1 
AE90
      
      * remotes/sstabellini/tags/xen-20170718-tag:
        xen: don't use xenstore to save/restore physmap anymore
        xen/mapcache: introduce xen_replace_cache_entry()
        xen/mapcache: add an ability to create dummy mappings
        xen: move physmap saving into a separate function
        xen-platform: separate unplugging of NVMe disks
        xen_pt_msi.c: Check for xen_host_pci_get_* failures in 
xen_pt_msix_init()
        hw/xen: Set emu_mask for igd_opregion register
      
      Signed-off-by: Peter Maydell <peter.maydell@xxxxxxxxxx>
  
  commit 331b5189d756d431b1d18ae7097527ba3d3ea809
  Author: Igor Druzhinin <igor.druzhinin@xxxxxxxxxx>
  Date:   Mon Jul 10 23:40:03 2017 +0100
  
      xen: don't use xenstore to save/restore physmap anymore
      
      If we have a system with xenforeignmemory_map2() implemented
      we don't need to save/restore physmap on suspend/restore
      anymore. In case we resume a VM without physmap - try to
      recreate the physmap during memory region restore phase and
      remap map cache entries accordingly. The old code is left
      for compatibility reasons.
      
      Signed-off-by: Igor Druzhinin <igor.druzhinin@xxxxxxxxxx>
      Reviewed-by: Paul Durrant <paul.durrant@xxxxxxxxxx>
      Reviewed-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>
      Signed-off-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>
  
  commit 5ba3d7564593c55292056ef5af84d50b55ebcf0e
  Author: Igor Druzhinin <igor.druzhinin@xxxxxxxxxx>
  Date:   Mon Jul 10 23:40:02 2017 +0100
  
      xen/mapcache: introduce xen_replace_cache_entry()
      
      This new call is trying to update a requested map cache entry
      according to the changes in the physmap. The call is searching
      for the entry, unmaps it and maps again at the same place using
      a new guest address. If the mapping is dummy this call will
      make it real.
      
      This function makes use of a new xenforeignmemory_map2() call
      with an extended interface that was recently introduced in
      libxenforeignmemory [1].
      
      [1] https://www.mail-archive.com/xen-devel@xxxxxxxxxxxxx/msg113007.html
      
      Signed-off-by: Igor Druzhinin <igor.druzhinin@xxxxxxxxxx>
      Reviewed-by: Paul Durrant <paul.durrant@xxxxxxxxxx>
      Reviewed-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>
      Signed-off-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>
  
  commit 759235653de427e4e7b62d8e6fb1ef9cb68bac7d
  Author: Igor Druzhinin <igor.druzhinin@xxxxxxxxxx>
  Date:   Mon Jul 10 23:40:01 2017 +0100
  
      xen/mapcache: add an ability to create dummy mappings
      
      Dummys are simple anonymous mappings that are placed instead
      of regular foreign mappings in certain situations when we need
      to postpone the actual mapping but still have to give a
      memory region to QEMU to play with.
      
      This is planned to be used for restore on Xen.
      
      Signed-off-by: Igor Druzhinin <igor.druzhinin@xxxxxxxxxx>
      Reviewed-by: Paul Durrant <paul.durrant@xxxxxxxxxx>
      Reviewed-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>
  
  commit 697b66d006676620a56fb5b79720ce457158204b
  Author: Igor Druzhinin <igor.druzhinin@xxxxxxxxxx>
  Date:   Mon Jul 10 23:40:00 2017 +0100
  
      xen: move physmap saving into a separate function
      
      Non-functional change.
      
      Signed-off-by: Igor Druzhinin <igor.druzhinin@xxxxxxxxxx>
      Reviewed-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>
      Reviewed-by: Paul Durrant <paul.durrant@xxxxxxxxxx>
  
  commit 04d6da4ff6084a3cb1b7a981769d9aa17e469348
  Author: Stefano Stabellini <sstabellini@xxxxxxxxxx>
  Date:   Tue Jul 18 13:28:12 2017 -0700
  
      xen-platform: separate unplugging of NVMe disks
      
      Commit 090fa1c8 "add support for unplugging NVMe disks..." extended the
      existing disk unplug flag to cover NVMe disks as well as IDE and SCSI.
      
      The recent thread on the xen-devel mailing list [1] has highlighted that
      this is not desirable behaviour: PV frontends should be able to 
distinguish
      NVMe disks from other types of disk and should have separate control over
      whether they are unplugged.
      
      This patch defines a new bit in the unplug mask for this purpose (see Xen
      commit [2]) and also tidies up the definitions of, and improves the
      comments regarding, the previously exiting bits in the protocol.
      
      [1] https://lists.xen.org/archives/html/xen-devel/2017-03/msg02924.html
      [2] http://xenbits.xen.org/gitweb/?p=xen.git;a=commit;h=1096aa02
      
      Signed-off-by: Paul Durrant <paul.durrant@xxxxxxxxxx>
      Reviewed-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>
      Signed-off-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>
  
  commit 64c7c1175b4e3e6fe005934cde63259d8adad392
  Author: Peter Maydell <peter.maydell@xxxxxxxxxx>
  Date:   Sun Jul 9 17:37:22 2017 +0100
  
      xen_pt_msi.c: Check for xen_host_pci_get_* failures in xen_pt_msix_init()
      
      Check the return status of the xen_host_pci_get_* functions we call in
      xen_pt_msix_init(), and fail device init if the reads failed rather than
      ploughing ahead. (Spotted by Coverity: CID 777338.)
      
      Signed-off-by: Peter Maydell <peter.maydell@xxxxxxxxxx>
      Reviewed-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>
      Signed-off-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>
  
  commit a19bae42e341a05f43f685fc9ff0e19ba6f129c3
  Author: Xiong Zhang <xiong.y.zhang@xxxxxxxxx>
  Date:   Fri Jul 7 12:07:58 2017 +0800
  
      hw/xen: Set emu_mask for igd_opregion register
      
      In igd passthrough environment, guest could only access opregion at the
      first bootup time. Once guest shutdown, later guest couldn't access
      opregion anymore.
      This is because qemu set emulated guest opregion base address to host
      register. Later guest get a wrong host opregion base address, and couldn't
      access it anymore.
      
      This patch set emu_mask for igd_opregion register, so guest won't set
      guest opregion base address to host.
      
      Signed-off-by: Xiong Zhang <xiong.y.zhang@xxxxxxxxx>
      Acked-by: Anthony PERARD <anthony.perard@xxxxxxxxxx>
      Signed-off-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>

Revision graph left in 
/home/logs/results/bisect/qemu-mainline/build-armhf-xsm.xen-build.{dot,ps,png,html,svg}.
----------------------------------------
112237: tolerable ALL FAIL

flight 112237 qemu-mainline real-bisect [real]
http://logs.test-lab.xenproject.org/osstest/logs/112237/

Failures :-/ but no regressions.

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


jobs:
 build-armhf-xsm                                              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@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.