|
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [Xen-devel] [xen-unstable bisection] complete test-amd64-i386-qemut-rhel6hvm-amd
branch xen-unstable
xen branch xen-unstable
job test-amd64-i386-qemut-rhel6hvm-amd
test redhat-install
Tree: linux git://xenbits.xen.org/linux-pvops.git
Tree: linuxfirmware git://xenbits.xen.org/osstest/linux-firmware.git
Tree: qemu git://xenbits.xen.org/staging/qemu-xen-unstable.git
Tree: qemuu git://xenbits.xen.org/staging/qemu-upstream-unstable.git
Tree: xen git://xenbits.xen.org/xen.git
*** Found and reproduced problem changeset ***
Bug is in tree: xen git://xenbits.xen.org/xen.git
Bug introduced: 8bad6c5626129ffba04dbab3a38115b6f3669596
Bug not present: 8d134c2e12730a4a3dce9873f4671f6dd8860baf
commit 8bad6c5626129ffba04dbab3a38115b6f3669596
Author: Jan Beulich <jbeulich@xxxxxxxx>
Date: Fri Mar 28 13:31:23 2014 +0100
x86/HVM: fix preemption handling in do_hvm_op()
Just like previously done for some mem-op hypercalls, undo preemption
using the interface structures (altering it in ways the caller may not
expect) and replace it by storing the continuation point in the high
bits of sub-operation argument.
This also changes the "nr" fields of struct xen_hvm_track_dirty_vram
(operation already limited to 1Gb worth of pages) and struct
xen_hvm_modified_memory to be only 32 bits wide, consistent with those
of struct xen_set_mem{type,access}. If that's not acceptable for some
reason, we'd need to shrink the HVMOP_op_bits (while still enforcing
the [then higher] limit resulting from the need to be able to encode
the continuation).
Whether (and if so how) to adjust xc_hvm_track_dirty_vram(),
xc_hvm_modified_memory(), xc_hvm_set_mem_type(), and
xc_hvm_set_mem_access() to reflect the 32-bit restriction on "nr" is
unclear: If the APIs need to remain stable, all four functions should
probably check that there was no truncation. Preferably their
parameters would be changed to uint32_t or unsigned int, though.
As a minor cleanup, along with introducing the switch-wide "pfn" the
redundant "d" is also being converted to a switch-wide one.
Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
Reviewed-by: Tim Deegan <tim@xxxxxxx>
For bisection revision-tuple graph see:
http://www.chiark.greenend.org.uk/~xensrcts/results/bisect.xen-unstable.test-amd64-i386-qemut-rhel6hvm-amd.redhat-install.html
Revision IDs in each graph node refer, respectively, to the Trees above.
----------------------------------------
Searching for failure / basis pass:
25693 fail [host=lace-bug] / 25685 [host=moss-bug] 25684 [host=potato-beetle]
25679 [host=leaf-beetle] 25676 ok.
Failure / basis pass flights: 25693 / 25676
Tree: linux git://xenbits.xen.org/linux-pvops.git
Tree: linuxfirmware git://xenbits.xen.org/osstest/linux-firmware.git
Tree: qemu git://xenbits.xen.org/staging/qemu-xen-unstable.git
Tree: qemuu git://xenbits.xen.org/staging/qemu-upstream-unstable.git
Tree: xen git://xenbits.xen.org/xen.git
Latest b1cee752ee0c04f424cb71ba0c8013c793008296
c530a75c1e6a472b0eb9558310b518f0dfcd8860
7f5b3c338e0f8938ba575dec18255dcbee0c2ee2
65fc9b78ba3d868a26952db0d8e51cecf01d47b4
b021348231e942a342fd82e7a60193256236274d
Basis pass b1cee752ee0c04f424cb71ba0c8013c793008296
c530a75c1e6a472b0eb9558310b518f0dfcd8860
7f5b3c338e0f8938ba575dec18255dcbee0c2ee2
65fc9b78ba3d868a26952db0d8e51cecf01d47b4
29f7ec2a9231352de05523e88619e9cef9ed695c
Generating revisions with ./adhoc-revtuple-generator
git://xenbits.xen.org/linux-pvops.git#b1cee752ee0c04f424cb71ba0c8013c793008296-b1cee752ee0c04f424cb71ba0c8013c793008296
git://xenbits.xen.org/osstest/linux-firmware.git#c530a75c1e6a472b0eb9558310b518f0dfcd8860-c530a75c1e6a472b0eb9558310b518f0dfcd8860
git://xenbits.xen.org/staging/qemu-xen-unstable.git#7f5b3c338e0f8938ba575dec18255dcbee0c2ee2-7f5b3c338e0f8938ba575dec18255dcbee0c2ee2
git://xenbits.xen.org/staging/qemu-upstream-unstable.git#65fc9b78ba3d868a26952db0d8e51cecf01d47b4-65fc9b78ba3d868a26952db0d8e51cecf01d47b4
git://xenbits.xen.org/xen.git#29f7ec2a9231352de05523e88619e9cef9ed695c-b021348231e942a342fd82e7a60193256236274d
Cloning into bare repository /export/home/osstest/repos/xen...
Cloning into bare repository /export/home/osstest/repos/xen...
Loaded 1001 nodes in revision graph
Searching for test results:
25672 []
25675 []
25676 pass b1cee752ee0c04f424cb71ba0c8013c793008296
c530a75c1e6a472b0eb9558310b518f0dfcd8860
7f5b3c338e0f8938ba575dec18255dcbee0c2ee2
65fc9b78ba3d868a26952db0d8e51cecf01d47b4
29f7ec2a9231352de05523e88619e9cef9ed695c
25679 [host=leaf-beetle]
25684 [host=potato-beetle]
25685 [host=moss-bug]
25689 fail b1cee752ee0c04f424cb71ba0c8013c793008296
c530a75c1e6a472b0eb9558310b518f0dfcd8860
7f5b3c338e0f8938ba575dec18255dcbee0c2ee2
65fc9b78ba3d868a26952db0d8e51cecf01d47b4
b021348231e942a342fd82e7a60193256236274d
25692 pass b1cee752ee0c04f424cb71ba0c8013c793008296
c530a75c1e6a472b0eb9558310b518f0dfcd8860
7f5b3c338e0f8938ba575dec18255dcbee0c2ee2
65fc9b78ba3d868a26952db0d8e51cecf01d47b4
29f7ec2a9231352de05523e88619e9cef9ed695c
25696 fail b1cee752ee0c04f424cb71ba0c8013c793008296
c530a75c1e6a472b0eb9558310b518f0dfcd8860
7f5b3c338e0f8938ba575dec18255dcbee0c2ee2
65fc9b78ba3d868a26952db0d8e51cecf01d47b4
b021348231e942a342fd82e7a60193256236274d
25698 pass b1cee752ee0c04f424cb71ba0c8013c793008296
c530a75c1e6a472b0eb9558310b518f0dfcd8860
7f5b3c338e0f8938ba575dec18255dcbee0c2ee2
65fc9b78ba3d868a26952db0d8e51cecf01d47b4
9011c2615c18b92f10cda8d78622e0c2a9e1f846
25700 fail b1cee752ee0c04f424cb71ba0c8013c793008296
c530a75c1e6a472b0eb9558310b518f0dfcd8860
7f5b3c338e0f8938ba575dec18255dcbee0c2ee2
65fc9b78ba3d868a26952db0d8e51cecf01d47b4
e5ae6eefdfbc1816b050d02998f69f0b78d5c814
25702 fail b1cee752ee0c04f424cb71ba0c8013c793008296
c530a75c1e6a472b0eb9558310b518f0dfcd8860
7f5b3c338e0f8938ba575dec18255dcbee0c2ee2
65fc9b78ba3d868a26952db0d8e51cecf01d47b4
8bad6c5626129ffba04dbab3a38115b6f3669596
25704 pass b1cee752ee0c04f424cb71ba0c8013c793008296
c530a75c1e6a472b0eb9558310b518f0dfcd8860
7f5b3c338e0f8938ba575dec18255dcbee0c2ee2
65fc9b78ba3d868a26952db0d8e51cecf01d47b4
8d134c2e12730a4a3dce9873f4671f6dd8860baf
25693 fail b1cee752ee0c04f424cb71ba0c8013c793008296
c530a75c1e6a472b0eb9558310b518f0dfcd8860
7f5b3c338e0f8938ba575dec18255dcbee0c2ee2
65fc9b78ba3d868a26952db0d8e51cecf01d47b4
b021348231e942a342fd82e7a60193256236274d
25707 fail b1cee752ee0c04f424cb71ba0c8013c793008296
c530a75c1e6a472b0eb9558310b518f0dfcd8860
7f5b3c338e0f8938ba575dec18255dcbee0c2ee2
65fc9b78ba3d868a26952db0d8e51cecf01d47b4
8bad6c5626129ffba04dbab3a38115b6f3669596
25709 pass b1cee752ee0c04f424cb71ba0c8013c793008296
c530a75c1e6a472b0eb9558310b518f0dfcd8860
7f5b3c338e0f8938ba575dec18255dcbee0c2ee2
65fc9b78ba3d868a26952db0d8e51cecf01d47b4
8d134c2e12730a4a3dce9873f4671f6dd8860baf
25713 fail b1cee752ee0c04f424cb71ba0c8013c793008296
c530a75c1e6a472b0eb9558310b518f0dfcd8860
7f5b3c338e0f8938ba575dec18255dcbee0c2ee2
65fc9b78ba3d868a26952db0d8e51cecf01d47b4
8bad6c5626129ffba04dbab3a38115b6f3669596
25715 pass b1cee752ee0c04f424cb71ba0c8013c793008296
c530a75c1e6a472b0eb9558310b518f0dfcd8860
7f5b3c338e0f8938ba575dec18255dcbee0c2ee2
65fc9b78ba3d868a26952db0d8e51cecf01d47b4
8d134c2e12730a4a3dce9873f4671f6dd8860baf
25716 fail b1cee752ee0c04f424cb71ba0c8013c793008296
c530a75c1e6a472b0eb9558310b518f0dfcd8860
7f5b3c338e0f8938ba575dec18255dcbee0c2ee2
65fc9b78ba3d868a26952db0d8e51cecf01d47b4
8bad6c5626129ffba04dbab3a38115b6f3669596
Searching for interesting versions
Result found: flight 25676 (pass), for basis pass
Result found: flight 25689 (fail), for basis failure
Repro found: flight 25692 (pass), for basis pass
Repro found: flight 25693 (fail), for basis failure
0 revisions at b1cee752ee0c04f424cb71ba0c8013c793008296
c530a75c1e6a472b0eb9558310b518f0dfcd8860
7f5b3c338e0f8938ba575dec18255dcbee0c2ee2
65fc9b78ba3d868a26952db0d8e51cecf01d47b4
8d134c2e12730a4a3dce9873f4671f6dd8860baf
No revisions left to test, checking graph state.
Result found: flight 25704 (pass), for last pass
Result found: flight 25707 (fail), for first failure
Repro found: flight 25709 (pass), for last pass
Repro found: flight 25713 (fail), for first failure
Repro found: flight 25715 (pass), for last pass
Repro found: flight 25716 (fail), for first failure
*** Found and reproduced problem changeset ***
Bug is in tree: xen git://xenbits.xen.org/xen.git
Bug introduced: 8bad6c5626129ffba04dbab3a38115b6f3669596
Bug not present: 8d134c2e12730a4a3dce9873f4671f6dd8860baf
Cloning into bare repository /export/home/osstest/repos/xen...
commit 8bad6c5626129ffba04dbab3a38115b6f3669596
Author: Jan Beulich <jbeulich@xxxxxxxx>
Date: Fri Mar 28 13:31:23 2014 +0100
x86/HVM: fix preemption handling in do_hvm_op()
Just like previously done for some mem-op hypercalls, undo preemption
using the interface structures (altering it in ways the caller may not
expect) and replace it by storing the continuation point in the high
bits of sub-operation argument.
This also changes the "nr" fields of struct xen_hvm_track_dirty_vram
(operation already limited to 1Gb worth of pages) and struct
xen_hvm_modified_memory to be only 32 bits wide, consistent with those
of struct xen_set_mem{type,access}. If that's not acceptable for some
reason, we'd need to shrink the HVMOP_op_bits (while still enforcing
the [then higher] limit resulting from the need to be able to encode
the continuation).
Whether (and if so how) to adjust xc_hvm_track_dirty_vram(),
xc_hvm_modified_memory(), xc_hvm_set_mem_type(), and
xc_hvm_set_mem_access() to reflect the 32-bit restriction on "nr" is
unclear: If the APIs need to remain stable, all four functions should
probably check that there was no truncation. Preferably their
parameters would be changed to uint32_t or unsigned int, though.
As a minor cleanup, along with introducing the switch-wide "pfn" the
redundant "d" is also being converted to a switch-wide one.
Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
Reviewed-by: Tim Deegan <tim@xxxxxxx>
Revision graph left in
/home/xc_osstest/results/bisect.xen-unstable.test-amd64-i386-qemut-rhel6hvm-amd.redhat-install.{dot,ps,png,html}.
----------------------------------------
25716: tolerable ALL FAIL
flight 25716 xen-unstable real-bisect [real]
http://www.chiark.greenend.org.uk/~xensrcts/logs/25716/
Failures :-/ but no regressions.
Tests which did not succeed,
including tests which could not be run:
test-amd64-i386-qemut-rhel6hvm-amd 7 redhat-install fail baseline untested
jobs:
test-amd64-i386-qemut-rhel6hvm-amd fail
------------------------------------------------------------
sg-report-flight on osstest.cam.xci-test.com
logs: /home/xc_osstest/logs
images: /home/xc_osstest/images
Logs, config files, etc. are available at
http://www.chiark.greenend.org.uk/~xensrcts/logs
Test harness code can be found at
http://xenbits.xensource.com/gitweb?p=osstest.git;a=summary
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel
|
![]() |
Lists.xenproject.org is hosted with RackSpace, monitoring our |