qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Laszlo Ersek <lersek@redhat.com>
To: Gerd Hoffmann <kraxel@redhat.com>
Cc: qemu-devel@nongnu.org, "Michael S. Tsirkin" <mst@redhat.com>,
	Alex Williamson <alex.williamson@redhat.com>,
	Ard Biesheuvel <ard.biesheuvel@linaro.org>,
	Marc Zyngier <marc.zyngier@arm.com>
Subject: Re: [Qemu-devel] [PATCH v2 0/7] ramfb: simple boot framebuffer, no legacy vga
Date: Tue, 5 Jun 2018 15:23:23 +0200	[thread overview]
Message-ID: <20d457f0-647f-04e3-76c3-ed58f3494208@redhat.com> (raw)
In-Reply-To: <20180605131649.fu3cellm7vwaf4ao@sirius.home.kraxel.org>

On 06/05/18 15:16, Gerd Hoffmann wrote:
> On Tue, Jun 05, 2018 at 02:07:27PM +0200, Laszlo Ersek wrote:
>> On 06/05/18 13:06, Gerd Hoffmann wrote:
>>>   Hi,
>>>
>>>>>> I could imagine an OvmfPkg-specific PCI capability that said, "all PCI
>>>>>> drivers in OvmfPkg that could otherwise drive this device, ignore it --
>>>>>> another (platform) driver in OvmfPkg will pick it up instead".
>>>>>
>>>>> pci capability for ramfb could be useful (also for linux).  I'll keep it
>>>>> in mind for now.
>>>>
>>>> Please do. :)
>>>
>>> Hmm, well.  Virtio 1.0 uses vendor specific capabilities already to
>>> define the regions, and they don't have a fixed field saying "this is
>>> for virtio".  So adding another vendor specific capability for something
>>> else on the same device is a bit problematic ...
>>
>> Can we invent a non-PCI method, e.g. fw_cfg, that tells QemuVideoDxe and
>> Virtio10Dxe not to bind some PCI S/B/D/Fs? Something like:
> 
> Well, from edk2 point of view Virtio10Dxe is the only problematic case
> because there is a is a native driver.  For cirrus/stdvga a version with
> ramfb is rather pointless.

Good catch; thinko on my part!

> A qxl-ramfb device might make sense, but
> QemuVideoDxe would ignore such a device like it ignores qxl today as it
> can only handle the vga mode of qxl-vga devices.

Indeed.

> Also I'd prefer to provide informations (device foo has ramfb at <addr>)
> not instructions (please ignore device foo).
> 
> Maybe we should for now just scratch the idea of an virtio-ramfb device.
> Linux doesn't need it, and windows wouldn't use the virtio part of it so
> a standalone ramfb device would work equally well.

If that works for you, it works for me best!

Thanks!
Laszlo

  reply	other threads:[~2018-06-05 13:23 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-03-23 12:25 [Qemu-devel] [PATCH v2 0/7] ramfb: simple boot framebuffer, no legacy vga Gerd Hoffmann
2018-03-23 12:25 ` [Qemu-devel] [PATCH v2 1/7] [testing] update bios, add vgabios-ramfb Gerd Hoffmann
2018-03-23 12:25 ` [Qemu-devel] [PATCH v2 2/7] [testing] add ovmf build with ramfb support Gerd Hoffmann
2018-03-23 12:25 ` [Qemu-devel] [PATCH v2 3/7] hw/display: add ramfb, a simple boot framebuffer living in guest ram Gerd Hoffmann
2018-03-23 12:25 ` [Qemu-devel] [PATCH v2 4/7] hw/display: add ramfb-testdev Gerd Hoffmann
2018-03-23 12:25 ` [Qemu-devel] [PATCH v2 5/7] hw/display: add virtio-ramfb Gerd Hoffmann
2018-03-23 12:25 ` [Qemu-devel] [PATCH v2 6/7] hw/vfio/display: add ramfb support Gerd Hoffmann
2018-03-23 12:25 ` [Qemu-devel] [PATCH v2 7/7] [wip] hw/display: add qxl-ramfb Gerd Hoffmann
2018-03-23 13:27 ` [Qemu-devel] [PATCH v2 0/7] ramfb: simple boot framebuffer, no legacy vga Laszlo Ersek
2018-03-23 14:51   ` Gerd Hoffmann
2018-03-23 15:12     ` Laszlo Ersek
2018-03-23 17:07       ` Gerd Hoffmann
2018-03-23 18:29         ` Laszlo Ersek
2018-03-24  3:51   ` Ard Biesheuvel
2018-04-25 14:07   ` Gerd Hoffmann
2018-04-25 20:43     ` Laszlo Ersek
2018-05-31  8:43       ` Gerd Hoffmann
2018-05-31  9:11         ` Laszlo Ersek
2018-06-05 11:06           ` Gerd Hoffmann
2018-06-05 12:07             ` Laszlo Ersek
2018-06-05 13:16               ` Gerd Hoffmann
2018-06-05 13:23                 ` Laszlo Ersek [this message]
2018-06-06  8:49                   ` Gerd Hoffmann
2018-06-06 12:43                     ` Laszlo Ersek
2018-06-06 13:21                       ` Gerd Hoffmann

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=20d457f0-647f-04e3-76c3-ed58f3494208@redhat.com \
    --to=lersek@redhat.com \
    --cc=alex.williamson@redhat.com \
    --cc=ard.biesheuvel@linaro.org \
    --cc=kraxel@redhat.com \
    --cc=marc.zyngier@arm.com \
    --cc=mst@redhat.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).