Hi Kshitiz, > On 09.04.2024, at 11:48, Kshitiz Varshney <kshitiz.varsh...@nxp.com> wrote: > > Hi Jarkko, > > >> -----Original Message----- >> From: Jarkko Sakkinen <jar...@kernel.org> >> Sent: Wednesday, April 3, 2024 9:18 PM >> To: David Gstir <da...@sigma-star.at>; Mimi Zohar <zo...@linux.ibm.com>; >> James Bottomley <j...@linux.ibm.com>; Herbert Xu >> <herb...@gondor.apana.org.au>; David S. Miller <da...@davemloft.net> >> Cc: Shawn Guo <shawn...@kernel.org>; Jonathan Corbet >> <cor...@lwn.net>; Sascha Hauer <s.ha...@pengutronix.de>; Pengutronix >> Kernel Team <ker...@pengutronix.de>; Fabio Estevam >> <feste...@gmail.com>; dl-linux-imx <linux-...@nxp.com>; Ahmad Fatoum >> <a.fat...@pengutronix.de>; sigma star Kernel Team >> <upstream+...@sigma-star.at>; David Howells <dhowe...@redhat.com>; Li >> Yang <leoyang...@nxp.com>; Paul Moore <p...@paul-moore.com>; James >> Morris <jmor...@namei.org>; Serge E. Hallyn <se...@hallyn.com>; Paul E. >> McKenney <paul...@kernel.org>; Randy Dunlap <rdun...@infradead.org>; >> Catalin Marinas <catalin.mari...@arm.com>; Rafael J. Wysocki >> <rafael.j.wyso...@intel.com>; Tejun Heo <t...@kernel.org>; Steven Rostedt >> (Google) <rost...@goodmis.org>; linux-...@vger.kernel.org; linux- >> ker...@vger.kernel.org; linux-integr...@vger.kernel.org; >> keyri...@vger.kernel.org; linux-crypto@vger.kernel.org; linux-arm- >> ker...@lists.infradead.org; linuxppc-...@lists.ozlabs.org; linux-security- >> mod...@vger.kernel.org; Richard Weinberger <rich...@nod.at>; David >> Oberhollenzer <david.oberhollen...@sigma-star.at> >> Subject: [EXT] Re: [PATCH v8 6/6] docs: trusted-encrypted: add DCP as new >> trust source >> >> Caution: This is an external email. Please take care when clicking links or >> opening attachments. When in doubt, report the message using the 'Report >> this email' button >> >> >> On Wed Apr 3, 2024 at 10:21 AM EEST, David Gstir wrote: >>> Update the documentation for trusted and encrypted KEYS with DCP as >>> new trust source: >>> >>> - Describe security properties of DCP trust source >>> - Describe key usage >>> - Document blob format >>> >>> Co-developed-by: Richard Weinberger <rich...@nod.at> >>> Signed-off-by: Richard Weinberger <rich...@nod.at> >>> Co-developed-by: David Oberhollenzer >>> <david.oberhollen...@sigma-star.at> >>> Signed-off-by: David Oberhollenzer <david.oberhollen...@sigma-star.at> >>> Signed-off-by: David Gstir <da...@sigma-star.at> >>> --- >>> .../security/keys/trusted-encrypted.rst | 53 +++++++++++++++++++ >>> security/keys/trusted-keys/trusted_dcp.c | 19 +++++++ >>> 2 files changed, 72 insertions(+) >>> >>> diff --git a/Documentation/security/keys/trusted-encrypted.rst >>> b/Documentation/security/keys/trusted-encrypted.rst >>> index e989b9802f92..f4d7e162d5e4 100644 >>> --- a/Documentation/security/keys/trusted-encrypted.rst >>> +++ b/Documentation/security/keys/trusted-encrypted.rst >>> @@ -42,6 +42,14 @@ safe. >>> randomly generated and fused into each SoC at manufacturing time. >>> Otherwise, a common fixed test key is used instead. >>> >>> + (4) DCP (Data Co-Processor: crypto accelerator of various i.MX >>> + SoCs) >>> + >>> + Rooted to a one-time programmable key (OTP) that is generally >> burnt >>> + in the on-chip fuses and is accessible to the DCP encryption >>> engine >> only. >>> + DCP provides two keys that can be used as root of trust: the OTP >> key >>> + and the UNIQUE key. Default is to use the UNIQUE key, but >>> selecting >>> + the OTP key can be done via a module parameter >> (dcp_use_otp_key). >>> + >>> * Execution isolation >>> >>> (1) TPM >>> @@ -57,6 +65,12 @@ safe. >>> >>> Fixed set of operations running in isolated execution environment. >>> >>> + (4) DCP >>> + >>> + Fixed set of cryptographic operations running in isolated >>> execution >>> + environment. Only basic blob key encryption is executed there. >>> + The actual key sealing/unsealing is done on main processor/kernel >> space. >>> + >>> * Optional binding to platform integrity state >>> >>> (1) TPM >>> @@ -79,6 +93,11 @@ safe. >>> Relies on the High Assurance Boot (HAB) mechanism of NXP SoCs >>> for platform integrity. >>> >>> + (4) DCP >>> + >>> + Relies on Secure/Trusted boot process (called HAB by vendor) for >>> + platform integrity. >>> + >>> * Interfaces and APIs >>> >>> (1) TPM >>> @@ -94,6 +113,11 @@ safe. >>> >>> Interface is specific to silicon vendor. >>> >>> + (4) DCP >>> + >>> + Vendor-specific API that is implemented as part of the DCP crypto >> driver in >>> + ``drivers/crypto/mxs-dcp.c``. >>> + >>> * Threat model >>> >>> The strength and appropriateness of a particular trust source >>> for a given @@ -129,6 +153,13 @@ selected trust source: >>> CAAM HWRNG, enable CRYPTO_DEV_FSL_CAAM_RNG_API and ensure >> the device >>> is probed. >>> >>> + * DCP (Data Co-Processor: crypto accelerator of various i.MX SoCs) >>> + >>> + The DCP hardware device itself does not provide a dedicated RNG >> interface, >>> + so the kernel default RNG is used. SoCs with DCP like the i.MX6ULL do >> have >>> + a dedicated hardware RNG that is independent from DCP which can be >> enabled >>> + to back the kernel RNG. >>> + >>> Users may override this by specifying ``trusted.rng=kernel`` on the >>> kernel command-line to override the used RNG with the kernel's random >> number pool. >>> >>> @@ -231,6 +262,19 @@ Usage:: >>> CAAM-specific format. The key length for new keys is always in bytes. >>> Trusted Keys can be 32 - 128 bytes (256 - 1024 bits). >>> >>> +Trusted Keys usage: DCP >>> +----------------------- >>> + >>> +Usage:: >>> + >>> + keyctl add trusted name "new keylen" ring >>> + keyctl add trusted name "load hex_blob" ring >>> + keyctl print keyid >>> + >>> +"keyctl print" returns an ASCII hex copy of the sealed key, which is >>> +in format specific to this DCP key-blob implementation. The key >>> +length for new keys is always in bytes. Trusted Keys can be 32 - 128 bytes >> (256 - 1024 bits). >>> + >>> Encrypted Keys usage >>> -------------------- >>> >>> @@ -426,3 +470,12 @@ string length. >>> privkey is the binary representation of TPM2B_PUBLIC excluding the >>> initial TPM2B header which can be reconstructed from the ASN.1 octed >>> string length. >>> + >>> +DCP Blob Format >>> +--------------- >>> + >>> +.. kernel-doc:: security/keys/trusted-keys/trusted_dcp.c >>> + :doc: dcp blob format >>> + >>> +.. kernel-doc:: security/keys/trusted-keys/trusted_dcp.c >>> + :identifiers: struct dcp_blob_fmt >>> diff --git a/security/keys/trusted-keys/trusted_dcp.c >>> b/security/keys/trusted-keys/trusted_dcp.c >>> index 16c44aafeab3..b5f81a05be36 100644 >>> --- a/security/keys/trusted-keys/trusted_dcp.c >>> +++ b/security/keys/trusted-keys/trusted_dcp.c >>> @@ -19,6 +19,25 @@ >>> #define DCP_BLOB_VERSION 1 >>> #define DCP_BLOB_AUTHLEN 16 >>> >>> +/** >>> + * DOC: dcp blob format >>> + * >>> + * The Data Co-Processor (DCP) provides hardware-bound AES keys using >>> +its >>> + * AES encryption engine only. It does not provide direct key >> sealing/unsealing. >>> + * To make DCP hardware encryption keys usable as trust source, we >>> +define >>> + * our own custom format that uses a hardware-bound key to secure the >>> +sealing >>> + * key stored in the key blob. >>> + * >>> + * Whenever a new trusted key using DCP is generated, we generate a >>> +random 128-bit >>> + * blob encryption key (BEK) and 128-bit nonce. The BEK and nonce are >>> +used to >>> + * encrypt the trusted key payload using AES-128-GCM. >>> + * >>> + * The BEK itself is encrypted using the hardware-bound key using the >>> +DCP's AES >>> + * encryption engine with AES-128-ECB. The encrypted BEK, generated >>> +nonce, >>> + * BEK-encrypted payload and authentication tag make up the blob >>> +format together >>> + * with a version number, payload length and authentication tag. >>> + */ >>> + >>> /** >>> * struct dcp_blob_fmt - DCP BLOB format. >>> * >> >> Reviewed-by: Jarkko Sakkinen <jar...@kernel.org> >> >> I can only test that this does not break a machine without the hardware >> feature. >> >> Is there anyone who could possibly peer test these patches? > I am already working on testing this patchset on i.MX6 platform.
Did you get around to testing this? I’d greatly appreciate a Tested-by for this. :-) Thanks! BR, David