<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Apr 20, 2020 at 9:45 PM Michel Dänzer <<a href="mailto:michel@daenzer.net">michel@daenzer.net</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 2020-04-20 6:04 p.m., uday kiran pichika wrote:<br>
> <br>
> Even in amdgpu_present_flip(), there is a check<br>
> for amdgpu_window_has_variable_refresh() which actually checks whether<br>
> window has a variable_refresh property set from the MESA or not ? this<br>
> check is failing in my case and never calls amdgpu_present_set_screen_vrr.<br>
<br>
This should be set by<br>
<br>
        get_window_priv(window)->variable_refresh = variable_refresh;<br>
<br>
in amdgpu_vrr_property_update.<br></blockquote><div><br></div><div>I amdgpu_vrr_property_update method gets called from amdgpu_change_property when the property is being added to the window. though variable_refresh property is updating from the below call, this window is not same as the one(info->flip_window) in amdgpu_present_flip.</div><div><b>get_window_priv(window)</b>->variable_refresh = variable_refresh; </div><div><br></div><div>Could you please let me know you IRC ID to have a chat in that for more information. </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
<br>
> Is there any way that i can check whether MESA is setting this property ?<br>
> <br>
> To my understanding, MESA will set this property to true or false based on<br>
> whether application is running in Full screen mode or in normal mode.<br>
<br>
Mesa always sets the window property by default, unless the driconf<br>
option "adaptive_sync" is disabled, e.g. via the application blacklist<br>
in /usr/share/drirc.d/00-mesa-defaults.conf (mostly for compositors /<br>
browsers / video players).<br></blockquote><div>Yes.. I had verified this too from <a href="https://patchwork.freedesktop.org/patch/257941/?series=51388&rev=1">https://patchwork.freedesktop.org/patch/257941/?series=51388&rev=1</a></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
<br>
-- <br>
Earthling Michel Dänzer               |               <a href="https://redhat.com" rel="noreferrer" target="_blank">https://redhat.com</a><br>
Libre software enthusiast             |             Mesa and X developer<br>
</blockquote></div></div>