[edk2-devel] [PATCH v3 0/6] SEV Encrypted Boot for Ovmf
James Bottomley
jejb at linux.ibm.com
Thu Dec 3 14:27:14 UTC 2020
On Thu, 2020-12-03 at 13:26 +0100, Laszlo Ersek wrote:
> Hi James,
>
> On 11/30/20 21:28, James Bottomley wrote:
> > v3:
> >
> > - More grub and boot stripping (I think I got everything out, but
> > there may be something that strayed in the boot panic
> > resolution).
> > - grub.sh tidy up with tabs->spaces.
> > - Move the reset vector GUIDisation patch to the front so it can be
> > applied independently
> > - Update the .dsc and .fdf files for variable policy
>
> In preparation for submitting the github PR for merging this series,
> I first ran PatchCheck.py locally.
>
> It doesn't like that I converted "OvmfPkg/AmdSev/Grub/grub.cfg" to
> LF, in addition to "OvmfPkg/AmdSev/Grub/grub.sh".
>
> PatchCheck.py recognizes the ".sh" suffix, so it's not complaining
> about "OvmfPkg/AmdSev/Grub/grub.sh". But, the config file is a
> problem.
>
> Can you please confirm that grub works fine if the config file has
> CRLF line terminators? Because then I'll just convert the config file
> back to CRLF, and merge the series.
I converted the entire file with unix2dos and grub does seem to be fine
with the CRLF line endings (probably because it wants to be a boot
loader beyond linux), so I think converting the file to CRLF is the
right way to go.
James
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#68249): https://edk2.groups.io/g/devel/message/68249
Mute This Topic: https://groups.io/mt/78617825/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-
More information about the edk2-devel-archive
mailing list