| From 1d7f97d5e7e1ec3cfc0847c62aeeec4c9d72bf53 Mon Sep 17 00:00:00 2001 |
| From: Sasha Levin <sashal@kernel.org> |
| Date: Fri, 13 Jun 2025 10:15:46 -0700 |
| Subject: ptp: Use ratelimite for freerun error message |
| |
| From: Breno Leitao <leitao@debian.org> |
| |
| [ Upstream commit e9a7795e75b78b56997fb0070c18d6e1057b6462 ] |
| |
| Replace pr_err() with pr_err_ratelimited() in ptp_clock_settime() to |
| prevent log flooding when the physical clock is free running, which |
| happens on some of my hosts. This ensures error messages are |
| rate-limited and improves kernel log readability. |
| |
| Signed-off-by: Breno Leitao <leitao@debian.org> |
| Reviewed-by: Simon Horman <horms@kernel.org> |
| Link: https://patch.msgid.link/20250613-ptp-v1-1-ee44260ce9e2@debian.org |
| Signed-off-by: Jakub Kicinski <kuba@kernel.org> |
| Signed-off-by: Sasha Levin <sashal@kernel.org> |
| --- |
| drivers/ptp/ptp_clock.c | 2 +- |
| 1 file changed, 1 insertion(+), 1 deletion(-) |
| |
| diff --git a/drivers/ptp/ptp_clock.c b/drivers/ptp/ptp_clock.c |
| index 642c939d4523..e6bcccf87cd6 100644 |
| --- a/drivers/ptp/ptp_clock.c |
| +++ b/drivers/ptp/ptp_clock.c |
| @@ -79,7 +79,7 @@ static int ptp_clock_settime(struct posix_clock *pc, const struct timespec64 *tp |
| struct ptp_clock *ptp = container_of(pc, struct ptp_clock, clock); |
| |
| if (ptp_clock_freerun(ptp)) { |
| - pr_err("ptp: physical clock is free running\n"); |
| + pr_err_ratelimited("ptp: physical clock is free running\n"); |
| return -EBUSY; |
| } |
| |
| -- |
| 2.39.5 |
| |