Re: [PATCH v1] tpm2: add longer timeout for verify signature command
From: Jarkko Sakkinen <jarkko@kernel.org>
Date: 2021-05-10 17:51:27
Also in:
lkml
On Mon, May 10, 2021 at 05:27:19PM +0300, amirmizi6@gmail.com wrote:
From: Amir Mizinski <redacted> TPM2_CC_VERIFY_SIGNATURE(0x177) Current timeout does not apply to usage with RSA 3070-bit keys.
I don't understand what this sentence means. Better excuse for making the whole change would be to: 1. If possible put a snippet of the klog transcript what happens to you. 2. You probably want rationalize this change for the reason that, since the TPM PC Client specification does not specify any specific number, and you have a corner case to show, it's best to pick the longest timeout, i.e. TPM_LONG_LONG.
Additional time may be required for usage with RSA 3070-bit keys. Therefore, the timeout of TPM2_CC_VERIFY_SIGNATURE is set to 3 minutes (TPM_LONG_LONG). Signed-off-by: Amir Mizinski <redacted>
Please, re-phrase : "Set duration for TPM2_CC_VERIFY_SIGNATUE to TPM_LONG_LONG (3 minutes)". Imperative way to express things is always better, and "timeout" is a concept of its own, separate from "duration" in TPM jargon.
quoted hunk ↗ jump to hunk
--- drivers/char/tpm/tpm2-cmd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)diff --git a/drivers/char/tpm/tpm2-cmd.c b/drivers/char/tpm/tpm2-cmd.c index eff1f12..235a454 100644 --- a/drivers/char/tpm/tpm2-cmd.c +++ b/drivers/char/tpm/tpm2-cmd.c@@ -87,7 +87,7 @@ static u8 tpm2_ordinal_duration_index(u32 ordinal) return TPM_MEDIUM; case TPM2_CC_VERIFY_SIGNATURE: /* 177 */ - return TPM_LONG; + return TPM_LONG_LONG; case TPM2_CC_PCR_EXTEND: /* 182 */ return TPM_MEDIUM;-- 2.7.4
The commit message needs rework because now it makes no sense. With the correct rationalization this probably would make sense. /Jarkko