On 6/12/2019 12:49 PM, David Marchand wrote:
This section is about checking dpdk, we can focus on just its version.
Then update the version to 18.11.1 in the commands output.

Fixes: 03f3f9c0faf8 ("dpdk: Update to use DPDK 18.11.")
Fixes: b5355b0d6e66 ("dpdk: Use DPDK 18.11.1 release.")

Signed-off-by: David Marchand <[email protected]>
---
  Documentation/intro/install/dpdk.rst | 7 +++----
  1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/Documentation/intro/install/dpdk.rst 
b/Documentation/intro/install/dpdk.rst
index 149548b..e4a95ba 100644
--- a/Documentation/intro/install/dpdk.rst
+++ b/Documentation/intro/install/dpdk.rst
@@ -281,11 +281,10 @@ initialization succeeded::
  Additionally, the library version linked to ovs-vswitchd can be confirmed
  with either the ovs-vswitchd logs, or by running either of the commands::
- $ ovs-vswitchd --version
-  ovs-vswitchd (Open vSwitch) 2.9.0
-  DPDK 17.11.0
+  $ ovs-vswitchd --version |grep -i dpdk
+  DPDK 18.11.1
    $ ovs-vsctl get Open_vSwitch . dpdk_version
-  "DPDK 17.11.0"
+  "DPDK 18.11.1"

Hi David,

thanks for this. I had submitted a similar change originally in the v2 for supporting 18.11 as you have above, but the feedback was to drop the change as we could run into a case where on the master branch we support a DPDK version that is different to what the latest OVS release supports

https://patchwork.ozlabs.org/patch/1011038/

As this is just an example command, the consensus seemed to be it was ok to leave it as is.

The DPDK version wasn't listed before OVS 2.10 release, so an alternative approach could be to tag that this was introduced in OVS 2.10 using the version changed tag

.. versionchanged:: 2.10.0

and flag above is an example output, users should check the mapping of OVS to DPDK using the OVS release faq (that will be kept up to date for the latest DPDK release anyhow). Thoughts?

Regards
Ian


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

Reply via email to