Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   "Chris Mason" <clm@fb.com>
To:     Christoph Hellwig <hch@infradead.org>
CC:     Nick Terrell <nickrterrell@gmail.com>,
        Herbert Xu <herbert@gondor.apana.org.au>,
        <linux-crypto@vger.kernel.org>, <linux-btrfs@vger.kernel.org>,
        <squashfs-devel@lists.sourceforge.net>,
        <linux-f2fs-devel@lists.sourceforge.net>,
        <linux-kernel@vger.kernel.org>, Kernel Team <Kernel-team@fb.com>,
        Nick Terrell <terrelln@fb.com>, Petr Malat <oss@malat.biz>,
        Johannes Weiner <jweiner@fb.com>,
        Niket Agarwal <niketa@fb.com>, Yann Collet <cyan@fb.com>
Subject: Re: [PATCH 5/9] btrfs: zstd: Switch to the zstd-1.4.6 API
Date:   Wed, 16 Sep 2020 10:43:04 -0400
Message-ID: <1CAB33F1-95DB-4BC5-9023-35DD2E4E0C20@fb.com>
In-Reply-To: <20200916143046.GA13543@infradead.org>
References: <20200916034307.2092020-1-nickrterrell@gmail.com>
 <20200916034307.2092020-7-nickrterrell@gmail.com>
 <20200916084958.GC31608@infradead.org>
 <CCDAB4AB-DE8D-4ADE-9221-02AE732CBAE2@fb.com>
 <20200916143046.GA13543@infradead.org>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 8bit
