From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759554AbdEOPvW (ORCPT ); Mon, 15 May 2017 11:51:22 -0400 Received: from quartz.orcorp.ca ([184.70.90.242]:56754 "EHLO quartz.orcorp.ca" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751952AbdEOPvV (ORCPT ); Mon, 15 May 2017 11:51:21 -0400 Date: Mon, 15 May 2017 09:51:10 -0600 From: Jason Gunthorpe To: "Winkler, Tomas" Cc: Josh Zimmerman , Greg Kroah-Hartman , "linux-kernel@vger.kernel.org" , "tpmdd-devel@lists.sourceforge.net" , "Rafael J. Wysocki" Subject: Re: [tpmdd-devel] [PATCH] Add "shutdown" to "struct class". Message-ID: <20170515155110.GC3433@obsidianresearch.com> References: <20170512233203.23659-1-joshz@google.com> <5B8DA87D05A7694D9FA63FD143655C1B641D7A27@hasmsx108.ger.corp.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <5B8DA87D05A7694D9FA63FD143655C1B641D7A27@hasmsx108.ger.corp.intel.com> User-Agent: Mutt/1.5.24 (2015-08-30) X-Broken-Reverse-DNS: no host name found for IP address 10.0.0.156 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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? Can the kernel really shut down the eMMC while the hidden platform stuff is still possibly using it? Goofy.. Jason