On Mon, Dec 10, 2018 at 11:21:47AM +0100, Thierry Reding wrote:
> On Sat, Dec 08, 2018 at 02:54:45PM +0000, Marcel Ziswiler wrote:
> > Hi Thierry et al.
> >
> > I noticed that since commit 3dde5a2342cd ("ARM: tegra: Add VIC on
> > Tegra124") graphics on Apalis TK1 is broken. During boot it fails
> > loading the vic firmware:
> >
> > [ 1.595824] tegra-vic 54340000.vic: Direct firmware load for
> > nvidia/tegra124/vic03_ucode.bin failed with error -2
> > [ 1.606140] tegra-vic: probe of 54340000.vic failed with error -2
> >
> > Subsequently Tegra HDMI seems to fail completely:
> >
> > [ 2.379860] tegra-hdmi 54280000.hdmi: failed to get PLL regulator
> >
> > And finally, Nouveau even crashes:
> >
> > [ 8.241115] nouveau 57000000.gpu: Linked as a consumer to
> > regulator.31
> > [ 8.247889] nouveau 57000000.gpu: NVIDIA GK20A (0ea000a1)
> > [ 8.253396] nouveau 57000000.gpu: imem: using IOMMU
> > [ 8.270210] Unable to handle kernel NULL pointer dereference at
> > virtual address 0000006c
> > [ 8.278340] pgd = (ptrval)
> > [ 8.281250] [0000006c] *pgd=00000000
> > [ 8.284944] Internal error: Oops: 5 [#1] PREEMPT SMP ARM
> > [ 8.290260] Modules linked in: nouveau(+) ttm
> > [ 8.294625] CPU: 2 PID: 203 Comm: systemd-udevd Not tainted 4.20.0-
> > rc5-next-20181207-00008-g85b0f8e25f86-dirty #110
> > [ 8.305055] Hardware name: NVIDIA Tegra SoC (Flattened Device Tree)
> > [ 8.311331] PC is at drm_plane_register_all+0x18/0x50
> > [ 8.316373] LR is at drm_modeset_register_all+0xc/0x70
> > [ 8.321513] pc : [<c056200c>] lr : [<c0564cc8>] psr: a0060013
> > [ 8.327768] sp : ed527c70 ip : ecc43ec0 fp : 00000000
> > [ 8.332993] r10: 00000016 r9 : ecc43e80 r8 : 00000000
> > [ 8.338209] r7 : bf182c80 r6 : 00000000 r5 : ed61b24c r4 :
> > fffffffc
> > [ 8.344735] r3 : 0002f000 r2 : ffffffff r1 : 2e124000 r0 :
> > ed61b000
> > [ 8.351260] Flags: NzCv IRQs on FIQs on Mode SVC_32 ISA
> > ARM Segment none
> > [ 8.358383] Control: 10c5387d Table: ad64c06a DAC: 00000051
> > [ 8.364127] Process systemd-udevd (pid: 203, stack limit =
> > 0x(ptrval))
> > [ 8.370654] Stack: (0xed527c70 to 0xed528000)
> > [ 8.375004] 7c60: ed61b000
> > ed61b000 00000000 c0564cc8
> > [ 8.383177] 7c80: ed61b000 00000000 00000000 c054b5b8 00000001
> > 00000001 ffffffff ffffffff
> > [ 8.391355] 7ca0: ed527cc0 c0f08c48 ed61b000 00000000 00000000
> > 00000000 bf180c5c bf0dc900
> > [ 8.399531] 7cc0: eda29208 5dfe844b 00000000 ee9f2a10 00000000
> > bf180c5c 00000000 c05a9328
> > [ 8.407695] 7ce0: c1006828 ee9f2a10 c100682c 00000000 00000000
> > c05a744c ee9f2a10 bf180c5c
> > [ 8.415871] 7d00: ee9f2a44 c05a77a8 00000000 c0f08c48 bf182980
> > c05a769c eefd14d0 c05a77a8
> > [ 8.424048] 7d20: 00000000 ee9f2a10 bf180c5c ee9f2a44 c05a77a8
> > 00000000 c0f08c48 bf182980
> > [ 8.432226] 7d40: 00000000 c05a7884 ee9ebfb4 c0f08c48 bf180c5c
> > c05a5790 00000000 ee88135c
> > [ 8.440405] 7d60: ee9ebfb4 5dfe844b c0f71168 bf180c5c ee379e80
> > c0f71168 00000000 c05a692c
> > [ 8.448570] 7d80: bf15dc00 bf180ac8 ffffe000 bf180c5c bf180ac8
> > ffffe000 bf1aa000 c05a84a0
> > [ 8.456746] 7da0: bf182b80 bf180ac8 ffffe000 bf1aa170 c0fbd220
> > c0f08c48 ffffe000 c0102ed0
> > [ 8.464924] 7dc0: ed53f4c0 006000c0 c01b3d98 0000000c 60000113
> > bf182980 00000040 c02592d0
> > [ 8.473102] 7de0: eda60200 2e124000 ee800000 006000c0 006000c0
> > c01b3d98 0000000c c025a8cc
> > [ 8.481281] 7e00: c024ce54 a0000113 bf182980 5dfe844b bf182980
> > 00000002 ed53f4c0 00000002
> > [ 8.489459] 7e20: eceba000 c01b3dd4 c0f08c48 bf182980 00000000
> > ed527f40 00000002 eceb9fc0
> > [ 8.497625] 7e40: 00000002 c01b61a4 bf18298c 00007fff bf182980
> > c01b2f88 00000000 c01b279c
> > [ 8.505800] 7e60: bf1829c8 bf182a80 bf182b6c bf182ab0 c0b03ab0
> > c0d58964 c0ca726c c0ca7278
> > [ 8.513978] 7e80: c0ca72d0 c0f08c48 00000000 c02654a0 00000000
> > 00000000 ffffe000 bf000000
> > [ 8.522157] 7ea0: 00000000 00000000 00000000 00000000 00000000
> > 00000000 6e72656b 00006c65
> > [ 8.530336] 7ec0: 00000000 00000000 00000000 00000000 00000000
> > 00000000 00000000 00000000
> > [ 8.538502] 7ee0: 00000000 00000000 00000000 00000000 00000000
> > 5dfe844b 7fffffff c0f08c48
> > [ 8.546677] 7f00: 00000000 0000000f b6f761cc c0101204 ed526000
> > 0000017b 004a3270 c01b66a4
> > [ 8.554855] 7f20: 7fffffff 00000000 00000003 00000001 004a3270
> > f0ced000 06e8994c 00000000
> > [ 8.563032] 7f40: f0e37f3a f0e50a40 f0ced000 06e8994c f7b75f9c
> > f7b75d34 f63e62dc 0016b000
> > [ 8.571209] 7f60: 0017f6f0 00000000 00000000 00000000 00050a48
> > 0000003b 0000003c 00000023
> > [ 8.579388] 7f80: 00000000 00000014 00000000 5dfe844b 00000000
> > 004c0ec0 00000000 00000001
> > [ 8.587554] 7fa0: 0000017b c0101000 004c0ec0 00000000 0000000f
> > b6f761cc 00000000 00020000
> > [ 8.595730] 7fc0: 004c0ec0 00000000 00000001 0000017b 0048e114
> > 00000000 00000000 004a3270
> > [ 8.603908] 7fe0: bea8f990 bea8f980 b6f71269 b6e9f6c0 400d0010
> > 0000000f 00000000 00000000
> > [ 8.612096] [<c056200c>] (drm_plane_register_all) from [<c0564cc8>]
> > (drm_modeset_register_all+0xc/0x70)
> > [ 8.621499] [<c0564cc8>] (drm_modeset_register_all) from
> > [<c054b5b8>] (drm_dev_register+0x168/0x1c4)
> > [ 8.630855] [<c054b5b8>] (drm_dev_register) from [<bf0dc900>]
> > (nouveau_platform_probe+0x6c/0x88 [nouveau])
> > [ 8.640739] [<bf0dc900>] (nouveau_platform_probe [nouveau]) from
> > [<c05a9328>] (platform_drv_probe+0x48/0x98)
> > [ 8.650574] [<c05a9328>] (platform_drv_probe) from [<c05a744c>]
> > (really_probe+0x1e0/0x2cc)
> > [ 8.658827] [<c05a744c>] (really_probe) from [<c05a769c>]
> > (driver_probe_device+0x60/0x16c)
> > [ 8.667096] [<c05a769c>] (driver_probe_device) from [<c05a7884>]
> > (__driver_attach+0xdc/0xe0)
> > [ 8.675543] [<c05a7884>] (__driver_attach) from [<c05a5790>]
> > (bus_for_each_dev+0x74/0xb4)
> > [ 8.683729] [<c05a5790>] (bus_for_each_dev) from [<c05a692c>]
> > (bus_add_driver+0x1c0/0x204)
> > [ 8.692004] [<c05a692c>] (bus_add_driver) from [<c05a84a0>]
> > (driver_register+0x74/0x108)
> > [ 8.700324] [<c05a84a0>] (driver_register) from [<bf1aa170>]
> > (nouveau_drm_init+0x170/0x1000 [nouveau])
> > [ 8.709857] [<bf1aa170>] (nouveau_drm_init [nouveau]) from
> > [<c0102ed0>] (do_one_initcall+0x54/0x284)
> > [ 8.718980] [<c0102ed0>] (do_one_initcall) from [<c01b3dd4>]
> > (do_init_module+0x64/0x214)
> > [ 8.727079] [<c01b3dd4>] (do_init_module) from [<c01b61a4>]
> > (load_module+0x21b8/0x246c)
> > [ 8.735094] [<c01b61a4>] (load_module) from [<c01b66a4>]
> > (sys_finit_module+0xc4/0xdc)
> > [ 8.742937] [<c01b66a4>] (sys_finit_module) from [<c0101000>]
> > (ret_fast_syscall+0x0/0x54)
> > [ 8.751114] Exception stack(0xed527fa8 to 0xed527ff0)
> > [ 8.756157] 7fa0: 004c0ec0 00000000 0000000f
> > b6f761cc 00000000 00020000
> > [ 8.764333] 7fc0: 004c0ec0 00000000 00000001 0000017b 0048e114
> > 00000000 00000000 004a3270
> > [ 8.772510] 7fe0: bea8f990 bea8f980 b6f71269 b6e9f6c0
> > [ 8.777556] Code: e5b5424c e1550004 0a00000c e2444004 (e5943070)
> > [ 8.784011] ---[ end trace ad8c21587c118655 ]---
> >
> > Of course my root file system does include resp. vic firmware:
> >
> > 7ef01d2e3f507c91ca79584e89edcc64 /lib/firmware/nvidia/tegra124/vic03_u
> > code.bin
> >
> > If I bake that one into the kernel binary, Nouveau still crashes like
> > above albeit VIC loading and Tegra DRM now at least showing something
> > on HDMI.
>
> Yeah, this is a fairly common pitfall. The general rule of thumb is that
> the firmware has to live on the same medium as the module. So if you've
> built Tegra DRM as a loadable kernel module and installed it in the root
> filesystem, then that's where your firmware file also needs to be. If
> the driver is built-in (or a loadable module installed in the initial
> ramdisk), then the firmware needs to be in the initial ramdisk (or built
> into the kernel image itself). That's somewhat annoying, but it is what
> it is. At least it's logical.
>
> > Just reverting above mentioned commit still leaves Nouveau crashing.
> >
> > This has been observed using latest next-20181207.
> >
> > Does anybody know what exactly is going on and how exactly one may get
> > graphics working again as before?
>
> So this is something that should be fixed by this:
>
> https://patchwork.freedesktop.org/patch/260547/
>
> And there's another patch that fixes a subsequent crash when you
> actually start to use the GPU:
>
> https://patchwork.freedesktop.org/patch/263588/
>
> It'd be great if you could apply both and verify that they fix the crash
> for you. If so, can you provide a Tested-by? Both were Cc'ed to
> linux-tegra, so you should have a copy to reply to. If not, let me know
> and I can bounce it.
>
> Ben, can you pick up the two patches above? They're kind of high-
> priority because they fix issues that crept into v4.20-rc1, so should
> ideally be fixed before v4.20 final.
Actually, it looks as if only the last patch is needed, since it
superseeds the first. The second one calls drm_mode_config_init() via
nouveau_display_create() and nouveau_drm_device_init(), making the
first patch obsolete.
There's more confirmation here:
https://lists.freedesktop.org/archives/nouveau/2018-December/031636.html
So Ben, correction, please only apply:
https://patchwork.freedesktop.org/patch/263587/
Preferably in time for v4.20 final.
Thanks,
Thierry
Attachment:
signature.asc
Description: PGP signature
