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 X-Spam-Level: X-Spam-Status: No, score=-15.7 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 3C312C4361B for ; Mon, 14 Dec 2020 13:53:39 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 854AB22B40 for ; Mon, 14 Dec 2020 13:53:38 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 854AB22B40 Authentication-Results: mail.kernel.org; dmarc=fail (p=quarantine dis=none) header.from=suse.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 8B5236B005D; Mon, 14 Dec 2020 08:53:37 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 866586B0068; Mon, 14 Dec 2020 08:53:37 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 754EE6B006C; Mon, 14 Dec 2020 08:53:37 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0069.hostedemail.com [216.40.44.69]) by kanga.kvack.org (Postfix) with ESMTP id 5F4A06B005D for ; Mon, 14 Dec 2020 08:53:37 -0500 (EST) Received: from smtpin16.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay04.hostedemail.com (Postfix) with ESMTP id 17E652484 for ; Mon, 14 Dec 2020 13:53:37 +0000 (UTC) X-FDA: 77592030474.16.prose97_2b018022741b Received: from filter.hostedemail.com (10.5.16.251.rfc1918.com [10.5.16.251]) by smtpin16.hostedemail.com (Postfix) with ESMTP id EB3B4100E6903 for ; Mon, 14 Dec 2020 13:53:36 +0000 (UTC) X-HE-Tag: prose97_2b018022741b X-Filterd-Recvd-Size: 2894 Received: from mx2.suse.de (mx2.suse.de [195.135.220.15]) by imf29.hostedemail.com (Postfix) with ESMTP for ; Mon, 14 Dec 2020 13:53:36 +0000 (UTC) X-Virus-Scanned: by amavisd-new at test-mx.suse.de DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=susede1; t=1607954015; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=GAdjzavcb3mFBEHJB874Dk5WCRXNv3K3HXp41RD9ubI=; b=qcicvaw/qXksidM4WAVQXpf06NH7u8U/WthgWbvG71oMO+P8DvLLjWleNjevKNMUD2Eqov XIBd3er3QqDwU11Lytxc5wTf4VcwMC4lmbynDIsfQbKuzd5FLw5Mbx3ptlC3Vpu1hzD6CG 4JHbh0qPwaVovCnYAISNGq2IhA4luFY= Received: from relay2.suse.de (unknown [195.135.221.27]) by mx2.suse.de (Postfix) with ESMTP id E383DAC7F; Mon, 14 Dec 2020 13:53:34 +0000 (UTC) Date: Mon, 14 Dec 2020 14:53:34 +0100 From: Michal Hocko To: Laurent Dufour Cc: linux-mm@kvack.org, linux-kernel@vger.kernel.org, akpm@linux-foundation.org, nathanl@linux.ibm.com, cheloha@linux.ibm.com Subject: Re: [PATCH] mm/memory_hotplug: quieting offline operation Message-ID: <20201214135334.GC32193@dhcp22.suse.cz> References: <20201211150157.91399-1-ldufour@linux.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20201211150157.91399-1-ldufour@linux.ibm.com> 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 Fri 11-12-20 16:01:57, Laurent Dufour wrote: > On PowerPC, when dymically removing memory from a system we can see in the console a > lot of messages like this: > [ 186.575389] Offlined Pages 4096 > > This message is displayed on each LMB (256MB) removed, which means that we > removing 1TB of memory, this message is displayed 4096 times. > > Moving it to DEBUG to not flood the console. > > Signed-off-by: Laurent Dufour I am wondering whether we should be dropping this altogether. Offlining is returning an error status to its caller so e.g. userspace can find out the failure and the error code. Anyway, feel free to add Acked-by: Michal Hocko > --- > mm/memory_hotplug.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/mm/memory_hotplug.c b/mm/memory_hotplug.c > index b44d4c7ba73b..c47a53a16782 100644 > --- a/mm/memory_hotplug.c > +++ b/mm/memory_hotplug.c > @@ -1587,7 +1587,7 @@ int __ref offline_pages(unsigned long start_pfn, unsigned long nr_pages) > > /* Mark all sections offline and remove free pages from the buddy. */ > __offline_isolated_pages(start_pfn, end_pfn); > - pr_info("Offlined Pages %ld\n", nr_pages); > + pr_debug("Offlined Pages %ld\n", nr_pages); > > /* > * The memory sections are marked offline, and the pageblock flags > -- > 2.29.2 -- Michal Hocko SUSE Labs