[PATCH v3 18/30] baseband/acc100: enable input validation by default
Maxime Coquelin
maxime.coquelin at redhat.com
Fri Oct 21 10:06:27 CEST 2022
Hi Nicolas,
On 10/20/22 00:12, Chautru, Nicolas wrote:
> Hi Maxime,
>
>> From: Maxime Coquelin <maxime.coquelin at redhat.com>
>> On 10/12/22 04:53, Hernan Vargas wrote:
>>> Enable validation functions by default and provide a new flag
>>> RTE_LIBRTE_SKIP_VALIDATE if the user wants to run without validating
>>> input to save cycles.
>>
>> I would prefer a devarg, so that it can be enabled/disabled at runtime, instead
>> of build time. The extra if condition would minimal.
>
> Could you please review the previous discussion on https://patches.dpdk.org/project/dpdk/cover/20221012025346.204394-1-hernan.vargas@intel.com/
> Basically we would keep this build time flag and update documentation for this release. Then for next release consider devarg.
> Let us know what you think?
>
>
That's not ideal, but if you commit to do it in the next release this is
OK for me.
Thanks,
Maxime
More information about the dev
mailing list