diff mbox series

[ovs-dev,3/3] netdev-dpdk: Document rx-steering status options.

Message ID 20231004083221.29507-1-jmeng@redhat.com
State Superseded, archived
Headers show
Series [ovs-dev,1/3] netdev-dpdk: Document status options for VF MAC address. | expand

Checks

Context Check Description
ovsrobot/apply-robot warning apply and check: warning
ovsrobot/github-robot-_Build_and_Test success github build: passed
ovsrobot/intel-ovs-compilation success test: success

Commit Message

Jakob Meng Oct. 4, 2023, 8:32 a.m. UTC
From: Jakob Meng <code@jakobmeng.de>

Fixes: fc06ea9a1883 ("netdev-dpdk: Add custom rx-steering configuration.")
---
 vswitchd/vswitch.xml | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

Comments

Simon Horman Oct. 4, 2023, 9:29 a.m. UTC | #1
On Wed, Oct 04, 2023 at 10:32:22AM +0200, jmeng@redhat.com wrote:
> From: Jakob Meng <code@jakobmeng.de>
> 
> Fixes: fc06ea9a1883 ("netdev-dpdk: Add custom rx-steering configuration.")

Acked-by: Simon Horman <horms@ovn.org>
0-day Robot Oct. 4, 2023, 10:04 a.m. UTC | #2
Bleep bloop.  Greetings Jakob Meng, I am a robot and I have tried out your patch.
Thanks for your contribution.

I encountered some error that I wasn't expecting.  See the details below.


checkpatch:
ERROR: Author Jakob Meng <code@jakobmeng.de> needs to sign off.
Lines checked: 40, Warnings: 0, Errors: 1


Please check this out.  If you feel there has been an error, please email aconole@redhat.com

Thanks,
0-day Robot
Jakob Meng Oct. 4, 2023, 10:22 a.m. UTC | #3
On 04.10.23 10:32, jmeng@redhat.com wrote:
> From: Jakob Meng <code@jakobmeng.de>
>
> Fixes: fc06ea9a1883 ("netdev-dpdk: Add custom rx-steering configuration.")
> ---
>  vswitchd/vswitch.xml | 14 ++++++++++++++
>  1 file changed, 14 insertions(+)
>
> diff --git a/vswitchd/vswitch.xml b/vswitchd/vswitch.xml
> index 006d1e6a4..1e2a1267d 100644
> --- a/vswitchd/vswitch.xml
> +++ b/vswitchd/vswitch.xml
> @@ -3806,6 +3806,20 @@ ovs-vsctl add-port br0 p0 -- set Interface p0 type=patch options:peer=p1 \
>              Ethernet address set for this VF interface. Only reported for dpdk
>              VF representors.
>            </column>
> +
> +          <column name="status" key="rx_steering">
> +            Hardware Rx queue steering policy in use.
> +          </column>
> +
> +          <column name="status" key="rx_steering_queue">
> +            ID of rx steering queue. Only reported if <code>rx-steering</code>
> +            is supported by hardware.
> +          </column>
> +
> +          <column name="status" key="rss_queues">
> +            IDs of rss queues. Only reported if <code>rx-steering</code> is
> +            supported by hardware.
> +          </column>
>        </group>
>      </group>
>  

Signed-off-by: Jakob Meng <code@jakobmeng.de>
Eelco Chaudron Oct. 5, 2023, 12:53 p.m. UTC | #4
On 4 Oct 2023, at 10:32, jmeng@redhat.com wrote:

> From: Jakob Meng <code@jakobmeng.de>
>
> Fixes: fc06ea9a1883 ("netdev-dpdk: Add custom rx-steering configuration.")

This looks good to me!

Acked-by: Eelco Chaudron <echaudro@redhat.com>
Kevin Traynor Oct. 5, 2023, 3:55 p.m. UTC | #5
On 04/10/2023 09:32, jmeng@redhat.com wrote:
> From: Jakob Meng <code@jakobmeng.de>
> 
> Fixes: fc06ea9a1883 ("netdev-dpdk: Add custom rx-steering configuration.")
> ---
>   vswitchd/vswitch.xml | 14 ++++++++++++++
>   1 file changed, 14 insertions(+)
> 
> diff --git a/vswitchd/vswitch.xml b/vswitchd/vswitch.xml
> index 006d1e6a4..1e2a1267d 100644
> --- a/vswitchd/vswitch.xml
> +++ b/vswitchd/vswitch.xml
> @@ -3806,6 +3806,20 @@ ovs-vsctl add-port br0 p0 -- set Interface p0 type=patch options:peer=p1 \
>               Ethernet address set for this VF interface. Only reported for dpdk
>               VF representors.
>             </column>
> +
> +          <column name="status" key="rx_steering">
> +            Hardware Rx queue steering policy in use.
> +          </column>
> +
> +          <column name="status" key="rx_steering_queue">
> +            ID of rx steering queue. Only reported if <code>rx-steering</code>
> +            is supported by hardware.
> +          </column>
> +
> +          <column name="status" key="rss_queues">
> +            IDs of rss queues. Only reported if <code>rx-steering</code> is
> +            supported by hardware.
> +          </column>
>         </group>
>       </group>
>   

Acked-by: Kevin Traynor <ktraynor@redhat.com>
diff mbox series

Patch

diff --git a/vswitchd/vswitch.xml b/vswitchd/vswitch.xml
index 006d1e6a4..1e2a1267d 100644
--- a/vswitchd/vswitch.xml
+++ b/vswitchd/vswitch.xml
@@ -3806,6 +3806,20 @@  ovs-vsctl add-port br0 p0 -- set Interface p0 type=patch options:peer=p1 \
             Ethernet address set for this VF interface. Only reported for dpdk
             VF representors.
           </column>
+
+          <column name="status" key="rx_steering">
+            Hardware Rx queue steering policy in use.
+          </column>
+
+          <column name="status" key="rx_steering_queue">
+            ID of rx steering queue. Only reported if <code>rx-steering</code>
+            is supported by hardware.
+          </column>
+
+          <column name="status" key="rss_queues">
+            IDs of rss queues. Only reported if <code>rx-steering</code> is
+            supported by hardware.
+          </column>
       </group>
     </group>