Re: [PATCH v6 06/14] x86: Add early SHA support for Secure Launch early measurements

From: Matthew Garrett
Date: Sun May 14 2023 - 15:12:05 EST


On Sun, May 14, 2023 at 11:18:17AM -0700, Eric Biggers wrote:
> On Fri, May 12, 2023 at 01:24:22PM +0100, Andrew Cooper wrote:
> > You're suggesting that because Linux has been slow to take D-RTM over
> > the past decade, you're going to intentionally break people with older
> > hardware just because you don't feel like using an older algorithm?
> >
> > That's about the worst possible reason to not take support.
> >
> > There really are people in the world with older TPM 1.2 systems where
> > this D-RTM using SHA1 only is an improvement over using the incumbent tboot.
> >
> > ~Andrew
>
> This patchset is proposing a new kernel feature. So by definition, there are no
> existing users of it that can be broken.

The patchset reimplements a more extensible version of an existing
feature which people already consume, and presumably people will be
encouraged to transition to it. There is plenty of hardware that
supports this feature that only implements SHA-1. If you want to propose
that the kernel not implement any functionality that uses deprecated
hash algorithms then that seems like a larger conversation rather than
one that should focus on a single pachset.