On Wed, Jun 13, 2018 at 2:27 PM, Kai-Heng Feng
<kai.heng.feng at canonical.com> wrote:
> at 01:41, Alex Deucher <alexdeucher at gmail.com> wrote:
>
>> On Tue, Jun 5, 2018 at 2:47 AM, Kai-Heng Feng
>> <kai.heng.feng at canonical.com> wrote:
>>>
>>> The original issue on these laptops was about _PR3, not audio controller
>>> prevents gfx auto suspending.
>>
>>
>> Have you verified that this this patch is still necessary with the HDA
>> driver fix in place?
>
>
> Yes I did. And the HDA fix doesn't work for these laptops.
>
> The HDA fix is to let HDA controller can be runtime suspended. OTOH, the
> ATPX quirk is to fix the "atombios stuck" for these laptops.
I'll just drop the original patch.
Thanks,
Alex
>
> Kai-Heng
>
>
>>
>> Alex
>>
>>> Commit 444d95f0eeef ("Partially revert: drm/amdgpu: add atpx quirk
>>> handling (v2)") breaks these laptops:
>>>
>>> [ 29.572055] [drm:atom_op_jump [amdgpu]] *ERROR* atombios stuck in loop
>>> for more than 5secs aborting
>>> [ 29.572738] [drm:amdgpu_atom_execute_table_locked [amdgpu]] *ERROR*
>>> atombios stuck executing 7C36 (len 272, WS 0, PS 4) @ 0x7C7F
>>> [ 29.573436] [drm:amdgpu_atom_execute_table_locked [amdgpu]] *ERROR*
>>> atombios stuck executing 6444 (len 70, WS 0, PS 8) @ 0x646A
>>> [ 29.574125] [drm:amdgpu_device_resume [amdgpu]] *ERROR* amdgpu asic
>>> init failed
>>> [ 29.991377] amdgpu 0000:01:00.0: Wait for MC idle timedout !
>>> [ 30.407480] amdgpu 0000:01:00.0: Wait for MC idle timedout !
>>> [ 30.417279] [drm] PCIE GART of 256M enabled (table at
>>> 0x000000F400000000).
>>> [ 30.426550] amdgpu: [powerplay] smu not running, upload firmware again
>>> [ 30.435710] BUG: unable to handle kernel paging request at
>>> ffffa52b90080fec
>>> [ 30.436982] IP: smu7_populate_single_firmware_entry.isra.5+0x65/0xe0
>>> [amdgpu]
>>> [ 30.438056] PGD 14e942067 P4D 14e942067 PUD 0
>>> [ 30.439280] Oops: 0002 [#1] SMP NOPTI
>>> [ 30.440339] Modules linked in: cmac bnep nls_iso8859_1 arc4 ath10k_pci
>>> ath10k_core rtsx_usb_ms memstick dell_wmi uvcvideo sparse_keymap
>>> videobuf2_vmalloc dell_laptop videobuf2_memops dell_smbios
>>> dell_wmi_descriptor videobuf2_v4l2 btusb btrtl wmi_bmof joydev dcdbas btbcm
>>> dell_smm_hwmon videobuf2_common kvm_amd btintel ath videodev
>>> snd_hda_codec_realtek snd_hda_codec_hdmi snd_hda_codec_generic cdc_acm
>>> mac80211 media bluetooth snd_hda_intel kvm snd_hda_codec snd_hwdep irqbypass
>>> snd_hda_core ecdh_generic crct10dif_pclmul snd_pcm crc32_pclmul snd_seq
>>> ghash_clmulni_intel pcbc snd_timer snd_seq_device aesni_intel cfg80211 snd
>>> aes_x86_64 soundcore input_leds crypto_simd cryptd tpm_crb hid_multitouch
>>> ucsi_acpi glue_helper typec_ucsi serio_raw typec video i2c_piix4 mac_hid
>>> dell_rbtn shpchp wmi parport_pc ppdev
>>> [ 30.445678] lp parport autofs4 btrfs xor zstd_decompress
>>> zstd_compress xxhash raid6_pq dm_mirror dm_region_hash dm_log amdkfd
>>> amd_iommu_v2 amdgpu rtsx_usb_sdmmc rtsx_usb chash i2c_algo_bit gpu_sched
>>> drm_kms_helper syscopyarea sysfillrect sysimgblt fb_sys_fops ttm drm r8169
>>> ahci libahci i2c_hid mii hid
>>> [ 30.448702] CPU: 7 PID: 1021 Comm: gpu-manager Not tainted 4.16.0-rc7+
>>> #1
>>> [ 30.450256] Hardware name: Dell Inc. Inspiron 5775/Inspiron 5775, BIOS
>>> 1.1.0 03/26/2018
>>> [ 30.451959] RIP:
>>> 0010:smu7_populate_single_firmware_entry.isra.5+0x65/0xe0 [amdgpu]
>>> [ 30.453492] RSP: 0018:ffffa50f816f3a58 EFLAGS: 00010246
>>> [ 30.455116] RAX: 000000000000008c RBX: ffffa52b90080fec RCX:
>>> 0000000000000000
>>> [ 30.456676] RDX: 0000000000000004 RSI: 0000000000000004 RDI:
>>> ffff917dfb3a5a90
>>> [ 30.458203] RBP: ffffa50f816f3aa8 R08: ffff917dfb3a5a90 R09:
>>> 0000000000033930
>>> [ 30.459727] R10: 0000000000000000 R11: 0000000000000412 R12:
>>> 0000000000000003
>>> [ 30.461246] R13: ffff917dfb194c14 R14: ffff917dfac65000 R15:
>>> 00000000000005fe
>>> [ 30.462733] FS: 00007f5248978700(0000) GS:ffff917e0edc0000(0000)
>>> knlGS:0000000000000000
>>> [ 30.464302] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033
>>> [ 30.465830] CR2: ffffa52b90080fec CR3: 00000001358c2000 CR4:
>>> 00000000003406e0
>>> [ 30.467468] Call Trace:
>>> [ 30.469068] smu7_request_smu_load_fw+0xa9/0x360 [amdgpu]
>>> [ 30.470630] ? vga_switcheroo_fini_domain_pm_ops+0x20/0x20
>>> [ 30.472416] iceland_start_smu+0x39/0x70 [amdgpu]
>>> [ 30.473492] hwmgr_resume+0x2b/0xa0 [amdgpu]
>>> [ 30.474500] pp_resume+0x15/0x20 [amdgpu]
>>> [ 30.475472] amdgpu_device_ip_resume_phase2+0x58/0xb0 [amdgpu]
>>> [ 30.476431] amdgpu_device_resume+0xd8/0x370 [amdgpu]
>>> [ 30.477379] ? __pci_set_master+0x34/0xe0
>>> [ 30.478345] ? vga_switcheroo_fini_domain_pm_ops+0x20/0x20
>>> [ 30.479317] amdgpu_pmops_runtime_resume+0x76/0xa0 [amdgpu]
>>> [ 30.480265] pci_pm_runtime_resume+0x76/0xb0
>>> [ 30.481216] vga_switcheroo_runtime_resume+0x59/0x60
>>> [ 30.482201] __rpm_callback+0xc4/0x200
>>> [ 30.483179] ? vga_switcheroo_fini_domain_pm_ops+0x20/0x20
>>> [ 30.484075] rpm_callback+0x24/0x80
>>> [ 30.485025] ? vga_switcheroo_fini_domain_pm_ops+0x20/0x20
>>> [ 30.486005] rpm_resume+0x499/0x6a0
>>> [ 30.486946] __pm_runtime_resume+0x4e/0x80
>>> [ 30.487880] pci_config_pm_runtime_get+0x53/0x60
>>> [ 30.488789] pci_read_config+0x8f/0x280
>>> [ 30.489771] sysfs_kf_bin_read+0x4a/0x70
>>> [ 30.490750] kernfs_fop_read+0xa9/0x190
>>> [ 30.491648] __vfs_read+0x37/0x160
>>> [ 30.492579] ? security_file_permission+0x9b/0xc0
>>> [ 30.493530] vfs_read+0x93/0x130
>>> [ 30.494445] SyS_pread64+0x95/0xb0
>>> [ 30.495347] do_syscall_64+0x6d/0x120
>>> [ 30.496223] entry_SYSCALL_64_after_hwframe+0x3d/0xa2
>>> [ 30.497124] RIP: 0033:0x7f524807ea43
>>> [ 30.498032] RSP: 002b:00007ffcbd7ca258 EFLAGS: 00000246 ORIG_RAX:
>>> 0000000000000011
>>> [ 30.498972] RAX: ffffffffffffffda RBX: 0000000000000030 RCX:
>>> 00007f524807ea43
>>> [ 30.499907] RDX: 0000000000000030 RSI: 00007ffcbd7ca490 RDI:
>>> 0000000000000005
>>> [ 30.500796] RBP: 00007ffcbd7ca490 R08: 0000000000000000 R09:
>>> 0000000000000028
>>> [ 30.501776] R10: 0000000000000000 R11: 0000000000000246 R12:
>>> 0000000000000005
>>> [ 30.502758] R13: 0000000000000030 R14: 00007ffcbd7ca3f8 R15:
>>> 0000000000000000
>>> [ 30.503659] Code: 83 fc 23 f3 48 ab 48 8b 06 be 0d 00 00 00 48 8b 40
>>> 20 77 0a 44 89 e1 0f b6 b1 a0 46 68 c0 4c 89 c7 ff d0 85 c0 75 3e 0f b7 45
>>> b2 <66> 44 89 23 c7 43 0c 00 00 00 00 c7 43 10 00 00 00 00 66 89 43
>>> [ 30.504668] RIP: smu7_populate_single_firmware_entry.isra.5+0x65/0xe0
>>> [amdgpu] RSP: ffffa50f816f3a58
>>> [ 30.505643] CR2: ffffa52b90080fec
>>> [ 30.506618] ---[ end trace b443cb7ec0f49d4f ]---
>>>
>>> So add these IDs back to atpx quirk table.
>>>
>>> Fixes: 444d95f0eeef ("Partially revert: drm/amdgpu: add atpx quirk
>>> handling (v2)")
>>> Fixes: c6f5b3155fbc ("Revert "drm/amdgpu: add new device to use atpx
>>> quirk"")
>>> Signed-off-by: Kai-Heng Feng <kai.heng.feng at canonical.com>
>>> ---
>>> drivers/gpu/drm/amd/amdgpu/amdgpu_atpx_handler.c | 4 ++++
>>> 1 file changed, 4 insertions(+)
>>>
>>> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_atpx_handler.c
>>> b/drivers/gpu/drm/amd/amdgpu/amdgpu_atpx_handler.c
>>> index 9c493e8a48a5..1b8cb076c378 100644
>>> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_atpx_handler.c
>>> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_atpx_handler.c
>>> @@ -565,6 +565,10 @@ static const struct vga_switcheroo_handler
>>> amdgpu_atpx_handler = {
>>> };
>>>
>>> static const struct amdgpu_px_quirk amdgpu_px_quirk_list[] = {
>>> + /* Dell Inspiron 5575 */
>>> + { 0x1002, 0x6900, 0x1028, 0x0812, AMDGPU_PX_QUIRK_FORCE_ATPX },
>>> + /* Dell Inspiron 5775 */
>>> + { 0x1002, 0x6900, 0x1028, 0x0813, AMDGPU_PX_QUIRK_FORCE_ATPX },
>>> { 0, 0, 0, 0, 0 },
>>> };
>>>
>>> --
>>> 2.17.0
>>>
>>> _______________________________________________
>>> amd-gfx mailing list
>>> amd-gfx at lists.freedesktop.org
>>> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
>
>
>
