On 7/30/2024 10:31 PM, Shradha Gupta wrote:
Currently the values of WQs for RX and TX queues for MANA devices
are hardcoded to default sizes.
Allow configuring these values for MANA devices as ringparam
configuration(get/set) through ethtool_ops.

Signed-off-by: Shradha Gupta <shradhagu...@linux.microsoft.com>
Reviewed-by: Haiyang Zhang <haiya...@microsoft.com>
Reviewed-by: Long Li <lon...@microsoft.com>
---
  Changes in v2:
  * Removed unnecessary validations in mana_set_ringparam()
  * Fixed codespell error
  * Improved error message to indicate issue with the parameter
---
  drivers/net/ethernet/microsoft/mana/mana_en.c | 20 +++---
  .../ethernet/microsoft/mana/mana_ethtool.c    | 66 +++++++++++++++++++
  include/net/mana/mana.h                       | 21 +++++-
  3 files changed, 96 insertions(+), 11 deletions(-)


From what I understand, we are adding support for "ethtool -G --set-
ring"  command.
Please correct me if I am wrong.

Maybe it would be good to capture the benefit/purpose of this patch in
the commit msg, as in which use-cases/scenarios we are now trying to
support that previously were not supported. The "why?" part basically.



Regards,
Naman Jain

Reply via email to