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

Chris Mason
2020-09-16T20:45:55Z

On 16 Sep 2020, at 10:46, 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?

Is the zlib upstream active? Or trying to sync active development with
the kernel? I’d suggest the same path for them if they were.

>
>> 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.

I’m happy to advocate for more constructive discussion for anyone’s
project. I tend to pick threads where I have context and I know the
people involved.

The kernel best practices are pragmatic. As one of many users of any
established-non-kernel project, there’s a compromise between the APIs
they are using for a broad base of users and us. I’m sure they are
interested in improving life for all of their users, while also
improving maintainability for us.

-chris