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 E2837C19F29 for ; Wed, 27 Jul 2022 09:33:41 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 5C98A94000D; Wed, 27 Jul 2022 05:33:41 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 57758940009; Wed, 27 Jul 2022 05:33:41 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 4667894000D; Wed, 27 Jul 2022 05:33:41 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0013.hostedemail.com [216.40.44.13]) by kanga.kvack.org (Postfix) with ESMTP id 3822B940009 for ; Wed, 27 Jul 2022 05:33:41 -0400 (EDT) Received: from smtpin16.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay10.hostedemail.com (Postfix) with ESMTP id BBCE8C0B5A for ; Wed, 27 Jul 2022 09:33:40 +0000 (UTC) X-FDA: 79732367400.16.30DFE28 Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by imf07.hostedemail.com (Postfix) with ESMTP id 15A9440084 for ; Wed, 27 Jul 2022 09:33:38 +0000 (UTC) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 03FA9617C4; Wed, 27 Jul 2022 09:33:38 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 3F1E8C433C1; Wed, 27 Jul 2022 09:33:35 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1658914417; bh=6Mk43boL7n4AfNnssO3TRzTylRUQVCu7yWdz07hbC5g=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=npDt2sUoMwyKg8WwtJeoFBJwPiY+9K+He+v6kiOSBPXh0JOfeLqelTMS6uGRqBB9g mIQT2BJ2B5bcEp6PSm4MyDv1vl/wp8omH6D7gVAmFI1U5h3BeYxUYPx+mykSESVjWR zvw6vD1DT3aa/7f66oeJ2Hupd5UGxXR+L7hDdFRhNNJOFYZ0whblf+bxe/PGd9EGZz aKBZkx457mFkzxf6ta1jzIrIFkQG1FmeV0IVD90Siy8fIu6cfwHOye9yLg3yHp6BNS FWGSOrNZW7ZhhVG6PyshKkH/T3GbHtCjPcrKBcOoGRJmkTut0tnSVlIPa4TxoZvq/k ntSsno6Chr42g== Date: Wed, 27 Jul 2022 12:33:24 +0300 From: Mike Rapoport To: Liu Xinpeng Cc: mawupeng1@huawei.com, jrdr.linux@gmail.com, david@redhat.com, linux-mm@kvack.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v3] memblock tests: compiling error Message-ID: References: <1658889585-20778-1-git-send-email-liuxp11@chinatelecom.cn> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1658889585-20778-1-git-send-email-liuxp11@chinatelecom.cn> ARC-Authentication-Results: i=1; imf07.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=npDt2sUo; spf=pass (imf07.hostedemail.com: domain of rppt@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=rppt@kernel.org; dmarc=pass (policy=none) header.from=kernel.org ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1658914419; a=rsa-sha256; cv=none; b=bIRl158TI0BDMQreJbD906gcKyfTiKHp/ZRQDdntA+kc03CJtpLsr+iMfgT8X/CfXjzfRl A92Rmuc5MUkGKVxuRgfKxYWeuz6IpNlPC6WrTz98KN9UoBTb7k3Pm+nlPIcC0MLXOuRPPO 0bqgan5neiuhSEzXNrdtZTxd62MQvQo= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1658914419; 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=towN4j8h+cCHbrNSu7IpSY9rHCJs3lDPE+3LaXyfy3I=; b=XUJGBFH/1oT8c9hMYAbZ8k1KxyhdJTfBMfoRB08uHJZ6zB2LMwk6HUVsbi0XHyzmMyhMJe UedkceHpZFbKFcIDPQaZMKDqi5Cyjd7HLZ5sJmTcNwp4MHhjMnUFqpppDOEM7nsQiKBb8W Jv/c4VwQqN+E6ptMGacN2qrEf9JGJUQ= X-Rspamd-Server: rspam11 X-Rspamd-Queue-Id: 15A9440084 Authentication-Results: imf07.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=npDt2sUo; spf=pass (imf07.hostedemail.com: domain of rppt@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=rppt@kernel.org; dmarc=pass (policy=none) header.from=kernel.org X-Stat-Signature: 6wwdt9dhzfs9namuiqf3n31w4ibst58w X-Rspam-User: X-HE-Tag: 1658914418-993435 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: Hi, On Wed, Jul 27, 2022 at 10:39:45AM +0800, Liu Xinpeng wrote: > Subject: [PATCH v3] memblock tests: compiling error "fix compilation errors" looks better, don't you think? Another thing, is when you post vN of a patch please add a "patch changelog" that describes the changes to the previous version. I'd suggest to take a look at sections "Respond to review comments" and "Don't get discouraged - or impatient" of the submitting patches guide. https://kernel.org/doc/html/latest/process/submitting-patches.html > memblock.o: In function `memblock_find_in_range.constprop.9': > memblock.c:(.text+0x4651): undefined reference to `pr_warn_ratelimited' > memblock.o: In function `memblock_mark_mirror': > memblock.c:(.text+0x7171): undefined reference to `mirrored_kernelcore' As people pointed in v1 thread, the description on how to reproduce this is missing. > Fixes: 902c2d91582c ("memblock: Disable mirror feature if kernelcore is not specified") > Fixes: 14d9a675fd0d ("mm: Ratelimited mirrored memory related warning messages") > > Signed-off-by: Liu Xinpeng > Tested-by: Ma Wupeng > --- > tools/testing/memblock/internal.h | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/tools/testing/memblock/internal.h b/tools/testing/memblock/internal.h > index c2a492c05e0c..fdb7f5db7308 100644 > --- a/tools/testing/memblock/internal.h > +++ b/tools/testing/memblock/internal.h > @@ -9,6 +9,10 @@ > static int memblock_debug = 1; > #endif > > +#define pr_warn_ratelimited(fmt, ...) printf(fmt, ##__VA_ARGS__) > + > +bool mirrored_kernelcore = false; > + > struct page {}; > > void memblock_free_pages(struct page *page, unsigned long pfn, > -- > 1.8.3.1 > -- Sincerely yours, Mike.