Commit 090b7aff authored by Gregory Haskins's avatar Gregory Haskins Committed by Avi Kivity
Browse files

KVM: make io_bus interface more robust 



Today kvm_io_bus_regsiter_dev() returns void and will internally BUG_ON
if it fails.  We want to create dynamic MMIO/PIO entries driven from
userspace later in the series, so we need to enhance the code to be more
robust with the following changes:

   1) Add a return value to the registration function
   2) Fix up all the callsites to check the return code, handle any
      failures, and percolate the error up to the caller.
   3) Add an unregister function that collapses holes in the array

Signed-off-by: default avatarGregory Haskins <ghaskins@novell.com>
Acked-by: default avatarMichael S. Tsirkin <mst@redhat.com>
Signed-off-by: default avatarAvi Kivity <avi@redhat.com>
parent fef07aae

arch/x86/kvm/i8254.c

+18 −2
Original line number Diff line number Diff line
@@ -605,6 +605,7 @@ struct kvm_pit *kvm_create_pit(struct kvm *kvm, u32 flags) 
{

	struct kvm_pit *pit;

	struct kvm_kpit_state *pit_state;

	int ret;



	pit = kzalloc(sizeof(struct kvm_pit), GFP_KERNEL);

	if (!pit)
@@ -639,14 +640,29 @@ struct kvm_pit *kvm_create_pit(struct kvm *kvm, u32 flags) 
	kvm_register_irq_mask_notifier(kvm, 0, &pit->mask_notifier);



	kvm_iodevice_init(&pit->dev, &pit_dev_ops);

	__kvm_io_bus_register_dev(&kvm->pio_bus, &pit->dev);

	ret = __kvm_io_bus_register_dev(&kvm->pio_bus, &pit->dev);

	if (ret < 0)

		goto fail;



	if (flags & KVM_PIT_SPEAKER_DUMMY) {

		kvm_iodevice_init(&pit->speaker_dev, &speaker_dev_ops);

		__kvm_io_bus_register_dev(&kvm->pio_bus, &pit->speaker_dev);

		ret = __kvm_io_bus_register_dev(&kvm->pio_bus,

						&pit->speaker_dev);

		if (ret < 0)

			goto fail_unregister;

	}



	return pit;



fail_unregister:

	__kvm_io_bus_unregister_dev(&kvm->pio_bus, &pit->dev);



fail:

	if (pit->irq_source_id >= 0)

		kvm_free_irq_source_id(kvm, pit->irq_source_id);



	kfree(pit);

	return NULL;

}



void kvm_free_pit(struct kvm *kvm)

arch/x86/kvm/i8259.c

+8 −1
Original line number Diff line number Diff line
@@ -539,6 +539,8 @@ static const struct kvm_io_device_ops picdev_ops = { 
struct kvm_pic *kvm_create_pic(struct kvm *kvm)

{

	struct kvm_pic *s;

	int ret;



	s = kzalloc(sizeof(struct kvm_pic), GFP_KERNEL);

	if (!s)

		return NULL;
@@ -555,6 +557,11 @@ struct kvm_pic *kvm_create_pic(struct kvm *kvm) 
	 * Initialize PIO device

	 */

	kvm_iodevice_init(&s->dev, &picdev_ops);

	kvm_io_bus_register_dev(kvm, &kvm->pio_bus, &s->dev);

	ret = kvm_io_bus_register_dev(kvm, &kvm->pio_bus, &s->dev);

	if (ret < 0) {

		kfree(s);

		return NULL;

	}



	return s;

}

include/linux/kvm_host.h

+7 −3
Original line number Diff line number Diff line
@@ -64,9 +64,13 @@ int kvm_io_bus_write(struct kvm_io_bus *bus, gpa_t addr, int len, 
		     const void *val);

int kvm_io_bus_read(struct kvm_io_bus *bus, gpa_t addr, int len,

		    void *val);

void __kvm_io_bus_register_dev(struct kvm_io_bus *bus,

int __kvm_io_bus_register_dev(struct kvm_io_bus *bus,

			       struct kvm_io_device *dev);

void kvm_io_bus_register_dev(struct kvm *kvm, struct kvm_io_bus *bus,

int kvm_io_bus_register_dev(struct kvm *kvm, struct kvm_io_bus *bus,

			    struct kvm_io_device *dev);

void __kvm_io_bus_unregister_dev(struct kvm_io_bus *bus,

				 struct kvm_io_device *dev);

void kvm_io_bus_unregister_dev(struct kvm *kvm, struct kvm_io_bus *bus,

			       struct kvm_io_device *dev);



struct kvm_vcpu {

virt/kvm/coalesced_mmio.c

+6 −2
Original line number Diff line number Diff line
@@ -92,6 +92,7 @@ static const struct kvm_io_device_ops coalesced_mmio_ops = { 
int kvm_coalesced_mmio_init(struct kvm *kvm)

{

	struct kvm_coalesced_mmio_dev *dev;

	int ret;



	dev = kzalloc(sizeof(struct kvm_coalesced_mmio_dev), GFP_KERNEL);

	if (!dev)
@@ -100,9 +101,12 @@ int kvm_coalesced_mmio_init(struct kvm *kvm) 
	kvm_iodevice_init(&dev->dev, &coalesced_mmio_ops);

	dev->kvm = kvm;

	kvm->coalesced_mmio_dev = dev;

	kvm_io_bus_register_dev(kvm, &kvm->mmio_bus, &dev->dev);



	return 0;

	ret = kvm_io_bus_register_dev(kvm, &kvm->mmio_bus, &dev->dev);

	if (ret < 0)

		kfree(dev);



	return ret;

}



int kvm_vm_ioctl_register_coalesced_mmio(struct kvm *kvm,

virt/kvm/ioapic.c

+6 −2
Original line number Diff line number Diff line
@@ -342,6 +342,7 @@ static const struct kvm_io_device_ops ioapic_mmio_ops = { 
int kvm_ioapic_init(struct kvm *kvm)

{

	struct kvm_ioapic *ioapic;

	int ret;



	ioapic = kzalloc(sizeof(struct kvm_ioapic), GFP_KERNEL);

	if (!ioapic)
@@ -350,7 +351,10 @@ int kvm_ioapic_init(struct kvm *kvm) 
	kvm_ioapic_reset(ioapic);

	kvm_iodevice_init(&ioapic->dev, &ioapic_mmio_ops);

	ioapic->kvm = kvm;

	kvm_io_bus_register_dev(kvm, &kvm->mmio_bus, &ioapic->dev);

	return 0;

	ret = kvm_io_bus_register_dev(kvm, &kvm->mmio_bus, &ioapic->dev);

	if (ret < 0)

		kfree(ioapic);



	return ret;

}

CRA Git | Maintained and supported by SUSTech CRA and CCSE