Re: [PATCH v1 0/8] Batched zswap_store() with compression batching
From: Herbert Xu
Date: Sat Oct 17 2026 - 00:12:48 EST
On Fri, Oct 16, 2026 at 08:03:07PM -0700, Kanchana P. Sridhar wrote:
>
> The Average throughput of 10 iterations usemem is 19,058 KB/s. IOW, I
> don't get the same performance boost as making the localized change in
> crypto_acomp_compress(), likely because of added computes per compress
> call (one function call, two conditionals).
What are we comparing against?
The thing is that the conditional will have to be there at some
point as otherwise this becomes pointless.
What if you just do the conditional and short-circuit in
crypto_acomp_compress?
Thanks,
--
Email: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt