I would like to confirm the intended behavior here:
1. Should last_comp_len be updated to comp_len after every compression
iteration to enable this comparison?
2. Is this logic necessary for scenarios where compression lengths
differ between iterations, or can this block be removed altogether?
To address this, I propose a fix where last_comp_len is updated after the
conditional block as shown below, ensuring it reflects the last compression
length:
if (last_comp_len && (last_comp_len != comp_len)) {
zs_free(zram->mem_pool, handle);
andle = -ENOMEM;
}
last_comp_len = comp_len;
Could you please clarify whether this approach aligns with the intended
functionality of the code?
Thank you for your time and assistance.
-Dheeraj
