On 09/01/2016 08:22 PM, Jarkko Sakkinen wrote:
> 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.
Sure Jarkko. Thanks for this !! I have included the feedbacks and have
posted V4 version of the patches just now.
Thanks & Regards,
- Nayna
>
> /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
