[dpdk-dev] [PATCH 0/6] net/mlx5: reduce Tx datapath compile time

[Slava Ovsiienko]

> -----Original Message-----
> From: David Marchand <david.marchand at redhat.com>
> Sent: Tuesday, April 6, 2021 12:34
> To: Michael Baum <michaelba at nvidia.com>
> Cc: dev <dev at dpdk.org>; Matan Azrad <matan at nvidia.com>; Raslan
> Darawsheh <rasland at nvidia.com>; Slava Ovsiienko
> <viacheslavo at nvidia.com>
> Subject: Re: [dpdk-dev] [PATCH 0/6] net/mlx5: reduce Tx datapath compile
> time
> 
> On Mon, Apr 5, 2021 at 4:01 PM Michael Baum <michaelba at nvidia.com>
> wrote:
> >
> > The mlx5_rxtx.c file contains a lot of Tx burst functions, each of those is
> performance-optimized for the specific set of requested offloads.
> > These ones are generated on the basis of the template function and it
> takes significant time to compile, just due to a large number of giant
> functions generated in the same file and this compilation is not being done in
> parallel with using multithreading.
> >
> > Therefore, in this series we split the mlx5_rxtx.c file into several separate
> files to allow different functions to be compiled simultaneously.
> >
> > Michael Baum (6):
> >   net/mlx5: separate Rx function declarations to another file
> >   net/mlx5: separate Rx function implementations to new file
> >   net/mlx5: separate Tx function declarations to another file
> >   net/mlx5: separate Tx burst template to header file
> >   net/mlx5: separate Tx function implementations to new file
> >   net/mlx5: separate Tx burst functions to different files
> 
> Please fix the indentation in meson.build updates.
> Out of curiosity, what is the gain in compilation time?
Om my development setup, with this patch the partial compilation
(for example if we changed the mlx5.h header) takes just several seconds,
before one (especially with full debug option)  - up to 1 minute - all this
this time was eaten by mlx5_rxtx.c.

Also this patch is the preparation step for mlx5 Rx datapath consolidation,
we aare thing about implementing the rx_burst template routine as we
done for the tx_burst, so - it would be nice to resolve the compilation time
issue at early stage.

With best regards,
Slava


> 
> 
> --
> David Marchand