Re: [meta-intel] [meta-dpdk][PATCH 2/2] dpdk: merge recipe and include
Hi Cal, >> Could one of you review my patch and maybe take a look at the configure >> block as a whole? I've provided comments against your patch. Regarding the review of do_configure I believe we can go through it once the current patches are merged. BR, Awais From: Cal Sullivan Sent: Saturday, March 31, 2018 2:26 AM To: Belal, Awais; Kevin Hao Cc: meta-intel@yoctoproject.org Subject: Re: [meta-intel] [meta-dpdk][PATCH 2/2] dpdk: merge recipe and include Hi Awais, Kevin, In testing I ran into some compilation problems: https://pastebin.com/n8dhUmZT I was able to fix it by disabling some options, but I don't know the ramifications of these changes. In working on this I noticed most of the sed commands in that do_configure block could use a review, as some are trying to modify options that no longer exist there, and others assume =n in the string to replace. Could one of you review my patch and maybe take a look at the configure block as a whole? Thanks, Cal On 03/29/2018 11:36 PM, Belal, Awais wrote: > Do we know when this will be merged? > > BR, > Awais -- ___ meta-intel mailing list meta-intel@yoctoproject.org https://lists.yoctoproject.org/listinfo/meta-intel
Re: [meta-intel] [meta-dpdk][PATCH 2/2] dpdk: merge recipe and include
Hi Awais, Kevin, In testing I ran into some compilation problems: https://pastebin.com/n8dhUmZT I was able to fix it by disabling some options, but I don't know the ramifications of these changes. In working on this I noticed most of the sed commands in that do_configure block could use a review, as some are trying to modify options that no longer exist there, and others assume =n in the string to replace. Could one of you review my patch and maybe take a look at the configure block as a whole? Thanks, Cal On 03/29/2018 11:36 PM, Belal, Awais wrote: Do we know when this will be merged? BR, Awais -- ___ meta-intel mailing list meta-intel@yoctoproject.org https://lists.yoctoproject.org/listinfo/meta-intel
Re: [meta-intel] [meta-dpdk][PATCH 2/2] dpdk: merge recipe and include
>> Is patch 1/2 good to merge? > Yes. Do we know when this will be merged? BR, Awais From: Kevin Hao Sent: Thursday, March 29, 2018 6:25 AM To: Cal Sullivan Cc: Belal, Awais; meta-intel@yoctoproject.org Subject: Re: [meta-intel] [meta-dpdk][PATCH 2/2] dpdk: merge recipe and include On Wed, Mar 28, 2018 at 11:34:42AM -0700, Cal Sullivan wrote: > Is patch 1/2 good to merge? Yes. > > It looks like in order for 18.02 and 17.11 to use the same .inc file we will > need to move the LIC_FILES_CHKSUM out of it, but that can be done later. 17.11 > seems to have upstreamed the same patch that was dropped in this upgrade, so > we're good on that front as well. Agreed. Thanks, Kevin > > Thanks, > Cal > > On 03/28/2018 12:46 AM, Kevin Hao wrote: > > On Wed, Mar 28, 2018 at 12:27:48PM +0500, Awais Belal wrote: > > Since we only have a single version supported at the > moment, which is the latest and the greatest. It is > best that the include and base recipe files are merged > to allow better maintainability and readability. > > Please don't do this because I plan to add the 17.11 (LTS) version to the > Yocto. > > Thanks, > Kevin > > > Signed-off-by: Awais Belal > --- > recipes-extended/dpdk/dpdk.inc | 167 > -- > recipes-extended/dpdk/dpdk_18.02.bb | 175 > ++-- > 2 files changed, 167 insertions(+), 175 deletions(-) > delete mode 100644 recipes-extended/dpdk/dpdk.inc > > diff --git a/recipes-extended/dpdk/dpdk.inc > b/recipes-extended/dpdk/dpdk.inc > deleted file mode 100644 > index ce4e8bc..000 > --- a/recipes-extended/dpdk/dpdk.inc > +++ /dev/null > @@ -1,167 +0,0 @@ > -DESCRIPTION = "Intel(r) Data Plane Development Kit" > -HOMEPAGE = "http://dpdk.org"; > -LICENSE = "BSD & LGPLv2 & GPLv2" > -LIC_FILES_CHKSUM = > "file://license/gpl-2.0.txt;md5=b234ee4d69f5fce4486a80fdaf4a4263 \ > - > file://license/lgpl-2.1.txt;md5=4b54a1fd55a448865a0b32d41598759d \ > - > file://license/bsd-3-clause.txt;md5=0f00d99239d922ffd13cabef83b33444" > - > -SRC_URI = "http://fast.dpdk.org/rel/${BP}.tar.gz;name=dpdk \ > - > file://dpdk-16.04-add-RTE_KERNELDIR_OUT-to-split-kernel-bu.patch \ > - > file://dpdk-16.07-add-sysroot-option-within-app-makefile.patch \ > - > file://dpdk-16.04-Fix-for-misleading-indentation-error.patch \ > - file://dpdk-16.07-dpdk-fix-for-parellel-make-issue.patch \ > - > file://dpdk-17.02-dpdk-fix-installation-warning-and-issue.patch \ > - " > - > -# A machine needs to enable this using: > -# COMPATIBLE_MACHINE_pn-dpdk-dev-libibverbs = "" > - > -COMPATIBLE_MACHINE = "null" > -COMPATIBLE_HOST_libc-musl_class-target = "null" > - > - > -# dpdk example apps dpdk_qat and vhost have dependancy on fuse and > qat. > -# fuse is in meta-filesystems and qat is not yet upstreamed. > -# So adding mechanism to explicitly disable the use of fuse and qat. > -# To enable, uncomment the below line or include in .bbappend. > -# PACKAGECONFIG ?= " dpdk_qat vhost libvirt" > - > -PACKAGECONFIG[dpdk_qat] = ",,virtual/qat" > -PACKAGECONFIG[vhost] = ",,fuse" > -PACKAGECONFIG[libvirt] = ",,libvirt" > - > -export CONFIG_EXAMPLE_DPDK_QAT = > "${@bb.utils.contains('PACKAGECONFIG', 'dpdk_qat', 'y', 'n', d)}" > -export CONFIG_EXAMPLE_VM_POWER_MANAGER = > "${@bb.utils.contains('PACKAGECONFIG', 'libvirt', 'y', 'n', d)}" > -export CONFIG_VHOST_ENABLED = "${@bb.utils.contains('PACKAGECONFIG', > 'vhost', 'y', 'n', d)}" > - > -RDEPENDS_${PN} += "python-subprocess virtual/libibverbs" > -DEPENDS = "virtual/kernel virtual/libibverbs" > -do_configure[depends] += "virtual/kernel:do_shared_workdir" > - > -inherit module > - > -export MODULE_DIR="/lib/modules/${KERNEL_VERSION}/kernel/drivers/net" > -export RTE_SDK = "
Re: [meta-intel] [meta-dpdk][PATCH 2/2] dpdk: merge recipe and include
On Wed, Mar 28, 2018 at 11:34:42AM -0700, Cal Sullivan wrote: > Is patch 1/2 good to merge? Yes. > > It looks like in order for 18.02 and 17.11 to use the same .inc file we will > need to move the LIC_FILES_CHKSUM out of it, but that can be done later. 17.11 > seems to have upstreamed the same patch that was dropped in this upgrade, so > we're good on that front as well. Agreed. Thanks, Kevin > > Thanks, > Cal > > On 03/28/2018 12:46 AM, Kevin Hao wrote: > > On Wed, Mar 28, 2018 at 12:27:48PM +0500, Awais Belal wrote: > > Since we only have a single version supported at the > moment, which is the latest and the greatest. It is > best that the include and base recipe files are merged > to allow better maintainability and readability. > > Please don't do this because I plan to add the 17.11 (LTS) version to the > Yocto. > > Thanks, > Kevin > > > Signed-off-by: Awais Belal > --- > recipes-extended/dpdk/dpdk.inc | 167 > -- > recipes-extended/dpdk/dpdk_18.02.bb | 175 > ++-- > 2 files changed, 167 insertions(+), 175 deletions(-) > delete mode 100644 recipes-extended/dpdk/dpdk.inc > > diff --git a/recipes-extended/dpdk/dpdk.inc > b/recipes-extended/dpdk/dpdk.inc > deleted file mode 100644 > index ce4e8bc..000 > --- a/recipes-extended/dpdk/dpdk.inc > +++ /dev/null > @@ -1,167 +0,0 @@ > -DESCRIPTION = "Intel(r) Data Plane Development Kit" > -HOMEPAGE = "http://dpdk.org"; > -LICENSE = "BSD & LGPLv2 & GPLv2" > -LIC_FILES_CHKSUM = > "file://license/gpl-2.0.txt;md5=b234ee4d69f5fce4486a80fdaf4a4263 \ > - > file://license/lgpl-2.1.txt;md5=4b54a1fd55a448865a0b32d41598759d \ > - > file://license/bsd-3-clause.txt;md5=0f00d99239d922ffd13cabef83b33444" > - > -SRC_URI = "http://fast.dpdk.org/rel/${BP}.tar.gz;name=dpdk \ > - > file://dpdk-16.04-add-RTE_KERNELDIR_OUT-to-split-kernel-bu.patch \ > - > file://dpdk-16.07-add-sysroot-option-within-app-makefile.patch \ > - > file://dpdk-16.04-Fix-for-misleading-indentation-error.patch \ > - file://dpdk-16.07-dpdk-fix-for-parellel-make-issue.patch \ > - > file://dpdk-17.02-dpdk-fix-installation-warning-and-issue.patch \ > - " > - > -# A machine needs to enable this using: > -# COMPATIBLE_MACHINE_pn-dpdk-dev-libibverbs = "" > - > -COMPATIBLE_MACHINE = "null" > -COMPATIBLE_HOST_libc-musl_class-target = "null" > - > - > -# dpdk example apps dpdk_qat and vhost have dependancy on fuse and > qat. > -# fuse is in meta-filesystems and qat is not yet upstreamed. > -# So adding mechanism to explicitly disable the use of fuse and qat. > -# To enable, uncomment the below line or include in .bbappend. > -# PACKAGECONFIG ?= " dpdk_qat vhost libvirt" > - > -PACKAGECONFIG[dpdk_qat] = ",,virtual/qat" > -PACKAGECONFIG[vhost] = ",,fuse" > -PACKAGECONFIG[libvirt] = ",,libvirt" > - > -export CONFIG_EXAMPLE_DPDK_QAT = > "${@bb.utils.contains('PACKAGECONFIG', 'dpdk_qat', 'y', 'n', d)}" > -export CONFIG_EXAMPLE_VM_POWER_MANAGER = > "${@bb.utils.contains('PACKAGECONFIG', 'libvirt', 'y', 'n', d)}" > -export CONFIG_VHOST_ENABLED = "${@bb.utils.contains('PACKAGECONFIG', > 'vhost', 'y', 'n', d)}" > - > -RDEPENDS_${PN} += "python-subprocess virtual/libibverbs" > -DEPENDS = "virtual/kernel virtual/libibverbs" > -do_configure[depends] += "virtual/kernel:do_shared_workdir" > - > -inherit module > - > -export MODULE_DIR="/lib/modules/${KERNEL_VERSION}/kernel/drivers/net" > -export RTE_SDK = "${S}" > -export RTE_TARGET="${@bb.utils.contains("TUNE_FEATURES", "m64", > "x86_64-native-linuxapp-gcc", "i686-native-linuxapp-gcc", d)}" > - > -export ICP_ROOT = "${PKG_CONFIG_SYSROOT_DIR}/usr/include" > -export ICP_LIB_ROOT= "${PKG_CONFIG_SYSROOT_DIR}/usr/lib" > -export RTE_KERNELDIR = "${STAGING_KERNEL_DIR}" > -export RTE_KERNELDIR_OUT = "${STAGING_KERNEL_BUILDDIR}" > -export INSTALL_PATH = "${prefix}/share" > -export RTE_OUTPUT = "${S}/${RTE_TARGET}" > -export ETHTOOL_LIB_PATH = "${S}/examples/ethtool/lib/${RTE_TARGET}/" > -export SYSROOTPATH = "--sysroot=${STAGING_DIR_HOST}" > -export DPDK_TARGET_MACH = "${@get_dpdk_target_mach(bb,d)}" > -export ICP_LAC_API_DIR = "${STAGING_DIR_TARGET}${includedir}/lac" > - > -# The list of intel Comms platforms and their target machine > -# process mapping. The supported target machine is lis
Re: [meta-intel] [meta-dpdk][PATCH 2/2] dpdk: merge recipe and include
Is patch 1/2 good to merge? It looks like in order for 18.02 and 17.11 to use the same .inc file we will need to move the LIC_FILES_CHKSUM out of it, but that can be done later. 17.11 seems to have upstreamed the same patch that was dropped in this upgrade, so we're good on that front as well. Thanks, Cal On 03/28/2018 12:46 AM, Kevin Hao wrote: On Wed, Mar 28, 2018 at 12:27:48PM +0500, Awais Belal wrote: Since we only have a single version supported at the moment, which is the latest and the greatest. It is best that the include and base recipe files are merged to allow better maintainability and readability. Please don't do this because I plan to add the 17.11 (LTS) version to the Yocto. Thanks, Kevin Signed-off-by: Awais Belal --- recipes-extended/dpdk/dpdk.inc | 167 -- recipes-extended/dpdk/dpdk_18.02.bb | 175 ++-- 2 files changed, 167 insertions(+), 175 deletions(-) delete mode 100644 recipes-extended/dpdk/dpdk.inc diff --git a/recipes-extended/dpdk/dpdk.inc b/recipes-extended/dpdk/dpdk.inc deleted file mode 100644 index ce4e8bc..000 --- a/recipes-extended/dpdk/dpdk.inc +++ /dev/null @@ -1,167 +0,0 @@ -DESCRIPTION = "Intel(r) Data Plane Development Kit" -HOMEPAGE = "http://dpdk.org"; -LICENSE = "BSD & LGPLv2 & GPLv2" -LIC_FILES_CHKSUM = "file://license/gpl-2.0.txt;md5=b234ee4d69f5fce4486a80fdaf4a4263 \ - file://license/lgpl-2.1.txt;md5=4b54a1fd55a448865a0b32d41598759d \ - file://license/bsd-3-clause.txt;md5=0f00d99239d922ffd13cabef83b33444" - -SRC_URI = "http://fast.dpdk.org/rel/${BP}.tar.gz;name=dpdk \ - file://dpdk-16.04-add-RTE_KERNELDIR_OUT-to-split-kernel-bu.patch \ - file://dpdk-16.07-add-sysroot-option-within-app-makefile.patch \ - file://dpdk-16.04-Fix-for-misleading-indentation-error.patch \ - file://dpdk-16.07-dpdk-fix-for-parellel-make-issue.patch \ - file://dpdk-17.02-dpdk-fix-installation-warning-and-issue.patch \ - " - -# A machine needs to enable this using: -# COMPATIBLE_MACHINE_pn-dpdk-dev-libibverbs = "" - -COMPATIBLE_MACHINE = "null" -COMPATIBLE_HOST_libc-musl_class-target = "null" - - -# dpdk example apps dpdk_qat and vhost have dependancy on fuse and qat. -# fuse is in meta-filesystems and qat is not yet upstreamed. -# So adding mechanism to explicitly disable the use of fuse and qat. -# To enable, uncomment the below line or include in .bbappend. -# PACKAGECONFIG ?= " dpdk_qat vhost libvirt" - -PACKAGECONFIG[dpdk_qat] = ",,virtual/qat" -PACKAGECONFIG[vhost] = ",,fuse" -PACKAGECONFIG[libvirt] = ",,libvirt" - -export CONFIG_EXAMPLE_DPDK_QAT = "${@bb.utils.contains('PACKAGECONFIG', 'dpdk_qat', 'y', 'n', d)}" -export CONFIG_EXAMPLE_VM_POWER_MANAGER = "${@bb.utils.contains('PACKAGECONFIG', 'libvirt', 'y', 'n', d)}" -export CONFIG_VHOST_ENABLED = "${@bb.utils.contains('PACKAGECONFIG', 'vhost', 'y', 'n', d)}" - -RDEPENDS_${PN} += "python-subprocess virtual/libibverbs" -DEPENDS = "virtual/kernel virtual/libibverbs" -do_configure[depends] += "virtual/kernel:do_shared_workdir" - -inherit module - -export MODULE_DIR="/lib/modules/${KERNEL_VERSION}/kernel/drivers/net" -export RTE_SDK = "${S}" -export RTE_TARGET="${@bb.utils.contains("TUNE_FEATURES", "m64", "x86_64-native-linuxapp-gcc", "i686-native-linuxapp-gcc", d)}" - -export ICP_ROOT = "${PKG_CONFIG_SYSROOT_DIR}/usr/include" -export ICP_LIB_ROOT= "${PKG_CONFIG_SYSROOT_DIR}/usr/lib" -export RTE_KERNELDIR = "${STAGING_KERNEL_DIR}" -export RTE_KERNELDIR_OUT = "${STAGING_KERNEL_BUILDDIR}" -export INSTALL_PATH = "${prefix}/share" -export RTE_OUTPUT = "${S}/${RTE_TARGET}" -export ETHTOOL_LIB_PATH = "${S}/examples/ethtool/lib/${RTE_TARGET}/" -export SYSROOTPATH = "--sysroot=${STAGING_DIR_HOST}" -export DPDK_TARGET_MACH = "${@get_dpdk_target_mach(bb,d)}" -export ICP_LAC_API_DIR = "${STAGING_DIR_TARGET}${includedir}/lac" - -# The list of intel Comms platforms and their target machine -# process mapping. The supported target machine is listed under -# dpdk/mk/machine -def get_dpdk_target_mach(bb, d): -target_arch = d.getVar('DPDK_TARGET_MACHINE', True) -if target_arch: -target_arch -return "default" - -do_configure () { - # - ### default value for prefix is "usr", unsetting it, so it - ### will not be concatenated in ${RTE_TARGET}/Makefile - ### which will cause compilation failure - # - unset prefix - - # Fix-up CONFIG_RTE_MACHINE based on target machine - sed -e "s#CONFIG_RTE_MACHINE=\"native\"#CONFIG_RTE_MACHINE=\"${DPDK_TARGET_MACH}\"#" -i ${S}/config/defconfig_x86_64-native-linuxapp-gcc - sed -e "s#CONFIG_RTE_MACHINE=\"native\"#CONFIG_RTE_MACHINE=\"${DPDK_TARGET_MACH}\"#" -i ${S}/config/defconfig_i686-native-linuxapp-gcc - - # Fix-up vhost configs
Re: [meta-intel] [meta-dpdk][PATCH 2/2] dpdk: merge recipe and include
On Wed, Mar 28, 2018 at 12:27:48PM +0500, Awais Belal wrote: > Since we only have a single version supported at the > moment, which is the latest and the greatest. It is > best that the include and base recipe files are merged > to allow better maintainability and readability. Please don't do this because I plan to add the 17.11 (LTS) version to the Yocto. Thanks, Kevin > > Signed-off-by: Awais Belal > --- > recipes-extended/dpdk/dpdk.inc | 167 -- > recipes-extended/dpdk/dpdk_18.02.bb | 175 > ++-- > 2 files changed, 167 insertions(+), 175 deletions(-) > delete mode 100644 recipes-extended/dpdk/dpdk.inc > > diff --git a/recipes-extended/dpdk/dpdk.inc b/recipes-extended/dpdk/dpdk.inc > deleted file mode 100644 > index ce4e8bc..000 > --- a/recipes-extended/dpdk/dpdk.inc > +++ /dev/null > @@ -1,167 +0,0 @@ > -DESCRIPTION = "Intel(r) Data Plane Development Kit" > -HOMEPAGE = "http://dpdk.org"; > -LICENSE = "BSD & LGPLv2 & GPLv2" > -LIC_FILES_CHKSUM = > "file://license/gpl-2.0.txt;md5=b234ee4d69f5fce4486a80fdaf4a4263 \ > - > file://license/lgpl-2.1.txt;md5=4b54a1fd55a448865a0b32d41598759d \ > - > file://license/bsd-3-clause.txt;md5=0f00d99239d922ffd13cabef83b33444" > - > -SRC_URI = "http://fast.dpdk.org/rel/${BP}.tar.gz;name=dpdk \ > -file://dpdk-16.04-add-RTE_KERNELDIR_OUT-to-split-kernel-bu.patch \ > -file://dpdk-16.07-add-sysroot-option-within-app-makefile.patch \ > -file://dpdk-16.04-Fix-for-misleading-indentation-error.patch \ > -file://dpdk-16.07-dpdk-fix-for-parellel-make-issue.patch \ > -file://dpdk-17.02-dpdk-fix-installation-warning-and-issue.patch \ > - " > - > -# A machine needs to enable this using: > -# COMPATIBLE_MACHINE_pn-dpdk-dev-libibverbs = "" > - > -COMPATIBLE_MACHINE = "null" > -COMPATIBLE_HOST_libc-musl_class-target = "null" > - > - > -# dpdk example apps dpdk_qat and vhost have dependancy on fuse and qat. > -# fuse is in meta-filesystems and qat is not yet upstreamed. > -# So adding mechanism to explicitly disable the use of fuse and qat. > -# To enable, uncomment the below line or include in .bbappend. > -# PACKAGECONFIG ?= " dpdk_qat vhost libvirt" > - > -PACKAGECONFIG[dpdk_qat] = ",,virtual/qat" > -PACKAGECONFIG[vhost] = ",,fuse" > -PACKAGECONFIG[libvirt] = ",,libvirt" > - > -export CONFIG_EXAMPLE_DPDK_QAT = "${@bb.utils.contains('PACKAGECONFIG', > 'dpdk_qat', 'y', 'n', d)}" > -export CONFIG_EXAMPLE_VM_POWER_MANAGER = > "${@bb.utils.contains('PACKAGECONFIG', 'libvirt', 'y', 'n', d)}" > -export CONFIG_VHOST_ENABLED = "${@bb.utils.contains('PACKAGECONFIG', > 'vhost', 'y', 'n', d)}" > - > -RDEPENDS_${PN} += "python-subprocess virtual/libibverbs" > -DEPENDS = "virtual/kernel virtual/libibverbs" > -do_configure[depends] += "virtual/kernel:do_shared_workdir" > - > -inherit module > - > -export MODULE_DIR="/lib/modules/${KERNEL_VERSION}/kernel/drivers/net" > -export RTE_SDK = "${S}" > -export RTE_TARGET="${@bb.utils.contains("TUNE_FEATURES", "m64", > "x86_64-native-linuxapp-gcc", "i686-native-linuxapp-gcc", d)}" > - > -export ICP_ROOT = "${PKG_CONFIG_SYSROOT_DIR}/usr/include" > -export ICP_LIB_ROOT= "${PKG_CONFIG_SYSROOT_DIR}/usr/lib" > -export RTE_KERNELDIR = "${STAGING_KERNEL_DIR}" > -export RTE_KERNELDIR_OUT = "${STAGING_KERNEL_BUILDDIR}" > -export INSTALL_PATH = "${prefix}/share" > -export RTE_OUTPUT = "${S}/${RTE_TARGET}" > -export ETHTOOL_LIB_PATH = "${S}/examples/ethtool/lib/${RTE_TARGET}/" > -export SYSROOTPATH = "--sysroot=${STAGING_DIR_HOST}" > -export DPDK_TARGET_MACH = "${@get_dpdk_target_mach(bb,d)}" > -export ICP_LAC_API_DIR = "${STAGING_DIR_TARGET}${includedir}/lac" > - > -# The list of intel Comms platforms and their target machine > -# process mapping. The supported target machine is listed under > -# dpdk/mk/machine > -def get_dpdk_target_mach(bb, d): > -target_arch = d.getVar('DPDK_TARGET_MACHINE', True) > -if target_arch: > -target_arch > -return "default" > - > -do_configure () { > - # > - ### default value for prefix is "usr", unsetting it, so it > - ### will not be concatenated in ${RTE_TARGET}/Makefile > - ### which will cause compilation failure > - # > - unset prefix > - > - # Fix-up CONFIG_RTE_MACHINE based on target machine > - sed -e > "s#CONFIG_RTE_MACHINE=\"native\"#CONFIG_RTE_MACHINE=\"${DPDK_TARGET_MACH}\"#" > -i ${S}/config/defconfig_x86_64-native-linuxapp-gcc > - sed -e > "s#CONFIG_RTE_MACHINE=\"native\"#CONFIG_RTE_MACHINE=\"${DPDK_TARGET_MACH}\"#" > -i ${S}/config/defconfig_i686-native-linuxapp-gcc > - > - # Fix-up vhost configs based on package config > - sed -e > "s#CONFIG_RTE_KNI_VHOST=n#CONFIG_RTE_KNI_VHOST=${CONFIG_VHOST_ENABLED}#" -i > ${S}/config/common_linuxapp > - sed -e > "s#CONFI
[meta-intel] [meta-dpdk][PATCH 2/2] dpdk: merge recipe and include
Since we only have a single version supported at the moment, which is the latest and the greatest. It is best that the include and base recipe files are merged to allow better maintainability and readability. Signed-off-by: Awais Belal --- recipes-extended/dpdk/dpdk.inc | 167 -- recipes-extended/dpdk/dpdk_18.02.bb | 175 ++-- 2 files changed, 167 insertions(+), 175 deletions(-) delete mode 100644 recipes-extended/dpdk/dpdk.inc diff --git a/recipes-extended/dpdk/dpdk.inc b/recipes-extended/dpdk/dpdk.inc deleted file mode 100644 index ce4e8bc..000 --- a/recipes-extended/dpdk/dpdk.inc +++ /dev/null @@ -1,167 +0,0 @@ -DESCRIPTION = "Intel(r) Data Plane Development Kit" -HOMEPAGE = "http://dpdk.org"; -LICENSE = "BSD & LGPLv2 & GPLv2" -LIC_FILES_CHKSUM = "file://license/gpl-2.0.txt;md5=b234ee4d69f5fce4486a80fdaf4a4263 \ - file://license/lgpl-2.1.txt;md5=4b54a1fd55a448865a0b32d41598759d \ - file://license/bsd-3-clause.txt;md5=0f00d99239d922ffd13cabef83b33444" - -SRC_URI = "http://fast.dpdk.org/rel/${BP}.tar.gz;name=dpdk \ - file://dpdk-16.04-add-RTE_KERNELDIR_OUT-to-split-kernel-bu.patch \ - file://dpdk-16.07-add-sysroot-option-within-app-makefile.patch \ - file://dpdk-16.04-Fix-for-misleading-indentation-error.patch \ - file://dpdk-16.07-dpdk-fix-for-parellel-make-issue.patch \ - file://dpdk-17.02-dpdk-fix-installation-warning-and-issue.patch \ - " - -# A machine needs to enable this using: -# COMPATIBLE_MACHINE_pn-dpdk-dev-libibverbs = "" - -COMPATIBLE_MACHINE = "null" -COMPATIBLE_HOST_libc-musl_class-target = "null" - - -# dpdk example apps dpdk_qat and vhost have dependancy on fuse and qat. -# fuse is in meta-filesystems and qat is not yet upstreamed. -# So adding mechanism to explicitly disable the use of fuse and qat. -# To enable, uncomment the below line or include in .bbappend. -# PACKAGECONFIG ?= " dpdk_qat vhost libvirt" - -PACKAGECONFIG[dpdk_qat] = ",,virtual/qat" -PACKAGECONFIG[vhost] = ",,fuse" -PACKAGECONFIG[libvirt] = ",,libvirt" - -export CONFIG_EXAMPLE_DPDK_QAT = "${@bb.utils.contains('PACKAGECONFIG', 'dpdk_qat', 'y', 'n', d)}" -export CONFIG_EXAMPLE_VM_POWER_MANAGER = "${@bb.utils.contains('PACKAGECONFIG', 'libvirt', 'y', 'n', d)}" -export CONFIG_VHOST_ENABLED = "${@bb.utils.contains('PACKAGECONFIG', 'vhost', 'y', 'n', d)}" - -RDEPENDS_${PN} += "python-subprocess virtual/libibverbs" -DEPENDS = "virtual/kernel virtual/libibverbs" -do_configure[depends] += "virtual/kernel:do_shared_workdir" - -inherit module - -export MODULE_DIR="/lib/modules/${KERNEL_VERSION}/kernel/drivers/net" -export RTE_SDK = "${S}" -export RTE_TARGET="${@bb.utils.contains("TUNE_FEATURES", "m64", "x86_64-native-linuxapp-gcc", "i686-native-linuxapp-gcc", d)}" - -export ICP_ROOT = "${PKG_CONFIG_SYSROOT_DIR}/usr/include" -export ICP_LIB_ROOT= "${PKG_CONFIG_SYSROOT_DIR}/usr/lib" -export RTE_KERNELDIR = "${STAGING_KERNEL_DIR}" -export RTE_KERNELDIR_OUT = "${STAGING_KERNEL_BUILDDIR}" -export INSTALL_PATH = "${prefix}/share" -export RTE_OUTPUT = "${S}/${RTE_TARGET}" -export ETHTOOL_LIB_PATH = "${S}/examples/ethtool/lib/${RTE_TARGET}/" -export SYSROOTPATH = "--sysroot=${STAGING_DIR_HOST}" -export DPDK_TARGET_MACH = "${@get_dpdk_target_mach(bb,d)}" -export ICP_LAC_API_DIR = "${STAGING_DIR_TARGET}${includedir}/lac" - -# The list of intel Comms platforms and their target machine -# process mapping. The supported target machine is listed under -# dpdk/mk/machine -def get_dpdk_target_mach(bb, d): -target_arch = d.getVar('DPDK_TARGET_MACHINE', True) -if target_arch: -target_arch -return "default" - -do_configure () { - # - ### default value for prefix is "usr", unsetting it, so it - ### will not be concatenated in ${RTE_TARGET}/Makefile - ### which will cause compilation failure - # - unset prefix - - # Fix-up CONFIG_RTE_MACHINE based on target machine - sed -e "s#CONFIG_RTE_MACHINE=\"native\"#CONFIG_RTE_MACHINE=\"${DPDK_TARGET_MACH}\"#" -i ${S}/config/defconfig_x86_64-native-linuxapp-gcc - sed -e "s#CONFIG_RTE_MACHINE=\"native\"#CONFIG_RTE_MACHINE=\"${DPDK_TARGET_MACH}\"#" -i ${S}/config/defconfig_i686-native-linuxapp-gcc - - # Fix-up vhost configs based on package config - sed -e "s#CONFIG_RTE_KNI_VHOST=n#CONFIG_RTE_KNI_VHOST=${CONFIG_VHOST_ENABLED}#" -i ${S}/config/common_linuxapp - sed -e "s#CONFIG_RTE_KNI_VHOST_VNET_HDR_EN=n#CONFIG_RTE_KNI_VHOST_VNET_HDR_EN=${CONFIG_VHOST_ENABLED}#" -i ${S}/config/common_linuxapp - sed -e "s#CONFIG_RTE_LIBRTE_VHOST=n#CONFIG_RTE_LIBRTE_VHOST=${CONFIG_VHOST_ENABLED}#" -i ${S}/config/common_linuxapp - - make O=$RTE_TARGET T=$RTE_TARGET config -} - -do_compile () { - unset LDFLAGS TARGET_LDF