Re: [PATCH] tpm: Add explicit chip->ops locking for sysfs attributes.

From: Jason Gunthorpe
Date: Mon Nov 20 2017 - 18:17:40 EST

On Mon, Nov 20, 2017 at 02:45:23PM -0800, Guenter Roeck wrote:

> "tpm: Enable sysfs support for TPM2 devices
> Access to chip->ops on TPM2 devices requires an explicit lock,
> since the pointer is set to NULL in tpm_class_shutdown().
> Implement that lock for sysfs access functions and enable sysfs
> support for TPM2 devices."

Wait.. one of the reasons we let it go with no sysfs for so long was
because there was not many sysfs files that were compatible with tpm2?

For TPM2 we have sort of had an API break of sorts from TPM1 in a
couple places around sysfs, and I would like to not re-introduce any
badly designed sysfs files for TPM2..

So.. When you apply this patch, what changes actually happen in the
sysfs directory?