From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jason Gunthorpe Subject: Re: [PATCH] tpm_tis: Issue a TPM2_Shutdown for TPM2 devices. Date: Fri, 28 Apr 2017 12:43:25 -0600 Message-ID: <20170428184325.GA15327@obsidianresearch.com> References: <20170427182659.403-1-joshz@google.com> <20170428121226.7xnpfbxcjszrnz5d@intel.com> <20170428152748.GA32017@obsidianresearch.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: tpmdd-devel-bounces-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org To: Josh Zimmerman Cc: tpmdd-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org, rafael-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org List-Id: tpmdd-devel@lists.sourceforge.net On Fri, Apr 28, 2017 at 11:27:16AM -0700, Josh Zimmerman wrote: > It sounds like there are several intertwined issues here. I'd like to > keep this patch relatively minimal, so here are my current thoughts on > a path forward: > > * Refactor references to chip->ops to go through tpm_try_get_ops, as > mentioned in the previous patch As I said, if you rely on the fact that sysfs does not exist for TPM2 then this should already be done for the TPM2 case and an incremental later patch could solve this problem in sysfs to turn shutdown on for tpm1. As long as this logic is clearly documented it is probably an OK step for now. > * Have a common tpm_shutdown method in tpm-chip.c that device specific > drivers call (or that is just registered automatically for all > devices?) that nulls out chip->ops (to prevent any commands being > issued after tpm2_shutdown) and sends tpm2_shutdown I think the original patch got a bit stuck on exactly how to do this, but yes, somehow the common shutdown method is called. Either from the driver core (preferred, IHMO) or via patching every single TPM driver. > Before I spend too much time on this, does this seem like a reasonable > approach? Anything I'm missing? Don't think so, but suggest re-reading that huge original thread to be sure. Jason ------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, Slashdot.org! http://sdm.link/slashdot