On Tue, May 28, 2024 at 3:31 PM Pavan Nikhilesh Bhagavatula <pbhagavat...@marvell.com> wrote: > > > > > -----Original Message----- > > From: Bruce Richardson <bruce.richard...@intel.com> > > Sent: Tuesday, May 28, 2024 3:10 PM > > To: Pavan Nikhilesh Bhagavatula <pbhagavat...@marvell.com> > > Cc: Jerin Jacob <jer...@marvell.com>; Nithin Kumar Dabilpuram > > <ndabilpu...@marvell.com>; Kiran Kumar Kokkilagadda > > <kirankum...@marvell.com>; Sunil Kumar Kori <sk...@marvell.com>; Satha > > Koteswara Rao Kottidi <skotesh...@marvell.com>; Harman Kalra > > <hka...@marvell.com>; Shijith Thotton <sthot...@marvell.com>; Anatoly > > Burakov <anatoly.bura...@intel.com>; dev@dpdk.org > > Subject: Re: [EXTERNAL] Re: [PATCH v5] cnxk: disable building template files > > > > On Tue, May 28, 2024 at 09:23:12AM +0000, Pavan Nikhilesh Bhagavatula > > wrote: > > > > On Mon, May 27, 2024 at 09:04:29PM +0530, > > pbhagavat...@marvell.com > > > > wrote: > > > > > From: Pavan Nikhilesh <pbhagavat...@marvell.com> > > > > > > > > > > Disable building template files when CNXK_DIS_TMPLT_FUNC is defined > > > > > as a part of c_args. This option can be used when reworking datapath > > > > > or debugging issues to reduce Rx/Tx path compilation time. > > > > > > > > > > Example command: meson build -Dc_args='-DCNXK_DIS_TMPLT_FUNC' > > > > > -Dexamples=all --cross-file config/arm/arm64_cn10k_linux_gcc > > > > > > > > > Should this option be set in CI by default, or in test-meson-builds by > > > > default? When do we need to avoid setting this flag, vs setting it? > > > > > > > > > > Yes, we can set this option when testing builds to speed things up. We > > > would want to avoid setting the flag when we are packaging or generating > > > release binaries as it will negatively affect fastpath performance. > > > > > > I am not sure how to integrate to test-meson-builds as there is no target > > > option for cn10k/cn9k. > > > > > So, are these template files already not built when building for other > > platforms other than cn10k/cn9k? > > I rechecked they are only built when target is aarch64.
Bruce, planning to merge this patch. Please let me know if there is a difference in opinion.