[Xen-devel] [qemu-upstream-4.11-testing bisection] complete test-arm64-arm64-xl-xsm

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

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


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel