mbox series

[SRU,J:linux-bluefield,v1,0/1] UBUNTU: SAUCE: ptm debugfs: add ddr thermal event nodes

Message ID cover.1691079366.git.jlanka@nvidia.com
Headers show
Series UBUNTU: SAUCE: ptm debugfs: add ddr thermal event nodes | expand

Message

Jitendra Lanka Aug. 3, 2023, 4:26 p.m. UTC 
BugLink: https://bugs.launchpad.net/bugs/2029505

SRU Justification:

[Impact]

mlxbf-ptm is a kernel driver that provides debufgs interface for system
software to monitor Bluefield devices' power and thermal management
parameters.

[Fix]

* This change adds additional debugfs nodes that provides event counter
  and throttling states of ddr thermal management state machine.

[Test Case]

* After installing the kernel module, debugfs entry at
  /sys/kernel/debug/mlxbf-ptm will be created
* monitors/status folder contains read-only parameters reported from the
  Bluefield device

[Regression Potential]

* Minimal - as these are read-only parameters that report Bluefield device
  information.

[Other]

* This code is likely to change depending on feedback we received from
  maintainers.

Jitendra Lanka (1):
  ptm debugfs: add ddr thermal event nodes

 drivers/platform/mellanox/mlxbf-ptm.c | 23 +++++++++++++++++++++++
 1 file changed, 23 insertions(+)

Comments

Tim Gardner Aug. 3, 2023, 6:45 p.m. UTC | #1 
On 8/3/23 10:26 AM, Jitendra Lanka wrote:
> BugLink: https://bugs.launchpad.net/bugs/2029505
> 
> SRU Justification:
> 
> [Impact]
> 
> mlxbf-ptm is a kernel driver that provides debufgs interface for system
> software to monitor Bluefield devices' power and thermal management
> parameters.
> 
> [Fix]
> 
> * This change adds additional debugfs nodes that provides event counter
>    and throttling states of ddr thermal management state machine.
> 
> [Test Case]
> 
> * After installing the kernel module, debugfs entry at
>    /sys/kernel/debug/mlxbf-ptm will be created
> * monitors/status folder contains read-only parameters reported from the
>    Bluefield device
> 
> [Regression Potential]
> 
> * Minimal - as these are read-only parameters that report Bluefield device
>    information.
> 
> [Other]
> 
> * This code is likely to change depending on feedback we received from
>    maintainers.
> 
> Jitendra Lanka (1):
>    ptm debugfs: add ddr thermal event nodes
> 
>   drivers/platform/mellanox/mlxbf-ptm.c | 23 +++++++++++++++++++++++
>   1 file changed, 23 insertions(+)
> 
Acked-by: Tim Gardner <tim.gardner@canonical.com>
Bartlomiej Zolnierkiewicz Aug. 4, 2023, 4:15 p.m. UTC | #2 
Acked-by: Bartlomiej Zolnierkiewicz <bartlomiej.zolnierkiewicz@canonical.com>

On Thu, Aug 3, 2023 at 6:27 PM Jitendra Lanka <jlanka@nvidia.com> wrote:
>
> BugLink: https://bugs.launchpad.net/bugs/2029505
>
> SRU Justification:
>
> [Impact]
>
> mlxbf-ptm is a kernel driver that provides debufgs interface for system
> software to monitor Bluefield devices' power and thermal management
> parameters.
>
> [Fix]
>
> * This change adds additional debugfs nodes that provides event counter
>   and throttling states of ddr thermal management state machine.
>
> [Test Case]
>
> * After installing the kernel module, debugfs entry at
>   /sys/kernel/debug/mlxbf-ptm will be created
> * monitors/status folder contains read-only parameters reported from the
>   Bluefield device
>
> [Regression Potential]
>
> * Minimal - as these are read-only parameters that report Bluefield device
>   information.
>
> [Other]
>
> * This code is likely to change depending on feedback we received from
>   maintainers.
>
> Jitendra Lanka (1):
>   ptm debugfs: add ddr thermal event nodes
>
>  drivers/platform/mellanox/mlxbf-ptm.c | 23 +++++++++++++++++++++++
>  1 file changed, 23 insertions(+)
>
Bartlomiej Zolnierkiewicz Aug. 17, 2023, 3:25 p.m. UTC | #3 
Applied to jammy:linux-bluefield/master-next. Thanks.

--
Best regards,
Bartlomiej

On Thu, Aug 3, 2023 at 6:27 PM Jitendra Lanka <jlanka@nvidia.com> wrote:
>
> BugLink: https://bugs.launchpad.net/bugs/2029505
>
> SRU Justification:
>
> [Impact]
>
> mlxbf-ptm is a kernel driver that provides debufgs interface for system
> software to monitor Bluefield devices' power and thermal management
> parameters.
>
> [Fix]
>
> * This change adds additional debugfs nodes that provides event counter
>   and throttling states of ddr thermal management state machine.
>
> [Test Case]
>
> * After installing the kernel module, debugfs entry at
>   /sys/kernel/debug/mlxbf-ptm will be created
> * monitors/status folder contains read-only parameters reported from the
>   Bluefield device
>
> [Regression Potential]
>
> * Minimal - as these are read-only parameters that report Bluefield device
>   information.
>
> [Other]
>
> * This code is likely to change depending on feedback we received from
>   maintainers.
>
> Jitendra Lanka (1):
>   ptm debugfs: add ddr thermal event nodes
>
>  drivers/platform/mellanox/mlxbf-ptm.c | 23 +++++++++++++++++++++++
>  1 file changed, 23 insertions(+)
>