On Thu, Sep 01, 2016 at 04:45:01PM +0300, Jarkko Sakkinen wrote:
> On Wed, Aug 31, 2016 at 11:26:24PM +0530, Nayna wrote:
> > Thanks Jarkko for the review. I will address all your comments in my next
> > version of patches.
>
> OK maybe the point I'm trying to make if you forget all the whining is
> that with this particular feature we have to be extremely careful
> because of the number of stakeholders that depend on it.
>
> This is not something I would put into 4.10 and no matter who is doing
> it, it might take a few more iterations to get right. So take your time.
> There's no rush (or more like there can't be rush).
Sorry, a typo. I meant the 4.9 release :) 4.10 release is an open
question. This should be in production quality by 4.9-rc4/5 in order
to make that happen.
The good timeline for topic branch would be end of Oct before LPC so
that I could carry a setup involving Minnowboard and a discrete TPM
module and demo this. I already have a topic branch called 'tabrm'
in place for the same conference.
/Jarkko
> In the meanwhile I'm still in progress on getting the suitable hardware
> so that I could test at least the DT stuff.
>
> PS. It's better to keep the attribute names the same since the code is
> already depending those names even if I don't like the naming :)
>
> /Jarkko
>
> > Thanks & Regards,
> > - Nayna
> >
> > On 08/30/2016 12:40 PM, Jarkko Sakkinen wrote:
> > >On Tue, Aug 30, 2016 at 12:50:12AM -0400, Nayna Jain wrote:
> > >>Existing TPM2.0 support lacks the support for eventlog securityfs file.
> > >>This patch adds the binary_bios_measurements to TPM2.0 eventlog
> > >>securityfs file.
> > >>
> > >>Additionally, it also includes the review feedbacks as suggested by
> > >>Jason.
> > >>
> > >>Further, commit msg subject line is prefixed with tpm as was suggested
> > >>by Jarkko.
> > >
> > >Please start using get_maintainers.pl...
> > >
> > >>Changelog v3:
> > >>
> > >>* Includes the review feedbacks as suggested by Jason
> > >> * Split of patches into one patch per idea
> > >> * Generic open() method for ascii/bios measurements
> > >> * Replacement of of **bios_dir with *bios_dir[3]
> > >> * Verifying readlog() is successful before creating
> > >> securityfs entries
> > >> * Generic readlog() to check for ACPI/OF in sequence
> > >> * read_log_of() method now uses of_node propertry rather than
> > >> calling find_device_by_name
> > >> * read_log differentiates vtpm/tpm using its compatible property
> > >> * Cleans pr_err with dev_dbg
> > >> * Commit msgs subject line prefixed with tpm
> > >
> > >Where is the changlog for v2?
> > >
> > >/Jarkko
> > >
> > >>
> > >>Nayna Jain (7):
> > >> tpm: Define a generic open() method for ascii & bios measurements.
> > >> tpm: Replace the dynamically allocated bios_dir as struct dentry
> > >> array.
> > >> tpm: Validate the eventlog access before tpm_bios_log_setup
> > >> tpm: Redefine the read_log method to check for ACPI/OF properties
> > >> sequentially
> > >> tpm: Replace the of_find_node_by_name() with dev of_node property
> > >> tpm: Moves the eventlog init functions to tpm_eventlog_init.c
> > >> tpm: Adds securityfs support for TPM2.0 eventlog
> > >>
> > >> drivers/char/tpm/Makefile| 13 +-
> > >> drivers/char/tpm/tpm-chip.c | 21 +---
> > >> drivers/char/tpm/tpm.h | 7 +-
> > >> drivers/char/tpm/tpm2.h | 85 +
> > >> drivers/char/tpm/tpm2_eventlog.c | 224
> > >> +++
> > >> drivers/char/tpm/tpm_acpi.c | 19 +--
> > >> drivers/char/tpm/tpm_eventlog.c | 154 +---
> > >> drivers/char/tpm/tpm_eventlog.h | 26 ++--
> > >> drivers/char/tpm/tpm_eventlog_init.c | 153
> > >> drivers/char/tpm/tpm_of.c| 65 ++
> > >> 10 files changed, 543 insertions(+), 224 deletions(-)
> > >> create mode 100644 drivers/char/tpm/tpm2.h
> > >> create mode 100644 drivers/char/tpm/tpm2_eventlog.c
> > >> create mode 100644 drivers/char/tpm/tpm_eventlog_init.c
> > >>
> > >>--
> > >>2.5.0
> > >>
> > >>
> > >>--
> > >>___
> > >>tpmdd-devel mailing list
> > >>tpmdd-devel@lists.sourceforge.net
> > >>https://lists.sourceforge.net/lists/listinfo/tpmdd-devel
> > >
> >
--
___
tpmdd-devel mailing list
tpmdd-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/tpmdd-devel