[dpdk-dev] [PATCH v2] mempool: reduce rte_mempool structure size
Panu Matilainen
pmatilai at redhat.com
Fri Feb 12 12:52:53 CET 2016
On 02/10/2016 08:02 PM, Thomas Monjalon wrote:
> 2016-02-10 18:01, Wiles, Keith:
>>>>> --- a/config/defconfig_x86_64-native-linuxapp-gcc
>>>>> +++ b/config/defconfig_x86_64-native-linuxapp-gcc
>>>>> @@ -40,3 +40,8 @@ CONFIG_RTE_ARCH_64=y
>>>>>
>>>>> CONFIG_RTE_TOOLCHAIN="gcc"
>>>>> CONFIG_RTE_TOOLCHAIN_GCC=y
>>>>> +CONFIG_RTE_BUILD_SHARED_LIB=y
>>>>> +CONFIG_RTE_NEXT_ABI=n
>>>>> +CONFIG_RTE_EAL_IGB_UIO=n
>>>>> +CONFIG_RTE_LIBRTE_KNI=n
>>>>> +CONFIG_RTE_KNI_KMOD=n
>>>
>>> Hmm, not sure where this came from, but will remove it.
>>
>> I think this was from the ABI-Checker I ran and the tool should leave the repo in its original state.
>
> Yes you're right. The ABI checker modify the defconfig instead of modifying
> the generated .config file.
Its "by design" according to Neil (this was discussed when the abi
checker was introduced, see
http://dpdk.org/ml/archives/dev/2015-March/014636.html)
I actually agree with Neil that changing .config after make config seems
counter-intuitive compared to how this works in other projects, but that
doesn't make modifying the default templates the right thing to do in DPDK.
> Anyone for a patch?
I can add it to my TODO among a couple of other things - it should not
leave clutter like this behind no matter what. As to how exactly it
should be fixed, it could of course just change .config, or
alternatively create (and later clean up) a temporary defconfig file
with the necessary contents, which would be closer to the current
approach I guess.
- Panu -
More information about the dev
mailing list