[Intel-gfx] [PATCH v2 00/12] drm/atomic: Make drm_atomic_helper_swap_state waiting interruptible.

Maarten Lankhorst maarten.lankhorst at linux.intel.com
Wed Jul 19 10:10:15 UTC 2017


Op 14-07-17 om 17:01 schreef Sean Paul:
> On Tue, Jul 11, 2017 at 04:33:02PM +0200, Maarten Lankhorst wrote:
>> drm_atomic_helper_swap_state could previously never fail, in order to still
>> continue it would set a limit of 10s on waiting for previous updates to complete,
>> then it moved forward. This can be very bad when ignoring previous updates,
>> because the hw state and sw state may get out of sync when for example a modeset
>> is ignored.
>>
>> By converting the swap_state to interruptible and handling the error in each driver,
>> we fix this issue because if a update takes forever it can be aborted through signals.
>>
>> Changes since first version:
>> - Split out driver conversions.
>> - Fix nouveau error handling first.
>>
>> Maarten Lankhorst (12):
>>   drm/nouveau: Fix error handling in nv50_disp_atomic_commit
>>   drm/atomic: Change drm_atomic_helper_swap_state to return an error.
>>   drm/nouveau: Handle drm_atomic_helper_swap_state failure
>>   drm/atmel-hlcdc: Handle drm_atomic_helper_swap_state failure
>>   drm/i915: Handle drm_atomic_helper_swap_state failure
>>   drm/mediatek: Handle drm_atomic_helper_swap_state failure
>>   drm/msm: Handle drm_atomic_helper_swap_state failure
>>   drm/tegra: Handle drm_atomic_helper_swap_state failure
>>   drm/tilcdc: Handle drm_atomic_helper_swap_state failure
>>   drm/vc4: Handle drm_atomic_helper_swap_state failure
>>   drm/atomic: Add __must_check to drm_atomic_helper_swap_state.
>>   drm/atomic: Allow drm_atomic_helper_swap_state to fail
> Hi Maarten,
> The set looks good to me. The core changes make sense and seem like a good step
> forward, and the driver changes seem like they do the right thing.
>
> Normally, I'd suggest waiting a week or so for maintainer acks to trickle in,
> but since this is fixing behavior in i915, it makes sense to expedite it.
> Perhaps ping individuals on IRC?
>
> For the whole set,
> Reviewed-by: Sean Paul <seanpaul at chromium.org>
Series applied, thanks for review. :)


More information about the dri-devel mailing list