diff options
Diffstat (limited to 'common/recipes-kernel/linux/linux-yocto-4.19.8/0868-drm-amdgpu-Reorder-uvd-ring-init-before-uvd-resume.patch')
| -rw-r--r-- | common/recipes-kernel/linux/linux-yocto-4.19.8/0868-drm-amdgpu-Reorder-uvd-ring-init-before-uvd-resume.patch | 194 |
1 files changed, 194 insertions, 0 deletions
diff --git a/common/recipes-kernel/linux/linux-yocto-4.19.8/0868-drm-amdgpu-Reorder-uvd-ring-init-before-uvd-resume.patch b/common/recipes-kernel/linux/linux-yocto-4.19.8/0868-drm-amdgpu-Reorder-uvd-ring-init-before-uvd-resume.patch new file mode 100644 index 00000000..e5ad1753 --- /dev/null +++ b/common/recipes-kernel/linux/linux-yocto-4.19.8/0868-drm-amdgpu-Reorder-uvd-ring-init-before-uvd-resume.patch @@ -0,0 +1,194 @@ +From c39b4218dba21d47a4d9c7d95ea084ddb27ac014 Mon Sep 17 00:00:00 2001 +From: Chris Wilson <chris@chris-wilson.co.uk> +Date: Fri, 16 Nov 2018 09:34:24 +0000 +Subject: [PATCH 0868/2940] drm/amdgpu: Reorder uvd ring init before uvd resume +MIME-Version: 1.0 +Content-Type: text/plain; charset=UTF-8 +Content-Transfer-Encoding: 8bit + +As amd_uvd_resume() accesses the uvd ring, it must be initialised first +or else we trigger errors like: + +[ 5.595963] [drm] Found UVD firmware Version: 1.87 Family ID: 17 +[ 5.595969] [drm] PSP loading UVD firmware +[ 5.596266] ------------[ cut here ]------------ +[ 5.596268] ODEBUG: assert_init not available (active state 0) object type: timer_list hint: (null) +[ 5.596285] WARNING: CPU: 0 PID: 507 at lib/debugobjects.c:329 debug_print_object+0x6a/0x80 +[ 5.596286] Modules linked in: amdgpu(+) hid_logitech_hidpp(+) chash gpu_sched amd_iommu_v2 ttm drm_kms_helper crc32c_intel drm hid_sony ff_memless igb hid_logitech_dj nvme dca i2c_algo_bit nvme_core wmi pinctrl_amd uas usb_storage +[ 5.596299] CPU: 0 PID: 507 Comm: systemd-udevd Tainted: G W 4.20.0-0.rc1.git4.1.fc30.x86_64 #1 +[ 5.596301] Hardware name: System manufacturer System Product Name/ROG STRIX X470-I GAMING, BIOS 0901 07/23/2018 +[ 5.596303] RIP: 0010:debug_print_object+0x6a/0x80 +[ 5.596305] Code: 8b 43 10 83 c2 01 8b 4b 14 4c 89 e6 89 15 e6 82 b0 02 4c 8b 45 00 48 c7 c7 60 fd 34 a6 48 8b 14 c5 a0 da 08 a6 e8 6a 6a b8 ff <0f> 0b 5b 83 05 d0 45 3e 01 01 5d 41 5c c3 83 05 c5 45 3e 01 01 c3 +[ 5.596306] RSP: 0018:ffffa02ac863f8c0 EFLAGS: 00010282 +[ 5.596307] RAX: 0000000000000000 RBX: ffffa02ac863f8e0 RCX: 0000000000000006 +[ 5.596308] RDX: 0000000000000007 RSI: ffff9160e9a7bfe8 RDI: ffff9160f91d6c60 +[ 5.596310] RBP: ffffffffa6742740 R08: 0000000000000002 R09: 0000000000000000 +[ 5.596311] R10: 0000000000000000 R11: 0000000000000000 R12: ffffffffa634ff69 +[ 5.596312] R13: 00000000000b79d0 R14: ffffffffa80f76d8 R15: 0000000000266000 +[ 5.596313] FS: 00007f762abf7940(0000) GS:ffff9160f9000000(0000) knlGS:0000000000000000 +[ 5.596314] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033 +[ 5.596315] CR2: 000055fdc593f000 CR3: 00000007e999c000 CR4: 00000000003406f0 +[ 5.596317] Call Trace: +[ 5.596321] debug_object_assert_init+0x14a/0x180 +[ 5.596327] del_timer+0x2e/0x90 +[ 5.596383] amdgpu_fence_process+0x47/0x100 [amdgpu] +[ 5.596430] amdgpu_uvd_resume+0xf6/0x120 [amdgpu] +[ 5.596475] uvd_v7_0_sw_init+0xe0/0x280 [amdgpu] +[ 5.596523] amdgpu_device_init.cold.30+0xf97/0x14b6 [amdgpu] +[ 5.596563] ? amdgpu_driver_load_kms+0x53/0x330 [amdgpu] +[ 5.596604] amdgpu_driver_load_kms+0x86/0x330 [amdgpu] +[ 5.596614] drm_dev_register+0x115/0x150 [drm] +[ 5.596654] amdgpu_pci_probe+0xbd/0x120 [amdgpu] +[ 5.596658] local_pci_probe+0x41/0x90 +[ 5.596661] pci_device_probe+0x188/0x1a0 +[ 5.596666] really_probe+0xf8/0x3b0 +[ 5.596669] driver_probe_device+0xb3/0xf0 +[ 5.596672] __driver_attach+0xe1/0x110 +[ 5.596674] ? driver_probe_device+0xf0/0xf0 +[ 5.596676] bus_for_each_dev+0x79/0xc0 +[ 5.596679] bus_add_driver+0x155/0x230 +[ 5.596681] ? 0xffffffffc07d9000 +[ 5.596683] driver_register+0x6b/0xb0 +[ 5.596685] ? 0xffffffffc07d9000 +[ 5.596688] do_one_initcall+0x5d/0x2be +[ 5.596691] ? rcu_read_lock_sched_held+0x79/0x80 +[ 5.596693] ? kmem_cache_alloc_trace+0x264/0x290 +[ 5.596695] ? do_init_module+0x22/0x210 +[ 5.596698] do_init_module+0x5a/0x210 +[ 5.596701] load_module+0x2137/0x2430 +[ 5.596703] ? lockdep_hardirqs_on+0xed/0x180 +[ 5.596714] ? __do_sys_init_module+0x150/0x1a0 +[ 5.596715] __do_sys_init_module+0x150/0x1a0 +[ 5.596722] do_syscall_64+0x60/0x1f0 +[ 5.596725] entry_SYSCALL_64_after_hwframe+0x49/0xbe +[ 5.596726] RIP: 0033:0x7f762b877dee +[ 5.596728] Code: 48 8b 0d 9d 20 0c 00 f7 d8 64 89 01 48 83 c8 ff c3 66 2e 0f 1f 84 00 00 00 00 00 90 f3 0f 1e fa 49 89 ca b8 af 00 00 00 0f 05 <48> 3d 01 f0 ff ff 73 01 c3 48 8b 0d 6a 20 0c 00 f7 d8 64 89 01 48 +[ 5.596729] RSP: 002b:00007ffc777b8558 EFLAGS: 00000246 ORIG_RAX: 00000000000000af +[ 5.596730] RAX: ffffffffffffffda RBX: 000055fdc48da320 RCX: 00007f762b877dee +[ 5.596731] RDX: 00007f762b9f284d RSI: 00000000006c5fc6 RDI: 000055fdc527a060 +[ 5.596732] RBP: 00007f762b9f284d R08: 0000000000000003 R09: 0000000000000002 +[ 5.596733] R10: 000055fdc48ad010 R11: 0000000000000246 R12: 000055fdc527a060 +[ 5.596734] R13: 000055fdc48dca20 R14: 0000000000020000 R15: 0000000000000000 +[ 5.596740] irq event stamp: 134618 +[ 5.596743] hardirqs last enabled at (134617): [<ffffffffa513d52e>] console_unlock+0x45e/0x610 +[ 5.596744] hardirqs last disabled at (134618): [<ffffffffa50037e8>] trace_hardirqs_off_thunk+0x1a/0x1c +[ 5.596746] softirqs last enabled at (133146): [<ffffffffa5e00365>] __do_softirq+0x365/0x47c +[ 5.596748] softirqs last disabled at (133139): [<ffffffffa50c64f9>] irq_exit+0x119/0x120 +[ 5.596749] ---[ end trace eaee508abfebccdc ]--- + +Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=108709 +Reviewed-by: Christian König <christian.koenig@amd.com> +Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> +Cc: Alex Deucher <alexdeucher@gmail.com> +Signed-off-by: Alex Deucher <alexander.deucher@amd.com> +--- + drivers/gpu/drm/amd/amdgpu/uvd_v4_2.c | 8 ++++---- + drivers/gpu/drm/amd/amdgpu/uvd_v5_0.c | 8 ++++---- + drivers/gpu/drm/amd/amdgpu/uvd_v6_0.c | 8 ++++---- + drivers/gpu/drm/amd/amdgpu/uvd_v7_0.c | 8 ++++---- + 4 files changed, 16 insertions(+), 16 deletions(-) + +diff --git a/drivers/gpu/drm/amd/amdgpu/uvd_v4_2.c b/drivers/gpu/drm/amd/amdgpu/uvd_v4_2.c +index 90bbcee00f28..d69c8f6daaf8 100644 +--- a/drivers/gpu/drm/amd/amdgpu/uvd_v4_2.c ++++ b/drivers/gpu/drm/amd/amdgpu/uvd_v4_2.c +@@ -116,16 +116,16 @@ static int uvd_v4_2_sw_init(void *handle) + if (r) + return r; + +- r = amdgpu_uvd_resume(adev); +- if (r) +- return r; +- + ring = &adev->uvd.inst->ring; + sprintf(ring->name, "uvd"); + r = amdgpu_ring_init(adev, ring, 512, &adev->uvd.inst->irq, 0); + if (r) + return r; + ++ r = amdgpu_uvd_resume(adev); ++ if (r) ++ return r; ++ + r = amdgpu_uvd_entity_init(adev); + + return r; +diff --git a/drivers/gpu/drm/amd/amdgpu/uvd_v5_0.c b/drivers/gpu/drm/amd/amdgpu/uvd_v5_0.c +index 1c5e12703103..ee8cd06ddc38 100644 +--- a/drivers/gpu/drm/amd/amdgpu/uvd_v5_0.c ++++ b/drivers/gpu/drm/amd/amdgpu/uvd_v5_0.c +@@ -113,16 +113,16 @@ static int uvd_v5_0_sw_init(void *handle) + if (r) + return r; + +- r = amdgpu_uvd_resume(adev); +- if (r) +- return r; +- + ring = &adev->uvd.inst->ring; + sprintf(ring->name, "uvd"); + r = amdgpu_ring_init(adev, ring, 512, &adev->uvd.inst->irq, 0); + if (r) + return r; + ++ r = amdgpu_uvd_resume(adev); ++ if (r) ++ return r; ++ + r = amdgpu_uvd_entity_init(adev); + + return r; +diff --git a/drivers/gpu/drm/amd/amdgpu/uvd_v6_0.c b/drivers/gpu/drm/amd/amdgpu/uvd_v6_0.c +index f184842ef2a2..d4f4a66f8324 100644 +--- a/drivers/gpu/drm/amd/amdgpu/uvd_v6_0.c ++++ b/drivers/gpu/drm/amd/amdgpu/uvd_v6_0.c +@@ -400,16 +400,16 @@ static int uvd_v6_0_sw_init(void *handle) + DRM_INFO("UVD ENC is disabled\n"); + } + +- r = amdgpu_uvd_resume(adev); +- if (r) +- return r; +- + ring = &adev->uvd.inst->ring; + sprintf(ring->name, "uvd"); + r = amdgpu_ring_init(adev, ring, 512, &adev->uvd.inst->irq, 0); + if (r) + return r; + ++ r = amdgpu_uvd_resume(adev); ++ if (r) ++ return r; ++ + if (uvd_v6_0_enc_support(adev)) { + for (i = 0; i < adev->uvd.num_enc_rings; ++i) { + ring = &adev->uvd.inst->ring_enc[i]; +diff --git a/drivers/gpu/drm/amd/amdgpu/uvd_v7_0.c b/drivers/gpu/drm/amd/amdgpu/uvd_v7_0.c +index 3680a013743b..089645e78f98 100644 +--- a/drivers/gpu/drm/amd/amdgpu/uvd_v7_0.c ++++ b/drivers/gpu/drm/amd/amdgpu/uvd_v7_0.c +@@ -430,10 +430,6 @@ static int uvd_v7_0_sw_init(void *handle) + DRM_INFO("PSP loading UVD firmware\n"); + } + +- r = amdgpu_uvd_resume(adev); +- if (r) +- return r; +- + for (j = 0; j < adev->uvd.num_uvd_inst; j++) { + if (adev->uvd.harvest_config & (1 << j)) + continue; +@@ -465,6 +461,10 @@ static int uvd_v7_0_sw_init(void *handle) + } + } + ++ r = amdgpu_uvd_resume(adev); ++ if (r) ++ return r; ++ + r = amdgpu_uvd_entity_init(adev); + if (r) + return r; +-- +2.17.1 + |