On 05/10/2023 13:52, Eelco Chaudron wrote:


On 4 Oct 2023, at 10:31, [email protected] wrote:

From: Jakob Meng <[email protected]>

The status options pci-vendor_id and pci-device_id for dpdk netdevs
have been replaced by bus_info. This patch updates the documentation
in vswitchd/vswitch.xml accordingly.

Fixes: a77c7796f23a ("dpdk: Update to use v22.11.1.")

Thanks for fixing the documentation, however I think the commit message should 
somehow state this is related to the documentation.


Message it probably ok, but yeah title might mislead so should mention docs.

Acked-by: Kevin Traynor <[email protected]>

//Eelco

---
  vswitchd/vswitch.xml | 9 +++------
  1 file changed, 3 insertions(+), 6 deletions(-)

diff --git a/vswitchd/vswitch.xml b/vswitchd/vswitch.xml
index 797fb05bf..006d1e6a4 100644
--- a/vswitchd/vswitch.xml
+++ b/vswitchd/vswitch.xml
@@ -3797,12 +3797,9 @@ ovs-vsctl add-port br0 p0 -- set Interface p0 type=patch 
options:peer=p1 \
              Interface description string.
            </column>

-          <column name="status" key="pci-vendor_id">
-            Vendor ID of PCI device.
-          </column>
-
-          <column name="status" key="pci-device_id">
-            Device ID of PCI device.
+          <column name="status" key="bus_info">
+            Bus name and bus info such as Vendor ID and Device ID of PCI
+            device.
            </column>

            <column name="status" key="dpdk-vf-mac">
--
2.39.2

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev


_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to