Herbert, On Thu, Feb 28, 2019 at 06:37:15PM +0800, Herbert Xu wrote: > On Thu, Feb 28, 2019 at 01:33:37PM +0300, Vitaly Chikunov wrote: > > > > To make the same for set_{pub,priv}_key it will require patching RSA > > drivers anyway, since length of the key is stored just once as keylen > > argument. > > No we don't need to use the same format for different algorithms. > RSA should stay as is. I will rework as you suggest. But, just want to state that I disagree with this approach of implicitly appending parameters data to the key stream without any argument signifying it or length covering it. This fitting into the old API is also somewhat disagree to your words that we could change internal API: On Thu, Feb 28, 2019 at 02:14:44PM +0800, Herbert Xu wrote: > On Sun, Feb 24, 2019 at 09:48:40AM +0300, Vitaly Chikunov wrote: > ... > This compatibility does not matter. We can always add translating > layers into the crypto API to deal with this. The only ABI that > matters is the one to user-space.