svn commit: r302350 - in head: share/man/man9 sys/kern sys/sys

[Bryan Drewery]

On 7/13/16 3:11 AM, Hans Petter Selasky wrote:
> Hi,
> 
> Can we agree on the following return values for callout_stop() and
> callout_reset() and their variants:
> 
> If the callout was scheduled regardless of being serviced or not,
> .Dv CALLOUT_RET_CANCELLED
> is returned.
> If the callout was stopped and is still being serviced
> .Dv CALLOUT_RET_DRAINING
> is returned.
> If the callout was stopped and is no longer being serviced
> .Dv CALLOUT_RET_STOPPED
> is returned.
> 
> --HPS
> 

I won't comment on the specific cases, but I do think we should move to
using enums or macros for these return values and other parts of the KPI.

The callout API is getting very abused lately, since r290664, and there
seems to be a lack of review or consensus around most changes.  Even in
the recent changes, the review brought up several points that were
seemingly ignored, and more that are still being ignored.  The constant
churn and reverts on this API is not acceptable.

-- 
Regards,
Bryan Drewery