The series provides more changes to self-encapsulate the thermal zone device
structure in order to protect wild accesses to the thermal zone device
internals, especially the trip points and the locks.
The first patch adds a macro to warn at compilation time if the
thermal_core.h is included in a file not belonging to the thermal core
code. One warning will happen with the nvidia drivers but this is in
the way to be solved.
The second patch reorders the headers inclusion in the core code.
The next patches makes the int340x drivers to use the thermal trip
update above and the different accessors for thermal zone structure.
Daniel Lezcano (4):
thermal/core: Hardening the self-encapsulation
thermal/core: Reorder the headers inclusion
thermal/drivers/int3400: Use thermal zone device wrappers
thermal/drivers/int340x: Do not check the thermal zone state
drivers/thermal/gov_bang_bang.c | 1 +
drivers/thermal/gov_fair_share.c | 1 +
drivers/thermal/gov_power_allocator.c | 7 +--
drivers/thermal/gov_step_wise.c | 1 +
drivers/thermal/gov_user_space.c | 1 +
.../intel/int340x_thermal/int3400_thermal.c | 44 +++++++++----------
drivers/thermal/thermal_acpi.c | 1 +
drivers/thermal/thermal_core.c | 7 +--
drivers/thermal/thermal_core.h | 4 ++
drivers/thermal/thermal_helpers.c | 1 +
drivers/thermal/thermal_hwmon.c | 1 +
drivers/thermal/thermal_netlink.c | 1 +
drivers/thermal/thermal_of.c | 1 +
drivers/thermal/thermal_sysfs.c | 1 +
drivers/thermal/thermal_trip.c | 1 +
15 files changed, 45 insertions(+), 28 deletions(-)
--
2.34.1
On 07/07/2023 22:37, Daniel Lezcano wrote:
> The series provides more changes to self-encapsulate the thermal zone device
> structure in order to protect wild accesses to the thermal zone device
> internals, especially the trip points and the locks.
>
> The first patch adds a macro to warn at compilation time if the
> thermal_core.h is included in a file not belonging to the thermal core
> code. One warning will happen with the nvidia drivers but this is in
> the way to be solved.
>
> The second patch reorders the headers inclusion in the core code.
>
> The next patches makes the int340x drivers to use the thermal trip
> update above and the different accessors for thermal zone structure.
>
> Daniel Lezcano (4):
> thermal/core: Hardening the self-encapsulation
> thermal/core: Reorder the headers inclusion
> thermal/drivers/int3400: Use thermal zone device wrappers
> thermal/drivers/int340x: Do not check the thermal zone state
>
> drivers/thermal/gov_bang_bang.c | 1 +
> drivers/thermal/gov_fair_share.c | 1 +
> drivers/thermal/gov_power_allocator.c | 7 +--
> drivers/thermal/gov_step_wise.c | 1 +
> drivers/thermal/gov_user_space.c | 1 +
> .../intel/int340x_thermal/int3400_thermal.c | 44 +++++++++----------
> drivers/thermal/thermal_acpi.c | 1 +
> drivers/thermal/thermal_core.c | 7 +--
> drivers/thermal/thermal_core.h | 4 ++
> drivers/thermal/thermal_helpers.c | 1 +
> drivers/thermal/thermal_hwmon.c | 1 +
> drivers/thermal/thermal_netlink.c | 1 +
> drivers/thermal/thermal_of.c | 1 +
> drivers/thermal/thermal_sysfs.c | 1 +
> drivers/thermal/thermal_trip.c | 1 +
> 15 files changed, 45 insertions(+), 28 deletions(-)
Applied
--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs
Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog
On Thu, Jul 13, 2023 at 11:33 AM Daniel Lezcano
<[email protected]> wrote:
>
> On 07/07/2023 22:37, Daniel Lezcano wrote:
> > The series provides more changes to self-encapsulate the thermal zone device
> > structure in order to protect wild accesses to the thermal zone device
> > internals, especially the trip points and the locks.
> >
> > The first patch adds a macro to warn at compilation time if the
> > thermal_core.h is included in a file not belonging to the thermal core
> > code. One warning will happen with the nvidia drivers but this is in
> > the way to be solved.
> >
> > The second patch reorders the headers inclusion in the core code.
> >
> > The next patches makes the int340x drivers to use the thermal trip
> > update above and the different accessors for thermal zone structure.
> >
> > Daniel Lezcano (4):
> > thermal/core: Hardening the self-encapsulation
> > thermal/core: Reorder the headers inclusion
> > thermal/drivers/int3400: Use thermal zone device wrappers
> > thermal/drivers/int340x: Do not check the thermal zone state
> >
> > drivers/thermal/gov_bang_bang.c | 1 +
> > drivers/thermal/gov_fair_share.c | 1 +
> > drivers/thermal/gov_power_allocator.c | 7 +--
> > drivers/thermal/gov_step_wise.c | 1 +
> > drivers/thermal/gov_user_space.c | 1 +
> > .../intel/int340x_thermal/int3400_thermal.c | 44 +++++++++----------
> > drivers/thermal/thermal_acpi.c | 1 +
> > drivers/thermal/thermal_core.c | 7 +--
> > drivers/thermal/thermal_core.h | 4 ++
> > drivers/thermal/thermal_helpers.c | 1 +
> > drivers/thermal/thermal_hwmon.c | 1 +
> > drivers/thermal/thermal_netlink.c | 1 +
> > drivers/thermal/thermal_of.c | 1 +
> > drivers/thermal/thermal_sysfs.c | 1 +
> > drivers/thermal/thermal_trip.c | 1 +
> > 15 files changed, 45 insertions(+), 28 deletions(-)
>
> Applied
OK
On 13/07/2023 13:30, Rafael J. Wysocki wrote:
> On Thu, Jul 13, 2023 at 11:33 AM Daniel Lezcano
> <[email protected]> wrote:
>>
>> On 07/07/2023 22:37, Daniel Lezcano wrote:
>>> The series provides more changes to self-encapsulate the thermal zone device
>>> structure in order to protect wild accesses to the thermal zone device
>>> internals, especially the trip points and the locks.
>>>
>>> The first patch adds a macro to warn at compilation time if the
>>> thermal_core.h is included in a file not belonging to the thermal core
>>> code. One warning will happen with the nvidia drivers but this is in
>>> the way to be solved.
>>>
>>> The second patch reorders the headers inclusion in the core code.
>>>
>>> The next patches makes the int340x drivers to use the thermal trip
>>> update above and the different accessors for thermal zone structure.
>>>
>>> Daniel Lezcano (4):
>>> thermal/core: Hardening the self-encapsulation
>>> thermal/core: Reorder the headers inclusion
>>> thermal/drivers/int3400: Use thermal zone device wrappers
>>> thermal/drivers/int340x: Do not check the thermal zone state
>>>
>>> drivers/thermal/gov_bang_bang.c | 1 +
>>> drivers/thermal/gov_fair_share.c | 1 +
>>> drivers/thermal/gov_power_allocator.c | 7 +--
>>> drivers/thermal/gov_step_wise.c | 1 +
>>> drivers/thermal/gov_user_space.c | 1 +
>>> .../intel/int340x_thermal/int3400_thermal.c | 44 +++++++++----------
>>> drivers/thermal/thermal_acpi.c | 1 +
>>> drivers/thermal/thermal_core.c | 7 +--
>>> drivers/thermal/thermal_core.h | 4 ++
>>> drivers/thermal/thermal_helpers.c | 1 +
>>> drivers/thermal/thermal_hwmon.c | 1 +
>>> drivers/thermal/thermal_netlink.c | 1 +
>>> drivers/thermal/thermal_of.c | 1 +
>>> drivers/thermal/thermal_sysfs.c | 1 +
>>> drivers/thermal/thermal_trip.c | 1 +
>>> 15 files changed, 45 insertions(+), 28 deletions(-)
>>
>> Applied
Sorry, I did not think you may wanted pull them in.
I can drop them from the tree if you prefer?
--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs
Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog
On Thu, Jul 13, 2023 at 2:24 PM Daniel Lezcano
<[email protected]> wrote:
>
> On 13/07/2023 13:30, Rafael J. Wysocki wrote:
> > On Thu, Jul 13, 2023 at 11:33 AM Daniel Lezcano
> > <[email protected]> wrote:
> >>
> >> On 07/07/2023 22:37, Daniel Lezcano wrote:
> >>> The series provides more changes to self-encapsulate the thermal zone device
> >>> structure in order to protect wild accesses to the thermal zone device
> >>> internals, especially the trip points and the locks.
> >>>
> >>> The first patch adds a macro to warn at compilation time if the
> >>> thermal_core.h is included in a file not belonging to the thermal core
> >>> code. One warning will happen with the nvidia drivers but this is in
> >>> the way to be solved.
> >>>
> >>> The second patch reorders the headers inclusion in the core code.
> >>>
> >>> The next patches makes the int340x drivers to use the thermal trip
> >>> update above and the different accessors for thermal zone structure.
> >>>
> >>> Daniel Lezcano (4):
> >>> thermal/core: Hardening the self-encapsulation
> >>> thermal/core: Reorder the headers inclusion
> >>> thermal/drivers/int3400: Use thermal zone device wrappers
> >>> thermal/drivers/int340x: Do not check the thermal zone state
> >>>
> >>> drivers/thermal/gov_bang_bang.c | 1 +
> >>> drivers/thermal/gov_fair_share.c | 1 +
> >>> drivers/thermal/gov_power_allocator.c | 7 +--
> >>> drivers/thermal/gov_step_wise.c | 1 +
> >>> drivers/thermal/gov_user_space.c | 1 +
> >>> .../intel/int340x_thermal/int3400_thermal.c | 44 +++++++++----------
> >>> drivers/thermal/thermal_acpi.c | 1 +
> >>> drivers/thermal/thermal_core.c | 7 +--
> >>> drivers/thermal/thermal_core.h | 4 ++
> >>> drivers/thermal/thermal_helpers.c | 1 +
> >>> drivers/thermal/thermal_hwmon.c | 1 +
> >>> drivers/thermal/thermal_netlink.c | 1 +
> >>> drivers/thermal/thermal_of.c | 1 +
> >>> drivers/thermal/thermal_sysfs.c | 1 +
> >>> drivers/thermal/thermal_trip.c | 1 +
> >>> 15 files changed, 45 insertions(+), 28 deletions(-)
> >>
> >> Applied
>
> Sorry, I did not think you may wanted pull them in.
>
> I can drop them from the tree if you prefer?
Let me apply them directly.
On 13/07/2023 15:02, Rafael J. Wysocki wrote:
> On Thu, Jul 13, 2023 at 2:24 PM Daniel Lezcano
> <[email protected]> wrote:
>>
>> On 13/07/2023 13:30, Rafael J. Wysocki wrote:
>>> On Thu, Jul 13, 2023 at 11:33 AM Daniel Lezcano
>>> <[email protected]> wrote:
>>>>
>>>> On 07/07/2023 22:37, Daniel Lezcano wrote:
>>>>> The series provides more changes to self-encapsulate the thermal zone device
>>>>> structure in order to protect wild accesses to the thermal zone device
>>>>> internals, especially the trip points and the locks.
>>>>>
>>>>> The first patch adds a macro to warn at compilation time if the
>>>>> thermal_core.h is included in a file not belonging to the thermal core
>>>>> code. One warning will happen with the nvidia drivers but this is in
>>>>> the way to be solved.
>>>>>
>>>>> The second patch reorders the headers inclusion in the core code.
>>>>>
>>>>> The next patches makes the int340x drivers to use the thermal trip
>>>>> update above and the different accessors for thermal zone structure.
>>>>>
>>>>> Daniel Lezcano (4):
>>>>> thermal/core: Hardening the self-encapsulation
>>>>> thermal/core: Reorder the headers inclusion
>>>>> thermal/drivers/int3400: Use thermal zone device wrappers
>>>>> thermal/drivers/int340x: Do not check the thermal zone state
>>>>>
>>>>> drivers/thermal/gov_bang_bang.c | 1 +
>>>>> drivers/thermal/gov_fair_share.c | 1 +
>>>>> drivers/thermal/gov_power_allocator.c | 7 +--
>>>>> drivers/thermal/gov_step_wise.c | 1 +
>>>>> drivers/thermal/gov_user_space.c | 1 +
>>>>> .../intel/int340x_thermal/int3400_thermal.c | 44 +++++++++----------
>>>>> drivers/thermal/thermal_acpi.c | 1 +
>>>>> drivers/thermal/thermal_core.c | 7 +--
>>>>> drivers/thermal/thermal_core.h | 4 ++
>>>>> drivers/thermal/thermal_helpers.c | 1 +
>>>>> drivers/thermal/thermal_hwmon.c | 1 +
>>>>> drivers/thermal/thermal_netlink.c | 1 +
>>>>> drivers/thermal/thermal_of.c | 1 +
>>>>> drivers/thermal/thermal_sysfs.c | 1 +
>>>>> drivers/thermal/thermal_trip.c | 1 +
>>>>> 15 files changed, 45 insertions(+), 28 deletions(-)
>>>>
>>>> Applied
>>
>> Sorry, I did not think you may wanted pull them in.
>>
>> I can drop them from the tree if you prefer?
>
> Let me apply them directly.
Ok
--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs
Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog