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 B3ECB430D2; Tue, 22 Aug 2023 18:08:56 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id A46EF4021D; Tue, 22 Aug 2023 18:08:56 +0200 (CEST) Received: from mail-yw1-f175.google.com (mail-yw1-f175.google.com [209.85.128.175]) by mails.dpdk.org (Postfix) with ESMTP id A9AC040041 for ; Tue, 22 Aug 2023 18:08:54 +0200 (CEST) Received: by mail-yw1-f175.google.com with SMTP id 00721157ae682-58d31f142eeso50888517b3.0 for ; Tue, 22 Aug 2023 09:08:54 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=iol.unh.edu; s=unh-iol; t=1692720534; x=1693325334; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date:message-id:reply-to; bh=mfZZvITpxxgPjCD8kAkR3jjFO1Tz79i0CFkDUwy4M5I=; b=aRyHTe3fvo+twyuiJ0JOXMJ9IDSHLKMw7cz4YxPnFUqZMd/s/ONWJfIFhQcXX78PSG pRGQI/LW/+ycfh/avL//0uPYxDMP651bw5z4cqAHrEsTkhkZaAWy4OFfICF7+HeLZ0Xr CnjuPDvU+WSUGQoeb6Iq8AOQHmnnhwf9WynbU= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1692720534; x=1693325334; 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=mfZZvITpxxgPjCD8kAkR3jjFO1Tz79i0CFkDUwy4M5I=; b=cSzI3EPf57mprHMUWXSvwgI263hTdbGMwDP2z4TteV1T2mISfinQWIySuZSH+p/JmL 63ppvSqypjj5owL4vUxhnkFGM1BBcQua6iHUlGbOu3LBCldaM9TERSddycUGcZIwYvnT OI7a8vyBL1KL1B2xQu28L8LGu+GHhqvI/vRguR73DCZSjxPARJ4ZR8RJtleDLiZzVrqg cKAwjnMHcY5pcUZ9k9o9KsW2Nc/C4IIfoBkaSIn4+Xivwet7MxxzpMLpyL+/xWSntAuB xYW5mjtbkHG1hhHQJZBX/Y5pAv1m/uLnLo+EBT+P9Tv8yZccjW+JWeIAEuh6UPz8XjsR myuw== X-Gm-Message-State: AOJu0Ywpfk0bGKFXojfDf7Ir61EiBG2V9bEAuuBzSVZN4FhIdwTOdfkR k55Ao5n59K39O0XFCOciukevUOsWGu+0ElP8pUFuFoQa/j+EUxVX9ZvlEA== X-Google-Smtp-Source: AGHT+IE/3FztYLighqthejf9TSBCzrLdT9kWLKbcnWT1dEvu9mA1jc8sd7S2Rv0KrXFAEzKfidkNghJEgmM7CjL0xEc= X-Received: by 2002:a81:7246:0:b0:58a:4b39:b62d with SMTP id n67-20020a817246000000b0058a4b39b62dmr9625716ywc.29.1692720533945; Tue, 22 Aug 2023 09:08:53 -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:09:21 -0400 Message-ID: Subject: Re: [PATCH 00/14] Cleanup PCI(e) drivers To: David Marchand Cc: dev@dpdk.org Content-Type: multipart/alternative; boundary="000000000000215736060385366e" 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 --000000000000215736060385366e 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. Testing the behavior of replying to a superseded patch. 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 --000000000000215736060385366e 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. Testi= ng the behavior of replying to a superseded patch.

On Thu, Aug 3, = 2023 at 3:50=E2=80=AFAM David Marchand <david.marchand@redhat.com> 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):
=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
--000000000000215736060385366e--