MIME-Version: 1.0
X-MS-Exchange-MessageSentRepresentingType: 1
Received: from [100.109.1.1] (2620:10d:c091:480::1:60bc) by MN2PR19CA0062.namprd19.prod.outlook.com (2603:10b6:208:19b::39) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3391.11 via Frontend Transport; Wed, 16 Sep 2020 14:43:06 +0000
X-Mailer: MailMate (1.13.2r5673)
X-Originating-IP: [2620:10d:c091:480::1:60bc]
X-MS-PublicTrafficType: Email
X-MS-Office365-Filtering-Correlation-Id: d8e45f87-73fb-4441-699c-08d85a4ed37c
X-MS-TrafficTypeDiagnostic: BL0PR1501MB2130:
X-MS-Exchange-Transport-Forked: True
X-Microsoft-Antispam-PRVS: <BL0PR1501MB2130335FAF87C026B464C3E9D3210@BL0PR1501MB2130.namprd15.prod.outlook.com>
X-FB-Source: Internal
X-MS-Oob-TLC-OOBClassifiers: OLM:213;
X-MS-Exchange-SenderADCheck: 1
X-Microsoft-Antispam: BCL:0;
X-Microsoft-Antispam-Message-Info: OOVi0X+Pq3jnBnkt2KSVJ+zdTmRmKJWAont4IYrkxwcOgLD2Yfd45ahGiobV2+WFp/NrtTdjEnSKtLh+aPaagBirq1fDfnhfnniOw+JLKFq4SRktUUEW3IiYcoolHxTVookSFqvtC5ekMdtYgm9nkLLdlPb4Etb8PplW/2vqNQk1WYOv1I0S2ykNdgZ1jpIYRLWaHyW1vc6OBQlrNUAuorCZCNt+q2thgzaY1KQ5IdPsO2Qd1pSMOuSjkMerPrxIyremGco2kjiSgOjM49bXLNS9hEIv1JL9bplhDp8dgUWexgLs+shoBHq8zMqK3Gg9Cs21/Od7eKy188px7sh0WRtp5hovF/FyOoacd70j+wdrdnUNzmLcyFr+k7bICnqi
X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:MN2PR15MB3582.namprd15.prod.outlook.com;PTR:;CAT:NONE;SFS:(376002)(346002)(366004)(136003)(39860400002)(396003)(2616005)(956004)(8936002)(83380400001)(2906002)(478600001)(52116002)(86362001)(66946007)(66476007)(66556008)(316002)(53546011)(6916009)(4326008)(5660300002)(186003)(16526019)(54906003)(36756003)(8676002)(33656002)(6486002)(78286007);DIR:OUT;SFP:1102;
X-MS-Exchange-AntiSpam-MessageData: GDIgRxwbLOJF5maRjeRld/E8FunLVHHIjfHJlMvmoMaWAIdp1oe5hIAZSh5kcyXod/7fFcqJ8J1sTM5RLFoojq9M+J2Hxi3QRK3UdQieAB48U4xCa9iAmhBkLq7ecQYVbU558H66MsV3QRZuyDDOZv4WLvgOvHyBp0v0jBGTduAT2mHGieaaa4WRZZC1KVIZLF+zGpSahCxR6KtOZ8nB5PQzuRpfUjvbtd/t3W1/AYFuz1yd4loLh6bc5+1VP8NhPQLfgMkyCjadmbuCTzWznQJZNRuk57pidxjH6SetKP4vUYzSnx51mSBql43PtaYZenyIZNCG5T7IHuRr9oNPEA+Tx2co+OzQJn1xqRX7WqanU/kACcF7Ab7NqXMMX0NR34cy9Guw2YbyAZ6kGuK0RT3y7kkjI/nB16EFR9qkHV5iDTdBrmShaa28d+QIQb5ilOrGCHxtBD08z2+d9c61+SLGWwzqn+f/TxWBz9cJBo2bPWcX2Eq7phmJmJIk8chu32VyLZ/+s3FnTbSzubcOvsqTnuLDk1XaW8Q/Y9wqNtp/Go1XiYum0nxmb27ghgLjHpD78rBXvmHFka2Mi1RUIFgeMJYHZ9k93USuGm0Nl56GWrIdj3nl9YHxp/uKvE3de6khRWBrPX9fQoARrrRC02x2Vdu2clYqMmKI16KpcztI1ImcTOq0JkpiM8lLkEP+
X-MS-Exchange-CrossTenant-Network-Message-Id: d8e45f87-73fb-4441-699c-08d85a4ed37c
X-MS-Exchange-CrossTenant-AuthSource: MN2PR15MB3582.namprd15.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 Sep 2020 14:43:07.9487
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 8ae927fe-1255-47a7-a2af-5f3a069daaa2
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: jz4NptFbpw0ZxNiAtwEwmGHdo9ZQXTqy5PunPTvqcWr5VAX9Xy6f7GIUDBMjni/N
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL0PR1501MB2130
X-OriginatorOrg: fb.com
X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:6.0.235,18.0.687
 definitions=2020-09-16_10:2020-09-16,2020-09-16 signatures=0
X-Proofpoint-Spam-Details: rule=fb_default_notspam policy=fb_default score=0 lowpriorityscore=0
 mlxlogscore=999 adultscore=0 impostorscore=0 suspectscore=0 bulkscore=0
 malwarescore=0 priorityscore=1501 spamscore=0 mlxscore=0 phishscore=0
 clxscore=1015 classifier=spam adjust=0 reason=mlx scancount=1
 engine=8.12.0-2006250000 definitions=main-2009160109
X-FB-Internal: deliver
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 16 Sep 2020, at 10:30, Christoph Hellwig wrote:

> On Wed, Sep 16, 2020 at 10:20:52AM -0400, Chris Mason wrote:
>> It???s not completely clear what you???re asking for here.  If the 
>> API
>> matches what???s in zstd-1.4.6, that seems like a reasonable way to 
>> label
>> it.  That???s what the upstream is for this code.
>>
>> I???m also not sure why we???re taking extra time to shit on the zstd
>> userspace package.  Can we please be constructive or at least 
>> actionable?
>
> Because it really doesn't matter that these crappy APIs he is
> introducing match anything, especially not something done as horribly
> as the zstd API.  We'll need to do this properly, and claiming
> compliance to some version of this lousy API is completely irrelevant
> for the kernel.

If the underlying goal is to closely follow the upstream of another 
project, we’re much better off using those APIs as provided.

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.

There are probably 1000 constructive ways to have that conversation.  
Please choose one of those instead of being an asshole.

-chris