Branch: refs/heads/master
  Home:   https://github.com/tianocore/edk2
  Commit: 2d233af64b8f73d1b1e138b302e6344f7c2e0f4e
      
https://github.com/tianocore/edk2/commit/2d233af64b8f73d1b1e138b302e6344f7c2e0f4e
  Author: Ard Biesheuvel <[email protected]>
  Date:   2020-06-17 (Wed, 17 Jun 2020)

  Changed paths:
    M ArmPkg/Library/PlatformBootManagerLib/PlatformBm.c

  Log Message:
  -----------
  ArmPkg/PlatformBootManagerLib: regenerate boot options on boot failure

One of the side effects of the recent changes to PlatformBootManagerLib
changes to avoid connecting all devices on every boot is that we no
longer default to network boot on a virgin boot, but end up in the
UiApp menu. At this point, the UiApp will instantiate the autogenerated
boot options that we used to rely on as before, but since we are already
sitting idle in the root UiApp menu at that point, it does break the
unattended boot case where devices are expected to attempt a network
boot on the very first power on.

Let's work around this by refreshing all boot options explicitly in
the UnableToBoot() handler, and rebooting the system if doing so
resulted in a change to the total number of configured boot options.
This way, we ultimately end up in the UiApp as before if no boot
options could be started, but only after all the autogenerated ones
have been attempted as well.

Signed-off-by: Ard Biesheuvel <[email protected]>
Reviewed-by: Leif Lindholm <[email protected]>
Acked-by: Laszlo Ersek <[email protected]>
Reviewed-by: Andrei Warkentin <[email protected]>


  Commit: 8f22a331b955bd3f8077c7fa83bafeec566d6718
      
https://github.com/tianocore/edk2/commit/8f22a331b955bd3f8077c7fa83bafeec566d6718
  Author: Ard Biesheuvel <[email protected]>
  Date:   2020-06-17 (Wed, 17 Jun 2020)

  Changed paths:
    M EmbeddedPkg/Library/NonCoherentDmaLib/NonCoherentDmaLib.c

  Log Message:
  -----------
  EmbeddedPkg/NonCoherentDmaLib: avoid dereferencing bogus buffer address

The bounce buffering code in NonCoherentDmaLib copies data into the
bounce buffer using CopyMem(), but passes Map->HostAddress as the
source of the copy before it has been assigned its correct value.

Signed-off-by: Ard Biesheuvel <[email protected]>
Reviewed-by: Leif Lindholm <[email protected]>
Reviewed-by: Andrei Warkentin <[email protected]>


Compare: https://github.com/tianocore/edk2/compare/58ae92a99368...8f22a331b955


_______________________________________________
edk2-commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/edk2-commits

Reply via email to