Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752832AbdHKLO1 (ORCPT ); Fri, 11 Aug 2017 07:14:27 -0400 Received: from mga11.intel.com ([192.55.52.93]:40230 "EHLO mga11.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751618AbdHKLOZ (ORCPT ); Fri, 11 Aug 2017 07:14:25 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.41,357,1498546800"; d="scan'208";a="1002565027" Date: Fri, 11 Aug 2017 14:14:21 +0300 From: Jarkko Sakkinen To: Peter Huewe Cc: Ken Goldman , linux-ima-devel@lists.sourceforge.net, linux-security-module@vger.kernel.org, tpmdd-devel@lists.sourceforge.net, linux-kernel@vger.kernel.org Subject: Re: Re: [tpmdd-devel] [PATCH] tpm: improve tpm_tis send() performance by ignoring burstcount Message-ID: <20170811111421.bg2we53rdeecjtac@linux.intel.com> References: <20170807114632.1339-1-nayna@linux.vnet.ibm.com> <20170808191145.kggmoczd5laiccrn@linux.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo User-Agent: NeoMutt/20170609 (1.8.3) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2413 Lines: 56 On Wed, Aug 09, 2017 at 11:00:36PM +0200, Peter Huewe wrote: > Hi Ken, > (again speaking only on my behalf, not my employer) > > > Does anyone know of platforms where this occurs? > > I suspect (but not sure) that the days of SuperIO connecting floppy > > drives, printer ports, and PS/2 mouse ports on the LPC bus are over, and > > such legacy systems will not have a TPM. Would SuperIO even support the > > special TPM LPC bus cycles? > > Since we are the linux kernel, we do have to care for legacy devices. > And a system with LPC, PS2Mouse on SuperIO and a TPM are not that uncommon. > > And heck, we even have support for 1.1b TPM devices.... > > > >> One more viewpoint: TCG must added the burst count for a reason (might > >> be very well related what Peter said). Is ignoring it something that TCG > >> recommends? Not following standard exactly in the driver code sometimes > >> makes sense on *small details* but I would not say that this a small > >> detail... > > > I checked with the TCG's device driver work group (DDWG). Both the spec > > editor and 3 TPM vendors - Infineon, Nuvoton, and ST Micro - agreed that > > ignoring burst count may incur wait states but nothing more. Operations > > will still be successful. > > Interesting - let me check with Georg tomorrow. > Unfortunately I do not have access to my tcg mails from home (since I'm not working :), > but did you _explicitly_ talk about LPC and the system? > I'm sure the TPM does not care about the waitstates... > > If my memory does not betray me, > it is actually possible to "freeze up" a system completly by flooding the lpc bus. > Let me double check tomorrow... > > > In anycase - I really would like to see a much more performant tpm subsystem - > however it will be quite an effort with a lot of legacy testing. > (which I unfortunately cannot spend on my private time ... and also of course lacking test systems). > > Thanks, > Peter I would like to see tpm_msleep() wrapper to replace current msleep() usage across the subsystem before considering this. I.e. wrapper that internally uses usleep_range(). This way we can mechanically convert everything to a more low latency option. This should have been done already for patch that Mini and Nayna provided instead of open coding stuff. That change is something that can be applied right now. On the other hand, this is a very controversial change. /Jarkko