Hi Tom

> On Fri, Jan 06, 2023 at 11:26:36PM +0000, Maier, Brandon L:
> > > From: Tom Rini <tr...@konsulko.com>
> > > It's all not ideal, yes, but it looks like BTRFS is the main user, right 
> > > now, which
> > > isn't widely enabled. So maybe we can look towards improving upstream a
> > > bit here, if motivated.
> >
> > I can't speak for the BTRFS code as I don't have a suitable platform
> > to test it on. But the zstd code does appear to be broken on master.
> > Running the zstd compression test in sandbox or sandbox64 both
> > segfault. And I have similar problems trying to decompress FIT images
> > on our AARCH64 platform. The primary motivation to this patch is that
> > the new version does work. And this seems like the appropriate fix
> > given Linux has abandoned the original version of this code.
> 
> Well, I guess in the end, non-functional smaller code is factually worse
> than larger functional code, so, I'll pick this up post v2023.01,
> thanks!

I did some deeper investigation and discovered a bunch of functions were being 
inlined that caused about 3k of growth. As well as a BMI2 feature that can be 
disabled to save about 800B on x86 platforms. With those it brings the code 
size change to about -3k for arm64, and +1k for arm. I'll send a v4 patch 
series shortly.

> 
> --
> Tom

Reply via email to