<div dir="ltr"><div dir="ltr"><div></div></div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Am Do., 11. Jan. 2024 um 18:13 Uhr schrieb Simon Ser <<a href="mailto:contact@emersion.fr">contact@emersion.fr</a>>:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Are we sure that all drivers handle these two props properly with async<br>
page-flips? This is a new codepath not taken by the legacy uAPI.<br></blockquote><div>I've only tested on amdgpu so far. Afacs the other drivers that would need</div><div>testing / that support atomic and async pageflips are</div><div>- i915<div></div><div>- noueveau (though atomic is disabled by default, so maybe it doesn't matter?)<br></div></div><div>- vc4</div><div>- atmel-hlcdc</div><div><br></div><div>The first two I can test, the latter I don't have the hardware for. I don't know if I can</div><div>extensively test fb_damage_clips either / how I'd even know if it's being applied</div><div>correctly, but in the worst case I'd expect the driver to not do the optimizations the</div><div>property allows.<br></div><div><br></div><div>As an alternative to this, would it be okay to expose a driver hook for optional</div><div>driver-specific checks that drm_atomic_set_property can delegate to, and only</div><div>allow this with the properties and hardware that's been tested? Then more</div><div>properties (like cursor position changes on amdgpu) could be easily added later</div><div>on too.</div></div></div>