DPDK patches and discussions
 help / color / mirror / Atom feed
From: Stephen Hemminger <stephen@networkplumber.org>
To: Junlong Wang <wang.junlong1@zte.com.cn>
Cc: yang.yonggang@zte.com.cn, dev@dpdk.org,
	Bingbin Chen <chen.bingbin@zte.com.cn>
Subject: Re: [PATCH v4 2/2] net/zxdh: add support flow director ops
Date: Fri, 8 Aug 2025 09:12:22 -0700	[thread overview]
Message-ID: <20250808091222.1cc721a8@hermes.local> (raw)
In-Reply-To: <20250808071048.1906724-3-wang.junlong1@zte.com.cn>

On Fri,  8 Aug 2025 15:10:37 +0800
Junlong Wang <wang.junlong1@zte.com.cn> wrote:

> From: Bingbin Chen <chen.bingbin@zte.com.cn>
> 
> Provide support for ETH, VLAN, IPv4/IPv6, TCP/UDP, VXLAN,
> and mask matching, supporting multiple actions
> include drop/count/mark/queue/rss,and vxlan decap/encap.
> 
> Signed-off-by: Bingbin Chen <chen.bingbin@zte.com.cn>
> ---
>  doc/guides/nics/features/zxdh.ini  |   16 +
>  doc/guides/nics/zxdh.rst           |    1 +
>  drivers/net/zxdh/meson.build       |    1 +
>  drivers/net/zxdh/zxdh_common.h     |    1 +
>  drivers/net/zxdh/zxdh_ethdev.c     |   26 +
>  drivers/net/zxdh/zxdh_ethdev.h     |   12 +-
>  drivers/net/zxdh/zxdh_ethdev_ops.c |    2 +-
>  drivers/net/zxdh/zxdh_ethdev_ops.h |    1 +
>  drivers/net/zxdh/zxdh_flow.c       | 2000 ++++++++++++++++++++++++++++
>  drivers/net/zxdh/zxdh_flow.h       |  237 ++++
>  drivers/net/zxdh/zxdh_msg.c        |  264 +++-
>  drivers/net/zxdh/zxdh_msg.h        |   31 +-
>  drivers/net/zxdh/zxdh_tables.h     |   10 +-
>  13 files changed, 2529 insertions(+), 73 deletions(-)
>  create mode 100644 drivers/net/zxdh/zxdh_flow.c
>  create mode 100644 drivers/net/zxdh/zxdh_flow.h

Missing include of stdlib.h

==== 20 line log output for CentOS Stream 10 (dpdk_meson_compile): ====
../drivers/net/zxdh/zxdh_flow.c:664:25: error: implicit declaration of function 'free' [-Wimplicit-function-declaration]
664 |                         free(result);
|                         ^~~~
../drivers/net/zxdh/zxdh_flow.c:664:25: note: include '<stdlib.h>' or provide a declaration of 'free'
../drivers/net/zxdh/zxdh_flow.c:664:25: error: incompatible implicit declaration of built-in function 'free' [-Werror=builtin-declaration-mismatch]
../drivers/net/zxdh/zxdh_flow.c:664:25: note: include '<stdlib.h>' or provide a declaration of 'free'
../drivers/net/zxdh/zxdh_flow.c:669:33: error: incompatible implicit declaration of built-in function 'calloc' [-Werror=builtin-declaration-mismatch]
669 |                         entry = calloc(1, sizeof(struct rte_flow));
|                                 ^~~~~~
../drivers/net/zxdh/zxdh_flow.c:669:33: note: include '<stdlib.h>' or provide a declaration of 'calloc'
../drivers/net/zxdh/zxdh_flow.c:677:25: error: incompatible implicit declaration of built-in function 'free' [-Werror=builtin-declaration-mismatch]
677 |                         free(entry_flow);
|                         ^~~~
../drivers/net/zxdh/zxdh_flow.c:677:25: note: include '<stdlib.h>' or provide a declaration of 'free'
../drivers/net/zxdh/zxdh_flow.c:685:9: error: incompatible implicit declaration of built-in function 'free' [-Werror=builtin-declaration-mismatch]
685 |         free(result);
|         ^~~~
../drivers/net/zxdh/zxdh_flow.c:685:9: note: include '<stdlib.h>' or provide a declaration of 'free'
cc1: all warnings being treated as errors
ninja: build stopped: subcommand failed.

      parent reply	other threads:[~2025-08-08 16:12 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-06-17  9:31 [PATCH v1 0/2] " Bingbin Chen
2025-06-17  9:32 ` [PATCH v1 1/2] net/zxdh: npsdk add flow director table ops Bingbin Chen
2025-06-17 14:07   ` Stephen Hemminger
2025-06-17 14:08   ` Stephen Hemminger
2025-06-17  9:32 ` [PATCH v1 2/2] net/zxdh: add support flow director ops Bingbin Chen
2025-06-18  7:49 ` [PATCH v2 0/2] " Bingbin Chen
2025-06-18  7:49   ` [PATCH v2 1/2] net/zxdh: npsdk add flow director table ops Bingbin Chen
2025-06-18  7:49   ` [PATCH v2 2/2] net/zxdh: add support flow director ops Bingbin Chen
2025-06-30 16:56     ` Stephen Hemminger
2025-07-02  7:34   ` [PATCH v3 0/2] " Bingbin Chen
2025-07-02  7:34     ` [PATCH v3 1/2] net/zxdh: npsdk add flow director table ops Bingbin Chen
2025-07-02  7:34     ` [PATCH v3 2/2] net/zxdh: add support flow director ops Bingbin Chen
2025-07-02 15:02       ` Stephen Hemminger
2025-08-03 17:34       ` Stephen Hemminger
2025-08-08  7:10     ` [PATCH v4 0/2] " Junlong Wang
2025-08-08  7:10       ` [PATCH v4 1/2] net/zxdh: npsdk add flow director table ops Junlong Wang
2025-08-08  7:10       ` [PATCH v4 2/2] net/zxdh: add support flow director ops Junlong Wang
2025-08-08  9:15         ` Ivan Malov
2025-08-08 16:12         ` Stephen Hemminger [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=20250808091222.1cc721a8@hermes.local \
    --to=stephen@networkplumber.org \
    --cc=chen.bingbin@zte.com.cn \
    --cc=dev@dpdk.org \
    --cc=wang.junlong1@zte.com.cn \
    --cc=yang.yonggang@zte.com.cn \
    /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).