From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1750919AbdAaVqn (ORCPT ); Tue, 31 Jan 2017 16:46:43 -0500 Received: from namei.org ([65.99.196.166]:34103 "EHLO namei.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750726AbdAaVqm (ORCPT ); Tue, 31 Jan 2017 16:46:42 -0500 Date: Wed, 1 Feb 2017 08:46:32 +1100 (AEDT) From: James Morris To: Jarkko Sakkinen cc: Kenneth Goldman , "moderated list:TPM DEVICE DRIVER" , open list , linux-security-module@vger.kernel.org Subject: Re: Fwd: Re: [tpmdd-devel] [PATCH v9 2/2] tpm: add securityfs support,for TPM 2.0 firmware event log In-Reply-To: <20170131203452.523ckw4aphxdsndq@intel.com> Message-ID: References: <588F09A2.4090502@linux.vnet.ibm.com> <20170131174659.b6njebycqzd5ur6f@intel.com> <20170131203452.523ckw4aphxdsndq@intel.com> User-Agent: Alpine 2.20 (LRH 67 2015-01-07) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, 31 Jan 2017, Jarkko Sakkinen wrote: > James, > > The discussion is about two features: > > 1. Extension to tpm_pcr_extend() (used by IMA) to extend all PCR banks > instead of just SHA-1 banks. It is recommended by TCG to do so in > order to prevent malicious use of PCRs. > 2. TPM 2.0 event log with backend support for OF device tree (for > getting address where you can grab it). > > These are required as baseline to implement full TPM 2.0 support for > IMA. The commits are fairly well baked and went through many iterations. > I've tested tpm_pcr_extend() patches. I haven't tested event log patches > but have extensively reviewed them and Ken Goldman has tested them with > POWER hardware. > > I don't believe that there is major risk to put them already into 4.11 > but it is fairly late so I just want a second opinion before putting > them into pull request. > I'll take this for 4.11. IMA + TPM 2.0 is still developmental and not in wide use, afaik. -- James Morris From mboxrd@z Thu Jan 1 00:00:00 1970 From: James Morris Subject: Re: Fwd: Re: [tpmdd-devel] [PATCH v9 2/2] tpm: add securityfs support,for TPM 2.0 firmware event log Date: Wed, 1 Feb 2017 08:46:32 +1100 (AEDT) Message-ID: References: <588F09A2.4090502@linux.vnet.ibm.com> <20170131174659.b6njebycqzd5ur6f@intel.com> <20170131203452.523ckw4aphxdsndq@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Return-path: In-Reply-To: <20170131203452.523ckw4aphxdsndq@intel.com> Sender: owner-linux-security-module@vger.kernel.org To: Jarkko Sakkinen Cc: Kenneth Goldman , "moderated list:TPM DEVICE DRIVER" , open list , linux-security-module@vger.kernel.org List-Id: tpmdd-devel@lists.sourceforge.net On Tue, 31 Jan 2017, Jarkko Sakkinen wrote: > James, > > The discussion is about two features: > > 1. Extension to tpm_pcr_extend() (used by IMA) to extend all PCR banks > instead of just SHA-1 banks. It is recommended by TCG to do so in > order to prevent malicious use of PCRs. > 2. TPM 2.0 event log with backend support for OF device tree (for > getting address where you can grab it). > > These are required as baseline to implement full TPM 2.0 support for > IMA. The commits are fairly well baked and went through many iterations. > I've tested tpm_pcr_extend() patches. I haven't tested event log patches > but have extensively reviewed them and Ken Goldman has tested them with > POWER hardware. > > I don't believe that there is major risk to put them already into 4.11 > but it is fairly late so I just want a second opinion before putting > them into pull request. > I'll take this for 4.11. IMA + TPM 2.0 is still developmental and not in wide use, afaik. -- James Morris