Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Feedback-ID: i56a14606:Fastmail
User-Agent: Cyrus-JMAP/3.9.0-alpha0-334-g8c072af647-fm-20230330.001-g8c072af6
Mime-Version: 1.0
Message-Id: <3c5d33f8-3669-4486-8986-d75fed8d882b@app.fastmail.com>
In-Reply-To: <202304151422.5e4d380b-oliver.sang@intel.com>
References: <202304151422.5e4d380b-oliver.sang@intel.com>
Date:   Sat, 15 Apr 2023 10:09:12 +0200
From:   "Arnd Bergmann" <arnd@arndb.de>
To:     "kernel test robot" <oliver.sang@intel.com>,
        "Arnd Bergmann" <arnd@kernel.org>,
        "Andrew Morton" <akpm@linux-foundation.org>
Cc:     oe-lkp@lists.linux.dev, "kernel test robot" <lkp@intel.com>,
        linux-snps-arc@lists.infradead.org, linux-mm@kvack.org,
        "Vineet Gupta" <vgupta@kernel.org>,
        "Mike Rapoport" <rppt@kernel.org>,
        "Nathan Chancellor" <nathan@kernel.org>,
        "Nick Desaulniers" <ndesaulniers@google.com>,
        "David Hildenbrand" <david@redhat.com>,
        "Vlastimil Babka" <vbabka@suse.cz>, llvm@lists.linux.dev,
        "Tom Rix" <trix@redhat.com>,
        "Liam R. Howlett" <Liam.Howlett@oracle.com>,
        "Matthew Wilcox" <willy@infradead.org>,
        "Suren Baghdasaryan" <surenb@google.com>,
        "Naoya Horiguchi" <naoya.horiguchi@nec.com>,
        linux-kernel@vger.kernel.org, "Oscar Salvador" <osalvador@suse.de>
Subject: Re: [PATCH] mm: make arch_has_descending_max_zone_pfns() static
Content-Type: text/plain
Precedence: bulk

On Sat, Apr 15, 2023, at 08:42, kernel test robot wrote:
> Hello,
>
> kernel test robot noticed "kernel_BUG_at_lib/list_debug.c" on:
>
> commit: c31fe5eb2ec68f6d2331fabc924e0030cc6bb2b3 ("[PATCH] mm: make 
> arch_has_descending_max_zone_pfns() static")

Thanks for the report, that was a logic error on my end, the fix is:

--- a/mm/mm_init.c
+++ b/mm/mm_init.c
@@ -1756,7 +1756,7 @@ static void __init free_area_init_memoryless_node(int nid)
  */
 static bool arch_has_descending_max_zone_pfns(void)
 {
-       return !IS_ENABLED(CONFIG_ARC_HAS_PAE40);
+       return IS_ENABLED(CONFIG_ARC) && !IS_ENABLED(CONFIG_ARC_HAS_PAE40);
 }
 
 /**

I'll send a replacement v2 that has this folded in.

    Arnd