drm/msm/dpu: Ratelimit framedone timeout msgs
[ Upstream commit 2b72e50c62de60ad2d6bcd86aa38d4ccbdd633f2 ] When we start getting these, we get a *lot*. So ratelimit it to not flood dmesg. Signed-off-by: Rob Clark <robdclark@chromium.org> Reviewed-by: Abhinav Kumar <quic_abhinavk@quicinc.com> Reviewed-by: Marijn Suijten <marijn.suijten@somainline.org> Patchwork: https://patchwork.freedesktop.org/patch/571584/ Link: https://lore.kernel.org/r/20231211182000.218088-1-robdclark@gmail.com Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
parent
42a5df71b9
commit
d9ae14cf3c
2 changed files with 5 additions and 1 deletions
|
@ -45,6 +45,9 @@
|
||||||
(p) ? ((p)->hw_pp ? (p)->hw_pp->idx - PINGPONG_0 : -1) : -1, \
|
(p) ? ((p)->hw_pp ? (p)->hw_pp->idx - PINGPONG_0 : -1) : -1, \
|
||||||
##__VA_ARGS__)
|
##__VA_ARGS__)
|
||||||
|
|
||||||
|
#define DPU_ERROR_ENC_RATELIMITED(e, fmt, ...) DPU_ERROR_RATELIMITED("enc%d " fmt,\
|
||||||
|
(e) ? (e)->base.base.id : -1, ##__VA_ARGS__)
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* Two to anticipate panels that can do cmd/vid dynamic switching
|
* Two to anticipate panels that can do cmd/vid dynamic switching
|
||||||
* plan is to create all possible physical encoder types, and switch between
|
* plan is to create all possible physical encoder types, and switch between
|
||||||
|
@ -2135,7 +2138,7 @@ static void dpu_encoder_frame_done_timeout(struct timer_list *t)
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
DPU_ERROR_ENC(dpu_enc, "frame done timeout\n");
|
DPU_ERROR_ENC_RATELIMITED(dpu_enc, "frame done timeout\n");
|
||||||
|
|
||||||
event = DPU_ENCODER_FRAME_EVENT_ERROR;
|
event = DPU_ENCODER_FRAME_EVENT_ERROR;
|
||||||
trace_dpu_enc_frame_done_timeout(DRMID(drm_enc), event);
|
trace_dpu_enc_frame_done_timeout(DRMID(drm_enc), event);
|
||||||
|
|
|
@ -52,6 +52,7 @@
|
||||||
} while (0)
|
} while (0)
|
||||||
|
|
||||||
#define DPU_ERROR(fmt, ...) pr_err("[dpu error]" fmt, ##__VA_ARGS__)
|
#define DPU_ERROR(fmt, ...) pr_err("[dpu error]" fmt, ##__VA_ARGS__)
|
||||||
|
#define DPU_ERROR_RATELIMITED(fmt, ...) pr_err_ratelimited("[dpu error]" fmt, ##__VA_ARGS__)
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* ktime_compare_safe - compare two ktime structures
|
* ktime_compare_safe - compare two ktime structures
|
||||||
|
|
Loading…
Reference in a new issue