Re: RFR: 8331008: KDF Implementation (Preview) [v26]

2024-05-10 Thread Alan Bateman
On Sat, 11 May 2024 02:06:09 GMT, Kevin Driver wrote: >> Introduce an API for Key Derivation Functions (KDFs), which are >> cryptographic algorithms for deriving additional keys from a secret key and >> other data. See [JEP 478](https://openjdk.org/jeps/478). > > Kevin Driver has updated the pu

Re: RFR: 8325513: Export method for Cipher [v3]

2024-05-10 Thread Anthony Scarpino
On Fri, 10 May 2024 22:40:30 GMT, Weijun Wang wrote: > One use case for this method is HPKE key export. Obviously, the exported key > won't have algorithm name being "HPKE". If `Cipher::init` is passed an AES SecretKey, wouldn't the `Cipher::export` return an AES SecretKey? I don't see the ca

Re: RFR: 8331008: KDF Implementation (Preview) [v26]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation (Preview) [v25]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation (Preview) [v24]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation (Preview) [v23]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8325513: Export method for Cipher [v3]

2024-05-10 Thread Weijun Wang
On Fri, 10 May 2024 14:00:55 GMT, Weijun Wang wrote: >> Add `Cipher::export` API. > > Weijun Wang has updated the pull request incrementally with one additional > commit since the last revision: > > change new method to non final One use case for this method is HPKE key export. Obviously, th

Re: RFR: 8331008: KDF Implementation (Preview) [v22]

2024-05-10 Thread Kevin Driver
On Fri, 10 May 2024 22:11:04 GMT, Kevin Driver wrote: >> Introduce an API for Key Derivation Functions (KDFs), which are >> cryptographic algorithms for deriving additional keys from a secret key and >> other data. See [JEP 478](https://openjdk.org/jeps/478). > > Kevin Driver has updated the pu

Re: RFR: 8331008: KDF Implementation (Preview) [v22]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with three additional commits since

Re: RFR: 8331008: KDF Implementation (Preview) [v21]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation (Preview) [v18]

2024-05-10 Thread Kevin Driver
On Fri, 10 May 2024 14:54:18 GMT, Sean Mullan wrote: >> src/java.base/share/classes/javax/crypto/spec/HKDFParameterSpec.java line 70: >> >>> 68: */ >>> 69: public Extract extractOnly() { >>> 70: if (this.ikms.isEmpty() && this.salts.isEmpty()) { >> >> I don't think

Re: RFR: 8331008: KDF Implementation (Preview) [v20]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8293345: SunPKCS11 provider checks on PKCS11 Mechanism are problematic [v2]

2024-05-10 Thread dems54
On Tue, 16 Apr 2024 22:30:57 GMT, Valerie Peng wrote: >> What about testing? > > @mcpowers I am about to leave for vacation. Will wait for your review and > resume on this PR after I return. Thanks! Many thanks @valeriepeng for this feature. Possible to backport on JDK 17 or 21 ? -

Re: RFR: 8331008: KDF Implementation (Preview) [v2]

2024-05-10 Thread Kevin Driver
On Thu, 9 May 2024 17:29:15 GMT, Sean Mullan wrote: >> src/java.base/share/classes/javax/crypto/KDFSpi.java line 107: >> >>> 105: * if the key derivation implementation cannot support >>> additional calls to >>> 106: * {@code deriveData } or if all {@code >>> DerivationParame

Re: RFR: 8331008: KDF Implementation (Preview) [v2]

2024-05-10 Thread Kevin Driver
On Thu, 9 May 2024 16:42:09 GMT, Weijun Wang wrote: >> Kevin Driver has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains ten additional >> commits si

Re: RFR: 8331008: KDF Implementation (Preview) [v19]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation (Preview) [v19]

2024-05-10 Thread Kevin Driver
On Thu, 9 May 2024 15:14:48 GMT, Weijun Wang wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> replace in-place SecretKeySpec alg for IKM/salt with "Generic" > > src/java.base/share/classes/javax/crypto/spec/HKDFPara

Re: RFR: 8331008: KDF Implementation (Preview) [v18]

2024-05-10 Thread Kevin Driver
On Thu, 9 May 2024 15:01:24 GMT, Weijun Wang wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> a start on the preview wire on/off > > src/java.base/share/classes/javax/crypto/spec/HKDFParameterSpec.java line 47: > >

Re: RFR: 8331008: KDF Implementation (Preview)

2024-05-10 Thread Kevin Driver
On Thu, 9 May 2024 14:43:55 GMT, Weijun Wang wrote: > Two comments on HKDF: > > 1. Expand length cannot exceed 255 * HashLen. See > https://datatracker.ietf.org/doc/html/rfc5869#section-2.3. > 2. Why disallow empty ikms and salts? For the salt side, RFC5869 allows [no > salt](https://datatrack

Re: RFR: 8331008: KDF Implementation (Preview) [v18]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation (Preview) [v17]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8325513: Export method for Cipher [v3]

2024-05-10 Thread Anthony Scarpino
On Fri, 10 May 2024 14:00:55 GMT, Weijun Wang wrote: >> Add `Cipher::export` API. > > Weijun Wang has updated the pull request incrementally with one additional > commit since the last revision: > > change new method to non final Why is the `algorithm` necessary for this new method? Couldn

Re: RFR: 8331008: KDF Implementation (Preview) [v16]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation (Preview) [v3]

2024-05-10 Thread Kevin Driver
On Thu, 9 May 2024 21:21:55 GMT, Weijun Wang wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> some code review comments > > src/java.base/share/classes/javax/crypto/KDF.java line 1: > >> 1: /* > > Delayed provider

Re: RFR: 8331008: KDF Implementation (Preview) [v15]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation (Preview) [v3]

2024-05-10 Thread Kevin Driver
On Thu, 9 May 2024 20:33:57 GMT, Weijun Wang wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> some code review comments > > src/java.base/share/classes/com/sun/crypto/provider/HkdfKeyDerivation.java > line 365: >

Re: RFR: 8331008: KDF Implementation (Preview) [v14]

2024-05-10 Thread Kevin Driver
On Wed, 8 May 2024 20:53:54 GMT, Kevin Driver wrote: >> src/java.base/share/classes/com/sun/crypto/provider/HkdfKeyDerivation.java >> line 370: >> >>> 368: } >>> 369: int rounds = (outLen + hmacLen - 1) / hmacLen; >>> 370: kdfOutput = new byte[rounds * hmacLen]; >> >> A

Re: RFR: 8331008: KDF Implementation (Preview) [v14]

2024-05-10 Thread Kevin Driver
On Thu, 9 May 2024 12:31:24 GMT, Sean Mullan wrote: >> src/java.base/share/classes/javax/crypto/KDF.java line 54: >> >>> 52: * the algorithm specifier may be omitted if the KDF algorithm has a >>> fixed or default PRF. >>> 53: * >>> 54: * TODO: finish this javadoc >> >> You should state th

Re: RFR: 8331008: KDF Implementation (Preview) [v14]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation (Preview) [v13]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation (Preview) [v12]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation (Preview) [v11]

2024-05-10 Thread Kevin Driver
On Thu, 9 May 2024 15:30:22 GMT, Weijun Wang wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> continued code review comments > > src/java.base/share/classes/javax/crypto/KDF.java line 58: > >> 56: >> 57: public fi

Re: RFR: 8331008: KDF Implementation (Preview) [v11]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation (Preview) [v10]

2024-05-10 Thread Kevin Driver
On Wed, 8 May 2024 20:25:36 GMT, Sean Mullan wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> code review comment > > src/java.base/share/classes/java/security/Provider.java line 1607: > >> 1605: addEngine(

Re: RFR: 8331008: KDF Implementation [v10]

2024-05-10 Thread Kevin Driver
On Thu, 9 May 2024 14:10:37 GMT, Weijun Wang wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> code review comment > > src/java.base/share/classes/javax/crypto/spec/HKDFParameterSpec.java line 36: > >> 34: * Parame

Re: RFR: 8331008: KDF Implementation [v10]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Integrated: 8293345: SunPKCS11 provider checks on PKCS11 Mechanism are problematic

2024-05-10 Thread Valerie Peng
On Wed, 20 Mar 2024 02:44:19 GMT, Valerie Peng wrote: > Existing legacy mechanism check disables mechanism(s) when the support is > partial, e.g. supports decryption but not encryption, or supports > verification but not signing. Some mechanisms can be used for both > encryption/decryption and

Re: RFR: 8331008: KDF Implementation [v7]

2024-05-10 Thread Kevin Driver
On Thu, 9 May 2024 13:14:45 GMT, Sean Mullan wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> additional delayed provider selection work > > src/java.base/share/classes/javax/crypto/KDF.java line 147: > >> 145:

Re: RFR: 8331008: KDF Implementation [v9]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation [v9]

2024-05-10 Thread Kevin Driver
On Thu, 9 May 2024 13:16:05 GMT, Sean Mullan wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> code review comment > > src/java.base/share/classes/javax/crypto/KDF.java line 150: > >> 148: KDF instance = nul

Re: RFR: 8331008: KDF Implementation [v8]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation [v8]

2024-05-10 Thread Kevin Driver
On Thu, 9 May 2024 13:07:41 GMT, Sean Mullan wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> code review comment > > src/java.base/share/classes/javax/crypto/KDF.java line 132: > >> 130: >> 131: private Strin

Re: RFR: 8331008: KDF Implementation [v7]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation [v6]

2024-05-10 Thread Sean Mullan
On Fri, 10 May 2024 14:56:55 GMT, Kevin Driver wrote: >> Introduce an API for Key Derivation Functions (KDFs), which are >> cryptographic algorithms for deriving additional keys from a secret key and >> other data. See [JEP 478](https://openjdk.org/jeps/478). > > Kevin Driver has updated the pu

Re: RFR: 8331008: KDF Implementation [v6]

2024-05-10 Thread Sean Mullan
On Thu, 9 May 2024 15:11:39 GMT, Weijun Wang wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> change algorithm standard name for HKDFs in SunJCE provider > > src/java.base/share/classes/javax/crypto/spec/HKDFParamet

Re: RFR: 8331008: KDF Implementation [v6]

2024-05-10 Thread Alan Bateman
On Fri, 10 May 2024 14:56:55 GMT, Kevin Driver wrote: >> Introduce an API for Key Derivation Functions (KDFs), which are >> cryptographic algorithms for deriving additional keys from a secret key and >> other data. See [JEP 478](https://openjdk.org/jeps/478). > > Kevin Driver has updated the pu

Re: RFR: 8331008: KDF Implementation [v4]

2024-05-10 Thread Sean Mullan
On Fri, 10 May 2024 00:15:32 GMT, Kevin Driver wrote: >> Introduce an API for Key Derivation Functions (KDFs), which are >> cryptographic algorithms for deriving additional keys from a secret key and >> other data. See [JEP 478](https://openjdk.org/jeps/478). > > Kevin Driver has updated the pu

Re: RFR: 8331008: KDF Implementation [v6]

2024-05-10 Thread Sean Mullan
On Thu, 9 May 2024 15:04:53 GMT, Weijun Wang wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> change algorithm standard name for HKDFs in SunJCE provider > > src/java.base/share/classes/javax/crypto/spec/HKDFParamet

Re: RFR: 8331008: KDF Implementation [v6]

2024-05-10 Thread Kevin Driver
On Wed, 8 May 2024 20:29:01 GMT, Sean Mullan wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> change algorithm standard name for HKDFs in SunJCE provider > > src/java.base/share/classes/com/sun/crypto/provider/SunJC

Re: RFR: 8331008: KDF Implementation [v6]

2024-05-10 Thread Kevin Driver
On Fri, 10 May 2024 14:50:51 GMT, Kevin Driver wrote: >> src/java.base/share/classes/com/sun/crypto/provider/SunJCE.java line 463: >> >>> 461: * Key Derivation engines >>> 462: */ >>> 463: ps("KDF", "HKDF/HmacSHA256", >> >> In the JEP, the example uses "HkdfSHA256". Th

Re: RFR: 8331008: KDF Implementation [v6]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8331008: KDF Implementation [v3]

2024-05-10 Thread Kevin Driver
On Thu, 9 May 2024 20:20:00 GMT, Weijun Wang wrote: >> src/java.base/share/classes/com/sun/crypto/provider/HkdfKeyDerivation.java >> line 53: >> >>> 51: * and Expand-only variants. >>> 52: */ >>> 53: abstract class HkdfKeyDerivation extends KDFSpi { >> >> How about just name it `HKDF`? > > M

Re: RFR: 8331008: KDF Implementation [v4]

2024-05-10 Thread Kevin Driver
On Fri, 10 May 2024 12:12:06 GMT, Sean Mullan wrote: >> src/java.base/share/classes/com/sun/crypto/provider/SunJCE.java line 467: >> >>> 465: ps("KDF", "HkdfSHA384", >>> 466: >>> "com.sun.crypto.provider.HkdfKeyDerivation$HkdfSHA384"); >>> 467: ps("KDF", "HkdfSHA

Re: RFR: 8331008: KDF Implementation [v5]

2024-05-10 Thread Kevin Driver
> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic > algorithms for deriving additional keys from a secret key and other data. See > [JEP 478](https://openjdk.org/jeps/478). Kevin Driver has updated the pull request incrementally with one additional commit since the

Re: RFR: 8325513: Export method for Cipher [v3]

2024-05-10 Thread Weijun Wang
> Add `Cipher::export` API. Weijun Wang has updated the pull request incrementally with one additional commit since the last revision: change new method to non final - Changes: - all: https://git.openjdk.org/jdk/pull/18409/files - new: https://git.openjdk.org/jdk/pull/18409/f

Re: RFR: 8325513: Export method for Cipher [v2]

2024-05-10 Thread Weijun Wang
On Fri, 10 May 2024 13:08:00 GMT, Alan Bateman wrote: >> Weijun Wang has updated the pull request incrementally with one additional >> commit since the last revision: >> >> rename > > src/java.base/share/classes/javax/crypto/Cipher.java line 2625: > >> 2623: * @since 23 >> 2624: */

Re: RFR: 8325513: Export method for Cipher [v2]

2024-05-10 Thread Alan Bateman
On Fri, 10 May 2024 13:07:30 GMT, Weijun Wang wrote: >> Add `Cipher::export` API. > > Weijun Wang has updated the pull request incrementally with one additional > commit since the last revision: > > rename src/java.base/share/classes/javax/crypto/Cipher.java line 2625: > 2623: * @since

Re: RFR: 8325513: Export method for Cipher [v2]

2024-05-10 Thread Weijun Wang
> Add `Cipher::export` API. Weijun Wang has updated the pull request incrementally with one additional commit since the last revision: rename - Changes: - all: https://git.openjdk.org/jdk/pull/18409/files - new: https://git.openjdk.org/jdk/pull/18409/files/8834f04e..b8658a76

Re: RFR: 8325513: Export method for Cipher

2024-05-10 Thread Sean Mullan
On Wed, 20 Mar 2024 20:37:46 GMT, Weijun Wang wrote: > Add `Cipher::export` API. src/java.base/share/classes/javax/crypto/Cipher.java line 2625: > 2623: * @since 23 > 2624: */ > 2625: public final SecretKey export(byte[] context, String algorithm, int > length) { Change name to

Re: RFR: 8325513: Export method for Cipher

2024-05-10 Thread Weijun Wang
On Fri, 10 May 2024 12:58:06 GMT, Sean Mullan wrote: >> Add `Cipher::export` API. > > src/java.base/share/classes/javax/crypto/Cipher.java line 2625: > >> 2623: * @since 23 >> 2624: */ >> 2625: public final SecretKey export(byte[] context, String algorithm, >> int length) { > > C

RFR: 8325513: Export method for Cipher

2024-05-10 Thread Weijun Wang
Add `Cipher::export` API. - Commit messages: - Merge branch 'master' into 8325513 - make test work - Add test - Wording - Wording - relax requirement - wording - the fix Changes: https://git.openjdk.org/jdk/pull/18409/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=1

Re: RFR: 8331008: KDF Implementation [v4]

2024-05-10 Thread Sean Mullan
On Thu, 9 May 2024 20:18:10 GMT, Kevin Driver wrote: >> src/java.base/share/classes/javax/crypto/KDF.java line 115: >> >>> 113: * {@code getInstance} calls that created this {@code >>> KeyDerivation} object. >>> 114: * >>> 115: * @return the algorithm name of this {@code KeyDeriv

Re: RFR: 8331008: KDF Implementation [v4]

2024-05-10 Thread Sean Mullan
On Fri, 10 May 2024 02:27:31 GMT, Weijun Wang wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> adding in delayed provider selection > > src/java.base/share/classes/com/sun/crypto/provider/SunJCE.java line 467: > >>

Re: RFR: 8261433: Better pkcs11 performance for libpkcs11:C_EncryptInit/libpkcs11:C_DecryptInit [v4]

2024-05-10 Thread Daniel Jeliński
On Fri, 10 May 2024 08:04:26 GMT, Prajwal Kumaraswamy wrote: >> This fix intends to eliminate additional library call to C_EncryptInit or >> C_DecryptInit for Ciphers running through the CKM_AES_GCM. >> >> Background: >> >> There are two types of CK_GCM_PARAMS struct that are used, one with

Re: RFR: 8261433: Better pkcs11 performance for libpkcs11:C_EncryptInit/libpkcs11:C_DecryptInit [v2]

2024-05-10 Thread Prajwal Kumaraswamy
On Thu, 9 May 2024 20:59:25 GMT, Valerie Peng wrote: > Please make sure the testing covers old and new NSS versions just to be safe. > Thanks! Done, I have attached test results of old and new NSS, However I have also tested few other NSS between 3.46 and the latest one [nsstest.zip](https://g

Re: RFR: 8261433: Better pkcs11 performance for libpkcs11:C_EncryptInit/libpkcs11:C_DecryptInit [v2]

2024-05-10 Thread Prajwal Kumaraswamy
On Thu, 9 May 2024 19:40:31 GMT, Valerie Peng wrote: >> Prajwal Kumaraswamy has updated the pull request with a new target base due >> to a merge or a rebase. The incremental webrev excludes the unrelated >> changes brought in by the merge/rebase. The pull request contains three >> additional

Re: RFR: 8261433: Better pkcs11 performance for libpkcs11:C_EncryptInit/libpkcs11:C_DecryptInit [v4]

2024-05-10 Thread Prajwal Kumaraswamy
On Mon, 25 Mar 2024 10:02:18 GMT, Daniel Jeliński wrote: >> Prajwal Kumaraswamy has updated the pull request incrementally with one >> additional commit since the last revision: >> >> use getversion instead to get spec version > > src/jdk.crypto.cryptoki/share/classes/sun/security/pkcs11/P11A

Re: RFR: 8261433: Better pkcs11 performance for libpkcs11:C_EncryptInit/libpkcs11:C_DecryptInit [v4]

2024-05-10 Thread Prajwal Kumaraswamy
> This fix intends to eliminate additional library call to C_EncryptInit or > C_DecryptInit for Ciphers running through the CKM_AES_GCM. > > Background: > > There are two types of CK_GCM_PARAMS struct that are used, one with IV bits > and the other without it. > > Initially there was issue in

Re: RFR: 8261433: Better pkcs11 performance for libpkcs11:C_EncryptInit/libpkcs11:C_DecryptInit [v3]

2024-05-10 Thread Prajwal Kumaraswamy
> This fix intends to eliminate additional library call to C_EncryptInit or > C_DecryptInit for Ciphers running through the CKM_AES_GCM. > > Background: > > There are two types of CK_GCM_PARAMS struct that are used, one with IV bits > and the other without it. > > Initially there was issue in