Re: [PATCH v5 2/6] mm/cma: introduce new zone, ZONE_CMA

2016-09-21 Thread Vlastimil Babka
On 08/29/2016 07:07 AM, js1...@gmail.com wrote: From: Joonsoo Kim Attached cover-letter: This series try to solve problems of current CMA implementation. CMA is introduced to provide physically contiguous pages at runtime without exclusive reserved memory area. But, current implementation wor

Re: [PATCH v5 2/6] mm/cma: introduce new zone, ZONE_CMA

2016-08-31 Thread Joonsoo Kim
On Tue, Aug 30, 2016 at 06:10:46PM +0530, Aneesh Kumar K.V wrote: > "Aneesh Kumar K.V" writes: > > > > > > >> static inline void check_highest_zone(enum zone_type k) > >> { > >> - if (k > policy_zone && k != ZONE_MOVABLE) > >> + if (k > policy_zone && k != ZONE_MOVABLE && !is_zone_cma_id

Re: [PATCH v5 2/6] mm/cma: introduce new zone, ZONE_CMA

2016-08-30 Thread Aneesh Kumar K.V
"Aneesh Kumar K.V" writes: > > >> static inline void check_highest_zone(enum zone_type k) >> { >> -if (k > policy_zone && k != ZONE_MOVABLE) >> +if (k > policy_zone && k != ZONE_MOVABLE && !is_zone_cma_idx(k)) >> policy_zone = k; >> } >> > > > Should we apply policy t

Re: [PATCH v5 2/6] mm/cma: introduce new zone, ZONE_CMA

2016-08-30 Thread Aneesh Kumar K.V
> static inline void check_highest_zone(enum zone_type k) > { > - if (k > policy_zone && k != ZONE_MOVABLE) > + if (k > policy_zone && k != ZONE_MOVABLE && !is_zone_cma_idx(k)) > policy_zone = k; > } > Should we apply policy to allocation from ZONE CMA ?. CMA reser

[PATCH v5 2/6] mm/cma: introduce new zone, ZONE_CMA

2016-08-28 Thread js1304
From: Joonsoo Kim Attached cover-letter: This series try to solve problems of current CMA implementation. CMA is introduced to provide physically contiguous pages at runtime without exclusive reserved memory area. But, current implementation works like as previous reserved memory approach, beca