Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Sun, 18 Sep 2022 10:32:05 +0300
From:   Mike Rapoport <rppt@kernel.org>
To:     Rebecca Mckeever <remckee0@gmail.com>
Cc:     linux-mm@kvack.org, linux-kernel@vger.kernel.org,
        David Hildenbrand <david@redhat.com>
Subject: Re: [PATCH v6 0/4] memblock tests: add NUMA tests for
 memblock_alloc_try_nid*
Message-ID: <YybJdXgUm1csEsWj@kernel.org>
References: <cover.1663046060.git.remckee0@gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <cover.1663046060.git.remckee0@gmail.com>
Precedence: bulk

On Tue, Sep 13, 2022 at 12:21:08AM -0500, Rebecca Mckeever wrote:
> These patches add additional tests for memblock_alloc_try_nid() and
> memblock_alloc_try_nid_raw() that use a simulated physical
> memory that is set up with multiple NUMA nodes. Additionally, most of
> these tests set nid != NUMA_NO_NODE.
> 
> To set up a simulated physical memory with multiple NUMA nodes, patch 1
> introduces setup_numa_memblock(). This function uses a previously
> allocated dummy physical memory. It can be used in place of
> setup_memblock() in tests that need to simulate a NUMA system.
> 
> These tests are run twice, once for memblock_alloc_try_nid() and once
> for memblock_alloc_try_nid_raw(), so that both functions are tested with
> the same set of tests. When the tests run memblock_alloc_try_nid(), they
> test that the entire memory region is zero. When the tests run
> memblock_alloc_try_nid_raw(), they test that the entire memory region is
> nonzero.
> 
> Rebecca Mckeever (4):
>   memblock tests: add simulation of physical memory with multiple NUMA
>     nodes
>   memblock tests: add top-down NUMA tests for memblock_alloc_try_nid*
>   memblock tests: add bottom-up NUMA tests for memblock_alloc_try_nid*
>   memblock tests: add generic NUMA tests for memblock_alloc_try_nid*
> 
>  .../testing/memblock/scripts/Makefile.include |    2 +-
>  tools/testing/memblock/tests/alloc_nid_api.c  | 1466 ++++++++++++++++-
>  tools/testing/memblock/tests/alloc_nid_api.h  |   16 +
>  tools/testing/memblock/tests/common.c         |   31 +
>  tools/testing/memblock/tests/common.h         |   22 +-
>  5 files changed, 1524 insertions(+), 13 deletions(-)

Applied, thanks! 

-- 
Sincerely yours,
Mike.