Re: [meta-freescale] New branch 5.4-2.1.x-imx for linux-fslc

["Philippe Schenker" <philippe.schenker@toradex.com>]

On Mon, 2020-08-10 at 22:02 +0200, Andrey Zhizhikin wrote:
> On Mon, Aug 10, 2020 at 5:33 PM Philippe Schenker
> <philippe.schenker@toradex.com> wrote:
> > On Mon, 2020-08-10 at 16:10 +0200, Andrey Zhizhikin wrote:
> > > Hey Philippe,
> > > 
> > > On Mon, Aug 10, 2020 at 1:52 PM Philippe Schenker
> > > <philippe.schenker@toradex.com> wrote:
> > > > On Mon, 2020-08-10 at 12:42 +0200, Andrey Zhizhikin wrote:
> > > > > Hello Philippe,
> > > > > 
> > > > > On Mon, Aug 10, 2020 at 12:01 PM Philippe Schenker via
> > > > > lists.yoctoproject.org
> > > > > <philippe.schenker=toradex.com@lists.yoctoproject.org> wrote:
> > > > > > Hi,
> > > > > > 
> > > > > > Toradex is currently working on integrating the new NXP
> > > > > > release
> > > > > > 'L5.4.24_2.1.0'. For this we created a new kernel branch
> > > > > > '5.4-
> > > > > > 2.1.x-
> > > > > > imx'
> > > > > > based on NXP's kernel 'imx_5.4.24_2.1.0'.
> > > > > > We already merged patches from stable linux 5.4.47 into
> > > > > > '5.4-
> > > > > > 2.1.x-
> > > > > > imx'.
> > > > > 
> > > > > This is great news! I was half the way of working on the same
> > > > > update,
> > > > > but now since you have the merge done - I can take your
> > > > > version
> > > > > instead.
> > > > > 
> > > > > > @Otavio: I sincerely ask you if you could place the branch
> > > > > > '5.4-
> > > > > > 2.1.x-
> > > > > > imx' that resides currently in my private github account:
> > > > > > 
> > > > > > https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fphilschenker%2Flinux-fslc%2Ftree%2F5.4-2.1.x-imx&amp;data=01%7C01%7Cphilippe.schenker%40toradex.com%7C1b2edaea486b43f0d2bf08d83d1a29be%7Cd99958660d9b42518315093f062abab4%7C0&amp;sdata=BZDTVKY%2BQJDwIgsBjAeRshdTw2i7Ksh0RjGtw92ZZHw%3D&amp;reserved=0
> > > > > 
> > > > > I'll grab this one to test on i.MX8M Mini EVK now and try it
> > > > > out.
> > > 
> > > I've tried to build the branch from your repo with latest master
> > > of
> > > meta-freescale, and saw couple of issue with imx8mmevk machine:
> > > 
> > > 1. imx-gpu-sdk recipe fails to build, as it requires the header
> > > which
> > > is most probably missing in the kernel:
> > > > make: *** No rule to make target
> > > '/development/projects/master/build-output/work/cortexa53-crypto-
> > > mx8mm-fsl-linux/imx-gpu-sdk/5.0.2-r0/recipe-
> > > sysroot/usr/include/bits/sys_errlist.h',
> > > needed by
> > > 'build/Yocto/obj/Release/source/FslBase/Bits/ByteArrayUtil.o'.
> > > Stop.
> > 
> > I guess any user-space stuff need updating to versions that NXP
> > provides
> > with BSP 'L5.4.24_2.1.0'.
> > 
> > OE-core updated glibc which is the provider of 'sys_errlist.h' so
> > maybe
> > that is the reason for build failure?
> 
> Correct! My bad: I did a pull of OE-Core and then your Kernel branch,
> which I should not have been doing in the first place. This issue is
> not from Kernel, it is a pure glibc one. Thanks for pointing it out!
> 
> > > 2. Kernel panics on the startup, there is something related to FEC
> > > driver:
> > > [    0.994893] Unable to handle kernel paging request at virtual
> > > address 0000000000023779
> > > [    1.002814] Mem abort info:
> > > [    1.005614]   ESR = 0x96000004
> > > [    1.008668]   EC = 0x25: DABT (current EL), IL = 32 bits
> > > [    1.013983]   SET = 0, FnV = 0
> > > [    1.017040]   EA = 0, S1PTW = 0
> > > [    1.020178] Data abort info:
> > > [    1.023062]   ISV = 0, ISS = 0x00000004
> > > [    1.026901]   CM = 0, WnR = 0
> > > [    1.029872] [0000000000023779] user address but active_mm is
> > > swapper
> > > [    1.036262] Internal error: Oops: 96000004 [#1] PREEMPT SMP
> > > [    1.041833] Modules linked in:
> > > [    1.044891] CPU: 2 PID: 1 Comm: swapper/0 Not tainted
> > > 5.4.47+g1310c126dd1e #1
> > > [    1.052022] Hardware name: FSL i.MX8MM EVK board (DT)
> > > [    1.057073] pstate: 80000005 (Nzcv daif -PAN -UAO)
> > > [    1.061868] pc : fec_probe+0x280/0x1340
> > > [    1.065703] lr : fec_probe+0x80/0x1340
> > > [    1.069449] sp : ffff80001003baf0
> > > [    1.072761] x29: ffff80001003baf0 x28: 0000000000000000
> > > [    1.078072] x27: 0000000000000000 x26: ffff80001114d9b8
> > > [    1.083383] x25: 0000000000000000 x24: ffff000076090000
> > > [    1.088694] x23: ffff00007dc03bd8 x22: ffff000076404c10
> > > [    1.094005] x21: ffff000076404c00 x20: ffff0000762f8000
> > > [    1.099316] x19: ffff0000762f87c0 x18: 0000000000000020
> > > [    1.104626] x17: 0000000000000001 x16: 0000000000000019
> > > [    1.109937] x15: ffff000076090470 x14: 0000000000000000
> > > [    1.115248] x13: ffff000076090000 x12: 0000000000000000
> > > [    1.120559] x11: 0101010101010101 x10: 7f7f7f7f7f7f7f7f
> > > [    1.125870] x9 : 49fffeff06fefeff x8 : 7f7f7f7f7f7f7f7f
> > > [    1.131180] x7 : 01fefefefefefeff x6 : 8080808080808000
> > > [    1.136491] x5 : 0080808080808080 x4 : 0000000000000000
> > > [    1.141802] x3 : ffff80001114ea48 x2 : 0000000000000000
> > > [    1.147113] x1 : ffff000076090000 x0 : 0000000000023779
> > > [    1.152424] Call trace:
> > > [    1.154871]  fec_probe+0x280/0x1340
> > > [    1.158361]  platform_drv_probe+0x50/0xa0
> > > [    1.162371]  really_probe+0xd8/0x410
> > > [    1.165946]  driver_probe_device+0x54/0xe4
> > > [    1.170042]  device_driver_attach+0xb4/0xc0
> > > [    1.174225]  __driver_attach+0x80/0x114
> > > [    1.178060]  bus_for_each_dev+0x6c/0xbc
> > > [    1.181895]  driver_attach+0x20/0x30
> > > [    1.185469]  bus_add_driver+0xfc/0x1e0
> > > [    1.189218]  driver_register+0x74/0x120
> > > [    1.193052]  __platform_driver_register+0x44/0x50
> > > [    1.197756]  fec_driver_init+0x18/0x20
> > > [    1.201506]  do_one_initcall+0x50/0x190
> > > [    1.205343]  kernel_init_freeable+0x194/0x22c
> > > [    1.209700]  kernel_init+0x10/0x100
> > > [    1.213188]  ret_from_fork+0x10/0x18
> > > [    1.216765] Code: 2a1303e2 b940afe1 b900b3f3 17ffff75
> > > (b9400000)
> > > [    1.222866] ---[ end trace 31a90d201ff67100 ]---
> > > [    1.227505] Kernel panic - not syncing: Attempted to kill init!
> > > exitcode=0x0000000b
> > > [    1.235161] SMP: stopping secondary CPUs
> > > [    1.239085] Kernel Offset: disabled
> > > [    1.242572] CPU features: 0x0002,2000200c
> > > [    1.246579] Memory Limit: none
> > > [    1.249634] ---[ end Kernel panic - not syncing: Attempted to
> > > kill
> > > init! exitcode=0x0000000b ]---
> > 
> > Thanks for pointing this out, I could reproduce the issue and
> > checked.
> > There were still two commits missing. One revert that we believe is
> > not
> > applicable to downstream fec_main.c
> > This one also solved the issue. The other IPU commit was also
> > missing.
> > Now it works on our side. Can you confirm?
> 
> Yeap, Kernel build off commit 1e10771673463a332a7e477fcba0b9488ec5362b
> boots fine on i.MX8M Mini EVK. I haven't verified whether graphics and
> multimedia are OK, but in general at this state this merge is
> absolutely fine!
> 
> I believe you can make a PR in https://github.com/Freescale/linux-fslc
> for Otavio to have a look at it.

Thanks Andrey for testing!
I guess Otavio needs to pull it from my branch, I'm not able to create a
new branch as PR.

Philippe
> 
> With that said:
> Tested-By: Andrey Zhizhikin <andrey.z@gmail.com>
> 
> > Philippe
> > > 
> > > > > Have you ported the patch [1] from upstream? Without it - the
> > > > > latest
> > > > > master build would fail (it is needed for binutils-2.35).
> > > > 
> > > > Hi Andrey,
> > > > 
> > > > We were building binutils-2.34 so didn't notice this issue. I've
> > > > cherry-
> > > > picked the patch you mentioned.
> > > 
> > > That's great, I've synced up the tree to your latest commit and
> > > the
> > > perf build build went through.
> > > 
> > > > Philippe
> > > > > > into the repository
> > > > > > 
> > > > > > https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FFreescale%2Flinux-fslc&amp;data=01%7C01%7Cphilippe.schenker%40toradex.com%7C1b2edaea486b43f0d2bf08d83d1a29be%7Cd99958660d9b42518315093f062abab4%7C0&amp;sdata=qkHhXyQfUuvj1C9u3FbPaqrmx6TsszVHpiRi9azUyR0%3D&amp;reserved=0
> > > > > > 
> > > > > > Thanks and best regards,
> > > > > > Philippe
> > > > > > 
> > > > > 
> > > > > [1]:
> > > > > https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgit.kernel.org%2Fpub%2Fscm%2Flinux%2Fkernel%2Fgit%2Ftorvalds%2Flinux.git%2Fcommit%2F%3Fid%3D39efdd94e314336f4acbac4c07e0f37bdc3bef71&amp;data=01%7C01%7Cphilippe.schenker%40toradex.com%7C1b2edaea486b43f0d2bf08d83d1a29be%7Cd99958660d9b42518315093f062abab4%7C0&amp;sdata=gDSV0ochz6GpX1DQ6EU%2Bn6BtpvDVBNbgPkWGWEqNegY%3D&amp;reserved=0
> > > > > 
> 
>