* [PATCH v1 0/1] s390/crypto: Generate iintermediate chaining value on export @ 2026-08-14 14:20 Holger Dengler 2026-08-14 14:20 ` [PATCH v1 1/1] s390/crypto: Generate intermediate CV for API partial block handling Holger Dengler 2026-09-17 9:13 ` [PATCH v1 0/1] s390/crypto: Generate iintermediate chaining value on export Holger Dengler 0 siblings, 2 replies; 6+ messages in thread From: Holger Dengler @ 2026-08-14 14:20 UTC (permalink / raw) To: Harald Freudenberger, Herbert Xu Cc: dengler, linux-s390, Heiko Carstens, Vasily Gorbik, Alexander Gordeev, Eric Biggers, linux-crypto, ifranzki While exporting the state of the hmac chipher, the intermediate chaining value (CV) has to be calculated. Calling the KIMD instruction on s390 will generate the correct value. The code already sets the correct function code for the instruction, but it calls the wrong instruction (KLMD). The patch just fixes this. This series applies on top of Linus' master branch. Holger Dengler (1): s390/crypto: Generate intermediate CV for API partial block handling arch/s390/crypto/hmac_s390.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) base-commit: 2f1baf1fc8929e6c48370be543ad028ac7ad4131 -- 2.55.0 ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH v1 1/1] s390/crypto: Generate intermediate CV for API partial block handling 2026-08-14 14:20 [PATCH v1 0/1] s390/crypto: Generate iintermediate chaining value on export Holger Dengler @ 2026-08-14 14:20 ` Holger Dengler 2026-08-14 14:29 ` Harald Freudenberger 2026-09-17 9:13 ` [PATCH v1 0/1] s390/crypto: Generate iintermediate chaining value on export Holger Dengler 1 sibling, 1 reply; 6+ messages in thread From: Holger Dengler @ 2026-08-14 14:20 UTC (permalink / raw) To: Harald Freudenberger, Herbert Xu Cc: dengler, linux-s390, Heiko Carstens, Vasily Gorbik, Alexander Gordeev, Eric Biggers, linux-crypto, ifranzki The API partial block handling requires a intermediate chaining value (CV). The internal function hash_data() sets the function code correctly, so also call cpacf_kimd() instruction for intermediate CV generation, as cpacf_klmd() always generate the final hash value. Cc: stable@vger.kernel.org # 6.15+ Fixes: 08811169ac01 ("crypto: s390/hmac - Use API partial block handling") Signed-off-by: Holger Dengler <dengler@linux.ibm.com> --- arch/s390/crypto/hmac_s390.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/arch/s390/crypto/hmac_s390.c b/arch/s390/crypto/hmac_s390.c index f8cd09f341d4..445fa7bbd958 100644 --- a/arch/s390/crypto/hmac_s390.c +++ b/arch/s390/crypto/hmac_s390.c @@ -150,7 +150,10 @@ static int hash_data(const u8 *in, unsigned int inlen, #undef PARAM_INIT - cpacf_klmd(func, ¶m, in, inlen); + if (final) + cpacf_klmd(func, ¶m, in, inlen); + else + cpacf_kimd(func, ¶m, in, inlen); memcpy(digest, ¶m, digestsize); -- 2.55.0 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v1 1/1] s390/crypto: Generate intermediate CV for API partial block handling 2026-08-14 14:20 ` [PATCH v1 1/1] s390/crypto: Generate intermediate CV for API partial block handling Holger Dengler @ 2026-08-14 14:29 ` Harald Freudenberger 0 siblings, 0 replies; 6+ messages in thread From: Harald Freudenberger @ 2026-08-14 14:29 UTC (permalink / raw) To: Holger Dengler Cc: Herbert Xu, linux-s390, Heiko Carstens, Vasily Gorbik, Alexander Gordeev, Eric Biggers, linux-crypto, ifranzki On 2026-08-14 16:20, Holger Dengler wrote: > The API partial block handling requires a intermediate chaining > value (CV). The internal function hash_data() sets the function code > correctly, so also call cpacf_kimd() instruction for intermediate CV > generation, as cpacf_klmd() always generate the final hash value. > > Cc: stable@vger.kernel.org # 6.15+ > Fixes: 08811169ac01 ("crypto: s390/hmac - Use API partial block > handling") > Signed-off-by: Holger Dengler <dengler@linux.ibm.com> > --- > arch/s390/crypto/hmac_s390.c | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/arch/s390/crypto/hmac_s390.c > b/arch/s390/crypto/hmac_s390.c > index f8cd09f341d4..445fa7bbd958 100644 > --- a/arch/s390/crypto/hmac_s390.c > +++ b/arch/s390/crypto/hmac_s390.c > @@ -150,7 +150,10 @@ static int hash_data(const u8 *in, unsigned int > inlen, > > #undef PARAM_INIT > > - cpacf_klmd(func, ¶m, in, inlen); > + if (final) > + cpacf_klmd(func, ¶m, in, inlen); > + else > + cpacf_kimd(func, ¶m, in, inlen); > > memcpy(digest, ¶m, digestsize); Reviewed-by: Harald Freudenberger <freude@linux.ibm.com> ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v1 0/1] s390/crypto: Generate iintermediate chaining value on export 2026-08-14 14:20 [PATCH v1 0/1] s390/crypto: Generate iintermediate chaining value on export Holger Dengler 2026-08-14 14:20 ` [PATCH v1 1/1] s390/crypto: Generate intermediate CV for API partial block handling Holger Dengler @ 2026-09-17 9:13 ` Holger Dengler 2026-09-18 15:26 ` Heiko Carstens 1 sibling, 1 reply; 6+ messages in thread From: Holger Dengler @ 2026-09-17 9:13 UTC (permalink / raw) To: Herbert Xu, Alexander Gordeev, Vasily Gorbik, Heiko Carstens Cc: Harald Freudenberger, linux-s390, Eric Biggers, linux-crypto, ifranzki Can someone please pick this series? Either via crypto or s390 maintainers. Thx. -- Mit freundlichen Grüßen / Kind regards Holger Dengler ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v1 0/1] s390/crypto: Generate iintermediate chaining value on export 2026-09-17 9:13 ` [PATCH v1 0/1] s390/crypto: Generate iintermediate chaining value on export Holger Dengler @ 2026-09-18 15:26 ` Heiko Carstens 2026-09-19 1:53 ` Herbert Xu 0 siblings, 1 reply; 6+ messages in thread From: Heiko Carstens @ 2026-09-18 15:26 UTC (permalink / raw) To: Holger Dengler Cc: Herbert Xu, Alexander Gordeev, Vasily Gorbik, Harald Freudenberger, linux-s390, Eric Biggers, linux-crypto, ifranzki On Thu, Sep 17, 2026 at 11:13:05AM +0200, Holger Dengler wrote: > Can someone please pick this series? Either via crypto or s390 > maintainers. Thx. This should go via the crypto tree. Herbert? ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v1 0/1] s390/crypto: Generate iintermediate chaining value on export 2026-09-18 15:26 ` Heiko Carstens @ 2026-09-19 1:53 ` Herbert Xu 0 siblings, 0 replies; 6+ messages in thread From: Herbert Xu @ 2026-09-19 1:53 UTC (permalink / raw) To: Heiko Carstens Cc: Holger Dengler, Alexander Gordeev, Vasily Gorbik, Harald Freudenberger, linux-s390, Eric Biggers, linux-crypto, ifranzki On Fri, Sep 18, 2026 at 05:26:41PM +0200, Heiko Carstens wrote: > On Thu, Sep 17, 2026 at 11:13:05AM +0200, Holger Dengler wrote: > > Can someone please pick this series? Either via crypto or s390 > > maintainers. Thx. > > This should go via the crypto tree. Herbert? Patch applied. Thanks. -- Email: Herbert Xu <herbert@gondor.apana.org.au> Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2026-09-19 1:53 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2026-08-14 14:20 [PATCH v1 0/1] s390/crypto: Generate iintermediate chaining value on export Holger Dengler 2026-08-14 14:20 ` [PATCH v1 1/1] s390/crypto: Generate intermediate CV for API partial block handling Holger Dengler 2026-08-14 14:29 ` Harald Freudenberger 2026-09-17 9:13 ` [PATCH v1 0/1] s390/crypto: Generate iintermediate chaining value on export Holger Dengler 2026-09-18 15:26 ` Heiko Carstens 2026-09-19 1:53 ` Herbert Xu
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox