|
[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
|
 |
Lists.xenproject.org is hosted with RackSpace, monitoring our |