RE: [PATCH v2] KVM: move KVM_CAP_DEVICE_CTRL to the generic check
From: Wang, Wei W
Date: Wed Mar 08 2023 - 08:35:45 EST
On Thursday, February 2, 2023 7:30 AM, Sean Christopherson wrote:
> On Thu, Dec 29, 2022, Wei Wang wrote:
> > KVM_CAP_DEVICE_CTRL allows userspace to check if the kvm_device
> > framework (e.g. KVM_CREATE_DEVICE) is supported by KVM. Move
> > KVM_CAP_DEVICE_CTRL to the generic check for the two reasons:
> > 1) it already supports arch agnostic usages (i.e. KVM_DEV_TYPE_VFIO).
> > For example, userspace VFIO implementation may needs to create
> > KVM_DEV_TYPE_VFIO on x86, riscv, or arm etc. It is simpler to have it
> > checked at the generic code than at each arch's code.
> > 2) KVM_CREATE_DEVICE has been added to the generic code.
>
> I think it's worth adding a link to the v1 patch, as that thread provides the
> context for how KVM ended up unconditionally advertising the feature, e.g.
> instead of trying to be more precise.
>
> Link: https://lore.kernel.org/all/20221215115207.14784-1-
> wei.w.wang@xxxxxxxxx
>
> > Signed-off-by: Wei Wang <wei.w.wang@xxxxxxxxx>
> > ---
>
> Reviewed-by: Sean Christopherson <seanjc@xxxxxxxxxx>
Thanks.
Is this patch ready to be merged along with the link or
need me to re-send to have the link added?