DPDK patches and discussions
 help / color / mirror / Atom feed
From: "Wei Hu (Xavier)" <xavier.huwei@huawei.com>
To: <dev@dpdk.org>, Aaron Conole <aconole@redhat.com>,
	Ferruh Yigit <ferruh.yigit@intel.com>, <test-report@dpdk.org>
Cc: <xavier_huwei@163.com>, <xavier.huwei@tom.com>,
	<linuxarm@huawei.com>, <forest.zhouchang@huawei.com>
Subject: Re: [dpdk-dev] [PATCH v3 00/22] add hns3 ethernet PMD driver
Date: Fri, 27 Sep 2019 14:52:58 +0800	[thread overview]
Message-ID: <1d5167d8-307a-ba20-632a-e45f945b854c@huawei.com> (raw)
In-Reply-To: <1569506528-60464-1-git-send-email-xavier.huwei@huawei.com>

Hi, Ferruh Yigit

  I sent out PATCH V3 to fix the conflict that occurs in
doc/guides/rel_notes/release_19_11.rst when performing
'git am patch(PATCH V2 series)' operation based on the latest
dpdk-next-net because of the recent change in the repository.

  The page in patches.dpdk.org indicates that CI checking
against this series failed and terminated, the content of 'S/W/F' field is '---', the page as follows:
http://patches.dpdk.org/project/dpdk/list/?series=&submitter=1405&state=*&q=&archive=&delegate=

  The information of CI building as follows, and in fact
there was non incompatible pointer type error based on
the latest repo in my local server.
http://mails.dpdk.org/archives/test-report/2019-September/099170.html

/home-local/jenkins-local/jenkins-agent/workspace/Apply-Custom-Patch-Set/dpdk/drivers/net/hns3/hns3_ethdev.c:4723:24: error: initialization from incompatible pointer type [-Werror=incompatible-pointer-types]
  .promiscuous_enable = hns3_dev_promiscuous_enable,
                        ^~~~~~~~~~~~~~~~~~~~~~~~~~~
/home-local/jenkins-local/jenkins-agent/workspace/Apply-Custom-Patch-Set/dpdk/drivers/net/hns3/hns3_ethdev.c:4723:24: note: (near initialization for ‘hns3_eth_dev_ops.promiscuous_enable’)
/home-local/jenkins-local/jenkins-agent/workspace/Apply-Custom-Patch-Set/dpdk/drivers/net/hns3/hns3_ethdev.c:4724:25: error: initialization from incompatible pointer type [-Werror=incompatible-pointer-types]
  .promiscuous_disable = hns3_dev_promiscuous_disable,
                         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
/home-local/jenkins-local/jenkins-agent/workspace/Apply-Custom-Patch-Set/dpdk/drivers/net/hns3/hns3_ethdev.c:4724:25: note: (near initialization for ‘hns3_eth_dev_ops.promiscuous_disable’)
/home-local/jenkins-local/jenkins-agent/workspace/Apply-Custom-Patch-Set/dpdk/drivers/net/hns3/hns3_ethdev.c:4725:26: error: initialization from incompatible pointer type [-Werror=incompatible-pointer-types]
  .allmulticast_enable  = hns3_dev_allmulticast_enable,
                          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
/home-local/jenkins-local/jenkins-agent/workspace/Apply-Custom-Patch-Set/dpdk/drivers/net/hns3/hns3_ethdev.c:4725:26: note: (near initialization for ‘hns3_eth_dev_ops.allmulticast_enable’)
/home-local/jenkins-local/jenkins-agent/workspace/Apply-Custom-Patch-Set/dpdk/drivers/net/hns3/hns3_ethdev.c:4726:26: error: initialization from incompatible pointer type [-Werror=incompatible-pointer-types]
  .allmulticast_disable = hns3_dev_allmulticast_disable,
                          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/home-local/jenkins-local/jenkins-agent/workspace/Apply-Custom-Patch-Set/dpdk/drivers/net/hns3/hns3_ethdev.c:4726:26: note: (near initialization for ‘hns3_eth_dev_ops.allmulticast_disable’)
