mirror of
https://github.com/Motorhead1991/qemu.git
synced 2025-08-11 03:24:58 -06:00
vfio-pci: Don't use kvm_irqchip_in_kernel
kvm_irqchip_in_kernel() has an architecture specific meaning, so we shouldn't be using it to determine whether to enabled KVM INTx bypass. kvm_irqfds_enabled() seems most appropriate. Also use this to protect our other call to kvm_check_extension() as that explodes when KVM isn't enabled. Signed-off-by: Alex Williamson <alex.williamson@redhat.com> Cc: qemu-stable@nongnu.org
This commit is contained in:
parent
1c97e303d4
commit
d281084d3e
1 changed files with 3 additions and 2 deletions
|
@ -275,7 +275,7 @@ static void vfio_enable_intx_kvm(VFIODevice *vdev)
|
||||||
int ret, argsz;
|
int ret, argsz;
|
||||||
int32_t *pfd;
|
int32_t *pfd;
|
||||||
|
|
||||||
if (!kvm_irqchip_in_kernel() ||
|
if (!kvm_irqfds_enabled() ||
|
||||||
vdev->intx.route.mode != PCI_INTX_ENABLED ||
|
vdev->intx.route.mode != PCI_INTX_ENABLED ||
|
||||||
!kvm_check_extension(kvm_state, KVM_CAP_IRQFD_RESAMPLE)) {
|
!kvm_check_extension(kvm_state, KVM_CAP_IRQFD_RESAMPLE)) {
|
||||||
return;
|
return;
|
||||||
|
@ -438,7 +438,8 @@ static int vfio_enable_intx(VFIODevice *vdev)
|
||||||
* Only conditional to avoid generating error messages on platforms
|
* Only conditional to avoid generating error messages on platforms
|
||||||
* where we won't actually use the result anyway.
|
* where we won't actually use the result anyway.
|
||||||
*/
|
*/
|
||||||
if (kvm_check_extension(kvm_state, KVM_CAP_IRQFD_RESAMPLE)) {
|
if (kvm_irqfds_enabled() &&
|
||||||
|
kvm_check_extension(kvm_state, KVM_CAP_IRQFD_RESAMPLE)) {
|
||||||
vdev->intx.route = pci_device_route_intx_to_irq(&vdev->pdev,
|
vdev->intx.route = pci_device_route_intx_to_irq(&vdev->pdev,
|
||||||
vdev->intx.pin);
|
vdev->intx.pin);
|
||||||
}
|
}
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue