]> git.karo-electronics.de Git - karo-tx-linux.git/commitdiff
kvm tools: Separate pci layer out of virtio-console
authorSasha Levin <levinsasha928@gmail.com>
Wed, 24 Aug 2011 09:09:10 +0000 (12:09 +0300)
committerPekka Enberg <penberg@kernel.org>
Wed, 24 Aug 2011 14:46:56 +0000 (17:46 +0300)
Signed-off-by: Sasha Levin <levinsasha928@gmail.com>
Signed-off-by: Pekka Enberg <penberg@kernel.org>
tools/kvm/virtio/console.c

index cb5a15ee855ab2721f75c732339051db9e18ea8a..035554cefc877000bf344c1a403a4d35bf48d083 100644 (file)
@@ -11,6 +11,7 @@
 #include "kvm/threadpool.h"
 #include "kvm/irq.h"
 #include "kvm/guest_compat.h"
+#include "kvm/virtio-pci.h"
 
 #include <linux/virtio_console.h>
 #include <linux/virtio_ring.h>
 #define VIRTIO_CONSOLE_RX_QUEUE                0
 #define VIRTIO_CONSOLE_TX_QUEUE                1
 
-static struct pci_device_header virtio_console_pci_device = {
-       .vendor_id              = PCI_VENDOR_ID_REDHAT_QUMRANET,
-       .device_id              = PCI_DEVICE_ID_VIRTIO_CONSOLE,
-       .header_type            = PCI_HEADER_TYPE_NORMAL,
-       .revision_id            = 0,
-       .class                  = 0x078000,
-       .subsys_vendor_id       = PCI_SUBSYSTEM_VENDOR_ID_REDHAT_QUMRANET,
-       .subsys_id              = VIRTIO_ID_CONSOLE,
-};
-
 struct con_dev {
        pthread_mutex_t                 mutex;
 
+       struct virtio_pci               vpci;
        struct virt_queue               vqs[VIRTIO_CONSOLE_NUM_QUEUES];
-       struct virtio_console_config    console_config;
-       u32                             host_features;
-       u32                             guest_features;
-       u16                             config_vector;
-       u8                              status;
-       u8                              isr;
-       u16                             queue_selector;
-       u16                             base_addr;
+       struct virtio_console_config    config;
+       u32                             features;
        int                             compat_id;
 
        struct thread_pool__job         jobs[VIRTIO_CONSOLE_NUM_QUEUES];
@@ -59,13 +45,11 @@ struct con_dev {
 static struct con_dev cdev = {
        .mutex                          = PTHREAD_MUTEX_INITIALIZER,
 
-       .console_config = {
+       .config = {
                .cols                   = 80,
                .rows                   = 24,
                .max_nr_ports           = 1,
        },
-
-       .host_features                  = 0,
 };
 
 /*
@@ -87,7 +71,7 @@ static void virtio_console__inject_interrupt_callback(struct kvm *kvm, void *par
                head = virt_queue__get_iov(vq, iov, &out, &in, kvm);
                len = term_getc_iov(CONSOLE_VIRTIO, iov, in);
                virt_queue__set_used_elem(vq, head, len);
-               virt_queue__trigger_irq(vq, virtio_console_pci_device.irq_line, &cdev.isr, kvm);
+               virtio_pci__signal_vq(kvm, &cdev.vpci, vq - cdev.vqs);
        }
 
        mutex_unlock(&cdev.mutex);
@@ -98,65 +82,6 @@ void virtio_console__inject_interrupt(struct kvm *kvm)
        thread_pool__do_job(&cdev.jobs[VIRTIO_CONSOLE_RX_QUEUE]);
 }
 
-static bool virtio_console_pci_io_device_specific_in(void *data, unsigned long offset, int size)
-{
-       u8 *config_space = (u8 *) &cdev.console_config;
-
-       if (size != 1)
-               return false;
-
-       if ((offset - VIRTIO_MSI_CONFIG_VECTOR) > sizeof(struct virtio_console_config))
-               pr_error("config offset is too big: %li", offset - VIRTIO_MSI_CONFIG_VECTOR);
-
-       ioport__write8(data, config_space[offset - VIRTIO_MSI_CONFIG_VECTOR]);
-
-       return true;
-}
-
-static bool virtio_console_pci_io_in(struct ioport *ioport, struct kvm *kvm, u16 port, void *data, int size)
-{
-       unsigned long offset = port - cdev.base_addr;
-       bool ret = true;
-
-       mutex_lock(&cdev.mutex);
-
-       switch (offset) {
-       case VIRTIO_PCI_HOST_FEATURES:
-               ioport__write32(data, cdev.host_features);
-               break;
-       case VIRTIO_PCI_GUEST_FEATURES:
-               ret = false;
-               break;
-       case VIRTIO_PCI_QUEUE_PFN:
-               ioport__write32(data, cdev.vqs[cdev.queue_selector].pfn);
-               break;
-       case VIRTIO_PCI_QUEUE_NUM:
-               ioport__write16(data, VIRTIO_CONSOLE_QUEUE_SIZE);
-               break;
-       case VIRTIO_PCI_QUEUE_SEL:
-       case VIRTIO_PCI_QUEUE_NOTIFY:
-               ret = false;
-               break;
-       case VIRTIO_PCI_STATUS:
-               ioport__write8(data, cdev.status);
-               break;
-       case VIRTIO_PCI_ISR:
-               ioport__write8(data, cdev.isr);
-               kvm__irq_line(kvm, virtio_console_pci_device.irq_line, VIRTIO_IRQ_LOW);
-               cdev.isr = VIRTIO_IRQ_LOW;
-               break;
-       case VIRTIO_MSI_CONFIG_VECTOR:
-               ioport__write16(data, cdev.config_vector);
-               break;
-       default:
-               ret = virtio_console_pci_io_device_specific_in(data, offset, size);
-       };
-
-       mutex_unlock(&cdev.mutex);
-
-       return ret;
-}
-
 static void virtio_console_handle_callback(struct kvm *kvm, void *param)
 {
        struct iovec iov[VIRTIO_CONSOLE_QUEUE_SIZE];
@@ -181,83 +106,87 @@ static void virtio_console_handle_callback(struct kvm *kvm, void *param)
 
 }
 
-static bool virtio_console_pci_io_out(struct ioport *ioport, struct kvm *kvm, u16 port, void *data, int size)
+static void set_config(struct kvm *kvm, void *dev, u8 data, u32 offset)
 {
-       unsigned long offset = port - cdev.base_addr;
-       bool ret = true;
+       struct con_dev *cdev = dev;
 
-       mutex_lock(&cdev.mutex);
+       ((u8 *)(&cdev->config))[offset] = data;
+}
+
+static u8 get_config(struct kvm *kvm, void *dev, u32 offset)
+{
+       struct con_dev *cdev = dev;
 
-       switch (offset) {
-       case VIRTIO_PCI_GUEST_FEATURES:
-               cdev.guest_features     = ioport__read32(data);
-               break;
-       case VIRTIO_PCI_QUEUE_PFN: {
-               struct virt_queue *queue;
-               void *p;
+       return ((u8 *)(&cdev->config))[offset];
+}
+
+static u32 get_host_features(struct kvm *kvm, void *dev)
+{
+       return 0;
+}
 
-               assert(cdev.queue_selector < VIRTIO_CONSOLE_NUM_QUEUES);
+static void set_guest_features(struct kvm *kvm, void *dev, u32 features)
+{
+       /* Unused */
+}
 
-               compat__remove_message(cdev.compat_id);
+static int init_vq(struct kvm *kvm, void *dev, u32 vq, u32 pfn)
+{
+       struct virt_queue *queue;
+       void *p;
 
-               queue                   = &cdev.vqs[cdev.queue_selector];
-               queue->pfn              = ioport__read32(data);
-               p                       = guest_pfn_to_host(kvm, queue->pfn);
+       assert(vq < VIRTIO_CONSOLE_NUM_QUEUES);
 
-               vring_init(&queue->vring, VIRTIO_CONSOLE_QUEUE_SIZE, p, VIRTIO_PCI_VRING_ALIGN);
+       compat__remove_message(cdev.compat_id);
 
-               if (cdev.queue_selector == VIRTIO_CONSOLE_TX_QUEUE)
-                       thread_pool__init_job(&cdev.jobs[cdev.queue_selector], kvm, virtio_console_handle_callback, queue);
-               else if (cdev.queue_selector == VIRTIO_CONSOLE_RX_QUEUE)
-                       thread_pool__init_job(&cdev.jobs[cdev.queue_selector], kvm, virtio_console__inject_interrupt_callback, queue);
+       queue                   = &cdev.vqs[vq];
+       queue->pfn              = pfn;
+       p                       = guest_pfn_to_host(kvm, queue->pfn);
 
-               break;
-       }
-       case VIRTIO_PCI_QUEUE_SEL:
-               cdev.queue_selector     = ioport__read16(data);
-               break;
-       case VIRTIO_PCI_QUEUE_NOTIFY: {
-               u16 queue_index         = ioport__read16(data);
-               thread_pool__do_job(&cdev.jobs[queue_index]);
-               break;
-       }
-       case VIRTIO_PCI_STATUS:
-               cdev.status             = ioport__read8(data);
-               break;
-       case VIRTIO_MSI_CONFIG_VECTOR:
-               cdev.config_vector      = VIRTIO_MSI_NO_VECTOR;
-               break;
-       case VIRTIO_MSI_QUEUE_VECTOR:
-               break;
-       default:
-               ret                     = false;
-               break;
-       };
+       vring_init(&queue->vring, VIRTIO_CONSOLE_QUEUE_SIZE, p, VIRTIO_PCI_VRING_ALIGN);
 
-       mutex_unlock(&cdev.mutex);
+       if (vq == VIRTIO_CONSOLE_TX_QUEUE)
+               thread_pool__init_job(&cdev.jobs[vq], kvm, virtio_console_handle_callback, queue);
+       else if (vq == VIRTIO_CONSOLE_RX_QUEUE)
+               thread_pool__init_job(&cdev.jobs[vq], kvm, virtio_console__inject_interrupt_callback, queue);
 
-       return ret;
+       return 0;
 }
 
