[v1] vfio: Remove 'vfio-platform'

[PATCH for-10.2 0/4] vfio: Remove 'vfio-platform'

Cédric Le Goater posted 4 patches 3 months, 2 weeks ago

Diff against v2
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20250731121947.1346927-1-clg@redhat.com

Maintainers: Paolo Bonzini <pbonzini@redhat.com>, Peter Maydell <peter.maydell@linaro.org>, Alex Williamson <alex.williamson@redhat.com>, "Cédric Le Goater" <clg@redhat.com>

There is a newer version of this series

docs/about/deprecated.rst             |  25 -
docs/about/removed-features.rst       |  25 +
docs/devel/kconfig.rst                |   2 -
{include/hw => hw}/vfio/vfio-region.h |   0
include/hw/vfio/vfio-amd-xgbe.h       |  46 --
include/hw/vfio/vfio-calxeda-xgmac.h  |  43 --
include/hw/vfio/vfio-device.h         |   2 +-
include/hw/vfio/vfio-platform.h       |  78 ---
hw/arm/virt.c                         |   5 -
hw/core/sysbus-fdt.c                  | 383 --------------
hw/vfio/amd-xgbe.c                    |  61 ---
hw/vfio/calxeda-xgmac.c               |  61 ---
hw/vfio/platform.c                    | 716 --------------------------
hw/arm/Kconfig                        |   3 -
hw/vfio/Kconfig                       |  16 -
hw/vfio/meson.build                   |   3 -
hw/vfio/trace-events                  |  11 -
17 files changed, 26 insertions(+), 1454 deletions(-)
rename {include/hw => hw}/vfio/vfio-region.h (100%)
delete mode 100644 include/hw/vfio/vfio-amd-xgbe.h
delete mode 100644 include/hw/vfio/vfio-calxeda-xgmac.h
delete mode 100644 include/hw/vfio/vfio-platform.h
delete mode 100644 hw/vfio/amd-xgbe.c
delete mode 100644 hw/vfio/calxeda-xgmac.c
delete mode 100644 hw/vfio/platform.c

Expand all Fold all

[PATCH for-10.2 0/4] vfio: Remove 'vfio-platform'

Posted by Cédric Le Goater 3 months, 2 weeks ago

Hello, 

The VFIO_PLATFORM device type has been deprecated in the QEMU 10.0
timeframe. QEMU 10.2 is the time for removal.

The last patch moves vfio-region.h under hw/vfio since it no longer
needs to be a public VFIO interface.

Thanks,

C.


Cédric Le Goater (4):
  vfio: Remove 'vfio-amd-xgbe' device
  vfio: Remove 'vfio-calxeda-xgmac' device
  vfio: Remove 'vfio-platform'
  vfio: Move vfio-region.h under hw/vfio/

 docs/about/deprecated.rst             |  25 -
 docs/about/removed-features.rst       |  25 +
 docs/devel/kconfig.rst                |   2 -
 {include/hw => hw}/vfio/vfio-region.h |   0
 include/hw/vfio/vfio-amd-xgbe.h       |  46 --
 include/hw/vfio/vfio-calxeda-xgmac.h  |  43 --
 include/hw/vfio/vfio-device.h         |   2 +-
 include/hw/vfio/vfio-platform.h       |  78 ---
 hw/arm/virt.c                         |   5 -
 hw/core/sysbus-fdt.c                  | 383 --------------
 hw/vfio/amd-xgbe.c                    |  61 ---
 hw/vfio/calxeda-xgmac.c               |  61 ---
 hw/vfio/platform.c                    | 716 --------------------------
 hw/arm/Kconfig                        |   3 -
 hw/vfio/Kconfig                       |  16 -
 hw/vfio/meson.build                   |   3 -
 hw/vfio/trace-events                  |  11 -
 17 files changed, 26 insertions(+), 1454 deletions(-)
 rename {include/hw => hw}/vfio/vfio-region.h (100%)
 delete mode 100644 include/hw/vfio/vfio-amd-xgbe.h
 delete mode 100644 include/hw/vfio/vfio-calxeda-xgmac.h
 delete mode 100644 include/hw/vfio/vfio-platform.h
 delete mode 100644 hw/vfio/amd-xgbe.c
 delete mode 100644 hw/vfio/calxeda-xgmac.c
 delete mode 100644 hw/vfio/platform.c

