Justin! On Fri, May 24 2024 at 15:43, Justin Stitt wrote: > I appreciate you reviewing my patches.
You're welcome! > On Fri, May 24, 2024 at 5:09 AM Thomas Gleixner <[email protected]> wrote: >> So instead of turning the clock back, we might be better off to actually >> put the normalization in place at the assignment: >> >> time_maxerror = min(max(0, txc->maxerror), NTP_PHASE_LIMIT); > > A saturating resolution strategy is one that I've taken with some of > my other overflow patches. > > ... but how about: clamp(txc->maxerror, 0, NTP_PHASE_LIMIT) Duh. You are right, but that's too obvious :) Thanks, tglx