-static struct ioport_operations virtio_console_io_ops = {
-       .io_in                  = virtio_console_pci_io_in,
-       .io_out                 = virtio_console_pci_io_out,
-};
+static int notify_vq(struct kvm *kvm, void *dev, u32 vq)
+{
+       struct con_dev *cdev = dev;
+
+       thread_pool__do_job(&cdev->jobs[vq]);
+
+       return 0;
+}
+
+static int get_pfn_vq(struct kvm *kvm, void *dev, u32 vq)
+{
+       struct con_dev *cdev = dev;
+
+       return cdev->vqs[vq].pfn;
+}
+
+static int get_size_vq(struct kvm *kvm, void *dev, u32 vq)
+{
+       return VIRTIO_CONSOLE_QUEUE_SIZE;
+}
 
 void virtio_console__init(struct kvm *kvm)
 {
-       u8 dev, line, pin;
-       u16 console_base_addr;
-
-       if (irq__register_device(VIRTIO_ID_CONSOLE, &dev, &pin, &line) < 0)
-               return;
-
-       virtio_console_pci_device.irq_pin       = pin;
-       virtio_console_pci_device.irq_line      = line;
-       console_base_addr                       = ioport__register(IOPORT_EMPTY, &virtio_console_io_ops, IOPORT_SIZE, NULL);
-       virtio_console_pci_device.bar[0]        = console_base_addr | PCI_BASE_ADDRESS_SPACE_IO;
-       cdev.base_addr                          = console_base_addr;
-       pci__register(&virtio_console_pci_device, dev);
+       virtio_pci__init(kvm, &cdev.vpci, &cdev, PCI_DEVICE_ID_VIRTIO_CONSOLE, VIRTIO_ID_CONSOLE);
+       cdev.vpci.ops = (struct virtio_pci_ops) {
+               .set_config             = set_config,
+               .get_config             = get_config,
+               .get_host_features      = get_host_features,
+               .set_guest_features     = set_guest_features,
+               .init_vq                = init_vq,
+               .notify_vq              = notify_vq,
+               .get_pfn_vq             = get_pfn_vq,
+               .get_size_vq            = get_size_vq,
+       };
 
        cdev.compat_id = compat__add_message("virtio-console device was not detected",
                                                "While you have requested a virtio-console device, "