mbox series

[net,V4,0/3] Fix PFC related issues

Message ID 20230824081032.436432-1-sumang@marvell.com (mailing list archive)
Headers show
Series Fix PFC related issues | expand

Message

Suman Ghosh Aug. 24, 2023, 8:10 a.m. UTC
This patchset fixes multiple PFC related issues related to Octeon.

Patch #1: octeontx2-pf: Fix PFC TX scheduler free

Patch #2: octeontx2-af: CN10KB: fix PFC configuration

Patch #3: octeonxt2-pf: Fix backpressure config for multiple PFC
priorities to work simultaneously

Hariprasad Kelam (1):
  octeontx2-af: CN10KB: fix PFC configuration

Suman Ghosh (2):
  octeontx2-pf: Fix PFC TX scheduler free
  cteonxt2-pf: Fix backpressure config for multiple PFC priorities to
    work simultaneously

---
v4 changes:
	Added detailed commit description for patch #2

v3 changes:
	Removed patch #1 from v2 patchset due to review comment from Jakub.
	I will find a more accurate fix for that and will push a separate
	patch.

v2 changes:
	1. Fixed compilation error in patch #2
	ERROR: modpost: "otx2_txschq_free_one"
	[drivers/net/ethernet/marvell/octeontx2/nic/rvu_nicvf.ko] undefined!
	2. Added new patch #4 to the patch set. This patch fixes another PFC
	related issue.

 .../net/ethernet/marvell/octeontx2/af/rpm.c   | 17 +++++++++--------
 .../marvell/octeontx2/nic/otx2_common.c       |  1 +
 .../marvell/octeontx2/nic/otx2_dcbnl.c        | 19 ++++++-------------
 3 files changed, 16 insertions(+), 21 deletions(-)

Comments

Simon Horman Aug. 25, 2023, 10:10 a.m. UTC | #1
On Thu, Aug 24, 2023 at 01:40:29PM +0530, Suman Ghosh wrote:
> This patchset fixes multiple PFC related issues related to Octeon.
> 
> Patch #1: octeontx2-pf: Fix PFC TX scheduler free
> 
> Patch #2: octeontx2-af: CN10KB: fix PFC configuration
> 
> Patch #3: octeonxt2-pf: Fix backpressure config for multiple PFC
> priorities to work simultaneously
> 
> Hariprasad Kelam (1):
>   octeontx2-af: CN10KB: fix PFC configuration
> 
> Suman Ghosh (2):
>   octeontx2-pf: Fix PFC TX scheduler free
>   cteonxt2-pf: Fix backpressure config for multiple PFC priorities to
>     work simultaneously

For series,

Reviewed-by: Simon Horman <horms@kernel.org>
Jakub Kicinski Aug. 26, 2023, 1:50 a.m. UTC | #2
On Thu, 24 Aug 2023 13:40:29 +0530 Suman Ghosh wrote:
> This patchset fixes multiple PFC related issues related to Octeon.
> 
> Patch #1: octeontx2-pf: Fix PFC TX scheduler free
> 
> Patch #2: octeontx2-af: CN10KB: fix PFC configuration
> 
> Patch #3: octeonxt2-pf: Fix backpressure config for multiple PFC
> priorities to work simultaneously

Minor note for the future, if you don't mind.
Please add the prefix to the subject of the series as well
(octeonxt2: ...), that way it's clear what the driver is
in all the places which use the subject of the cover letter
to identify the series.
patchwork-bot+netdevbpf@kernel.org Aug. 26, 2023, 2 a.m. UTC | #3
Hello:

This series was applied to netdev/net.git (main)
by Jakub Kicinski <kuba@kernel.org>:

On Thu, 24 Aug 2023 13:40:29 +0530 you wrote:
> This patchset fixes multiple PFC related issues related to Octeon.
> 
> Patch #1: octeontx2-pf: Fix PFC TX scheduler free
> 
> Patch #2: octeontx2-af: CN10KB: fix PFC configuration
> 
> Patch #3: octeonxt2-pf: Fix backpressure config for multiple PFC
> priorities to work simultaneously
> 
> [...]

Here is the summary with links:
  - [net,V4,1/3] octeontx2-pf: Fix PFC TX scheduler free
    https://git.kernel.org/netdev/net/c/a9ac2e187795
  - [net,V4,2/3] octeontx2-af: CN10KB: fix PFC configuration
    https://git.kernel.org/netdev/net/c/47bcc9c1cf6a
  - [net,V4,3/3] cteonxt2-pf: Fix backpressure config for multiple PFC priorities to work simultaneously
    https://git.kernel.org/netdev/net/c/597d0ec0e4ca

You are awesome, thank you!
Suman Ghosh Aug. 26, 2023, 2:27 a.m. UTC | #4
>
>Minor note for the future, if you don't mind.
>Please add the prefix to the subject of the series as well
>(octeonxt2: ...), that way it's clear what the driver is in all the
>places which use the subject of the cover letter to identify the series.
[Suman] Sure Jakub.