vfio: selftests: Clean up <uapi/linux/types.h> includes

[PATCH 0/2] vfio: selftests: Clean up <uapi/linux/types.h> includes

David Matlack posted 2 patches 1 month, 2 weeks ago

Download series mbox

tools/include/linux/types.h                               | 8 ++++++++
.../selftests/vfio/lib/include/libvfio/iova_allocator.h   | 1 -
tools/testing/selftests/vfio/lib/iommu.c                  | 1 -
tools/testing/selftests/vfio/lib/iova_allocator.c         | 1 -
tools/testing/selftests/vfio/lib/vfio_pci_device.c        | 1 -
tools/testing/selftests/vfio/vfio_dma_mapping_test.c      | 1 -
tools/testing/selftests/vfio/vfio_iommufd_setup_test.c    | 1 -
7 files changed, 8 insertions(+), 6 deletions(-)

Expand all Fold all

[PATCH 0/2] vfio: selftests: Clean up <uapi/linux/types.h> includes

Posted by David Matlack 1 month, 2 weeks ago

Small clean up series to eliminate the extra includes of
<uapi/linux/types.h> from various VFIO selftests files. This include is
not causing any problems now, but it is causing benign typedef
redifinitions. Those redifinitions will become a problem when the VFIO
selftests library is built into KVM selftests, since KVM selftests build
with -std=gnu99.

Cc: Yosry Ahmed <yosryahmed@google.com>
Cc: Josh Hilke <jrhilke@google.com>

David Matlack (2):
  tools include: Add definitions for __aligned_{l,b}e64
  vfio: selftests: Drop <uapi/linux/types.h> includes

 tools/include/linux/types.h                               | 8 ++++++++
 .../selftests/vfio/lib/include/libvfio/iova_allocator.h   | 1 -
 tools/testing/selftests/vfio/lib/iommu.c                  | 1 -
 tools/testing/selftests/vfio/lib/iova_allocator.c         | 1 -
 tools/testing/selftests/vfio/lib/vfio_pci_device.c        | 1 -
 tools/testing/selftests/vfio/vfio_dma_mapping_test.c      | 1 -
 tools/testing/selftests/vfio/vfio_iommufd_setup_test.c    | 1 -
 7 files changed, 8 insertions(+), 6 deletions(-)

base-commit: d721f52e31553a848e0e9947ca15a49c5674aef3
-- 
2.52.0.322.g1dd061c0dc-goog

Re: [PATCH 0/2] vfio: selftests: Clean up <uapi/linux/types.h> includes

Posted by Alex Williamson 1 month, 2 weeks ago

On Fri, 19 Dec 2025 23:38:16 +0000
David Matlack <dmatlack@google.com> wrote:

> Small clean up series to eliminate the extra includes of
> <uapi/linux/types.h> from various VFIO selftests files. This include is
> not causing any problems now, but it is causing benign typedef
> redifinitions. Those redifinitions will become a problem when the VFIO
> selftests library is built into KVM selftests, since KVM selftests build
> with -std=gnu99.
> 
> Cc: Yosry Ahmed <yosryahmed@google.com>
> Cc: Josh Hilke <jrhilke@google.com>
> 
> David Matlack (2):
>   tools include: Add definitions for __aligned_{l,b}e64
>   vfio: selftests: Drop <uapi/linux/types.h> includes
> 
>  tools/include/linux/types.h                               | 8 ++++++++
>  .../selftests/vfio/lib/include/libvfio/iova_allocator.h   | 1 -
>  tools/testing/selftests/vfio/lib/iommu.c                  | 1 -
>  tools/testing/selftests/vfio/lib/iova_allocator.c         | 1 -
>  tools/testing/selftests/vfio/lib/vfio_pci_device.c        | 1 -
>  tools/testing/selftests/vfio/vfio_dma_mapping_test.c      | 1 -
>  tools/testing/selftests/vfio/vfio_iommufd_setup_test.c    | 1 -
>  7 files changed, 8 insertions(+), 6 deletions(-)
> 
> 
> base-commit: d721f52e31553a848e0e9947ca15a49c5674aef3

Applied to vfio for-linus branch for v6.19.  Thanks,

Alex