DPDK patches and discussions
 help / color / mirror / Atom feed
From: Bruce Richardson <bruce.richardson@intel.com>
To: Khadem Ullah <14pwcse1224@uetpeshawar.edu.pk>
Cc: Ciara Loftus <ciara.loftus@intel.com>, <dev@dpdk.org>
Subject: Re: [PATCH 3/4] net/ice: remove SSE vector path
Date: Tue, 26 Aug 2025 08:54:30 +0100	[thread overview]
Message-ID: <aK1oNgPbHGeJdgUp@bricha3-mobl1.ger.corp.intel.com> (raw)
In-Reply-To: <CA++2-x4mGrxvu7vxRgvQLbtGkS3bMBnLt+1NLpqa9P-GQobsAA@mail.gmail.com>

On Tue, Aug 26, 2025 at 12:31:26PM +0500, Khadem Ullah wrote:
>    Hi Ciara Loftus,
>    We may need some legacy support for the SSE path for earlier systems.
>    Or it will not be a problem ?
>    As you specified that modern x86 systems all support AVX2, if not
>    AVX-512.
>    Still, we might need some type of support for the SSE path.
>    Best Regards,
>    Khadem

Hi Kadem,

can you perhaps share what systems you are using that do not have AVX2
support? Do you do new deployments with latest DPDK on those systems?
However, if it helps, we can defer this patch till after 25.11
release, so that we have one more LTS with SSE support still in it. That
would mean that we have a supported DPDK release with SSE in it until
potentially end of 2028. As it stands, even if this patch goes into the
release, 24.11 will be supported till end of 2027.

/Bruce

>    On Mon, Aug 25, 2025 at 5:36 PM Ciara Loftus
>    <[1]ciara.loftus@intel.com> wrote:
> 
>      Modern x86 systems all support AVX2, if not AVX-512, so the SSE path
>      is no longer widely used. This change will not result in any feature
>      loss, as in the rare cases where the SSE path would have been used
>      in
>      the absence of AVX2 or AVX-512, the fallback scalar path will be
>      used
>      which has feature parity with SSE.
>      Signed-off-by: Ciara Loftus <[2]ciara.loftus@intel.com>
>      ---
> 
> References
> 
>    1. mailto:ciara.loftus@intel.com
>    2. mailto:ciara.loftus@intel.com

  reply	other threads:[~2025-08-26  7:54 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-08-25 12:35 [PATCH 0/4] Remove SSE from some Intel drivers Ciara Loftus
2025-08-25 12:35 ` [PATCH 1/4] net/i40e: remove SSE vector path Ciara Loftus
2025-08-25 12:35 ` [PATCH 2/4] net/iavf: " Ciara Loftus
2025-08-25 12:35 ` [PATCH 3/4] net/ice: " Ciara Loftus
2025-08-26  7:31   ` Khadem Ullah
2025-08-26  7:54     ` Bruce Richardson [this message]
2025-08-26  9:29       ` Khadem Ullah
2025-08-25 12:35 ` [PATCH 4/4] doc: add release note entry for SSE removal Ciara Loftus
2025-08-25 13:20   ` Bruce Richardson

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=aK1oNgPbHGeJdgUp@bricha3-mobl1.ger.corp.intel.com \
    --to=bruce.richardson@intel.com \
    --cc=14pwcse1224@uetpeshawar.edu.pk \
    --cc=ciara.loftus@intel.com \
    --cc=dev@dpdk.org \
    /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).