On Fri, Mar 31, 2017 at 02:18:23PM -0700, Petr Vandrovec wrote: > Jarkko Sakkinen wrote: > > On Wed, Mar 29, 2017 at 12:43:20AM -0700, Petr Vandrovec wrote: > > > Hi Peter, > > > > > > This series of 4 patches adds support for handling TPM2 > > > log when it is reported through ACPI TPM2 table, as > > > described in latest TPM ACPI draft (Version 1.2, Revision 8, > > > from February 27th, 2017). > > > > > > I've tested patch on x86 only - I do not have PPC64 system > > > with TPM, and handling of endianness between tpm1_eventlog.c > > > and tpm2_eventlog.c seems inconsistent: tpm1_eventlog.c uses > > > be32_to_cpu() for all log fields on PPC64, while > > > tpm2_eventlog.c uses log uses native endianness. If it > > > is intentional, and PPC64 has TPM1 logs in big endian > > > while TPM2 logs in native endianness, then 3rd patch > > > needs to be amended. > > > > > > Four patches split task of improving TPM2 support into following > > > subtasks: > > > > This cover letter is a bit chaotic. A cover letter should give an > > understandable description of the what is done. Now you have a 2nd > > version of commit messages here. This looks confusing. > > Hi Jarkko, > > sorry about that. I wanted to provide additional details in the cover > letter. As far as understandable description, it is more or less covered in > first paragraph. Would this summary work for you: > > This series adds support for handling TPM2 log when it is reported through > ACPI TPM2 table, as described in latest TPM ACPI draft (Version 1.2, > Revision 8, from February 27th, 2017). > > Second half of the series then improves support for TPM2 log parsing, so > kernel does not crash when firmware provides TPM log in different format > than 'Spec ID Event03', or if it uses Event03 format with more than 3 > hashes. > > > Thanks, > Petr
Great, thank you. I'll look into this in more detail after we are done with 4.12. /Jarkko ------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, Slashdot.org! http://sdm.link/slashdot _______________________________________________ tpmdd-devel mailing list tpmdd-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/tpmdd-devel