Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752065AbdIEQS6 (ORCPT ); Tue, 5 Sep 2017 12:18:58 -0400 Received: from mail-oi0-f41.google.com ([209.85.218.41]:35455 "EHLO mail-oi0-f41.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751382AbdIEQS4 (ORCPT ); Tue, 5 Sep 2017 12:18:56 -0400 X-Google-Smtp-Source: ADKCNb4HF10wvkiRZQCfOukz9sQS/9R9Nkcc05DXAHA4FZ8U5uEnTJId4EMxlRpHxJS05DLMZb1n9ef+g8y4WkgI3pU= MIME-Version: 1.0 In-Reply-To: <20170905135017.GA19397@redhat.com> References: <20170817000548.32038-1-jglisse@redhat.com> <20170817000548.32038-20-jglisse@redhat.com> <20170904155123.GA3161@redhat.com> <7026dfda-9fd0-2661-5efc-66063dfdf6bc@huawei.com> <20170905023826.GA4836@redhat.com> <20170905135017.GA19397@redhat.com> From: Dan Williams Date: Tue, 5 Sep 2017 09:18:55 -0700 Message-ID: Subject: Re: [HMM-v25 19/19] mm/hmm: add new helper to hotplug CDM memory region v3 To: Jerome Glisse Cc: Bob Liu , Andrew Morton , "linux-kernel@vger.kernel.org" , Linux MM , John Hubbard , David Nellans , Balbir Singh , majiuyue , "xieyisheng (A)" , Ross Zwisler , Mel Gorman , Rik van Riel , Michal Hocko Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from quoted-printable to 8bit by nfs id v85GJ6eQ013845 Content-Length: 5636 Lines: 112 On Tue, Sep 5, 2017 at 6:50 AM, Jerome Glisse wrote: > On Tue, Sep 05, 2017 at 11:50:57AM +0800, Bob Liu wrote: >> On 2017/9/5 10:38, Jerome Glisse wrote: >> > On Tue, Sep 05, 2017 at 09:13:24AM +0800, Bob Liu wrote: >> >> On 2017/9/4 23:51, Jerome Glisse wrote: >> >>> On Mon, Sep 04, 2017 at 11:09:14AM +0800, Bob Liu wrote: >> >>>> On 2017/8/17 8:05, Jérôme Glisse wrote: >> >>>>> Unlike unaddressable memory, coherent device memory has a real >> >>>>> resource associated with it on the system (as CPU can address >> >>>>> it). Add a new helper to hotplug such memory within the HMM >> >>>>> framework. >> >>>>> >> >>>> >> >>>> Got an new question, coherent device( e.g CCIX) memory are likely reported to OS >> >>>> through ACPI and recognized as NUMA memory node. >> >>>> Then how can their memory be captured and managed by HMM framework? >> >>>> >> >>> >> >>> Only platform that has such memory today is powerpc and it is not reported >> >>> as regular memory by the firmware hence why they need this helper. >> >>> >> >>> I don't think anyone has defined anything yet for x86 and acpi. As this is >> >> >> >> Not yet, but now the ACPI spec has Heterogeneous Memory Attribute >> >> Table (HMAT) table defined in ACPI 6.2. >> >> The HMAT can cover CPU-addressable memory types(though not non-cache >> >> coherent on-device memory). >> >> >> >> Ross from Intel already done some work on this, see: >> >> https://lwn.net/Articles/724562/ >> >> >> >> arm64 supports APCI also, there is likely more this kind of device when CCIX >> >> is out (should be very soon if on schedule). >> > >> > HMAT is not for the same thing, AFAIK HMAT is for deep "hierarchy" memory ie >> > when you have several kind of memory each with different characteristics: >> > - HBM very fast (latency) and high bandwidth, non persistent, somewhat >> > small (ie few giga bytes) >> > - Persistent memory, slower (both latency and bandwidth) big (tera bytes) >> > - DDR (good old memory) well characteristics are between HBM and persistent >> > >> >> Okay, then how the kernel handle the situation of "kind of memory each with different characteristics"? >> Does someone have any suggestion? I thought HMM can do this. >> Numa policy/node distance is good but perhaps require a few extending, e.g a HBM node can't be >> swap, can't accept DDR fallback allocation. > > I don't think there is any consensus for this. I put forward the idea that NUMA > needed to be extended as with deep hierarchy it is not only the distance between > two nodes but also others factors like persistency, bandwidth, latency ... > > >> > So AFAICT this has nothing to do with what HMM is for, ie device memory. Note >> > that device memory can have a hierarchy of memory themself (HBM, GDDR and in >> > maybe even persistent memory). >> > >> >> This looks like a subset of HMAT when CPU can address device memory directly in cache-coherent way. > > It is not, it is much more complex than that. Linux kernel has no idea on what is > going on a device and thus do not have any usefull informations to make proper > decission regarding device memory. Here device is real device ie something with > processing capability, not something like HBM or persistent memory even if the > latter is associated with a struct device inside linux kernel. > >> >> >> >>> memory on PCIE like interface then i don't expect it to be reported as NUMA >> >>> memory node but as io range like any regular PCIE resources. Device driver >> >>> through capabilities flags would then figure out if the link between the >> >>> device and CPU is CCIX capable if so it can use this helper to hotplug it >> >>> as device memory. >> >>> >> >> >> >> From my point of view, Cache coherent device memory will popular soon and >> >> reported through ACPI/UEFI. Extending NUMA policy still sounds more reasonable >> >> to me. >> > >> > Cache coherent device will be reported through standard mecanisms defined by >> > the bus standard they are using. To my knowledge all the standard are either >> > on top of PCIE or are similar to PCIE. >> > >> > It is true that on many platform PCIE resource is manage/initialize by the >> > bios (UEFI) but it is platform specific. In some case we reprogram what the >> > bios pick. >> > >> > So like i was saying i don't expect the BIOS/UEFI to report device memory as >> >> But it's happening. >> In my understanding, that's why HMAT was introduced. >> For reporting device memory as regular memory(with different characteristics). > > That is not my understanding but only Intel can confirm. HMAT was introduced > for things like HBM or persistent memory. Which i do not consider as device > memory. Sure persistent memory is assign a device struct because it is easier > for integration with the block system i assume. But it does not make it a > device in my view. For me a device is a piece of hardware that has some > processing capabilities (network adapter, sound card, GPU, ...) > > But we can argue about semantic and what a device is. For all intent and purposes > device in HMM context is some piece of hardware with processing capabilities and > local device memory. I would say that device memory at its base-level is a memory range whose availability is dependent on a device-driver. HMM layers some additional functionality on top, but ZONE_DEVICE should only be seen as the device-driver controlled lifetime and not conflated with the incremental HMM functionality. HMAT simply allows you to associate a memory range with a numa-node / proximity-domain number that represents a set of performance / feature characteristics.