/home-local/jenkins-local/jenkins-agent/workspace/Apply-Custom-Patch-Set/dpdk/drivers/net/hns3/hns3_ethdev.c:4729:24: error: initialization from incompatible pointer type [-Werror=incompatible-pointer-types]
  .stats_reset        = hns3_stats_reset,
                        ^~~~~~~~~~~~~~~~
/home-local/jenkins-local/jenkins-agent/workspace/Apply-Custom-Patch-Set/dpdk/drivers/net/hns3/hns3_ethdev.c:4729:24: note: (near initialization for ‘hns3_eth_dev_ops.stats_reset’)
/home-local/jenkins-local/jenkins-agent/workspace/Apply-Custom-Patch-Set/dpdk/drivers/net/hns3/hns3_ethdev.c:4732:24: error: initialization from incompatible pointer type [-Werror=incompatible-pointer-types]
  .xstats_reset       = hns3_dev_xstats_reset,
                        ^~~~~~~~~~~~~~~~~~~~~
/home-local/jenkins-local/jenkins-agent/workspace/Apply-Custom-Patch-Set/dpdk/drivers/net/hns3/hns3_ethdev.c:4732:24: note: (near initialization for ‘hns3_eth_dev_ops.xstats_reset’)
/home-local/jenkins-local/jenkins-agent/workspace/Apply-Custom-Patch-Set/dpdk/drivers/net/hns3/hns3_ethdev.c:4735:28: error: initialization from incompatible pointer type [-Werror=incompatible-pointer-types]
  .dev_infos_get          = hns3_dev_infos_get,
                            ^~~~~~~~~~~~~~~~~~
/home-local/jenkins-local/jenkins-agent/workspace/Apply-Custom-Patch-Set/dpdk/drivers/net/hns3/hns3_ethdev.c:4735:28: note: (near initialization for ‘hns3_eth_dev_ops.dev_infos_get’)
cc1: error: unrecognized command line option ‘-Wno-address-of-packed-member’ [-Werror]
cc1: all warnings being treated as errors
/home-local/jenkins-local/jenkins-agent/workspace/Apply-Custom-Patch-Set/dpdk/mk/internal/rte.compile-pre.mk:114: recipe for target 'hns3_ethdev.o' failed
make[6]: *** [hns3_ethdev.o] Error 1


  The detail information of the repository when making PATCH V3 on as follows: 

repo: http://dpdk.org/git/next/dpdk-next-net
branch: master

git log --oneline
3be6962 app/testpmd: fix unused variable compile error
5453153 app/testpmd: fix crash on port reset
944ee10 net/ipn3ke: setup MTU during HW init
52e3ab3 net/ice: remove Rx legacy descriptor definition
d27982e net/ice: switch to Rx flexible descriptor in AVX path
0973c28 net/ice: switch to flexible descriptor in SSE path
98ed8c1 net/ice: add protocol extraction support for per Rx queue
03ff0d1 net/ice: handle the Rx flex descriptor
2962f75 net/ice: add Rx flex descriptor definition
a03e11f net/i40e: limit the number of VF messages
644032b net/ice/base: remove unused code
c02031f net/ice/base: add switch support for IPv6 tc field
05859a5 net/ice/base: fix PTYPE bitmap
4f07dc0 net/ice/base: fix alignment
9d1c626 net/ice/base: use bitmap copy where appropriate
3efd0a4 net/ice/base: remove unnecessary error log
82c1c29 net/ice/base: fix 4 bytes alignment for PPPoE dummy packet
17db50c net/ice/base: search field vector indices for result slots
9ef53fb net/ice/base: remove unused DDP package macros
665293f net/ice/base: fix segment in remove existing RSS rule
079ca8c net/ice/base: fix the bitmap for TCP in RSS
d3e1ebd net/ice/base: add FDIR support for GTPU QFI field

  
  Is there a way to reconstruct PATCH V3 in CI? Do we
resend PATCH V3 to trigger CI reconstruct them again?
Thanks for your suggestion.

  Regards
Xavier


