• Rafael J. Wysocki's avatar
    thermal: core: Change PM notifier priority to the minimum · 494c7d05
    Rafael J. Wysocki authored
    It is reported that commit 5a5efdaf ("thermal: core: Resume thermal
    zones asynchronously") causes battery data in sysfs on Thinkpad P1 Gen2
    to become invalid after a resume from S3 (and it is necessary to reboot
    the machine to restore correct battery data).  Some investigation into
    the problem indicated that it happened because, after the commit in
    question, the ACPI battery PM notifier ran in parallel with
    thermal_zone_device_resume() for one of the thermal zones which
    apparently confused the platform firmware on the affected system.
    
    While the exact reason for the firmware confusion remains unclear, it
    is arguably not particularly relevant, and the expected behavior of the
    affected system can be restored by making the thermal PM notifier run
    at the lowest priority which avoids interference between work items
    spawned by it and the other PM notifiers (that will run before those
    work items now).
    
    Fixes: 5a5efdaf ("thermal: core: Resume thermal zones asynchronously")
    Closes: https://bugzilla.kernel.org/show_bug.cgi?id=218881
    Reported-by: fhortner@yahoo.de
    Tested-by: fhortner@yahoo.de
    Cc: 6.8+ <stable@vger.kernel.org> # 6.8+
    Signed-off-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
    494c7d05
thermal_core.c 45 KB