A bit large, maybe 100ms is enough.
Apart from that the set now looks good on first glance, but I really
don't know that code that well.
Christian.
Am 30.07.2018 um 12:25 schrieb Zhu, Rex:
>
> One problem,
>
>
> I delay 500ms to enable gfx off feature.
>
>
> Any concern about the delay time?
>
>
> Best Regards
>
> Rex
>
>
> ------------------------------------------------------------------------
> *From:* amd-gfx <amd-gfx-bounces at lists.freedesktop.org> on behalf of
> Rex Zhu <Rex.Zhu at amd.com>
> *Sent:* Monday, July 30, 2018 5:23 PM
> *To:* amd-gfx at lists.freedesktop.org
> *Cc:* Zhu, Rex
> *Subject:* [PATCH v2 2/4] drm/amdgpu: Put enable gfx off feature to a
> delay thread
> delay to enable gfx off feature to avoid gfx on/off frequently
> suggested by Alex and Evan.
>
> Signed-off-by: Rex Zhu <Rex.Zhu at amd.com>
> ---
>  drivers/gpu/drm/amd/amdgpu/amdgpu.h       | 2 ++
> Â drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 15 +++++++++++++++
>  drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.c   | 8 ++++++--
> Â 3 files changed, 23 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu.h
> b/drivers/gpu/drm/amd/amdgpu/amdgpu.h
> index 188bc53..ff16689 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu.h
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu.h
> @@ -992,6 +992,8 @@ struct amdgpu_gfx {
> Â Â Â Â Â Â Â Â bool gfx_off_state; /* true: enabled, false: disabled */
> Â Â Â Â Â Â Â Â struct mutex gfx_off_mutex;
> Â Â Â Â Â Â Â Â uint32_t gfx_off_req_count; /* default 1, enable gfx off: dec
> 1, disable gfx off: add 1 */
> +Â Â Â Â Â Â struct delayed_work gfx_off_delay_work;
> +
> Â Â Â Â Â Â Â Â /* pipe reservation */
> Â Â Â Â Â Â Â Â struct mutex pipe_reserve_mutex;
> Â Â Â Â Â Â Â Â DECLARE_BITMAP (pipe_reserve_bitmap, AMDGPU_MAX_COMPUTE_QUEUES);
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
> b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
> index 122653f..f6a6fb5 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
> @@ -1925,6 +1925,19 @@ static void
> amdgpu_device_ip_late_init_func_handler(struct work_struct *work)
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â DRM_ERROR("ib ring test failed (%d).\n", r);
> Â }
>
> +static void amdgpu_device_delay_enable_gfx_off(struct work_struct *work)
> +{
> +Â Â Â Â Â Â struct amdgpu_device *adev =
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â container_of(work, struct amdgpu_device,
> gfx.gfx_off_delay_work.work);
> +
> +Â Â Â Â Â Â mutex_lock(&adev->gfx.gfx_off_mutex);
> +Â Â Â Â Â Â if (!adev->gfx.gfx_off_state && !adev->gfx.gfx_off_req_count) {
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â if (!amdgpu_dpm_set_powergating_by_smu(adev,
> AMD_IP_BLOCK_TYPE_GFX, true))
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â adev->gfx.gfx_off_state = true;
> +Â Â Â Â Â Â }
> +Â Â Â Â Â Â mutex_unlock(&adev->gfx.gfx_off_mutex);
> +}
> +
> Â /**
> Â * amdgpu_device_ip_suspend_phase1 - run suspend for hardware IPs
> (phase 1)
> Â *
> @@ -2394,6 +2407,8 @@ int amdgpu_device_init(struct amdgpu_device *adev,
>
> INIT_DELAYED_WORK(&adev->late_init_work,
> amdgpu_device_ip_late_init_func_handler);
> + INIT_DELAYED_WORK(&adev->gfx.gfx_off_delay_work,
> + amdgpu_device_delay_enable_gfx_off);
>
> Â Â Â Â Â Â Â Â adev->gfx.gfx_off_req_count = 1;
> Â Â Â Â Â Â Â Â adev->pm.ac_power = power_supply_is_system_supplied() > 0 ?
> true : false;
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.c
> b/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.c
> index 1cdb264..11d4d9f 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.c
> @@ -26,6 +26,9 @@
> Â #include "amdgpu.h"
> Â #include "amdgpu_gfx.h"
>
> +/* 0.5 second timeout */
> +#define GFX_OFF_DELAY_ENABLE msecs_to_jiffies(500)
> +
> Â /*
> Â * GPU scratch registers helpers function.
> Â */
> @@ -360,6 +363,7 @@ void amdgpu_gfx_off_ctrl(struct amdgpu_device
> *adev, bool enable)
> Â Â Â Â Â Â Â Â if (!adev->powerplay.pp_funcs->set_powergating_by_smu)
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â return;
>
> +
> Â Â Â Â Â Â Â Â mutex_lock(&adev->gfx.gfx_off_mutex);
>
> Â Â Â Â Â Â Â Â if (!enable)
> @@ -368,11 +372,11 @@ void amdgpu_gfx_off_ctrl(struct amdgpu_device
> *adev, bool enable)
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â adev->gfx.gfx_off_req_count--;
>
> Â Â Â Â Â Â Â Â if (enable && !adev->gfx.gfx_off_state &&
> !adev->gfx.gfx_off_req_count) {
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â if (!amdgpu_dpm_set_powergating_by_smu(adev,
> AMD_IP_BLOCK_TYPE_GFX, true))
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â adev->gfx.gfx_off_state = true;
> + schedule_delayed_work(&adev->gfx.gfx_off_delay_work,
> GFX_OFF_DELAY_ENABLE);
> Â Â Â Â Â Â Â Â } else if (!enable && adev->gfx.gfx_off_state) {
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â if (!amdgpu_dpm_set_powergating_by_smu(adev,
> AMD_IP_BLOCK_TYPE_GFX, false))
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â adev->gfx.gfx_off_state = false;
> Â Â Â Â Â Â Â Â }
> +
> mutex_unlock(&adev->gfx.gfx_off_mutex);
> Â }
> --
> 1.9.1
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
> amd-gfx Info Page - freedesktop.org
> <https://lists.freedesktop.org/mailman/listinfo/amd-gfx>
> lists.freedesktop.org
> Subscribing to amd-gfx: Subscribe to amd-gfx by filling out the
> following form. Use of all freedesktop.org lists is subject to our
> Code of Conduct.
>
>
>
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/amd-gfx/attachments/20180730/43c661df/attachment-0001.html>