On 2019/9/26 22:01, Wei Hu (Xavier) wrote:
> The Hisilicon Network Subsystem is a long term evolution IP which is
> supposed to be used in Hisilicon ICT SoCs such as Kunpeng 920.
>
> This series add DPDK rte_ethdev poll mode driver for Hisilicon
> Network Subsystem 3(hns3) network engine.
>
> v2 -> v3:
> 	1. Resolve the conflict problem when performing git operation
> 	   based on the current repository. The conflict occurs in
> 	   doc/guides/rel_notes/release_19_11.rst.
>
> v1 -> v2:
> 	1. Address some comments from Jerin Jacob Kollanukkaran,
> 	   Stephen Hemminger and Ferruh Yigit.
>
> Wei Hu (Xavier) (22):
>   net/hns3: add build and doc infrastructure
>   net/hns3: add hardware registers definition
>   net/hns3: add some definitions for data structure and macro
>   net/hns3: register hns3 PMD driver and add the log interface
>     definition
>   net/hns3: add support for cmd of hns3 PMD driver
>   net/hns3: add the initialization of hns3 PMD driver
>   net/hns3: add support for MAC address related operations
>   net/hns3: add support for some misc operations
>   net/hns3: add support for link_update operation
>   net/hns3: add support for flow directory of hns3 PMD driver
>   net/hns3: add support for RSS of hns3 PMD driver
>   net/hns3: add support for flow control of hns3 PMD driver
>   net/hns3: add support for vlan of hns3 PMD driver
>   net/hns3: add support for mailbox of hns3 PMD driver
>   net/hns3: add support for hns3 VF PMD driver
>   net/hns3: add RX/TX package burst and queue related operation
>   net/hns3: add start stop configure promiscuous ops
>   net/hns3: add dump register ops for hns3 PMD driver
>   net/hns3: add abnormal interrupt process for hns3 PMD driver
>   net/hns3: add stats related ops for hns3 PMD driver
>   net/hns3: add reset related process for hns3 PMD driver
>   net/hns3: add multiple process support for hns3 PMD driver
>
>  MAINTAINERS                                  |    8 +
>  config/common_base                           |    5 +
>  config/common_linux                          |    5 +
>  config/defconfig_arm-armv7a-linuxapp-gcc     |    1 +
>  config/defconfig_i686-native-linuxapp-gcc    |    5 +
>  config/defconfig_i686-native-linuxapp-icc    |    5 +
>  config/defconfig_ppc_64-power8-linuxapp-gcc  |    1 +
>  config/defconfig_x86_64-native-linuxapp-icc  |    5 +
>  config/defconfig_x86_x32-native-linuxapp-gcc |    5 +
>  doc/guides/nics/features/hns3.ini            |   34 +
>  doc/guides/nics/features/hns3_vf.ini         |   29 +
>  doc/guides/nics/hns3.rst                     |   60 +
>  doc/guides/nics/index.rst                    |    1 +
>  doc/guides/rel_notes/release_19_11.rst       |    6 +
>  drivers/net/Makefile                         |    1 +
>  drivers/net/hns3/Makefile                    |   45 +
>  drivers/net/hns3/hns3_cmd.c                  |  565 +++
>  drivers/net/hns3/hns3_cmd.h                  |  761 ++++
>  drivers/net/hns3/hns3_dcb.c                  | 1642 +++++++++
>  drivers/net/hns3/hns3_dcb.h                  |  166 +
>  drivers/net/hns3/hns3_ethdev.c               | 4947 ++++++++++++++++++++++++++
>  drivers/net/hns3/hns3_ethdev.h               |  645 ++++
>  drivers/net/hns3/hns3_ethdev_vf.c            | 1725 +++++++++
>  drivers/net/hns3/hns3_fdir.c                 | 1059 ++++++
>  drivers/net/hns3/hns3_fdir.h                 |  205 ++
>  drivers/net/hns3/hns3_flow.c                 | 1903 ++++++++++
>  drivers/net/hns3/hns3_intr.c                 | 1166 ++++++
>  drivers/net/hns3/hns3_intr.h                 |   79 +
>  drivers/net/hns3/hns3_logs.h                 |   34 +
>  drivers/net/hns3/hns3_mbx.c                  |  353 ++
>  drivers/net/hns3/hns3_mbx.h                  |  136 +
>  drivers/net/hns3/hns3_mp.c                   |  214 ++
>  drivers/net/hns3/hns3_mp.h                   |   14 +
>  drivers/net/hns3/hns3_regs.c                 |  368 ++
>  drivers/net/hns3/hns3_regs.h                 |   99 +
>  drivers/net/hns3/hns3_rss.c                  |  584 +++
>  drivers/net/hns3/hns3_rss.h                  |  124 +
>  drivers/net/hns3/hns3_rxtx.c                 | 1676 +++++++++
>  drivers/net/hns3/hns3_rxtx.h                 |  317 ++
>  drivers/net/hns3/hns3_stats.c                |  975 +++++
>  drivers/net/hns3/hns3_stats.h                |  152 +
>  drivers/net/hns3/meson.build                 |   37 +
>  drivers/net/hns3/rte_pmd_hns3_version.map    |    3 +
>  drivers/net/meson.build                      |    1 +
>  mk/rte.app.mk                                |    1 +
>  45 files changed, 20167 insertions(+)
>  create mode 100644 doc/guides/nics/features/hns3.ini
>  create mode 100644 doc/guides/nics/features/hns3_vf.ini
>  create mode 100644 doc/guides/nics/hns3.rst
>  create mode 100644 drivers/net/hns3/Makefile
>  create mode 100644 drivers/net/hns3/hns3_cmd.c
>  create mode 100644 drivers/net/hns3/hns3_cmd.h
>  create mode 100644 drivers/net/hns3/hns3_dcb.c
>  create mode 100644 drivers/net/hns3/hns3_dcb.h
>  create mode 100644 drivers/net/hns3/hns3_ethdev.c
>  create mode 100644 drivers/net/hns3/hns3_ethdev.h
>  create mode 100644 drivers/net/hns3/hns3_ethdev_vf.c
>  create mode 100644 drivers/net/hns3/hns3_fdir.c
>  create mode 100644 drivers/net/hns3/hns3_fdir.h
>  create mode 100644 drivers/net/hns3/hns3_flow.c
>  create mode 100644 drivers/net/hns3/hns3_intr.c
>  create mode 100644 drivers/net/hns3/hns3_intr.h
>  create mode 100644 drivers/net/hns3/hns3_logs.h
>  create mode 100644 drivers/net/hns3/hns3_mbx.c
>  create mode 100644 drivers/net/hns3/hns3_mbx.h
>  create mode 100644 drivers/net/hns3/hns3_mp.c
>  create mode 100644 drivers/net/hns3/hns3_mp.h
>  create mode 100644 drivers/net/hns3/hns3_regs.c
>  create mode 100644 drivers/net/hns3/hns3_regs.h
>  create mode 100644 drivers/net/hns3/hns3_rss.c
>  create mode 100644 drivers/net/hns3/hns3_rss.h
>  create mode 100644 drivers/net/hns3/hns3_rxtx.c
>  create mode 100644 drivers/net/hns3/hns3_rxtx.h
>  create mode 100644 drivers/net/hns3/hns3_stats.c
>  create mode 100644 drivers/net/hns3/hns3_stats.h
>  create mode 100644 drivers/net/hns3/meson.build
>  create mode 100644 drivers/net/hns3/rte_pmd_hns3_version.map
>



  parent reply	other threads:[~2019-09-27  6:53 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-09-26 14:01 Wei Hu (Xavier)
2019-09-26 14:01 ` [dpdk-dev] [PATCH v3 01/22] net/hns3: add build and doc infrastructure Wei Hu (Xavier)
2019-09-26 14:01 ` [dpdk-dev] [PATCH v3 02/22] net/hns3: add hardware registers definition Wei Hu (Xavier)
2019-09-26 14:01 ` [dpdk-dev] [PATCH v3 03/22] net/hns3: add some definitions for data structure and macro Wei Hu (Xavier)
2019-09-26 14:01 ` [dpdk-dev] [PATCH v3 04/22] net/hns3: register hns3 PMD driver and add the log interface definition Wei Hu (Xavier)
2019-09-26 14:01 ` [dpdk-dev] [PATCH v3 05/22] net/hns3: add support for cmd of hns3 PMD driver Wei Hu (Xavier)
2019-09-26 14:01 ` [dpdk-dev] [PATCH v3 06/22] net/hns3: add the initialization " Wei Hu (Xavier)
2019-09-26 14:01 ` [dpdk-dev] [PATCH v3 07/22] net/hns3: add support for MAC address related operations Wei Hu (Xavier)
2019-09-26 14:01 ` [dpdk-dev] [PATCH v3 08/22] net/hns3: add support for some misc operations Wei Hu (Xavier)
2019-09-26 14:01 ` [dpdk-dev] [PATCH v3 09/22] net/hns3: add support for link_update operation Wei Hu (Xavier)
2019-09-26 14:01 ` [dpdk-dev] [PATCH v3 10/22] net/hns3: add support for flow directory of hns3 PMD driver Wei Hu (Xavier)
2019-09-26 14:01 ` [dpdk-dev] [PATCH v3 11/22] net/hns3: add support for RSS " Wei Hu (Xavier)
2019-09-26 14:01 ` [dpdk-dev] [PATCH v3 12/22] net/hns3: add support for flow control " Wei Hu (Xavier)
2019-09-26 14:01 ` [dpdk-dev] [PATCH v3 13/22] net/hns3: add support for vlan " Wei Hu (Xavier)
2019-09-26 14:02 ` [dpdk-dev] [PATCH v3 14/22] net/hns3: add support for mailbox " Wei Hu (Xavier)
2019-09-26 14:02 ` [dpdk-dev] [PATCH v3 15/22] net/hns3: add support for hns3 VF " Wei Hu (Xavier)
2019-09-26 14:02 ` [dpdk-dev] [PATCH v3 16/22] net/hns3: add RX/TX package burst and queue related operation Wei Hu (Xavier)
2019-09-26 14:02 ` [dpdk-dev] [PATCH v3 17/22] net/hns3: add start stop configure promiscuous ops Wei Hu (Xavier)
2019-09-26 14:02 ` [dpdk-dev] [PATCH v3 18/22] net/hns3: add dump register ops for hns3 PMD driver Wei Hu (Xavier)
2019-09-27 18:31   ` Ferruh Yigit
2019-09-29 12:09     ` Wei Hu (Xavier)
2019-09-26 14:02 ` [dpdk-dev] [PATCH v3 19/22] net/hns3: add abnormal interrupt process " Wei Hu (Xavier)
2019-09-26 14:02 ` [dpdk-dev] [PATCH v3 20/22] net/hns3: add stats related ops " Wei Hu (Xavier)
2019-09-26 14:02 ` [dpdk-dev] [PATCH v3 21/22] net/hns3: add reset related process " Wei Hu (Xavier)
2019-09-26 14:02 ` [dpdk-dev] [PATCH v3 22/22] net/hns3: add multiple process support " Wei Hu (Xavier)
2019-09-27  6:52 ` Wei Hu (Xavier) [this message]
2019-09-27  8:47   ` [dpdk-dev] [PATCH v3 00/22] add hns3 ethernet " Ferruh Yigit
2019-09-27 19:16     ` Jeremy Plsek
2019-09-27 18:30 ` Ferruh Yigit
2019-09-29 12:12   ` Wei Hu (Xavier)
2019-09-30  8:52     ` Ferruh Yigit
2019-10-11  1:30       ` Wei Hu (Xavier)
2019-10-10 17:10   ` Ferruh Yigit
2019-10-11  1:39     ` Wei Hu (Xavier)
2019-10-11  2:40       ` Wei Hu (Xavier)

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=1d5167d8-307a-ba20-632a-e45f945b854c@huawei.com \
    --to=xavier.huwei@huawei.com \
    --cc=aconole@redhat.com \
    --cc=dev@dpdk.org \
    --cc=ferruh.yigit@intel.com \
    --cc=forest.zhouchang@huawei.com \
    --cc=linuxarm@huawei.com \
    --cc=test-report@dpdk.org \
    --cc=xavier.huwei@tom.com \
    --cc=xavier_huwei@163.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).