Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752876AbdGGFbJ (ORCPT ); Fri, 7 Jul 2017 01:31:09 -0400 Received: from hqemgate16.nvidia.com ([216.228.121.65]:13447 "EHLO hqemgate16.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752279AbdGGFbH (ORCPT ); Fri, 7 Jul 2017 01:31:07 -0400 X-PGP-Universal: processed; by hqpgpgate101.nvidia.com on Thu, 06 Jul 2017 22:30:47 -0700 Subject: Re: [RFC v2 0/5] surface heterogeneous memory performance information To: Ross Zwisler , CC: "Anaczkowski, Lukasz" , "Box, David E" , "Kogut, Jaroslaw" , "Lahtinen, Joonas" , "Moore, Robert" , "Nachimuthu, Murugasamy" , "Odzioba, Lukasz" , "Rafael J. Wysocki" , "Rafael J. Wysocki" , "Schmauss, Erik" , "Verma, Vishal L" , "Zheng, Lv" , Andrew Morton , Dan Williams , Dave Hansen , Greg Kroah-Hartman , Jerome Glisse , Len Brown , Tim Chen , , , , References: <20170706215233.11329-1-ross.zwisler@linux.intel.com> X-Nvconfidentiality: public From: John Hubbard Message-ID: <7cb3b9c4-9082-97e9-ebfd-542243bf652b@nvidia.com> Date: Thu, 6 Jul 2017 22:30:46 -0700 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 MIME-Version: 1.0 In-Reply-To: <20170706215233.11329-1-ross.zwisler@linux.intel.com> X-Originating-IP: [172.17.160.221] X-ClientProxiedBy: HQMAIL102.nvidia.com (172.18.146.10) To HQMAIL107.nvidia.com (172.20.187.13) Content-Type: text/plain; charset="utf-8" Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1159 Lines: 30 On 07/06/2017 02:52 PM, Ross Zwisler wrote: [...] > > The naming collision between Jerome's "Heterogeneous Memory Management > (HMM)" and this "Heterogeneous Memory (HMEM)" series is unfortunate, but I > was trying to stick with the word "Heterogeneous" because of the naming of > the ACPI 6.2 Heterogeneous Memory Attribute Table table. Suggestions for > better naming are welcome. > Hi Ross, Say, most of the places (file names, function and variable names, and even print statements) where this patchset uses hmem or HMEM, it really seems to mean, the Heterogeneous Memory Attribute Table. That's not *always* true, but given that it's a pretty severe naming conflict, how about just changing: hmem --> hmat HMEM --> HMAT ...everywhere? Then you still have Heterogeneous Memory in the name, but there is enough lexical distance (is that a thing? haha) between HMM and HMAT to keep us all sane. :) With or without the above suggestion, there are a few places (Kconfig, comments, prints) where we can more easily make it clear that HMM != HMEM (or HMAT), so for those I can just comment on them separately in the individual patches. thanks, john h