-- 
2.50.1

Re: [PATCH for-10.2 0/4] vfio: Remove 'vfio-platform'

Posted by Eric Auger 3 months, 1 week ago

Hi Cédric,

On 7/31/25 2:19 PM, Cédric Le Goater wrote:
> Hello, 
>
> The VFIO_PLATFORM device type has been deprecated in the QEMU 10.0
> timeframe. QEMU 10.2 is the time for removal.
>
> The last patch moves vfio-region.h under hw/vfio since it no longer
> needs to be a public VFIO interface.
"tout ça... pour ça!" as we say in french ;-)

Reviewed-by: Eric Auger <eric.auger@redhat.com>

Eric
> Thanks,
>
> C.
>
>
> Cédric Le Goater (4):
>   vfio: Remove 'vfio-amd-xgbe' device
>   vfio: Remove 'vfio-calxeda-xgmac' device
>   vfio: Remove 'vfio-platform'
>   vfio: Move vfio-region.h under hw/vfio/
>
>  docs/about/deprecated.rst             |  25 -
>  docs/about/removed-features.rst       |  25 +
>  docs/devel/kconfig.rst                |   2 -
>  {include/hw => hw}/vfio/vfio-region.h |   0
>  include/hw/vfio/vfio-amd-xgbe.h       |  46 --
>  include/hw/vfio/vfio-calxeda-xgmac.h  |  43 --
>  include/hw/vfio/vfio-device.h         |   2 +-
>  include/hw/vfio/vfio-platform.h       |  78 ---
>  hw/arm/virt.c                         |   5 -
>  hw/core/sysbus-fdt.c                  | 383 --------------
>  hw/vfio/amd-xgbe.c                    |  61 ---
>  hw/vfio/calxeda-xgmac.c               |  61 ---
>  hw/vfio/platform.c                    | 716 --------------------------
>  hw/arm/Kconfig                        |   3 -
>  hw/vfio/Kconfig                       |  16 -
>  hw/vfio/meson.build                   |   3 -
>  hw/vfio/trace-events                  |  11 -
>  17 files changed, 26 insertions(+), 1454 deletions(-)
>  rename {include/hw => hw}/vfio/vfio-region.h (100%)
>  delete mode 100644 include/hw/vfio/vfio-amd-xgbe.h
>  delete mode 100644 include/hw/vfio/vfio-calxeda-xgmac.h
>  delete mode 100644 include/hw/vfio/vfio-platform.h
>  delete mode 100644 hw/vfio/amd-xgbe.c
>  delete mode 100644 hw/vfio/calxeda-xgmac.c
>  delete mode 100644 hw/vfio/platform.c
>

Re: [PATCH for-10.2 0/4] vfio: Remove 'vfio-platform'

Posted by Cédric Le Goater 3 months, 1 week ago

Hello Eric,

On 8/6/25 14:51, Eric Auger wrote:
> Hi Cédric,
> 
> On 7/31/25 2:19 PM, Cédric Le Goater wrote:
>> Hello,
>>
>> The VFIO_PLATFORM device type has been deprecated in the QEMU 10.0
>> timeframe. QEMU 10.2 is the time for removal.
>>
>> The last patch moves vfio-region.h under hw/vfio since it no longer
>> needs to be a public VFIO interface.
> "tout ça... pour ça!" as we say in french ;-)

It was not for nothing !

vfio-platform explored device passthrough on DT platforms.
It had limitations which were difficult to address without
common FW specifications to expose devices. The result is
neither good or bad, it had its time.

However, the wrong choice would be to keep unused code in an
upstream project. This would be a burden for the maintainer
and the overall community.

> Reviewed-by: Eric Auger <eric.auger@redhat.com>

