* [PATCH net RESEND] ptp: ocp: have adjtime handle negative delta_ns correctly
@ 2022-05-06 22:37 Jonathan Lemon
2022-05-06 22:37 ` [PATCH net RESEND] ptp: ocp: Use DIV64_U64_ROUND_UP for rounding Jonathan Lemon
0 siblings, 1 reply; 3+ messages in thread
From: Jonathan Lemon @ 2022-05-06 22:37 UTC (permalink / raw)
To: netdev; +Cc: richardcochran, kernel-team, davem, kuba, pabeni, edumazet
delta_ns is a s64, but it was being passed ptp_ocp_adjtime_coarse
as an u64. Also, it turns out that timespec64_add_ns() only handles
positive values, so the math needs to be updated.
Fix by passing in the correct signed value, then adding to a
nanosecond version of the timespec.
Fixes: 90f8f4c0e3ce ("ptp: ocp: Add ptp_ocp_adjtime_coarse for large adjustments")
Signed-off-by: Jonathan Lemon <jonathan.lemon@gmail.com>
---
Resend to fix misquoted Fixes: tag
---
drivers/ptp/ptp_ocp.c | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/drivers/ptp/ptp_ocp.c b/drivers/ptp/ptp_ocp.c
index dd45471f6780..65e592ec272e 100644
--- a/drivers/ptp/ptp_ocp.c
+++ b/drivers/ptp/ptp_ocp.c
@@ -841,16 +841,18 @@ __ptp_ocp_adjtime_locked(struct ptp_ocp *bp, u32 adj_val)
}
static void
-ptp_ocp_adjtime_coarse(struct ptp_ocp *bp, u64 delta_ns)
+ptp_ocp_adjtime_coarse(struct ptp_ocp *bp, s64 delta_ns)
{
struct timespec64 ts;
unsigned long flags;
int err;
+ s64 ns;
spin_lock_irqsave(&bp->lock, flags);
err = __ptp_ocp_gettime_locked(bp, &ts, NULL);
if (likely(!err)) {
- timespec64_add_ns(&ts, delta_ns);
+ ns = timespec64_to_ns(&ts) + delta_ns;
+ ts = ns_to_timespec64(ns);
__ptp_ocp_settime_locked(bp, &ts);
}
spin_unlock_irqrestore(&bp->lock, flags);
--
2.31.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [PATCH net RESEND] ptp: ocp: Use DIV64_U64_ROUND_UP for rounding.
2022-05-06 22:37 [PATCH net RESEND] ptp: ocp: have adjtime handle negative delta_ns correctly Jonathan Lemon
@ 2022-05-06 22:37 ` Jonathan Lemon
2022-05-10 0:30 ` patchwork-bot+netdevbpf
0 siblings, 1 reply; 3+ messages in thread
From: Jonathan Lemon @ 2022-05-06 22:37 UTC (permalink / raw)
To: netdev; +Cc: richardcochran, kernel-team, davem, kuba, pabeni, edumazet
The initial code used roundup() to round the starting time to
a multiple of a period. This generated an error on 32-bit
systems, so was replaced with DIV_ROUND_UP_ULL().
However, this truncates to 32-bits on a 64-bit system. Replace
with DIV64_U64_ROUND_UP() instead.
Fixes: b325af3cfab9 ("ptp: ocp: Add signal generators and update sysfs nodes")
Signed-off-by: Jonathan Lemon <jonathan.lemon@gmail.com>
---
Resend to fix misquoted Fixes: tag
---
drivers/ptp/ptp_ocp.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/ptp/ptp_ocp.c b/drivers/ptp/ptp_ocp.c
index 0feaa4b45317..dd45471f6780 100644
--- a/drivers/ptp/ptp_ocp.c
+++ b/drivers/ptp/ptp_ocp.c
@@ -1557,7 +1557,7 @@ ptp_ocp_signal_set(struct ptp_ocp *bp, int gen, struct ptp_ocp_signal *s)
start_ns = ktime_set(ts.tv_sec, ts.tv_nsec) + NSEC_PER_MSEC;
if (!s->start) {
/* roundup() does not work on 32-bit systems */
- s->start = DIV_ROUND_UP_ULL(start_ns, s->period);
+ s->start = DIV64_U64_ROUND_UP(start_ns, s->period);
s->start = ktime_add(s->start, s->phase);
}
--
2.31.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH net RESEND] ptp: ocp: Use DIV64_U64_ROUND_UP for rounding.
2022-05-06 22:37 ` [PATCH net RESEND] ptp: ocp: Use DIV64_U64_ROUND_UP for rounding Jonathan Lemon
@ 2022-05-10 0:30 ` patchwork-bot+netdevbpf
0 siblings, 0 replies; 3+ messages in thread
From: patchwork-bot+netdevbpf @ 2022-05-10 0:30 UTC (permalink / raw)
To: Jonathan Lemon
Cc: netdev, richardcochran, kernel-team, davem, kuba, pabeni, edumazet
Hello:
This patch was applied to netdev/net.git (master)
by Jakub Kicinski <kuba@kernel.org>:
On Fri, 6 May 2022 15:37:39 -0700 you wrote:
> The initial code used roundup() to round the starting time to
> a multiple of a period. This generated an error on 32-bit
> systems, so was replaced with DIV_ROUND_UP_ULL().
>
> However, this truncates to 32-bits on a 64-bit system. Replace
> with DIV64_U64_ROUND_UP() instead.
>
> [...]
Here is the summary with links:
- [net,RESEND] ptp: ocp: Use DIV64_U64_ROUND_UP for rounding.
https://git.kernel.org/netdev/net/c/4bd46bb037f8
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2022-05-10 0:30 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-05-06 22:37 [PATCH net RESEND] ptp: ocp: have adjtime handle negative delta_ns correctly Jonathan Lemon
2022-05-06 22:37 ` [PATCH net RESEND] ptp: ocp: Use DIV64_U64_ROUND_UP for rounding Jonathan Lemon
2022-05-10 0:30 ` patchwork-bot+netdevbpf
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.