[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH 00/14] Automatic Disk Unlock with TPM2

From: Daniel Kiper
Subject: Re: [PATCH 00/14] Automatic Disk Unlock with TPM2
Date: Tue, 28 Feb 2023 18:39:22 +0100
User-agent: NeoMutt/20170113 (1.7.2)

On Wed, Feb 22, 2023 at 03:00:40PM +0800, Gary Lin via Grub-devel wrote:
> The patch series "Automatic TPM Disk Unlock" posted by Hernan Gatta
> introduces the key protector framework and TPM2 stack to GRUB2, and it's
> a useful feature for the systems to implement full disk encryption.
> However, it seems the development was stalled for a while, and I'd like
> to push it forward.

Thank you for updating this patch set. Certainly I want to have it in
the GRUB upstream. Sadly I cannot consider this as a 2.12 release
material. So, I will not review these patches immediately due to
their low priority. Though I have a few comments below...

> Patch 1~5 are Hernan Gatta's patch series(*) with a few modifications:
> - Converting 8 spaces into 1 tab
> - Merging the minor build fix from Michael Chang
>   - Replacing "lu" with "PRIuGRUB_SIZE" for grub_dprintf
>   - Adding "enable = efi" to the tpm2 module in grub-core/Makefile.core.def
> - Rebasing "cryptodisk: Support key protectors" to the git master

Thank you for doing all of this!

> To minimize the changes to Patch 1~5, the follow-up fixes (Patch 6~14)
> from my colleagues and me are committed separately. Those patches fix
> the problems we found while testing the original patchset.

Please merge all patches after #5 into original patch series and add
your and your colleagues SOBs to relevant patches.

And of course take into account James comments...


reply via email to

[Prev in Thread] Current Thread [Next in Thread]