From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 3D934A0C45; Mon, 13 Sep 2021 14:06:20 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id B335540151; Mon, 13 Sep 2021 14:06:19 +0200 (CEST) Received: from mail-108-mta31.mxroute.com (mail-108-mta31.mxroute.com [136.175.108.31]) by mails.dpdk.org (Postfix) with ESMTP id 9DCCC4014F for ; Mon, 13 Sep 2021 14:06:17 +0200 (CEST) Received: from filter004.mxroute.com ([149.28.56.236] filter004.mxroute.com) (Authenticated sender: mN4UYu2MZsgR) by mail-108-mta31.mxroute.com (ZoneMTA) with ESMTPSA id 17bdf0cfb6e00074ba.00e for (version=TLSv1/SSLv3 cipher=ECDHE-RSA-AES128-GCM-SHA256); Mon, 13 Sep 2021 12:06:15 +0000 X-Zone-Loop: 43455fa6120cec1d9fcce0bec1f8e41ccc7136c5c901 X-Originating-IP: [149.28.56.236] DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=ashroe.eu; s=x; h=Content-Transfer-Encoding:Content-Type:In-Reply-To:MIME-Version:Date: Message-ID:From:References:Cc:To:Subject:Sender:Reply-To:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:List-Id:List-Help:List-Unsubscribe:List-Subscribe: List-Post:List-Owner:List-Archive; bh=I3kP+77Gr3Wr/4PUHTH+pxQa2yDjOtdLpmhgv7516hc=; b=mnB5TODbKsFd2ZzKlxWn3JyYeP zpSsYE0GsXSEuOEjRjRT+0c3pCtNlYcqfc+mzLnxivljl8mcxFwGsuAaJQit79pdfcjGICsK+PpZc 5pctogoY2jHufd2WUVnqRS+CykE00/o8sa9yrxqACudE3YoksrdN447tuOVvLT5fV0siEAJuBovUh IUEdLzqqcNkyIkkYL34/jwqfpEVWWy3kXbHX6YN6C0EVyuNUJhbfSayjb7bL18Vhtrjc6QSfQQyWu O7QG8vu08WyoPvkQgTF+USTBvHzXoNxqI52hhqpPX5ofuKwR5C/N5xcRHGHEYcS8B8yy6iTinn4rY wjrvhFXw==; To: Chenbo Xia , dev@dpdk.org Cc: Nicolas Chautru , Ferruh Yigit , Anatoly Burakov , Nithin Dabilpuram , Kiran Kumar K , Sunil Kumar Kori , Satha Rao , Matan Azrad , Shahaf Shuler , Viacheslav Ovsiienko , Jerin Jacob , Anoob Joseph , Fiona Trahe , John Griffin , Deepak Kumar Jain , Andrew Rybchenko , Ashish Gupta , Somalapuram Amaranath , Ankur Dwivedi , Tejasree Kondoj , Nagadheeraj Rottela , Srikanth Jampala , Jay Zhou , Timothy McDaniel , Pavan Nikhilesh , Ashwin Sekhar T K , Harman Kalra , Shepard Siegel , Ed Czeck , John Miller , Steven Webster , Matt Peters , Rasesh Mody , Shahed Shaikh , Ajit Khaparde , Somnath Kotur , Chas Williams , "Min Hu (Connor)" , Rahul Lakkireddy , Haiyue Wang , Marcin Wojtas , Michal Krawczyk , Shai Brandes , Evgeny Schemeilin , Igor Chauskin , John Daley , Hyong Youb Kim , Ziyang Xuan , Xiaoyun Wang , Guoyang Zhou , Yisen Zhuang , Lijun Ou , Beilei Xing , Andrew Boyer , Rosen Xu , Stephen Hemminger , Long Li , Devendra Singh Rawat , Maciej Czekaj , Jiawen Wu , Jian Wang , Maxime Coquelin , Yong Wang , Jakub Palider , Tomasz Duszynski , Tianfei zhang , Bruce Richardson , Xiaoyun Li , Jingjing Wu , Radha Mohan Chintakuntla , Veerasenareddy Burru , Ori Kam , Xiao Wang , Thomas Monjalon References: <20210910022402.26620-1-chenbo.xia@intel.com> <20210910022402.26620-9-chenbo.xia@intel.com> From: "Kinsella, Ray" Message-ID: <08847c18-c42b-7f59-af79-a3e990e72f0f@ashroe.eu> Date: Mon, 13 Sep 2021 13:06:01 +0100 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101 Thunderbird/78.14.0 MIME-Version: 1.0 In-Reply-To: <20210910022402.26620-9-chenbo.xia@intel.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit X-AuthUser: mdr@ashroe.eu Subject: Re: [dpdk-dev] [PATCH 8/8] bus/pci: remove ABIs in PCI bus X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On 10/09/2021 03:24, Chenbo Xia wrote: > 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 > --- > 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: Ray Kinsella