[PATCH] vhost: remove dependency on PCI library

[PATCH] vhost: remove dependency on PCI library

[David Marchand]

Since the async datapath code switched to dmadev, there is no reason to
depend on the pci library.

Fixes: 53d3f4778c1d ("vhost: integrate dmadev in asynchronous data-path")

Signed-off-by: David Marchand <david.marchand@redhat.com>
---
 lib/vhost/meson.build | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/vhost/meson.build b/lib/vhost/meson.build
index 6a24981d10..64698182cd 100644
--- a/lib/vhost/meson.build
+++ b/lib/vhost/meson.build
@@ -44,4 +44,4 @@ headers = files(
 driver_sdk_headers = files(
         'vdpa_driver.h',
 )
-deps += ['ethdev', 'cryptodev', 'hash', 'pci', 'dmadev']
+deps += ['ethdev', 'cryptodev', 'hash', 'dmadev']
-- 
2.54.0

[PATCH v2] vhost: remove dependency on PCI library

[David Marchand]

Since the async datapath code switched to dmadev, there is no reason to
depend on the pci library.

Fixes: 53d3f4778c1d ("vhost: integrate dmadev in asynchronous data-path")

Signed-off-by: David Marchand <david.marchand@redhat.com>
---
Changes since v1:
- fixed "internal" dependencies of examples,

---
 examples/vdpa/meson.build  | 1 +
 examples/vhost/meson.build | 1 +
 lib/vhost/meson.build      | 2 +-
 3 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/examples/vdpa/meson.build b/examples/vdpa/meson.build
index 45f8a6fa31..3a319dd539 100644
--- a/examples/vdpa/meson.build
+++ b/examples/vdpa/meson.build
@@ -12,6 +12,7 @@ if not is_linux
 endif
 
 deps += 'vhost'
+deps += 'pci'
 allow_experimental_apis = true
 sources = files(
         'main.c',
diff --git a/examples/vhost/meson.build b/examples/vhost/meson.build
index e938be8f45..1cf061c597 100644
--- a/examples/vhost/meson.build
+++ b/examples/vhost/meson.build
@@ -13,6 +13,7 @@ endif
 
 deps += 'vhost'
 deps += 'dmadev'
+deps += 'pci'
 allow_experimental_apis = true
 sources = files(
         'main.c',
diff --git a/lib/vhost/meson.build b/lib/vhost/meson.build
index 6a24981d10..64698182cd 100644
--- a/lib/vhost/meson.build
+++ b/lib/vhost/meson.build
@@ -44,4 +44,4 @@ headers = files(
 driver_sdk_headers = files(
         'vdpa_driver.h',
 )
-deps += ['ethdev', 'cryptodev', 'hash', 'pci', 'dmadev']
+deps += ['ethdev', 'cryptodev', 'hash', 'dmadev']
-- 
2.54.0

Re: [PATCH v2] vhost: remove dependency on PCI library

[Bruce Richardson]

On Tue, Sep 01, 2026 at 09:00:24AM +0200, David Marchand wrote:
> Since the async datapath code switched to dmadev, there is no reason to
> depend on the pci library.
> 
> Fixes: 53d3f4778c1d ("vhost: integrate dmadev in asynchronous data-path")
> 
> Signed-off-by: David Marchand <david.marchand@redhat.com>
> ---
> Changes since v1:
> - fixed "internal" dependencies of examples,
> 
> ---
>  examples/vdpa/meson.build  | 1 +
>  examples/vhost/meson.build | 1 +
>  lib/vhost/meson.build      | 2 +-
>  3 files changed, 3 insertions(+), 1 deletion(-)
> 
No objection to this patch, but rather than changing this now, would we not
be better just waiting to see what happens with the sync datapath itself?
If it gets removed from vhost lib, then a larger cleanup is likely which
should encompass these changes and more.

Re: [PATCH v2] vhost: remove dependency on PCI library

[David Marchand]

On Tue, 1 Sept 2026 at 10:00, Bruce Richardson
<bruce.richardson@intel.com> wrote:
>
> On Tue, Sep 01, 2026 at 09:00:24AM +0200, David Marchand wrote:
> > Since the async datapath code switched to dmadev, there is no reason to
> > depend on the pci library.
> >
> > Fixes: 53d3f4778c1d ("vhost: integrate dmadev in asynchronous data-path")
> >
> > Signed-off-by: David Marchand <david.marchand@redhat.com>
> > ---
> > Changes since v1:
> > - fixed "internal" dependencies of examples,
> >
> > ---
> >  examples/vdpa/meson.build  | 1 +
> >  examples/vhost/meson.build | 1 +
> >  lib/vhost/meson.build      | 2 +-
> >  3 files changed, 3 insertions(+), 1 deletion(-)
> >
> No objection to this patch, but rather than changing this now, would we not
> be better just waiting to see what happens with the sync datapath itself?

*a*sync I guess.

> If it gets removed from vhost lib, then a larger cleanup is likely which
> should encompass these changes and more.

On second thought, I agree, it is not worth having as a simple
(backportable) fix, it is only an internal dependency.

I don't mind merging it in the series
https://patchwork.dpdk.org/project/dpdk/list/?series=39064&state=*


-- 
David Marchand