Re: [PATCH 5/9] btrfs: zstd: Switch to the zstd-1.4.6 API

Eric Biggers
2020-09-16T18:42:48Z

On Wed, Sep 16, 2020 at 03:46:18PM +0100, Christoph Hellwig wrote:
> On Wed, Sep 16, 2020 at 10:43:04AM -0400, Chris Mason wrote:
> > Otherwise we just end up with drift and kernel-specific bugs that are harder
> > to debug. To the extent those APIs make us contort the kernel code, I???m
> > sure Nick is interested in improving things in both places.
>
> Seriously, we do not care elsewhere. Why would zlib be any different?
>
> > There are probably 1000 constructive ways to have that conversation. Please
> > choose one of those instead of being an asshole.
>
> I think you are the asshole here by ignoring the practices we are using
> elsewhere and think your employers pet project is somehow special. It
> is not, and claiming so is everything but constructive.
>

The userspace Zstandard library is widely used and has been heavily reviewed,
tested, and fuzzed.

The options are either (a) write and maintain a separate kernel implementation
of Zstandard, or (b) periodically sync from upstream and make minimal, easily
reviewable changes to integrate with the kernel.

I don't see option (a) working for Zstandard. For short and simple code, it's
the usual Linux kernel practice and it works fine. But it's far too hard to
write and maintain a good implementation of Zstandard -- meaning correct, fast,
fully fuzzed, and supporting all needed compression levels. Optimizing
compressors and decompressors is really hard. A "naive" implementation wouldn't
be too hard, but it would be slow and wouldn't support high compression ratios.

Similarly, some of the crypto assembly code in the kernel is taken from the
OpenSSL project, since the kernel community doesn't have the capacity to
properly optimize algorithms like Poly1305 for x86, arm, arm64, mips, ...

If your main concern is about the camel case function naming, that doesn't seem
very important, relatively speaking.

- Eric