From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752155AbdCNNda (ORCPT ); Tue, 14 Mar 2017 09:33:30 -0400 Received: from mx0a-001b2d01.pphosted.com ([148.163.156.1]:43853 "EHLO mx0a-001b2d01.pphosted.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751049AbdCNNd1 (ORCPT ); Tue, 14 Mar 2017 09:33:27 -0400 Subject: Re: [PATCH 1/2] mm: Change generic FALLBACK zonelist creation process To: John Hubbard , Anshuman Khandual , linux-kernel@vger.kernel.org, linux-mm@kvack.org References: <1d67f38b-548f-26a2-23f5-240d6747f286@linux.vnet.ibm.com> <20170308092146.5264-1-khandual@linux.vnet.ibm.com> <0f787fb7-e299-9afb-8c87-4afdb937fdbb@nvidia.com> Cc: mhocko@suse.com, vbabka@suse.cz, mgorman@suse.de, minchan@kernel.org, aneesh.kumar@linux.vnet.ibm.com, bsingharora@gmail.com, srikar@linux.vnet.ibm.com, haren@linux.vnet.ibm.com, jglisse@redhat.com, dave.hansen@intel.com, dan.j.williams@intel.com, zi.yan@cs.rutgers.edu From: Anshuman Khandual Date: Tue, 14 Mar 2017 19:03:04 +0530 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.5.1 MIME-Version: 1.0 In-Reply-To: <0f787fb7-e299-9afb-8c87-4afdb937fdbb@nvidia.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit X-TM-AS-MML: disable x-cbid: 17031413-0024-0000-0000-000003B2C325 X-IBM-AV-DETECTION: SAVI=unused REMOTE=unused XFE=unused x-cbparentid: 17031413-0025-0000-0000-00001130445D Message-Id: <13c1a501-0ab9-898c-f749-efecca787661@linux.vnet.ibm.com> X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:,, definitions=2017-03-14_08:,, signatures=0 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 spamscore=0 suspectscore=0 malwarescore=0 phishscore=0 adultscore=0 bulkscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.0.1-1702020001 definitions=main-1703140108 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 03/08/2017 04:37 PM, John Hubbard wrote: > On 03/08/2017 01:21 AM, Anshuman Khandual wrote: >> Kernel allocation to CDM node has already been prevented by putting it's >> entire memory in ZONE_MOVABLE. But the CDM nodes must also be isolated >> from implicit allocations happening on the system. >> >> Any isolation seeking CDM node requires isolation from implicit memory >> allocations from user space but at the same time there should also have >> an explicit way to do the memory allocation. >> >> Platform node's both zonelists are fundamental to where the memory comes >> from when there is an allocation request. In order to achieve these two >> objectives as stated above, zonelists building process has to change as >> both zonelists (i.e FALLBACK and NOFALLBACK) gives access to the node's >> memory zones during any kind of memory allocation. The following changes >> are implemented in this regard. >> >> * CDM node's zones are not part of any other node's FALLBACK zonelist >> * CDM node's FALLBACK list contains it's own memory zones followed by >> all system RAM zones in regular order as before > > There was a discussion, on an earlier version of this patchset, in which > someone pointed out that a slight over-allocation on a device that has > much more memory than the CPU has, could use up system memory. Your > latest approach here does not address this. Hmm, I dont remember this. Could you please be more specific and point me to the discussion on this. > > I'm thinking that, until oversubscription between NUMA nodes is more > fully implemented in a way that can be properly controlled, you'd I did not get you. What does over subscription mean in this context ? FALLBACK zonelist on each node has memory from every node including it's own. Hence the allocation request targeted towards any node is symmetrical with respect to from where the memory will be allocated. > probably better just not fallback to system memory. In other words, a > CDM node really is *isolated* from other nodes--no automatic use in > either direction. That is debatable. With this proposed solution the CDM FALLBACK zonelist contains system RAM zones as fallback option which will be used in case CDM memory is depleted. IMHO, I think thats the right thing to do as it still maintains the symmetry to some extent. > > Also, naming and purpose: maybe this is a "Limited NUMA Node", rather > than a Coherent Device Memory node. Because: the real point of this > thing is to limit the normal operation of NUMA, just enough to work with > what I am *told* is memory-that-is-too-fragile-for-kernel-use (I remain > soemwhat on the fence, there, even though you did talk me into it > earlier, heh). :) Naming can be debated later after we all agree on the proposal in principle. We have already discussed about kernel memory on CDM in detail. > > On process: it would probably help if you gathered up previous > discussion points and carefully, concisely addressed each one, > somewhere, (maybe in a cover letter). Because otherwise, it's too easy > for earlier, important problems to be forgotten. And reviewers don't > want to have to repeat themselves, of course. Will do.