Thanks for the detailed explanation Mitchell. I know that this has been
a problem for a long time - waiting for several minutes for a VM to
start to boot is not reasonable.

> I am *not* planning to submit this patch to upstream edk2 since this
option should not be necessary for any users running newer kernels, once
the requisite patches[1] land in the mainline tree.

While newer kernels should (eventually) avoid this (yay aw!), older
kernels will be in production for many years, and users will want to
keep running them on newer hardware. This isn't just an Ubuntu issue -
so I think it would be worth submitting it upstream to gauge interest
from others.

If upstream is uninterested, then it is likely reasonable to carry a
SAUCE patch.

One thing I'd suggest for the patch itself is to move the logic for
whether or not to create a dynamic window into the caller(s) of
PlatformDynamicMmioWindow(). IMO, a function should do what its name
says - if the code needs to decide whether or not to do that thing, it
should do that before calling the function.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/2101903

Title:
  UBUNTU: SAUCE: Introduce X-PciMmioClassicWindow option to fw_cfg

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/edk2/+bug/2101903/+subscriptions


-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to