[PATCH v3 04/15] bus/pci: find PCI capability

[Xia, Chenbo]

> -----Original Message-----
> From: David Marchand <david.marchand at redhat.com>
> Sent: Thursday, September 14, 2023 8:36 PM
> To: dev at dpdk.org
> Cc: thomas at monjalon.net; ferruh.yigit at amd.com; Xia, Chenbo
> <chenbo.xia at intel.com>; nipun.gupta at amd.com; Richardson, Bruce
> <bruce.richardson at intel.com>; Burakov, Anatoly <anatoly.burakov at intel.com>;
> Jay Zhou <jianjay.zhou at huawei.com>; Sevincer, Abdullah
> <abdullah.sevincer at intel.com>; Julien Aube <julien_dpdk at jaube.fr>; Rahul
> Lakkireddy <rahul.lakkireddy at chelsio.com>; Guo, Junfeng
> <junfeng.guo at intel.com>; Jeroen de Borst <jeroendb at google.com>; Rushil
> Gupta <rushilg at google.com>; Joshua Washington <joshwash at google.com>;
> Dongdong Liu <liudongdong3 at huawei.com>; Yisen Zhuang
> <yisen.zhuang at huawei.com>; Maxime Coquelin <maxime.coquelin at redhat.com>;
> Gaetan Rivet <grive at u256.net>
> Subject: [PATCH v3 04/15] bus/pci: find PCI capability
> 
> Introduce two helpers so that drivers stop reinventing the wheel when it
> comes to finding capabilities in a device PCI configuration space.
> Use it in existing drivers.
> 
> Note:
> - base/ drivers code is left untouched, only some wrappers in cxgbe
>   are touched,
> - bnx2x maintained a per device cache of capabilities, this code has been
>   reworked to only cache the capabilities used in this driver,
> 
> Acked-by: Bruce Richardson <bruce.richardson at intel.com>
> Signed-off-by: David Marchand <david.marchand at redhat.com>
> ---
> Changes since v2:
> - added rte_pci_find_next_capability for vendor capa used with virtio,
> 
> Changes since v1:
> - updated commitlog,
> - separated VFIO changes for using standard PCI helper in a separate
>   patch,
> - marked new experimental symbols with current version,
> - reordered defines in rte_pci.h,
> 
> ---
>  drivers/bus/pci/linux/pci_vfio.c   |  74 ++++--------------
>  drivers/bus/pci/pci_common.c       |  54 +++++++++++++
>  drivers/bus/pci/rte_bus_pci.h      |  53 +++++++++++++
>  drivers/bus/pci/version.map        |   5 ++
>  drivers/crypto/virtio/virtio_pci.c |  57 +++++---------
>  drivers/event/dlb2/pf/dlb2_main.c  |  42 +---------
>  drivers/net/bnx2x/bnx2x.c          |  41 +++++-----
>  drivers/net/cxgbe/base/adapter.h   |  28 +------
>  drivers/net/gve/gve_ethdev.c       |  46 ++---------
>  drivers/net/gve/gve_ethdev.h       |   4 -
>  drivers/net/hns3/hns3_ethdev_vf.c  |  79 +++----------------
>  drivers/net/virtio/virtio_pci.c    | 121 ++++++-----------------------
>  lib/pci/rte_pci.h                  |  12 +++
>  13 files changed, 223 insertions(+), 393 deletions(-)
> 

Reviewed-by: Chenbo Xia <chenbo.xia at intel.com>