[dpdk-dev] [PATCH 8/8] bus/pci: remove ABIs in PCI bus
Xu, Rosen
rosen.xu at intel.com
Tue Sep 14 10:15:26 CEST 2021
> -----Original Message-----
> From: Xia, Chenbo <chenbo.xia at intel.com>
> Sent: Friday, September 10, 2021 10:24
> To: dev at dpdk.org
> Cc: Chautru, Nicolas <nicolas.chautru at intel.com>; Yigit, Ferruh
> <ferruh.yigit at intel.com>; Burakov, Anatoly <anatoly.burakov at intel.com>;
> Ray Kinsella <mdr at ashroe.eu>; Nithin Dabilpuram
> <ndabilpuram at marvell.com>; Kiran Kumar K <kirankumark at marvell.com>;
> Sunil Kumar Kori <skori at marvell.com>; Satha Rao
> <skoteshwar at marvell.com>; Matan Azrad <matan at nvidia.com>; Shahaf
> Shuler <shahafs at nvidia.com>; Viacheslav Ovsiienko
> <viacheslavo at nvidia.com>; Jerin Jacob <jerinj at marvell.com>; Anoob Joseph
> <anoobj at marvell.com>; Trahe, Fiona <fiona.trahe at intel.com>; Griffin, John
> <john.griffin at intel.com>; Jain, Deepak K <deepak.k.jain at intel.com>;
> Andrew Rybchenko <andrew.rybchenko at oktetlabs.ru>; Ashish Gupta
> <ashish.gupta at marvell.com>; Somalapuram Amaranath
> <asomalap at amd.com>; Ankur Dwivedi <adwivedi at marvell.com>; Tejasree
> Kondoj <ktejasree at marvell.com>; Nagadheeraj Rottela
> <rnagadheeraj at marvell.com>; Srikanth Jampala <jsrikanth at marvell.com>;
> Jay Zhou <jianjay.zhou at huawei.com>; McDaniel, Timothy
> <timothy.mcdaniel at intel.com>; Pavan Nikhilesh
> <pbhagavatula at marvell.com>; Ashwin Sekhar T K <asekhar at marvell.com>;
> Harman Kalra <hkalra at marvell.com>; Shepard Siegel
> <shepard.siegel at atomicrules.com>; Ed Czeck <ed.czeck at atomicrules.com>;
> John Miller <john.miller at atomicrules.com>; Steven Webster
> <steven.webster at windriver.com>; Peters, Matt
> <matt.peters at windriver.com>; Rasesh Mody <rmody at marvell.com>;
> Shahed Shaikh <shshaikh at marvell.com>; Ajit Khaparde
> <ajit.khaparde at broadcom.com>; Somnath Kotur
> <somnath.kotur at broadcom.com>; Chas Williams <chas3 at att.com>; Min Hu
> (Connor) <humin29 at huawei.com>; Rahul Lakkireddy
> <rahul.lakkireddy at chelsio.com>; Wang, Haiyue <haiyue.wang at intel.com>;
> Marcin Wojtas <mw at semihalf.com>; Michal Krawczyk <mk at semihalf.com>;
> Shai Brandes <shaibran at amazon.com>; Evgeny Schemeilin
> <evgenys at amazon.com>; Igor Chauskin <igorch at amazon.com>; Daley, John
> <johndale at cisco.com>; Hyong Youb Kim <hyonkim at cisco.com>; Ziyang
> Xuan <xuanziyang2 at huawei.com>; Xiaoyun Wang
> <cloud.wangxiaoyun at huawei.com>; Guoyang Zhou
> <zhouguoyang at huawei.com>; Yisen Zhuang <yisen.zhuang at huawei.com>;
> Lijun Ou <oulijun at huawei.com>; Xing, Beilei <beilei.xing at intel.com>;
> Andrew Boyer <aboyer at pensando.io>; Xu, Rosen <rosen.xu at intel.com>;
> Stephen Hemminger <sthemmin at microsoft.com>; Long Li
> <longli at microsoft.com>; Devendra Singh Rawat
> <dsinghrawat at marvell.com>; Maciej Czekaj <mczekaj at marvell.com>;
> Jiawen Wu <jiawenwu at trustnetic.com>; Jian Wang
> <jianwang at trustnetic.com>; Maxime Coquelin
> <maxime.coquelin at redhat.com>; Yong Wang <yongwang at vmware.com>;
> Jakub Palider <jpalider at marvell.com>; Tomasz Duszynski
> <tduszynski at marvell.com>; Zhang, Tianfei <tianfei.zhang at intel.com>;
> Richardson, Bruce <bruce.richardson at intel.com>; Li, Xiaoyun
> <xiaoyun.li at intel.com>; Wu, Jingjing <jingjing.wu at intel.com>; Radha Mohan
> Chintakuntla <radhac at marvell.com>; Veerasenareddy Burru
> <vburru at marvell.com>; Ori Kam <orika at nvidia.com>; Wang, Xiao W
> <xiao.w.wang at intel.com>; Thomas Monjalon <thomas at monjalon.net>
> Subject: [PATCH 8/8] bus/pci: remove ABIs in PCI bus
>
> As announced in the deprecation note, most of ABIs in PCI bus are
> removed in this patch. Only the function rte_pci_dump is still ABI
> and experimental APIs are kept for future promotion.
>
> This patch creates a new file named pci_driver.h and moves most of
> the content in original rte_bus_pci.h to it. After that, pci_driver.h
> is considered the interface for drivers and rte_bus_pci.h for
> applications. pci_driver.h is defined as driver_sdk_headers so that
> out-of-tree drivers can use it.
>
> Then this patch replaces the including of rte_bus_pci.h with pci_driver.h
> in all related drivers.
>
> Signed-off-by: Chenbo Xia <chenbo.xia at intel.com>
> ---
> app/test/virtual_pmd.c | 2 +-
> doc/guides/rel_notes/release_21_11.rst | 2 +
> drivers/baseband/acc100/rte_acc100_pmd.c | 2 +-
> .../fpga_5gnr_fec/rte_fpga_5gnr_fec.c | 2 +-
> drivers/baseband/fpga_lte_fec/fpga_lte_fec.c | 2 +-
> drivers/bus/pci/bsd/pci.c | 1 -
> drivers/bus/pci/linux/pci.c | 1 -
> drivers/bus/pci/linux/pci_uio.c | 1 -
> drivers/bus/pci/linux/pci_vfio.c | 1 -
> drivers/bus/pci/meson.build | 4 +
> drivers/bus/pci/pci_common_uio.c | 1 -
> drivers/bus/pci/pci_driver.h | 402 ++++++++++++++++++
> drivers/bus/pci/pci_params.c | 1 -
> drivers/bus/pci/private.h | 3 +-
> drivers/bus/pci/rte_bus_pci.h | 375 +---------------
> drivers/bus/pci/version.map | 32 +-
> drivers/common/cnxk/roc_platform.h | 2 +-
> drivers/common/mlx5/linux/mlx5_common_verbs.c | 2 +-
> drivers/common/mlx5/mlx5_common_pci.c | 2 +-
> drivers/common/octeontx2/otx2_dev.h | 2 +-
> drivers/common/octeontx2/otx2_sec_idev.c | 2 +-
> drivers/common/qat/qat_device.h | 2 +-
> drivers/common/qat/qat_qp.c | 2 +-
> drivers/common/sfc_efx/sfc_efx.h | 2 +-
> drivers/compress/mlx5/mlx5_compress.c | 2 +-
> drivers/compress/octeontx/otx_zip.h | 2 +-
> drivers/compress/qat/qat_comp.c | 2 +-
> drivers/crypto/ccp/ccp_dev.h | 2 +-
> drivers/crypto/ccp/ccp_pci.h | 2 +-
> drivers/crypto/ccp/rte_ccp_pmd.c | 2 +-
> drivers/crypto/cnxk/cn10k_cryptodev.c | 2 +-
> drivers/crypto/cnxk/cn9k_cryptodev.c | 2 +-
> drivers/crypto/mlx5/mlx5_crypto.c | 2 +-
> drivers/crypto/nitrox/nitrox_device.h | 2 +-
> drivers/crypto/octeontx/otx_cryptodev.c | 2 +-
> drivers/crypto/octeontx/otx_cryptodev_ops.c | 2 +-
> drivers/crypto/octeontx2/otx2_cryptodev.c | 2 +-
> drivers/crypto/qat/qat_sym.c | 2 +-
> drivers/crypto/qat/qat_sym_pmd.c | 2 +-
> drivers/crypto/virtio/virtio_cryptodev.c | 2 +-
> drivers/crypto/virtio/virtio_pci.h | 2 +-
> drivers/event/dlb2/pf/dlb2_main.h | 2 +-
> drivers/event/dlb2/pf/dlb2_pf.c | 2 +-
> drivers/event/octeontx/ssovf_probe.c | 2 +-
> drivers/event/octeontx/timvf_probe.c | 2 +-
> drivers/event/octeontx2/otx2_evdev.c | 2 +-
> drivers/mempool/cnxk/cnxk_mempool.c | 2 +-
> drivers/mempool/octeontx/octeontx_fpavf.c | 2 +-
> drivers/mempool/octeontx2/otx2_mempool.c | 2 +-
> drivers/mempool/octeontx2/otx2_mempool.h | 2 +-
> drivers/mempool/octeontx2/otx2_mempool_irq.c | 2 +-
> drivers/meson.build | 4 +
> drivers/net/ark/ark_ethdev.c | 2 +-
> drivers/net/avp/avp_ethdev.c | 2 +-
> drivers/net/bnx2x/bnx2x.h | 2 +-
> drivers/net/bnxt/bnxt.h | 2 +-
> drivers/net/bonding/rte_eth_bond_args.c | 2 +-
> drivers/net/cxgbe/base/adapter.h | 2 +-
> drivers/net/cxgbe/cxgbe_ethdev.c | 2 +-
> drivers/net/e1000/em_ethdev.c | 2 +-
> drivers/net/e1000/em_rxtx.c | 2 +-
> drivers/net/e1000/igb_ethdev.c | 2 +-
> drivers/net/e1000/igb_pf.c | 2 +-
> drivers/net/ena/ena_ethdev.h | 2 +-
> drivers/net/enic/base/vnic_dev.h | 2 +-
> drivers/net/enic/enic_ethdev.c | 2 +-
> drivers/net/enic/enic_main.c | 2 +-
> drivers/net/enic/enic_vf_representor.c | 2 +-
> drivers/net/hinic/base/hinic_pmd_hwdev.c | 2 +-
> drivers/net/hinic/base/hinic_pmd_hwif.c | 2 +-
> drivers/net/hinic/base/hinic_pmd_nicio.c | 2 +-
> drivers/net/hinic/hinic_pmd_ethdev.c | 2 +-
> drivers/net/hns3/hns3_ethdev.c | 2 +-
> drivers/net/hns3/hns3_rxtx.c | 2 +-
> drivers/net/i40e/i40e_ethdev.c | 2 +-
> drivers/net/i40e/i40e_ethdev_vf.c | 2 +-
> drivers/net/i40e/i40e_vf_representor.c | 2 +-
> drivers/net/igc/igc_ethdev.c | 2 +-
> drivers/net/ionic/ionic.h | 2 +-
> drivers/net/ionic/ionic_ethdev.c | 2 +-
> drivers/net/ipn3ke/ipn3ke_ethdev.c | 2 +-
> drivers/net/ipn3ke/ipn3ke_representor.c | 2 +-
> drivers/net/ipn3ke/ipn3ke_tm.c | 2 +-
> drivers/net/ixgbe/ixgbe_ethdev.c | 2 +-
> drivers/net/ixgbe/ixgbe_ethdev.h | 2 +-
> drivers/net/mlx4/mlx4_ethdev.c | 2 +-
> drivers/net/mlx5/linux/mlx5_ethdev_os.c | 2 +-
> drivers/net/mlx5/linux/mlx5_os.c | 2 +-
> drivers/net/mlx5/mlx5.c | 2 +-
> drivers/net/mlx5/mlx5_ethdev.c | 2 +-
> drivers/net/mlx5/mlx5_txq.c | 2 +-
> drivers/net/netvsc/hn_vf.c | 2 +-
> drivers/net/octeontx/base/octeontx_pkivf.c | 2 +-
> drivers/net/octeontx/base/octeontx_pkovf.c | 2 +-
> drivers/net/octeontx2/otx2_ethdev_irq.c | 2 +-
> drivers/net/qede/base/bcm_osal.h | 2 +-
> drivers/net/sfc/sfc.h | 2 +-
> drivers/net/sfc/sfc_ethdev.c | 2 +-
> drivers/net/sfc/sfc_sriov.c | 2 +-
> drivers/net/thunderx/nicvf_ethdev.c | 2 +-
> drivers/net/txgbe/txgbe_ethdev.h | 2 +-
> drivers/net/txgbe/txgbe_flow.c | 2 +-
> drivers/net/txgbe/txgbe_pf.c | 2 +-
> drivers/net/virtio/virtio_pci.h | 2 +-
> drivers/net/virtio/virtio_pci_ethdev.c | 2 +-
> drivers/net/vmxnet3/vmxnet3_ethdev.c | 2 +-
> drivers/raw/cnxk_bphy/cnxk_bphy.c | 2 +-
> drivers/raw/cnxk_bphy/cnxk_bphy_cgx.c | 2 +-
> drivers/raw/cnxk_bphy/cnxk_bphy_irq.c | 2 +-
> drivers/raw/cnxk_bphy/cnxk_bphy_irq.h | 2 +-
> drivers/raw/ifpga/ifpga_rawdev.c | 2 +-
> drivers/raw/ifpga/rte_pmd_ifpga.c | 2 +-
> drivers/raw/ioat/idxd_pci.c | 2 +-
> drivers/raw/ioat/ioat_rawdev.c | 2 +-
> drivers/raw/ntb/ntb.c | 2 +-
> drivers/raw/ntb/ntb_hw_intel.c | 2 +-
> drivers/raw/octeontx2_dma/otx2_dpi_rawdev.c | 2 +-
> drivers/raw/octeontx2_ep/otx2_ep_enqdeq.c | 2 +-
> drivers/raw/octeontx2_ep/otx2_ep_rawdev.c | 2 +-
> drivers/regex/mlx5/mlx5_regex.c | 2 +-
> drivers/regex/mlx5/mlx5_regex_fastpath.c | 2 +-
> drivers/vdpa/ifc/base/ifcvf_osdep.h | 2 +-
> drivers/vdpa/ifc/ifcvf_vdpa.c | 2 +-
> drivers/vdpa/mlx5/mlx5_vdpa.c | 2 +-
> lib/ethdev/ethdev_pci.h | 2 +-
> lib/eventdev/eventdev_pmd_pci.h | 2 +-
> 126 files changed, 546 insertions(+), 508 deletions(-)
> create mode 100644 drivers/bus/pci/pci_driver.h
>
Acked-by: Rosen Xu <rosen.xu at intel.com>
More information about the dev
mailing list