KVM: Clean up unneeded void pointer casts
[linux-2.6.git] / virt / kvm / coalesced_mmio.c
index 03ea280..a6ec206 100644 (file)
@@ -2,6 +2,7 @@
  * KVM coalesced MMIO
  *
  * Copyright (c) 2008 Bull S.A.S.
+ * Copyright 2009 Red Hat, Inc. and/or its affiliates.
  *
  *  Author: Laurent Vivier <Laurent.Vivier@bull.net>
  *
 #include "iodev.h"
 
 #include <linux/kvm_host.h>
+#include <linux/slab.h>
 #include <linux/kvm.h>
 
 #include "coalesced_mmio.h"
 
-static int coalesced_mmio_in_range(struct kvm_io_device *this,
-                                  gpa_t addr, int len, int is_write)
+static inline struct kvm_coalesced_mmio_dev *to_mmio(struct kvm_io_device *dev)
 {
-       struct kvm_coalesced_mmio_dev *dev =
-                               (struct kvm_coalesced_mmio_dev*)this->private;
-       struct kvm_coalesced_mmio_zone *zone;
-       int next;
-       int i;
+       return container_of(dev, struct kvm_coalesced_mmio_dev, dev);
+}
 
-       if (!is_write)
-               return 0;
+static int coalesced_mmio_in_range(struct kvm_coalesced_mmio_dev *dev,
+                                  gpa_t addr, int len)
+{
+       /* is it in a batchable area ?
+        * (addr,len) is fully included in
+        * (zone->addr, zone->size)
+        */
+
+       return (dev->zone.addr <= addr &&
+               addr + len <= dev->zone.addr + dev->zone.size);
+}
 
