From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pa0-f45.google.com ([209.85.220.45]:33722 "EHLO mail-pa0-f45.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1761108AbcIWQwD (ORCPT ); Fri, 23 Sep 2016 12:52:03 -0400 Received: by mail-pa0-f45.google.com with SMTP id hm5so41904345pac.0 for ; Fri, 23 Sep 2016 09:52:02 -0700 (PDT) Date: Fri, 23 Sep 2016 09:52:01 -0700 From: Omar Sandoval To: dsterba@suse.cz, linux-btrfs@vger.kernel.org, kernel-team@fb.com, Chandan Rajendra , Anatoly Pugachev Subject: Re: [PATCH v2 6/6] Btrfs: use less memory for delalloc sanity tests Message-ID: <20160923165201.GA28767@vader> References: <20160923092727.GU16983@twin.jikos.cz> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <20160923092727.GU16983@twin.jikos.cz> Sender: linux-btrfs-owner@vger.kernel.org List-ID: On Fri, Sep 23, 2016 at 11:27:27AM +0200, David Sterba wrote: > On Thu, Sep 22, 2016 at 05:24:25PM -0700, Omar Sandoval wrote: > > From: Omar Sandoval > > > > test_find_delalloc() allocates 256 MB worth of pages. That's all of the > > RAM that my MIPS emulator has, so it ends up panicking after it OOM > > kills everything. We don't actually need to use that much for this test. > > I'm not sure we should limit it that way as more bytes can give it more > stress. Can we do it somehow dynamically ? Like start with 256M and fall > back to the numbers you've used. > > Or maybe start from the low bound and allocate until it fails with first > ENOMEM. I figured out how to get qemu to do highmem on MIPS, so we can drop this patch. It's reasonable to assume that the sanity tests are running on a machine with a decent amount of memory. -- Omar