amd_sfh modprobe fails when no sensor reported from AMD MP2

Bug #1993240 reported by You-Sheng Yang
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
HWE Next
New
Undecided
Unassigned
linux (Ubuntu)
Fix Committed
Undecided
Unassigned
Jammy
Invalid
Undecided
Unassigned
Kinetic
Invalid
Undecided
Unassigned
linux-oem-6.0 (Ubuntu)
Invalid
Undecided
Unassigned
Jammy
Fix Released
High
You-Sheng Yang
Kinetic
Invalid
Undecided
Unassigned

Bug Description

[SRU Justification]

[Impact]

Some AMD platforms might not have sensor reported from MP2 controller,
and that fails device init.

[Fix]

Two commits targeting v6.1 release in mainline kernel.

[Test Case]

On such platforms, driver loading should be bailed out with:

  $ sudo dmesg | grep amd_sfh1_1_hid_client_init
  [ 1.204896] pcie_mp2_amd 0000:62:00.7: amd_sfh1_1_hid_client_init failed

[Where problems could occur]

This skips driver loading against ineffective hardware only.

[Other Info]

While this targets oem-6.0 and above, and is already in mainline
v6.1-rc1, only oem-6.0 is nominated for fix.

========== original bug report ==========

Fix amd_sfh to make each sensor with proper initialization & resource allocation during MP2 loaded.

To be in v6.1

https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/drivers/hid/amd-sfh-hid?h=next-20221010&id=68266bdcceec10ea364e62c63732cd6fe5a256a8

https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/drivers/hid/amd-sfh-hid?h=next-20221010&id=beb18bb22cd4fb88648bb2925d56f36131c1ac21

You-Sheng Yang (vicamo)
Changed in linux-oem-6.0 (Ubuntu Kinetic):
status: New → Invalid
Changed in linux-oem-6.0 (Ubuntu Jammy):
status: New → Confirmed
tags: added: amd oem-priority originate-from-1992402
You-Sheng Yang (vicamo)
Changed in linux-oem-6.0 (Ubuntu Jammy):
status: Confirmed → In Progress
importance: Undecided → High
assignee: nobody → You-Sheng Yang (vicamo)
Revision history for this message
You-Sheng Yang (vicamo) wrote :
description: updated
Timo Aaltonen (tjaalton)
Changed in linux-oem-6.0 (Ubuntu Jammy):
status: In Progress → Fix Committed
Changed in linux (Ubuntu Jammy):
status: New → Invalid
Changed in linux (Ubuntu Kinetic):
status: New → Invalid
Changed in linux (Ubuntu):
status: New → Fix Committed
Timo Aaltonen (tjaalton)
tags: added: verification-needed-jammy
Revision history for this message
You-Sheng Yang (vicamo) wrote :

verified linux-oem-6.0/jammy version 6.0.0-1008.8.

tags: added: verification-done-jammy
removed: verification-needed-jammy
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (48.0 KiB)

This bug was fixed in the package linux-oem-6.0 - 6.0.0-1008.8

---------------
linux-oem-6.0 (6.0.0-1008.8) jammy; urgency=medium

  * jammy/linux-oem-6.0: 6.0.0-1008.8 -proposed tracker (LP: #1996347)

  * Packaging resync (LP: #1786013)
    - debian/dkms-versions -- update from kernel-versions (main/2022.11.14)

  * Fix a race condition with AMD PMF and Thinkpad-ACPI (LP: #1993242)
    - platform/x86/amd/pmf: Fix undefined reference to platform_profile
    - platform/x86/amd/pmf: Fix clang unused variable warning
    - platform/x86/amd/pmf: install notify handler after acpi init

  * Screen freeze after resuming from suspend (nvme0: I/O timeout)
    (LP: #1996048)
    - SAUCE: PCI: vmd: Disable MSI remapping after suspend

  * Fibocom WWAN FM350-GL suspend error (notebook not suspend) (LP: #1990700)
    - net: wwan: t7xx: Add AP CLDMA

  * Add some ACPI device IDs for Intel HID device (LP: #1995453)
    - platform/x86/intel/hid: Add some ACPI device IDs

  * amd_sfh modprobe fails when no sensor reported from AMD MP2 (LP: #1993240)
    - HID: amd_sfh: Change dev_err to dev_dbg for additional debug info

  * Jammy update: v6.0.9 upstream stable release (LP: #1996785)
    - drm/i915/gvt: Add missing vfio_unregister_group_dev() call
    - m68k: Rework BI_VIRT_RNG_SEED as BI_RNG_SEED
    - KVM: debugfs: Return retval of simple_attr_open() if it fails
    - drm/i915: Allow more varied alternate fixed modes for panels
    - drm/i915: Simplify intel_panel_add_edid_alt_fixed_modes()
    - drm/i915/sdvo: Grab mode_config.mutex during LVDS init to avoid WARNs
    - drm/amd/display: Acquire FCLK DPM levels on DCN32
    - drm/amd/display: Limit dcn32 to 1950Mhz display clock
    - drm/amd/display: Set memclk levels to be at least 1 for dcn32
    - drm/amdkfd: handle CPU fault on COW mapping
    - drm/amdkfd: Fix NULL pointer dereference in svm_migrate_to_ram()
    - cxl/region: Recycle region ids
    - HID: wacom: Fix logic used for 3rd barrel switch emulation
    - hwspinlock: qcom: correct MMIO max register for newer SoCs
    - phy: stm32: fix an error code in probe
    - wifi: cfg80211: silence a sparse RCU warning
    - wifi: cfg80211: fix memory leak in query_regdb_file()
    - soundwire: qcom: reinit broadcast completion
    - soundwire: qcom: check for outanding writes before doing a read
    - ALSA: arm: pxa: pxa2xx-ac97-lib: fix return value check of
      platform_get_irq()
    - spi: mediatek: Fix package division error
    - bpf, verifier: Fix memory leak in array reallocation for stack state
    - bpf, sockmap: Fix the sk->sk_forward_alloc warning of sk_stream_kill_queues
    - wifi: mac80211: fix general-protection-fault in ieee80211_subif_start_xmit()
    - wifi: mac80211: Set TWT Information Frame Disabled bit as 1
    - bpftool: Fix NULL pointer dereference when pin {PROG, MAP, LINK} without
      FILE
    - HID: hyperv: fix possible memory leak in mousevsc_probe()
    - drm/vc4: hdmi: Fix HSM clock too low on Pi4
    - bpf, sock_map: Move cancel_work_sync() out of sock lock
    - PCI: hv: Fix the definition of vector in hv_compose_msi_msg()
    - bpf: Add helper macro bpf_for_each_reg_in_vstate
    - bpf: Fix wrong reg type conversi...

Changed in linux-oem-6.0 (Ubuntu Jammy):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.