On 2018-02-08 10:10 AM, Harry Wentland wrote:
> On 2018-02-08 10:07 AM, Zhu, Rex wrote:
>> when autowattman enabled�we will update uphyst/downhyst/min-sclk/mclk activity value to smu based on the workload.
>>
>
> Why is this incompatible with Freesync?
>
Just had a chat with the Windows guys. Apparently AutoWattman changes clocks quite aggressively which can make framerates jump significantly and lead to a subpar experience when using Freesync.
We'll still need a patch to call this interface from DC. Is that on your plate or is this something we should hook up?
Harry
> Harry
>
>> Best Regards
>> Rex
>>
>> ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
>> *From:* Wentland, Harry
>> *Sent:* Thursday, February 8, 2018 10:22:16 PM
>> *To:* Zhu, Rex; amd-gfx at lists.freedesktop.org
>> *Subject:* Re: [PATCH] drm/amd/pp: Expose new interface to DC to ctrl auto wattman
>> Â
>> On 2018-02-08 06:20 AM, Rex Zhu wrote:
>>> Disable AutoWattman (if enabled) when FreeSync is enabled.
>>
>> Do you have a DC change calling this?
>>
>> What's the use case for this and why do we need to disable AutoWattman when Freesync is enabled?
>>
>> What does AutoWattman do?
>>
>> Harry
>>
>>>
>>> Change-Id: I9a531321d7913b8b40e60070c569a01c4f202002
>>> Signed-off-by: Rex Zhu <Rex.Zhu at amd.com>
>>> ---
>>> Â drivers/gpu/drm/amd/include/kgd_pp_interface.h |Â 1 +
>>>  drivers/gpu/drm/amd/powerplay/amd_powerplay.c | 25 +++++++++++++++++++++++++
>>> Â 2 files changed, 26 insertions(+)
>>>
>>> diff --git a/drivers/gpu/drm/amd/include/kgd_pp_interface.h b/drivers/gpu/drm/amd/include/kgd_pp_interface.h
>>> index 22c2fa3..f7bb565 100644
>>> --- a/drivers/gpu/drm/amd/include/kgd_pp_interface.h
>>> +++ b/drivers/gpu/drm/amd/include/kgd_pp_interface.h
>>> @@ -313,6 +313,7 @@ struct amd_pm_funcs {
>>> Â Â Â Â Â Â Â int (*set_power_profile_mode)(void *handle, long *input, uint32_t size);
>>> Â Â Â Â Â Â Â int (*odn_edit_dpm_table)(void *handle, uint32_t type, long *input, uint32_t size);
>>> Â Â Â Â Â Â Â int (*set_mmhub_powergating_by_smu)(void *handle);
>>> +Â Â Â Â int (*notify_free_sync_change)(void *handle, bool en);
>>> Â };
>>> Â
>>> Â #endif
>>> diff --git a/drivers/gpu/drm/amd/powerplay/amd_powerplay.c b/drivers/gpu/drm/amd/powerplay/amd_powerplay.c
>>> index 376ed2d..d0306b6 100644
>>> --- a/drivers/gpu/drm/amd/powerplay/amd_powerplay.c
>>> +++ b/drivers/gpu/drm/amd/powerplay/amd_powerplay.c
>>> @@ -1555,6 +1555,30 @@ static int pp_set_mmhub_powergating_by_smu(void *handle)
>>> Â Â Â Â Â Â Â return hwmgr->hwmgr_func->set_mmhub_powergating_by_smu(hwmgr);
>>> Â }
>>> Â
>>> +static int pp_notify_free_sync_change(void *handle, bool en)
>>> +{
>>> +Â Â Â Â struct pp_hwmgr *hwmgr;
>>> +Â Â Â Â struct pp_instance *pp_handle = (struct pp_instance *)handle;
>>> +Â Â Â Â int ret = 0;
>>> +
>>> +Â Â Â Â ret = pp_check(pp_handle);
>>> +
>>> +Â Â Â Â if (ret)
>>> +Â Â Â Â Â Â Â Â Â Â Â Â return ret;
>>> +
>>> +Â Â Â Â hwmgr = pp_handle->hwmgr;
>>> +
>>> +Â Â Â Â mutex_lock(&pp_handle->pp_lock);
>>> +Â Â Â Â if (hwmgr->autowattman_enabled) {
>>> +Â Â Â Â Â Â Â Â Â Â Â Â if (hwmgr->hwmgr_func->start_auto_wattman != NULL) {
>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â if (!cancel_delayed_work_sync(&hwmgr->wattman_update_work))
>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â hwmgr->hwmgr_func->start_auto_wattman(hwmgr, en);
>>> +Â Â Â Â Â Â Â Â Â Â Â Â }
>>> +Â Â Â Â }
>>> +Â Â Â Â mutex_unlock(&pp_handle->pp_lock);
>>> +Â Â Â Â return 0;
>>> +}
>>> +
>>> Â const struct amd_pm_funcs pp_dpm_funcs = {
>>> Â Â Â Â Â Â Â .load_firmware = pp_dpm_load_fw,
>>> Â Â Â Â Â Â Â .wait_for_fw_loading_complete = pp_dpm_fw_loading_complete,
>>> @@ -1604,4 +1628,5 @@ static int pp_set_mmhub_powergating_by_smu(void *handle)
>>> Â Â Â Â Â Â Â .display_clock_voltage_request = pp_display_clock_voltage_request,
>>> Â Â Â Â Â Â Â .get_display_mode_validation_clocks = pp_get_display_mode_validation_clocks,
>>> Â Â Â Â Â Â Â .set_mmhub_powergating_by_smu = pp_set_mmhub_powergating_by_smu,
>>> +Â Â Â Â .notify_free_sync_change = pp_notify_free_sync_change,
>>> Â };
>>>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
>
