Re: [PATCH] drm/amd/powerplay: Ratelimit all "was not implemented" messages

From: Alex Deucher
Date: Thu Nov 15 2018 - 15:40:42 EST


On Thu, Nov 15, 2018 at 8:47 AM Joerg Roedel <joro@xxxxxxxxxx> wrote:
>
> From: Joerg Roedel <jroedel@xxxxxxx>
>
> Running kfdtest on an AMD Carizzo flooded the kernel log
> with thousands of these "was not implemented" messages,
> making it impossible to see other messages there.
>
> Ratelimit the messages to prevent user-space from flooding
> the kernel log.
>
> Signed-off-by: Joerg Roedel <jroedel@xxxxxxx>

Should we just make these debug only? Applied.

Thanks,

Alex

> ---
> drivers/gpu/drm/amd/powerplay/amd_powerplay.c | 52 +++++++++++++--------------
> 1 file changed, 26 insertions(+), 26 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/powerplay/amd_powerplay.c b/drivers/gpu/drm/amd/powerplay/amd_powerplay.c
> index d6aa1d414320..b68c2e0fef01 100644
> --- a/drivers/gpu/drm/amd/powerplay/amd_powerplay.c
> +++ b/drivers/gpu/drm/amd/powerplay/amd_powerplay.c
> @@ -300,7 +300,7 @@ static int pp_set_clockgating_by_smu(void *handle, uint32_t msg_id)
> return -EINVAL;
>
> if (hwmgr->hwmgr_func->update_clock_gatings == NULL) {
> - pr_info("%s was not implemented.\n", __func__);
> + pr_info_ratelimited("%s was not implemented.\n", __func__);
> return 0;
> }
>
> @@ -387,7 +387,7 @@ static uint32_t pp_dpm_get_sclk(void *handle, bool low)
> return 0;
>
> if (hwmgr->hwmgr_func->get_sclk == NULL) {
> - pr_info("%s was not implemented.\n", __func__);
> + pr_info_ratelimited("%s was not implemented.\n", __func__);
> return 0;
> }
> mutex_lock(&hwmgr->smu_lock);
> @@ -405,7 +405,7 @@ static uint32_t pp_dpm_get_mclk(void *handle, bool low)
> return 0;
>
> if (hwmgr->hwmgr_func->get_mclk == NULL) {
> - pr_info("%s was not implemented.\n", __func__);
> + pr_info_ratelimited("%s was not implemented.\n", __func__);
> return 0;
> }
> mutex_lock(&hwmgr->smu_lock);
> @@ -422,7 +422,7 @@ static void pp_dpm_powergate_vce(void *handle, bool gate)
> return;
>
> if (hwmgr->hwmgr_func->powergate_vce == NULL) {
> - pr_info("%s was not implemented.\n", __func__);
> + pr_info_ratelimited("%s was not implemented.\n", __func__);
> return;
> }
> mutex_lock(&hwmgr->smu_lock);
> @@ -438,7 +438,7 @@ static void pp_dpm_powergate_uvd(void *handle, bool gate)
> return;
>
> if (hwmgr->hwmgr_func->powergate_uvd == NULL) {
> - pr_info("%s was not implemented.\n", __func__);
> + pr_info_ratelimited("%s was not implemented.\n", __func__);
> return;
> }
> mutex_lock(&hwmgr->smu_lock);
> @@ -505,7 +505,7 @@ static void pp_dpm_set_fan_control_mode(void *handle, uint32_t mode)
> return;
>
> if (hwmgr->hwmgr_func->set_fan_control_mode == NULL) {
> - pr_info("%s was not implemented.\n", __func__);
> + pr_info_ratelimited("%s was not implemented.\n", __func__);
> return;
> }
> mutex_lock(&hwmgr->smu_lock);
> @@ -522,7 +522,7 @@ static uint32_t pp_dpm_get_fan_control_mode(void *handle)
> return 0;
>
> if (hwmgr->hwmgr_func->get_fan_control_mode == NULL) {
> - pr_info("%s was not implemented.\n", __func__);
> + pr_info_ratelimited("%s was not implemented.\n", __func__);
> return 0;
> }
> mutex_lock(&hwmgr->smu_lock);
> @@ -540,7 +540,7 @@ static int pp_dpm_set_fan_speed_percent(void *handle, uint32_t percent)
> return -EINVAL;
>
> if (hwmgr->hwmgr_func->set_fan_speed_percent == NULL) {
> - pr_info("%s was not implemented.\n", __func__);
> + pr_info_ratelimited("%s was not implemented.\n", __func__);
> return 0;
> }
> mutex_lock(&hwmgr->smu_lock);
> @@ -558,7 +558,7 @@ static int pp_dpm_get_fan_speed_percent(void *handle, uint32_t *speed)
> return -EINVAL;
>
> if (hwmgr->hwmgr_func->get_fan_speed_percent == NULL) {
> - pr_info("%s was not implemented.\n", __func__);
> + pr_info_ratelimited("%s was not implemented.\n", __func__);
> return 0;
> }
>
> @@ -594,7 +594,7 @@ static int pp_dpm_set_fan_speed_rpm(void *handle, uint32_t rpm)
> return -EINVAL;
>
> if (hwmgr->hwmgr_func->set_fan_speed_rpm == NULL) {
> - pr_info("%s was not implemented.\n", __func__);
> + pr_info_ratelimited("%s was not implemented.\n", __func__);
> return 0;
> }
> mutex_lock(&hwmgr->smu_lock);
> @@ -720,7 +720,7 @@ static int pp_dpm_force_clock_level(void *handle,
> return -EINVAL;
>
> if (hwmgr->hwmgr_func->force_clock_level == NULL) {
> - pr_info("%s was not implemented.\n", __func__);
> + pr_info_ratelimited("%s was not implemented.\n", __func__);
> return 0;
> }
>
> @@ -745,7 +745,7 @@ static int pp_dpm_print_clock_levels(void *handle,
> return -EINVAL;
>
> if (hwmgr->hwmgr_func->print_clock_levels == NULL) {
> - pr_info("%s was not implemented.\n", __func__);
> + pr_info_ratelimited("%s was not implemented.\n", __func__);
> return 0;
> }
> mutex_lock(&hwmgr->smu_lock);
> @@ -763,7 +763,7 @@ static int pp_dpm_get_sclk_od(void *handle)
> return -EINVAL;
>
> if (hwmgr->hwmgr_func->get_sclk_od == NULL) {
> - pr_info("%s was not implemented.\n", __func__);
> + pr_info_ratelimited("%s was not implemented.\n", __func__);
> return 0;
> }
> mutex_lock(&hwmgr->smu_lock);
> @@ -781,7 +781,7 @@ static int pp_dpm_set_sclk_od(void *handle, uint32_t value)
> return -EINVAL;
>
> if (hwmgr->hwmgr_func->set_sclk_od == NULL) {
> - pr_info("%s was not implemented.\n", __func__);
> + pr_info_ratelimited("%s was not implemented.\n", __func__);
> return 0;
> }
>
> @@ -800,7 +800,7 @@ static int pp_dpm_get_mclk_od(void *handle)
> return -EINVAL;
>
> if (hwmgr->hwmgr_func->get_mclk_od == NULL) {
> - pr_info("%s was not implemented.\n", __func__);
> + pr_info_ratelimited("%s was not implemented.\n", __func__);
> return 0;
> }
> mutex_lock(&hwmgr->smu_lock);
> @@ -818,7 +818,7 @@ static int pp_dpm_set_mclk_od(void *handle, uint32_t value)
> return -EINVAL;
>
> if (hwmgr->hwmgr_func->set_mclk_od == NULL) {
> - pr_info("%s was not implemented.\n", __func__);
> + pr_info_ratelimited("%s was not implemented.\n", __func__);
> return 0;
> }
> mutex_lock(&hwmgr->smu_lock);
> @@ -878,7 +878,7 @@ static int pp_get_power_profile_mode(void *handle, char *buf)
> return -EINVAL;
>
> if (hwmgr->hwmgr_func->get_power_profile_mode == NULL) {
> - pr_info("%s was not implemented.\n", __func__);
> + pr_info_ratelimited("%s was not implemented.\n", __func__);
> return snprintf(buf, PAGE_SIZE, "\n");
> }
>
> @@ -894,7 +894,7 @@ static int pp_set_power_profile_mode(void *handle, long *input, uint32_t size)
> return ret;
>
> if (hwmgr->hwmgr_func->set_power_profile_mode == NULL) {
> - pr_info("%s was not implemented.\n", __func__);
> + pr_info_ratelimited("%s was not implemented.\n", __func__);
> return ret;
> }
>
> @@ -917,7 +917,7 @@ static int pp_odn_edit_dpm_table(void *handle, uint32_t type, long *input, uint3
> return -EINVAL;
>
> if (hwmgr->hwmgr_func->odn_edit_dpm_table == NULL) {
> - pr_info("%s was not implemented.\n", __func__);
> + pr_info_ratelimited("%s was not implemented.\n", __func__);
> return -EINVAL;
> }
>
> @@ -935,7 +935,7 @@ static int pp_dpm_switch_power_profile(void *handle,
> return -EINVAL;
>
> if (hwmgr->hwmgr_func->set_power_profile_mode == NULL) {
> - pr_info("%s was not implemented.\n", __func__);
> + pr_info_ratelimited("%s was not implemented.\n", __func__);
> return -EINVAL;
> }
>
> @@ -972,7 +972,7 @@ static int pp_set_power_limit(void *handle, uint32_t limit)
> return -EINVAL;
>
> if (hwmgr->hwmgr_func->set_power_limit == NULL) {
> - pr_info("%s was not implemented.\n", __func__);
> + pr_info_ratelimited("%s was not implemented.\n", __func__);
> return -EINVAL;
> }
>
> @@ -1212,7 +1212,7 @@ static int pp_dpm_powergate_mmhub(void *handle)
> return -EINVAL;
>
> if (hwmgr->hwmgr_func->powergate_mmhub == NULL) {
> - pr_info("%s was not implemented.\n", __func__);
> + pr_info_ratelimited("%s was not implemented.\n", __func__);
> return 0;
> }
>
> @@ -1227,7 +1227,7 @@ static int pp_dpm_powergate_gfx(void *handle, bool gate)
> return 0;
>
> if (hwmgr->hwmgr_func->powergate_gfx == NULL) {
> - pr_info("%s was not implemented.\n", __func__);
> + pr_info_ratelimited("%s was not implemented.\n", __func__);
> return 0;
> }
>
> @@ -1242,7 +1242,7 @@ static void pp_dpm_powergate_acp(void *handle, bool gate)
> return;
>
> if (hwmgr->hwmgr_func->powergate_acp == NULL) {
> - pr_info("%s was not implemented.\n", __func__);
> + pr_info_ratelimited("%s was not implemented.\n", __func__);
> return;
> }
>
> @@ -1257,7 +1257,7 @@ static void pp_dpm_powergate_sdma(void *handle, bool gate)
> return;
>
> if (hwmgr->hwmgr_func->powergate_sdma == NULL) {
> - pr_info("%s was not implemented.\n", __func__);
> + pr_info_ratelimited("%s was not implemented.\n", __func__);
> return;
> }
>
> @@ -1303,7 +1303,7 @@ static int pp_notify_smu_enable_pwe(void *handle)
> return -EINVAL;
>
> if (hwmgr->hwmgr_func->smus_notify_pwe == NULL) {
> - pr_info("%s was not implemented.\n", __func__);
> + pr_info_ratelimited("%s was not implemented.\n", __func__);
> return -EINVAL;;
> }
>
> --
> 2.13.7
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx