[qemu-mainline bisection] 112187: testing build-i386

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

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

Tree: qemu git://xenbits.xen.org/qemu-xen-traditional.git
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/112183/


  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-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/qemu-mainline/build-i386.xen-build 
--summary-out=tmp/112187.bisection-summary --basis-template=111765 
--blessings=real,real-bisect --flight=112187 qemu-mainline build-i386 xen-build
Searching for failure / basis pass:
 112100 fail [host=nobling0] / 112011 [host=huxelrebe0] 111986 [host=merlot0] 
111963 [host=italia1] 111926 [host=huxelrebe0] 111889 [host=huxelrebe0] 111848 
[host=italia1] 111817 [host=baroque1] 111790 [host=baroque1] 111765 
[host=baroque1] 111732 ok.
Failure / basis pass flights: 112100 / 111732
(tree with no url: minios)
(tree with no url: ovmf)
(tree with no url: seabios)
Tree: qemu git://xenbits.xen.org/qemu-xen-traditional.git
Tree: qemuu git://git.qemu.org/qemu.git
Tree: xen git://xenbits.xen.org/xen.git
Latest 8051789e982499050680a26febeada7467e18a8d 
91939262ffcd3c85ea6a4793d3029326eea1d649 
d535d8922f571502252deaf607e82e7475cd1728
Basis pass 8051789e982499050680a26febeada7467e18a8d 
aa916e409c04cb614ec2fee8b6b33836bf5998bb 
89df98b77d28136c4d7aade13a1c8bc154d2919f
Generating revisions with ./adhoc-revtuple-generator  
git://xenbits.xen.org/qemu-xen-traditional.git#8051789e982499050680a26febeada7467e18a8d-8051789e982499050680a26febeada7467e18a8d
 
git://git.qemu.org/qemu.git#aa916e409c04cb614ec2fee8b6b33836bf5998bb-91939262ffcd3c85ea6a4793d3029326eea1d649
 
git://xenbits.xen.org/xen.git#89df98b77d28136c4d7aade13a1c8bc154d2919f-d535d8922f571502252deaf607e82e7475cd1728
Loaded 8008 nodes in revision graph
Searching for test results:
 111703 [host=italia1]
 111735 pass irrelevant
 111731 pass irrelevant
 111732 pass 8051789e982499050680a26febeada7467e18a8d 
aa916e409c04cb614ec2fee8b6b33836bf5998bb 
89df98b77d28136c4d7aade13a1c8bc154d2919f
 111765 [host=baroque1]
 111790 [host=baroque1]
 111817 [host=baroque1]
 111862 [host=baroque1]
 111871 pass irrelevant
 111848 [host=italia1]
 111876 [host=italia1]
 111910 [host=baroque1]
 111885 [host=huxelrebe0]
 111913 [host=italia1]
 111891 pass 8051789e982499050680a26febeada7467e18a8d 
92ddfade9f619977d47399bd360c03626629b1e2 
614a14736e33fb84872eb00f08799ebbc73a96c6
 111896 [host=baroque1]
 111889 [host=huxelrebe0]
 111905 pass 8051789e982499050680a26febeada7467e18a8d 
75b601602be0462acb4e0298e738df86b1de61a1 
614a14736e33fb84872eb00f08799ebbc73a96c6
 111926 [host=huxelrebe0]
 111964 [host=italia1]
 111986 [host=merlot0]
 111963 [host=italia1]
 111990 [host=italia0]
 112008 [host=huxelrebe0]
 112011 [host=huxelrebe0]
 112041 fail 8051789e982499050680a26febeada7467e18a8d 
d4e59218ab80e86015753782fb5378767a51ccd0 
d535d8922f571502252deaf607e82e7475cd1728
 112072 fail 8051789e982499050680a26febeada7467e18a8d 
25d0233c1ac6cd14a15fcc834f1de3b179037b1d 
d535d8922f571502252deaf607e82e7475cd1728
 112100 fail 8051789e982499050680a26febeada7467e18a8d 
91939262ffcd3c85ea6a4793d3029326eea1d649 
d535d8922f571502252deaf607e82e7475cd1728
 112178 pass 8051789e982499050680a26febeada7467e18a8d 
a51568b78ea011e0f1e67664b8b0c6b693f8ee5a 
d535d8922f571502252deaf607e82e7475cd1728
 112167 pass 8051789e982499050680a26febeada7467e18a8d 
df55c7e8c81db4b142ee157bc41d2ba401b8db20 
2b8a8a03f56e21381c7dd560b081002d357639e2
 112154 pass 8051789e982499050680a26febeada7467e18a8d 
aa916e409c04cb614ec2fee8b6b33836bf5998bb 
89df98b77d28136c4d7aade13a1c8bc154d2919f
 112156 fail 8051789e982499050680a26febeada7467e18a8d 
91939262ffcd3c85ea6a4793d3029326eea1d649 
d535d8922f571502252deaf607e82e7475cd1728
 112169 pass 8051789e982499050680a26febeada7467e18a8d 
c6d41b363caa921e1740f12d0e01f0911fa4aa0d 
d535d8922f571502252deaf607e82e7475cd1728
 112157 pass 8051789e982499050680a26febeada7467e18a8d 
