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 D92C0430D2; Tue, 22 Aug 2023 18:48:28 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id C6E3D40697; Tue, 22 Aug 2023 18:48:28 +0200 (CEST) Received: from mail-yw1-f173.google.com (mail-yw1-f173.google.com [209.85.128.173]) by mails.dpdk.org (Postfix) with ESMTP id C55B54021D for ; Tue, 22 Aug 2023 18:48:27 +0200 (CEST) Received: by mail-yw1-f173.google.com with SMTP id 00721157ae682-58c5642701fso52320727b3.0 for ; Tue, 22 Aug 2023 09:48:27 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=iol.unh.edu; s=unh-iol; t=1692722907; x=1693327707; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date:message-id:reply-to; bh=o5hGFq0B1jJKcAk4OfuG6ksamXpxT1mLm/2h1rgL1qk=; b=dzgUmK/jRRBRiklMh5yZyQTcP1ybWmjBlHgj5Ma8rXbC16cQ09mVc//zOiisctT4Fy 0G4xmex36LTRTVDT6LlTiaOYO1Ozg6NBP9TCyLfVnDRsXC6e02NZMYL/ATicgeF7jLiJ dKYSqHbHLRKweJMINf1tOR/sJNX9/FfadRT+c= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1692722907; x=1693327707; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=o5hGFq0B1jJKcAk4OfuG6ksamXpxT1mLm/2h1rgL1qk=; b=Gy2YHmqDnjoGMOniIjOfF3qe43POC+P8AxIg/VG2BkDi8h2TRlTsln7ycY8g6MyxJn lhhr0tZFRFd+oGpvYIc7gtvgv0Od5pEjoEhoVyJg0E8Tn0n4xjDWu3iCUGrvUmzAh2qg ft+FhjyQfO/6cFNTheR4TFiwcfDSOCv3nmZJLWMlzkmavi6zawAu32REkcCB1y9fbIJY MUG3+wv/yDJWQPtIgmbeJLNzqRRcxMrUzNeAWh1ororDB1uFJgwdEJY1iPa2jisRMqYR 6q53hSQXd5QFptb27XbR18CN9setsAapX8RpMZucKhMWLcQ+KT1OhY1YroMMNvSH94VY Aymg== X-Gm-Message-State: AOJu0YwUHI2YuI87dYZYjSpcw7NPWs+Wa6yOw9Y4u7NChQT/vf/z8tKs TGtwqbpZJ6Ul0nrMPzSFmZSHt4JapQHdmwLrdqn7bA== X-Google-Smtp-Source: AGHT+IFQWwaHisCd0y3dOwqw7I1LXUd3Fn8jv0ZfSWZDmqgqByZhGHfPWU8ueo6HaD5lKDH9Ny1mkIhl50+fKW2B8Sw= X-Received: by 2002:a0d:e502:0:b0:56d:330d:86e2 with SMTP id o2-20020a0de502000000b0056d330d86e2mr10332188ywe.28.1692722907004; Tue, 22 Aug 2023 09:48:27 -0700 (PDT) MIME-Version: 1.0 References: <20230803075038.307012-1-david.marchand@redhat.com> In-Reply-To: <20230803075038.307012-1-david.marchand@redhat.com> From: Adam Hassick Date: Tue, 22 Aug 2023 12:48:54 -0400 Message-ID: Subject: Re: [PATCH 00/14] Cleanup PCI(e) drivers To: David Marchand Cc: dev@dpdk.org Content-Type: multipart/alternative; boundary="000000000000935866060385c38e" 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 --000000000000935866060385c38e Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Recheck-request: iol-sample-apps-testing, This is a test email of the retesting framework, please ignore. On Thu, Aug 3, 2023 at 3:50=E2=80=AFAM David Marchand wrote: > Rather than rely on Linux headers to find some PCI(e) standard constants > or reinvent the same PCI capability helper, this series complements the > pci library and the pci bus driver. > PCI drivers can then use OS agnostic macros and helpers. > > WARNING: this is only compile tested. > > -- > David Marchand > > David Marchand (14): > drivers: remove duplicated PCI master control > bus/pci: add const to some experimental API > bus/pci: find PCI capability > pci: define some capability constants > pci: define some MSIX constants > pci: define some command constants > pci: define some BAR constants > pci: define some PM constants > pci: define some PCIe constants > pci: define some extended capability constants > pci: define some ACS constants > pci: define some PRI constants > pci: define some AER constants > devtools: forbid inclusion of Linux header for PCI > > devtools/checkpatches.sh | 8 ++ > drivers/bus/pci/linux/pci_init.h | 18 ---- > drivers/bus/pci/linux/pci_uio.c | 32 +----- > drivers/bus/pci/linux/pci_vfio.c | 142 ++++++------------------- > drivers/bus/pci/pci_common.c | 49 ++++++++- > drivers/bus/pci/rte_bus_pci.h | 35 ++++++- > drivers/bus/pci/version.map | 2 + > drivers/crypto/virtio/virtio_pci.c | 67 ++++-------- > drivers/event/dlb2/pf/dlb2_main.c | 156 ++++++++-------------------- > drivers/net/bnx2x/bnx2x.c | 86 ++++++++------- > drivers/net/bnx2x/bnx2x.h | 46 -------- > drivers/net/cxgbe/base/adapter.h | 31 +----- > drivers/net/gve/gve_ethdev.c | 46 +------- > drivers/net/gve/gve_ethdev.h | 14 +-- > drivers/net/hns3/hns3_ethdev_vf.c | 109 +++---------------- > drivers/net/ngbe/base/ngbe_hw.c | 20 +--- > drivers/net/ngbe/base/ngbe_osdep.h | 3 - > drivers/net/virtio/virtio_pci.c | 131 ++++------------------- > drivers/vdpa/ifc/base/ifcvf_osdep.h | 4 +- > lib/pci/rte_pci.h | 77 +++++++++++++- > 20 files changed, 346 insertions(+), 730 deletions(-) > > -- > 2.41.0 > > --=20 *Adam Hassick* Senior Developer UNH InterOperability Lab ahassick@iol.unh.edu iol.unh.edu +1 (603) 475-8248 --000000000000935866060385c38e Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
Recheck-request: iol-sample-apps-testing,

