Re: [PATCH 1/1] virtio: move pci drivers files to a directory

Re: [PATCH 1/1] virtio: move pci drivers files to a directory

[Michael S. Tsirkin]

On Mon, May 15, 2023 at 08:13:38PM +0300, Max Gurtovoy wrote:
> The PCI transport drivers have many files under virtio directory. Move
> them to a dedicated directory to ease on the subsystem maintenance.
> 
> Reviewed-by: Parav Pandit <parav@nvidia.com>
> Signed-off-by: Max Gurtovoy <mgurtovoy@nvidia.com>

The benefit here seems marginal, they all have
virtio_pci prefix, pci/virtio_pci_ just seems redundant. OTOH I
could see a benefit to have transports, core, and a bunch of
misc devices in 3 separate directories.

> ---
>  drivers/virtio/Makefile                          | 7 ++-----
>  drivers/virtio/pci/Makefile                      | 7 +++++++
>  drivers/virtio/{ => pci}/virtio_pci_common.c     | 0
>  drivers/virtio/{ => pci}/virtio_pci_common.h     | 0
>  drivers/virtio/{ => pci}/virtio_pci_legacy.c     | 0
>  drivers/virtio/{ => pci}/virtio_pci_legacy_dev.c | 0
>  drivers/virtio/{ => pci}/virtio_pci_modern.c     | 0
>  drivers/virtio/{ => pci}/virtio_pci_modern_dev.c | 0
>  8 files changed, 9 insertions(+), 5 deletions(-)
>  create mode 100644 drivers/virtio/pci/Makefile
>  rename drivers/virtio/{ => pci}/virtio_pci_common.c (100%)
>  rename drivers/virtio/{ => pci}/virtio_pci_common.h (100%)
>  rename drivers/virtio/{ => pci}/virtio_pci_legacy.c (100%)
>  rename drivers/virtio/{ => pci}/virtio_pci_legacy_dev.c (100%)
>  rename drivers/virtio/{ => pci}/virtio_pci_modern.c (100%)
>  rename drivers/virtio/{ => pci}/virtio_pci_modern_dev.c (100%)
> 
> diff --git a/drivers/virtio/Makefile b/drivers/virtio/Makefile
> index 8e98d24917cc..8c39c0594cea 100644
> --- a/drivers/virtio/Makefile
> +++ b/drivers/virtio/Makefile
> @@ -1,14 +1,11 @@
>  # SPDX-License-Identifier: GPL-2.0
>  obj-$(CONFIG_VIRTIO) += virtio.o virtio_ring.o
>  obj-$(CONFIG_VIRTIO_ANCHOR) += virtio_anchor.o
> -obj-$(CONFIG_VIRTIO_PCI_LIB) += virtio_pci_modern_dev.o
> -obj-$(CONFIG_VIRTIO_PCI_LIB_LEGACY) += virtio_pci_legacy_dev.o
>  obj-$(CONFIG_VIRTIO_MMIO) += virtio_mmio.o
> -obj-$(CONFIG_VIRTIO_PCI) += virtio_pci.o
> -virtio_pci-y := virtio_pci_modern.o virtio_pci_common.o
> -virtio_pci-$(CONFIG_VIRTIO_PCI_LEGACY) += virtio_pci_legacy.o
>  obj-$(CONFIG_VIRTIO_BALLOON) += virtio_balloon.o
>  obj-$(CONFIG_VIRTIO_INPUT) += virtio_input.o
>  obj-$(CONFIG_VIRTIO_VDPA) += virtio_vdpa.o
>  obj-$(CONFIG_VIRTIO_MEM) += virtio_mem.o
>  obj-$(CONFIG_VIRTIO_DMA_SHARED_BUFFER) += virtio_dma_buf.o
> +
> +obj-$(CONFIG_VIRTIO_PCI)		+= pci/
> diff --git a/drivers/virtio/pci/Makefile b/drivers/virtio/pci/Makefile
> new file mode 100644
> index 000000000000..673c7532430a
> --- /dev/null
> +++ b/drivers/virtio/pci/Makefile
> @@ -0,0 +1,7 @@
> +# SPDX-License-Identifier: GPL-2.0-only
> +obj-$(CONFIG_VIRTIO_PCI) += virtio_pci.o
> +obj-$(CONFIG_VIRTIO_PCI_LIB) += virtio_pci_modern_dev.o
> +obj-$(CONFIG_VIRTIO_PCI_LIB_LEGACY) += virtio_pci_legacy_dev.o
> +
> +virtio_pci-y := virtio_pci_modern.o virtio_pci_common.o
> +virtio_pci-$(CONFIG_VIRTIO_PCI_LEGACY) += virtio_pci_legacy.o
> diff --git a/drivers/virtio/virtio_pci_common.c b/drivers/virtio/pci/virtio_pci_common.c
> similarity index 100%
> rename from drivers/virtio/virtio_pci_common.c
> rename to drivers/virtio/pci/virtio_pci_common.c
> diff --git a/drivers/virtio/virtio_pci_common.h b/drivers/virtio/pci/virtio_pci_common.h
> similarity index 100%
> rename from drivers/virtio/virtio_pci_common.h
> rename to drivers/virtio/pci/virtio_pci_common.h
> diff --git a/drivers/virtio/virtio_pci_legacy.c b/drivers/virtio/pci/virtio_pci_legacy.c
> similarity index 100%
> rename from drivers/virtio/virtio_pci_legacy.c
> rename to drivers/virtio/pci/virtio_pci_legacy.c
> diff --git a/drivers/virtio/virtio_pci_legacy_dev.c b/drivers/virtio/pci/virtio_pci_legacy_dev.c
> similarity index 100%
> rename from drivers/virtio/virtio_pci_legacy_dev.c
> rename to drivers/virtio/pci/virtio_pci_legacy_dev.c
> diff --git a/drivers/virtio/virtio_pci_modern.c b/drivers/virtio/pci/virtio_pci_modern.c
> similarity index 100%
> rename from drivers/virtio/virtio_pci_modern.c
> rename to drivers/virtio/pci/virtio_pci_modern.c
> diff --git a/drivers/virtio/virtio_pci_modern_dev.c b/drivers/virtio/pci/virtio_pci_modern_dev.c
> similarity index 100%
> rename from drivers/virtio/virtio_pci_modern_dev.c
> rename to drivers/virtio/pci/virtio_pci_modern_dev.c
> -- 
> 2.18.1

_______________________________________________
Virtualization mailing list
Virtualization@lists.linux-foundation.org
https://lists.linuxfoundation.org/mailman/listinfo/virtualization