Re: [v4.0][v4.1][v4.2][Regression] tpm: fix raciness of PPI interface lookup

Jason Gunthorpe
2015-09-14T22:27:07Z

On Mon, Sep 14, 2015 at 05:36:07PM -0400, Joseph Salisbury wrote:
> I was hoping to get your feedback, since you are the patch author. Do
> you think gathering any additional data will help diagnose this
> issue,

Jarkko may have more to add, but from my perspective, this should have
been a no-functional change re-work patch. I'm surprised this has been
tagged, since it doesn't seem to really change much at all. I looked
again and all the code motion looks right to me.

The biggest change is that we don't call acpi_walk_namespace to get
the acpi_dev_handle, instead this now comes from pnp_acpi_device. It
would be useful to know if that is the source of the difference. ie
are the two methods are giving different answers on this hardware?

I'd also test with this line:

+ if (pnp_acpi_device(pnp_dev))
+ acpi_dev_handle = pnp_acpi_device(pnp_dev)->handle;

Changed to 'acpi_dev_handle = NULL;' - that should totally disable TPM
access to ACPI.

The ACPI code doesn't actually do anything if sysfs files are not
touched, I'm really not sure how it could be at fault.. Maybe also
comment out the sysfs_create_group, could something be touching the
files in userspace?

Honestly, I don't know enough about ACPI to guess what is going
on. Hopefully Jarkko does..

Jason