From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 03B08C00140 for ; Tue, 16 Aug 2022 02:15:26 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 66D8D8D0002; Mon, 15 Aug 2022 22:15:26 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 61CE78D0001; Mon, 15 Aug 2022 22:15:26 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 4BD348D0002; Mon, 15 Aug 2022 22:15:26 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0015.hostedemail.com [216.40.44.15]) by kanga.kvack.org (Postfix) with ESMTP id 39B338D0001 for ; Mon, 15 Aug 2022 22:15:26 -0400 (EDT) Received: from smtpin28.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay05.hostedemail.com (Postfix) with ESMTP id 06DDF4030D for ; Tue, 16 Aug 2022 02:15:26 +0000 (UTC) X-FDA: 79803839052.28.8EF13AB Received: from mail-qv1-f68.google.com (mail-qv1-f68.google.com [209.85.219.68]) by imf03.hostedemail.com (Postfix) with ESMTP id 9A4B5201B4 for ; Tue, 16 Aug 2022 02:15:25 +0000 (UTC) Received: by mail-qv1-f68.google.com with SMTP id ct13so6958075qvb.9 for ; Mon, 15 Aug 2022 19:15:25 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc; bh=gB3Bz5AjtMc1DQ28Gy11iyEZdEFjWI/WxGTjW9sxTk4=; b=kDpyqKy8UaICjCOfptA2Vxv2ycuXszlDZH6qt9rTGwwmjFFMw0as2C/jXQRMjXjaF0 QTfJg/qpsP1LtMhzd0UQdzAodu30VegCI1Er6f/wwJvIqXkhdKiyQHufvHyHDGnVRLdo WIg5v/KmBK2z6rBfvGZ5DG2BO94oNDow7ZMs7LCYm3l3EwoaOytdL1zaUOW9HYIzVEBd LVuDSSRcGxqDjGrBdW/m8Qo9rWAQAjCo3ej2fXpIjyxUPfG8PHYZZ58BRu/V/8JueCu4 mEzJcoLanv9hXQvTqh6wft8m3kJVsrGDBgkCg9jUqlxqMQqDpio6Euy3BehDX8RFDTvm +Okw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc; bh=gB3Bz5AjtMc1DQ28Gy11iyEZdEFjWI/WxGTjW9sxTk4=; b=scDYEAem66PazAqh+tG7qsr9I3+8aLDIdROSp0JcjBCUbAnKGFFLXt1L//av64CJOh mmJkChRuyKOKR9ENYmvzSOm5XMhAPqY1WDDYaPDQTdphkOpvfTEAQbtavS7aQhA9zQyw 3hYocnmNnMb8eVQp9uxEYrcqbT2TiKpVKLSYNY5jupjOZGX6q+uaWy0xN4FwjLq7oY3A 3pFIaqpfWkw+KgPJA8dfC3Gw+mvRSkgRS3jCMeJeuX2g2czKxGAZDaFK+TaQiTk7cuYB gEKiPps7ci/XtpI9QBbDAr+7DWkcMvk2+GMODvFHwxwU3/8R5V5G0hYirjVI2JNK5YN6 T0RQ== X-Gm-Message-State: ACgBeo1LSnTczz+skFNs7J5b14u1B1boxzBSkEKIkcgqwadUy4REzXuC iX+TIb5TnnZneXmT4M/bfyo= X-Google-Smtp-Source: AA6agR45rSHkYcQeehtkvWA9pSrdBHzeEoYrm7SlmlHjE205JkOiNLcJ4sXer3fnZSKH6KRAClWpAQ== X-Received: by 2002:ad4:5cc9:0:b0:475:5b35:3b03 with SMTP id iu9-20020ad45cc9000000b004755b353b03mr15928484qvb.42.1660616124808; Mon, 15 Aug 2022 19:15:24 -0700 (PDT) Received: from sophie ([185.156.46.184]) by smtp.gmail.com with ESMTPSA id ay36-20020a05620a17a400b006baf3ffba23sm7266045qkb.38.2022.08.15.19.15.24 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 15 Aug 2022 19:15:24 -0700 (PDT) Date: Mon, 15 Aug 2022 21:15:22 -0500 From: Rebecca Mckeever To: "Huang, Shaoqin" Cc: Mike Rapoport , linux-mm@kvack.org, linux-kernel@vger.kernel.org, David Hildenbrand Subject: Re: [PATCH 1/8] memblock tests: update tests to check if memblock_alloc zeroed memory Message-ID: <20220816021522.GA3590@sophie> References: <02d514d86f212c79e52792b8ecd919156d7a9c20.1660454730.git.remckee0@gmail.com> <5dde66a7-2236-a890-aaeb-a45876057bec@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <5dde66a7-2236-a890-aaeb-a45876057bec@intel.com> ARC-Authentication-Results: i=1; imf03.hostedemail.com; dkim=pass header.d=gmail.com header.s=20210112 header.b=kDpyqKy8; spf=pass (imf03.hostedemail.com: domain of remckee0@gmail.com designates 209.85.219.68 as permitted sender) smtp.mailfrom=remckee0@gmail.com; dmarc=pass (policy=none) header.from=gmail.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1660616125; a=rsa-sha256; cv=none; b=yE/JOe4OtX59YP3cGri/wGhoQyMaXU1Ftpcunam86r8zMXxJ/MW3MkO2iejvxgYX0ABNIN pQrZ3uohGaYVm/bIzNJUeBcT0gB0+j/AvAezgyVSQqKsPLraNmFJRmnTnQV34PhiscpfI5 y3waCyz4IqAnndi0hleLhaci0TCgr+0= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1660616125; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=gB3Bz5AjtMc1DQ28Gy11iyEZdEFjWI/WxGTjW9sxTk4=; b=pOUTq8l8A802Ci5SU7+/mVyN73EVVaL3h7urrE7Umwhff3Kk0Z6avHCUXXBHbnz5nK7dYI ZHyXmm26tbHlrQKQuO5k8+BjkL6MOXw+f+4PZLygaTPpJG2M5UTQj73GQa1LQxThDTuA0X zvkX19oCBeoq3et1mJqDCN4WcpMI0+0= Authentication-Results: imf03.hostedemail.com; dkim=pass header.d=gmail.com header.s=20210112 header.b=kDpyqKy8; spf=pass (imf03.hostedemail.com: domain of remckee0@gmail.com designates 209.85.219.68 as permitted sender) smtp.mailfrom=remckee0@gmail.com; dmarc=pass (policy=none) header.from=gmail.com X-Stat-Signature: p79x94ect3ezxt1i3msduc4pe34mcntn X-Rspamd-Server: rspam09 X-Rspam-User: X-Rspamd-Queue-Id: 9A4B5201B4 X-HE-Tag: 1660616125-472785 X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: On Mon, Aug 15, 2022 at 04:50:51PM +0800, Huang, Shaoqin wrote: > Hi, Rebecca. > > On 8/14/2022 1:53 PM, Rebecca Mckeever wrote: > > Add an assert in memblock_alloc() tests where allocation is expected to > > occur. The assert checks whether the entire chunk of allocated memory is > > cleared. > > > > The current memblock_alloc() tests do not check whether the allocated > > memory was zeroed. memblock_alloc() should zero the allocated memory since > > it is a wrapper for memblock_alloc_try_nid(). > > > > Signed-off-by: Rebecca Mckeever > > --- > > tools/testing/memblock/tests/alloc_api.c | 24 ++++++++++++++++++++++++ > > tools/testing/memblock/tests/common.c | 7 +++++++ > > tools/testing/memblock/tests/common.h | 13 +++++++++++++ > > 3 files changed, 44 insertions(+) > > > > diff --git a/tools/testing/memblock/tests/alloc_api.c b/tools/testing/memblock/tests/alloc_api.c > > index a14f38eb8a89..71c89cb9b2a8 100644 > > --- a/tools/testing/memblock/tests/alloc_api.c > > +++ b/tools/testing/memblock/tests/alloc_api.c > > @@ -22,6 +22,8 @@ static int alloc_top_down_simple_check(void) > > allocated_ptr = memblock_alloc(size, SMP_CACHE_BYTES); > > ASSERT_NE(allocated_ptr, NULL); > > + ASSERT_MEM_EQ((char *)allocated_ptr, 0, size); > > + > > ASSERT_EQ(rgn->size, size); > > ASSERT_EQ(rgn->base, expected_start); > > @@ -80,6 +82,8 @@ static int alloc_top_down_disjoint_check(void) > > allocated_ptr = memblock_alloc(r2_size, alignment); > > ASSERT_NE(allocated_ptr, NULL); > > + ASSERT_MEM_EQ((char *)allocated_ptr, 0, r2_size); > > + > > ASSERT_EQ(rgn1->size, r1.size); > > ASSERT_EQ(rgn1->base, r1.base); > > @@ -125,6 +129,8 @@ static int alloc_top_down_before_check(void) > > allocated_ptr = memblock_alloc(r2_size, SMP_CACHE_BYTES); > > ASSERT_NE(allocated_ptr, NULL); > > + ASSERT_MEM_EQ((char *)allocated_ptr, 0, r2_size); > > + > > ASSERT_EQ(rgn->size, total_size); > > ASSERT_EQ(rgn->base, memblock_end_of_DRAM() - total_size); > > @@ -173,6 +179,8 @@ static int alloc_top_down_after_check(void) > > allocated_ptr = memblock_alloc(r2_size, SMP_CACHE_BYTES); > > ASSERT_NE(allocated_ptr, NULL); > > + ASSERT_MEM_EQ((char *)allocated_ptr, 0, r2_size); > > + > > ASSERT_EQ(rgn->size, total_size); > > ASSERT_EQ(rgn->base, r1.base - r2_size); > > @@ -223,6 +231,8 @@ static int alloc_top_down_second_fit_check(void) > > allocated_ptr = memblock_alloc(r3_size, SMP_CACHE_BYTES); > > ASSERT_NE(allocated_ptr, NULL); > > + ASSERT_MEM_EQ((char *)allocated_ptr, 0, r3_size); > > + > > ASSERT_EQ(rgn->size, r2.size + r3_size); > > ASSERT_EQ(rgn->base, r2.base - r3_size); > > @@ -277,6 +287,8 @@ static int alloc_in_between_generic_check(void) > > allocated_ptr = memblock_alloc(r3_size, SMP_CACHE_BYTES); > > ASSERT_NE(allocated_ptr, NULL); > > + ASSERT_MEM_EQ((char *)allocated_ptr, 0, r3_size); > > + > > ASSERT_EQ(rgn->size, total_size); > > ASSERT_EQ(rgn->base, r1.base - r2.size - r3_size); > > @@ -418,6 +430,8 @@ static int alloc_limited_space_generic_check(void) > > allocated_ptr = memblock_alloc(available_size, SMP_CACHE_BYTES); > > ASSERT_NE(allocated_ptr, NULL); > > + ASSERT_MEM_EQ((char *)allocated_ptr, 0, available_size); > > + > > ASSERT_EQ(rgn->size, MEM_SIZE); > > ASSERT_EQ(rgn->base, memblock_start_of_DRAM()); > > @@ -442,6 +456,7 @@ static int alloc_no_memory_generic_check(void) > > PREFIX_PUSH(); > > reset_memblock_regions(); > > + fill_memblock(); > > Maybe we don't need this line, it has no effect at here. Anyway, Others > Looks Good to me. > Do you mean that it's not necessary since the test doesn't run verify_mem_content()? > Reviewed-by: Shaoqin Huang > > > allocated_ptr = memblock_alloc(SZ_1K, SMP_CACHE_BYTES); > > @@ -472,6 +487,8 @@ static int alloc_bottom_up_simple_check(void) > > allocated_ptr = memblock_alloc(SZ_2, SMP_CACHE_BYTES); > > ASSERT_NE(allocated_ptr, NULL); > > + ASSERT_MEM_EQ((char *)allocated_ptr, 0, SZ_2); > > + > > ASSERT_EQ(rgn->size, SZ_2); > > ASSERT_EQ(rgn->base, memblock_start_of_DRAM()); > > @@ -528,6 +545,7 @@ static int alloc_bottom_up_disjoint_check(void) > > allocated_ptr = memblock_alloc(r2_size, alignment); > > ASSERT_NE(allocated_ptr, NULL); > > + ASSERT_MEM_EQ((char *)allocated_ptr, 0, r2_size); > > ASSERT_EQ(rgn1->size, r1.size); > > ASSERT_EQ(rgn1->base, r1.base); > > @@ -571,6 +589,8 @@ static int alloc_bottom_up_before_check(void) > > allocated_ptr = memblock_alloc(r1_size, SMP_CACHE_BYTES); > > ASSERT_NE(allocated_ptr, NULL); > > + ASSERT_MEM_EQ((char *)allocated_ptr, 0, r1_size); > > + > > ASSERT_EQ(rgn->size, total_size); > > ASSERT_EQ(rgn->base, memblock_start_of_DRAM()); > > @@ -618,6 +638,8 @@ static int alloc_bottom_up_after_check(void) > > allocated_ptr = memblock_alloc(r2_size, SMP_CACHE_BYTES); > > ASSERT_NE(allocated_ptr, NULL); > > + ASSERT_MEM_EQ((char *)allocated_ptr, 0, r2_size); > > + > > ASSERT_EQ(rgn->size, total_size); > > ASSERT_EQ(rgn->base, r1.base); > > @@ -669,6 +691,8 @@ static int alloc_bottom_up_second_fit_check(void) > > allocated_ptr = memblock_alloc(r3_size, SMP_CACHE_BYTES); > > ASSERT_NE(allocated_ptr, NULL); > > + ASSERT_MEM_EQ((char *)allocated_ptr, 0, r3_size); > > + > > ASSERT_EQ(rgn->size, r2.size + r3_size); > > ASSERT_EQ(rgn->base, r2.base); > > diff --git a/tools/testing/memblock/tests/common.c b/tools/testing/memblock/tests/common.c > > index 76a8ad818f3a..0ca26fe12c38 100644 > > --- a/tools/testing/memblock/tests/common.c > > +++ b/tools/testing/memblock/tests/common.c > > @@ -60,16 +60,23 @@ void reset_memblock_attributes(void) > > memblock.current_limit = MEMBLOCK_ALLOC_ANYWHERE; > > } > > +void fill_memblock(void) > > +{ > > + memset(memory_block.base, 1, MEM_SIZE); > > +} > > + > > void setup_memblock(void) > > { > > reset_memblock_regions(); > > memblock_add((phys_addr_t)memory_block.base, MEM_SIZE); > > + fill_memblock(); > > } > > void dummy_physical_memory_init(void) > > { > > memory_block.base = malloc(MEM_SIZE); > > assert(memory_block.base); > > + fill_memblock(); > > } > > void dummy_physical_memory_cleanup(void) > > diff --git a/tools/testing/memblock/tests/common.h b/tools/testing/memblock/tests/common.h > > index d396e5423a8e..7a16a7dc8f2c 100644 > > --- a/tools/testing/memblock/tests/common.h > > +++ b/tools/testing/memblock/tests/common.h > > @@ -51,6 +51,18 @@ > > assert((_expected) < (_seen)); \ > > } while (0) > > +/** > > + * ASSERT_MEM_EQ(): > > + * Check that the first @_size bytes of @_seen are all equal to @_expected. > > + * If false, print failed test message (if running with --verbose) and then > > + * assert. > > + */ > > +#define ASSERT_MEM_EQ(_seen, _expected, _size) do { \ > > + for (int _i = 0; _i < (_size); _i++) { \ > > + ASSERT_EQ((_seen)[_i], (_expected)); \ > > + } \ > > +} while (0) > > + > > #define PREFIX_PUSH() prefix_push(__func__) > > /* > > @@ -70,6 +82,7 @@ struct region { > > void reset_memblock_regions(void); > > void reset_memblock_attributes(void); > > +void fill_memblock(void); > > void setup_memblock(void); > > void dummy_physical_memory_init(void); > > void dummy_physical_memory_cleanup(void);