> -----Original Message----- > From: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx> > Sent: Tuesday, September 6, 2022 12:13 PM > To: Pankaj Gupta <pankaj.gupta@xxxxxxx> > Cc: jarkko@xxxxxxxxxx; a.fatoum@xxxxxxxxxxxxxx; Jason@xxxxxxxxx; > jejb@xxxxxxxxxxxxx; zohar@xxxxxxxxxxxxx; dhowells@xxxxxxxxxx; > sumit.garg@xxxxxxxxxx; david@xxxxxxxxxxxxx; michael@xxxxxxxx; > john.ernberg@xxxxxxxx; jmorris@xxxxxxxxx; serge@xxxxxxxxxx; > davem@xxxxxxxxxxxxx; j.luebbe@xxxxxxxxxxxxxx; ebiggers@xxxxxxxxxx; > richard@xxxxxx; keyrings@xxxxxxxxxxxxxxx; linux-crypto@xxxxxxxxxxxxxxx; > linux-integrity@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; linux- > security-module@xxxxxxxxxxxxxxx; Sahil Malhotra > <sahil.malhotra@xxxxxxx>; Kshitiz Varshney <kshitiz.varshney@xxxxxxx>; > Horia Geanta <horia.geanta@xxxxxxx>; Varun Sethi <V.Sethi@xxxxxxx> > Subject: [EXT] Re: [RFC PATCH HBK: 2/8] hw-bound-key: flag-is_hbk added to > the tfm > > Caution: EXT Email > > On Tue, Sep 06, 2022 at 12:21:51PM +0530, Pankaj Gupta wrote: > > Consumer of the kernel crypto api, after allocating the > > transformation, sets this flag based on the basis of the type of key > > consumer has. This helps: > > > > - This helps to influence the core processing logic > > for the encapsulated algorithm. > > - This flag is set by the consumer after allocating > > the tfm and before calling the function crypto_xxx_setkey(). > > > > Signed-off-by: Pankaj Gupta <pankaj.gupta@xxxxxxx> > > --- > > include/linux/crypto.h | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/include/linux/crypto.h b/include/linux/crypto.h index > > 2324ab6f1846..b4fa83ca87bd 100644 > > --- a/include/linux/crypto.h > > +++ b/include/linux/crypto.h > > @@ -639,6 +639,8 @@ struct crypto_tfm { > > > > u32 crt_flags; > > > > + unsigned int is_hbk; > > + > > We already have plenty of drivers with hardware keys in the tree. > Plesae explain why the current support is inadequate and you need to do > this. > Current support, does not permit the hardware to support both types of keys: HBK & Plain Key, at the same time. This change is generic and permit any driver that supports both- HBK and plain-key, to differentiate and use the keys accordingly. > Thanks, > -- > Email: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx> Home Page: > https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fgondo > r.apana.org.au%2F~herbert%2F&data=05%7C01%7Cpankaj.gupta%40nx > p.com%7Cffd6105be88e4b82546c08da8fd33b6e%7C686ea1d3bc2b4c6fa92cd9 > 9c5c301635%7C0%7C0%7C637980434686220485%7CUnknown%7CTWFpbGZs > b3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn > 0%3D%7C3000%7C%7C%7C&sdata=AErrjpwh%2FRZSCNDDbtH%2FWW > %2B39J%2FrxPJwak2X2Fk%2BWBg%3D&reserved=0 > PGP Key: > https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fgondo > r.apana.org.au%2F~herbert%2Fpubkey.txt&data=05%7C01%7Cpankaj.g > upta%40nxp.com%7Cffd6105be88e4b82546c08da8fd33b6e%7C686ea1d3bc2 > b4c6fa92cd99c5c301635%7C0%7C0%7C637980434686220485%7CUnknown%7 > CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwi > LCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=%2F7axxa8nrYDiItrwcZi > AfRTVc%2F25Ln5IgUzr%2F1uK%2FNk%3D&reserved=0