From: Andrew Rybchenko <arybchenko@solarflare.com>
To: <dev@dpdk.org>
Cc: Thomas Monjalon <thomas@monjalon.net>,
David Marchand <david.marchand@redhat.com>,
Ferruh Yigit <ferruh.yigit@intel.com>
Subject: [dpdk-dev] [PATCH 00/14] net/sfc: factor out common driver library
Date: Tue, 8 Sep 2020 10:14:20 +0100 [thread overview]
Message-ID: <1599556475-27820-1-git-send-email-arybchenko@solarflare.com> (raw)
Network and vDPA drivers share libefx. So, libefx should be moved
to common drivers.
DPDK adaptation of the MCDI interface may be shared as well.
The new common driver name is sfc_efx since it a new home of
libefx base driver. sfc_ prefix is used to make it clear that
it is related to net/sfc (and vdpa/sfc in the future).
In theory, right now all exported libefx functions should be marked
as internal, but it requires corresponding markup in base driver
which sources should have no DPDK specifics since shared by many
drivers. So, it is unclear what to do and how to solve it.
The patch series makes these functions a part of DPDK_21 ABI to
pass build checks. checkpatches.sh does not like it.
Cc: Thomas Monjalon <thomas@monjalon.net>
Cc: David Marchand <david.marchand@redhat.com>
Cc: Ferruh Yigit <ferruh.yigit@intel.com>
Andrew Rybchenko (14):
net/sfc: include header with debug helpers directly
net/sfc: introduce common driver library
net/sfc: add dedicated header file with MCDI interface
net/sfc: move MCDI helper interface to dedicated namespace
net/sfc: make MCDI logging helper macros local
net/sfc: start to make MCDI helpers interface shareable
net/sfc: use own logging helper macros
net/sfc: avoid usage of NIC pointer from adapter context
net/sfc: avoid panic in the case of MCDI timeout
net/sfc: add MCDI callbacks to allocate/free DMA memory
net/sfc: add MCDI callback to schedule restart
net/sfc: add MCDI callback to poll management event queue
net/sfc: use MCDI control structure as libefx ops context
net/sfc: move MCDI helpers to common driver
MAINTAINERS | 1 +
drivers/common/Makefile | 4 +
drivers/common/meson.build | 2 +-
drivers/common/sfc_efx/Makefile | 112 ++++++
.../{net/sfc => common/sfc_efx}/base/README | 0
.../sfc => common/sfc_efx}/base/ef10_ev.c | 0
.../sfc => common/sfc_efx}/base/ef10_evb.c | 0
.../sfc => common/sfc_efx}/base/ef10_filter.c | 0
.../sfc_efx}/base/ef10_firmware_ids.h | 0
.../sfc => common/sfc_efx}/base/ef10_image.c | 0
.../sfc => common/sfc_efx}/base/ef10_impl.h | 0
.../sfc => common/sfc_efx}/base/ef10_intr.c | 0
.../sfc => common/sfc_efx}/base/ef10_mac.c | 0
.../sfc => common/sfc_efx}/base/ef10_mcdi.c | 0
.../sfc => common/sfc_efx}/base/ef10_nic.c | 0
.../sfc => common/sfc_efx}/base/ef10_nvram.c | 0
.../sfc => common/sfc_efx}/base/ef10_phy.c | 0
.../sfc => common/sfc_efx}/base/ef10_proxy.c | 0
.../sfc => common/sfc_efx}/base/ef10_rx.c | 0
.../sfc_efx}/base/ef10_signed_image_layout.h | 0
.../sfc_efx}/base/ef10_tlv_layout.h | 0
.../sfc => common/sfc_efx}/base/ef10_tx.c | 0
.../sfc => common/sfc_efx}/base/ef10_vpd.c | 0
.../{net/sfc => common/sfc_efx}/base/efx.h | 0
.../sfc => common/sfc_efx}/base/efx_annote.h | 0
.../sfc => common/sfc_efx}/base/efx_bootcfg.c | 0
.../sfc => common/sfc_efx}/base/efx_check.h | 0
.../sfc => common/sfc_efx}/base/efx_crc32.c | 0
.../{net/sfc => common/sfc_efx}/base/efx_ev.c | 0
.../sfc => common/sfc_efx}/base/efx_evb.c | 0
.../sfc => common/sfc_efx}/base/efx_filter.c | 0
.../sfc => common/sfc_efx}/base/efx_hash.c | 0
.../sfc => common/sfc_efx}/base/efx_impl.h | 0
.../sfc => common/sfc_efx}/base/efx_intr.c | 0
.../sfc => common/sfc_efx}/base/efx_lic.c | 0
.../sfc => common/sfc_efx}/base/efx_mac.c | 0
.../sfc => common/sfc_efx}/base/efx_mcdi.c | 0
.../sfc => common/sfc_efx}/base/efx_mcdi.h | 0
.../sfc => common/sfc_efx}/base/efx_mon.c | 0
.../sfc => common/sfc_efx}/base/efx_nic.c | 0
.../sfc => common/sfc_efx}/base/efx_nvram.c | 0
.../sfc => common/sfc_efx}/base/efx_phy.c | 0
.../sfc => common/sfc_efx}/base/efx_phy_ids.h | 0
.../sfc => common/sfc_efx}/base/efx_port.c | 0
.../sfc => common/sfc_efx}/base/efx_proxy.c | 0
.../sfc => common/sfc_efx}/base/efx_regs.h | 0
.../sfc_efx}/base/efx_regs_ef10.h | 0
.../sfc_efx}/base/efx_regs_mcdi.h | 0
.../sfc_efx}/base/efx_regs_mcdi_aoe.h | 0
.../sfc_efx}/base/efx_regs_mcdi_strs.h | 0
.../sfc_efx}/base/efx_regs_pci.h | 0
.../{net/sfc => common/sfc_efx}/base/efx_rx.c | 0
.../sfc => common/sfc_efx}/base/efx_sram.c | 0
.../sfc => common/sfc_efx}/base/efx_tunnel.c | 0
.../{net/sfc => common/sfc_efx}/base/efx_tx.c | 0
.../sfc => common/sfc_efx}/base/efx_types.h | 0
.../sfc => common/sfc_efx}/base/efx_vpd.c | 0
.../sfc => common/sfc_efx}/base/hunt_impl.h | 0
.../sfc => common/sfc_efx}/base/hunt_nic.c | 0
.../sfc => common/sfc_efx}/base/mcdi_mon.c | 0
.../sfc => common/sfc_efx}/base/mcdi_mon.h | 0
.../sfc_efx}/base/medford2_impl.h | 0
.../sfc_efx}/base/medford2_nic.c | 0
.../sfc_efx}/base/medford_impl.h | 0
.../sfc => common/sfc_efx}/base/medford_nic.c | 0
.../sfc => common/sfc_efx}/base/meson.build | 0
.../sfc => common/sfc_efx}/base/siena_flash.h | 0
.../sfc => common/sfc_efx}/base/siena_impl.h | 0
.../sfc => common/sfc_efx}/base/siena_mac.c | 0
.../sfc => common/sfc_efx}/base/siena_mcdi.c | 0
.../sfc => common/sfc_efx}/base/siena_nic.c | 0
.../sfc => common/sfc_efx}/base/siena_nvram.c | 0
.../sfc => common/sfc_efx}/base/siena_phy.c | 0
.../sfc => common/sfc_efx}/base/siena_sram.c | 0
.../sfc => common/sfc_efx}/base/siena_vpd.c | 0
drivers/{net/sfc => common/sfc_efx}/efsys.h | 58 +--
drivers/common/sfc_efx/meson.build | 40 ++
.../sfc_efx/rte_common_sfc_efx_version.map | 119 ++++++
drivers/common/sfc_efx/sfc_efx.c | 23 ++
drivers/common/sfc_efx/sfc_efx_debug.h | 29 ++
drivers/common/sfc_efx/sfc_efx_log.h | 22 ++
drivers/common/sfc_efx/sfc_efx_mcdi.c | 343 ++++++++++++++++++
drivers/common/sfc_efx/sfc_efx_mcdi.h | 75 ++++
drivers/net/sfc/Makefile | 70 +---
drivers/net/sfc/meson.build | 6 +-
drivers/net/sfc/sfc.c | 1 +
drivers/net/sfc/sfc.h | 26 +-
drivers/net/sfc/sfc_dp_tx.h | 1 +
drivers/net/sfc/sfc_ef10.h | 2 +
drivers/net/sfc/sfc_ef10_essb_rx.c | 1 +
drivers/net/sfc/sfc_ef10_rx.c | 1 +
drivers/net/sfc/sfc_ef10_rx_ev.h | 2 +
drivers/net/sfc/sfc_ef10_tx.c | 1 +
drivers/net/sfc/sfc_ethdev.c | 13 +
drivers/net/sfc/sfc_filter.c | 1 +
drivers/net/sfc/sfc_flow.c | 1 +
drivers/net/sfc/sfc_log.h | 21 +-
drivers/net/sfc/sfc_mcdi.c | 295 ++-------------
drivers/net/sfc/sfc_port.c | 1 +
mk/rte.app.mk | 1 +
100 files changed, 870 insertions(+), 402 deletions(-)
create mode 100644 drivers/common/sfc_efx/Makefile
rename drivers/{net/sfc => common/sfc_efx}/base/README (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/ef10_ev.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/ef10_evb.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/ef10_filter.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/ef10_firmware_ids.h (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/ef10_image.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/ef10_impl.h (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/ef10_intr.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/ef10_mac.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/ef10_mcdi.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/ef10_nic.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/ef10_nvram.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/ef10_phy.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/ef10_proxy.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/ef10_rx.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/ef10_signed_image_layout.h (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/ef10_tlv_layout.h (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/ef10_tx.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/ef10_vpd.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/efx.h (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/efx_annote.h (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/efx_bootcfg.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/efx_check.h (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/efx_crc32.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/efx_ev.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/efx_evb.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/efx_filter.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/efx_hash.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/efx_impl.h (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/efx_intr.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/efx_lic.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/efx_mac.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/efx_mcdi.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/efx_mcdi.h (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/efx_mon.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/efx_nic.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/efx_nvram.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/efx_phy.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/efx_phy_ids.h (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/efx_port.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/efx_proxy.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/efx_regs.h (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/efx_regs_ef10.h (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/efx_regs_mcdi.h (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/efx_regs_mcdi_aoe.h (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/efx_regs_mcdi_strs.h (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/efx_regs_pci.h (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/efx_rx.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/efx_sram.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/efx_tunnel.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/efx_tx.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/efx_types.h (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/efx_vpd.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/hunt_impl.h (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/hunt_nic.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/mcdi_mon.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/mcdi_mon.h (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/medford2_impl.h (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/medford2_nic.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/medford_impl.h (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/medford_nic.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/meson.build (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/siena_flash.h (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/siena_impl.h (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/siena_mac.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/siena_mcdi.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/siena_nic.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/siena_nvram.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/siena_phy.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/siena_sram.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/base/siena_vpd.c (100%)
rename drivers/{net/sfc => common/sfc_efx}/efsys.h (93%)
create mode 100644 drivers/common/sfc_efx/meson.build
create mode 100644 drivers/common/sfc_efx/rte_common_sfc_efx_version.map
create mode 100644 drivers/common/sfc_efx/sfc_efx.c
create mode 100644 drivers/common/sfc_efx/sfc_efx_debug.h
create mode 100644 drivers/common/sfc_efx/sfc_efx_log.h
create mode 100644 drivers/common/sfc_efx/sfc_efx_mcdi.c
create mode 100644 drivers/common/sfc_efx/sfc_efx_mcdi.h
--
2.17.1
next reply other threads:[~2020-09-08 9:15 UTC|newest]
Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-09-08 9:14 Andrew Rybchenko [this message]
2020-09-08 9:14 ` [dpdk-dev] [PATCH 01/14] net/iavf: downgrade error log Andrew Rybchenko
2020-09-08 9:26 ` Andrew Rybchenko
2020-09-08 9:14 ` [dpdk-dev] [PATCH 01/14] net/sfc: include header with debug helpers directly Andrew Rybchenko
2020-09-08 9:14 ` [dpdk-dev] [PATCH 02/14] net/sfc: introduce common driver library Andrew Rybchenko
2020-09-08 9:14 ` [dpdk-dev] [PATCH 03/14] net/sfc: add dedicated header file with MCDI interface Andrew Rybchenko
2020-09-08 9:14 ` [dpdk-dev] [PATCH 04/14] net/sfc: move MCDI helper interface to dedicated namespace Andrew Rybchenko
2020-09-08 9:14 ` [dpdk-dev] [PATCH 05/14] net/sfc: make MCDI logging helper macros local Andrew Rybchenko
2020-09-08 9:14 ` [dpdk-dev] [PATCH 06/14] net/sfc: start to make MCDI helpers interface shareable Andrew Rybchenko
2020-09-08 9:14 ` [dpdk-dev] [PATCH 07/14] net/sfc: use own logging helper macros Andrew Rybchenko
2020-09-08 9:14 ` [dpdk-dev] [PATCH 08/14] net/sfc: avoid usage of NIC pointer from adapter context Andrew Rybchenko
2020-09-08 9:14 ` [dpdk-dev] [PATCH 09/14] net/sfc: avoid panic in the case of MCDI timeout Andrew Rybchenko
2020-09-08 9:14 ` [dpdk-dev] [PATCH 10/14] net/sfc: add MCDI callbacks to allocate/free DMA memory Andrew Rybchenko
2020-09-08 9:14 ` [dpdk-dev] [PATCH 11/14] net/sfc: add MCDI callback to schedule restart Andrew Rybchenko
2020-09-08 9:14 ` [dpdk-dev] [PATCH 12/14] net/sfc: add MCDI callback to poll management event queue Andrew Rybchenko
2020-09-08 9:14 ` [dpdk-dev] [PATCH 13/14] net/sfc: use MCDI control structure as libefx ops context Andrew Rybchenko
2020-09-08 9:14 ` [dpdk-dev] [PATCH 14/14] net/sfc: move MCDI helpers to common driver Andrew Rybchenko
2020-09-09 14:32 ` [dpdk-dev] [PATCH 00/14] net/sfc: factor out common driver library Ferruh Yigit
2020-09-17 6:21 ` Andrew Rybchenko
2020-09-17 6:34 ` [dpdk-dev] [PATCH v2 00/17] " Andrew Rybchenko
2020-09-17 6:34 ` [dpdk-dev] [PATCH v2 01/17] net/sfc/base: add missing extern storage-class specifiers Andrew Rybchenko
2020-09-17 6:34 ` [dpdk-dev] [PATCH v2 02/17] net/sfc/base: decorate libefx API functions Andrew Rybchenko
2020-09-17 6:34 ` [dpdk-dev] [PATCH v2 03/17] net/sfc/base: decorate libefx internal extern functions Andrew Rybchenko
2020-09-17 6:34 ` [dpdk-dev] [PATCH v2 04/17] net/sfc: include header with debug helpers directly Andrew Rybchenko
2020-09-17 6:34 ` [dpdk-dev] [PATCH v2 05/17] net/sfc: introduce common driver library Andrew Rybchenko
2020-09-17 6:34 ` [dpdk-dev] [PATCH v2 06/17] net/sfc: add dedicated header file with MCDI interface Andrew Rybchenko
2020-09-17 6:34 ` [dpdk-dev] [PATCH v2 07/17] net/sfc: move MCDI helper interface to dedicated namespace Andrew Rybchenko
2020-09-17 6:34 ` [dpdk-dev] [PATCH v2 08/17] net/sfc: make MCDI logging helper macros local Andrew Rybchenko
2020-09-17 6:34 ` [dpdk-dev] [PATCH v2 09/17] net/sfc: start to make MCDI helpers interface shareable Andrew Rybchenko
2020-09-17 6:34 ` [dpdk-dev] [PATCH v2 10/17] net/sfc: use own logging helper macros Andrew Rybchenko
2020-09-17 6:34 ` [dpdk-dev] [PATCH v2 11/17] net/sfc: avoid usage of NIC pointer from adapter context Andrew Rybchenko
2020-09-17 6:34 ` [dpdk-dev] [PATCH v2 12/17] net/sfc: avoid panic in the case of MCDI timeout Andrew Rybchenko
2020-09-17 6:34 ` [dpdk-dev] [PATCH v2 13/17] net/sfc: add MCDI callbacks to allocate/free DMA memory Andrew Rybchenko
2020-09-17 6:34 ` [dpdk-dev] [PATCH v2 14/17] net/sfc: add MCDI callback to schedule restart Andrew Rybchenko
2020-09-17 6:34 ` [dpdk-dev] [PATCH v2 15/17] net/sfc: add MCDI callback to poll management event queue Andrew Rybchenko
2020-09-17 6:34 ` [dpdk-dev] [PATCH v2 16/17] net/sfc: use MCDI control structure as libefx ops context Andrew Rybchenko
2020-09-17 6:34 ` [dpdk-dev] [PATCH v2 17/17] net/sfc: move MCDI helpers to common driver Andrew Rybchenko
2020-09-21 22:38 ` [dpdk-dev] [PATCH v2 00/17] net/sfc: factor out common driver library 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=1599556475-27820-1-git-send-email-arybchenko@solarflare.com \
--to=arybchenko@solarflare.com \
--cc=david.marchand@redhat.com \
--cc=dev@dpdk.org \
--cc=ferruh.yigit@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).