Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Subject: Re: [PATCH 5.4 00/24] 5.4.105-rc1 review
To:     Greg KH <gregkh@linuxfoundation.org>
Cc:     linux-kernel@vger.kernel.org, Alexander Lobakin <alobakin@pm.me>,
        torvalds@linux-foundation.org, akpm@linux-foundation.org,
        linux@roeck-us.net, shuah@kernel.org, patches@kernelci.org,
        lkft-triage@lists.linaro.org, pavel@denx.de, jonathanh@nvidia.com,
        stable@vger.kernel.org
References: <20210310132320.550932445@linuxfoundation.org>
 <29dcd801-7f1e-ae09-9b88-ce17cb096f60@gmail.com> <YEoWT85kGVYbBnKY@kroah.com>
From:   Florian Fainelli <f.fainelli@gmail.com>
Message-ID: <61cef8f0-c40a-c4e4-5322-9939ed21bff7@gmail.com>
Date:   Thu, 11 Mar 2021 09:23:56 -0800
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101
 Thunderbird/78.7.1
MIME-Version: 1.0
In-Reply-To: <YEoWT85kGVYbBnKY@kroah.com>
Content-Type: text/plain; charset=utf-8
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Precedence: bulk

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@linuxfoundation.org wrote:
>>> From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
>>>
>>> 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?
--
Florian