All of lore.kernel.org
 help / color / mirror / Atom feed
From: Avi Kivity <avi@redhat.com>
To: Anthony Liguori <anthony@codemonkey.ws>
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] Cirrus bugs vs endian: how two bugs cancel each other out
Date: Mon, 30 Jul 2012 16:30:16 +0300	[thread overview]
Message-ID: <50168C68.9010103@redhat.com> (raw)
In-Reply-To: <878ve11j70.fsf@codemonkey.ws>

On 07/30/2012 04:18 PM, Anthony Liguori wrote:
> Avi Kivity <avi@redhat.com> writes:
> 
>> On 07/30/2012 02:54 PM, Benjamin Herrenschmidt wrote:
>>>> 
>>>> > 
>>>> > We can also make the fbdev/fbcon driver do the swapping in SW, but it's
>>>> > a relatively unusual code path and I don't think it works properly with
>>>> > X, I don't think it can be made to work properly with the generic X KMS
>>>> > at this point.
>>>> > 
>>>> > Now, cirrusdrmfb is already specific to the qemu cirrus variant in
>>>> > several ways, I wouldn't mind keeping it that way and if we "fix" the
>>>> > endianness model, maybe having a "hidden" register to flip it back to
>>>> > it's current mode of operation that cirrusdrmfb would use...
>>>> 
>>>> That's possible, but why not go all the way to qxl?
>>>>
>>>> That will give you better graphics performance with no need to hack.
>>> 
>>> Well, qxl is pretty awful from what I can see so far. I'm more tempted
>>> to continue improving qemu-vga, adding a virtio transport, and maybe
>>> adding a way to tunnel spice into it if that makes sense but so far,
>>> that's stuff was designed for Windows as far as I can tell and is pretty
>>> horrible whatever way you look at it...
>>
>> Let's balkanize some more then?
> 
> Minor improvements to stdvga actual help qxl (presumably).  qxl still
> provides a vga interface which is used when guest drivers aren't
> available.

The premise is that guest drivers will be used, otherwise you may as
well stay with stdvga.

> It's not clear to me why it doesn't enable VBE but presumably if it did,
> then accelerations could be mapped through VBE.

I believe the idea is that you don't want to map the framebuffer into
the guest, this allows one-directional communication so you can defer
rendering to the client and not suffer from the latency.  But I may be
mixing things up.

> 
>>
>> No, qxl is our paravirt vga, we should improve it instead of spawning
>> new ones (which will be horrible in the eyes of the next person to look
>> at them).  You should also be getting the drm driver for free.
> 
> Actually, Gerd et al have expressed interest in moving to a virtio-based
> device model for Spice in the past.
> 
> I think done correctly, it could help bring graphics to other platforms
> like S390 where PCI doesn't exist and will never exist.

I thought the plan was to render into a virtual card punch, then flip
through the cards at 60 fps?

Virtio makes sense for qxl, but for now we have the original pci model
which I don't see a reason why it can't work for ppc.

-- 
error compiling committee.c: too many arguments to function

  reply	other threads:[~2012-07-30 13:30 UTC|newest]

Thread overview: 55+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-07-30  6:24 [Qemu-devel] Cirrus bugs vs endian: how two bugs cancel each other out Benjamin Herrenschmidt
2012-07-30 10:08 ` Avi Kivity
2012-07-30 11:20   ` Benjamin Herrenschmidt
2012-07-30 11:25     ` Avi Kivity
2012-07-30 11:54       ` Benjamin Herrenschmidt
2012-07-30 11:58         ` Avi Kivity
2012-07-30 12:08           ` Benjamin Herrenschmidt
2012-07-30 12:15             ` Avi Kivity
2012-07-30 12:23               ` Benjamin Herrenschmidt
2012-07-30 16:24             ` Alon Levy
2012-07-30 20:19               ` Anthony Liguori
2012-07-30 22:24               ` Benjamin Herrenschmidt
2012-07-31  8:10                 ` Alon Levy
2012-08-01 14:35                 ` Avi Kivity
2012-08-06 12:57             ` Gerd Hoffmann
2012-07-30 13:18           ` Anthony Liguori
2012-07-30 13:30             ` Avi Kivity [this message]
2012-07-30 13:45               ` Anthony Liguori
2012-07-30 13:55                 ` Avi Kivity
2012-07-30 14:29                   ` Anthony Liguori
2012-07-30 14:36                     ` Avi Kivity
2012-07-30 16:01                       ` Anthony Liguori
2012-07-30 23:47                         ` Rusty Russell
2012-07-31  3:16                           ` Benjamin Herrenschmidt
2012-08-06 14:02                             ` Gerd Hoffmann
2012-08-06 21:13                               ` Benjamin Herrenschmidt
2012-08-01 23:29                         ` Andreas Färber
2012-08-06 13:47                         ` Gerd Hoffmann
2012-08-06 14:35                           ` Anthony Liguori
2012-07-31  8:20                     ` Alon Levy
2012-07-30 22:15                   ` Benjamin Herrenschmidt
2012-07-31  0:17                     ` Anthony Liguori
2012-07-31  3:26                       ` Benjamin Herrenschmidt
2012-08-06 13:20             ` Gerd Hoffmann
2012-08-06 21:16               ` Benjamin Herrenschmidt
2012-08-07  5:30                 ` Gerd Hoffmann
2012-08-07  6:07                   ` Benjamin Herrenschmidt
2012-07-30 16:19         ` Alon Levy
2012-08-01 15:42           ` Andreas Färber
2012-08-01 19:22             ` Anthony Liguori
2012-08-03  6:45               ` Alon Levy
2012-08-03 13:41                 ` Anthony Liguori
2012-08-07  7:00                   ` Alon Levy
2012-08-07  8:01                     ` Gerd Hoffmann
2012-08-07 13:05                       ` Erlon Cruz
2012-08-07 14:07                         ` Gerd Hoffmann
2012-08-07 19:43                           ` Erlon Cruz
2012-08-08  6:18                             ` Gerd Hoffmann
2012-08-08 14:14                               ` Erlon Cruz
2012-08-09  6:17                                 ` Gerd Hoffmann
2012-07-30 15:18 ` Blue Swirl
2012-07-30 15:30   ` Peter Maydell
2012-07-30 15:44     ` Blue Swirl
2012-07-31  8:44 ` ronnie sahlberg
2012-07-31 10:30   ` Benjamin Herrenschmidt

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=50168C68.9010103@redhat.com \
    --to=avi@redhat.com \
    --cc=anthony@codemonkey.ws \
    --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.