svn commit: r296320 - in head/sys: kern sys

[Konstantin Belousov]

On Thu, Mar 03, 2016 at 11:59:36AM +0100, Hans Petter Selasky wrote:
> I believe all of these callout quirks can be avoided by using a spinlock 
> to proctect the thread callout like done in projects/hps_head. Has 
> anyone tried to reproduce the issue with projects/hps_head, before 
> making this patch?
I did not looked at your branch, but I do not like this coupling between
callbacks and thread locks.  Even more, this probably cannot make work
since thread lock is not some fixed lock at all, and introducing one
more spinlock for the coupling is excessive.

> 
> BTW: The FreeBSD kernel version should have been bumped because 
> _callout_stop_safe() is a public callout API, used by all external 
> kernel modules, which now will have undefined behaviour until recompiled.
The change was done in a way which is compatible with the pre-commit
callout_stop_safe(9)  users.  I even consider it mergeable into the
stable/10 without requiring any shims.