Re: [PATCH RFC 6/9] lib/lz4: switch the HC compressor to the vendored sources

[sashiko-bot@kernel.org]

> Replace the forked LZ4HC with thin entry points over the vendored
> lz4hc.c.
> 
> The HC entry points clamp the level below LZ4HC_CLEVEL_OPT_MIN, since
> upstream routes levels 10 and up to the optimal parser whose ~64K
> workspace does not fit a kernel stack. Levels are clamped rather than
> rejected so existing f2fs and zram settings keep working; they now
> compress as level 9.
> 
> f2fs and zram both take user levels up to LZ4HC_MAX_CLEVEL, so
> <linux/lz4.h> gains LZ4HC_CLAMP_CLEVEL and says what happens at or above
> it. A later patch ties that constant to upstream's LZ4HC_CLEVEL_OPT_MIN
> with a static_assert, once <linux/lz4.h> is in scope of these files.
> 
> Levels 1 and 2 change in the other direction. Upstream routes them
> [ ... ]
> This retires one local patch:
> 
> commit b08918fb3f27 ("lz4: do not export static symbol")
> 
> Signed-off-by: Michal Wilczynski <m.wilczynski@samsung.com>

Sashiko has reviewed this patch and found no issues. It looks great!

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260925-lz4-vendor-upstream-v1-0-1c7ffbe21c4b@samsung.com?part=6