diff options
| author | Rafael J. Wysocki <[email protected]> | 2025-02-21 16:57:11 +0000 |
|---|---|---|
| committer | Rafael J. Wysocki <[email protected]> | 2025-02-25 11:19:10 +0000 |
| commit | 423de5b5bc5b267586b449abd1c4fde562aa0cf9 (patch) | |
| tree | 6d924b996581777cc17f0438f32fb101d3162ba0 /net/unix/sysctl_net_unix.c | |
| parent | thermal: gov_power_allocator: Fix incorrect calculation in divvy_up_power() (diff) | |
| download | kernel-423de5b5bc5b267586b449abd1c4fde562aa0cf9.tar.gz kernel-423de5b5bc5b267586b449abd1c4fde562aa0cf9.zip | |
thermal/of: Fix cdev lookup in thermal_of_should_bind()
Since thermal_of_should_bind() terminates the loop after processing
the first child found in cooling-maps, it will never match more than
one cdev to a given trip point which is incorrect, as there may be
cooling-maps associating one trip point with multiple cooling devices.
Address this by letting the loop continue until either all
children have been processed or a matching one has been found.
To avoid adding conditionals or goto statements, put the loop in
question into a separate function and make that function return
right away after finding a matching cooling-maps entry.
Fixes: 94c6110b0b13 ("thermal/of: Use the .should_bind() thermal zone callback")
Link: https://lore.kernel.org/linux-pm/[email protected]/
Reported-by: Yu-Che Cheng <[email protected]>
Signed-off-by: Rafael J. Wysocki <[email protected]>
Reviewed-by: Yu-Che Cheng <[email protected]>
Tested-by: Yu-Che Cheng <[email protected]>
Reviewed-by: Lukasz Luba <[email protected]>
Tested-by: Lukasz Luba <[email protected]>
Link: https://patch.msgid.link/[email protected]
Diffstat (limited to 'net/unix/sysctl_net_unix.c')
0 files changed, 0 insertions, 0 deletions
