lkml.org 
[lkml]   [2018]   [Nov]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v5 16/17] tpm: take TPM chip power gating out of tpm_transmit()
On Thu, Nov 08, 2018 at 06:38:59PM +0000, Winkler, Tomas wrote:
> > Call tpm_chip_start() and tpm_chip_stop() in
> >
> > * tpm_try_get_ops() and tpm_put_ops()
> > * tpm_chip_register()
> > * tpm2_del_space()
> >
> > And remove these calls from tpm_transmit(). The core reason for this change
> > is that in tpm_vtpm_proxy a locality change requires a virtual TPM command
> > (a command made up just for that driver).
> >
> I don't think you can do that, locality has to be request for each
> command, as for example tboot can request higher locality any time.

That could be a potential problem. How tboot intervention gets prevented
without this patch?

> Same for cmd_ready()/go_idle() powergatin, you will prevent the whole
> platform entering power save state.

Why would you want that in a middle of using the TPM anyway?

> Thanks
> Tomas

/Jarkko

\
 
 \ /
  Last update: 2018-11-09 00:07    [W:0.190 / U:2.572 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site