Re: [PATCH] drm: Require __GFP_NOFAIL for the legacy drm_modeset_lock_all

2017-11-02 Thread Daniel Vetter
On Wed, Nov 01, 2017 at 02:08:37PM +, Chris Wilson wrote: > Quoting Daniel Vetter (2017-10-31 16:38:26) > > On Tue, Oct 31, 2017 at 03:28:01PM +0200, Ville Syrjälä wrote: > > > On Tue, Oct 31, 2017 at 11:55:35AM +, Chris Wilson wrote: > > > > To acquire all modeset locks requires a ww_ctx

Re: [PATCH] drm: Require __GFP_NOFAIL for the legacy drm_modeset_lock_all

2017-11-01 Thread Chris Wilson
Quoting Daniel Vetter (2017-10-31 16:38:26) > On Tue, Oct 31, 2017 at 03:28:01PM +0200, Ville Syrjälä wrote: > > On Tue, Oct 31, 2017 at 11:55:35AM +, Chris Wilson wrote: > > > To acquire all modeset locks requires a ww_ctx to be allocated. As this > > > is the legacy path and the allocation

Re: [PATCH] drm: Require __GFP_NOFAIL for the legacy drm_modeset_lock_all

2017-10-31 Thread Daniel Vetter
On Tue, Oct 31, 2017 at 03:28:01PM +0200, Ville Syrjälä wrote: > On Tue, Oct 31, 2017 at 11:55:35AM +, Chris Wilson wrote: > > To acquire all modeset locks requires a ww_ctx to be allocated. As this > > is the legacy path and the allocation small, to reduce the changes > > required (and

Re: [PATCH] drm: Require __GFP_NOFAIL for the legacy drm_modeset_lock_all

2017-10-31 Thread Ville Syrjälä
On Tue, Oct 31, 2017 at 11:55:35AM +, Chris Wilson wrote: > To acquire all modeset locks requires a ww_ctx to be allocated. As this > is the legacy path and the allocation small, to reduce the changes > required (and complex untested error handling) to the legacy drivers, we > simply assume

Re: [PATCH] drm: Require __GFP_NOFAIL for the legacy drm_modeset_lock_all

2017-10-31 Thread Chris Wilson
Quoting Chris Wilson (2017-10-31 11:55:35) > To acquire all modeset locks requires a ww_ctx to be allocated. As this > is the legacy path and the allocation small, to reduce the changes > required (and complex untested error handling) to the legacy drivers, we > simply assume that the allocation

[PATCH] drm: Require __GFP_NOFAIL for the legacy drm_modeset_lock_all

2017-10-31 Thread Chris Wilson
To acquire all modeset locks requires a ww_ctx to be allocated. As this is the legacy path and the allocation small, to reduce the changes required (and complex untested error handling) to the legacy drivers, we simply assume that the allocation succeeds. At present, it relies on the