From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pg1-f198.google.com (mail-pg1-f198.google.com [209.85.215.198]) by kanga.kvack.org (Postfix) with ESMTP id 41C7C8E0002 for ; Thu, 17 Jan 2019 10:17:28 -0500 (EST) Received: by mail-pg1-f198.google.com with SMTP id p4so6304198pgj.21 for ; Thu, 17 Jan 2019 07:17:28 -0800 (PST) Received: from mga12.intel.com (mga12.intel.com. [192.55.52.136]) by mx.google.com with ESMTPS id j132si1960312pfc.84.2019.01.17.07.17.27 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Thu, 17 Jan 2019 07:17:27 -0800 (PST) Subject: Re: [PATCH 4/4] dax: "Hotplug" persistent memory for use like normal RAM References: <20190116181859.D1504459@viggo.jf.intel.com> <20190116181905.12E102B4@viggo.jf.intel.com> <5ef5d5e9-9d35-fb84-b69e-7456dcf4c241@linux.intel.com> From: Dave Hansen Message-ID: <1e9377c6-11a0-3bbd-763d-d9347bd556cf@intel.com> Date: Thu, 17 Jan 2019 07:17:24 -0800 MIME-Version: 1.0 In-Reply-To: <5ef5d5e9-9d35-fb84-b69e-7456dcf4c241@linux.intel.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 8bit Sender: owner-linux-mm@kvack.org List-ID: To: Yanmin Zhang , Dave Hansen , dave@sr71.net Cc: dan.j.williams@intel.com, dave.jiang@intel.com, zwisler@kernel.org, vishal.l.verma@intel.com, thomas.lendacky@amd.com, akpm@linux-foundation.org, mhocko@suse.com, linux-nvdimm@lists.01.org, linux-kernel@vger.kernel.org, linux-mm@kvack.org, ying.huang@intel.com, fengguang.wu@intel.com, bp@suse.de, bhelgaas@google.com, baiyaowei@cmss.chinamobile.com, tiwai@suse.de On 1/17/19 12:19 AM, Yanmin Zhang wrote: >> > I didn't try pmem and I am wondering it's slower than DRAM. > Should a flag, such like _GFP_PMEM, be added to distinguish it from > DRAM? Absolutely not. :) We already have performance-differentiated memory, and lots of ways to enumerate and select it in the kernel (all of our NUMA infrastructure). PMEM is also just the first of many "kinds" of memory that folks want to build in systems and use a "RAM". We literally don't have space to put a flag in for each type.