Hi Herbert, Thanks for your response >Hi Jan: On Mon, Dec 12, 2016 at 04:04:36PM +0100, Jan Glauber wrote: > > > >this series adds support for hardware accelerated compression & decompression > >as found on ThunderX (arm64) SOCs. I've been reviewing this driver internally > >for some time and would like to get feedback on the RFC to see if this goes > >into the right direction and to see if there are any concerns. > > > >We've discussed switching to the new acomp algorithm but for the time being > >decided against acomp because our test cases are not yet supported with it. > OK. Do you see any major problems in converting this over to acomp? Mahipal: One major issue is, the kernel use cases to validate the Cavium ThunderX ZIP driver are "ZSWAP" and "IPComp" which are not yet supported with scomp-acomp framework. Regards, -Mahipal From: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx> Sent: Tuesday, December 27, 2016 2:32 PM To: Jan Glauber Cc: linux-crypto@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; David S . Miller; Challa, Mahipal; Nair, Vishnu Subject: Re: [RFC PATCH 0/3] Cavium ThunderX ZIP driver Hi Jan: On Mon, Dec 12, 2016 at 04:04:36PM +0100, Jan Glauber wrote: > > this series adds support for hardware accelerated compression & decompression > as found on ThunderX (arm64) SOCs. I've been reviewing this driver internally > for some time and would like to get feedback on the RFC to see if this goes > into the right direction and to see if there are any concerns. > > We've discussed switching to the new acomp algorithm but for the time being > decided against acomp because our test cases are not yet supported with it. OK. Do you see any major problems in converting this over to acomp? Thanks, -- Email: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx> Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt -- To unsubscribe from this list: send the line "unsubscribe linux-crypto" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html