From: William Tu <u9012063@gmail.com>
To: dpdk-dev <dev@dpdk.org>
Cc: Dmitry Kozliuk <dmitry.kozliuk@gmail.com>,
Ranjit Menon <ranjit.menon@intel.com>,
Pallavi Kadam <pallavi.kadam@intel.com>
Subject: Re: [dpdk-dev] [PATCH v2] net/ixgbe: Add support for Windows
Date: Wed, 22 Sep 2021 14:01:35 -0700 [thread overview]
Message-ID: <CALDO+SbWgSuPRrZv7=m9vUnFcbmezdc_aAdSXSOoFq-G0BBioA@mail.gmail.com> (raw)
In-Reply-To: <20210922205823.38073-1-u9012063@gmail.com>
I forgot to add the CC list.
On Wed, Sep 22, 2021 at 1:58 PM William Tu <u9012063@gmail.com> wrote:
>
> This patch enables building the ixgbe driver for Windows.
> It also enables its dependencies on security and cryptodev.
> I tested on AWS using Intel Corporation 82599 Ethernet
> Controller Virtual Function [8086:10ed], using dpdk-testpmd.
>
> Signed-off-by: William Tu <u9012063@gmail.com>
> ---
> Depends-on: series-18880 ("cryptodev: build on Windows")
> Depends-on: series-18943 ("security: use the net library for IP structs")
>
> 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
> ---
> drivers/net/ixgbe/base/ixgbe_hv_vf.c | 1 +
> drivers/net/ixgbe/base/ixgbe_osdep.h | 5 +++++
> drivers/net/ixgbe/base/meson.build | 7 +++++++
> drivers/net/ixgbe/ixgbe_ethdev.c | 2 +-
> drivers/net/ixgbe/ixgbe_flow.c | 2 +-
> drivers/net/ixgbe/ixgbe_tm.c | 2 +-
> drivers/net/ixgbe/meson.build | 6 ------
> 7 files changed, 16 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/net/ixgbe/base/ixgbe_hv_vf.c b/drivers/net/ixgbe/base/ixgbe_hv_vf.c
> index b7ad44ab8c..4572411d39 100644
> --- a/drivers/net/ixgbe/base/ixgbe_hv_vf.c
> +++ b/drivers/net/ixgbe/base/ixgbe_hv_vf.c
> @@ -4,6 +4,7 @@
>
> #include "ixgbe_vf.h"
> #include "ixgbe_hv_vf.h"
> +#include "ixgbe_osdep.h"
>
> /**
> * Hyper-V variant - just a stub.
> diff --git a/drivers/net/ixgbe/base/ixgbe_osdep.h b/drivers/net/ixgbe/base/ixgbe_osdep.h
> index cacf724198..2c44e7bf35 100644
> --- a/drivers/net/ixgbe/base/ixgbe_osdep.h
> +++ b/drivers/net/ixgbe/base/ixgbe_osdep.h
> @@ -16,6 +16,7 @@
> #include <rte_log.h>
> #include <rte_byteorder.h>
> #include <rte_io.h>
> +#include <rte_ip.h>
>
> #include "../ixgbe_logs.h"
> #include "../ixgbe_bypass_defines.h"
> @@ -43,12 +44,16 @@
>
> #define false 0
> #define true 1
> +#ifndef RTE_EXEC_ENV_WINDOWS
> #define min(a,b) RTE_MIN(a,b)
> +#endif
>
> #define EWARN(hw, S, args...) DEBUGOUT1(S, ##args)
>
> /* Bunch of defines for shared code bogosity */
> +#ifndef UNREFERENCED_PARAMETER
> #define UNREFERENCED_PARAMETER(_p)
> +#endif
> #define UNREFERENCED_1PARAMETER(_p)
> #define UNREFERENCED_2PARAMETER(_p, _q)
> #define UNREFERENCED_3PARAMETER(_p, _q, _r)
> diff --git a/drivers/net/ixgbe/base/meson.build b/drivers/net/ixgbe/base/meson.build
> index 22972c6b56..51a58a3183 100644
> --- a/drivers/net/ixgbe/base/meson.build
> +++ b/drivers/net/ixgbe/base/meson.build
> @@ -28,7 +28,14 @@ foreach flag: error_cflags
> endif
> endforeach
>
> +inc = []
> +inc += include_directories('../../../../lib/net')
> +inc += include_directories('../../../../lib/mbuf')
> +inc += include_directories('../../../../lib/mempool')
> +inc += include_directories('../../../../lib/ring')
> +
> base_lib = static_library('ixgbe_base', sources,
> dependencies: static_rte_eal,
> + include_directories: inc,
> c_args: c_args)
> base_objs = base_lib.extract_all_objects(recursive: true)
> diff --git a/drivers/net/ixgbe/ixgbe_ethdev.c b/drivers/net/ixgbe/ixgbe_ethdev.c
> index 47693c0c47..8b33897ca1 100644
> --- a/drivers/net/ixgbe/ixgbe_ethdev.c
> +++ b/drivers/net/ixgbe/ixgbe_ethdev.c
> @@ -10,7 +10,6 @@
> #include <unistd.h>
> #include <stdarg.h>
> #include <inttypes.h>
> -#include <netinet/in.h>
> #include <rte_string_fns.h>
> #include <rte_byteorder.h>
> #include <rte_common.h>
> @@ -46,6 +45,7 @@
> #include "ixgbe_rxtx.h"
> #include "base/ixgbe_type.h"
> #include "base/ixgbe_phy.h"
> +#include "base/ixgbe_osdep.h"
> #include "ixgbe_regs.h"
>
> /*
> diff --git a/drivers/net/ixgbe/ixgbe_flow.c b/drivers/net/ixgbe/ixgbe_flow.c
> index 511b612f7f..27322ab903 100644
> --- a/drivers/net/ixgbe/ixgbe_flow.c
> +++ b/drivers/net/ixgbe/ixgbe_flow.c
> @@ -10,7 +10,6 @@
> #include <unistd.h>
> #include <stdarg.h>
> #include <inttypes.h>
> -#include <netinet/in.h>
> #include <rte_byteorder.h>
> #include <rte_common.h>
> #include <rte_cycles.h>
> @@ -37,6 +36,7 @@
> #include "base/ixgbe_api.h"
> #include "base/ixgbe_vf.h"
> #include "base/ixgbe_common.h"
> +#include "base/ixgbe_osdep.h"
> #include "ixgbe_ethdev.h"
> #include "ixgbe_bypass.h"
> #include "ixgbe_rxtx.h"
> diff --git a/drivers/net/ixgbe/ixgbe_tm.c b/drivers/net/ixgbe/ixgbe_tm.c
> index a8407e742e..ae03ea6e9d 100644
> --- a/drivers/net/ixgbe/ixgbe_tm.c
> +++ b/drivers/net/ixgbe/ixgbe_tm.c
> @@ -665,7 +665,7 @@ ixgbe_node_add(struct rte_eth_dev *dev, uint32_t node_id,
> }
> /* check level */
> if (level_id != RTE_TM_NODE_LEVEL_ID_ANY &&
> - level_id != parent_node_type + 1) {
> + level_id != (uint32_t)parent_node_type + 1) {
> error->type = RTE_TM_ERROR_TYPE_NODE_PARAMS;
> error->message = "Wrong level";
> return -EINVAL;
> diff --git a/drivers/net/ixgbe/meson.build b/drivers/net/ixgbe/meson.build
> index 55ac91fcd1..4b18e47273 100644
> --- a/drivers/net/ixgbe/meson.build
> +++ b/drivers/net/ixgbe/meson.build
> @@ -1,12 +1,6 @@
> # SPDX-License-Identifier: BSD-3-Clause
> # Copyright(c) 2017 Intel Corporation
>
> -if is_windows
> - build = false
> - reason = 'not supported on Windows'
> - subdir_done()
> -endif
> -
> cflags += ['-DRTE_LIBRTE_IXGBE_BYPASS']
>
> subdir('base')
> --
> 2.30.2
>
prev parent reply other threads:[~2021-09-22 21:02 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-09-15 18:44 [dpdk-dev] [PATCH v1] " William Tu
2021-09-15 19:05 ` Dmitry Kozlyuk
2021-09-15 20:22 ` William Tu
2021-09-18 14:24 ` William Tu
2021-09-18 14:55 ` Dmitry Kozlyuk
2021-09-18 15:10 ` William Tu
2021-09-22 20:58 ` [dpdk-dev] [PATCH v2] " William Tu
2021-09-22 21:01 ` William Tu [this message]
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='CALDO+SbWgSuPRrZv7=m9vUnFcbmezdc_aAdSXSOoFq-G0BBioA@mail.gmail.com' \
--to=u9012063@gmail.com \
--cc=dev@dpdk.org \
--cc=dmitry.kozliuk@gmail.com \
--cc=pallavi.kadam@intel.com \
--cc=ranjit.menon@intel.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).