Thanks,

C.

Re: [PATCH for-10.2 0/4] vfio: Remove 'vfio-platform'

Posted by Eric Auger 3 months, 1 week ago


On 8/6/25 5:54 PM, Cédric Le Goater wrote:
> Hello Eric,
>
> On 8/6/25 14:51, Eric Auger wrote:
>> Hi Cédric,
>>
>> On 7/31/25 2:19 PM, Cédric Le Goater wrote:
>>> Hello,
>>>
>>> The VFIO_PLATFORM device type has been deprecated in the QEMU 10.0
>>> timeframe. QEMU 10.2 is the time for removal.
>>>
>>> The last patch moves vfio-region.h under hw/vfio since it no longer
>>> needs to be a public VFIO interface.
>> "tout ça... pour ça!" as we say in french ;-)
>
> It was not for nothing !
>
> vfio-platform explored device passthrough on DT platforms.
> It had limitations which were difficult to address without
> common FW specifications to expose devices. The result is
> neither good or bad, it had its time.
>
> However, the wrong choice would be to keep unused code in an
> upstream project. This would be a burden for the maintainer
> and the overall community.

Yes I totally agree. Especially in this situation where we cannot test
anymore.

Thanks

Eric
>
>> Reviewed-by: Eric Auger <eric.auger@redhat.com>
>
>
> Thanks,
>
> C.
>

Re: [PATCH for-10.2 0/4] vfio: Remove 'vfio-platform'

Posted by Alex Williamson 3 months, 2 weeks ago

On Thu, 31 Jul 2025 14:19:43 +0200
Cédric Le Goater <clg@redhat.com> wrote:

> Hello, 
> 
> The VFIO_PLATFORM device type has been deprecated in the QEMU 10.0
> timeframe. QEMU 10.2 is the time for removal.
> 
> The last patch moves vfio-region.h under hw/vfio since it no longer
> needs to be a public VFIO interface.
> 
> Thanks,
> 
> C.
> 
> 
> Cédric Le Goater (4):
>   vfio: Remove 'vfio-amd-xgbe' device
>   vfio: Remove 'vfio-calxeda-xgmac' device
>   vfio: Remove 'vfio-platform'
>   vfio: Move vfio-region.h under hw/vfio/
> 
>  docs/about/deprecated.rst             |  25 -
>  docs/about/removed-features.rst       |  25 +
>  docs/devel/kconfig.rst                |   2 -
>  {include/hw => hw}/vfio/vfio-region.h |   0
>  include/hw/vfio/vfio-amd-xgbe.h       |  46 --
>  include/hw/vfio/vfio-calxeda-xgmac.h  |  43 --
>  include/hw/vfio/vfio-device.h         |   2 +-
>  include/hw/vfio/vfio-platform.h       |  78 ---
>  hw/arm/virt.c                         |   5 -
>  hw/core/sysbus-fdt.c                  | 383 --------------
>  hw/vfio/amd-xgbe.c                    |  61 ---
>  hw/vfio/calxeda-xgmac.c               |  61 ---
>  hw/vfio/platform.c                    | 716 --------------------------
>  hw/arm/Kconfig                        |   3 -
>  hw/vfio/Kconfig                       |  16 -
>  hw/vfio/meson.build                   |   3 -
>  hw/vfio/trace-events                  |  11 -
>  17 files changed, 26 insertions(+), 1454 deletions(-)
>  rename {include/hw => hw}/vfio/vfio-region.h (100%)
>  delete mode 100644 include/hw/vfio/vfio-amd-xgbe.h
>  delete mode 100644 include/hw/vfio/vfio-calxeda-xgmac.h
>  delete mode 100644 include/hw/vfio/vfio-platform.h
>  delete mode 100644 hw/vfio/amd-xgbe.c
>  delete mode 100644 hw/vfio/calxeda-xgmac.c
>  delete mode 100644 hw/vfio/platform.c
> 

Reviewed-by: Alex Williamson <alex.williamson@redhat.com>