[net-next] Documentation: devlink: Add blank line padding on numbered lists in Devlink Port documentation

Message ID 20221121035854.28411-1-bagasdotme@gmail.com
State New
Headers
Series [net-next] Documentation: devlink: Add blank line padding on numbered lists in Devlink Port documentation |

Commit Message

Bagas Sanjaya Nov. 21, 2022, 3:58 a.m. UTC
  kernel test robot reported indentation warnings:

Documentation/networking/devlink/devlink-port.rst:220: WARNING: Unexpected indentation.
Documentation/networking/devlink/devlink-port.rst:222: WARNING: Block quote ends without a blank line; unexpected unindent.

These warnings cause lists (arbitration flow for which the warnings blame to
and 3-step subfunction setup) to be rendered inline instead. Also, for the
former list, automatic list numbering is messed up.

Fix these warnings by adding missing blank line padding.

Link: https://lore.kernel.org/linux-doc/202211200926.kfOPiVti-lkp@intel.com/
Fixes: 242dd64375b80a ("Documentation: Add documentation for new devlink-rate attributes")
Reported-by: kernel test robot <lkp@intel.com>
Signed-off-by: Bagas Sanjaya <bagasdotme@gmail.com>
---
 Documentation/networking/devlink/devlink-port.rst | 13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)


base-commit: 8bd8dcc5e47f0f9dc40187c3b8b42d992181eee1
  

Comments

patchwork-bot+netdevbpf@kernel.org Nov. 23, 2022, 12:50 p.m. UTC | #1
Hello:

This patch was applied to netdev/net-next.git (master)
by David S. Miller <davem@davemloft.net>:

On Mon, 21 Nov 2022 10:58:55 +0700 you wrote:
> kernel test robot reported indentation warnings:
> 
> Documentation/networking/devlink/devlink-port.rst:220: WARNING: Unexpected indentation.
> Documentation/networking/devlink/devlink-port.rst:222: WARNING: Block quote ends without a blank line; unexpected unindent.
> 
> These warnings cause lists (arbitration flow for which the warnings blame to
> and 3-step subfunction setup) to be rendered inline instead. Also, for the
> former list, automatic list numbering is messed up.
> 
> [...]

Here is the summary with links:
  - [net-next] Documentation: devlink: Add blank line padding on numbered lists in Devlink Port documentation
    https://git.kernel.org/netdev/net-next/c/c84f6f6c2bb5

You are awesome, thank you!
  

Patch

diff --git a/Documentation/networking/devlink/devlink-port.rst b/Documentation/networking/devlink/devlink-port.rst
index 643f5903d1d8aa..98557c2ab1c11f 100644
--- a/Documentation/networking/devlink/devlink-port.rst
+++ b/Documentation/networking/devlink/devlink-port.rst
@@ -130,10 +130,11 @@  it is deployed. Subfunction is created and deployed in unit of 1. Unlike
 SRIOV VFs, a subfunction doesn't require its own PCI virtual function.
 A subfunction communicates with the hardware through the parent PCI function.
 
-To use a subfunction, 3 steps setup sequence is followed.
-(1) create - create a subfunction;
-(2) configure - configure subfunction attributes;
-(3) deploy - deploy the subfunction;
+To use a subfunction, 3 steps setup sequence is followed:
+
+1) create - create a subfunction;
+2) configure - configure subfunction attributes;
+3) deploy - deploy the subfunction;
 
 Subfunction management is done using devlink port user interface.
 User performs setup on the subfunction management device.
@@ -216,13 +217,17 @@  nodes with the same priority form a WFQ subgroup in the sibling group
 and arbitration among them is based on assigned weights.
 
 Arbitration flow from the high level:
+
 #. Choose a node, or group of nodes with the highest priority that stays
    within the BW limit and are not blocked. Use ``tx_priority`` as a
    parameter for this arbitration.
+
 #. If group of nodes have the same priority perform WFQ arbitration on
    that subgroup. Use ``tx_weight`` as a parameter for this arbitration.
+
 #. Select the winner node, and continue arbitration flow among it's children,
    until leaf node is reached, and the winner is established.
+
 #. If all the nodes from the highest priority sub-group are satisfied, or
    overused their assigned BW, move to the lower priority nodes.