mbox series

[SRU,J:linux-bluefield,v1,0/1] UBUNTU: SAUCE: mlxbf-ptm: use single SMC for total power

Message ID cover.1718287750.git.jlanka@nvidia.com
Headers show
Series UBUNTU: SAUCE: mlxbf-ptm: use single SMC for total power | expand

Message

Jitendra Lanka June 13, 2024, 3:18 p.m. UTC
BugLink: https://bugs.launchpad.net/bugs/2069303

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 performance improvements to total power reporting with a
  single secure monitor call.

[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):
  UBUNTU: SAUCE: mlxbf-ptm: use single SMC for total power

 drivers/platform/mellanox/mlxbf-ptm.c | 13 ++-----------
 1 file changed, 2 insertions(+), 11 deletions(-)

Comments

Bartlomiej Zolnierkiewicz June 13, 2024, 3:58 p.m. UTC | #1
Acked-by: Bartlomiej Zolnierkiewicz <bartlomiej.zolnierkiewicz@canonical.com>

On Thu, Jun 13, 2024 at 5:19 PM Jitendra Lanka <jlanka@nvidia.com> wrote:
>
> BugLink: https://bugs.launchpad.net/bugs/2069303
>
> 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 performance improvements to total power reporting with a
>   single secure monitor call.
>
> [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):
>   UBUNTU: SAUCE: mlxbf-ptm: use single SMC for total power
>
>  drivers/platform/mellanox/mlxbf-ptm.c | 13 ++-----------
>  1 file changed, 2 insertions(+), 11 deletions(-)
>
Andrei Gherzan June 14, 2024, 9:19 a.m. UTC | #2
On 24/06/13 03:18pm, Jitendra Lanka wrote:
> BugLink: https://bugs.launchpad.net/bugs/2069303
> 
> 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 performance improvements to total power reporting with a
>   single secure monitor call.
> 
> [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):
>   UBUNTU: SAUCE: mlxbf-ptm: use single SMC for total power
> 
>  drivers/platform/mellanox/mlxbf-ptm.c | 13 ++-----------
>  1 file changed, 2 insertions(+), 11 deletions(-)

Acked-by: Andrei Gherzan <andrei.gherzan@canonical.com>
Bartlomiej Zolnierkiewicz June 17, 2024, 10:44 a.m. UTC | #3
Applied to jammy:linux-bluefield/master-next. Thanks.

--
Best regards,
Bartlomiej

On Thu, Jun 13, 2024 at 5:19 PM Jitendra Lanka <jlanka@nvidia.com> wrote:
>
> BugLink: https://bugs.launchpad.net/bugs/2069303
>
> 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 performance improvements to total power reporting with a
>   single secure monitor call.
>
> [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):
>   UBUNTU: SAUCE: mlxbf-ptm: use single SMC for total power
>
>  drivers/platform/mellanox/mlxbf-ptm.c | 13 ++-----------
>  1 file changed, 2 insertions(+), 11 deletions(-)
>