Re: [PATCH v2 5/8] vfio: Change vfio_external_check_extension() to vfio_file_enforced_coherent()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Subject: Re: [PATCH v2 5/8] vfio: Change vfio_external_check_extension() to vfio_file_enforced_coherent()
From: Christoph Hellwig <hch@xxxxxx>
Date: Thu, 21 Apr 2022 07:41:16 +0200
Cc: Alex Williamson <alex.williamson@xxxxxxxxxx>, Cornelia Huck <cohuck@xxxxxxxxxx>, kvm@xxxxxxxxxxxxxxx, Paolo Bonzini <pbonzini@xxxxxxxxxx>, Eric Auger <eric.auger@xxxxxxxxxx>, Christoph Hellwig <hch@xxxxxx>, Kevin Tian <kevin.tian@xxxxxxxxx>, Yi Liu <yi.l.liu@xxxxxxxxx>
In-reply-to: <5-v2-6a528653a750+1578a-vfio_kvm_no_group_jgg@nvidia.com>
References: <0-v2-6a528653a750+1578a-vfio_kvm_no_group_jgg@nvidia.com> <5-v2-6a528653a750+1578a-vfio_kvm_no_group_jgg@nvidia.com>
User-agent: Mutt/1.5.17 (2007-11-01)

I can see why a specific error might be nice for some cases and am
open to that, but as a simple transformation this already looks good:

Reviewed-by: Christoph Hellwig <hch@xxxxxx>

Follow-Ups:
- RE: [PATCH v2 5/8] vfio: Change vfio_external_check_extension() to vfio_file_enforced_coherent()
  - From: Tian, Kevin

References:
- [PATCH v2 0/8] Remove vfio_group from the struct file facing VFIO API
  - From: Jason Gunthorpe
- [PATCH v2 5/8] vfio: Change vfio_external_check_extension() to vfio_file_enforced_coherent()
  - From: Jason Gunthorpe

Prev by Date: Re: [PATCH v2 4/8] vfio: Remove vfio_external_group_match_file()
Next by Date: Re: [PATCH v2 6/8] vfio: Change vfio_group_set_kvm() to vfio_file_set_kvm()
Previous by thread: RE: [PATCH v2 5/8] vfio: Change vfio_external_check_extension() to vfio_file_enforced_coherent()
Next by thread: RE: [PATCH v2 5/8] vfio: Change vfio_external_check_extension() to vfio_file_enforced_coherent()
Index(es):
- Date
- Thread

[Index of Archives] [KVM ARM] [KVM ia64] [KVM ppc] [Virtualization Tools] [Spice Development] [Libvirt] [Libvirt Users] [Linux USB Devel] [Linux Audio Users] [Yosemite Questions] [Linux Kernel] [Linux SCSI] [XFree86]