From: Slava Ovsiienko <viacheslavo@nvidia.com>
To: David Marchand <david.marchand@redhat.com>,
Michael Baum <michaelba@nvidia.com>
Cc: dev <dev@dpdk.org>, Matan Azrad <matan@nvidia.com>,
Raslan Darawsheh <rasland@nvidia.com>
Subject: Re: [dpdk-dev] [PATCH 0/6] net/mlx5: reduce Tx datapath compile time
Date: Tue, 6 Apr 2021 09:58:03 +0000 [thread overview]
Message-ID: <DM6PR12MB375336FD66BC72856904E04CDF769@DM6PR12MB3753.namprd12.prod.outlook.com> (raw)
In-Reply-To: <CAJFAV8w1CGFvbdrUVZTMm-GYzQWu0nD+Tko+tz+OiJ2-dugbag@mail.gmail.com>
> -----Original Message-----
> From: David Marchand <david.marchand@redhat.com>
> Sent: Tuesday, April 6, 2021 12:34
> To: Michael Baum <michaelba@nvidia.com>
> Cc: dev <dev@dpdk.org>; Matan Azrad <matan@nvidia.com>; Raslan
> Darawsheh <rasland@nvidia.com>; Slava Ovsiienko
> <viacheslavo@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@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
next prev parent reply other threads:[~2021-04-06 9:58 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-04-05 14:00 Michael Baum
2021-04-05 14:00 ` [dpdk-dev] [PATCH 1/6] net/mlx5: separate Rx function declarations to another file Michael Baum
2021-04-06 9:27 ` Slava Ovsiienko
2021-04-05 14:00 ` [dpdk-dev] [PATCH 2/6] net/mlx5: separate Rx function implementations to new file Michael Baum
2021-04-06 9:27 ` Slava Ovsiienko
2021-04-05 14:00 ` [dpdk-dev] [PATCH 3/6] net/mlx5: separate Tx function declarations to another file Michael Baum
2021-04-06 9:28 ` Slava Ovsiienko
2021-04-07 11:33 ` Raslan Darawsheh
2021-04-05 14:00 ` [dpdk-dev] [PATCH 4/6] net/mlx5: separate Tx burst template to header file Michael Baum
2021-04-06 9:28 ` Slava Ovsiienko
2021-04-05 14:00 ` [dpdk-dev] [PATCH 5/6] net/mlx5: separate Tx function implementations to new file Michael Baum
2021-04-06 9:29 ` Slava Ovsiienko
2021-04-05 14:00 ` [dpdk-dev] [PATCH 6/6] net/mlx5: separate Tx burst functions to different files Michael Baum
2021-04-06 9:30 ` Slava Ovsiienko
2021-04-06 9:33 ` [dpdk-dev] [PATCH 0/6] net/mlx5: reduce Tx datapath compile time David Marchand
2021-04-06 9:58 ` Slava Ovsiienko [this message]
2021-04-12 6:32 ` [dpdk-dev] [PATCH v2 " Michael Baum
2021-04-12 6:32 ` [dpdk-dev] [PATCH v2 1/6] net/mlx5: separate Rx function declarations to another file Michael Baum
2021-04-12 6:32 ` [dpdk-dev] [PATCH v2 2/6] net/mlx5: separate Rx function implementations to new file Michael Baum
2021-04-16 13:08 ` Ferruh Yigit
2021-04-12 6:32 ` [dpdk-dev] [PATCH v2 3/6] net/mlx5: separate Tx function declarations to another file Michael Baum
2021-04-12 6:32 ` [dpdk-dev] [PATCH v2 4/6] net/mlx5: separate Tx burst template to header file Michael Baum
2021-04-12 6:32 ` [dpdk-dev] [PATCH v2 5/6] net/mlx5: separate Tx function implementations to new file Michael Baum
2021-04-12 6:32 ` [dpdk-dev] [PATCH v2 6/6] net/mlx5: separate Tx burst functions to different files Michael Baum
2021-04-15 6:27 ` [dpdk-dev] [PATCH v2 0/6] net/mlx5: reduce Tx datapath compile time Raslan Darawsheh
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=DM6PR12MB375336FD66BC72856904E04CDF769@DM6PR12MB3753.namprd12.prod.outlook.com \
--to=viacheslavo@nvidia.com \
--cc=david.marchand@redhat.com \
--cc=dev@dpdk.org \
--cc=matan@nvidia.com \
--cc=michaelba@nvidia.com \
--cc=rasland@nvidia.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).