Re: [PATCH for-7.0] virtio: fix --enable-vhost-user build on non-Linux

[Jason Wang]

On Tue, Mar 29, 2022 at 3:36 PM Paolo Bonzini <pbonzini@redhat.com> wrote:
>
> The vhost-shadow-virtqueue.c build requires include files from
> linux-headers/, so it cannot be built on non-Linux systems.

We need decouple this dependency in the future.

> Fortunately it is only needed by vhost-vdpa, so move it there.

Acked-by: Jason Wang <jasowang@redhat.com>

>
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> ---
>  hw/virtio/meson.build | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/hw/virtio/meson.build b/hw/virtio/meson.build
> index 6047670804..67dc77e00f 100644
> --- a/hw/virtio/meson.build
> +++ b/hw/virtio/meson.build
> @@ -11,9 +11,9 @@ softmmu_ss.add(when: 'CONFIG_ALL', if_true: 
> files('vhost-stub.c'))
>
>  virtio_ss = ss.source_set()
>  virtio_ss.add(files('virtio.c'))
> -virtio_ss.add(when: 'CONFIG_VHOST', if_true: files('vhost.c', 
> 'vhost-backend.c', 'vhost-shadow-virtqueue.c', 'vhost-iova-tree.c'))
> +virtio_ss.add(when: 'CONFIG_VHOST', if_true: files('vhost.c', 
> 'vhost-backend.c', 'vhost-iova-tree.c'))
>  virtio_ss.add(when: 'CONFIG_VHOST_USER', if_true: files('vhost-user.c'))
> -virtio_ss.add(when: 'CONFIG_VHOST_VDPA', if_true: files('vhost-vdpa.c'))
> +virtio_ss.add(when: 'CONFIG_VHOST_VDPA', if_true: 
> files('vhost-shadow-virtqueue.c', 'vhost-vdpa.c'))
>  virtio_ss.add(when: 'CONFIG_VIRTIO_BALLOON', if_true: 
> files('virtio-balloon.c'))
>  virtio_ss.add(when: 'CONFIG_VIRTIO_CRYPTO', if_true: 
> files('virtio-crypto.c'))
>  virtio_ss.add(when: ['CONFIG_VIRTIO_CRYPTO', 'CONFIG_VIRTIO_PCI'], if_true: 
> files('virtio-crypto-pci.c'))
> --
> 2.35.1
>
>