From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga09.intel.com ([134.134.136.24]:21530 "EHLO mga09.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750846AbdLXTrI (ORCPT ); Sun, 24 Dec 2017 14:47:08 -0500 Date: Sun, 24 Dec 2017 21:47:03 +0200 From: Jarkko Sakkinen To: Azhar Shaikh Cc: jgg@ziepe.ca, peterhuewe@gmx.de, linux-security-module@vger.kernel.org, linux-integrity@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v6 0/2] Fix corner cases with disabling CLKRUN in tpm_tis Message-ID: <20171224194703.zo75mrwqmruqe65f@linux.intel.com> References: <1513973624-44226-1-git-send-email-azhar.shaikh@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <1513973624-44226-1-git-send-email-azhar.shaikh@intel.com> Sender: linux-integrity-owner@vger.kernel.org List-ID: On Fri, Dec 22, 2017 at 12:13:42PM -0800, Azhar Shaikh wrote: > Changes from v1: > - Patch 1: "tpm: Keep CLKRUN enabled throughout the duration of transmit_cmd()" > - Add NULL checks before calling clk_toggle callback > - Use IS_ENABLED instead of ifdef in tpm_tis_clkrun_toggle() > - Do not call tpm_platform_begin_xfer() and tpm_platform_end_xfer() > from tpm_tis_clkrun_toggle(). Make them static again. > > - Patch 2: "tpm_tis: Move ilb_base_addr to tpm_tis_tcg_phy" > - This is a new patch in this series as per suggestion from Jason. > - Is the current implementation ok or I should move the code in tpm_tis_pnp_remove() > and tpm_tis_plat_remove() inside tpm_tis_remove(). That way all the unmapping > can be done in one place, instead of 3 different places now. Also the unmapping > in tpm_tis_init() can be moved to tpm_tis_remove(), since in case of error > tpm_tis_core_init() calls tpm_tis_remove(). Kindly suggest. > > Changes from v2: > - Patch 1: "tpm: Keep CLKRUN enabled throughout the duration of transmit_cmd()" > - No changes > > - Patch 2: "tpm_tis: Move ilb_base_addr to tpm_tis_tcg_phy" > - Updated is_bsw() function to have the #ifdef CONFIG_X86 check within the function > itself. Also removed the #ifdef CONFIG_X86 from all other places around is_bsw() > > Changes from v3: > - Patch 1: "tpm: Keep CLKRUN enabled throughout the duration of transmit_cmd()" > - Change function name from clk_toggle to clk_enable > - Update the commit message. > > - Patch 2: "tpm_tis: Move ilb_base_addr to tpm_tis_tcg_phy" > - No changes > > Changes from v4: > - The numbering of patches is now interchanged. > > - Patch 1: "tpm_tis: Move ilb_base_addr to tpm_tis_data" > - Had to move ilb_base_addr to tpm_tis_data, from tpm_tis_tcg_phy. > Since the ioremapping of ilb_base_addr had to be done before any TPM access, > hence moved the variable to tpm_tis_data. > - Also move the ioremapping of ilb_base_addr from tpm_tis_init() to > tpm_tis_core_init() i.e. before any TPM access is done. > - Rename marco LPC_CNTRL_REG_OFFSET to LPC_CNTRL_OFFSET > - Update the commit message. > > - Patch 2: "tpm: Keep CLKRUN enabled throughout the duration of transmit_cmd()" > - Remove the functions tpm_platform_begin_xfer() and tpm_platform_end_xfer() > - Move the code from these functions to tpm_tis_clkrun_enable(). > > Changes from v5: > - Patch 1: "tpm_tis: Move ilb_base_addr to tpm_tis_data" > - No changes > > - Patch 2: "tpm: Keep CLKRUN enabled throughout the duration of transmit_cmd()" > - Update the commit message. > > Changes from v6: > - - Patch 1: "tpm_tis: Move ilb_base_addr to tpm_tis_data" > - No changes > > - Patch 2: "tpm: Keep CLKRUN enabled throughout the duration of transmit_cmd()" > - chip->ops was set to NULL in tpm_del_char_device() called from tpm_chip_unregister() in > error/driver exit path. Fix this code. (Suggested by Javier Martinez Canillas and Jason Gunthorpe) > - Add a comment in tpm_tis_clkrun_enable() function. > > Azhar Shaikh (2): > tpm_tis: Move ilb_base_addr to tpm_tis_data > tpm: Keep CLKRUN enabled throughout the duration of transmit_cmd() > > drivers/char/tpm/tpm-interface.c | 6 ++ > drivers/char/tpm/tpm_tis.c | 121 ++++++--------------------------------- > drivers/char/tpm/tpm_tis_core.c | 111 +++++++++++++++++++++++++++++++++-- > drivers/char/tpm/tpm_tis_core.h | 17 ++++++ > include/linux/tpm.h | 1 + > 5 files changed, 147 insertions(+), 109 deletions(-) > > -- > 1.9.1 > Reviewed-by: Jarkko Sakkinen Tested-by: Jarkko Sakkinen (with SPI TPM on Sandybridge machine) /Jarkko From mboxrd@z Thu Jan 1 00:00:00 1970 From: jarkko.sakkinen@linux.intel.com (Jarkko Sakkinen) Date: Sun, 24 Dec 2017 21:47:03 +0200 Subject: [PATCH v6 0/2] Fix corner cases with disabling CLKRUN in tpm_tis In-Reply-To: <1513973624-44226-1-git-send-email-azhar.shaikh@intel.com> References: <1513973624-44226-1-git-send-email-azhar.shaikh@intel.com> Message-ID: <20171224194703.zo75mrwqmruqe65f@linux.intel.com> To: linux-security-module@vger.kernel.org List-Id: linux-security-module.vger.kernel.org On Fri, Dec 22, 2017 at 12:13:42PM -0800, Azhar Shaikh wrote: > Changes from v1: > - Patch 1: "tpm: Keep CLKRUN enabled throughout the duration of transmit_cmd()" > - Add NULL checks before calling clk_toggle callback > - Use IS_ENABLED instead of ifdef in tpm_tis_clkrun_toggle() > - Do not call tpm_platform_begin_xfer() and tpm_platform_end_xfer() > from tpm_tis_clkrun_toggle(). Make them static again. > > - Patch 2: "tpm_tis: Move ilb_base_addr to tpm_tis_tcg_phy" > - This is a new patch in this series as per suggestion from Jason. > - Is the current implementation ok or I should move the code in tpm_tis_pnp_remove() > and tpm_tis_plat_remove() inside tpm_tis_remove(). That way all the unmapping > can be done in one place, instead of 3 different places now. Also the unmapping > in tpm_tis_init() can be moved to tpm_tis_remove(), since in case of error > tpm_tis_core_init() calls tpm_tis_remove(). Kindly suggest. > > Changes from v2: > - Patch 1: "tpm: Keep CLKRUN enabled throughout the duration of transmit_cmd()" > - No changes > > - Patch 2: "tpm_tis: Move ilb_base_addr to tpm_tis_tcg_phy" > - Updated is_bsw() function to have the #ifdef CONFIG_X86 check within the function > itself. Also removed the #ifdef CONFIG_X86 from all other places around is_bsw() > > Changes from v3: > - Patch 1: "tpm: Keep CLKRUN enabled throughout the duration of transmit_cmd()" > - Change function name from clk_toggle to clk_enable > - Update the commit message. > > - Patch 2: "tpm_tis: Move ilb_base_addr to tpm_tis_tcg_phy" > - No changes > > Changes from v4: > - The numbering of patches is now interchanged. > > - Patch 1: "tpm_tis: Move ilb_base_addr to tpm_tis_data" > - Had to move ilb_base_addr to tpm_tis_data, from tpm_tis_tcg_phy. > Since the ioremapping of ilb_base_addr had to be done before any TPM access, > hence moved the variable to tpm_tis_data. > - Also move the ioremapping of ilb_base_addr from tpm_tis_init() to > tpm_tis_core_init() i.e. before any TPM access is done. > - Rename marco LPC_CNTRL_REG_OFFSET to LPC_CNTRL_OFFSET > - Update the commit message. > > - Patch 2: "tpm: Keep CLKRUN enabled throughout the duration of transmit_cmd()" > - Remove the functions tpm_platform_begin_xfer() and tpm_platform_end_xfer() > - Move the code from these functions to tpm_tis_clkrun_enable(). > > Changes from v5: > - Patch 1: "tpm_tis: Move ilb_base_addr to tpm_tis_data" > - No changes > > - Patch 2: "tpm: Keep CLKRUN enabled throughout the duration of transmit_cmd()" > - Update the commit message. > > Changes from v6: > - - Patch 1: "tpm_tis: Move ilb_base_addr to tpm_tis_data" > - No changes > > - Patch 2: "tpm: Keep CLKRUN enabled throughout the duration of transmit_cmd()" > - chip->ops was set to NULL in tpm_del_char_device() called from tpm_chip_unregister() in > error/driver exit path. Fix this code. (Suggested by Javier Martinez Canillas and Jason Gunthorpe) > - Add a comment in tpm_tis_clkrun_enable() function. > > Azhar Shaikh (2): > tpm_tis: Move ilb_base_addr to tpm_tis_data > tpm: Keep CLKRUN enabled throughout the duration of transmit_cmd() > > drivers/char/tpm/tpm-interface.c | 6 ++ > drivers/char/tpm/tpm_tis.c | 121 ++++++--------------------------------- > drivers/char/tpm/tpm_tis_core.c | 111 +++++++++++++++++++++++++++++++++-- > drivers/char/tpm/tpm_tis_core.h | 17 ++++++ > include/linux/tpm.h | 1 + > 5 files changed, 147 insertions(+), 109 deletions(-) > > -- > 1.9.1 > Reviewed-by: Jarkko Sakkinen Tested-by: Jarkko Sakkinen (with SPI TPM on Sandybridge machine) /Jarkko -- To unsubscribe from this list: send the line "unsubscribe linux-security-module" in the body of a message to majordomo at vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html