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=-18.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,NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS, URIBL_BLOCKED,USER_AGENT_SANE_1 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 E7540C4338F for ; Fri, 20 Aug 2021 08:41:39 +0000 (UTC) Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) (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 8DCD861101 for ; Fri, 20 Aug 2021 08:41:39 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 8DCD861101 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=xen.org Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=lists.xenproject.org Received: from list by lists.xenproject.org with outflank-mailman.169327.309284 (Exim 4.92) (envelope-from ) id 1mH05b-0006pZ-B4; Fri, 20 Aug 2021 08:41:23 +0000 X-Outflank-Mailman: Message body and most headers restored to incoming version Received: by outflank-mailman (output) from mailman id 169327.309284; Fri, 20 Aug 2021 08:41:23 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1mH05b-0006pS-7v; Fri, 20 Aug 2021 08:41:23 +0000 Received: by outflank-mailman (input) for mailman id 169327; Fri, 20 Aug 2021 08:41:22 +0000 Received: from mail.xenproject.org ([104.130.215.37]) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1mH05a-0006pM-AN for xen-devel@lists.xenproject.org; Fri, 20 Aug 2021 08:41:22 +0000 Received: from xenbits.xenproject.org ([104.239.192.120]) by mail.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1mH05Z-0005og-DC; Fri, 20 Aug 2021 08:41:21 +0000 Received: from [54.239.6.184] (helo=a483e7b01a66.ant.amazon.com) by xenbits.xenproject.org with esmtpsa (TLS1.3:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.92) (envelope-from ) id 1mH05Z-0002ML-6v; Fri, 20 Aug 2021 08:41:21 +0000 X-BeenThere: xen-devel@lists.xenproject.org List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Precedence: list Sender: "Xen-devel" DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=xen.org; s=20200302mail; h=Content-Transfer-Encoding:Content-Type:In-Reply-To: MIME-Version:Date:Message-ID:From:References:Cc:To:Subject; bh=LIYsFxPsJDNA62TqrSk11n9tgyiR+UX+MfcUGqlEZug=; b=4cZuA4HtNztt7QowZktGSb0gub uuqys+Jpm4gKzpp08ZCDBdofzKKcjZJBspAp3C1L+IuxYx2EMOmaSEJWv+KZqElC7Qt1A6vWJlTsy 5/ZSBTYrRjLwqQYjJ2G4CCcM9aTqUI5R5UoixJkXh1D0e8LKq8IBOU9FPK1pJr26HnCo=; Subject: Re: [XEN RFC PATCH 17/40] xen/arm: Introduce DEVICE_TREE_NUMA Kconfig for arm64 To: Wei Chen , "xen-devel@lists.xenproject.org" , "sstabellini@kernel.org" , "jbeulich@suse.com" Cc: Bertrand Marquis References: <20210811102423.28908-1-wei.chen@arm.com> <20210811102423.28908-18-wei.chen@arm.com> <1fbcb8a6-67b6-e2d6-6524-cfce7a8a6557@xen.org> From: Julien Grall Message-ID: Date: Fri, 20 Aug 2021 09:41:19 +0100 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:78.0) Gecko/20100101 Thunderbird/78.13.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-GB Content-Transfer-Encoding: 8bit On 20/08/2021 03:30, Wei Chen wrote: > Hi Julien, Hi Wei, >> -----Original Message----- >> From: Julien Grall >> Sent: 2021年8月19日 21:38 >> To: Wei Chen ; xen-devel@lists.xenproject.org; >> sstabellini@kernel.org; jbeulich@suse.com >> Cc: Bertrand Marquis >> Subject: Re: [XEN RFC PATCH 17/40] xen/arm: Introduce DEVICE_TREE_NUMA >> Kconfig for arm64 >> >> Hi, >> >> On 11/08/2021 11:24, Wei Chen wrote: >>> We need a Kconfig option to distinguish with ACPI based >>> NUMA. So we introduce the new Kconfig option: >>> DEVICE_TREE_NUMA in this patch for Arm64. >>> >>> Signed-off-by: Wei Chen >>> --- >>> xen/arch/arm/Kconfig | 10 ++++++++++ >>> 1 file changed, 10 insertions(+) >>> >>> diff --git a/xen/arch/arm/Kconfig b/xen/arch/arm/Kconfig >>> index ecfa6822e4..678cc98ea3 100644 >>> --- a/xen/arch/arm/Kconfig >>> +++ b/xen/arch/arm/Kconfig >>> @@ -33,6 +33,16 @@ config ACPI >>> Advanced Configuration and Power Interface (ACPI) support for Xen >> is >>> an alternative to device tree on ARM64. >>> >>> +config DEVICE_TREE_NUMA >> >> The name suggests that NUMA should only be enabled for Device-Tree... >> But the description looks generic. >> >> However, I think the user should only have the choice to say whether >> they want NUMA to be enabled or not. We should not give them the choice >> to enable/disable the parsing for DT/ACPI. >> >> So we should have a generic config that will then select DT (and ACPI in >> the future). >> > > How about we select DT_NUMA default on Arm64. And DT_NUMA select NUMA > like what we have done in patch#6 in x86? And remove the description? I would rather not make NUMA supported by default on Arm64. Instead, we should go throught the same process as other new features and gate it behind UNSUPPORTED until it is mature enough. > > If we make generic NUMA as a selectable option, and depends on > NUMA to select DT or ACPI NUMA. It seems to be quite different from > the existing logic? I am a bit confused. You added just logic to select NUMA from ACPI, right? So are you talking about a different logic? Cheers, -- Julien Grall