All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jan Kiszka <jan.kiszka@siemens.com>
To: Gerhard Wiesinger <lists@wiesinger.com>
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] Missing patch in QEMU which is in QEMU-KVM
Date: Mon, 13 Feb 2012 12:25:36 +0100	[thread overview]
Message-ID: <4F38F330.1010402@siemens.com> (raw)
In-Reply-To: <alpine.LFD.2.02.1202130714390.32751@bbs.intern>

On 2012-02-13 07:21, Gerhard Wiesinger wrote:
> Hello,
> 
> I miss the following patch in QEMU which is in QEMU-KVM:
> http://article.gmane.org/gmane.comp.emulators.kvm.devel/13557
> 
> commit a7fe0297840908a4fd65a1cf742481ccd45960eb
> Author: Andreas Winkelbauer <andreas.winkelbauer@gmx.at>
> Date:   Sun Feb 24 10:33:27 2008 +0200
> 
>     Extend vram size to 16MB
> 
>     this is useful for high resolutions.
> 
>     Signed-off-by: Avi Kivity <avi@qumranet.com>

This patch no longer applies, the delta became much smaller:

diff --git a/hw/vga_int.h b/hw/vga_int.h
index c1e700f..21047a5 100644
--- a/hw/vga_int.h
+++ b/hw/vga_int.h
@@ -34,8 +34,8 @@
 /* bochs VBE support */
 #define CONFIG_BOCHS_VBE

-#define VBE_DISPI_MAX_XRES              1600
-#define VBE_DISPI_MAX_YRES              1200
+#define VBE_DISPI_MAX_XRES              2560
+#define VBE_DISPI_MAX_YRES              1600
 #define VBE_DISPI_MAX_BPP               32

 #define VBE_DISPI_INDEX_ID              0x0
@@ -224,7 +224,7 @@ void vga_init_vbe(VGACommonState *s, MemoryRegion
*address_space);
 extern const uint8_t sr_mask[8];
 extern const uint8_t gr_mask[16];

-#define VGA_RAM_SIZE (8192 * 1024)
+#define VGA_RAM_SIZE (16 * 1024 * 1024)
 #define VGABIOS_FILENAME "vgabios.bin"
 #define VGABIOS_CIRRUS_FILENAME "vgabios-cirrus.bin"

Someone needs to bake a proper patch out of this and post it. There must
be also some VGA BIOS differences, but I haven't looked at the details.
That should be resolved at this chance as well.

> 
> In general: Which patches are missing in QEMU which are in QEMU-KVM and
> vice versa?

Basically no more patches, only features that partially require some
refactoring work:

 - KVM in-kernel PIC (patches + dependencies pending)
 - TPR acceleration for Windows (series under review, needs some minor
   further work)
 - MSI rework and support for KVM in-kernel irqchip support
 - PCI device assignment (old-style and/or VFIO)

Then we are done and qemu-kvm can be closed. But I would not expect this
before 1.2, maybe even 1.3. However, once only PCI assignment is
pending, the vast majority of users can switch.

Jan

-- 
Siemens AG, Corporate Technology, CT T DE IT 1
Corporate Competence Center Embedded Linux

      reply	other threads:[~2012-02-13 11:25 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-02-13  6:21 [Qemu-devel] Missing patch in QEMU which is in QEMU-KVM Gerhard Wiesinger
2012-02-13 11:25 ` Jan Kiszka [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4F38F330.1010402@siemens.com \
    --to=jan.kiszka@siemens.com \
    --cc=lists@wiesinger.com \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.