mbox series

[v8,00/13] NXP DPAAx: move internal symbols to INTERNAL

Message ID 20200515094752.28490-1-hemant.agrawal@nxp.com
Headers show
Series NXP DPAAx: move internal symbols to INTERNAL | expand

Message

Hemant Agrawal May 15, 2020, 9:47 a.m. UTC 
v8: fix 32bit compilation

v7: using empty DPDK_20.0 to preserve the so name

v6: v5 was dirty

v5: use empty DPDK_20.0 to avoid so name erriin pmd_dpaa2


v4: pmd_dpaa, mempool_dpaa2 also has external symbols. So, changing
the libabigail.abignore exception to only for selected func calls
pmd_dpaa2 also has experimental external func call remaining.
However, it causes the so name to be changed. So, we need to maintain
the specific file name exception there for now. 

v3: adding entries in libabigail.ignore

This patchset moves the internal symbol to the INTERNAL sections

This patch also change some of the shared variables to function
calls. (Mostly control path used variables).

Hemant Agrawal (13):
  common/dpaax: move internal symbols into INTERNAL section
  bus/fslmc: move internal symbols into INTERNAL section
  bus/dpaa: move internal symbols into INTERNAL section
  mempool/dpaa2: move internal symbols into INTERNAL section
  net/dpaa: move internal symbols into INTERNAL section
  net/dpaa2: move internal symbols into INTERNAL section
  crypto: move internal symbols into INTERNAL section
  mempool/dpaa2: fix to add the include file
  bus/dpaa: share qman portal ids as function calls
  bus/dpaa: move log into to respective dpaa drivers
  bus/dpaa: change netcfg access as function
  bus/fslmc: share mcp ptr as function across libs
  bus/fslmc: remove unused symbols

 devtools/libabigail.abignore                  | 17 ++++++
 drivers/bus/dpaa/base/qbman/qman_driver.c     | 10 ++++
 drivers/bus/dpaa/dpaa_bus.c                   | 25 +++-----
 drivers/bus/dpaa/include/fsl_bman.h           |  6 ++
 drivers/bus/dpaa/include/fsl_fman.h           | 27 +++++++++
 drivers/bus/dpaa/include/fsl_qman.h           | 40 ++++++++++++-
 drivers/bus/dpaa/include/fsl_usd.h            |  8 ++-
 drivers/bus/dpaa/include/netcfg.h             |  2 +
 drivers/bus/dpaa/rte_bus_dpaa_version.map     | 17 +++---
 drivers/bus/dpaa/rte_dpaa_bus.h               | 11 +++-
 drivers/bus/dpaa/rte_dpaa_logs.h              | 58 -------------------
 drivers/bus/fslmc/fslmc_vfio.c                | 15 ++++-
 drivers/bus/fslmc/fslmc_vfio.h                |  5 ++
 drivers/bus/fslmc/mc/fsl_dpbp.h               |  7 +++
 drivers/bus/fslmc/mc/fsl_dpci.h               |  3 +
 drivers/bus/fslmc/mc/fsl_dpcon.h              |  2 +
 drivers/bus/fslmc/mc/fsl_dpdmai.h             | 10 ++++
 drivers/bus/fslmc/mc/fsl_dpio.h               | 11 ++++
 drivers/bus/fslmc/mc/fsl_dpmng.h              |  4 ++
 drivers/bus/fslmc/mc/fsl_mc_cmd.h             |  2 +
 drivers/bus/fslmc/portal/dpaa2_hw_dpbp.c      |  8 +--
 drivers/bus/fslmc/portal/dpaa2_hw_dpci.c      |  2 +-
 drivers/bus/fslmc/portal/dpaa2_hw_dpio.c      |  2 +-
 drivers/bus/fslmc/portal/dpaa2_hw_dpio.h      |  5 ++
 drivers/bus/fslmc/portal/dpaa2_hw_pvt.h       | 18 ++++--
 .../bus/fslmc/qbman/include/fsl_qbman_debug.h |  8 +++
 .../fslmc/qbman/include/fsl_qbman_portal.h    | 42 ++++++++++++++
 drivers/bus/fslmc/rte_bus_fslmc_version.map   | 23 ++++----
 drivers/bus/fslmc/rte_fslmc.h                 |  4 ++
 drivers/common/dpaax/dpaa_of.h                | 15 +++++
 drivers/common/dpaax/dpaax_iova_table.h       |  4 ++
 .../common/dpaax/rte_common_dpaax_version.map |  6 +-
 drivers/crypto/dpaa2_sec/dpaa2_sec_dpseci.c   |  2 +-
 drivers/crypto/dpaa2_sec/dpaa2_sec_event.h    |  5 +-
 .../dpaa2_sec/rte_pmd_dpaa2_sec_version.map   |  6 +-
 drivers/crypto/dpaa_sec/dpaa_sec.c            |  2 +-
 drivers/crypto/dpaa_sec/dpaa_sec_event.h      |  8 +--
 .../dpaa_sec/rte_pmd_dpaa_sec_version.map     |  6 +-
 drivers/event/dpaa/dpaa_eventdev.c            |  7 +++
 drivers/event/dpaa/dpaa_eventdev.h            | 16 +++++
 drivers/event/dpaa2/dpaa2_hw_dpcon.c          |  2 +-
 drivers/mempool/dpaa/dpaa_mempool.c           |  8 +++
 drivers/mempool/dpaa/dpaa_mempool.h           | 19 ++++++
 .../mempool/dpaa/rte_mempool_dpaa_version.map |  6 +-
 drivers/mempool/dpaa2/dpaa2_hw_mempool.h      |  1 +
 drivers/mempool/dpaa2/meson.build             |  2 +
 .../dpaa2/rte_mempool_dpaa2_version.map       |  9 ++-
 drivers/net/dpaa/dpaa_ethdev.c                | 12 +++-
 drivers/net/dpaa/dpaa_ethdev.h                | 24 ++++++++
 drivers/net/dpaa/rte_pmd_dpaa_version.map     |  9 ++-
 drivers/net/dpaa2/dpaa2_ethdev.c              |  2 +-
 drivers/net/dpaa2/dpaa2_ethdev.h              |  2 +
 drivers/net/dpaa2/dpaa2_mux.c                 |  2 +-
 drivers/net/dpaa2/dpaa2_ptp.c                 |  2 +-
 drivers/net/dpaa2/rte_pmd_dpaa2_version.map   | 12 ++--
 drivers/raw/dpaa2_qdma/dpaa2_qdma.c           |  2 +-
 56 files changed, 431 insertions(+), 152 deletions(-)

