Re: [Intel-gfx] [PATCH v2] drm/i915: Remove __GFP_NORETRY from our buffer allocator

2017-06-08 Thread Chris Wilson
Quoting Michal Hocko (2017-06-08 13:26:22) > Do you plan to merge this patch? I would like to post my > __GFP_RETRY_MAYFAIL series sometime soon and would like to cover this > one as well. Yes, just a strict policy on getting some sucker to review it first. -Chris _

Re: [Intel-gfx] [PATCH v2] drm/i915: Remove __GFP_NORETRY from our buffer allocator

2017-06-08 Thread Michal Hocko
Do you plan to merge this patch? I would like to post my __GFP_RETRY_MAYFAIL series sometime soon and would like to cover this one as well. On Mon 05-06-17 11:35:12, Chris Wilson wrote: > I tried __GFP_NORETRY in the belief that __GFP_RECLAIM was effective. It > struggles with handling reclaim via

Re: [Intel-gfx] [PATCH v2] drm/i915: Remove __GFP_NORETRY from our buffer allocator

2017-06-06 Thread Michal Hocko
On Mon 05-06-17 16:04:55, Chris Wilson wrote: > Quoting Michal Hocko (2017-06-05 14:08:10) > > On Mon 05-06-17 13:49:38, Chris Wilson wrote: > > > Quoting Michal Hocko (2017-06-05 13:26:30) > > > > On Mon 05-06-17 11:35:12, Chris Wilson wrote: > > > > > I tried __GFP_NORETRY in the belief that __GF

Re: [Intel-gfx] [PATCH v2] drm/i915: Remove __GFP_NORETRY from our buffer allocator

2017-06-06 Thread Michal Hocko
On Mon 05-06-17 11:35:12, Chris Wilson wrote: > I tried __GFP_NORETRY in the belief that __GFP_RECLAIM was effective. It > struggles with handling reclaim via kswapd (through inconsistency within > throttle_direct_reclaim() and even then the race between multiple > allocators makes the two step of

Re: [Intel-gfx] [PATCH v2] drm/i915: Remove __GFP_NORETRY from our buffer allocator

2017-06-06 Thread Michal Hocko
On Mon 05-06-17 14:39:17, Chris Wilson wrote: > Quoting Michal Hocko (2017-06-05 14:08:10) > > On Mon 05-06-17 13:49:38, Chris Wilson wrote: > > > Quoting Michal Hocko (2017-06-05 13:26:30) > > > > On Mon 05-06-17 11:35:12, Chris Wilson wrote: > > > > > I tried __GFP_NORETRY in the belief that __GF

Re: [Intel-gfx] [PATCH v2] drm/i915: Remove __GFP_NORETRY from our buffer allocator

2017-06-06 Thread Michal Hocko
On Mon 05-06-17 13:49:38, Chris Wilson wrote: > Quoting Michal Hocko (2017-06-05 13:26:30) > > On Mon 05-06-17 11:35:12, Chris Wilson wrote: > > > I tried __GFP_NORETRY in the belief that __GFP_RECLAIM was effective. It > > > struggles with handling reclaim via kswapd (through inconsistency within

Re: [Intel-gfx] [PATCH v2] drm/i915: Remove __GFP_NORETRY from our buffer allocator

2017-06-05 Thread Chris Wilson
Quoting Michal Hocko (2017-06-05 14:08:10) > On Mon 05-06-17 13:49:38, Chris Wilson wrote: > > Quoting Michal Hocko (2017-06-05 13:26:30) > > > On Mon 05-06-17 11:35:12, Chris Wilson wrote: > > > > I tried __GFP_NORETRY in the belief that __GFP_RECLAIM was effective. It > > > > struggles with handl

Re: [Intel-gfx] [PATCH v2] drm/i915: Remove __GFP_NORETRY from our buffer allocator

2017-06-05 Thread Chris Wilson
Quoting Michal Hocko (2017-06-05 14:08:10) > On Mon 05-06-17 13:49:38, Chris Wilson wrote: > > Quoting Michal Hocko (2017-06-05 13:26:30) > > > On Mon 05-06-17 11:35:12, Chris Wilson wrote: > > > > I tried __GFP_NORETRY in the belief that __GFP_RECLAIM was effective. It > > > > struggles with handl

Re: [Intel-gfx] [PATCH v2] drm/i915: Remove __GFP_NORETRY from our buffer allocator

2017-06-05 Thread Chris Wilson
Quoting Michal Hocko (2017-06-05 13:26:30) > On Mon 05-06-17 11:35:12, Chris Wilson wrote: > > I tried __GFP_NORETRY in the belief that __GFP_RECLAIM was effective. It > > struggles with handling reclaim via kswapd (through inconsistency within > > throttle_direct_reclaim() and even then the race b

Re: [Intel-gfx] [PATCH v2] drm/i915: Remove __GFP_NORETRY from our buffer allocator

2017-06-05 Thread Chris Wilson
Quoting Daniel Stone (2017-06-05 11:47:44) > Hi, > > On 5 June 2017 at 11:35, Chris Wilson wrote: > > I tried __GFP_NORETRY in the belief that __GFP_RECLAIM was effective. It > > struggles with handling reclaim via kswapd (through inconsistency within > > throttle_direct_reclaim() and even then t

Re: [Intel-gfx] [PATCH v2] drm/i915: Remove __GFP_NORETRY from our buffer allocator

2017-06-05 Thread Daniel Stone
Hi, On 5 June 2017 at 11:35, Chris Wilson wrote: > I tried __GFP_NORETRY in the belief that __GFP_RECLAIM was effective. It > struggles with handling reclaim via kswapd (through inconsistency within > throttle_direct_reclaim() and even then the race between multiple > allocators makes the two ste

[Intel-gfx] [PATCH v2] drm/i915: Remove __GFP_NORETRY from our buffer allocator

2017-06-05 Thread Chris Wilson
I tried __GFP_NORETRY in the belief that __GFP_RECLAIM was effective. It struggles with handling reclaim via kswapd (through inconsistency within throttle_direct_reclaim() and even then the race between multiple allocators makes the two step of reclaim then allocate fragile), and as our buffers are