> On May 20, 2020, at 2:53 AM, Thomas Monjalon <[email protected]> wrote: > > 20/05/2020 00:58, Dharmik Thakkar: >>> On May 19, 2020, at 2:42 AM, Thomas Monjalon <[email protected]> wrote: >>>>> Update testpmd documentation to include RECORD configuration options, >>>>> CONFIG_RTE_TEST_PMD_RECORD_CORE_CYCLES and >>>>> CONFIG_RTE_TEST_PMD_RECORD_BURST_STATS. >>>>> >>>>> Signed-off-by: Dharmik Thakkar <[email protected]> >>>>> Reviewed-by: Honnappa Nagarahalli <[email protected]> >>>>> Reviewed-by: Phil Yang <[email protected]> >>>> >>>> Acked-by: Bernard Iremonger <[email protected]> >>> >>> How these options are managed with meson? >> >> As per my understanding, currently, this is not implemented with meson. >> With ‘make’, the configuration options are saved within >> ./build/include/rte_config.h ( which gets generated during make config …). >> But this file (rte_config.h) does not get generated when using meson. > > That's also my understanding. > There is a gap which needs to be fixed, not sure what is the best approach.
IMO, for now, it is best to replicate what is being done for ‘make’. > Can it be made a runtime option? Yes, it can be made a runtime option but that will be a separate patch. >