-- 
2.17.1

Comments

David Marchand May 19, 2020, 1:49 p.m. UTC | #1 
On Fri, May 15, 2020 at 11:50 AM Hemant Agrawal <hemant.agrawal@nxp.com> wrote:
> This patchset moves the internal symbol to the INTERNAL sections

>

> This patch also change some of the shared variables to function

> calls. (Mostly control path used variables).

>

> Hemant Agrawal (13):

>   common/dpaax: move internal symbols into INTERNAL section

>   bus/fslmc: move internal symbols into INTERNAL section

>   bus/dpaa: move internal symbols into INTERNAL section

>   mempool/dpaa2: move internal symbols into INTERNAL section

>   net/dpaa: move internal symbols into INTERNAL section

>   net/dpaa2: move internal symbols into INTERNAL section

>   crypto: move internal symbols into INTERNAL section

>   mempool/dpaa2: fix to add the include file

>   bus/dpaa: share qman portal ids as function calls

>   bus/dpaa: move log into to respective dpaa drivers

>   bus/dpaa: change netcfg access as function

>   bus/fslmc: share mcp ptr as function across libs

>   bus/fslmc: remove unused symbols


Rebased series, updated rules following Ray comments, re-tested and applied.
Thanks Hemant, Ray.


-- 
David Marchand