On Fri, Dec 16, 2016 at 03:19:09PM -0800, John Baldwin wrote:
>
> So the hack in pause() is probably not as necessary now. In particular, I
> think we only need it for thread0, not for other threads. The patch below
> worked for me with SPEW's config:
>
> Index: kern_synch.c
> ===================================================================
> --- kern_synch.c (revision 310128)
> +++ kern_synch.c (working copy)
> @@ -321,7 +321,8 @@ pause_sbt(const char *wmesg, sbintime_t sbt, sbint
> if (sbt == 0)
> sbt = tick_sbt;
>
> - if (cold || kdb_active || SCHEDULER_STOPPED()) {
> + if ((cold && curthread == &thread0) || kdb_active ||
> + SCHEDULER_STOPPED()) {
> /*
> * We delay one second at a time to avoid overflowing the
> * system specific DELAY() function(s):
>
I'll give your patch a try on Monday. I'm heading out the door
for the weekend, and the system that has problems sits in my office.
System is currently rebuilding firefox-50, which seems to have
a penchant for becoming wedged. An unkillable program consuming
~1 GB of memory. :(
--
Steve
http://troutmask.apl.washington.edu/~kargl/
2. https://www.youtube.com/watch?v=Py6d6o2jbaE
1. https://www.youtube.com/watch?v=6hwgPfCcpyQ
_______________________________________________
freebsd-current@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"
