Re: [RFC V2 02/12] mm: Isolate HugeTLB allocations away from CDM nodes

2017-02-01 Thread Dave Hansen
On 02/01/2017 05:59 AM, Anshuman Khandual wrote: > So shall we write all these details in the comment section for each > patch after the SOB statement to be more visible ? Or some where > in-code documentation as FIXME or XXX or something. These are little > large paragraphs, hence was wondering.

Re: [RFC V2 02/12] mm: Isolate HugeTLB allocations away from CDM nodes

2017-02-01 Thread Dave Hansen
On 02/01/2017 05:59 AM, Anshuman Khandual wrote: > So shall we write all these details in the comment section for each > patch after the SOB statement to be more visible ? Or some where > in-code documentation as FIXME or XXX or something. These are little > large paragraphs, hence was wondering.

Re: [RFC V2 02/12] mm: Isolate HugeTLB allocations away from CDM nodes

2017-02-01 Thread Anshuman Khandual
On 01/31/2017 07:07 AM, Dave Hansen wrote: > On 01/30/2017 05:03 PM, Anshuman Khandual wrote: >> On 01/30/2017 10:49 PM, Dave Hansen wrote: >>> On 01/29/2017 07:35 PM, Anshuman Khandual wrote: HugeTLB allocation/release/accounting currently spans across all the nodes under N_MEMORY node

Re: [RFC V2 02/12] mm: Isolate HugeTLB allocations away from CDM nodes

2017-02-01 Thread Anshuman Khandual
On 01/31/2017 07:07 AM, Dave Hansen wrote: > On 01/30/2017 05:03 PM, Anshuman Khandual wrote: >> On 01/30/2017 10:49 PM, Dave Hansen wrote: >>> On 01/29/2017 07:35 PM, Anshuman Khandual wrote: HugeTLB allocation/release/accounting currently spans across all the nodes under N_MEMORY node

Re: [RFC V2 02/12] mm: Isolate HugeTLB allocations away from CDM nodes

2017-01-30 Thread Dave Hansen
On 01/30/2017 05:03 PM, Anshuman Khandual wrote: > On 01/30/2017 10:49 PM, Dave Hansen wrote: >> On 01/29/2017 07:35 PM, Anshuman Khandual wrote: >>> HugeTLB allocation/release/accounting currently spans across all the nodes >>> under N_MEMORY node mask. Coherent memory nodes should not be part of

Re: [RFC V2 02/12] mm: Isolate HugeTLB allocations away from CDM nodes

2017-01-30 Thread Dave Hansen
On 01/30/2017 05:03 PM, Anshuman Khandual wrote: > On 01/30/2017 10:49 PM, Dave Hansen wrote: >> On 01/29/2017 07:35 PM, Anshuman Khandual wrote: >>> HugeTLB allocation/release/accounting currently spans across all the nodes >>> under N_MEMORY node mask. Coherent memory nodes should not be part of

Re: [RFC V2 02/12] mm: Isolate HugeTLB allocations away from CDM nodes

2017-01-30 Thread Anshuman Khandual
On 01/30/2017 10:49 PM, Dave Hansen wrote: > On 01/29/2017 07:35 PM, Anshuman Khandual wrote: >> HugeTLB allocation/release/accounting currently spans across all the nodes >> under N_MEMORY node mask. Coherent memory nodes should not be part of these >> allocations. So use system_ram() call to

Re: [RFC V2 02/12] mm: Isolate HugeTLB allocations away from CDM nodes

2017-01-30 Thread Anshuman Khandual
On 01/30/2017 10:49 PM, Dave Hansen wrote: > On 01/29/2017 07:35 PM, Anshuman Khandual wrote: >> HugeTLB allocation/release/accounting currently spans across all the nodes >> under N_MEMORY node mask. Coherent memory nodes should not be part of these >> allocations. So use system_ram() call to

Re: [RFC V2 02/12] mm: Isolate HugeTLB allocations away from CDM nodes

2017-01-30 Thread Dave Hansen
On 01/29/2017 07:35 PM, Anshuman Khandual wrote: > HugeTLB allocation/release/accounting currently spans across all the nodes > under N_MEMORY node mask. Coherent memory nodes should not be part of these > allocations. So use system_ram() call to fetch system RAM only nodes on the > platform which

Re: [RFC V2 02/12] mm: Isolate HugeTLB allocations away from CDM nodes

2017-01-30 Thread Dave Hansen
On 01/29/2017 07:35 PM, Anshuman Khandual wrote: > HugeTLB allocation/release/accounting currently spans across all the nodes > under N_MEMORY node mask. Coherent memory nodes should not be part of these > allocations. So use system_ram() call to fetch system RAM only nodes on the > platform which

[RFC V2 02/12] mm: Isolate HugeTLB allocations away from CDM nodes

2017-01-29 Thread Anshuman Khandual
HugeTLB allocation/release/accounting currently spans across all the nodes under N_MEMORY node mask. Coherent memory nodes should not be part of these allocations. So use system_ram() call to fetch system RAM only nodes on the platform which can then be used for HugeTLB allocation purpose instead

[RFC V2 02/12] mm: Isolate HugeTLB allocations away from CDM nodes

2017-01-29 Thread Anshuman Khandual
HugeTLB allocation/release/accounting currently spans across all the nodes under N_MEMORY node mask. Coherent memory nodes should not be part of these allocations. So use system_ram() call to fetch system RAM only nodes on the platform which can then be used for HugeTLB allocation purpose instead