public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Gerd Hoffmann" <kraxel@redhat.com>
To: Ard Biesheuvel <ardb@kernel.org>
Cc: devel@edk2.groups.io, pedro.falcato@gmail.com,
	 Laszlo Ersek <lersek@redhat.com>,
	Oliver Steffen <osteffen@redhat.com>,
	 Ard Biesheuvel <ardb+tianocore@kernel.org>,
	Jiewen Yao <jiewen.yao@intel.com>,
	dun.tan@intel.com
Subject: Re: [edk2-devel] [PATCH 1/1] OvmfPkg/QemuVideoDxe: purge VbeShim
Date: Tue, 13 Feb 2024 16:28:07 +0100	[thread overview]
Message-ID: <yjdu3dza5rz5a3buscfr5vgl2b2sxobo3zb3mkdgt7w4xh4zam@m7eux5gyd433> (raw)
In-Reply-To: <CAMj1kXG26+8yToJMJ+0JgE6w6ZDoWdyo0hhpy01=SL6wUUH=+Q@mail.gmail.com>

  Hi,

> > > New theory: This is a NULL pointer dereference corrupting the zero page.
> >
> > Tangentially related: Maybe it's time to unmap the NULL page? :)

Suggestion: read mails to the end.  Had the same idea + question ;)

> That would break many 3rd party GPU and network drivers.

For virtual machine firmware it might be acceptable to just ignore
that, or at least default to an unmapped zero page and offer an
qemu fw-cfg override in case someone needs that when pci-assigning
nic or gpu to a VM.

take care,
  Gerd



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#115387): https://edk2.groups.io/g/devel/message/115387
Mute This Topic: https://groups.io/mt/104329097/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



  parent reply	other threads:[~2024-02-13 15:28 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-02-13  8:59 [edk2-devel] [PATCH 1/1] OvmfPkg/QemuVideoDxe: purge VbeShim Gerd Hoffmann
2024-02-13 10:41 ` Ard Biesheuvel
2024-02-13 10:44 ` Laszlo Ersek
2024-02-13 14:02   ` Gerd Hoffmann
2024-02-13 14:55     ` Pedro Falcato
2024-02-13 14:56       ` Ard Biesheuvel
2024-02-13 15:07         ` Pedro Falcato
2024-02-13 15:28         ` Gerd Hoffmann [this message]
2024-02-13 15:47           ` Pedro Falcato
2024-02-13 17:31             ` Laszlo Ersek
2024-02-13 17:28     ` Laszlo Ersek
2024-02-13 18:13       ` Laszlo Ersek
2024-02-13 20:25         ` Laszlo Ersek

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-list 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=yjdu3dza5rz5a3buscfr5vgl2b2sxobo3zb3mkdgt7w4xh4zam@m7eux5gyd433 \
    --to=devel@edk2.groups.io \
    /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