[dpdk-dev] [PATCH] config: remove duplicate configuration information

Wiles, Keith keith.wiles at intel.com
Fri Mar 4 15:11:39 CET 2016


>
>> -----Original Message-----
>> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Thomas Monjalon
>> Sent: Thursday, March 3, 2016 6:38 PM
>> To: Wiles, Keith <keith.wiles at intel.com>
>> Cc: dev at dpdk.org
>> Subject: Re: [dpdk-dev] [PATCH] config: remove duplicate configuration
>> information
>> 
>> 2016-02-22 07:53, Keith Wiles:
>> > --- /dev/null
>> > +++ b/config/common_base
>> > +CONFIG_RTE_EAL_IGB_UIO=y
>> > +CONFIG_RTE_EAL_VFIO=y
>> 
>> These options should be disabled in the base file
>> and enabled in Linux.
>> 
>> > +CONFIG_RTE_LIBRTE_PMD_AF_PACKET=y
>> 
>> Idem, it should be disabled.
>> 
>> > +CONFIG_RTE_LIBRTE_POWER=y
>> 
>> Idem?
>> 
>> > +CONFIG_RTE_LIBRTE_KNI=y
>> 
>> Should be disabled.
>> 
>> > +CONFIG_RTE_LIBRTE_VHOST=y
>> 
>> Should be disabled.
>
>Any reason this should be disabled? It was changed to =Y in DPDK 2.1.
>
>It means updating scripts/build instructions to set =Y for OVS, no big
>deal but it might catch people out.

I do not see why it can not be disable, unless someone disagrees.
> 
>
>Kevin.
>
>> 
>> > --- a/config/common_bsdapp
>> > +++ b/config/common_bsdapp
>> > +# Compile Environment Abstraction Layer for linux, FreeBSD, OS X, ...
>> > +CONFIG_RTE_LIBRTE_EAL_BSDAPP=y
>> 
>> Please keep the original comment:
>> Compile Environment Abstraction Layer for BSD
>> 
>> > +# Compile Environment Abstraction Layer
>> 
>> Why this comment before disabling UIO and VFIO?
>> 
>> > --- a/config/common_linuxapp
>> > +++ b/config/common_linuxapp
>> > -##
>> > -## machine can define specific variables or action for a specific board
>> > -## RTE_MACHINE values are the directories in mk/machine/
>> > -##
>> > -#CONFIG_RTE_MACHINE="native"
>> > -#
>> > -##
>> > -## define the architecture we compile for.
>> > -## RTE_ARCH values are the directories in mk/arch/
>> > -##
>> > -#CONFIG_RTE_ARCH="x86_64"
>> > -#CONFIG_RTE_ARCH_X86_64=y
>> > -#CONFIG_RTE_ARCH_X86=y
>> > -#
>> > -##
>> > -## The compiler we use.
>> > -## RTE_TOOLCHAIN values are the directories in mk/toolchain/
>> > -##
>> > -#CONFIG_RTE_TOOLCHAIN="gcc"
>> > -#CONFIG_RTE_TOOLCHAIN_GCC=y
>> 
>> Maybe we should keep these comments in common_base?
>> I would remove the values and uncomment CONFIG_RTE_MACHINE, CONFIG_RTE_ARCH
>> and CONFIG_RTE_TOOLCHAIN.
>> 
>> > --- a/config/defconfig_x86_64-native-bsdapp-clang
>> > +++ b/config/defconfig_x86_64-native-bsdapp-clang
>> > @@ -37,6 +37,7 @@ CONFIG_RTE_MACHINE="native"
>> >  CONFIG_RTE_ARCH="x86_64"
>> >  CONFIG_RTE_ARCH_X86_64=y
>> >  CONFIG_RTE_ARCH_X86=y
>> > +CONFIG_RTE_ARCH_64=y
>> >
>> >  CONFIG_RTE_TOOLCHAIN="clang"
>> >  CONFIG_RTE_TOOLCHAIN_CLANG=y
>> > diff --git a/config/defconfig_x86_64-native-bsdapp-gcc
>> b/config/defconfig_x86_64-native-bsdapp-gcc
>> > index 5a6a4e8..4ea4433 100644
>> > --- a/config/defconfig_x86_64-native-bsdapp-gcc
>> > +++ b/config/defconfig_x86_64-native-bsdapp-gcc
>> > @@ -37,6 +37,7 @@ CONFIG_RTE_MACHINE="native"
>> >  CONFIG_RTE_ARCH="x86_64"
>> >  CONFIG_RTE_ARCH_X86_64=y
>> >  CONFIG_RTE_ARCH_X86=y
>> > +CONFIG_RTE_ARCH_64=y
>> 
>> It should be a totally separate patch.
>> And there are other places where it is missing.
>


Regards,
Keith






More information about the dev mailing list