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=-2.3 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_2 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 A2943C3276D for ; Thu, 2 Jan 2020 15:27:25 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 8317020848 for ; Thu, 2 Jan 2020 15:27:25 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728747AbgABP1Y convert rfc822-to-8bit (ORCPT ); Thu, 2 Jan 2020 10:27:24 -0500 Received: from lhrrgout.huawei.com ([185.176.76.210]:2221 "EHLO huawei.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1728561AbgABP1Y (ORCPT ); Thu, 2 Jan 2020 10:27:24 -0500 Received: from LHREML714-CAH.china.huawei.com (unknown [172.18.7.106]) by Forcepoint Email with ESMTP id B45E849FCBA97A570CA8; Thu, 2 Jan 2020 15:27:22 +0000 (GMT) Received: from lhreml710-chm.china.huawei.com (10.201.108.61) by LHREML714-CAH.china.huawei.com (10.201.108.37) with Microsoft SMTP Server (TLS) id 14.3.408.0; Thu, 2 Jan 2020 15:27:22 +0000 Received: from localhost (10.202.226.57) by lhreml710-chm.china.huawei.com (10.201.108.61) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256) id 15.1.1713.5; Thu, 2 Jan 2020 15:27:22 +0000 Date: Thu, 2 Jan 2020 15:27:19 +0000 From: Jonathan Cameron To: Brice Goglin CC: , , , , , Keith Busch , , "Rafael J . Wysocki" , , "Andrew Morton" , Dan Williams , Tao Xu , Lorenzo Pieralisi , Hanjun Guo , Sudeep Holla Subject: Re: [PATCH V6 0/7] ACPI: Support Generic Initiator proximity domains Message-ID: <20200102152604.000039f1@Huawei.com> In-Reply-To: <1867024e-b0c4-c291-7190-262cc4b297a8@gmail.com> References: <20191216153809.105463-1-Jonathan.Cameron@huawei.com> <20191218145041.00005a11@Huawei.com> <1867024e-b0c4-c291-7190-262cc4b297a8@gmail.com> Organization: Huawei Technologies Research and Development (UK) Ltd. X-Mailer: Claws Mail 3.17.4 (GTK+ 2.24.32; i686-w64-mingw32) MIME-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 8BIT X-Originating-IP: [10.202.226.57] X-ClientProxiedBy: lhreml709-chm.china.huawei.com (10.201.108.58) To lhreml710-chm.china.huawei.com (10.201.108.61) X-CFilter-Loop: Reflected Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, 20 Dec 2019 22:40:18 +0100 Brice Goglin wrote: > Le 18/12/2019 à 15:50, Jonathan Cameron a écrit : > > On Wed, 18 Dec 2019 12:32:06 +0100 > > Brice Goglin wrote: > > > >> Le 16/12/2019 à 16:38, Jonathan Cameron a écrit : > >>> Introduces a new type of NUMA node for cases where we want to represent > >>> the access characteristics of a non CPU initiator of memory requests, > >>> as these differ from all those for existing nodes containing CPUs and/or > >>> memory. > >>> > >>> These Generic Initiators are presented by the node access0 class in > >>> sysfs in the same way as a CPU. It seems likely that there will be > >>> usecases in which the best 'CPU' is desired and Generic Initiators > >>> should be ignored. The final few patches in this series introduced > >>> access1 which is a new performance class in the sysfs node description > >>> which presents only CPU to memory relationships. Test cases for this > >>> are described below. > >> > >> Hello Jonathan > >> > >> If I want to test this with a fake GI, what are the minimal set of > >> changes I should put in my ACPI tables? Can I just specify a dummy GI in > >> SRAT? What handle should I use there? > > Exactly that for a dummy GI. Also extend HMAT and SLIT for the extra > > proximity domain / initiator. > > > I couldn't get this to work (your patches on top of 5.5-rc2). I added > the GI in SRAT, and extended HMAT and SLIT accordingly. > > I don't know if that's expected but I get an additional node in sysfs, > with 0kB memory. > > However the HMAT table gets ignored because find_mem_target() fails in > hmat_parse_proximity_domain(). The target should have been allocated in > alloc_memory_target() which is called in srat_parse_mem_affinity(), but > it seems to me that this function isn't called for GI nodes. Or should > SRAT also contain a normal Memory node with same PM as the GI? > Hi Brice, Yes you should see a node with 0kB memory. Same as you get for a processor only node I believe. srat_parse_mem_affinity shouldn't call alloc_memory_target for the GI nodes as they don't have any memory. The hmat table should only refer to GI domains as initiators. Just to check, do you have them listed as a target node? Or perhaps in some hmat proximity entry as memory_PD? To answer your question, SRAT should not contain a normal memory node with the same PXM as that would defeat the whole purpose as we would have been able to have such a domain without Generic Initiators. Also, just to check, x86 or arm64? Thanks for testing this. Jonathan > Brice > >