From: osstest service owner <osstest-admin@xenproject.org>
To: xen-devel@lists.xensource.com, osstest-admin@xenproject.org
Subject: [qemu-mainline bisection] complete build-armhf
Date: Sun, 23 Jul 2017 23:33:05 +0000 [thread overview]
Message-ID: <E1dZQMz-0008Ju-3z@osstest.test-lab.xenproject.org> (raw)
branch xen-unstable
xenbranch xen-unstable
job build-armhf
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/112254/
commit b3e46a89147493d4474dafe983befca2d6500275
Merge: a51568b 331b518
Author: Peter Maydell <peter.maydell@linaro.org>
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@eu.citrix.com>"
# gpg: aka "Stefano Stabellini <sstabellini@kernel.org>"
# 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@linaro.org>
commit 331b5189d756d431b1d18ae7097527ba3d3ea809
Author: Igor Druzhinin <igor.druzhinin@citrix.com>
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@citrix.com>
Reviewed-by: Paul Durrant <paul.durrant@citrix.com>
Reviewed-by: Stefano Stabellini <sstabellini@kernel.org>
Signed-off-by: Stefano Stabellini <sstabellini@kernel.org>
commit 5ba3d7564593c55292056ef5af84d50b55ebcf0e
Author: Igor Druzhinin <igor.druzhinin@citrix.com>
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@lists.xen.org/msg113007.html
Signed-off-by: Igor Druzhinin <igor.druzhinin@citrix.com>
Reviewed-by: Paul Durrant <paul.durrant@citrix.com>
Reviewed-by: Stefano Stabellini <sstabellini@kernel.org>
Signed-off-by: Stefano Stabellini <sstabellini@kernel.org>
commit 759235653de427e4e7b62d8e6fb1ef9cb68bac7d
Author: Igor Druzhinin <igor.druzhinin@citrix.com>
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@citrix.com>
Reviewed-by: Paul Durrant <paul.durrant@citrix.com>
Reviewed-by: Stefano Stabellini <sstabellini@kernel.org>
commit 697b66d006676620a56fb5b79720ce457158204b
Author: Igor Druzhinin <igor.druzhinin@citrix.com>
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@citrix.com>
Reviewed-by: Stefano Stabellini <sstabellini@kernel.org>
Reviewed-by: Paul Durrant <paul.durrant@citrix.com>
commit 04d6da4ff6084a3cb1b7a981769d9aa17e469348
Author: Stefano Stabellini <sstabellini@kernel.org>
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@citrix.com>
Reviewed-by: Stefano Stabellini <sstabellini@kernel.org>
Signed-off-by: Stefano Stabellini <sstabellini@kernel.org>
commit 64c7c1175b4e3e6fe005934cde63259d8adad392
Author: Peter Maydell <peter.maydell@linaro.org>
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@linaro.org>
Reviewed-by: Stefano Stabellini <sstabellini@kernel.org>
Signed-off-by: Stefano Stabellini <sstabellini@kernel.org>
commit a19bae42e341a05f43f685fc9ff0e19ba6f129c3
Author: Xiong Zhang <xiong.y.zhang@intel.com>
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@intel.com>
Acked-by: Anthony PERARD <anthony.perard@citrix.com>
Signed-off-by: Stefano Stabellini <sstabellini@kernel.org>
For bisection revision-tuple graph see:
http://logs.test-lab.xenproject.org/osstest/results/bisect/qemu-mainline/build-armhf.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.xen-build --summary-out=tmp/112254.bisection-summary --basis-template=111765 --blessings=real,real-bisect qemu-mainline build-armhf xen-build
Searching for failure / basis pass:
112155 fail [host=arndale-lakeside] / 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
112037 [host=arndale-bluewater]
112041 [host=arndale-bluewater]
112072 fail 25d0233c1ac6cd14a15fcc834f1de3b179037b1d d535d8922f571502252deaf607e82e7475cd1728
112100 [host=arndale-metrocentre]
112155 fail 91939262ffcd3c85ea6a4793d3029326eea1d649 d535d8922f571502252deaf607e82e7475cd1728
112251 fail b3e46a89147493d4474dafe983befca2d6500275 d535d8922f571502252deaf607e82e7475cd1728
112252 pass a51568b78ea011e0f1e67664b8b0c6b693f8ee5a d535d8922f571502252deaf607e82e7475cd1728
112254 fail b3e46a89147493d4474dafe983befca2d6500275 d535d8922f571502252deaf607e82e7475cd1728
112238 pass 63cb55783c5e8f783b1dcebd3a2935941f872d44 2b8a8a03f56e21381c7dd560b081002d357639e2
112240 fail 91939262ffcd3c85ea6a4793d3029326eea1d649 d535d8922f571502252deaf607e82e7475cd1728
112241 fail 9c489ea6bed134fecfd556b439c68bba48fbe102 d535d8922f571502252deaf607e82e7475cd1728
112243 fail 810d5cad4087236236e00fd3046a16adf26e9060 d535d8922f571502252deaf607e82e7475cd1728
112244 pass 6d60e295ef020759a03b90724d0342012c189ba2 d535d8922f571502252deaf607e82e7475cd1728
112246 fail 824dbfb45d4834c6538f6f70737014b1f19e2e55 d535d8922f571502252deaf607e82e7475cd1728
112247 pass a51568b78ea011e0f1e67664b8b0c6b693f8ee5a d535d8922f571502252deaf607e82e7475cd1728
112248 fail b3e46a89147493d4474dafe983befca2d6500275 d535d8922f571502252deaf607e82e7475cd1728
112249 pass a51568b78ea011e0f1e67664b8b0c6b693f8ee5a d535d8922f571502252deaf607e82e7475cd1728
Searching for interesting versions
Result found: flight 112011 (pass), for basis pass
Result found: flight 112155 (fail), for basis failure
Repro found: flight 112238 (pass), for basis pass
Repro found: flight 112240 (fail), for basis failure
0 revisions at a51568b78ea011e0f1e67664b8b0c6b693f8ee5a d535d8922f571502252deaf607e82e7475cd1728
No revisions left to test, checking graph state.
Result found: flight 112247 (pass), for last pass
Result found: flight 112248 (fail), for first failure
Repro found: flight 112249 (pass), for last pass
Repro found: flight 112251 (fail), for first failure
Repro found: flight 112252 (pass), for last pass
Repro found: flight 112254 (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/112254/
commit b3e46a89147493d4474dafe983befca2d6500275
Merge: a51568b 331b518
Author: Peter Maydell <peter.maydell@linaro.org>
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@eu.citrix.com>"
# gpg: aka "Stefano Stabellini <sstabellini@kernel.org>"
# 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@linaro.org>
commit 331b5189d756d431b1d18ae7097527ba3d3ea809
Author: Igor Druzhinin <igor.druzhinin@citrix.com>
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@citrix.com>
Reviewed-by: Paul Durrant <paul.durrant@citrix.com>
Reviewed-by: Stefano Stabellini <sstabellini@kernel.org>
Signed-off-by: Stefano Stabellini <sstabellini@kernel.org>
commit 5ba3d7564593c55292056ef5af84d50b55ebcf0e
Author: Igor Druzhinin <igor.druzhinin@citrix.com>
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@lists.xen.org/msg113007.html
Signed-off-by: Igor Druzhinin <igor.druzhinin@citrix.com>
Reviewed-by: Paul Durrant <paul.durrant@citrix.com>
Reviewed-by: Stefano Stabellini <sstabellini@kernel.org>
Signed-off-by: Stefano Stabellini <sstabellini@kernel.org>
commit 759235653de427e4e7b62d8e6fb1ef9cb68bac7d
Author: Igor Druzhinin <igor.druzhinin@citrix.com>
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@citrix.com>
Reviewed-by: Paul Durrant <paul.durrant@citrix.com>
Reviewed-by: Stefano Stabellini <sstabellini@kernel.org>
commit 697b66d006676620a56fb5b79720ce457158204b
Author: Igor Druzhinin <igor.druzhinin@citrix.com>
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@citrix.com>
Reviewed-by: Stefano Stabellini <sstabellini@kernel.org>
Reviewed-by: Paul Durrant <paul.durrant@citrix.com>
commit 04d6da4ff6084a3cb1b7a981769d9aa17e469348
Author: Stefano Stabellini <sstabellini@kernel.org>
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@citrix.com>
Reviewed-by: Stefano Stabellini <sstabellini@kernel.org>
Signed-off-by: Stefano Stabellini <sstabellini@kernel.org>
commit 64c7c1175b4e3e6fe005934cde63259d8adad392
Author: Peter Maydell <peter.maydell@linaro.org>
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@linaro.org>
Reviewed-by: Stefano Stabellini <sstabellini@kernel.org>
Signed-off-by: Stefano Stabellini <sstabellini@kernel.org>
commit a19bae42e341a05f43f685fc9ff0e19ba6f129c3
Author: Xiong Zhang <xiong.y.zhang@intel.com>
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@intel.com>
Acked-by: Anthony PERARD <anthony.perard@citrix.com>
Signed-off-by: Stefano Stabellini <sstabellini@kernel.org>
Revision graph left in /home/logs/results/bisect/qemu-mainline/build-armhf.xen-build.{dot,ps,png,html,svg}.
----------------------------------------
112254: tolerable ALL FAIL
flight 112254 qemu-mainline real-bisect [real]
http://logs.test-lab.xenproject.org/osstest/logs/112254/
Failures :-/ but no regressions.
Tests which did not succeed,
including tests which could not be run:
build-armhf 6 xen-build fail baseline untested
jobs:
build-armhf 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.xen.org
https://lists.xen.org/xen-devel
next reply other threads:[~2017-07-23 23:33 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-07-23 23:33 osstest service owner [this message]
-- strict thread matches above, loose matches on Subject: below --
2023-09-13 0:14 [qemu-mainline bisection] complete build-armhf osstest service owner
2023-05-23 0:35 osstest service owner
2023-01-11 0:38 osstest service owner
2021-10-27 18:17 osstest service owner
2020-10-21 9:37 osstest service owner
2019-05-19 8:15 osstest service owner
2017-11-01 4:14 osstest service owner
2017-05-02 17:59 osstest service owner
2017-02-06 15:52 osstest service owner
2015-01-22 3:21 xen.org
2015-01-22 10:04 ` Ian Campbell
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=E1dZQMz-0008Ju-3z@osstest.test-lab.xenproject.org \
--to=osstest-admin@xenproject.org \
--cc=xen-devel@lists.xensource.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).