CVS commit: [netbsd-10] src/sys/altq

["Martin Husemann" <martin%netbsd.org@localhost>]

Module Name:    src
Committed By:   martin
Date:           Thu Feb 20 19:30:38 UTC 2025

Modified Files:
        src/sys/altq [netbsd-10]: altq_rmclass.h

Log Message:
Pull up following revision(s) (requested by ozaki-r in ticket #1049):

        sys/altq/altq_rmclass.h: revision 1.14

altq, cbq: fix TS_ADD_DELTA on ILP32

TS_ADD_DELTA adds a nanosecond value, delta, to a timespec value while
delta is int64_t and tv_nsec is long.  delta is guaranteed as positive
in ALTQ/CBQ.  However, the original code used delta after casting it
into long.  So it could be negative on ILP32 systems.  TS_ADD_DELTA
didn't assume that delta was negative and resulting tv_nsec could be
negative unexpectedly.

This issue was likely to occur when bandwidth borrowing was used where
the borrowing class had a absolutely small bandwidth (e.g., less than
10 Mbps) or a relatively small bandwidth (e.g., 10 Mbps when the parent
class had 200 Mbps).

KASSERT has been added to guarantee resulting tv_nsec is always not
negative.  Additionally, another KASSERT has been added to make it clear
that delta is assumed to be non-negative.


To generate a diff of this commit:
cvs rdiff -u -r1.13 -r1.13.4.1 src/sys/altq/altq_rmclass.h

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.