]> CyberLeo.Net >> Repos - FreeBSD/FreeBSD.git/commit
geom_uzip(4), mkuzip(8): Add Zstd image mode
authorcem <cem@FreeBSD.org>
Tue, 13 Aug 2019 23:32:56 +0000 (23:32 +0000)
committercem <cem@FreeBSD.org>
Tue, 13 Aug 2019 23:32:56 +0000 (23:32 +0000)
commit604e65334e196a0977c08443be2c2eb85747162f
tree43be3a9c423158f83bcb63f72fc0e628f2bb25cf
parent07370ae4eed28a06cf0da45d2d441bc92b38c699
geom_uzip(4), mkuzip(8): Add Zstd image mode

The Zstd format bumps the CLOOP major number to 4 to avoid incompatibility
with older systems.  Support in geom_uzip(4) is conditional on the ZSTDIO
kernel option, which is enabled in amd64 GENERIC, but not all in-tree
configurations.

mkuzip(8) was modified slightly to always initialize the nblocks + 1'th
offset in the CLOOP file format.  Previously, it was only initialized in the
case where the final compressed block happened to be unaligned w.r.t.
DEV_BSIZE.  The "Fake" last+1 block change in r298619 means that the final
compressed block's 'blen' was never correct unless the compressed uzip image
happened to be BSIZE-aligned.  This happened in about 1 out of every 512
cases.  The zlib and lzma decompressors are probably tolerant of extra trash
following the frame they were told to decode, but Zstd complains that the
input size is incorrect.

Correspondingly, geom_uzip(4) was modified slightly to avoid trashing the
nblocks + 1'th offset when it is known to be initialized to a good value.
This corrects the calculated final real cluster compressed length to match
that printed by mkuzip(8).

mkuzip(8) was refactored somewhat to reduce code duplication and increase
ease of adding other compression formats.

  * Input block size validation was pulled out of individual compression
    init routines into main().

  * Init routines now validate a user-provided compression level or select
    an algorithm-specific default, if none was provided.

  * A new interface for calculating the maximal compressed size of an
    incompressible input block was added for each driver.  The generic code
    uses it to validate against MAXPHYS as well as to allocate compression
    result buffers in the generic code.

  * Algorithm selection is now driven by a table lookup, to increase ease of
    adding other formats in the future.

mkuzip(8) gained the ability to explicitly specify a compression level with
'-C'.  The prior defaults -- 9 for zlib and 6 for lzma -- are maintained.
The new zstd default is 9, to match zlib.

Rather than select lzma or zlib with '-L' or its absense, respectively, a
new argument '-A <algorithm>' is provided to select 'zlib', 'lzma', or
'zstd'.  '-L' is considered deprecated, but will probably never be removed.

All of the new features were documented in mkuzip.8; the page was also
cleaned up slightly.

Relnotes: yes
22 files changed:
share/man/man4/geom_uzip.4
sys/conf/NOTES
sys/conf/files
sys/geom/uzip/g_uzip.c
sys/geom/uzip/g_uzip_cloop.h
sys/geom/uzip/g_uzip_zstd.c [new file with mode: 0644]
sys/geom/uzip/g_uzip_zstd.h [new file with mode: 0644]
sys/modules/geom/geom_uzip/Makefile
usr.bin/mkuzip/Makefile
usr.bin/mkuzip/mkuz_cfg.h
usr.bin/mkuzip/mkuz_cloop.h
usr.bin/mkuzip/mkuz_conveyor.c
usr.bin/mkuzip/mkuz_format.h
usr.bin/mkuzip/mkuz_lzma.c
usr.bin/mkuzip/mkuz_lzma.h
usr.bin/mkuzip/mkuz_zlib.c
usr.bin/mkuzip/mkuz_zlib.h
usr.bin/mkuzip/mkuz_zstd.c [new file with mode: 0644]
usr.bin/mkuzip/mkuz_zstd.h [new file with mode: 0644]
usr.bin/mkuzip/mkuzip.8
usr.bin/mkuzip/mkuzip.c
usr.bin/mkuzip/mkuzip.h