On Tue, 13 Oct 2020 at 07:13, Jarkko Sakkinen jarkko.sakkinen@linux.intel.com wrote:
On Wed, Oct 07, 2020 at 03:37:45PM +0530, Sumit Garg wrote:
Current trusted keys framework is tightly coupled to use TPM device as an underlying implementation which makes it difficult for implementations like Trusted Execution Environment (TEE) etc. to provide trusted keys support in case platform doesn't posses a TPM device.
Add a generic trusted keys framework where underlying implementations can be easily plugged in. Create struct trusted_key_ops to achieve this, which contains necessary functions of a backend.
Also, add a module parameter in order to select a particular trust source in case a platform support multiple trust sources.
Suggested-by: Jarkko Sakkinen jarkko.sakkinen@linux.intel.com Signed-off-by: Sumit Garg sumit.garg@linaro.org
This is exactly kind of place where I think static_call() should be taken into use, which is a v5.10 feature [1]. For background and context, I'd read [2].
This looks like an interesting feature. But I am not sure about the real benefits that it will provide in case of trusted keys. If we are looking at it performance wise then I think the gain will be negligible when compared with slow TPM communication interface (eg. SPI, I2C) or when compared with context switching involved in TEE.
Also, it requires arch specific support too which currently seems to be limited to x86 only.
The other thing that I see that does not make much else than additional complexity, is trusted_tpm.ko. We can do with one trusted.ko.
Current implementation only builds a single trusted.ko module. There isn't any trusted_tpm.ko.
-Sumit
I'd also *guess* that the static_call() mechanism does not work accross module boundaries.
[1] https://lore.kernel.org/lkml/20201012155542.GA3557765@gmail.com/ [2] https://lwn.net/Articles/815908/
/Jarkko