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=-12.7 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH, MAILING_LIST_MULTI,NICE_REPLY_A,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS, URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=unavailable 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 3D607C4363D for ; Wed, 23 Sep 2020 04:46:47 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id BC0962068D for ; Wed, 23 Sep 2020 04:46:46 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="n7ab9AdK" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org BC0962068D Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=arm.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:Message-ID:References: To:Subject:From:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=cYtFNhKlZvwUC11gk8VLlGhmFDzyn5oRHfN+76HDrEg=; b=n7ab9AdKnwgUKWtBTc1Ud5i+S gz2i4L/hobPeYQimhl+AX+K3+s2+W4LdUHdV7LkVx8/cgGPatmnOBbQ96x9VmfILIh8t9E4c5DGgY hd5BP9i3uF6fFJiS/JqcXgV0u9PFiPGZR3+sTHxS1WP6UvRfrl6lKkQ+XSgZao/1DPE+B1kDHc6Kj xm49qzTX+MU06ALnghgQ8HbS9/inLSKCNl16OvbYnAPPmwnTAeSLQPeSgaV1nMVHR+AZxtiskQj7t wfEP8EvTpEwbYMgDC1O45PY7TCmqmrtm3HmEyUHRloYzK8ooqaZHb24iHYPQLvw45GCzMqRq/ThOn 2Wtw3yQ1w==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1kKweg-0003hg-IN; Wed, 23 Sep 2020 04:45:22 +0000 Received: from foss.arm.com ([217.140.110.172]) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1kKwed-0003hG-Tx for linux-arm-kernel@lists.infradead.org; Wed, 23 Sep 2020 04:45:20 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 9F8B6D6E; Tue, 22 Sep 2020 21:45:16 -0700 (PDT) Received: from [10.163.72.76] (unknown [10.163.72.76]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 1C2DB3F718; Tue, 22 Sep 2020 21:45:13 -0700 (PDT) From: Anshuman Khandual Subject: Re: [PATCH V3 2/3] arm64/mm/hotplug: Enable MEM_OFFLINE event handling To: linux-arm-kernel@lists.infradead.org References: <1600689908-28213-1-git-send-email-anshuman.khandual@arm.com> <1600689908-28213-3-git-send-email-anshuman.khandual@arm.com> Message-ID: <9b00cfc3-ef97-f476-28d4-93c2d5842e37@arm.com> Date: Wed, 23 Sep 2020 10:14:38 +0530 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 MIME-Version: 1.0 In-Reply-To: <1600689908-28213-3-git-send-email-anshuman.khandual@arm.com> Content-Language: en-US X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200923_004520_031146_E42E47A2 X-CRM114-Status: GOOD ( 25.06 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Mark Rutland , Will Deacon , Steve Capper , catalin.marinas@arm.com, linux-kernel@vger.kernel.org, Mark Brown , Marc Zyngier , will@kernel.org Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 09/21/2020 05:35 PM, Anshuman Khandual wrote: > This enables MEM_OFFLINE memory event handling. It will help intercept any > possible error condition such as if boot memory some how still got offlined > even after an explicit notifier failure, potentially by a future change in > generic hot plug framework. This would help detect such scenarios and help > debug further. > > Cc: Catalin Marinas > Cc: Will Deacon > Cc: Mark Rutland > Cc: Marc Zyngier > Cc: Steve Capper > Cc: Mark Brown > Cc: linux-arm-kernel@lists.infradead.org > Cc: linux-kernel@vger.kernel.org > Signed-off-by: Anshuman Khandual > --- > arch/arm64/mm/mmu.c | 37 ++++++++++++++++++++++++++++++++----- > 1 file changed, 32 insertions(+), 5 deletions(-) > > diff --git a/arch/arm64/mm/mmu.c b/arch/arm64/mm/mmu.c > index df3b7415b128..6b171bd88bcf 100644 > --- a/arch/arm64/mm/mmu.c > +++ b/arch/arm64/mm/mmu.c > @@ -1482,13 +1482,40 @@ static int prevent_bootmem_remove_notifier(struct notifier_block *nb, > unsigned long end_pfn = arg->start_pfn + arg->nr_pages; > unsigned long pfn = arg->start_pfn; > > - if (action != MEM_GOING_OFFLINE) > + if ((action != MEM_GOING_OFFLINE) && (action != MEM_OFFLINE)) > return NOTIFY_OK; > > - for (; pfn < end_pfn; pfn += PAGES_PER_SECTION) { > - ms = __pfn_to_section(pfn); > - if (early_section(ms)) > - return NOTIFY_BAD; > + if (action == MEM_GOING_OFFLINE) { > + for (; pfn < end_pfn; pfn += PAGES_PER_SECTION) { > + ms = __pfn_to_section(pfn); > + if (early_section(ms)) { > + pr_warn("Boot memory offlining attempted\n"); > + return NOTIFY_BAD; > + } > + } > + } else if (action == MEM_OFFLINE) { > + for (; pfn < end_pfn; pfn += PAGES_PER_SECTION) { > + ms = __pfn_to_section(pfn); > + if (early_section(ms)) { > + > + /* > + * This should have never happened. Boot memory > + * offlining should have been prevented by this > + * very notifier. Probably some memory removal > + * procedure might have changed which would then > + * require further debug. > + */ > + pr_err("Boot memory offlined\n"); It is returning in the first instance, when a section inside the offline range happen to be part of the boot memory. So wondering if it would be better to call out here, entire attempted offline range or just the first section inside that which overlaps with boot memory ? But some range information here will be helpful. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel