]> CyberLeo.Net >> Repos - FreeBSD/FreeBSD.git/commit
l2arc: restore correct rounding up of asize of compressed data
authorAndriy Gapon <avg@FreeBSD.org>
Mon, 17 Nov 2014 14:45:42 +0000 (14:45 +0000)
committerAndriy Gapon <avg@FreeBSD.org>
Mon, 17 Nov 2014 14:45:42 +0000 (14:45 +0000)
commit2c51c83bc8f5723eef5b1d5dfa5dcb9e93796870
tree3c4c73e7d0fb062ff6a53ea871bfed873551aa37
parent0908b20b7ee1d0e8f1628e137fe13ee3e85c2826
l2arc: restore correct rounding up of asize of compressed data

This rounding up was lost in a mismerge of illumos code.
See r268075 MFV r267565.
After that commit zio_compress_data() no longer performs any compressed
size adjustment, so it needs to be done externally.  On FreeBSD we round
up the size using vdev_ashift rather than SPA_MINBLOCKSIZE so that 4KB
devices are properly supported.

Additionally, zero out the buffer tail only if compression succeeds.
The compression is considered successful if the size of compressed
data after rounding up to account for the vdev ashift is less than the
original data size.  It does not make sense to have the data compressed
if all the savings are lost to rounding up.
With the new zio_compress_data() it could have been possible that the
rounded compressed size would be greater than the original size and thus
we could zero beyond the allocated buffer if the zeroing code was kept
at the original place.

Discussed with: delphij, gibbs
MFC after: 2 weeks
X-MFC with: r274627
sys/cddl/contrib/opensolaris/uts/common/fs/zfs/arc.c