This is a test email of the retesting framework, please ignore.

On Thu, Aug 3, 2023 at 3:50=E2=80=AFAM David Marchand <david.marchand@redhat.com> wrote:
Rather than rely on L= inux headers to find some PCI(e) standard constants
or reinvent the same PCI capability helper, this series complements the
pci library and the pci bus driver.
PCI drivers can then use OS agnostic macros and helpers.

WARNING: this is only compile tested.

--
David Marchand

David Marchand (14):
=C2=A0 drivers: remove duplicated PCI master control
=C2=A0 bus/pci: add const to some experimental API
=C2=A0 bus/pci: find PCI capability
=C2=A0 pci: define some capability constants
=C2=A0 pci: define some MSIX constants
=C2=A0 pci: define some command constants
=C2=A0 pci: define some BAR constants
=C2=A0 pci: define some PM constants
=C2=A0 pci: define some PCIe constants
=C2=A0 pci: define some extended capability constants
=C2=A0 pci: define some ACS constants
=C2=A0 pci: define some PRI constants
=C2=A0 pci: define some AER constants
=C2=A0 devtools: forbid inclusion of Linux header for PCI

=C2=A0devtools/checkpatches.sh=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 |= =C2=A0 =C2=A08 ++
=C2=A0drivers/bus/pci/linux/pci_init.h=C2=A0 =C2=A0 |=C2=A0 18 ----
=C2=A0drivers/bus/pci/linux/pci_uio.c=C2=A0 =C2=A0 =C2=A0|=C2=A0 32 +-----<= br> =C2=A0drivers/bus/pci/linux/pci_vfio.c=C2=A0 =C2=A0 | 142 ++++++-----------= --------
=C2=A0drivers/bus/pci/pci_common.c=C2=A0 =C2=A0 =C2=A0 =C2=A0 |=C2=A0 49 ++= ++++++-
=C2=A0drivers/bus/pci/rte_bus_pci.h=C2=A0 =C2=A0 =C2=A0 =C2=A0|=C2=A0 35 ++= ++++-
=C2=A0drivers/bus/pci/version.map=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0|=C2=A0 = =C2=A02 +
=C2=A0drivers/crypto/virtio/virtio_pci.c=C2=A0 |=C2=A0 67 ++++--------
=C2=A0drivers/event/dlb2/pf/dlb2_main.c=C2=A0 =C2=A0| 156 ++++++++---------= -----------
=C2=A0drivers/net/bnx2x/bnx2x.c=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0|= =C2=A0 86 ++++++++-------
=C2=A0drivers/net/bnx2x/bnx2x.h=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0|= =C2=A0 46 --------
=C2=A0drivers/net/cxgbe/base/adapter.h=C2=A0 =C2=A0 |=C2=A0 31 +-----
=C2=A0drivers/net/gve/gve_ethdev.c=C2=A0 =C2=A0 =C2=A0 =C2=A0 |=C2=A0 46 +-= ------
=C2=A0drivers/net/gve/gve_ethdev.h=C2=A0 =C2=A0 =C2=A0 =C2=A0 |=C2=A0 14 +-= -
=C2=A0drivers/net/hns3/hns3_ethdev_vf.c=C2=A0 =C2=A0| 109 +++--------------= --
=C2=A0drivers/net/ngbe/base/ngbe_hw.c=C2=A0 =C2=A0 =C2=A0|=C2=A0 20 +--- =C2=A0drivers/net/ngbe/base/ngbe_osdep.h=C2=A0 |=C2=A0 =C2=A03 -
=C2=A0drivers/net/virtio/virtio_pci.c=C2=A0 =C2=A0 =C2=A0| 131 ++++--------= -----------
=C2=A0drivers/vdpa/ifc/base/ifcvf_osdep.h |=C2=A0 =C2=A04 +-
=C2=A0lib/pci/rte_pci.h=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 =C2=A0|=C2=A0 77 +++++++++++++-
=C2=A020 files changed, 346 insertions(+), 730 deletions(-)

--
2.41.0



--
Adam Hassick
Senior Developer
UNH InterOperability Lab
+1 (603) 475-8248
--000000000000935866060385c38e--