[PATCH] drm/amdgpu: Add Dell Inspiron 5575/5775 back to atpx quirk table

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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
>
>
>


[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux