diff mbox

[1/1] ODP-DPDK README update

Message ID 1407406581-10935-1-git-send-email-venkatesh.vivekanandan@linaro.org
State New
Headers show

Commit Message

Venkatesh Vivekanandan Aug. 7, 2014, 10:16 a.m. UTC
From: Venkatesh Vivekanandan <venkatesh.vivekanandan@linaro.org>

- export RTE_SDK and RTE_TARGET before compiling ODP
- new commands for odp_l2fwd and pktio
- cleanup

Signed-off-by: Venkatesh Vivekanandan <venkatesh.vivekanandan@linaro.org>
---
 platform/linux-dpdk/README | 23 +++++++++++++++--------
 1 file changed, 15 insertions(+), 8 deletions(-)

Comments

Anders Roxell Aug. 7, 2014, 3:50 p.m. UTC | #1
On 2014-08-07 15:46, venkatesh.vivekanandan@linaro.org wrote:
> From: Venkatesh Vivekanandan <venkatesh.vivekanandan@linaro.org>
> 
> - export RTE_SDK and RTE_TARGET before compiling ODP
> - new commands for odp_l2fwd and pktio
> - cleanup
> 
> Signed-off-by: Venkatesh Vivekanandan <venkatesh.vivekanandan@linaro.org>
> ---
>  platform/linux-dpdk/README | 23 +++++++++++++++--------
>  1 file changed, 15 insertions(+), 8 deletions(-)
> 
> diff --git a/platform/linux-dpdk/README b/platform/linux-dpdk/README
> index 14ad4e8..da9891c 100644
> --- a/platform/linux-dpdk/README
> +++ b/platform/linux-dpdk/README
> @@ -22,16 +22,21 @@ $ git checkout -b 1.6.0 tags/v1.6.0r2
>  # Please refer "How to setup and compile DPDK" section in this document
>  
>  # To compile ODP with linux-dpdk
> +
> +$ export RTE_SDK=<dpdk-dir>
> +# For example
> +$ export RTE_SDK=/root/dpdk
> +$ export RTE_TARGET=x86_64-default-linuxapp-gcc
> +# RTE_TARGET is the configuration used while building DPDK
> +
>  $ cd <odp-dir>
>  $ ./bootstrap
> -$ ./configure --with-platform=linux-dpdk --with-sdk-install-path=<dpdk-install-path>
> -# For example
> -$ ./configure --with-platform=linux-dpdk --with-sdk-install-path=/root/dpdk/build
> +$ ./configure --with-platform=linux-dpdk --with-sdk-install-path=$RTE_SDK/$RTE_TARGET
>  $ make

I'm not convinced that this is easier than it was before... The only
time its easier is if you build dpdk from the same shell then you don't
need to export RTE_SDK and RTE_TARGET... Because you have to do that
when you build dpdk right?

>  
> -# Commands to test
> -l2fwding app - sudo ./test/l2fwd/odp_l2fwd -i 0,1 -m 0 -c 2
> -loopback app - sudo ./test/packet/odp_pktio -i 0,1 -m 0 -c 2
> +# App commands to test
> +l2fwding app - sudo ./example/l2fwd/odp_l2fwd -i 0,1 -m 0 -c 2
> +loopback app - sudo ./example/packet/odp_pktio -i 0,1 -m 0 -c 2
>  
>  	-i 0,1 - interface number
>  	-m 0   - burst mode
> @@ -42,8 +47,10 @@ How to setup and compile DPDK:
>  $ cd <dpdk-dir>
>  
>  # This has to be done only once.
> -# Following command will generate build dir as ./build in <dpdk-dir>
> -$ make install T=x86_64-default-linuxapp-gcc CONFIG_RTE_BUILD_COMBINE_LIBS=y EXTRA_CFLAGS="-fPIC"
> +$ make install T=x86_64-default-linuxapp-gcc EXTRA_CFLAGS="-fPIC"
> +# set CONFIG_RTE_BUILD_COMBINE_LIBS=y in .config file

Why is it better to split this up on two lines?
If its better to split it up on two lines:
Why do I want to change .config after I've done make install?

> +# Note: if non-intel SFP's are used in IXGBE, then set CONFIG_RTE_LIBRTE_IXGBE_ALLOW_UNSUPPORTED_SFP=y in .config file

I think this information can be good.
Maybe we should point out DPDKs README if there is one as well...

> +

Not needed


Cheers,
Anders

>  
>  # If "conflicting types for skb_set_hash" error happens during dpdk
>  # build, then please knock-off skb_set_hash function from kcompat.h as
> -- 
> 1.8.1.2
> 
> 
> _______________________________________________
> lng-odp mailing list
> lng-odp@lists.linaro.org
> http://lists.linaro.org/mailman/listinfo/lng-odp
Anders Roxell Aug. 7, 2014, 7:09 p.m. UTC | #2
On 2014-08-07 18:42, Wiles, Roger Keith wrote:
> Export the RTE_SDK/RTE_TARGET is a requirement for DPDK to build example directory or external applications to locate the includes/libs. When doing a ‘make install T=…’ they assume and build the two variables internally to that build only. Also you can ‘cd’ into x86_64-default-linuxapp-gcc and just type make as they know all of the values already. They did not force you to export the variables, but it does help to make sure you are building with the correct includes/libs. You can have multiple configuration directories in a single DPDK tree as long as you define the correct RTE_TARGET you can build the correct one.

Argh, if I remember correctly you mentioned this earlier as well!
Thank you for clarifying this again.

Cheers,
Anders

> 
> Keith Wiles, Principal Technologist with CTO office, Wind River
> mobile 972-213-5533
> 
> [Powering 30 Years of Innovation]<http://www.windriver.com/announces/wr30/>
> 
> On Aug 7, 2014, at 10:50 AM, Anders Roxell <anders.roxell@linaro.org<mailto:anders.roxell@linaro.org>> wrote:
> 
> On 2014-08-07 15:46, venkatesh.vivekanandan@linaro.org<mailto:venkatesh.vivekanandan@linaro.org> wrote:
> From: Venkatesh Vivekanandan <venkatesh.vivekanandan@linaro.org<mailto:venkatesh.vivekanandan@linaro.org>>
> 
> - export RTE_SDK and RTE_TARGET before compiling ODP
> - new commands for odp_l2fwd and pktio
> - cleanup
> 
> Signed-off-by: Venkatesh Vivekanandan <venkatesh.vivekanandan@linaro.org<mailto:venkatesh.vivekanandan@linaro.org>>
> ---
> platform/linux-dpdk/README | 23 +++++++++++++++--------
> 1 file changed, 15 insertions(+), 8 deletions(-)
> 
> diff --git a/platform/linux-dpdk/README b/platform/linux-dpdk/README
> index 14ad4e8..da9891c 100644
> --- a/platform/linux-dpdk/README
> +++ b/platform/linux-dpdk/README
> @@ -22,16 +22,21 @@ $ git checkout -b 1.6.0 tags/v1.6.0r2
> # Please refer "How to setup and compile DPDK" section in this document
> 
> # To compile ODP with linux-dpdk
> +
> +$ export RTE_SDK=<dpdk-dir>
> +# For example
> +$ export RTE_SDK=/root/dpdk
> +$ export RTE_TARGET=x86_64-default-linuxapp-gcc
> +# RTE_TARGET is the configuration used while building DPDK
> +
> $ cd <odp-dir>
> $ ./bootstrap
> -$ ./configure --with-platform=linux-dpdk --with-sdk-install-path=<dpdk-install-path>
> -# For example
> -$ ./configure --with-platform=linux-dpdk --with-sdk-install-path=/root/dpdk/build
> +$ ./configure --with-platform=linux-dpdk --with-sdk-install-path=$RTE_SDK/$RTE_TARGET
> $ make
> 
> I'm not convinced that this is easier than it was before... The only
> time its easier is if you build dpdk from the same shell then you don't
> need to export RTE_SDK and RTE_TARGET... Because you have to do that
> when you build dpdk right?
> 
> 
> -# Commands to test
> -l2fwding app - sudo ./test/l2fwd/odp_l2fwd -i 0,1 -m 0 -c 2
> -loopback app - sudo ./test/packet/odp_pktio -i 0,1 -m 0 -c 2
> +# App commands to test
> +l2fwding app - sudo ./example/l2fwd/odp_l2fwd -i 0,1 -m 0 -c 2
> +loopback app - sudo ./example/packet/odp_pktio -i 0,1 -m 0 -c 2
> 
> -i 0,1 - interface number
> -m 0   - burst mode
> @@ -42,8 +47,10 @@ How to setup and compile DPDK:
> $ cd <dpdk-dir>
> 
> # This has to be done only once.
> -# Following command will generate build dir as ./build in <dpdk-dir>
> -$ make install T=x86_64-default-linuxapp-gcc CONFIG_RTE_BUILD_COMBINE_LIBS=y EXTRA_CFLAGS="-fPIC"
> +$ make install T=x86_64-default-linuxapp-gcc EXTRA_CFLAGS="-fPIC"
> +# set CONFIG_RTE_BUILD_COMBINE_LIBS=y in .config file
> 
> Why is it better to split this up on two lines?
> If its better to split it up on two lines:
> Why do I want to change .config after I've done make install?
> 
> +# Note: if non-intel SFP's are used in IXGBE, then set CONFIG_RTE_LIBRTE_IXGBE_ALLOW_UNSUPPORTED_SFP=y in .config file
> 
> I think this information can be good.
> Maybe we should point out DPDKs README if there is one as well...
> 
> +
> 
> Not needed
> 
> 
> Cheers,
> Anders
> 
> 
> # If "conflicting types for skb_set_hash" error happens during dpdk
> # build, then please knock-off skb_set_hash function from kcompat.h as
> --
> 1.8.1.2
> 
> 
> _______________________________________________
> lng-odp mailing list
> lng-odp@lists.linaro.org<mailto:lng-odp@lists.linaro.org>
> http://lists.linaro.org/mailman/listinfo/lng-odp
> 
> --
> Anders Roxell
> anders.roxell@linaro.org<mailto:anders.roxell@linaro.org>
> M: +46 709 71 42 85 | IRC: roxell
> 
> _______________________________________________
> lng-odp mailing list
> lng-odp@lists.linaro.org<mailto:lng-odp@lists.linaro.org>
> http://lists.linaro.org/mailman/listinfo/lng-odp
>
diff mbox

Patch

diff --git a/platform/linux-dpdk/README b/platform/linux-dpdk/README
index 14ad4e8..da9891c 100644
--- a/platform/linux-dpdk/README
+++ b/platform/linux-dpdk/README
@@ -22,16 +22,21 @@  $ git checkout -b 1.6.0 tags/v1.6.0r2
 # Please refer "How to setup and compile DPDK" section in this document
 
 # To compile ODP with linux-dpdk
+
+$ export RTE_SDK=<dpdk-dir>
+# For example
+$ export RTE_SDK=/root/dpdk
+$ export RTE_TARGET=x86_64-default-linuxapp-gcc
+# RTE_TARGET is the configuration used while building DPDK
+
 $ cd <odp-dir>
 $ ./bootstrap
-$ ./configure --with-platform=linux-dpdk --with-sdk-install-path=<dpdk-install-path>
-# For example
-$ ./configure --with-platform=linux-dpdk --with-sdk-install-path=/root/dpdk/build
+$ ./configure --with-platform=linux-dpdk --with-sdk-install-path=$RTE_SDK/$RTE_TARGET
 $ make
 
-# Commands to test
-l2fwding app - sudo ./test/l2fwd/odp_l2fwd -i 0,1 -m 0 -c 2
-loopback app - sudo ./test/packet/odp_pktio -i 0,1 -m 0 -c 2
+# App commands to test
+l2fwding app - sudo ./example/l2fwd/odp_l2fwd -i 0,1 -m 0 -c 2
+loopback app - sudo ./example/packet/odp_pktio -i 0,1 -m 0 -c 2
 
 	-i 0,1 - interface number
 	-m 0   - burst mode
@@ -42,8 +47,10 @@  How to setup and compile DPDK:
 $ cd <dpdk-dir>
 
 # This has to be done only once.
-# Following command will generate build dir as ./build in <dpdk-dir>
-$ make install T=x86_64-default-linuxapp-gcc CONFIG_RTE_BUILD_COMBINE_LIBS=y EXTRA_CFLAGS="-fPIC"
+$ make install T=x86_64-default-linuxapp-gcc EXTRA_CFLAGS="-fPIC"
+# set CONFIG_RTE_BUILD_COMBINE_LIBS=y in .config file
+# Note: if non-intel SFP's are used in IXGBE, then set CONFIG_RTE_LIBRTE_IXGBE_ALLOW_UNSUPPORTED_SFP=y in .config file
+
 
 # If "conflicting types for skb_set_hash" error happens during dpdk
 # build, then please knock-off skb_set_hash function from kcompat.h as