All of lore.kernel.org
 help / color / mirror / Atom feed
From: Matthew Garrett <mjg59@srcf.ucam.org>
To: Benjamin Herrenschmidt <benh@au1.ibm.com>
Cc: "Anthony Liguori" <aliguori@us.ibm.com>,
	"Alexander Graf" <agraf@suse.de>,
	"Li Zhang" <zhlcindy@linux.vnet.ibm.com>,
	qemu-devel@nongnu.org, "Li Zhang" <zhlcindy@gmail.com>,
	airlied@redhat.com, qemu-ppc <qemu-ppc@nongnu.org>,
	"Andreas Färber" <afaerber@suse.de>
Subject: Re: [Qemu-devel] [Qemu-ppc] [PATCH 2/2] spapr: Add support for -vga option
Date: Mon, 23 Jul 2012 14:24:31 +0100	[thread overview]
Message-ID: <20120723132431.GA21495@srcf.ucam.org> (raw)
In-Reply-To: <1343025621.2957.27.camel@pasglop>

Dave Airlie wrote the dirty handling code, so Cc:ing him. I suspect 
transitioning offb over to a similar design would be fine by him.

On Mon, Jul 23, 2012 at 04:40:21PM +1000, Benjamin Herrenschmidt wrote:
> On Wed, 2012-06-27 at 17:25 -0500, Anthony Liguori wrote:
> 
> > If a user asks for something and we can't make it work, we should fail.
> 
> Note that QEMU's cirrus works fine with the new kernel cirrusdrmfb, so
> I say we should allow it (needs to be able to the powerpc device .mak
> as well tho.
> 
> It shouldn't be primary because old RHEL's iirc used to have cirrusfb
> enabled and that would blow due to guest kernel bugs among others.
> 
> I haven't had a chance to try whatever userspace DDX the Xorg folks have
> come up with to use on top of cirrusdrmfb, so that might need a bit of
> fixing but there's no reason not to allow -vga cirrus at this point.
> 
> Note to Matthew: cirrusdrmfb is a LOT SLOWER than offb for a similar
> SW only dumb framebuffer, probably has to do with the way it does the
> "dirty" stuff, not sure ...
> 
> Why not draw directly into the emulated vram ?
> 
> Cheers,
> Ben.
> 
> 
> 
-- 
Matthew Garrett | mjg59@srcf.ucam.org

      parent reply	other threads:[~2012-07-23 13:25 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-06-18  9:34 [Qemu-devel] [PATCH 2/2] spapr: Add support for -vga option Li Zhang
     [not found] ` <80058105-0CF7-4FBD-BD4A-EF7532A8F5B8@suse.de>
     [not found]   ` <1340833174.3732.65.camel@pasglop>
     [not found]     ` <9D81F5FD-C680-4CC4-B3E0-E737C84BC1DB@suse.de>
     [not found]       ` <1340834707.3732.81.camel@pasglop>
     [not found]         ` <92004E32-9F3E-4F4F-8287-B2EF25C7B4F9@suse.de>
     [not found]           ` <4FEB886A.5000605@us.ibm.com>
2012-07-23  6:40             ` [Qemu-devel] [Qemu-ppc] " Benjamin Herrenschmidt
2012-07-23  7:27               ` Benjamin Herrenschmidt
2012-07-23 13:24               ` Matthew Garrett [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=20120723132431.GA21495@srcf.ucam.org \
    --to=mjg59@srcf.ucam.org \
    --cc=afaerber@suse.de \
    --cc=agraf@suse.de \
    --cc=airlied@redhat.com \
    --cc=aliguori@us.ibm.com \
    --cc=benh@au1.ibm.com \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-ppc@nongnu.org \
    --cc=zhlcindy@gmail.com \
    --cc=zhlcindy@linux.vnet.ibm.com \
    /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.