On Wed, May 29, 2024 at 01:31:39PM +0530, Jerin Jacob wrote: > 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.
No, no concerns as such from my end. I was just wondering what the current status was and if we needed to take any other action to speed up builds for developers. /Bruce