|
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [Xen-devel] [linux-3.18 bisection] complete test-amd64-amd64-pair
branch xen-unstable xenbranch xen-unstable job test-amd64-amd64-pair testid xen-boot/dst_host Tree: linux git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git Tree: linuxfirmware git://xenbits.xen.org/osstest/linux-firmware.git Tree: qemu git://xenbits.xen.org/qemu-xen-traditional.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: linux git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git Bug introduced: 7b8052e19304865477e03a0047062d977309a22f Bug not present: d255d18a34a8d53ccc4a019dc07e17b6e8cf6bd1 Last fail repro: http://logs.test-lab.xenproject.org/osstest/logs/131278/ commit 7b8052e19304865477e03a0047062d977309a22f Author: Jan Beulich <JBeulich@xxxxxxxx> Date: Mon Oct 19 04:23:29 2015 -0600 igb: fix NULL derefs due to skipped SR-IOV enabling [ Upstream commit be06998f96ecb93938ad2cce46c4289bf7cf45bc ] The combined effect of commits 6423fc3416 ("igb: do not re-init SR-IOV during probe") and ceee3450b3 ("igb: make sure SR-IOV init uses the right number of queues") causes VFs no longer getting set up, leading to NULL pointer dereferences due to the adapter's ->vf_data being NULL while ->vfs_allocated_count is non-zero. The first commit not only neglected the side effect of igb_sriov_reinit() that the second commit tried to account for, but also that of setting IGB_FLAG_HAS_MSIX, without which igb_enable_sriov() is effectively a no-op. Calling igb_{,re}set_interrupt_capability() as done here seems to address this, but I'm not sure whether this is better than sinply reverting the other two commits. Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx> Tested-by: Aaron Brown <aaron.f.brown@xxxxxxxxx> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@xxxxxxxxx> Signed-off-by: Sasha Levin <sashal@xxxxxxxxxx> For bisection revision-tuple graph see: http://logs.test-lab.xenproject.org/osstest/results/bisect/linux-3.18/test-amd64-amd64-pair.xen-boot--dst_host.html Revision IDs in each graph node refer, respectively, to the Trees above. ---------------------------------------- Running cs-bisection-step --graph-out=/home/logs/results/bisect/linux-3.18/test-amd64-amd64-pair.xen-boot--dst_host --summary-out=tmp/131280.bisection-summary --basis-template=128858 --blessings=real,real-bisect --flight=131280 linux-3.18 test-amd64-amd64-pair xen-boot/dst_host Searching for failure / basis pass: 131231 fail [dst_host=debina1,src_host=debina0] / 130367 [dst_host=albana0,src_host=albana1] 130203 [dst_host=albana0,src_host=albana1] 130067 [dst_host=pinot1,src_host=pinot0] 129845 [dst_host=albana1,src_host=albana0] 129760 [dst_host=godello1,src_host=godello0] 128858 [dst_host=elbling0,src_host=elbling1] 128841 [dst_host=elbling0,src_host=elbling1] 128807 [dst_host=godello1,src_host=godello0] 128691 [dst_host=fiano1,src_host=fiano0] 128258 [dst_host=fiano1,src_host=fiano0] 128232 [dst_host=albana1,src_host=albana0] 128177 ok. Failure / basis pass flights: 131231 / 128177 (tree with no url: minios) (tree with no url: ovmf) (tree with no url: seabios) Tree: linux git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git Tree: linuxfirmware git://xenbits.xen.org/osstest/linux-firmware.git Tree: qemu git://xenbits.xen.org/qemu-xen-traditional.git Tree: qemuu git://xenbits.xen.org/qemu-xen.git Tree: xen git://xenbits.xen.org/xen.git Latest 3879c163e8681939b1d93139521aee983623884f c530a75c1e6a472b0eb9558310b518f0dfcd8860 d0d8ad39ecb51cd7497cd524484fe09f50876798 de5b678ca4dcdfa83e322491d478d66df56c1986 82855aba5bf91e50c81526167c11d4aeaf665e66 Basis pass 921b2fed6a79439ef1609ef4af0ada5cccb3555c c530a75c1e6a472b0eb9558310b518f0dfcd8860 9c0eed618f37dd5b4a57c8b3fbc48ef8913e3149 de5b678ca4dcdfa83e322491d478d66df56c1986 940185b2f6f343251c2b83bd96e599398cea51ec Generating revisions with ./adhoc-revtuple-generator git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git#921b2fed6a79439ef1609ef4af0ada5cccb3555c-3879c163e8681939b1d93139521aee983623884f git://xenbits.xen.org/osstest/linux-firmware.git#c530a75c1e6a472b0eb9558310b518f0dfcd8860-c530a75c1e6a472b0eb9558310b518f0dfcd8860 git://xenbits.xen.org/qemu-xen-traditional.git#9c0eed618f37dd5b4a57c8b3fbc48ef8913e3149-d0d8ad39ecb51cd7497cd524484fe09f50876798 git://xenbits.xen.org/qemu-xen.git#de5b678ca4dcdfa83e322491d478d66df56c1986-de5b678ca4dcdfa83e322491d478d66df56c1986 git://xenbits.xen.org/xen.git#940185b2f6f343251c2b83bd96e599398cea51ec-82855aba5bf91e50c81526167c11d4aeaf665e66 Loaded 3004 nodes in revision graph Searching for test results: 128096 [dst_host=chardonnay1,src_host=chardonnay0] 128177 pass 921b2fed6a79439ef1609ef4af0ada5cccb3555c c530a75c1e6a472b0eb9558310b518f0dfcd8860 9c0eed618f37dd5b4a57c8b3fbc48ef8913e3149 de5b678ca4dcdfa83e322491d478d66df56c1986 940185b2f6f343251c2b83bd96e599398cea51ec 128232 [dst_host=albana1,src_host=albana0] 128258 [dst_host=fiano1,src_host=fiano0] 128691 [dst_host=fiano1,src_host=fiano0] 128807 [dst_host=godello1,src_host=godello0] 128858 [dst_host=elbling0,src_host=elbling1] 128841 [dst_host=elbling0,src_host=elbling1] 129760 [dst_host=godello1,src_host=godello0] 129845 [dst_host=albana1,src_host=albana0] 130067 [dst_host=pinot1,src_host=pinot0] 130203 [dst_host=albana0,src_host=albana1] 130367 [dst_host=albana0,src_host=albana1] 130843 fail irrelevant 130876 fail irrelevant 130939 fail irrelevant 131095 fail 3879c163e8681939b1d93139521aee983623884f c530a75c1e6a472b0eb9558310b518f0dfcd8860 d0d8ad39ecb51cd7497cd524484fe09f50876798 de5b678ca4dcdfa83e322491d478d66df56c1986 82855aba5bf91e50c81526167c11d4aeaf665e66 131035 fail irrelevant 131149 fail 3879c163e8681939b1d93139521aee983623884f c530a75c1e6a472b0eb9558310b518f0dfcd8860 d0d8ad39ecb51cd7497cd524484fe09f50876798 de5b678ca4dcdfa83e322491d478d66df56c1986 82855aba5bf91e50c81526167c11d4aeaf665e66 131192 fail 3879c163e8681939b1d93139521aee983623884f c530a75c1e6a472b0eb9558310b518f0dfcd8860 d0d8ad39ecb51cd7497cd524484fe09f50876798 de5b678ca4dcdfa83e322491d478d66df56c1986 82855aba5bf91e50c81526167c11d4aeaf665e66 131271 pass d255d18a34a8d53ccc4a019dc07e17b6e8cf6bd1 c530a75c1e6a472b0eb9558310b518f0dfcd8860 d0d8ad39ecb51cd7497cd524484fe09f50876798 de5b678ca4dcdfa83e322491d478d66df56c1986 6d8ffac1f7a782dc2c7f8df3871a294729ae36bd 131272 fail 7b8052e19304865477e03a0047062d977309a22f c530a75c1e6a472b0eb9558310b518f0dfcd8860 d0d8ad39ecb51cd7497cd524484fe09f50876798 de5b678ca4dcdfa83e322491d478d66df56c1986 6d8ffac1f7a782dc2c7f8df3871a294729ae36bd 131254 pass 921b2fed6a79439ef1609ef4af0ada5cccb3555c c530a75c1e6a472b0eb9558310b518f0dfcd8860 9c0eed618f37dd5b4a57c8b3fbc48ef8913e3149 de5b678ca4dcdfa83e322491d478d66df56c1986 940185b2f6f343251c2b83bd96e599398cea51ec 131258 fail 3879c163e8681939b1d93139521aee983623884f c530a75c1e6a472b0eb9558310b518f0dfcd8860 d0d8ad39ecb51cd7497cd524484fe09f50876798 de5b678ca4dcdfa83e322491d478d66df56c1986 82855aba5bf91e50c81526167c11d4aeaf665e66 131273 pass d255d18a34a8d53ccc4a019dc07e17b6e8cf6bd1 c530a75c1e6a472b0eb9558310b518f0dfcd8860 d0d8ad39ecb51cd7497cd524484fe09f50876798 de5b678ca4dcdfa83e322491d478d66df56c1986 6d8ffac1f7a782dc2c7f8df3871a294729ae36bd 131259 pass 17a4b61833db0fd17164dfa7762cf6354d794e3f c530a75c1e6a472b0eb9558310b518f0dfcd8860 d0d8ad39ecb51cd7497cd524484fe09f50876798 de5b678ca4dcdfa83e322491d478d66df56c1986 6d8ffac1f7a782dc2c7f8df3871a294729ae36bd 131260 blocked 50057d1de0173052ef90783eee8b49287c5a7aff c530a75c1e6a472b0eb9558310b518f0dfcd8860 d0d8ad39ecb51cd7497cd524484fe09f50876798 de5b678ca4dcdfa83e322491d478d66df56c1986 568653e01b84be5ac15fbf949bbcffcd91d6e829 131231 fail 3879c163e8681939b1d93139521aee983623884f c530a75c1e6a472b0eb9558310b518f0dfcd8860 d0d8ad39ecb51cd7497cd524484fe09f50876798 de5b678ca4dcdfa83e322491d478d66df56c1986 82855aba5bf91e50c81526167c11d4aeaf665e66 131261 fail 78e0897dd8b321ba1b4a2137778ab7ae7d400af5 c530a75c1e6a472b0eb9558310b518f0dfcd8860 d0d8ad39ecb51cd7497cd524484fe09f50876798 de5b678ca4dcdfa83e322491d478d66df56c1986 84f08707182dc240156f30540517beecbf5918b4 131275 fail 7b8052e19304865477e03a0047062d977309a22f c530a75c1e6a472b0eb9558310b518f0dfcd8860 d0d8ad39ecb51cd7497cd524484fe09f50876798 de5b678ca4dcdfa83e322491d478d66df56c1986 6d8ffac1f7a782dc2c7f8df3871a294729ae36bd 131277 pass d255d18a34a8d53ccc4a019dc07e17b6e8cf6bd1 c530a75c1e6a472b0eb9558310b518f0dfcd8860 d0d8ad39ecb51cd7497cd524484fe09f50876798 de5b678ca4dcdfa83e322491d478d66df56c1986 6d8ffac1f7a782dc2c7f8df3871a294729ae36bd 131262 fail 7b8052e19304865477e03a0047062d977309a22f c530a75c1e6a472b0eb9558310b518f0dfcd8860 d0d8ad39ecb51cd7497cd524484fe09f50876798 de5b678ca4dcdfa83e322491d478d66df56c1986 6d8ffac1f7a782dc2c7f8df3871a294729ae36bd 131266 pass aa4b7331d3121555088cddc4de0c819768961d46 c530a75c1e6a472b0eb9558310b518f0dfcd8860 d0d8ad39ecb51cd7497cd524484fe09f50876798 de5b678ca4dcdfa83e322491d478d66df56c1986 6d8ffac1f7a782dc2c7f8df3871a294729ae36bd 131267 pass 21bfce667e8afb19265e65309af2a44737293f92 c530a75c1e6a472b0eb9558310b518f0dfcd8860 d0d8ad39ecb51cd7497cd524484fe09f50876798 de5b678ca4dcdfa83e322491d478d66df56c1986 6d8ffac1f7a782dc2c7f8df3871a294729ae36bd 131268 pass 1c857dc0e69fdba81e5169cf4b3d2a9daaa783eb c530a75c1e6a472b0eb9558310b518f0dfcd8860 d0d8ad39ecb51cd7497cd524484fe09f50876798 de5b678ca4dcdfa83e322491d478d66df56c1986 6d8ffac1f7a782dc2c7f8df3871a294729ae36bd 131278 fail 7b8052e19304865477e03a0047062d977309a22f c530a75c1e6a472b0eb9558310b518f0dfcd8860 d0d8ad39ecb51cd7497cd524484fe09f50876798 de5b678ca4dcdfa83e322491d478d66df56c1986 6d8ffac1f7a782dc2c7f8df3871a294729ae36bd 131269 pass ce2c2e07337e98eb42904ef27c3ce62972d22a60 c530a75c1e6a472b0eb9558310b518f0dfcd8860 d0d8ad39ecb51cd7497cd524484fe09f50876798 de5b678ca4dcdfa83e322491d478d66df56c1986 6d8ffac1f7a782dc2c7f8df3871a294729ae36bd 131270 pass 61fde28f92ab9a329ef06e0ab3f41269e3930213 c530a75c1e6a472b0eb9558310b518f0dfcd8860 d0d8ad39ecb51cd7497cd524484fe09f50876798 de5b678ca4dcdfa83e322491d478d66df56c1986 6d8ffac1f7a782dc2c7f8df3871a294729ae36bd Searching for interesting versions Result found: flight 128177 (pass), for basis pass Result found: flight 131095 (fail), for basis failure Repro found: flight 131254 (pass), for basis pass Repro found: flight 131258 (fail), for basis failure 0 revisions at d255d18a34a8d53ccc4a019dc07e17b6e8cf6bd1 c530a75c1e6a472b0eb9558310b518f0dfcd8860 d0d8ad39ecb51cd7497cd524484fe09f50876798 de5b678ca4dcdfa83e322491d478d66df56c1986 6d8ffac1f7a782dc2c7f8df3871a294729ae36bd No revisions left to test, checking graph state. Result found: flight 131271 (pass), for last pass Result found: flight 131272 (fail), for first failure Repro found: flight 131273 (pass), for last pass Repro found: flight 131275 (fail), for first failure Repro found: flight 131277 (pass), for last pass Repro found: flight 131278 (fail), for first failure *** Found and reproduced problem changeset *** Bug is in tree: linux git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git Bug introduced: 7b8052e19304865477e03a0047062d977309a22f Bug not present: d255d18a34a8d53ccc4a019dc07e17b6e8cf6bd1 Last fail repro: http://logs.test-lab.xenproject.org/osstest/logs/131278/ commit 7b8052e19304865477e03a0047062d977309a22f Author: Jan Beulich <JBeulich@xxxxxxxx> Date: Mon Oct 19 04:23:29 2015 -0600 igb: fix NULL derefs due to skipped SR-IOV enabling [ Upstream commit be06998f96ecb93938ad2cce46c4289bf7cf45bc ] The combined effect of commits 6423fc3416 ("igb: do not re-init SR-IOV during probe") and ceee3450b3 ("igb: make sure SR-IOV init uses the right number of queues") causes VFs no longer getting set up, leading to NULL pointer dereferences due to the adapter's ->vf_data being NULL while ->vfs_allocated_count is non-zero. The first commit not only neglected the side effect of igb_sriov_reinit() that the second commit tried to account for, but also that of setting IGB_FLAG_HAS_MSIX, without which igb_enable_sriov() is effectively a no-op. Calling igb_{,re}set_interrupt_capability() as done here seems to address this, but I'm not sure whether this is better than sinply reverting the other two commits. Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx> Tested-by: Aaron Brown <aaron.f.brown@xxxxxxxxx> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@xxxxxxxxx> Signed-off-by: Sasha Levin <sashal@xxxxxxxxxx> dot: graph is too large for cairo-renderer bitmaps. Scaling by 0.613442 to fit pnmtopng: 36 colors found Revision graph left in /home/logs/results/bisect/linux-3.18/test-amd64-amd64-pair.xen-boot--dst_host.{dot,ps,png,html,svg}. No revision to test. _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxxxxxxxxx https://lists.xenproject.org/mailman/listinfo/xen-devel
|
 |
Lists.xenproject.org is hosted with RackSpace, monitoring our |