Re: [igt-dev] [PATCH i-g-t v9 3/3] igt/tests/kms_atomic_transition: Tolerate if can't have all planes

[Maarten Lankhorst <maarten.lankhorst@linux.intel.com>]

Op 10-04-2019 om 17:06 schreef Maarten Lankhorst:
> Op 10-04-2019 om 13:32 schreef Stanislav Lisovskiy:
>> With some upcoming changes i915 might not allow
>> all sprite planes enabled, depending on available
>> bandwidth limitation. Thus the test need to decrement
>> amount of planes and try again, instead of panicking.
>>
>> v2: Removed excessive nested conditions, making code a bit
>>     more readable(hopefully).
>>
>> v3: Stopped using global n_planes variable as it might cause
>>     resources being unreleased.
>>     Using now parms[i].mask as a way to determine if plane
>>     has to be included into commit.
>>
>> v4: Removed unneeded n_overlays initialization.
>>
>> v5: Randomize which of sprite planes to remove if hitting
>>     resource limits.
>>
>> v6: Replace igt_warn with igt_info, to make IGT tests happier.
>>
>> v7: Removed unneeded retry logic, made plane random removal simplier.
>>     Great thanks to Maarten Lankhorst for suggestions.
>>
>> v8: Fail if we have less than 3 planes
>>
>> v9: Remove unneeded diffs
>>
>> Signed-off-by: Stanislav Lisovskiy <stanislav.lisovskiy@intel.com>
>> ---
>>  tests/kms_atomic_transition.c | 78 +++++++++++++++++++++--------------
>>  1 file changed, 47 insertions(+), 31 deletions(-)
> Assuming CI is happy:
>
> Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
>
Thanks, pushed. :)


_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev