Re: [PATCH v3 5/7] mm/compaction: respect compaction order when updating defer counter

2015-12-07 Thread Joonsoo Kim
On Fri, Dec 04, 2015 at 06:15:02PM +0100, Vlastimil Babka wrote: > On 12/03/2015 08:11 AM, Joonsoo Kim wrote: > >It doesn't make sense that we reset defer counter > >in compaction_defer_reset() when compaction request under the order of > >compact_order_failed succeed. Fix it. > > Right. > >

Re: [PATCH v3 5/7] mm/compaction: respect compaction order when updating defer counter

2015-12-07 Thread Joonsoo Kim
On Fri, Dec 04, 2015 at 06:15:02PM +0100, Vlastimil Babka wrote: > On 12/03/2015 08:11 AM, Joonsoo Kim wrote: > >It doesn't make sense that we reset defer counter > >in compaction_defer_reset() when compaction request under the order of > >compact_order_failed succeed. Fix it. > > Right. > >

Re: [PATCH v3 5/7] mm/compaction: respect compaction order when updating defer counter

2015-12-04 Thread Vlastimil Babka
On 12/03/2015 08:11 AM, Joonsoo Kim wrote: It doesn't make sense that we reset defer counter in compaction_defer_reset() when compaction request under the order of compact_order_failed succeed. Fix it. Right. And, it does make sense that giving enough chance for updated failed order

Re: [PATCH v3 5/7] mm/compaction: respect compaction order when updating defer counter

2015-12-04 Thread Vlastimil Babka
On 12/03/2015 08:11 AM, Joonsoo Kim wrote: It doesn't make sense that we reset defer counter in compaction_defer_reset() when compaction request under the order of compact_order_failed succeed. Fix it. Right. And, it does make sense that giving enough chance for updated failed order

[PATCH v3 5/7] mm/compaction: respect compaction order when updating defer counter

2015-12-02 Thread Joonsoo Kim
It doesn't make sense that we reset defer counter in compaction_defer_reset() when compaction request under the order of compact_order_failed succeed. Fix it. And, it does make sense that giving enough chance for updated failed order compaction before deferring. Change it. Signed-off-by: Joonsoo

[PATCH v3 5/7] mm/compaction: respect compaction order when updating defer counter

2015-12-02 Thread Joonsoo Kim
It doesn't make sense that we reset defer counter in compaction_defer_reset() when compaction request under the order of compact_order_failed succeed. Fix it. And, it does make sense that giving enough chance for updated failed order compaction before deferring. Change it. Signed-off-by: Joonsoo