[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/9] virtio-gpu/2d: add virtio gpu core code
From: |
Max Reitz |
Subject: |
Re: [Qemu-devel] [PATCH 2/9] virtio-gpu/2d: add virtio gpu core code |
Date: |
Mon, 16 Mar 2015 14:28:33 -0400 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.5.0 |
On 2015-03-13 at 05:47, Gerd Hoffmann wrote:
This patch adds the core code for virtio gpu emulation,
covering 2d support.
Written by Dave Airlie and Gerd Hoffmann.
Signed-off-by: Dave Airlie <address@hidden>
Signed-off-by: Gerd Hoffmann <address@hidden>
---
hw/display/Makefile.objs | 2 +
hw/display/virtio-gpu.c | 923 +++++++++++++++++++++++++++++++++++++++++
include/hw/virtio/virtio-gpu.h | 147 +++++++
trace-events | 14 +
4 files changed, 1086 insertions(+)
create mode 100644 hw/display/virtio-gpu.c
create mode 100644 include/hw/virtio/virtio-gpu.h
diff --git a/hw/display/virtio-gpu.c b/hw/display/virtio-gpu.c
new file mode 100644
index 0000000..ab71291
--- /dev/null
+++ b/hw/display/virtio-gpu.c
[snip]
+int virtio_gpu_create_mapping_iov(struct virtio_gpu_resource_attach_backing
*ab,
+ struct virtio_gpu_ctrl_command *cmd,
+ struct iovec **iov)
+{
+ struct virtio_gpu_mem_entry *ents;
+ size_t esize, s;
+ int i;
+
+ if (ab->nr_entries > 1024) {
+ qemu_log_mask(LOG_GUEST_ERROR,
+ "%s: nr_entries is too big (%d > 1024)\n",
+ __func__, ab->nr_entries);
You said that maybe you wanted to go for 16384 instead of 1024.
[snip]
diff --git a/include/hw/virtio/virtio-gpu.h b/include/hw/virtio/virtio-gpu.h
new file mode 100644
index 0000000..5d26ca9
--- /dev/null
+++ b/include/hw/virtio/virtio-gpu.h
@@ -0,0 +1,147 @@
+/*
+ * Virtio GPU Device
+ *
+ * Copyright Red Hat, Inc. 2013-2014
+ *
+ * Authors:
+ * Dave Airlie <address@hidden>
+ * Gerd Hoffmann <address@hidden>
+ *
+ * This work is licensed under the terms of the GNU GPL, version 2.
+ * See the COPYING file in the top-level directory.
+ */
+
+#ifndef _QEMU_VIRTIO_VGA_H
+#define _QEMU_VIRTIO_VGA_H
+
+#include "qemu/queue.h"
+#include "ui/qemu-pixman.h"
+#include "ui/console.h"
+#include "hw/virtio/virtio.h"
+#include "hw/pci/pci.h"
+
+#include "hw/virtio/virtgpu_hw.h"
+#define TYPE_VIRTIO_GPU "virtio-gpu-device"
+#define VIRTIO_GPU(obj) \
+ OBJECT_CHECK(VirtIOGPU, (obj), TYPE_VIRTIO_GPU)
+
+#define VIRTIO_ID_GPU 16
+
+#define VIRTIO_GPU_MAX_RES 16
Unused in this series. Is that intentional?
Independent of whether you keep it or remove it, and independent of
whether you increase the ab->nr_entries limit:
Reviewed-by: Max Reitz <address@hidden>
- [Qemu-devel] [PATCH 8/9] virtio-gpu: add to display-vga test, (continued)
- [Qemu-devel] [PATCH 8/9] virtio-gpu: add to display-vga test, Gerd Hoffmann, 2015/03/13
- [Qemu-devel] [PATCH 5/9] virtio-vga: add '-vga virtio' support, Gerd Hoffmann, 2015/03/13
- [Qemu-devel] [PATCH 1/9] virtio-gpu/2d: add hardware spec include file, Gerd Hoffmann, 2015/03/13
- [Qemu-devel] [PATCH 9/9] [hack] virtio-gpu: maskerade as -device VGA, Gerd Hoffmann, 2015/03/13
- [Qemu-devel] [PATCH 2/9] virtio-gpu/2d: add virtio gpu core code, Gerd Hoffmann, 2015/03/13
- [Qemu-devel] [PATCH 7/9] virtio-vga: add vgabios binary, Gerd Hoffmann, 2015/03/13
- [Qemu-devel] [PATCH 4/9] virtio-vga: add virtio gpu device with vga compatibility, Gerd Hoffmann, 2015/03/13
- [Qemu-devel] [PATCH] opengl: require glx, Gerd Hoffmann, 2015/03/13
- [Qemu-devel] [PATCH 3/9] virtio-gpu-pci: add virtio pci support, Gerd Hoffmann, 2015/03/13