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.8 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,NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS 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 13379C48BE5 for ; Tue, 15 Jun 2021 14:36:00 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 1E5BF61490 for ; Tue, 15 Jun 2021 14:35:59 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 1E5BF61490 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=telliq.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 6B5A26B0036; Tue, 15 Jun 2021 10:35:58 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 6651D6B006E; Tue, 15 Jun 2021 10:35:58 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 4DEA36B0070; Tue, 15 Jun 2021 10:35:58 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0220.hostedemail.com [216.40.44.220]) by kanga.kvack.org (Postfix) with ESMTP id 1B30F6B0036 for ; Tue, 15 Jun 2021 10:35:58 -0400 (EDT) Received: from smtpin01.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay05.hostedemail.com (Postfix) with ESMTP id AE83A181AEF07 for ; Tue, 15 Jun 2021 14:35:57 +0000 (UTC) X-FDA: 78256206504.01.263D192 Received: from mail-lj1-f171.google.com (mail-lj1-f171.google.com [209.85.208.171]) by imf27.hostedemail.com (Postfix) with ESMTP id 87D30801936A for ; Tue, 15 Jun 2021 14:35:47 +0000 (UTC) Received: by mail-lj1-f171.google.com with SMTP id d2so25289167ljj.11 for ; Tue, 15 Jun 2021 07:35:56 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=telliq.com; s=google; h=subject:from:to:cc:references:message-id:date:user-agent :mime-version:in-reply-to:content-transfer-encoding; bh=GbBjUIoFxvaBNvRzYWx5ZKwYntRTHGOfw5r31RdP6mA=; b=IMJSq8e5A+lG5eMcFWdHiHrDuLuqhIVnGW0T71EplYsa4Lur3IH8gUjOPcu9BHSbOj XOeed467PU2KrYTmMhj6ondu2QLwzwJgyUTSipsYXcmyA+7gM7XgRyzq/TGOYa/7Ktc+ L7mVSftetjKU2ZmW+VilAwIMA0yj2t1lRGlJ8gpuSEQ7dC6+D1GayrR6FojZOFKWXplb wcZaKzZ8gB816jMkaNfn4+wFGf3w7TjerNTGMTZGj1mPO/lW0NaANLJlZ8zoJ8UL8FWY zxRSYfrNXVVkWw+vlAaHQcJBW1/BlE2MH1t6ed6AqZtP3HqD2AHgjVT1XxVwwWwSjEFO NwUw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:from:to:cc:references:message-id:date :user-agent:mime-version:in-reply-to:content-transfer-encoding; bh=GbBjUIoFxvaBNvRzYWx5ZKwYntRTHGOfw5r31RdP6mA=; b=an3AQSKJXa8ObKqmp/KMkWQeFWwz4MyjIMKu5Vdmec81Db/UoeRRsBnnF949qvylA0 lgfnOBwMgU7Fhuen5hRuPQ1yhKiS+A8c66xzC91j7/n97CKvho3OOyeFVLxqRr5fDiUb NFQfMIP0cLkhlynpJM0zzpKzVMAoxeeoXBXea2GGpqAc3CK3/vK4xOJnNevmAenkKfhy /Kx90qXl5CWoD66ttCeTbvspmnEsgGz99YUSuYJbjMFk/9ktRDfPdMu5newYiHljlpSh 9sKn73kBi5dwgmRKkwsD4oFvvj/bcHvvFvQFIMUIFBfj3KS4ZXa2lYg5X8dI+otGTwlK azww== X-Gm-Message-State: AOAM533ZfNInl0cjHq7mDBySTLgXE3nx/Ja7LdFQPr6VZw97iJm3u9Jz XFRaPqe6gsBeyoDzQybqkY7T9TmWmlL2KR/b X-Google-Smtp-Source: ABdhPJyqTukCl34ZxxTUg+/JaHiogEsKb1WJUYNUJVIPVwBfYteGIrktkGf6WTUoEXJiOj01+8Gt8w== X-Received: by 2002:a2e:b0e7:: with SMTP id h7mr18808677ljl.2.1623767753954; Tue, 15 Jun 2021 07:35:53 -0700 (PDT) Received: from polera.kvasta (h77-53-209-86.cust.a3fiber.se. [77.53.209.86]) by smtp.gmail.com with ESMTPSA id s21sm1824786lfc.269.2021.06.15.07.35.53 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 15 Jun 2021 07:35:53 -0700 (PDT) Subject: Re: [PATCH] arm: Define arch_is_kernel_initmem_freed() for lockdep From: Jan Kardell To: Linux ARM Cc: linux-kernel@vger.kernel.org, Russell King , linux-omap@vger.kernel.org, Andrew Morton , linux-mm@kvack.org References: <20210329175318.21393-1-jan.kardell@telliq.com> Message-ID: <754bde27-df3b-d98e-2ca8-5b16ccdc5345@telliq.com> Date: Tue, 15 Jun 2021 16:35:52 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Firefox/60.0 SeaMonkey/2.53.7.1 MIME-Version: 1.0 In-Reply-To: <20210329175318.21393-1-jan.kardell@telliq.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Authentication-Results: imf27.hostedemail.com; dkim=pass header.d=telliq.com header.s=google header.b=IMJSq8e5; dmarc=none; spf=pass (imf27.hostedemail.com: domain of jan@arkub.com designates 209.85.208.171 as permitted sender) smtp.mailfrom=jan@arkub.com X-Stat-Signature: 95znc4m8ycdnw9asi337iwyww5185sq7 X-Rspamd-Server: rspam04 X-Rspamd-Queue-Id: 87D30801936A X-HE-Tag: 1623767747-720177 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: A ping on this as there is no reply yet. I'm not entirely sure who should be on the CC list, or maybe this patch is just forgotten for some reason, maybe I should have written a better subject? I don't mean to complain, I do know that core developers are very busy! //Jan Jan Kardell skrev: > This fixes the warning: > > WARNING: CPU: 0 PID: 1994 at kernel/locking/lockdep.c:1119 alloc_netdev_mqs+0xb4/0x3b0 > > This warning is because the check in static_obj() assumes that all > memory within [_stext, _end] belongs to static objects. The init > section is also part of this range, and freeing it allows the buddy > allocator to allocate memory from it. > > To fix this, define arch_is_kernel_initmem_freed() for arm, it will > return 1 if initmem has been freed and the address is in the range > [__init_begin, __init_end], and this function is called by the > static_obj() function in lockdep. > > Tested on TI am3352 (Cortex A8). > > This change is modelled after commit 7a5da02de8d6eafba995 > ("locking/lockdep: check for freed initmem in static_obj()") for s390 by > Gerald Schaefer > > Signed-off-by: Jan Kardell > --- > arch/arm/include/asm/sections.h | 13 +++++++++++++ > arch/arm/mm/init.c | 6 +++++- > 2 files changed, 18 insertions(+), 1 deletion(-) > > diff --git a/arch/arm/include/asm/sections.h b/arch/arm/include/asm/sections.h > index 700b8bcdf9bd..1dd64c90c1ac 100644 > --- a/arch/arm/include/asm/sections.h > +++ b/arch/arm/include/asm/sections.h > @@ -2,6 +2,8 @@ > #ifndef _ASM_ARM_SECTIONS_H > #define _ASM_ARM_SECTIONS_H > > +#define arch_is_kernel_initmem_freed arch_is_kernel_initmem_freed > + > #include > > extern char _exiprom[]; > @@ -11,6 +13,17 @@ extern char __idmap_text_end[]; > extern char __entry_text_start[]; > extern char __entry_text_end[]; > > +extern bool initmem_freed; > + > +static inline int arch_is_kernel_initmem_freed(unsigned long addr) > +{ > + if (!initmem_freed) > + return 0; > + return addr >= (unsigned long)__init_begin && > + addr < (unsigned long)__init_end; > +} > + > + > static inline bool in_entry_text(unsigned long addr) > { > return memory_contains(__entry_text_start, __entry_text_end, > diff --git a/arch/arm/mm/init.c b/arch/arm/mm/init.c > index 828a2561b229..cd7a4273797a 100644 > --- a/arch/arm/mm/init.c > +++ b/arch/arm/mm/init.c > @@ -41,6 +41,8 @@ > > #include "mm.h" > > +bool initmem_freed; > + > #ifdef CONFIG_CPU_CP15_MMU > unsigned long __init __clear_cr(unsigned long mask) > { > @@ -523,8 +525,10 @@ void free_initmem(void) > fix_kernmem_perms(); > > poison_init_mem(__init_begin, __init_end - __init_begin); > - if (!machine_is_integrator() && !machine_is_cintegrator()) > + if (!machine_is_integrator() && !machine_is_cintegrator()) { > + initmem_freed = true; > free_initmem_default(-1); > + } > } > > #ifdef CONFIG_BLK_DEV_INITRD