On Fri, Feb 08, 2019 at 04:18:40PM -0500, Stefan Berger wrote:
On 2/8/19 3:46 PM, Jarkko Sakkinen wrote:
On Fri, Feb 08, 2019 at 03:32:32PM -0500, Stefan Berger wrote:
tpm_del_char_device also needs the start/stop!
Done and updated the commit message to have all the call sites:
* tpm_chip_register() * tpm_class_shutdown() * tpm_del_char_device() * tpm_pm_suspend() * tpm_try_get_ops() and tpm_put_ops() * tpm2_del_space()I tested this now with tpm_vtpm_proxy, TPM 1.2, (TIS|CRB) + TPM 2.0. Looks good now - rock solid so to say. And while we are at it ... :-)
Thank you for your support. I don't think these quite distruptive changes would have made in without any growing pains. The way tpm_transmit() used to be had become quite a mess with all the nested calls and weird locking flags. Now we have a good baseline to forward :-)
I rebased the branches to the latest security/next-general. Probably do the PR after the middle week (ETA Thu).
/Jarkko