On Thu, Mar 11, 2021 at 09:23:56AM -0800, Florian Fainelli wrote: > On 3/11/21 5:08 AM, Greg KH wrote: > > On Wed, Mar 10, 2021 at 08:19:45PM -0800, Florian Fainelli wrote: > >> +Alex, > >> > >> On 3/10/2021 5:24 AM, gregkh@xxxxxxxxxxxxxxxxxxx wrote: > >>> From: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> > >>> > >>> This is the start of the stable review cycle for the 5.4.105 release. > >>> There are 24 patches in this series, all will be posted as a response > >>> to this one. If anyone has any issues with these being applied, please > >>> let me know. > >>> > >>> Responses should be made by Fri, 12 Mar 2021 13:23:09 +0000. > >>> Anything received after that time might be too late. > >>> > >>> The whole patch series can be found in one patch at: > >>> https://www.kernel.org/pub/linux/kernel/v5.x/stable-review/patch-5.4.105-rc1.gz > >>> or in the git tree and branch at: > >>> git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git linux-5.4.y > >>> and the diffstat can be found below. > >>> > >>> thanks, > >>> > >>> greg k-h > >> > >> I believe you need to drop "net: dsa: add GRO support via gro_cells" as > >> it causes the following kernel panic on a DSA-enabled platform: > >> > >> Configuring rgmii_2 interface > >> [ 10.170527] brcm-sf2 f0b00000.ethernet_switch rgmii_2: configuring > >> for fixed/rgmii-txid link mode > >> [ 10.179597] 8021q: adding VLAN 0 to HW filter on device rgmii_2 > >> [ 10.185608] brcm-sf2 f0b00000.ethernet_switch rgmii_2: Link is Up - > >> 1Gbps/Full - flow control off > >> [ 10.198631] IPv6: ADDRCONF(NETDEV_CHANGE): rgmii_2: link becomes ready > >> Configuring sit0 interface > >> [ 10.254346] 8<--- cut here --- > >> [ 10.257438] Unable to handle kernel paging request at virtual address > >> d6df6190 > >> [ 10.264685] pgd = (ptrval) > >> [ 10.267411] [d6df6190] *pgd=80000000007003, *pmd=00000000 > >> [ 10.272846] Internal error: Oops: 206 [#1] SMP ARM > >> [ 10.277661] Modules linked in: > >> [ 10.280739] CPU: 0 PID: 1886 Comm: sed Not tainted > >> 5.4.105-1.0pre-geff642e2af2b #4 > >> [ 10.288337] Hardware name: Broadcom STB (Flattened Device Tree) > >> [ 10.294292] PC is at gro_cells_receive+0x90/0x11c > >> [ 10.299020] LR is at dsa_switch_rcv+0x120/0x1d4 > >> [ 10.303562] pc : [<c0a57a28>] lr : [<c0b4a65c>] psr: 600f0113 > >> [ 10.309841] sp : c1d33cd0 ip : 000003e8 fp : c1d33ce4 > >> [ 10.315078] r10: c8901000 r9 : c8901000 r8 : c0b4a53c > >> [ 10.320314] r7 : c2208920 r6 : 00000000 r5 : 00000000 r4 : 00004000 > >> [ 10.326855] r3 : d6df6188 r2 : c4927000 r1 : c8adc300 r0 : c22069dc > >> [ 10.333398] Flags: nZCv IRQs on FIQs on Mode SVC_32 ISA ARM > >> Segment user > >> [ 10.340547] Control: 30c5387d Table: 04ac4c80 DAC: fffffffd > >> [ 10.346307] Process sed (pid: 1886, stack limit = 0x(ptrval)) > >> [ 10.352066] Stack: (0xc1d33cd0 to 0xc1d34000) > >> [ 10.356434] 3cc0: c8adc300 > >> c4927000 c1d33d04 c1d33ce8 > >> [ 10.364631] 3ce0: c0b4a65c c0a579a4 c1d33d24 c2208920 c1d33d24 > >> 00000000 c1d33d5c c1d33d08 > >> [ 10.372827] 3d00: c0a0b38c c0b4a548 c021e070 c2204cc8 00000000 > >> c89015c0 04b87700 c89015c0 > >> [ 10.381023] 3d20: c2208920 c1d33d24 c1d33d24 00976ec2 04b87700 > >> c8adc300 c89015c0 00000000 > >> [ 10.389218] 3d40: c1d33d74 c1d32000 00000000 c230742c c1d33dac > >> c1d33d60 c0a0b5c0 c0a0b180 > >> [ 10.397414] 3d60: 00000000 c2204cc8 00000000 c1d33d6c c1d33d6c > >> c1d33d80 c029daf8 00976ec2 > >> [ 10.405610] 3d80: 00000800 c8901540 c89015c0 c8901540 00000000 > >> 00000001 0000016c 00000162 > >> [ 10.413805] 3da0: c1d33dc4 c1d33db0 c0a0b7fc c0a0b3b8 00000000 > >> c8adc300 c1d33dfc c1d33dc8 > >> [ 10.422001] 3dc0: c0a0c660 c0a0b7e4 c8901540 c8adc300 c1d33dfc > >> c1d33de0 c8901540 c8adc300 > >> [ 10.430196] 3de0: 0000015e c8901000 00000001 0000016c c1d33e74 > >> c1d33e00 c083df00 c0a0c4fc > >> [ 10.438391] 3e00: 0000012c c22b0f14 c1d33e4c c1d33e18 c0fbd9b8 > >> c0fbd9cc c0fbd9e0 c0fbd98c > >> [ 10.446586] 3e20: 00000001 00000040 c8901500 00000001 00000000 > >> 00000000 00000000 00000000 > >> [ 10.454780] 3e40: 00000000 00000000 c02f65a0 c8901540 00000001 > >> 00000040 c22b07e4 0000012c > >> [ 10.462975] 3e60: d1003000 fffb942f c1d33edc c1d33e78 c0a0c94c > >> c083dafc d051ad80 c2204cc8 > >> [ 10.471170] 3e80: c2204cf0 c1d32000 c22b40b0 0e4a4000 c2076d80 > >> c2203d00 c022bc70 c1d33e9c > >> [ 10.479365] 3ea0: c1d33e9c c1d33ea4 c1d33ea4 00976ec2 c02f65a0 > >> c220308c 00000003 c1d32000 > >> [ 10.487561] 3ec0: c22b07e4 00000100 d1003000 00000008 c1d33f44 > >> c1d33ee0 c020238c c0a0c6cc > >> [ 10.495755] 3ee0: c1d33f14 c1d33ef0 00000001 00400000 c2203d00 > >> fffb942f c206b2e4 c2076040 > >> [ 10.503950] 3f00: c2076040 0000000a c2203080 c206b358 c1d33ee0 > >> 00000004 c90c9500 ffffe000 > >> [ 10.512145] 3f20: 00000000 00000000 00000001 c9019000 d1003000 > >> 00000000 c1d33f5c c1d33f48 > >> [ 10.520339] 3f40: c022bc70 c0202264 c2075fbc 00000000 c1d33f84 > >> c1d33f60 c027feb4 c022bb88 > >> [ 10.528535] 3f60: c226d668 c220565c d100200c c1d33fb0 d1002000 > >> d1003000 c1d33fac c1d33f88 > >> [ 10.536730] 3f80: c0202214 c027fe50 b6f1e0b6 000f0030 ffffffff > >> 30c5387d 30c5387d 00000000 > >> [ 10.544924] 3fa0: 00000000 c1d33fb0 c0201d8c c02021c4 b6ec1778 > >> b6f32094 62632e73 62740000 > >> [ 10.553120] 3fc0: 00172e73 00001700 b6f3293c 00000001 00000001 > >> 00000000 00000000 00000006 > >> [ 10.561316] 3fe0: ffffffff bef37ac8 b6f18151 b6f1e0b6 000f0030 > >> ffffffff 00000000 00000000 > >> [ 10.569508] Backtrace: > >> [ 10.571970] [<c0a57998>] (gro_cells_receive) from [<c0b4a65c>] > >> (dsa_switch_rcv+0x120/0x1d4) > >> [ 10.580338] r5:c4927000 r4:c8adc300 > >> [ 10.583929] [<c0b4a53c>] (dsa_switch_rcv) from [<c0a0b38c>] > >> (__netif_receive_skb_list_core+0x218/0x238) > >> [ 10.593343] r7:00000000 r6:c1d33d24 r5:c2208920 r4:c1d33d24 > >> [ 10.599017] [<c0a0b174>] (__netif_receive_skb_list_core) from > >> [<c0a0b5c0>] (netif_receive_skb_list_internal+0x214/0x2dc) > >> [ 10.609909] r10:c230742c r9:00000000 r8:c1d32000 r7:c1d33d74 > >> r6:00000000 r5:c89015c0 > >> [ 10.617755] r4:c8adc300 > >> [ 10.620300] [<c0a0b3ac>] (netif_receive_skb_list_internal) from > >> [<c0a0b7fc>] (gro_normal_list.part.42+0x24/0x38) > >> [ 10.630496] r10:00000162 r9:0000016c r8:00000001 r7:00000000 > >> r6:c8901540 r5:c89015c0 > >> [ 10.638342] r4:c8901540 > >> [ 10.640885] [<c0a0b7d8>] (gro_normal_list.part.42) from [<c0a0c660>] > >> (napi_complete_done+0x170/0x1d0) > >> [ 10.650123] r5:c8adc300 r4:00000000 > >> [ 10.653712] [<c0a0c4f0>] (napi_complete_done) from [<c083df00>] > >> (bcm_sysport_poll+0x410/0x4b4) > >> [ 10.662343] r9:0000016c r8:00000001 r7:c8901000 r6:0000015e > >> r5:c8adc300 r4:c8901540 > >> [ 10.670104] [<c083daf0>] (bcm_sysport_poll) from [<c0a0c94c>] > >> (net_rx_action+0x28c/0x44c) > >> [ 10.678301] r10:fffb942f r9:d1003000 r8:0000012c r7:c22b07e4 > >> r6:00000040 r5:00000001 > >> [ 10.686146] r4:c8901540 > >> [ 10.688693] [<c0a0c6c0>] (net_rx_action) from [<c020238c>] > >> (__do_softirq+0x134/0x414) > >> [ 10.696542] r10:00000008 r9:d1003000 r8:00000100 r7:c22b07e4 > >> r6:c1d32000 r5:00000003 > >> [ 10.704387] r4:c220308c > >> [ 10.706931] [<c0202258>] (__do_softirq) from [<c022bc70>] > >> (irq_exit+0xf4/0x100) > >> [ 10.714257] r10:00000000 r9:d1003000 r8:c9019000 r7:00000001 > >> r6:00000000 r5:00000000 > >> [ 10.722104] r4:ffffe000 > >> [ 10.724650] [<c022bb7c>] (irq_exit) from [<c027feb4>] > >> (__handle_domain_irq+0x70/0xc4) > >> [ 10.732497] r5:00000000 r4:c2075fbc > >> [ 10.736083] [<c027fe44>] (__handle_domain_irq) from [<c0202214>] > >> (gic_handle_irq+0x5c/0xa0) > >> [ 10.744453] r9:d1003000 r8:d1002000 r7:c1d33fb0 r6:d100200c > >> r5:c220565c r4:c226d668 > >> [ 10.752215] [<c02021b8>] (gic_handle_irq) from [<c0201d8c>] > >> (__irq_usr+0x4c/0x60) > >> [ 10.759713] Exception stack(0xc1d33fb0 to 0xc1d33ff8) > >> [ 10.764776] 3fa0: b6ec1778 > >> b6f32094 62632e73 62740000 > >> [ 10.772973] 3fc0: 00172e73 00001700 b6f3293c 00000001 00000001 > >> 00000000 00000000 00000006 > >> [ 10.781167] 3fe0: ffffffff bef37ac8 b6f18151 b6f1e0b6 000f0030 ffffffff > >> [ 10.787797] r9:00000000 r8:30c5387d r7:30c5387d r6:ffffffff > >> r5:000f0030 r4:b6f1e0b6 > >> [ 10.795559] Code: e30609dc e34c0220 e083300c e590c000 (e5930008) > >> [ 10.801670] ---[ end trace 97c3942fa73eff4c ]--- > >> [ 10.806300] Kernel panic - not syncing: Fatal exception in interrupt > >> [ 10.812678] CPU2: stopping > >> [ 10.815403] CPU: 2 PID: 0 Comm: swapper/2 Tainted: G D > >> 5.4.105-1.0pre-geff642e2af2b #4 > >> [ 10.824641] Hardware name: Broadcom STB (Flattened Device Tree) > >> [ 10.830573] Backtrace: > >> [ 10.833036] [<c020dd30>] (dump_backtrace) from [<c020e04c>] > >> (show_stack+0x20/0x24) > >> [ 10.840624] r7:c22a86d0 r6:00000000 r5:600c0193 r4:c22a86d0 > >> [ 10.846302] [<c020e02c>] (show_stack) from [<c0c09924>] > >> (dump_stack+0xb8/0xe4) > >> [ 10.853546] [<c0c0986c>] (dump_stack) from [<c0212160>] > >> (handle_IPI+0x344/0x3cc) > >> [ 10.860960] r10:00000000 r9:c2205400 r8:00000000 r7:00000002 > >> r6:c22b0744 r5:00000004 > >> [ 10.868807] r4:c22ce308 r3:00976ec2 > >> [ 10.872395] [<c0211e1c>] (handle_IPI) from [<c0202254>] > >> (gic_handle_irq+0x9c/0xa0) > >> [ 10.879983] r10:00000000 r9:d1003000 r8:d1002000 r7:c9131f18 > >> r6:d100200c r5:c220565c > >> [ 10.887829] r4:c226d668 > >> [ 10.890373] [<c02021b8>] (gic_handle_irq) from [<c0201a3c>] > >> (__irq_svc+0x5c/0x7c) > >> [ 10.897871] Exception stack(0xc9131f18 to 0xc9131f60) > >> [ 10.902934] 1f00: > >> c020a47c 00000000 > >> [ 10.911131] 1f20: 0e4ca000 600c0093 c9130000 c2204cf0 c2204d34 > >> 00000004 00000000 c20757b0 > >> [ 10.919327] 1f40: 00000000 c9131f74 c9130000 c9131f68 00000000 > >> c020a480 600c0013 ffffffff > >> [ 10.927523] r9:c9130000 r8:00000000 r7:c9131f4c r6:ffffffff > >> r5:600c0013 r4:c020a480 > >> [ 10.935291] [<c020a44c>] (arch_cpu_idle) from [<c0c1316c>] > >> (default_idle_call+0x34/0x48) > >> [ 10.943403] [<c0c13138>] (default_idle_call) from [<c02580c0>] > >> (do_idle+0x1d4/0x2c0) > >> [ 10.951164] [<c0257eec>] (do_idle) from [<c0258494>] > >> (cpu_startup_entry+0x28/0x2c) > >> [ 10.958752] r10:00000000 r9:420f00f3 r8:00007000 r7:c22ce318 > >> r6:30c0387d r5:00000002 > >> [ 10.966598] r4:0000008a > >> [ 10.969142] [<c025846c>] (cpu_startup_entry) from [<c0211644>] > >> (secondary_start_kernel+0x17c/0x1a0) > >> [ 10.978210] [<c02114c8>] (secondary_start_kernel) from [<0020270c>] > >> (0x20270c) > >> [ 10.985447] r5:00000000 r4:090eaa40 > >> [ 10.989032] CPU3: stopping > >> [ 10.991754] CPU: 3 PID: 0 Comm: swapper/3 Tainted: G D > >> 5.4.105-1.0pre-geff642e2af2b #4 > >> [ 11.000992] Hardware name: Broadcom STB (Flattened Device Tree) > >> [ 11.006924] Backtrace: > >> [ 11.009384] [<c020dd30>] (dump_backtrace) from [<c020e04c>] > >> (show_stack+0x20/0x24) > >> [ 11.016972] r7:c22a86d0 r6:00000000 r5:600f0193 r4:c22a86d0 > >> [ 11.022649] [<c020e02c>] (show_stack) from [<c0c09924>] > >> (dump_stack+0xb8/0xe4) > >> [ 11.029892] [<c0c0986c>] (dump_stack) from [<c0212160>] > >> (handle_IPI+0x344/0x3cc) > >> [ 11.037307] r10:00000000 r9:c2205400 r8:00000000 r7:00000003 > >> r6:c22b0744 r5:00000004 > >> [ 11.045154] r4:c22ce308 r3:00976ec2 > >> [ 11.048741] [<c0211e1c>] (handle_IPI) from [<c0202254>] > >> (gic_handle_irq+0x9c/0xa0) > >> [ 11.056329] r10:00000000 r9:d1003000 r8:d1002000 r7:c9133f18 > >> r6:d100200c r5:c220565c > >> [ 11.064176] r4:c226d668 > >> [ 11.066719] [<c02021b8>] (gic_handle_irq) from [<c0201a3c>] > >> (__irq_svc+0x5c/0x7c) > >> [ 11.074216] Exception stack(0xc9133f18 to 0xc9133f60) > >> [ 11.079280] 3f00: > >> c020a47c 00000000 > >> [ 11.087476] 3f20: 0e4dd000 600f0093 c9132000 c2204cf0 c2204d34 > >> 00000008 00000000 c20757b0 > >> [ 11.095671] 3f40: 00000000 c9133f74 c9132000 c9133f68 00000000 > >> c020a480 600f0013 ffffffff > >> [ 11.103867] r9:c9132000 r8:00000000 r7:c9133f4c r6:ffffffff > >> r5:600f0013 r4:c020a480 > >> [ 11.111633] [<c020a44c>] (arch_cpu_idle) from [<c0c1316c>] > >> (default_idle_call+0x34/0x48) > >> [ 11.119744] [<c0c13138>] (default_idle_call) from [<c02580c0>] > >> (do_idle+0x1d4/0x2c0) > >> [ 11.127506] [<c0257eec>] (do_idle) from [<c0258494>] > >> (cpu_startup_entry+0x28/0x2c) > >> [ 11.135093] r10:00000000 r9:420f00f3 r8:00007000 r7:c22ce318 > >> r6:30c0387d r5:00000003 > >> [ 11.142939] r4:0000008a > >> [ 11.145484] [<c025846c>] (cpu_startup_entry) from [<c0211644>] > >> (secondary_start_kernel+0x17c/0x1a0) > >> [ 11.154550] [<c02114c8>] (secondary_start_kernel) from [<0020270c>] > >> (0x20270c) > >> [ 11.161788] r5:00000000 r4:090eaa40 > >> [ 11.165372] CPU1: stopping > >> [ 11.168094] CPU: 1 PID: 0 Comm: swapper/1 Tainted: G D > >> 5.4.105-1.0pre-geff642e2af2b #4 > >> [ 11.177332] Hardware name: Broadcom STB (Flattened Device Tree) > >> [ 11.183264] Backtrace: > >> [ 11.185723] [<c020dd30>] (dump_backtrace) from [<c020e04c>] > >> (show_stack+0x20/0x24) > >> [ 11.193311] r7:c22a86d0 r6:00000000 r5:600c0193 r4:c22a86d0 > >> [ 11.198988] [<c020e02c>] (show_stack) from [<c0c09924>] > >> (dump_stack+0xb8/0xe4) > >> [ 11.206230] [<c0c0986c>] (dump_stack) from [<c0212160>] > >> (handle_IPI+0x344/0x3cc) > >> [ 11.213644] r10:00000000 r9:c2205400 r8:00000000 r7:00000001 > >> r6:c22b0744 r5:00000004 > >> [ 11.221491] r4:c22ce308 r3:00976ec2 > >> [ 11.225078] [<c0211e1c>] (handle_IPI) from [<c0202254>] > >> (gic_handle_irq+0x9c/0xa0) > >> [ 11.232666] r10:00000000 r9:d1003000 r8:d1002000 r7:c912ff18 > >> r6:d100200c r5:c220565c > >> [ 11.240512] r4:c226d668 > >> [ 11.243054] [<c02021b8>] (gic_handle_irq) from [<c0201a3c>] > >> (__irq_svc+0x5c/0x7c) > >> [ 11.250553] Exception stack(0xc912ff18 to 0xc912ff60) > >> [ 11.255617] ff00: > >> c020a47c 00000000 > >> [ 11.263814] ff20: 0e4b7000 600c0093 c912e000 c2204cf0 c2204d34 > >> 00000002 00000000 c20757b0 > >> [ 11.272010] ff40: 00000000 c912ff74 c912e000 c912ff68 00000000 > >> c020a480 600c0013 ffffffff > >> [ 11.280206] r9:c912e000 r8:00000000 r7:c912ff4c r6:ffffffff > >> r5:600c0013 r4:c020a480 > >> [ 11.287970] [<c020a44c>] (arch_cpu_idle) from [<c0c1316c>] > >> (default_idle_call+0x34/0x48) > >> [ 11.296081] [<c0c13138>] (default_idle_call) from [<c02580c0>] > >> (do_idle+0x1d4/0x2c0) > >> [ 11.303842] [<c0257eec>] (do_idle) from [<c0258494>] > >> (cpu_startup_entry+0x28/0x2c) > >> [ 11.311430] r10:00000000 r9:420f00f3 r8:00007000 r7:c22ce318 > >> r6:30c0387d r5:00000001 > >> [ 11.319275] r4:0000008a > >> [ 11.321819] [<c025846c>] (cpu_startup_entry) from [<c0211644>] > >> (secondary_start_kernel+0x17c/0x1a0) > >> [ 11.330886] [<c02114c8>] (secondary_start_kernel) from [<0020270c>] > >> (0x20270c) > >> [ 11.338124] r5:00000000 r4:090eaa40 > >> [ 11.341729] ---[ end Kernel panic - not syncing: Fatal exception in > >> interrupt ]--- > >> > >> it is not marked as fixing anything so I wonder how it landed in stable? > > > > It was requested to be merged. > > OK. > > > Do you have the same crash on newer kernels with this commit in it (like Linus's tree?) > > There are no crashes with 5.10 or upstream otherwise it would have been > noticed earlier, I have not kept track of which additional changes/fixes > we may need but I would suggest we drop this one for now. > > Alexander, do you want me to test additional patches if your change > somehow must be included in an upcoming 5.4? I thought the platform you > are working is still not upstream, so who is going to benefit from this > performance improvement? Is 4.19 also failing for you now? thanks, greg k-h