On Mon, Dec 10, 2018 at 03:20:19PM +0000, Marcel Ziswiler wrote:
> Hi Thierry
>
> On Mon, 2018-12-10 at 12:00 +0100, Thierry Reding wrote:
> > 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/v
> > > > ic03_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/
>
> Yes, that fixes it and I sent my tested-by. Thanks!
Excellent, thanks!
>
> > Preferably in time for v4.20 final.
>
> BTW: During testing I was also brave enough to try rmmodding nouveau
> which unfortunately also seems to fail:
>
> root@apalis-tk1-mainline:~# rmmod nouveau
> [ 3044.432527] [TTM] Finalizing pool allocator
> [ 3044.440007] [TTM] Zone kernel: Used memory at exit: 0 kiB
> [ 3044.445631] [TTM] Zone highmem: Used memory at exit: 0 kiB
> [ 3044.452841] Unable to handle kernel NULL pointer dereference at
> virtual address 0000038a
> [ 3044.461167] pgd = 537c0ac4
> [ 3044.463891] [0000038a] *pgd=fb95b835
> [ 3044.467487] Internal error: Oops: 17 [#1] PREEMPT SMP ARM
> [ 3044.472901] Modules linked in: nouveau(-) btusb btrtl btbcm btintel
> tegra_drm xhci_tegra host1x iova ttm
> [ 3044.482415] CPU: 3 PID: 616 Comm: rmmod Not tainted 4.20.0-rc6-next-
> 20181210-00001-gd70a977fd0d5-dirty #115
> [ 3044.492176] Hardware name: NVIDIA Tegra SoC (Flattened Device Tree)
> [ 3044.498455] PC is at pci_disable_device+0x8/0xd4
> [ 3044.503165] LR is at nouveau_drm_device_remove+0x50/0x7c [nouveau]
> [ 3044.509350] pc : [<c048d05c>] lr : [<bf254820>] psr: 60000113
> [ 3044.515638] sp : ee3abedc ip : ed625000 fp : 00000001
> [ 3044.520879] r10: 00000081 r9 : ee3aa000 r8 : ee9eb834
> [ 3044.526107] r7 : ed624000 r6 : 00000000 r5 : c0f08c48 r4 :
> 00000000
> [ 3044.532649] r3 : 5dfe844b r2 : 5dfe844b r1 : 2e135000 r0 :
> 00000000
> [ 3044.539181] Flags: nZCv IRQs on FIQs on Mode SVC_32 ISA
> ARM Segment none
> [ 3044.546333] Control: 10c5387d Table: acbc006a DAC: 00000051
> [ 3044.552098] Process rmmod (pid: 616, stack limit = 0xfc4e79a2)
> [ 3044.557934] Stack: (0xee3abedc to 0xee3ac000)
> [ 3044.562304]
> bec0: ed
> 62d400
> [ 3044.570503] bee0: c0f08c48 bf254820 eda76808 5dfe844b ee9f1c8c
> ee9f1c10 ee9f1c10 bf2f9c5c
> [ 3044.578701] bf00: ee9eb800 bf255914 ee9f1c10 c056aba8 ee9f1c10
> ee9f1c44 bf2f9c5c c05693bc
> [ 3044.587298] bf20: ee9f1c10 bf2f9c5c 0001f10c 00000800 c0101204
> c05694cc bf2f9c5c bf2fb980
> [ 3044.596316] bf40: 0001f10c c056829c c0f08c48 c01b3c34 76756f6e
> 00756165 00000000 00000000
> [ 3044.605356] bf60: c0f08c48 ec415000 00000002 5dfe844b 00000001
> c0141c10 ec415000 ec415000
> [ 3044.614431] bf80: ed67c100 5dfe844b 00000000 5dfe844b 00000000
> 00000002 bebb5ba8 00000000
> [ 3044.623529] bfa0: 00000081 c0101000 00000002 bebb5ba8 0001f10c
> 00000800 0000000a 00000000
> [ 3044.632631] bfc0: 00000002 bebb5ba8 00000000 00000081 bebb5e9b
> 0001f0d0 bebb5d8c 00000001
> [ 3044.641739] bfe0: b6e74730 bebb5b64 00012bdf b6e7473c 600d0010
> 0001f10c 00000000 00000000
> [ 3044.650884] [<c048d05c>] (pci_disable_device) from [<ee9f1c8c>]
> (0xee9f1c8c)
> [ 3044.658410] Code: eafffff0 ebf25973 e92d4030 e1a04000 (e5d0338a)
> [ 3044.665141] ---[ end trace 810af3dad648a902 ]---
> Segmentation fault
>
> Looks like with pci_disable_device() it may take a rather strange
> path...
Yikes... it has no business at all calling pci_disable_device() on
Tegra. Unless if you happen to have a GPU plugged into the PCIe slot.
I'm assuming that's not what you're doing?
I'll see if I can reproduce (and fix) that crash on unload. Admittedly
it's not something that I regularly test. Perhaps that's something that
I should change...
Thierry
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
