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=-5.5 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS, USER_AGENT_SANE_1 autolearn=no 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 9615FC433DF for ; Wed, 5 Aug 2020 08:38:36 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 81A3F21744 for ; Wed, 5 Aug 2020 08:38:36 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727794AbgHEIif (ORCPT ); Wed, 5 Aug 2020 04:38:35 -0400 Received: from elvis.franken.de ([193.175.24.41]:35123 "EHLO elvis.franken.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726175AbgHEIie (ORCPT ); Wed, 5 Aug 2020 04:38:34 -0400 Received: from uucp (helo=alpha) by elvis.franken.de with local-bsmtp (Exim 3.36 #1) id 1k3EwR-0008Mh-00; Wed, 05 Aug 2020 10:38:31 +0200 Received: by alpha.franken.de (Postfix, from userid 1000) id 50BAFC0BDA; Wed, 5 Aug 2020 10:37:20 +0200 (CEST) Date: Wed, 5 Aug 2020 10:37:20 +0200 From: Thomas Bogendoerfer To: Mike Rapoport Cc: kernel test robot , linux-kernel@vger.kernel.org, Paul Burton , linux-mips@vger.kernel.org Subject: Re: arch/mips/include/asm/mach-ip27/topology.h:19:7: error: implicit declaration of function 'hub_data' Message-ID: <20200805083720.GA7297@alpha.franken.de> References: <202008040108.xTarUIe8%lkp@intel.com> <20200803194910.GC72435@linux.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20200803194910.GC72435@linux.ibm.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Aug 03, 2020 at 10:49:10PM +0300, Mike Rapoport wrote: > On Tue, Aug 04, 2020 at 01:39:14AM +0800, kernel test robot wrote: > [...] > > > > All errors (new ones prefixed by >>): > > > > In file included from arch/mips/include/asm/topology.h:11, > > from include/linux/topology.h:36, > > from include/linux/gfp.h:9, > > from include/linux/slab.h:15, > > from include/linux/crypto.h:19, > > from include/crypto/hash.h:11, > > from include/linux/uio.h:10, > > from include/linux/socket.h:8, > > from include/linux/compat.h:15, > > from arch/mips/kernel/asm-offsets.c:12: > > arch/mips/include/asm/mach-ip27/topology.h:25:39: error: 'MAX_COMPACT_NODES' undeclared here (not in a function) > > 25 | extern unsigned char __node_distances[MAX_COMPACT_NODES][MAX_COMPACT_NODES]; > > | ^~~~~~~~~~~~~~~~~ > > include/linux/topology.h: In function 'numa_node_id': > > >> arch/mips/include/asm/mach-ip27/topology.h:16:27: error: implicit declaration of function 'cputonasid' [-Werror=implicit-function-declaration] > > 16 | #define cpu_to_node(cpu) (cputonasid(cpu)) > > | ^~~~~~~~~~ > > This happens when randconfig disables NUMA and has SGI_IP27 enabled. > Before switch from discontigmem to sparsemem, there always was > CONFIG_NEED_MULTIPLE_NODES=y because it was selected by DISCONTIGMEM. > Without DISCONTIGMEM it is possible to have SPARSEMEM without NUMA for > SGI_IP27 and as many things there rely on custom node definition, the > build breaks. > > I don't remember small Origin or Onyx systems so I think it would be > reasonable to make SGI_IP27 to select NUMA. IMHO there are right now too many places in IP27 code, which assumes NUMA enabled, so your patch makes sense. And if someone wants to get it supported without NUMA enabled, I'm taking that patch as well. > If the below patch makes sense I'll resend it formally. yes please. Thomas. -- Crap can work. Given enough thrust pigs will fly, but it's not necessarily a good idea. [ RFC1925, 2.3 ]