DPDK patches and discussions
 help / color / mirror / Atom feed
From: William Tu <u9012063@gmail.com>
To: dev@dpdk.org
Cc: Dmitry.Kozliuk@gmail.com, david.marchand@redhat.com
Subject: [dpdk-dev] [PATCH v4 0/4] net/ixgbe: build on Windows
Date: Wed,  6 Oct 2021 22:35:40 +0000	[thread overview]
Message-ID: <20211006223544.1882-1-u9012063@gmail.com> (raw)
In-Reply-To: <20211005170252.7-1-u9012063@gmail.com>

This patch enables building the ixgbe driver for Windows.
It also enables its dependencies on security and cryptodev.
I tested on AWS using ixgbe VF device, using dpdk-testpmd.

Tal Shnaiderman (3):
  security: use the net library for IP structs
  security: build on Windows
  cryptodev: build on Windows

William Tu (1):
  net/ixgbe: build on Windows
---
v5:
* fix error due to "invalid register for .seh_savexmm"
  However, I'm not able to reproduce the error on my mingw.
  I will check the DPDK CI...
* add release notes and documentation suggested by Pallavi

v4:
* fix mingw build by defining the IPPROTO_SCTP if needed
* simplify the meson file
* change patch title, rebase to main

v3:
* I include Tal's patch series due to some fixes.
* apply on master, after commit f1f6ebc0eaf6
  ("eal: remove sys/queue.h from public headers")
  ixgbe needs to add "include <sys/queue.h>"
* For "cryptodev: build on Windows",
  need to add "include <sys/queue.h>"
* For "security: build on Windows",
  remove unresolved external symbols, reported
  by Pallavi at:
  http://inbox.dpdk.org/dev/CALDO+SZ-iL4jhcSZPZZKkbY+dTa1OP+dGKZs86iRE6b2pUsvBw@mail.gmail.com/T/#m0160fb800fe8e8d83624f3bdb39a01b7ef9b5f35 

v2:
* add dependencies on cryptodev and security
* add #include <rte_ip.h>, instead of doing
    #ifndef RTE_EXEC_ENV_WINDOWS
    #include <netinet/in.h>
    #else
    #include <Ws2tcpip.h>
    #endif
  however, including rte_ip.h requires including another
  set of dependent headers, see the
    drivers/net/ixgbe/base/meson.build

 doc/guides/nics/features/ixgbe.ini     |  1 +
 doc/guides/nics/features/ixgbe_vf.ini  |  1 +
 doc/guides/nics/ixgbe.rst              | 19 +++++++++++++++++--
 doc/guides/rel_notes/release_21_11.rst |  4 ++++
 drivers/net/ixgbe/base/ixgbe_hv_vf.c   |  1 +
 drivers/net/ixgbe/base/ixgbe_osdep.h   |  9 +++++++++
 drivers/net/ixgbe/base/meson.build     |  2 +-
 drivers/net/ixgbe/ixgbe_ethdev.c       |  2 +-
 drivers/net/ixgbe/ixgbe_ethdev.h       |  1 +
 drivers/net/ixgbe/ixgbe_flow.c         |  2 +-
 drivers/net/ixgbe/ixgbe_tm.c           |  2 +-
 drivers/net/ixgbe/meson.build          |  9 +++------
 lib/cryptodev/cryptodev_pmd.c          |  2 ++
 lib/cryptodev/rte_cryptodev.c          |  1 -
 lib/cryptodev/version.map              |  2 --
 lib/meson.build                        |  2 ++
 lib/net/rte_ip.h                       |  1 +
 lib/security/meson.build               |  2 +-
 lib/security/rte_security.h            |  5 +----
 lib/security/version.map               |  3 ---
 20 files changed, 48 insertions(+), 23 deletions(-)

-- 
2.33.0.windows.2


  parent reply	other threads:[~2021-10-06 22:36 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-10-02  0:33 [dpdk-dev] [PATCH v3 0/4] net/ixgbe: Add support for Windows William Tu
2021-10-02  0:33 ` [dpdk-dev] [PATCH v3 1/4] security: use the net library for IP structs William Tu
2021-10-05 10:52   ` [dpdk-dev] [EXT] " Akhil Goyal
2021-10-02  0:33 ` [dpdk-dev] [PATCH v3 2/4] security: build on Windows William Tu
2021-10-05 10:53   ` [dpdk-dev] [EXT] " Akhil Goyal
2021-10-02  0:33 ` [dpdk-dev] [PATCH v3 3/4] cryptodev: " William Tu
2021-10-05 10:53   ` [dpdk-dev] [EXT] " Akhil Goyal
2021-10-02  0:33 ` [dpdk-dev] [PATCH v3 4/4] net/ixgbe: Add support for Windows William Tu
2021-10-01 19:03   ` Dmitry Kozlyuk
2021-10-05 17:02 ` [dpdk-dev] [PATCH v4 0/4] net/ixgbe: build on Windows William Tu
2021-10-05 17:02   ` [dpdk-dev] [PATCH v4 1/4] security: use the net library for IP structs William Tu
2021-10-05 17:02   ` [dpdk-dev] [PATCH v4 2/4] security: build on Windows William Tu
2021-10-05 17:02   ` [dpdk-dev] [PATCH v4 3/4] cryptodev: " William Tu
2021-10-05 17:02   ` [dpdk-dev] [PATCH v4 4/4] net/ixgbe: " William Tu
2021-10-05 22:52     ` Dmitry Kozlyuk
2021-10-06  5:32     ` Kadam, Pallavi
2021-10-06 21:26       ` William Tu
2021-10-06 12:09   ` [dpdk-dev] [PATCH v4 0/4] " David Marchand
2021-10-06 21:48     ` William Tu
2021-10-06 22:35   ` William Tu [this message]
2021-10-06 22:35     ` [dpdk-dev] [PATCH v5 1/4] security: use the net library for IP structs William Tu
2021-10-06 22:35     ` [dpdk-dev] [PATCH v5 2/4] security: build on Windows William Tu
2021-10-07  7:44       ` Thomas Monjalon
2021-10-06 22:35     ` [dpdk-dev] [PATCH v5 3/4] cryptodev: " William Tu
2021-10-06 22:35     ` [dpdk-dev] [PATCH v5 4/4] net/ixgbe: " William Tu
2021-10-07  7:51       ` Thomas Monjalon
2021-10-07 11:53     ` [dpdk-dev] [PATCH v4 0/4] " Thomas Monjalon

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=20211006223544.1882-1-u9012063@gmail.com \
    --to=u9012063@gmail.com \
    --cc=Dmitry.Kozliuk@gmail.com \
    --cc=david.marchand@redhat.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).