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@xxxxxxxxxx>
> Signed-off-by: Max Gurtovoy <mgurtovoy@xxxxxxxxxx>
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@xxxxxxxxxxxxxxxxxxxxxxxxxx
https://lists.linuxfoundation.org/mailman/listinfo/virtualization
