[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [qemu-upstream-4.11-testing bisection] complete test-arm64-arm64-xl-xsm
branch xen-4.11-testing xenbranch xen-4.11-testing job test-arm64-arm64-xl-xsm testid xen-boot Tree: linux git://xenbits.xen.org/linux-pvops.git Tree: linuxfirmware git://xenbits.xen.org/osstest/linux-firmware.git Tree: qemuu git://xenbits.xen.org/qemu-xen.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: e202feb7131e66ed9186ad8766c9582502c98998 Bug not present: 198672807ec6c6a1dee07841433a3064cb446a89 Last fail repro: http://logs.test-lab.xenproject.org/osstest/logs/135778/ commit e202feb7131e66ed9186ad8766c9582502c98998 Author: Andrew Cooper <andrew.cooper3@xxxxxxxxxx> Date: Fri Feb 1 11:34:35 2019 +0100 xen/cmdline: Fix buggy strncmp(s, LITERAL, ss - s) construct When the command line parsing was updated to use const strings and no longer tokenise with NUL characters, string matches could no longer be made with strcmp(). Unfortunately, the replacement was buggy. strncmp(s, "opt", ss - s) matches "o", "op" and "opt" on the command line, as ss - s may be shorter than the passed literal. Furthermore, parse_bool() is affected by this, so substrings such as "d", "e" and "o" are considered valid, with the latter being ambiguous between "on" and "off". Introduce a new strcmp-like function for the task, which looks for exact string matches, but declares success when the NUL of the literal matches a comma, colon or semicolon in the command line fragment. No change to the intended parsing functionality, but fixes cases where a partial string on the command line will inadvertently trigger options. A few areas were more than just a trivial change: * parse_irq_vector_map_param() gained some style corrections. * parse_vpmu_params() was rewritten to use the normal list-of-options form, rather than just fixing up parse_vpmu_param() and leaving the parsing being hard to follow. * Instead of making the trivial fix of adding an explicit length check in parse_bool(), use the length to select which token to we search for, which is more efficient than the previous linear search over all possible tokens. Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx> Reviewed-by: Jan Beulich <jbeulich@xxxxxxxx> Acked-by: Julien Grall <julien.grall@xxxxxxx> master commit: 2ddf7e3e341df3ccf21613ff7ffd4b7693abe9e9 master date: 2019-01-15 12:58:34 +0000 For bisection revision-tuple graph see: http://logs.test-lab.xenproject.org/osstest/results/bisect/qemu-upstream-4.11-testing/test-arm64-arm64-xl-xsm.xen-boot.html Revision IDs in each graph node refer, respectively, to the Trees above. ---------------------------------------- Running cs-bisection-step --graph-out=/home/logs/results/bisect/qemu-upstream-4.11-testing/test-arm64-arm64-xl-xsm.xen-boot --summary-out=tmp/135778.bisection-summary --basis-template=125575 --blessings=real,real-bisect qemu-upstream-4.11-testing test-arm64-arm64-xl-xsm xen-boot Searching for failure / basis pass: 135603 fail [host=rochester1] / 125575 [host=laxton1] 125534 [host=laxton1] 125508 [host=laxton1] 125465 [host=laxton0] 125345 [host=laxton1] 125273 [host=laxton0] template as basis? using template as basis. Failure / basis pass flights: 135603 / 125575 (tree in latest but not in basispass: ovmf) Tree: linux git://xenbits.xen.org/linux-pvops.git Tree: linuxfirmware git://xenbits.xen.org/osstest/linux-firmware.git Tree: qemuu git://xenbits.xen.org/qemu-xen.git Tree: xen git://xenbits.xen.org/xen.git Latest e64ac26749dc2c0f390caccd04274608ab31c8cf c530a75c1e6a472b0eb9558310b518f0dfcd8860 2871355a6957f1b3c16f858e3143e0fff0737b6a e983e8ae84efd5e43045a3d20a820f13cb4a75bf Basis pass 50484ee133c2219a26fd98b39770187221b5e0bc c530a75c1e6a472b0eb9558310b518f0dfcd8860 20c76f9a5fbf16d58c6add2ace2ff0fabd785926 1fd87ba1cd0312b743a48756a91c6962b1514aac Generating revisions with ./adhoc-revtuple-generator git://xenbits.xen.org/linux-pvops.git#50484ee133c2219a26fd98b39770187221b5e0bc-e64ac26749dc2c0f390caccd04274608ab31c8cf git://xenbits.xen.org/osstest/linux-firmware.git#c530a75c1e6a472b0eb9558310b518f0dfcd8860-c530a75c1e6a472b0eb9558310b518f0dfcd8860 git://xenbits.xen.org/qemu-xen.git#20c76f9a5fbf16d58c6add2ace2ff0fabd785926-2871355a6957f1b3c16f858e3143e0fff0737b6a git://xenbits.xen.org/xen.git#1fd87ba1cd0312b743a48756a91c6962b1514aac-e983e8a\ e84efd5e43045a3d20a820f13cb4a75bf adhoc-revtuple-generator: tree discontiguous: linux-pvops Loaded 2011 nodes in revision graph Searching for test results: 125273 [host=laxton0] 125345 [host=laxton1] 125465 [host=laxton0] 125534 [host=laxton1] 125508 [host=laxton1] 125575 [host=laxton1] 134697 fail 50484ee133c2219a26fd98b39770187221b5e0bc c530a75c1e6a472b0eb9558310b518f0dfcd8860 2871355a6957f1b3c16f858e3143e0fff0737b6a 718a8d2e9cbacb14f86f6dee40732727cd79bcff 134815 fail 50484ee133c2219a26fd98b39770187221b5e0bc c530a75c1e6a472b0eb9558310b518f0dfcd8860 2871355a6957f1b3c16f858e3143e0fff0737b6a 718a8d2e9cbacb14f86f6dee40732727cd79bcff 135029 fail e64ac26749dc2c0f390caccd04274608ab31c8cf c530a75c1e6a472b0eb9558310b518f0dfcd8860 2871355a6957f1b3c16f858e3143e0fff0737b6a 718a8d2e9cbacb14f86f6dee40732727cd79bcff 134961 fail 50484ee133c2219a26fd98b39770187221b5e0bc c530a75c1e6a472b0eb9558310b518f0dfcd8860 2871355a6957f1b3c16f858e3143e0fff0737b6a 718a8d2e9cbacb14f86f6dee40732727cd79bcff 135205 fail e64ac26749dc2c0f390caccd04274608ab31c8cf c530a75c1e6a472b0eb9558310b518f0dfcd8860 2871355a6957f1b3c16f858e3143e0fff0737b6a 348922b16a0901ccc7b8980c92703b2ce20d1a2d 135446 fail e64ac26749dc2c0f390caccd04274608ab31c8cf c530a75c1e6a472b0eb9558310b518f0dfcd8860 2871355a6957f1b3c16f858e3143e0fff0737b6a 348922b16a0901ccc7b8980c92703b2ce20d1a2d 135418 fail e64ac26749dc2c0f390caccd04274608ab31c8cf c530a75c1e6a472b0eb9558310b518f0dfcd8860 2871355a6957f1b3c16f858e3143e0fff0737b6a 348922b16a0901ccc7b8980c92703b2ce20d1a2d 135603 fail e64ac26749dc2c0f390caccd04274608ab31c8cf c530a75c1e6a472b0eb9558310b518f0dfcd8860 2871355a6957f1b3c16f858e3143e0fff0737b6a e983e8ae84efd5e43045a3d20a820f13cb4a75bf 135750 pass 50484ee133c2219a26fd98b39770187221b5e0bc c530a75c1e6a472b0eb9558310b518f0dfcd8860 20c76f9a5fbf16d58c6add2ace2ff0fabd785926 1fd87ba1cd0312b743a48756a91c6962b1514aac 135769 fail 50484ee133c2219a26fd98b39770187221b5e0bc c530a75c1e6a472b0eb9558310b518f0dfcd8860 20c76f9a5fbf16d58c6add2ace2ff0fabd785926 514dccd049f8338495b6c992b9c6a1ec2ece2932 135771 fail 50484ee133c2219a26fd98b39770187221b5e0bc c530a75c1e6a472b0eb9558310b518f0dfcd8860 20c76f9a5fbf16d58c6add2ace2ff0fabd785926 e202feb7131e66ed9186ad8766c9582502c98998 135752 fail e64ac26749dc2c0f390caccd04274608ab31c8cf c530a75c1e6a472b0eb9558310b518f0dfcd8860 2871355a6957f1b3c16f858e3143e0fff0737b6a e983e8ae84efd5e43045a3d20a820f13cb4a75bf 135736 pass 50484ee133c2219a26fd98b39770187221b5e0bc c530a75c1e6a472b0eb9558310b518f0dfcd8860 20c76f9a5fbf16d58c6add2ace2ff0fabd785926 1fd87ba1cd0312b743a48756a91c6962b1514aac 135759 pass 50484ee133c2219a26fd98b39770187221b5e0bc c530a75c1e6a472b0eb9558310b518f0dfcd8860 20c76f9a5fbf16d58c6add2ace2ff0fabd785926 ff9f8730a993b0ffac06af28af0b0132e1f123fe 135740 fail e64ac26749dc2c0f390caccd04274608ab31c8cf c530a75c1e6a472b0eb9558310b518f0dfcd8860 2871355a6957f1b3c16f858e3143e0fff0737b6a 348922b16a0901ccc7b8980c92703b2ce20d1a2d 135743 pass 50484ee133c2219a26fd98b39770187221b5e0bc c530a75c1e6a472b0eb9558310b518f0dfcd8860 20c76f9a5fbf16d58c6add2ace2ff0fabd785926 1fd87ba1cd0312b743a48756a91c6962b1514aac 135745 fail e64ac26749dc2c0f390caccd04274608ab31c8cf c530a75c1e6a472b0eb9558310b518f0dfcd8860 2871355a6957f1b3c16f858e3143e0fff0737b6a e983e8ae84efd5e43045a3d20a820f13cb4a75bf 135773 pass 50484ee133c2219a26fd98b39770187221b5e0bc c530a75c1e6a472b0eb9558310b518f0dfcd8860 20c76f9a5fbf16d58c6add2ace2ff0fabd785926 198672807ec6c6a1dee07841433a3064cb446a89 135760 fail 50484ee133c2219a26fd98b39770187221b5e0bc c530a75c1e6a472b0eb9558310b518f0dfcd8860 20c76f9a5fbf16d58c6add2ace2ff0fabd785926 dd492b8f64d9ad9d413289565bd074f025f8edde 135776 fail 50484ee133c2219a26fd98b39770187221b5e0bc c530a75c1e6a472b0eb9558310b518f0dfcd8860 20c76f9a5fbf16d58c6add2ace2ff0fabd785926 e202feb7131e66ed9186ad8766c9582502c98998 135764 pass 50484ee133c2219a26fd98b39770187221b5e0bc c530a75c1e6a472b0eb9558310b518f0dfcd8860 20c76f9a5fbf16d58c6add2ace2ff0fabd785926 14b7dc115b8a5d90cf842ac003100542408f671c 135766 pass 50484ee133c2219a26fd98b39770187221b5e0bc c530a75c1e6a472b0eb9558310b518f0dfcd8860 20c76f9a5fbf16d58c6add2ace2ff0fabd785926 63d71138a4d3521cf42ff28b0dd3e79b82d79230 135768 pass 50484ee133c2219a26fd98b39770187221b5e0bc c530a75c1e6a472b0eb9558310b518f0dfcd8860 20c76f9a5fbf16d58c6add2ace2ff0fabd785926 198672807ec6c6a1dee07841433a3064cb446a89 135777 pass 50484ee133c2219a26fd98b39770187221b5e0bc c530a75c1e6a472b0eb9558310b518f0dfcd8860 20c76f9a5fbf16d58c6add2ace2ff0fabd785926 198672807ec6c6a1dee07841433a3064cb446a89 135778 fail 50484ee133c2219a26fd98b39770187221b5e0bc c530a75c1e6a472b0eb9558310b518f0dfcd8860 20c76f9a5fbf16d58c6add2ace2ff0fabd785926 e202feb7131e66ed9186ad8766c9582502c98998 Searching for interesting versions Result found: flight 135736 (pass), for basis pass Result found: flight 135745 (fail), for basis failure Repro found: flight 135750 (pass), for basis pass Repro found: flight 135752 (fail), for basis failure 0 revisions at 50484ee133c2219a26fd98b39770187221b5e0bc c530a75c1e6a472b0eb9558310b518f0dfcd8860 20c76f9a5fbf16d58c6add2ace2ff0fabd785926 198672807ec6c6a1dee07841433a3064cb446a89 No revisions left to test, checking graph state. Result found: flight 135768 (pass), for last pass Result found: flight 135771 (fail), for first failure Repro found: flight 135773 (pass), for last pass Repro found: flight 135776 (fail), for first failure Repro found: flight 135777 (pass), for last pass Repro found: flight 135778 (fail), for first failure *** Found and reproduced problem changeset *** Bug is in tree: xen git://xenbits.xen.org/xen.git Bug introduced: e202feb7131e66ed9186ad8766c9582502c98998 Bug not present: 198672807ec6c6a1dee07841433a3064cb446a89 Last fail repro: http://logs.test-lab.xenproject.org/osstest/logs/135778/ commit e202feb7131e66ed9186ad8766c9582502c98998 Author: Andrew Cooper <andrew.cooper3@xxxxxxxxxx> Date: Fri Feb 1 11:34:35 2019 +0100 xen/cmdline: Fix buggy strncmp(s, LITERAL, ss - s) construct When the command line parsing was updated to use const strings and no longer tokenise with NUL characters, string matches could no longer be made with strcmp(). Unfortunately, the replacement was buggy. strncmp(s, "opt", ss - s) matches "o", "op" and "opt" on the command line, as ss - s may be shorter than the passed literal. Furthermore, parse_bool() is affected by this, so substrings such as "d", "e" and "o" are considered valid, with the latter being ambiguous between "on" and "off". Introduce a new strcmp-like function for the task, which looks for exact string matches, but declares success when the NUL of the literal matches a comma, colon or semicolon in the command line fragment. No change to the intended parsing functionality, but fixes cases where a partial string on the command line will inadvertently trigger options. A few areas were more than just a trivial change: * parse_irq_vector_map_param() gained some style corrections. * parse_vpmu_params() was rewritten to use the normal list-of-options form, rather than just fixing up parse_vpmu_param() and leaving the parsing being hard to follow. * Instead of making the trivial fix of adding an explicit length check in parse_bool(), use the length to select which token to we search for, which is more efficient than the previous linear search over all possible tokens. Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx> Reviewed-by: Jan Beulich <jbeulich@xxxxxxxx> Acked-by: Julien Grall <julien.grall@xxxxxxx> master commit: 2ddf7e3e341df3ccf21613ff7ffd4b7693abe9e9 master date: 2019-01-15 12:58:34 +0000 Revision graph left in /home/logs/results/bisect/qemu-upstream-4.11-testing/test-arm64-arm64-xl-xsm.xen-boot.{dot,ps,png,html,svg}. ---------------------------------------- 135778: tolerable ALL FAIL flight 135778 qemu-upstream-4.11-testing real-bisect [real] http://logs.test-lab.xenproject.org/osstest/logs/135778/ Failures :-/ but no regressions. Tests which did not succeed, including tests which could not be run: test-arm64-arm64-xl-xsm 7 xen-boot fail baseline untested jobs: test-arm64-arm64-xl-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 _______________________________________________ osstest-output mailing list osstest-output@xxxxxxxxxxxxxxxxxxxx https://lists.xenproject.org/mailman/listinfo/osstest-output
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |