* [Regression] net/phy/micrel.c v4.9.94 @ 2018-04-18 1:34 Chris Ruehl 2018-04-18 5:09 ` Chris Ruehl 2018-04-18 12:56 ` Andrew Lunn 0 siblings, 2 replies; 8+ messages in thread From: Chris Ruehl @ 2018-04-18 1:34 UTC (permalink / raw) To: f.fainelli, netdev Hello, I like to get your heads up at a regression introduced in 4.9.94 commitment lead to a kernel ops and make the network unusable on my MX6DL customized board. Race condition resume is called on startup and the phy not yet initialized. [ 7.313366] Unable to handle kernel NULL pointer dereference at virtual address 00000008 [ 7.321602] pgd = ecfc0000 [ 7.324950] [00000008] *pgd=8e901831 [ 7.328652] Internal error: Oops: 17 [#1] PREEMPT SMP ARM [ 7.334061] Modules linked in: [ 7.337146] CPU: 0 PID: 269 Comm: ip Not tainted 4.9.94 #11 [ 7.342725] Hardware name: Freescale i.MX6 Quad/DualLite (Device Tree) [ 7.349259] task: ece59900 task.stack: ec9ea000 [ 7.353809] PC is at kszphy_config_reset+0x14/0x148 [ 7.358703] LR is at kszphy_resume+0x1c/0x6c [ 7.362983] pc : [<c056a24c>] lr : [<c056a4fc>] psr: 60030013 [ 7.362983] sp : ec9eb918 ip : ec9eb938 fp : ec9eb934 [ 7.374467] r10: 00000007 r9 : 00000000 r8 : ee693c00 [ 7.379700] r7 : 00000000 r6 : 00000000 r5 : 00000000 r4 : ee6fc000 [ 7.386234] r3 : 00000001 r2 : 00000000 r1 : 00000110 r0 : ee6fc000 [ 7.392768] Flags: nZCv IRQs on FIQs on Mode SVC_32 ISA ARM Segment none [ 7.399911] Control: 10c5387d Table: 3cfc004a DAC: 00000051 [ 7.405663] Process ip (pid: 269, stack limit = 0xec9ea210) [ 7.411244] Stack: (0xec9eb918 to 0xec9ec000) [ 7.415611] b900: ee6fc000 00000000 [ 7.423800] b920: ee031000 00000000 ec9eb94c ec9eb938 c056a4fc c056a244 ee6fc000 00000000 [ 7.431988] b940: ec9eb97c ec9eb950 c05681e4 c056a4ec 00000007 ee6fc000 ee6fc000 c056ce7c [ 7.440174] b960: c056ce7c ee031000 ee55c818 00000000 ec9eb99c ec9eb980 c05683cc c0568134 [ 7.448364] b980: 00000007 ec9eba10 ee6fc000 00000007 ec9eb9c4 ec9eb9a0 c0568450 c05683bc [ 7.456550] b9a0: 00000007 00000005 ee031000 ec9eb9d3 00000200 c1508da4 ec9eba6c ec9eb9c8 [ 7.464736] b9c0: c056ce24 c0568410 00000005 ee03162c 32000001 30383831 652e3030 72656874 [ 7.472921] b9e0: 2d74656e 00000031 000003e8 000000c8 c01732ec c0172adc 000003e8 000000c8 [ 7.481109] ba00: 024000c0 ee55c000 c150e454 024000c0 38383132 2e303030 65687465 74656e72 [ 7.489296] ba20: 303a312d ee000035 ec9eba6c ec9eba38 c0224b50 c0175eb8 ec9eba6c c056eb44 [ 7.497482] ba40: c056bbe0 f0c16000 ee031000 ee55c000 00000200 f0c16000 ee031000 ee55c000 [ 7.505667] ba60: ec9ebaa4 ec9eba70 c056eba4 c056cd1c 00000001 ee03162c ec9ebaa4 ee031000 [ 7.513855] ba80: 00000000 c09566ec ee031030 00000000 ec9ccd10 ecb39900 ec9ebacc ec9ebaa8 [ 7.522043] baa0: c06ad6e0 c056e92c ec9ebacc ee031000 ee031000 00000001 00001003 00001002 [ 7.530229] bac0: ec9ebaf4 ec9ebad0 c06ad99c c06ad63c 00001002 ee031000 ee031148 00001002 [ 7.538414] bae0: 00000000 00000000 ec9ebb1c ec9ebaf8 c06ada6c c06ad90c 00001002 00000000 [ 7.546601] bb00: ee031000 ec9ebc28 00000000 c09566ec ec9ebb94 ec9ebb20 c06c1034 c06ada58 [ 7.554787] bb20: c0c50df8 2e184000 ec9ebb44 ec9ebb38 c0173528 c0173320 ec9ebbd4 c0e82b6c [ 7.562972] bb40: 00000000 ece59dc8 ebb4e9d0 c9eae3f3 ece59900 00000003 ece59900 0000005e [ 7.571157] bb60: c14e30ec c0d1e51c ece59900 00000000 ee031000 ec9ccd00 00000000 00000000 [ 7.579346] bb80: ec9ebb98 00000000 ec9ebd04 ec9ebb98 c06c30cc c06c0d68 ec9ebbc4 00000000 [ 7.587531] bba0: c01758bc ecb39900 c09eb3a0 ec9ccd20 00000000 ec9ccd10 00000001 ece59900 [ 7.595715] bbc0: c01e0e64 00000000 00000000 00000001 ec9ebbfc 00000000 00000000 00000000 [ 7.603900] bbe0: 00000000 00000000 00000000 ffffff00 ec9ebc0c ec9ebc00 c0173528 c0173320 [ 7.612084] bc00: ec9ebc9c ec9ebc10 c01e0e64 c0173520 00000000 0000000e ece59900 00000096 [ 7.620269] bc20: c14e30ec c0d1e51c 00000000 00000000 00000000 00000000 00000000 00000000 [ 7.628452] bc40: 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000 [ 7.636636] bc60: 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000 [ 7.644819] bc80: 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000 [ 7.653003] bca0: 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000 [ 7.661186] bcc0: 00000000 00000000 00000000 00000000 00000000 00000000 c06d3870 00000000 [ 7.669372] bce0: ec9ccd00 ecb39900 c15226e4 00000000 00000000 ecb39900 ec9ebd44 ec9ebd08 [ 7.677556] bd00: c06c343c c06c2bdc c0869c2c c0173520 00000001 00000000 c06c06e4 00000000 [ 7.685741] bd20: 00000000 ec9ccd00 c06c32b8 ecb39900 ecb39900 00000000 ec9ebd64 ec9ebd48 [ 7.693926] bd40: c06d86cc c06c32c4 00000000 ecb39900 00000020 ec970400 ec9ebd7c ec9ebd68 [ 7.702110] bd60: c06c06f4 c06d8630 c06c06c4 ee15f400 ec9ebdac ec9ebd80 c06d802c c06c06d0 [ 7.710294] bd80: ec9ebf50 7fffffff ec970400 ec9ebf48 ec970400 00000000 00000020 00000000 [ 7.718477] bda0: ec9ebe0c ec9ebdb0 c06d84e8 c06d7ec8 0000000c ec9ebe48 0000000c 00000000 [ 7.726661] bdc0: beee97bc 00000008 00000000 ee0cbd80 00000000 0000010d 00000000 00000000 [ 7.734845] bde0: ec9ebe24 ec9ebf48 00000000 eda478c0 00000128 00000000 00000000 ec9ebe28 [ 7.743029] be00: ec9ebe1c ec9ebe10 c068b124 c06d8180 ec9ebf34 ec9ebe20 c068bcac c068b114 [ 7.751213] be20: ec9ebe8c 00000000 c01759cc c01704b0 ec9ebea4 ec9ebe40 c0209960 c01187ac [ 7.759396] be40: 00000000 00000000 beeed828 00000020 00000000 00000000 600f0113 ec98ce70 [ 7.767580] be60: beee9000 ed7ac714 ece59900 ed7ac6c0 00000817 beee977c ed7ac714 00000055 [ 7.775764] be80: ec9ebea4 ec9ebe90 00000010 00000000 00000000 ec9ebfb0 ec9ebefc ec9ebea8 [ 7.783948] bea0: c0115b74 c016ec18 00000000 c0d55a61 c025e178 ffffffea ee022f10 00000000 [ 7.792131] bec0: 00000000 00000000 00000800 00000000 c025ddc0 c0d0a44c 00000817 c0115990 [ 7.800315] bee0: beee977c ec9ebfb0 00054694 beeedec0 ec9ebfac ec9ebf00 c0101368 c011599c [ 7.808500] bf00: ec9ebf1c ec9ebf10 c025a578 eda478c0 00000000 beee97a4 00000128 c0107ee4 [ 7.816685] bf20: ec9ea000 00000000 ec9ebf94 ec9ebf38 c068ca54 c068bad4 00000000 00000000 [ 7.824868] bf40: 00000000 fffffff7 ec9ebe88 0000000c 00000001 00000000 00000000 ec9ebe50 [ 7.833052] bf60: 00000000 00000001 00000000 00000000 00000000 00000000 ec9ebf94 0000000c [ 7.841235] bf80: 00000010 beee9790 ec9ebfa4 ec9ebf98 c068ca94 c068ca18 00000000 ec9ebfa8 [ 7.849419] bfa0: c0107d20 c068ca90 0000000c 00000010 00000003 beee97a4 00000000 00000005 [ 7.857604] bfc0: 0000000c 00000010 beee9790 00000128 00000000 00054694 beee97a4 beee97c0 [ 7.865789] bfe0: 00000000 beee9774 00034d1d b6eaaf16 400f0030 00000003 3fffd861 3fffdc61 [ 7.873968] Backtrace: [ 7.876450] [<c056a238>] (kszphy_config_reset) from [<c056a4fc>] (kszphy_resume+0x1c/0x6c) [ 7.884723] r7:00000000 r6:ee031000 r5:00000000 r4:ee6fc000 [ 7.890392] [<c056a4e0>] (kszphy_resume) from [<c05681e4>] (phy_attach_direct+0xbc/0x1bc) [ 7.898575] r5:00000000 r4:ee6fc000 [ 7.902158] [<c0568128>] (phy_attach_direct) from [<c05683cc>] (phy_connect_direct+0x1c/0x54) [ 7.910691] r10:00000000 r9:ee55c818 r8:ee031000 r7:c056ce7c r6:c056ce7c r5:ee6fc000 [ 7.918525] r4:ee6fc000 r3:00000007 [ 7.922108] [<c05683b0>] (phy_connect_direct) from [<c0568450>] (phy_connect+0x4c/0x80) [ 7.930117] r6:00000007 r5:ee6fc000 r4:ec9eba10 r3:00000007 [ 7.935785] [<c0568404>] (phy_connect) from [<c056ce24>] (fec_enet_mii_probe+0x114/0x16c) [ 7.943969] r8:c1508da4 r7:00000200 r6:ec9eb9d3 r5:ee031000 r4:00000005 r3:00000007 [ 7.951720] [<c056cd10>] (fec_enet_mii_probe) from [<c056eba4>] (fec_enet_open+0x284/0x320) [ 7.960076] r6:ee55c000 r5:ee031000 r4:f0c16000 [ 7.964704] [<c056e920>] (fec_enet_open) from [<c06ad6e0>] (__dev_open+0xb0/0x114) [ 7.972281] r10:ecb39900 r9:ec9ccd10 r8:00000000 r7:ee031030 r6:c09566ec r5:00000000 [ 7.980113] r4:ee031000 [ 7.982655] [<c06ad630>] (__dev_open) from [<c06ad99c>] (__dev_change_flags+0x9c/0x14c) [ 7.990665] r7:00001002 r6:00001003 r5:00000001 r4:ee031000 [ 7.996333] [<c06ad900>] (__dev_change_flags) from [<c06ada6c>] (dev_change_flags+0x20/0x50) [ 8.004777] r8:00000000 r7:00000000 r6:00001002 r5:ee031148 r4:ee031000 r3:00001002 [ 8.012533] [<c06ada4c>] (dev_change_flags) from [<c06c1034>] (do_setlink+0x2d8/0x838) [ 8.020456] r8:c09566ec r7:00000000 r6:ec9ebc28 r5:ee031000 r4:00000000 r3:00001002 [ 8.028207] [<c06c0d5c>] (do_setlink) from [<c06c30cc>] (rtnl_newlink+0x4fc/0x6e8) [ 8.035784] r10:00000000 r9:ec9ebb98 r8:00000000 r7:00000000 r6:ec9ccd00 r5:ee031000 [ 8.043616] r4:00000000 [ 8.046159] [<c06c2bd0>] (rtnl_newlink) from [<c06c343c>] (rtnetlink_rcv_msg+0x184/0x234) [ 8.054343] r10:ecb39900 r9:00000000 r8:00000000 r7:c15226e4 r6:ecb39900 r5:ec9ccd00 [ 8.062175] r4:00000000 [ 8.064720] [<c06c32b8>] (rtnetlink_rcv_msg) from [<c06d86cc>] (netlink_rcv_skb+0xa8/0xc4) [ 8.072990] r8:00000000 r7:ecb39900 r6:ecb39900 r5:c06c32b8 r4:ec9ccd00 [ 8.079701] [<c06d8624>] (netlink_rcv_skb) from [<c06c06f4>] (rtnetlink_rcv+0x30/0x38) [ 8.087622] r6:ec970400 r5:00000020 r4:ecb39900 r3:00000000 [ 8.093291] [<c06c06c4>] (rtnetlink_rcv) from [<c06d802c>] (netlink_unicast+0x170/0x1f8) [ 8.101384] r4:ee15f400 r3:c06c06c4 [ 8.104968] [<c06d7ebc>] (netlink_unicast) from [<c06d84e8>] (netlink_sendmsg+0x374/0x388) [ 8.113238] r8:00000000 r7:00000020 r6:00000000 r5:ec970400 r4:ec9ebf48 [ 8.119952] [<c06d8174>] (netlink_sendmsg) from [<c068b124>] (sock_sendmsg+0x1c/0x2c) [ 8.127789] r10:ec9ebe28 r9:00000000 r8:00000000 r7:00000128 r6:eda478c0 r5:00000000 [ 8.135621] r4:ec9ebf48 [ 8.138167] [<c068b108>] (sock_sendmsg) from [<c068bcac>] (___sys_sendmsg+0x1e4/0x20c) [ 8.146095] [<c068bac8>] (___sys_sendmsg) from [<c068ca54>] (__sys_sendmsg+0x48/0x78) [ 8.153932] r10:00000000 r9:ec9ea000 r8:c0107ee4 r7:00000128 r6:beee97a4 r5:00000000 [ 8.161764] r4:eda478c0 [ 8.164304] [<c068ca0c>] (__sys_sendmsg) from [<c068ca94>] (SyS_sendmsg+0x10/0x14) [ 8.171880] r6:beee9790 r5:00000010 r4:0000000c [ 8.176510] [<c068ca84>] (SyS_sendmsg) from [<c0107d20>] (ret_fast_syscall+0x0/0x1c) [ 8.184261] Code: e92dd8f0 e24cb004 e590628c e1a04000 (e5d63008) [ 8.192713] ---[ end trace 07c02ee14784bc48 ]--- Kind regards Chris -- GTSYS Limited RFID Technology 9/F, Unit E, R07, Kwai Shing Industrial Building Phase 2, 42-46 Tai Lin Pai Road, Kwai Chung, N.T., Hong Kong Tel (852) 9079 9521 Disclaimer: https://www.gtsys.com.hk/email/classified.html ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Regression] net/phy/micrel.c v4.9.94 2018-04-18 1:34 [Regression] net/phy/micrel.c v4.9.94 Chris Ruehl @ 2018-04-18 5:09 ` Chris Ruehl 2018-04-18 12:43 ` Andrew Lunn 2018-04-18 12:56 ` Andrew Lunn 1 sibling, 1 reply; 8+ messages in thread From: Chris Ruehl @ 2018-04-18 5:09 UTC (permalink / raw) To: f.fainelli, netdev On Wednesday, April 18, 2018 09:34 AM, Chris Ruehl wrote: > Hello, > > I like to get your heads up at a regression introduced in 4.9.94 > commitment lead to a kernel ops and make the network unusable on my MX6DL > customized board. > > Race condition resume is called on startup and the phy not yet initialized. > > [ 7.313366] Unable to handle kernel NULL pointer dereference at virtual > address 00000008 > [ 7.321602] pgd = ecfc0000 > > [ 7.324950] [00000008] *pgd=8e901831 > > [ 7.328652] Internal error: Oops: 17 [#1] PREEMPT SMP ARM > > [ 7.334061] Modules linked in: > > [ 7.337146] CPU: 0 PID: 269 Comm: ip Not tainted 4.9.94 #11 > > [ 7.342725] Hardware name: Freescale i.MX6 Quad/DualLite (Device Tree) > > [ 7.349259] task: ece59900 task.stack: ec9ea000 > > [ 7.353809] PC is at kszphy_config_reset+0x14/0x148 > > [ 7.358703] LR is at kszphy_resume+0x1c/0x6c > > [ 7.362983] pc : [<c056a24c>] lr : [<c056a4fc>] psr: 60030013 > > [ 7.362983] sp : ec9eb918 ip : ec9eb938 fp : ec9eb934 > > [ 7.374467] r10: 00000007 r9 : 00000000 r8 : ee693c00 > > [ 7.379700] r7 : 00000000 r6 : 00000000 r5 : 00000000 r4 : ee6fc000 > > [ 7.386234] r3 : 00000001 r2 : 00000000 r1 : 00000110 r0 : ee6fc000 > > [ 7.392768] Flags: nZCv IRQs on FIQs on Mode SVC_32 ISA ARM Segment none > > [ 7.399911] Control: 10c5387d Table: 3cfc004a DAC: 00000051 > > [ 7.405663] Process ip (pid: 269, stack limit = 0xec9ea210) > > [ 7.411244] Stack: (0xec9eb918 to 0xec9ec000) > > [ 7.415611] b900: ee6fc000 00000000 > [ 7.423800] b920: ee031000 00000000 ec9eb94c ec9eb938 c056a4fc c056a244 > ee6fc000 00000000 > [ 7.431988] b940: ec9eb97c ec9eb950 c05681e4 c056a4ec 00000007 ee6fc000 > ee6fc000 c056ce7c > [ 7.440174] b960: c056ce7c ee031000 ee55c818 00000000 ec9eb99c ec9eb980 > c05683cc c0568134 > [ 7.448364] b980: 00000007 ec9eba10 ee6fc000 00000007 ec9eb9c4 ec9eb9a0 > c0568450 c05683bc > [ 7.456550] b9a0: 00000007 00000005 ee031000 ec9eb9d3 00000200 c1508da4 > ec9eba6c ec9eb9c8 > [ 7.464736] b9c0: c056ce24 c0568410 00000005 ee03162c 32000001 30383831 > 652e3030 72656874 > [ 7.472921] b9e0: 2d74656e 00000031 000003e8 000000c8 c01732ec c0172adc > 000003e8 000000c8 > [ 7.481109] ba00: 024000c0 ee55c000 c150e454 024000c0 38383132 2e303030 > 65687465 74656e72 > [ 7.489296] ba20: 303a312d ee000035 ec9eba6c ec9eba38 c0224b50 c0175eb8 > ec9eba6c c056eb44 > [ 7.497482] ba40: c056bbe0 f0c16000 ee031000 ee55c000 00000200 f0c16000 > ee031000 ee55c000 > [ 7.505667] ba60: ec9ebaa4 ec9eba70 c056eba4 c056cd1c 00000001 ee03162c > ec9ebaa4 ee031000 > [ 7.513855] ba80: 00000000 c09566ec ee031030 00000000 ec9ccd10 ecb39900 > ec9ebacc ec9ebaa8 > [ 7.522043] baa0: c06ad6e0 c056e92c ec9ebacc ee031000 ee031000 00000001 > 00001003 00001002 > [ 7.530229] bac0: ec9ebaf4 ec9ebad0 c06ad99c c06ad63c 00001002 ee031000 > ee031148 00001002 > [ 7.538414] bae0: 00000000 00000000 ec9ebb1c ec9ebaf8 c06ada6c c06ad90c > 00001002 00000000 > [ 7.546601] bb00: ee031000 ec9ebc28 00000000 c09566ec ec9ebb94 ec9ebb20 > c06c1034 c06ada58 > [ 7.554787] bb20: c0c50df8 2e184000 ec9ebb44 ec9ebb38 c0173528 c0173320 > ec9ebbd4 c0e82b6c > [ 7.562972] bb40: 00000000 ece59dc8 ebb4e9d0 c9eae3f3 ece59900 00000003 > ece59900 0000005e > [ 7.571157] bb60: c14e30ec c0d1e51c ece59900 00000000 ee031000 ec9ccd00 > 00000000 00000000 > [ 7.579346] bb80: ec9ebb98 00000000 ec9ebd04 ec9ebb98 c06c30cc c06c0d68 > ec9ebbc4 00000000 > [ 7.587531] bba0: c01758bc ecb39900 c09eb3a0 ec9ccd20 00000000 ec9ccd10 > 00000001 ece59900 > [ 7.595715] bbc0: c01e0e64 00000000 00000000 00000001 ec9ebbfc 00000000 > 00000000 00000000 > [ 7.603900] bbe0: 00000000 00000000 00000000 ffffff00 ec9ebc0c ec9ebc00 > c0173528 c0173320 > [ 7.612084] bc00: ec9ebc9c ec9ebc10 c01e0e64 c0173520 00000000 0000000e > ece59900 00000096 > [ 7.620269] bc20: c14e30ec c0d1e51c 00000000 00000000 00000000 00000000 > 00000000 00000000 > [ 7.628452] bc40: 00000000 00000000 00000000 00000000 00000000 00000000 > 00000000 00000000 > [ 7.636636] bc60: 00000000 00000000 00000000 00000000 00000000 00000000 > 00000000 00000000 > [ 7.644819] bc80: 00000000 00000000 00000000 00000000 00000000 00000000 > 00000000 00000000 > [ 7.653003] bca0: 00000000 00000000 00000000 00000000 00000000 00000000 > 00000000 00000000 > [ 7.661186] bcc0: 00000000 00000000 00000000 00000000 00000000 00000000 > c06d3870 00000000 > [ 7.669372] bce0: ec9ccd00 ecb39900 c15226e4 00000000 00000000 ecb39900 > ec9ebd44 ec9ebd08 > [ 7.677556] bd00: c06c343c c06c2bdc c0869c2c c0173520 00000001 00000000 > c06c06e4 00000000 > [ 7.685741] bd20: 00000000 ec9ccd00 c06c32b8 ecb39900 ecb39900 00000000 > ec9ebd64 ec9ebd48 > [ 7.693926] bd40: c06d86cc c06c32c4 00000000 ecb39900 00000020 ec970400 > ec9ebd7c ec9ebd68 > [ 7.702110] bd60: c06c06f4 c06d8630 c06c06c4 ee15f400 ec9ebdac ec9ebd80 > c06d802c c06c06d0 > [ 7.710294] bd80: ec9ebf50 7fffffff ec970400 ec9ebf48 ec970400 00000000 > 00000020 00000000 > [ 7.718477] bda0: ec9ebe0c ec9ebdb0 c06d84e8 c06d7ec8 0000000c ec9ebe48 > 0000000c 00000000 > [ 7.726661] bdc0: beee97bc 00000008 00000000 ee0cbd80 00000000 0000010d > 00000000 00000000 > [ 7.734845] bde0: ec9ebe24 ec9ebf48 00000000 eda478c0 00000128 00000000 > 00000000 ec9ebe28 > [ 7.743029] be00: ec9ebe1c ec9ebe10 c068b124 c06d8180 ec9ebf34 ec9ebe20 > c068bcac c068b114 > [ 7.751213] be20: ec9ebe8c 00000000 c01759cc c01704b0 ec9ebea4 ec9ebe40 > c0209960 c01187ac > [ 7.759396] be40: 00000000 00000000 beeed828 00000020 00000000 00000000 > 600f0113 ec98ce70 > [ 7.767580] be60: beee9000 ed7ac714 ece59900 ed7ac6c0 00000817 beee977c > ed7ac714 00000055 > [ 7.775764] be80: ec9ebea4 ec9ebe90 00000010 00000000 00000000 ec9ebfb0 > ec9ebefc ec9ebea8 > [ 7.783948] bea0: c0115b74 c016ec18 00000000 c0d55a61 c025e178 ffffffea > ee022f10 00000000 > [ 7.792131] bec0: 00000000 00000000 00000800 00000000 c025ddc0 c0d0a44c > 00000817 c0115990 > [ 7.800315] bee0: beee977c ec9ebfb0 00054694 beeedec0 ec9ebfac ec9ebf00 > c0101368 c011599c > [ 7.808500] bf00: ec9ebf1c ec9ebf10 c025a578 eda478c0 00000000 beee97a4 > 00000128 c0107ee4 > [ 7.816685] bf20: ec9ea000 00000000 ec9ebf94 ec9ebf38 c068ca54 c068bad4 > 00000000 00000000 > [ 7.824868] bf40: 00000000 fffffff7 ec9ebe88 0000000c 00000001 00000000 > 00000000 ec9ebe50 > [ 7.833052] bf60: 00000000 00000001 00000000 00000000 00000000 00000000 > ec9ebf94 0000000c > [ 7.841235] bf80: 00000010 beee9790 ec9ebfa4 ec9ebf98 c068ca94 c068ca18 > 00000000 ec9ebfa8 > [ 7.849419] bfa0: c0107d20 c068ca90 0000000c 00000010 00000003 beee97a4 > 00000000 00000005 > [ 7.857604] bfc0: 0000000c 00000010 beee9790 00000128 00000000 00054694 > beee97a4 beee97c0 > [ 7.865789] bfe0: 00000000 beee9774 00034d1d b6eaaf16 400f0030 00000003 > 3fffd861 3fffdc61 > [ 7.873968] Backtrace: > > [ 7.876450] [<c056a238>] (kszphy_config_reset) from [<c056a4fc>] > (kszphy_resume+0x1c/0x6c) > [ 7.884723] r7:00000000 r6:ee031000 r5:00000000 r4:ee6fc000 > > [ 7.890392] [<c056a4e0>] (kszphy_resume) from [<c05681e4>] > (phy_attach_direct+0xbc/0x1bc) > [ 7.898575] r5:00000000 r4:ee6fc000 > > [ 7.902158] [<c0568128>] (phy_attach_direct) from [<c05683cc>] > (phy_connect_direct+0x1c/0x54) > [ 7.910691] r10:00000000 r9:ee55c818 r8:ee031000 r7:c056ce7c r6:c056ce7c > r5:ee6fc000 > [ 7.918525] r4:ee6fc000 r3:00000007 > > [ 7.922108] [<c05683b0>] (phy_connect_direct) from [<c0568450>] > (phy_connect+0x4c/0x80) > [ 7.930117] r6:00000007 r5:ee6fc000 r4:ec9eba10 r3:00000007 > > [ 7.935785] [<c0568404>] (phy_connect) from [<c056ce24>] > (fec_enet_mii_probe+0x114/0x16c) > [ 7.943969] r8:c1508da4 r7:00000200 r6:ec9eb9d3 r5:ee031000 r4:00000005 > r3:00000007 > [ 7.951720] [<c056cd10>] (fec_enet_mii_probe) from [<c056eba4>] > (fec_enet_open+0x284/0x320) > [ 7.960076] r6:ee55c000 r5:ee031000 r4:f0c16000 > > [ 7.964704] [<c056e920>] (fec_enet_open) from [<c06ad6e0>] > (__dev_open+0xb0/0x114) > [ 7.972281] r10:ecb39900 r9:ec9ccd10 r8:00000000 r7:ee031030 r6:c09566ec > r5:00000000 > [ 7.980113] r4:ee031000 > > [ 7.982655] [<c06ad630>] (__dev_open) from [<c06ad99c>] > (__dev_change_flags+0x9c/0x14c) > [ 7.990665] r7:00001002 r6:00001003 r5:00000001 r4:ee031000 > > [ 7.996333] [<c06ad900>] (__dev_change_flags) from [<c06ada6c>] > (dev_change_flags+0x20/0x50) > [ 8.004777] r8:00000000 r7:00000000 r6:00001002 r5:ee031148 r4:ee031000 > r3:00001002 > [ 8.012533] [<c06ada4c>] (dev_change_flags) from [<c06c1034>] > (do_setlink+0x2d8/0x838) > [ 8.020456] r8:c09566ec r7:00000000 r6:ec9ebc28 r5:ee031000 r4:00000000 > r3:00001002 > [ 8.028207] [<c06c0d5c>] (do_setlink) from [<c06c30cc>] > (rtnl_newlink+0x4fc/0x6e8) > [ 8.035784] r10:00000000 r9:ec9ebb98 r8:00000000 r7:00000000 r6:ec9ccd00 > r5:ee031000 > [ 8.043616] r4:00000000 > > [ 8.046159] [<c06c2bd0>] (rtnl_newlink) from [<c06c343c>] > (rtnetlink_rcv_msg+0x184/0x234) > [ 8.054343] r10:ecb39900 r9:00000000 r8:00000000 r7:c15226e4 r6:ecb39900 > r5:ec9ccd00 > [ 8.062175] r4:00000000 > > [ 8.064720] [<c06c32b8>] (rtnetlink_rcv_msg) from [<c06d86cc>] > (netlink_rcv_skb+0xa8/0xc4) > [ 8.072990] r8:00000000 r7:ecb39900 r6:ecb39900 r5:c06c32b8 r4:ec9ccd00 > [ 8.079701] [<c06d8624>] (netlink_rcv_skb) from [<c06c06f4>] > (rtnetlink_rcv+0x30/0x38) > [ 8.087622] r6:ec970400 r5:00000020 r4:ecb39900 r3:00000000 > [ 8.093291] [<c06c06c4>] (rtnetlink_rcv) from [<c06d802c>] > (netlink_unicast+0x170/0x1f8) > [ 8.101384] r4:ee15f400 r3:c06c06c4 > [ 8.104968] [<c06d7ebc>] (netlink_unicast) from [<c06d84e8>] > (netlink_sendmsg+0x374/0x388) > [ 8.113238] r8:00000000 r7:00000020 r6:00000000 r5:ec970400 r4:ec9ebf48 > [ 8.119952] [<c06d8174>] (netlink_sendmsg) from [<c068b124>] > (sock_sendmsg+0x1c/0x2c) > [ 8.127789] r10:ec9ebe28 r9:00000000 r8:00000000 r7:00000128 r6:eda478c0 > r5:00000000 > [ 8.135621] r4:ec9ebf48 > [ 8.138167] [<c068b108>] (sock_sendmsg) from [<c068bcac>] > (___sys_sendmsg+0x1e4/0x20c) > [ 8.146095] [<c068bac8>] (___sys_sendmsg) from [<c068ca54>] > (__sys_sendmsg+0x48/0x78) > [ 8.153932] r10:00000000 r9:ec9ea000 r8:c0107ee4 r7:00000128 r6:beee97a4 > r5:00000000 > [ 8.161764] r4:eda478c0 > [ 8.164304] [<c068ca0c>] (__sys_sendmsg) from [<c068ca94>] > (SyS_sendmsg+0x10/0x14) > [ 8.171880] r6:beee9790 r5:00000010 r4:0000000c > [ 8.176510] [<c068ca84>] (SyS_sendmsg) from [<c0107d20>] > (ret_fast_syscall+0x0/0x1c) > [ 8.184261] Code: e92dd8f0 e24cb004 e590628c e1a04000 (e5d63008) > [ 8.192713] ---[ end trace 07c02ee14784bc48 ]--- > > Kind regards > Chris > > If I look at the patch I think it should call kszphy_config_init() not _reset() in the resume function: @@ -715,8 +723,14 @@ static int kszphy_suspend(struct phy_device *phydev) static int kszphy_resume(struct phy_device *phydev) { + int ret; + genphy_resume(phydev); - ret = kszphy_config_reset(phydev); + ret = kszphy_config_init(phydev); + if (ret) + return ret; + ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Regression] net/phy/micrel.c v4.9.94 2018-04-18 5:09 ` Chris Ruehl @ 2018-04-18 12:43 ` Andrew Lunn 0 siblings, 0 replies; 8+ messages in thread From: Andrew Lunn @ 2018-04-18 12:43 UTC (permalink / raw) To: Chris Ruehl; +Cc: f.fainelli, netdev > If I look at the patch I think it should call kszphy_config_init() not _reset() > in the resume function: > > > @@ -715,8 +723,14 @@ static int kszphy_suspend(struct phy_device *phydev) > > static int kszphy_resume(struct phy_device *phydev) > { > + int ret; > + > genphy_resume(phydev); > > - ret = kszphy_config_reset(phydev); > + ret = kszphy_config_init(phydev); > + if (ret) > + return ret; > + > Hi Chris I think there has been a patch for this posted. If i remember correctly, the PHY you have does not call probe, hence phydev->priv is a NULL pointer, so priv->rmii_ref_clk_sel does not work. It would be good to find the patch and make sure it has been accepted, and marked for stable. Andrew ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Regression] net/phy/micrel.c v4.9.94 2018-04-18 1:34 [Regression] net/phy/micrel.c v4.9.94 Chris Ruehl 2018-04-18 5:09 ` Chris Ruehl @ 2018-04-18 12:56 ` Andrew Lunn 2018-04-18 13:02 ` Andrew Lunn 1 sibling, 1 reply; 8+ messages in thread From: Andrew Lunn @ 2018-04-18 12:56 UTC (permalink / raw) To: Chris Ruehl; +Cc: f.fainelli, netdev On Wed, Apr 18, 2018 at 09:34:16AM +0800, Chris Ruehl wrote: > Hello, > > I like to get your heads up at a regression introduced in 4.9.94 > commitment lead to a kernel ops and make the network unusable on my MX6DL > customized board. > > Race condition resume is called on startup and the phy not yet initialized. Hi Chris Please could you try bfe72442578b ("net: phy: micrel: fix crash when statistic requested for KSZ9031 phy") Andrew ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Regression] net/phy/micrel.c v4.9.94 2018-04-18 12:56 ` Andrew Lunn @ 2018-04-18 13:02 ` Andrew Lunn 2018-04-19 1:21 ` Chris Ruehl 0 siblings, 1 reply; 8+ messages in thread From: Andrew Lunn @ 2018-04-18 13:02 UTC (permalink / raw) To: Chris Ruehl; +Cc: f.fainelli, netdev On Wed, Apr 18, 2018 at 02:56:01PM +0200, Andrew Lunn wrote: > On Wed, Apr 18, 2018 at 09:34:16AM +0800, Chris Ruehl wrote: > > Hello, > > > > I like to get your heads up at a regression introduced in 4.9.94 > > commitment lead to a kernel ops and make the network unusable on my MX6DL > > customized board. > > > > Race condition resume is called on startup and the phy not yet initialized. > > Hi Chris > > Please could you try > > bfe72442578b ("net: phy: micrel: fix crash when statistic requested for KSZ9031 phy") I don't think it is a complete fix. I suspect "Micrel KSZ8795", "Micrel KSZ886X Switch", "Micrel KSZ8061", and "Micrel KS8737" will still have problems. Those four probably need a: .probe = kszphy_probe, Andrew ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Regression] net/phy/micrel.c v4.9.94 2018-04-18 13:02 ` Andrew Lunn @ 2018-04-19 1:21 ` Chris Ruehl 2018-04-19 1:34 ` Chris Ruehl 0 siblings, 1 reply; 8+ messages in thread From: Chris Ruehl @ 2018-04-19 1:21 UTC (permalink / raw) To: Andrew Lunn; +Cc: f.fainelli, netdev On Wednesday, April 18, 2018 09:02 PM, Andrew Lunn wrote: > On Wed, Apr 18, 2018 at 02:56:01PM +0200, Andrew Lunn wrote: >> On Wed, Apr 18, 2018 at 09:34:16AM +0800, Chris Ruehl wrote: >>> Hello, >>> >>> I like to get your heads up at a regression introduced in 4.9.94 >>> commitment lead to a kernel ops and make the network unusable on my MX6DL >>> customized board. >>> >>> Race condition resume is called on startup and the phy not yet initialized. >> >> Hi Chris >> >> Please could you try >> >> bfe72442578b ("net: phy: micrel: fix crash when statistic requested for KSZ9031 phy") > > I don't think it is a complete fix. I suspect "Micrel KSZ8795", > "Micrel KSZ886X Switch", "Micrel KSZ8061", and "Micrel KS8737" will > still have problems. > > Those four probably need a: > > .probe = kszphy_probe, > > Andrew > Indeed I have the [ 7.385851] Micrel KSZ9031 Gigabit PHY 2188000.ethernet-1:05: attached PHY driver [Micrel KSZ9031 Gigabit PHY] (mii_bus:phy_addr=2188000.ethernet-1:05, irq=-1) first I rollback to a non crashing stable kernel. As "bfe72442578b" gonna fix it I check with the next update and verify its works for me. Thanks Chris ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Regression] net/phy/micrel.c v4.9.94 2018-04-19 1:21 ` Chris Ruehl @ 2018-04-19 1:34 ` Chris Ruehl 2018-04-19 2:34 ` Chris Ruehl 0 siblings, 1 reply; 8+ messages in thread From: Chris Ruehl @ 2018-04-19 1:34 UTC (permalink / raw) To: Andrew Lunn; +Cc: f.fainelli, netdev On Thursday, April 19, 2018 09:21 AM, Chris Ruehl wrote: > On Wednesday, April 18, 2018 09:02 PM, Andrew Lunn wrote: >> On Wed, Apr 18, 2018 at 02:56:01PM +0200, Andrew Lunn wrote: >>> On Wed, Apr 18, 2018 at 09:34:16AM +0800, Chris Ruehl wrote: >>>> Hello, >>>> >>>> I like to get your heads up at a regression introduced in 4.9.94 >>>> commitment lead to a kernel ops and make the network unusable on my MX6DL >>>> customized board. >>>> >>>> Race condition resume is called on startup and the phy not yet initialized. >>> >>> Hi Chris >>> >>> Please could you try >>> >>> bfe72442578b ("net: phy: micrel: fix crash when statistic requested for >>> KSZ9031 phy") >> >> I don't think it is a complete fix. I suspect "Micrel KSZ8795", >> "Micrel KSZ886X Switch", "Micrel KSZ8061", and "Micrel KS8737" will >> still have problems. >> >> Those four probably need a: >> >> .probe = kszphy_probe, >> >> Andrew >> > > Indeed I have the > [ 7.385851] Micrel KSZ9031 Gigabit PHY 2188000.ethernet-1:05: attached PHY > driver [Micrel KSZ9031 Gigabit PHY] (mii_bus:phy_addr=2188000.ethernet-1:05, > irq=-1) > > first I rollback to a non crashing stable kernel. > > As "bfe72442578b" gonna fix it I check with the next update and verify its works > for me. > > Thanks > Chris Andrew, Change my mind. Find the patch you mentioned and will apply and test. Chris ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Regression] net/phy/micrel.c v4.9.94 2018-04-19 1:34 ` Chris Ruehl @ 2018-04-19 2:34 ` Chris Ruehl 0 siblings, 0 replies; 8+ messages in thread From: Chris Ruehl @ 2018-04-19 2:34 UTC (permalink / raw) To: Andrew Lunn; +Cc: f.fainelli, netdev On Thursday, April 19, 2018 09:34 AM, Chris Ruehl wrote: > > > On Thursday, April 19, 2018 09:21 AM, Chris Ruehl wrote: >> On Wednesday, April 18, 2018 09:02 PM, Andrew Lunn wrote: >>> On Wed, Apr 18, 2018 at 02:56:01PM +0200, Andrew Lunn wrote: >>>> On Wed, Apr 18, 2018 at 09:34:16AM +0800, Chris Ruehl wrote: >>>>> Hello, >>>>> >>>>> I like to get your heads up at a regression introduced in 4.9.94 >>>>> commitment lead to a kernel ops and make the network unusable on my MX6DL >>>>> customized board. >>>>> >>>>> Race condition resume is called on startup and the phy not yet initialized. >>>> >>>> Hi Chris >>>> >>>> Please could you try >>>> >>>> bfe72442578b ("net: phy: micrel: fix crash when statistic requested for >>>> KSZ9031 phy") >>> >>> I don't think it is a complete fix. I suspect "Micrel KSZ8795", >>> "Micrel KSZ886X Switch", "Micrel KSZ8061", and "Micrel KS8737" will >>> still have problems. >>> >>> Those four probably need a: >>> >>> .probe = kszphy_probe, >>> >>> Andrew >>> >> >> Indeed I have the >> [ 7.385851] Micrel KSZ9031 Gigabit PHY 2188000.ethernet-1:05: attached PHY >> driver [Micrel KSZ9031 Gigabit PHY] (mii_bus:phy_addr=2188000.ethernet-1:05, >> irq=-1) >> >> first I rollback to a non crashing stable kernel. >> >> As "bfe72442578b" gonna fix it I check with the next update and verify its >> works for me. >> >> Thanks >> Chris > > Andrew, > > Change my mind. Find the patch you mentioned and will apply and test. > > Chris Andrew, make a micro patch only and can confirm to add the .probe fix the ops. But the imx serial is broken :-( ! have to have a look into it. --- linux-4.9/drivers/net/phy/micrel.c.orig 2018-04-19 09:37:45.648000000 +0800 +++ linux-4.9/drivers/net/phy/micrel.c 2018-04-19 09:44:54.356000000 +0800 @@ -974,6 +974,7 @@ .features = (PHY_GBIT_FEATURES | SUPPORTED_Pause), .flags = PHY_HAS_MAGICANEG | PHY_HAS_INTERRUPT, .driver_data = &ksz9021_type, + .probe = kszphy_probe, .config_init = ksz9021_config_init, .config_aneg = genphy_config_aneg, .read_status = genphy_read_status, @@ -993,6 +994,7 @@ .features = (PHY_GBIT_FEATURES | SUPPORTED_Pause), .flags = PHY_HAS_MAGICANEG | PHY_HAS_INTERRUPT, .driver_data = &ksz9021_type, + .probe = kszphy_probe, .config_init = ksz9031_config_init, .config_aneg = genphy_config_aneg, .read_status = ksz9031_read_status, [ 7.219735] Micrel KSZ9031 Gigabit PHY 2188000.ethernet-1:04: attached PHY driver [Micrel KSZ9031 Gigabit PHY] (mii_bus:phy_addr=2188000.ethernet-1:04, irq=-1) Regards Chris -- GTSYS Limited RFID Technology 9/F, Unit E, R07, Kwai Shing Industrial Building Phase 2, 42-46 Tai Lin Pai Road, Kwai Chung, N.T., Hong Kong Tel (852) 9079 9521 Disclaimer: https://www.gtsys.com.hk/email/classified.html ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2018-04-19 2:34 UTC | newest] Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2018-04-18 1:34 [Regression] net/phy/micrel.c v4.9.94 Chris Ruehl 2018-04-18 5:09 ` Chris Ruehl 2018-04-18 12:43 ` Andrew Lunn 2018-04-18 12:56 ` Andrew Lunn 2018-04-18 13:02 ` Andrew Lunn 2018-04-19 1:21 ` Chris Ruehl 2018-04-19 1:34 ` Chris Ruehl 2018-04-19 2:34 ` Chris Ruehl
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.