DPDK patches and discussions
 help / color / mirror / Atom feed
From: David Marchand <david.marchand@redhat.com>
To: Pallavi Kadam <pallavi.kadam@intel.com>
Cc: dev <dev@dpdk.org>, Thomas Monjalon <thomas@monjalon.net>,
	 Ranjit Menon <ranjit.menon@intel.com>,
	Qiming Yang <qiming.yang@intel.com>,
	 Qi Zhang <qi.z.zhang@intel.com>,
	"Yigit, Ferruh" <ferruh.yigit@intel.com>
Subject: Re: [dpdk-dev] [PATCH 2/4] net/ice: build on Windows
Date: Thu, 4 Mar 2021 13:56:05 +0100	[thread overview]
Message-ID: <CAJFAV8wu3FaEdbJmh=36bZi_r=SQd+ZkuMbAKFOD2ewVi3cNaw@mail.gmail.com> (raw)
In-Reply-To: <20210303215929.5616-3-pallavi.kadam@intel.com>

On Wed, Mar 3, 2021 at 10:27 PM Pallavi Kadam <pallavi.kadam@intel.com> wrote:
>
> Add Intel ice PMD support on Windows.
> Remove #include sys/ioctl header file as it is not needed.
> Replace x86intrin.h with rte_vect.h to avoid __m_prefetchw conflicting
> types.
> Replace POSIX usleep() API with rte API.
> Add a new macro for the access() API as the original function
> has been deprecated on Windows.
> Add extra cflags '-fno-asynchronous-unwind-tables'
> to avoid MinGW build error:
>         Error: invalid register for .seh_savexmm

Fyi, caught at the UNH lab:

http://mails.dpdk.org/archives/test-report/2021-March/180813.html

[205/277] Compiling C object drivers/net/ice/base/a42f040@@ice_base at
sta/ice_acl_ctrl.c.obj.
[206/277] Compiling C object drivers/net/ice/base/a42f040@@ice_base at
sta/ice_flow.c.obj.
FAILED: drivers/net/ice/base/a42f040@@ice_base at sta/ice_flow.c.obj
gcc @drivers/net/ice/base/a42f040@@ice_base at sta/ice_flow.c.obj.rsp
{standard input}: Assembler messages:
{standard input}:1677: Error: invalid register for .seh_savexmm
{standard input}:3985: Error: invalid register for .seh_savexmm
[207/277] Linking static target drivers/libtmp_rte_net_i40e.a.
[208/277] Compiling C object drivers/net/ice/base/a42f040@@ice_base at
sta/ice_sched.c.obj.
[209/277] Compiling C object drivers/net/ice/base/a42f040@@ice_base at
sta/ice_dcb.c.obj.
[210/277] Compiling C object drivers/net/ice/base/a42f040@@ice_base at
sta/ice_flex_pipe.c.obj.
[211/277] Compiling C object drivers/a715181@@tmp_rte_net_ice at
sta/net_ice_ice_ethdev.c.obj.
ninja: build stopped: subcommand failed.

I suppose this is because "-fno-asynchronous-unwind-tables" is not
passed to the base driver cflags.
Bonus question: should we make this workaround global for mingw compilation?


-- 
David Marchand


  reply	other threads:[~2021-03-04 12:56 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-03-03 21:59 [dpdk-dev] [PATCH 0/4] Support ice PMD " Pallavi Kadam
2021-03-03 21:59 ` [dpdk-dev] [PATCH 1/4] build: enable iavf driver " Pallavi Kadam
2021-03-03 21:59 ` [dpdk-dev] [PATCH 2/4] net/ice: build " Pallavi Kadam
2021-03-04 12:56   ` David Marchand [this message]
2021-03-08 23:30     ` Kadam, Pallavi
2021-03-03 21:59 ` [dpdk-dev] [PATCH 3/4] net/ice: disable ice DDP package " Pallavi Kadam
2021-03-03 23:12   ` Dmitry Kozlyuk
2021-03-05  0:31     ` Kadam, Pallavi
2021-03-03 21:59 ` [dpdk-dev] [PATCH 4/4] doc: update ice PMD and 21.05 release notes Pallavi Kadam
2021-03-03 23:13   ` Dmitry Kozlyuk
2021-03-05  0:25     ` Kadam, Pallavi
2021-03-11  1:58 ` [dpdk-dev] [PATCH v2 0/3] Support ice PMD on Windows Pallavi Kadam
2021-03-11  1:58   ` [dpdk-dev] [PATCH v2 1/3] build: enable iavf base code to build on windows Pallavi Kadam
2021-03-25 16:41     ` Jie Zhou
2021-03-26 11:41     ` Ferruh Yigit
2021-03-28  2:05       ` Kadam, Pallavi
2021-03-11  1:58   ` [dpdk-dev] [PATCH v2 2/3] net/ice: build on Windows Pallavi Kadam
2021-03-25 16:48     ` Jie Zhou
2021-03-26 11:51     ` Ferruh Yigit
2021-03-28  2:00       ` Kadam, Pallavi
2021-03-26 11:58     ` Ferruh Yigit
2021-03-28  2:04       ` Kadam, Pallavi
2021-03-11  1:58   ` [dpdk-dev] [PATCH v2 3/3] net/ice: disable ice DDP package " Pallavi Kadam
2021-03-25 16:55     ` Jie Zhou
2021-03-26 11:53     ` Ferruh Yigit
2021-03-28  2:02       ` Kadam, Pallavi
2021-04-02  1:26   ` [dpdk-dev] [PATCH v3 0/3] Support ice PMD " Pallavi Kadam
2021-04-02  1:26     ` [dpdk-dev] [PATCH v3 1/3] build: enable iavf base code to build on windows Pallavi Kadam
2021-04-02  1:26     ` [dpdk-dev] [PATCH v3 2/3] net/ice: build on Windows Pallavi Kadam
2021-04-02  1:26     ` [dpdk-dev] [PATCH v3 3/3] net/ice: disable ice DDP package " Pallavi Kadam
2021-04-06 16:40     ` [dpdk-dev] [PATCH v3 0/3] Support ice PMD " Ferruh Yigit
2021-04-06 17:22       ` Ferruh Yigit

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='CAJFAV8wu3FaEdbJmh=36bZi_r=SQd+ZkuMbAKFOD2ewVi3cNaw@mail.gmail.com' \
    --to=david.marchand@redhat.com \
    --cc=dev@dpdk.org \
    --cc=ferruh.yigit@intel.com \
    --cc=pallavi.kadam@intel.com \
    --cc=qi.z.zhang@intel.com \
    --cc=qiming.yang@intel.com \
    --cc=ranjit.menon@intel.com \
    --cc=thomas@monjalon.net \
    /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).