bisecting fixing commit since 6fbc7275c7a9ba97877050335f290341a1fd8dbf building syzkaller on 907bf74686129436f81aa40336ee89f7cc01b0b4 testing commit 6fbc7275c7a9ba97877050335f290341a1fd8dbf with gcc (GCC) 8.1.0 kernel signature: c56a9b44baeb73fef677332b07ae51fa03c67fea all runs: crashed: WARNING in perf_group_attach testing current HEAD 9455d25f4e3b3d009fa1b810862e5b06229530e4 testing commit 9455d25f4e3b3d009fa1b810862e5b06229530e4 with gcc (GCC) 8.1.0 kernel signature: 7c08c73d99965a13b273a65977310417dd21dbeb all runs: OK # git bisect start 9455d25f4e3b3d009fa1b810862e5b06229530e4 6fbc7275c7a9ba97877050335f290341a1fd8dbf Bisecting: 21685 revisions left to test after this (roughly 15 steps) [81160dda9a7aad13c04e78bb2cfd3c4630e3afab] Merge git://git.kernel.org/pub/scm/linux/kernel/git/netdev/net-next testing commit 81160dda9a7aad13c04e78bb2cfd3c4630e3afab with gcc (GCC) 8.1.0 kernel signature: 897304517be9fc7c1cdcd20e4c493b73d714fd09 all runs: OK # git bisect bad 81160dda9a7aad13c04e78bb2cfd3c4630e3afab Bisecting: 11628 revisions left to test after this (roughly 14 steps) [7636b7589f81940c6d6518786f93de74495575fa] Merge tag 'rpmsg-v5.3' of git://github.com/andersson/remoteproc testing commit 7636b7589f81940c6d6518786f93de74495575fa with gcc (GCC) 8.1.0 kernel signature: b5b005ba3f0dc47e13c7e1f2f489040444c00761 all runs: OK # git bisect bad 7636b7589f81940c6d6518786f93de74495575fa Bisecting: 5811 revisions left to test after this (roughly 13 steps) [39656e83dab918861931ef96e5c41731b0899e56] mm: lift the x86_32 PAE version of gup_get_pte to common code testing commit 39656e83dab918861931ef96e5c41731b0899e56 with gcc (GCC) 8.1.0 kernel signature: 0edf829eaa277f4ac00d046191faeb334cc1ef72 run #0: crashed: general protection fault in send_hsr_supervision_frame run #1: crashed: general protection fault in send_hsr_supervision_frame run #2: crashed: general protection fault in send_hsr_supervision_frame run #3: crashed: general protection fault in send_hsr_supervision_frame run #4: OK run #5: OK run #6: OK run #7: OK run #8: OK run #9: OK # git bisect good 39656e83dab918861931ef96e5c41731b0899e56 Bisecting: 3006 revisions left to test after this (roughly 12 steps) [168869492e7009b6861b615f1d030c99bc805e83] docs: kbuild: fix build with pdf and fix some minor issues testing commit 168869492e7009b6861b615f1d030c99bc805e83 with gcc (GCC) 8.1.0 kernel signature: 87544cc8d400d14ab08535fe7066bde51f2baa75 all runs: OK # git bisect bad 168869492e7009b6861b615f1d030c99bc805e83 Bisecting: 1380 revisions left to test after this (roughly 11 steps) [d06e4156430e7c5eb4f04dabcaa0d9e2fba335e3] Merge tag 'devicetree-for-5.3' of git://git.kernel.org/pub/scm/linux/kernel/git/robh/linux testing commit d06e4156430e7c5eb4f04dabcaa0d9e2fba335e3 with gcc (GCC) 8.1.0 kernel signature: 01850f2d126d295272e79a601bc8dce62184e928 run #0: crashed: general protection fault in send_hsr_supervision_frame run #1: OK run #2: OK run #3: OK run #4: OK run #5: OK run #6: OK run #7: OK run #8: OK run #9: OK # git bisect good d06e4156430e7c5eb4f04dabcaa0d9e2fba335e3 Bisecting: 691 revisions left to test after this (roughly 10 steps) [073c916bc00571d8662b89a294eba265481b6fbb] Merge branch 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/dtor/input testing commit 073c916bc00571d8662b89a294eba265481b6fbb with gcc (GCC) 8.1.0 kernel signature: d2321e3fd51c5ea9db9f75d2b6904aa1135233c0 run #0: crashed: general protection fault in send_hsr_supervision_frame run #1: crashed: general protection fault in send_hsr_supervision_frame run #2: crashed: general protection fault in send_hsr_supervision_frame run #3: OK run #4: OK run #5: OK run #6: OK run #7: OK run #8: OK run #9: OK # git bisect good 073c916bc00571d8662b89a294eba265481b6fbb Bisecting: 281 revisions left to test after this (roughly 9 steps) [192f0f8e9db7efe4ac98d47f5fa4334e43c1204d] Merge tag 'powerpc-5.3-1' of git://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux testing commit 192f0f8e9db7efe4ac98d47f5fa4334e43c1204d with gcc (GCC) 8.1.0 kernel signature: 54e37fd97f56ef3b2ee60e3d2b44d76c7e29fb23 all runs: OK # git bisect bad 192f0f8e9db7efe4ac98d47f5fa4334e43c1204d Bisecting: 212 revisions left to test after this (roughly 8 steps) [1fa91854dcdf7829e2694873b19f96b65947112d] Merge git://git.kernel.org/pub/scm/linux/kernel/git/davem/ide testing commit 1fa91854dcdf7829e2694873b19f96b65947112d with gcc (GCC) 8.1.0 kernel signature: 183e477bfa5df5d1415aca49534547bf10cd6a30 run #0: crashed: general protection fault in send_hsr_supervision_frame run #1: OK run #2: OK run #3: OK run #4: OK run #5: OK run #6: OK run #7: OK run #8: OK run #9: OK # git bisect good 1fa91854dcdf7829e2694873b19f96b65947112d Bisecting: 106 revisions left to test after this (roughly 7 steps) [41732bdc9ccd26829cf647711c4b93384e03e529] powerpc/powernv-eeh: Consisely desribe what this file does testing commit 41732bdc9ccd26829cf647711c4b93384e03e529 with gcc (GCC) 8.1.0 kernel signature: c36c786f2566718eef64582409a6dd03b273c564 all runs: crashed: WARNING in perf_group_attach # git bisect good 41732bdc9ccd26829cf647711c4b93384e03e529 Bisecting: 53 revisions left to test after this (roughly 6 steps) [a2b6f26c264e510f1d76ce347e34a21fbda9d8b2] powerpc/module64: Use symbolic instructions names. testing commit a2b6f26c264e510f1d76ce347e34a21fbda9d8b2 with gcc (GCC) 8.1.0 kernel signature: 9c1125111301d62852e0ac2fe9d4bcae57119029 all runs: crashed: WARNING in perf_group_attach # git bisect good a2b6f26c264e510f1d76ce347e34a21fbda9d8b2 Bisecting: 26 revisions left to test after this (roughly 5 steps) [6b660c4177aaebdc73df7a3378f0e8b110aa4b51] net: openvswitch: do not update max_headroom if new headroom is equal to old headroom testing commit 6b660c4177aaebdc73df7a3378f0e8b110aa4b51 with gcc (GCC) 8.1.0 kernel signature: 63036e5edb5d65e164c80c9d298cd0efd13fbc77 all runs: OK # git bisect bad 6b660c4177aaebdc73df7a3378f0e8b110aa4b51 Bisecting: 14 revisions left to test after this (roughly 4 steps) [08d14c49ac949b0e42f2aa36fb8d5c860cfe130d] Merge branch 'mlx5-build-fixes' testing commit 08d14c49ac949b0e42f2aa36fb8d5c860cfe130d with gcc (GCC) 8.1.0 kernel signature: 13fd37a98ef7e87a5ff599c477c7ee87efd0043a all runs: OK # git bisect bad 08d14c49ac949b0e42f2aa36fb8d5c860cfe130d Bisecting: 5 revisions left to test after this (roughly 3 steps) [052e0690f1f62f76493ba996d73847c7ca9fd132] ipv6: tcp: fix flowlabels reflection for RST packets testing commit 052e0690f1f62f76493ba996d73847c7ca9fd132 with gcc (GCC) 8.1.0 kernel signature: 407f7f1891094061fe35ec92168533cfd7fbb1c6 all runs: OK # git bisect bad 052e0690f1f62f76493ba996d73847c7ca9fd132 Bisecting: 2 revisions left to test after this (roughly 2 steps) [aa4c0c9091b0bb4cb261bbe0718d17c2834c4690] net: stmmac: Fix misuses of GENMASK macro testing commit aa4c0c9091b0bb4cb261bbe0718d17c2834c4690 with gcc (GCC) 8.1.0 kernel signature: 62229b1dbb32c63ce554a51e044b7c4b2ddf6808 run #0: crashed: general protection fault in send_hsr_supervision_frame run #1: crashed: general protection fault in send_hsr_supervision_frame run #2: crashed: general protection fault in send_hsr_supervision_frame run #3: OK run #4: OK run #5: OK run #6: OK run #7: OK run #8: OK run #9: OK # git bisect good aa4c0c9091b0bb4cb261bbe0718d17c2834c4690 Bisecting: 0 revisions left to test after this (roughly 1 step) [416e8126a2672f6e91e9e81c6f5c07cf46808b13] ipv6: Use ipv6_authlen for len testing commit 416e8126a2672f6e91e9e81c6f5c07cf46808b13 with gcc (GCC) 8.1.0 kernel signature: 7b8bb4696b04da761248a0a180a85bbeab16eb39 all runs: OK # git bisect bad 416e8126a2672f6e91e9e81c6f5c07cf46808b13 Bisecting: 0 revisions left to test after this (roughly 0 steps) [311633b604063a8a5d3fbc74d0565b42df721f68] hsr: switch ->dellink() to ->ndo_uninit() testing commit 311633b604063a8a5d3fbc74d0565b42df721f68 with gcc (GCC) 8.1.0 kernel signature: 31d4eea41388ea691b2bcc1a64ed4828a86c96d6 all runs: OK # git bisect bad 311633b604063a8a5d3fbc74d0565b42df721f68 311633b604063a8a5d3fbc74d0565b42df721f68 is the first bad commit commit 311633b604063a8a5d3fbc74d0565b42df721f68 Author: Cong Wang Date: Tue Jul 9 23:24:54 2019 -0700 hsr: switch ->dellink() to ->ndo_uninit() Switching from ->priv_destructor to dellink() has an unexpected consequence: existing RCU readers, that is, hsr_port_get_hsr() callers, may still be able to read the port list. Instead of checking the return value of each hsr_port_get_hsr(), we can just move it to ->ndo_uninit() which is called after device unregister and synchronize_net(), and we still have RTNL lock there. Fixes: b9a1e627405d ("hsr: implement dellink to clean up resources") Fixes: edf070a0fb45 ("hsr: fix a NULL pointer deref in hsr_dev_xmit()") Reported-by: syzbot+097ef84cdc95843fbaa8@syzkaller.appspotmail.com Cc: Arvid Brodin Signed-off-by: Cong Wang Signed-off-by: David S. Miller net/hsr/hsr_device.c | 18 ++++++++---------- net/hsr/hsr_device.h | 1 - net/hsr/hsr_netlink.c | 7 ------- 3 files changed, 8 insertions(+), 18 deletions(-) kernel signature: 31d4eea41388ea691b2bcc1a64ed4828a86c96d6 previous signature: 62229b1dbb32c63ce554a51e044b7c4b2ddf6808 revisions tested: 18, total time: 4h46m28.033625284s (build: 1h47m8.791896642s, test: 2h57m22.67660038s) first good commit: 311633b604063a8a5d3fbc74d0565b42df721f68 hsr: switch ->dellink() to ->ndo_uninit() cc: ["arvid.brodin@alten.se" "davem@davemloft.net" "linux-kernel@vger.kernel.org" "netdev@vger.kernel.org" "xiyou.wangcong@gmail.com"]