Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754055AbdGXNor (ORCPT ); Mon, 24 Jul 2017 09:44:47 -0400 Received: from mail-it0-f68.google.com ([209.85.214.68]:33959 "EHLO mail-it0-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751837AbdGXNoj (ORCPT ); Mon, 24 Jul 2017 09:44:39 -0400 Subject: Re: [PATCH v3 0/4] Add xxhash and zstd modules To: Adam Borowski Cc: Nick Terrell , Herbert Xu , kernel-team@fb.com, Chris Mason , Yann Collet , David Sterba , squashfs-devel@lists.sourceforge.net, linux-btrfs@vger.kernel.org, linux-kernel@vger.kernel.org References: <20170720212743.3351201-1-terrelln@fb.com> <20170722113503.nf227bdrzx6637wl@angband.pl> From: "Austin S. Hemmelgarn" Message-ID: Date: Mon, 24 Jul 2017 09:44:35 -0400 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 MIME-Version: 1.0 In-Reply-To: <20170722113503.nf227bdrzx6637wl@angband.pl> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2267 Lines: 64 On 2017-07-22 07:35, Adam Borowski wrote: > On Fri, Jul 21, 2017 at 11:56:21AM -0400, Austin S. Hemmelgarn wrote: >> On 2017-07-20 17:27, Nick Terrell wrote: >>> This patch set adds xxhash, zstd compression, and zstd decompression >>> modules. It also adds zstd support to BtrFS and SquashFS. >>> >>> Each patch has relevant summaries, benchmarks, and tests. >> >> For patches 2-3, I've compile tested and had runtime testing running for >> about 18 hours now with no issues, so you can add: >> >> Tested-by: Austin S. Hemmelgarn > > I assume you haven't tried it on arm64, right? > > I had no time to get 'round to it before, and just got the following build > failure: > > CC fs/btrfs/zstd.o > In file included from fs/btrfs/zstd.c:28:0: > fs/btrfs/compression.h:39:2: error: unknown type name ‘refcount_t’ > refcount_t pending_bios; > ^~~~~~~~~~ > scripts/Makefile.build:302: recipe for target 'fs/btrfs/zstd.o' failed > > It's trivially fixably by: > --- a/fs/btrfs/zstd.c > +++ b/fs/btrfs/zstd.c > @@ -24,6 +24,7 @@ > #include > #include > #include > +#include > #include > #include "compression.h" > > after which it works fine, although half an hour of testing isn't exactly > exhaustive. I did, and didn't hit this somehow... Off to go verify my tool-chain and scripts then... > > > Alas, the armhf machine I ran stress tests (Debian archive rebuilds) on > doesn't boot with 4.13-rc1 due to some unrelated regression, bisecting that > would be quite painful so I did not try yet. I guess re-testing your patch > set on 4.12, even with btrfs-for-4.13 (which it had for a while), wouldn't > be of much help. So far, previous versions have been running for weeks, > with no issue since you fixed workspace flickering. I also didn't see this, but I test on some seriously bare-bones configurations for both the 32-bit ARM tests I run. On further inspection, it looks like my scripts decided to use btrfs-for-4.13 as the base, not 4.13-rc1 like I thought they did, so I don't know anymore how helpful my testing may have been. > > > On amd64 all is fine. > > > I haven't tested SquashFS at all. > > > Meow! >