mbox series

[v1,0/1,SRU,J:linux-bluefield,0/1] UBUNTU: SAUCE: mlxbf-ptm: power and thermal management debugfs interface

Message ID cover.1673641685.git.jlanka@nvidia.com
Headers show
Series UBUNTU: SAUCE: mlxbf-ptm: power and thermal management debugfs interface | expand

Message

Jitendra Lanka Jan. 13, 2023, 8:54 p.m. UTC
BugLink: https://bugs.launchpad.net/bugs/2002852

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]

* New driver added

[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):
  mlxbf-ptm: power and thermal management debugfs interface

 debian.bluefield/config/config.common.ubuntu |   1 +
 drivers/platform/mellanox/Kconfig            |  10 +
 drivers/platform/mellanox/Makefile           |   1 +
 drivers/platform/mellanox/mlxbf-ptm.c        | 215 +++++++++++++++++++
 4 files changed, 227 insertions(+)
 create mode 100644 drivers/platform/mellanox/mlxbf-ptm.c

Comments

Stefan Bader Jan. 16, 2023, 9:59 a.m. UTC | #1
On 13.01.23 21:54, Jitendra Lanka wrote:
> BugLink: https://bugs.launchpad.net/bugs/2002852
> 
> 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]
> 
> * New driver added
> 
> [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):
>    mlxbf-ptm: power and thermal management debugfs interface
> 
>   debian.bluefield/config/config.common.ubuntu |   1 +
>   drivers/platform/mellanox/Kconfig            |  10 +
>   drivers/platform/mellanox/Makefile           |   1 +
>   drivers/platform/mellanox/mlxbf-ptm.c        | 215 +++++++++++++++++++
>   4 files changed, 227 insertions(+)
>   create mode 100644 drivers/platform/mellanox/mlxbf-ptm.c
> 

I see 4 emails here. 2 cover emails and 2 patches which seem like only one 
submission to me. The message-id's and references seem to be badly messed up. 
This results in a rather unclear and confusing situation. I will reject all the 
pieces for now. Please submit again after checking that there will be only one 
thread done.

-Stefan
Stefan Bader Jan. 16, 2023, 10 a.m. UTC | #2
On 14.01.23 16:34, Jitendra Lanka wrote:
> BugLink: https://bugs.launchpad.net/bugs/2002852
> 
> 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]
> 
> * New driver added
> 
> [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):
>    mlxbf-ptm: power and thermal management debugfs interface
> 
>   debian.bluefield/config/config.common.ubuntu |   1 +
>   drivers/platform/mellanox/Kconfig            |  10 +
>   drivers/platform/mellanox/Makefile           |   1 +
>   drivers/platform/mellanox/mlxbf-ptm.c        | 215 +++++++++++++++++++
>   4 files changed, 227 insertions(+)
>   create mode 100644 drivers/platform/mellanox/mlxbf-ptm.c
> 
Corrupted submission