From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751615AbcGRSIC (ORCPT ); Mon, 18 Jul 2016 14:08:02 -0400 Received: from mga02.intel.com ([134.134.136.20]:17588 "EHLO mga02.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751437AbcGRSIA (ORCPT ); Mon, 18 Jul 2016 14:08:00 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.28,385,1464678000"; d="scan'208";a="1009221776" Date: Mon, 18 Jul 2016 21:07:36 +0300 From: Jarkko Sakkinen To: Ed Swierk Cc: tpmdd-devel@lists.sourceforge.net, linux-kernel@vger.kernel.org, linux-security-module@vger.kernel.org, jgunthorpe@obsidianresearch.com, stefanb@us.ibm.com Subject: Re: [PATCH v9 0/5] tpm: Command duration logging and chip-specific override Message-ID: <20160718180736.GB31463@intel.com> References: <1466557831-113440-1-git-send-email-eswierk@skyportsystems.com> <1468426776-42762-1-git-send-email-eswierk@skyportsystems.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1468426776-42762-1-git-send-email-eswierk@skyportsystems.com> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Jul 13, 2016 at 09:19:31AM -0700, Ed Swierk wrote: > v9: Include command duration in existing error messages rather than > logging an extra debug message. Rebase onto Jarkko's tree. > > v8: Fix v7 goof-up in tpm_getcap(). > > v7: Use tpm_getcap() instead of a redundant new function. > > v6: Split tpm_get_cap_prop() out of tpm_get_timeouts(); always return > error on TPM command failure. > > v5: Use msecs_to_jiffies() instead of * HZ / 1000. > > v4: Rework tpm_get_timeouts() to allow overriding both timeouts and > durations via a single callback. > > This series > - improves TPM command error reporting > - adds optional logging of TPM command durations > - allows chip-specific override of command durations as well as protocol > timeouts > - overrides ST19NP18 TPM command duration to avoid lockups For the next version please write something more readable. Describe the motivation and solution in big picture, why and how. This list does no good because it just mimics the list of patches. /Jarkko > Ed Swierk (5): > tpm_tis: Improve reporting of IO errors > tpm: Add optional logging of TPM command durations > tpm: Clean up reading of timeout and duration capabilities > tpm: Allow TPM chip drivers to override reported command durations > tpm_tis: Increase ST19NP18 TPM command duration to avoid chip lockup > > drivers/char/tpm/tpm-interface.c | 219 ++++++++++++++++++++------------------- > drivers/char/tpm/tpm_tis_core.c | 46 ++++---- > include/linux/tpm.h | 3 +- > 3 files changed, 136 insertions(+), 132 deletions(-) > > -- > 1.9.1 >