RE: [tpmdd-devel] [PATCH] Add "shutdown" to "struct class".

From	"Winkler, Tomas" <tomas.winkler@intel.com>
Newsgroups	linux.kernel
Subject	RE: [tpmdd-devel] [PATCH] Add "shutdown" to "struct class".
Date	2017-05-15 18:20 +0200
Message-ID	<tHqT0-1Zv-15@gated-at.bofh.it> (permalink)
References	<tGsk9-3Ib-3@gated-at.bofh.it> <tGEEF-3FE-1@gated-at.bofh.it> <tHqzF-1DW-31@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

> 
> On Sat, May 13, 2017 at 12:43:11PM +0000, Winkler, Tomas wrote:
> > > The TPM class has some common shutdown code that must be executed
> > > for all drivers. This adds some needed functionality for that
> >
> > The issue with this is, that on some platforms the only storage can be
> > eMMC and TPM is using it,. It has to be ensured that the storage
> > device won't go down before TPM2_shutdown is called.  And there is no
> > direct device hierarchy to ensure an orderly shutdown.
> 
> Something will have to use the new device links stuff to define that
> dependency, but that seems unrelated to this patch?


Yep, it's not directly related to this specific patch, this is more relevant particularly to TPM2_shutdown. 

> Can the kernel really shut down the eMMC while the hidden platform stuff is
> still possibly using it? Goofy..

eMMC is single headed, so yes that can happen.

Tomas

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[PATCH] Add "shutdown" to "struct class". Josh Zimmerman <joshz@google.com> - 2017-05-13 01:40 +0200
  RE: [tpmdd-devel] [PATCH] Add "shutdown" to "struct class". "Winkler, Tomas" <tomas.winkler@intel.com> - 2017-05-13 14:50 +0200
    Re: [tpmdd-devel] [PATCH] Add "shutdown" to "struct class". Jason Gunthorpe <jgunthorpe@obsidianresearch.com> - 2017-05-15 18:00 +0200
      RE: [tpmdd-devel] [PATCH] Add "shutdown" to "struct class". "Winkler, Tomas" <tomas.winkler@intel.com> - 2017-05-15 18:20 +0200

csiph-web