-       /* kvm->lock is taken by the caller and must be not released before
-         * dev.read/write
-         */
+static int coalesced_mmio_has_room(struct kvm_coalesced_mmio_dev *dev)
+{
+       struct kvm_coalesced_mmio_ring *ring;
+       unsigned avail;
 
        /* Are we able to batch it ? */
 
@@ -36,38 +44,31 @@ static int coalesced_mmio_in_range(struct kvm_io_device *this,
         * check if we don't meet the first used entry
         * there is always one unused entry in the buffer
         */
-
-       next = (dev->kvm->coalesced_mmio_ring->last + 1) %
-                                                       KVM_COALESCED_MMIO_MAX;
-       if (next == dev->kvm->coalesced_mmio_ring->first) {
+       ring = dev->kvm->coalesced_mmio_ring;
+       avail = (ring->first - ring->last - 1) % KVM_COALESCED_MMIO_MAX;
+       if (avail == 0) {
                /* full */
                return 0;
        }
 
-       /* is it in a batchable area ? */
-
-       for (i = 0; i < dev->nb_zones; i++) {
-               zone = &dev->zone[i];
-
-               /* (addr,len) is fully included in
-                * (zone->addr, zone->size)
-                */
-
-               if (zone->addr <= addr &&
-                   addr + len <= zone->addr + zone->size)
-                       return 1;
-       }
-       return 0;
+       return 1;
 }
 
-static void coalesced_mmio_write(struct kvm_io_device *this,
-                                gpa_t addr, int len, const void *val)
+static int coalesced_mmio_write(struct kvm_io_device *this,
+                               gpa_t addr, int len, const void *val)
 {
-       struct kvm_coalesced_mmio_dev *dev =
-                               (struct kvm_coalesced_mmio_dev*)this->private;
+       struct kvm_coalesced_mmio_dev *dev = to_mmio(this);
        struct kvm_coalesced_mmio_ring *ring = dev->kvm->coalesced_mmio_ring;
 
-       /* kvm->lock must be taken by caller before call to in_range()*/
+       if (!coalesced_mmio_in_range(dev, addr, len))
+               return -EOPNOTSUPP;
+
+       spin_lock(&dev->kvm->ring_lock);
+
+       if (!coalesced_mmio_has_room(dev)) {
+               spin_unlock(&dev->kvm->ring_lock);
+               return -EOPNOTSUPP;
+       }
 
        /* copy data in first free entry of the ring */
 
@@ -76,84 +77,104 @@ static void coalesced_mmio_write(struct kvm_io_device *this,
        memcpy(ring->coalesced_mmio[ring->last].data, val, len);
        smp_wmb();
        ring->last = (ring->last + 1) % KVM_COALESCED_MMIO_MAX;
+       spin_unlock(&dev->kvm->ring_lock);
+       return 0;
 }
 
 static void coalesced_mmio_destructor(struct kvm_io_device *this)
 {
-       struct kvm_coalesced_mmio_dev *dev =
-               (struct kvm_coalesced_mmio_dev *)this->private;
+       struct kvm_coalesced_mmio_dev *dev = to_mmio(this);
+
+       list_del(&dev->list);
 
        kfree(dev);
 }
 
+static const struct kvm_io_device_ops coalesced_mmio_ops = {
+       .write      = coalesced_mmio_write,
+       .destructor = coalesced_mmio_destructor,
+};
+
 int kvm_coalesced_mmio_init(struct kvm *kvm)
 {
+       struct page *page;
+       int ret;
+
+       ret = -ENOMEM;
+       page = alloc_page(GFP_KERNEL | __GFP_ZERO);
+       if (!page)
+               goto out_err;
+
+       ret = 0;
+       kvm->coalesced_mmio_ring = page_address(page);
+
+       /*
+        * We're using this spinlock to sync access to the coalesced ring.
+        * The list doesn't need it's own lock since device registration and
+        * unregistration should only happen when kvm->slots_lock is held.
+        */
+       spin_lock_init(&kvm->ring_lock);
+       INIT_LIST_HEAD(&kvm->coalesced_zones);
+
+out_err:
+       return ret;
+}
+
+void kvm_coalesced_mmio_free(struct kvm *kvm)
+{
+       if (kvm->coalesced_mmio_ring)
+               free_page((unsigned long)kvm->coalesced_mmio_ring);
+}
+
+int kvm_vm_ioctl_register_coalesced_mmio(struct kvm *kvm,
+                                        struct kvm_coalesced_mmio_zone *zone)
+{
+       int ret;
        struct kvm_coalesced_mmio_dev *dev;
 
        dev = kzalloc(sizeof(struct kvm_coalesced_mmio_dev), GFP_KERNEL);
        if (!dev)
                return -ENOMEM;
-       dev->dev.write  = coalesced_mmio_write;
-       dev->dev.in_range  = coalesced_mmio_in_range;
-       dev->dev.destructor  = coalesced_mmio_destructor;
-       dev->dev.private  = dev;
+
+       kvm_iodevice_init(&dev->dev, &coalesced_mmio_ops);
        dev->kvm = kvm;
-       kvm->coalesced_mmio_dev = dev;
-       kvm_io_bus_register_dev(&kvm->mmio_bus, &dev->dev);
+       dev->zone = *zone;
 
-       return 0;
-}
+       mutex_lock(&kvm->slots_lock);
+       ret = kvm_io_bus_register_dev(kvm, KVM_MMIO_BUS, zone->addr,
+                                     zone->size, &dev->dev);
+       if (ret < 0)
+               goto out_free_dev;
+       list_add_tail(&dev->list, &kvm->coalesced_zones);
+       mutex_unlock(&kvm->slots_lock);
 
-int kvm_vm_ioctl_register_coalesced_mmio(struct kvm *kvm,
-                                        struct kvm_coalesced_mmio_zone *zone)
-{
-       struct kvm_coalesced_mmio_dev *dev = kvm->coalesced_mmio_dev;
+       return ret;
 
-       if (dev == NULL)
-               return -EINVAL;
+out_free_dev:
+       mutex_unlock(&kvm->slots_lock);
 
-       mutex_lock(&kvm->lock);
-       if (dev->nb_zones >= KVM_COALESCED_MMIO_ZONE_MAX) {
-               mutex_unlock(&kvm->lock);
-               return -ENOBUFS;
-       }
+       kfree(dev);
 
-       dev->zone[dev->nb_zones] = *zone;
-       dev->nb_zones++;
+       if (dev == NULL)
+               return -ENXIO;
 
-       mutex_unlock(&kvm->lock);
        return 0;
 }
 
 int kvm_vm_ioctl_unregister_coalesced_mmio(struct kvm *kvm,
                                           struct kvm_coalesced_mmio_zone *zone)
 {
-       int i;
-       struct kvm_coalesced_mmio_dev *dev = kvm->coalesced_mmio_dev;
-       struct kvm_coalesced_mmio_zone *z;
+       struct kvm_coalesced_mmio_dev *dev, *tmp;
 
-       if (dev == NULL)
-               return -EINVAL;
-
-       mutex_lock(&kvm->lock);
-
-       i = dev->nb_zones;
-       while(i) {
-               z = &dev->zone[i - 1];
+       mutex_lock(&kvm->slots_lock);
 
-               /* unregister all zones
-                * included in (zone->addr, zone->size)
-                */
-
-               if (zone->addr <= z->addr &&
-                   z->addr + z->size <= zone->addr + zone->size) {
-                       dev->nb_zones--;
-                       *z = dev->zone[dev->nb_zones];
+       list_for_each_entry_safe(dev, tmp, &kvm->coalesced_zones, list)
+               if (coalesced_mmio_in_range(dev, zone->addr, zone->size)) {
+                       kvm_io_bus_unregister_dev(kvm, KVM_MMIO_BUS, &dev->dev);
+                       kvm_iodevice_destructor(&dev->dev);
                }
-               i--;
-       }
 
-       mutex_unlock(&kvm->lock);
+       mutex_unlock(&kvm->slots_lock);
 
        return 0;
 }