6d60e295ef020759a03b90724d0342012c189ba2 
d535d8922f571502252deaf607e82e7475cd1728
 112159 pass 8051789e982499050680a26febeada7467e18a8d 
a51568b78ea011e0f1e67664b8b0c6b693f8ee5a 
d535d8922f571502252deaf607e82e7475cd1728
 112179 fail 8051789e982499050680a26febeada7467e18a8d 
b3e46a89147493d4474dafe983befca2d6500275 
d535d8922f571502252deaf607e82e7475cd1728
 112160 pass 8051789e982499050680a26febeada7467e18a8d 
f9dada2baabb639feb988b3a564df7a06d214e18 
b9cd216f74411a699c3e5ce3d25a375af37f096c
 112170 pass 8051789e982499050680a26febeada7467e18a8d 
cc5eff0186001790e6c15f155e376b9e9c43fc56 
d535d8922f571502252deaf607e82e7475cd1728
 112161 pass 8051789e982499050680a26febeada7467e18a8d 
acbaa0f4fd0491d222b718688244e629aa188b3c 
2b8a8a03f56e21381c7dd560b081002d357639e2
 112162 pass 8051789e982499050680a26febeada7467e18a8d 
346ebfc6fbf1a0c07cbdfc52beef0f0b9b5180d4 
2b8a8a03f56e21381c7dd560b081002d357639e2
 112163 pass 8051789e982499050680a26febeada7467e18a8d 
fbc8ea1ed0704b0cd393e5dc429816a36cb5ed05 
2b8a8a03f56e21381c7dd560b081002d357639e2
 112172 fail 8051789e982499050680a26febeada7467e18a8d 
b3e46a89147493d4474dafe983befca2d6500275 
d535d8922f571502252deaf607e82e7475cd1728
 112164 pass 8051789e982499050680a26febeada7467e18a8d 
1e886639791762e89b51aa0507f523c6a1448831 
2b8a8a03f56e21381c7dd560b081002d357639e2
 112183 fail 8051789e982499050680a26febeada7467e18a8d 
b3e46a89147493d4474dafe983befca2d6500275 
d535d8922f571502252deaf607e82e7475cd1728
 112166 pass 8051789e982499050680a26febeada7467e18a8d 
63dba7e6b18aa85392800f0cb7c897e3a05b097b 
2b8a8a03f56e21381c7dd560b081002d357639e2
 112174 pass 8051789e982499050680a26febeada7467e18a8d 
697b66d006676620a56fb5b79720ce457158204b 
b9cd216f74411a699c3e5ce3d25a375af37f096c
 112175 fail 8051789e982499050680a26febeada7467e18a8d 
5ba3d7564593c55292056ef5af84d50b55ebcf0e 
b9cd216f74411a699c3e5ce3d25a375af37f096c
 112185 fail 8051789e982499050680a26febeada7467e18a8d 
5ba3d7564593c55292056ef5af84d50b55ebcf0e 
b9cd216f74411a699c3e5ce3d25a375af37f096c
 112176 pass 8051789e982499050680a26febeada7467e18a8d 
759235653de427e4e7b62d8e6fb1ef9cb68bac7d 
b9cd216f74411a699c3e5ce3d25a375af37f096c
 112181 pass 8051789e982499050680a26febeada7467e18a8d 
a51568b78ea011e0f1e67664b8b0c6b693f8ee5a 
d535d8922f571502252deaf607e82e7475cd1728
Searching for interesting versions
 Result found: flight 111732 (pass), for basis pass
 Result found: flight 112100 (fail), for basis failure
 Repro found: flight 112154 (pass), for basis pass
 Repro found: flight 112156 (fail), for basis failure
 0 revisions at 8051789e982499050680a26febeada7467e18a8d 
759235653de427e4e7b62d8e6fb1ef9cb68bac7d 
b9cd216f74411a699c3e5ce3d25a375af37f096c
No revisions left to test, checking graph state.
 Result found: flight 112159 (pass), for last pass
 Result found: flight 112172 (fail), for first failure
 Repro found: flight 112178 (pass), for last pass
 Repro found: flight 112179 (fail), for first failure
 Repro found: flight 112181 (pass), for last pass
 Repro found: flight 112183 (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/112183/


  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>

 Result found: flight 112176 (pass), for last pass
 Result found: flight 112185 (fail), for first failure
Need to reproduce last pass (pass); had 1 already.
Should test 8051789e982499050680a26febeada7467e18a8d 
759235653de427e4e7b62d8e6fb1ef9cb68bac7d 
b9cd216f74411a699c3e5ce3d25a375af37f096c.
Revision graph left in 
/home/logs/results/bisect/qemu-mainline/build-i386.xen-build.{dot,ps,png,html,svg}.
Populating 112187...
Need 112187.build-i386 (like 112100)
 revision_qemu=8051789e982499050680a26febeada7467e18a8d
 revision_qemuu=759235653de427e4e7b62d8e6fb1ef9cb68bac7d
 revision_xen=b9cd216f74411a699c3e5ce3d25a375af37f096c
Creating 112187.build-i386 (copying from 112100)
Checking for flail (since 111732)...
Flight 112187 ready to go.

_______________________________________________
osstest-output mailing list
osstest-output@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/cgi-bin/mailman/listinfo/osstest-output