Re: Revision 309657 to stack_machdep.c renders unbootable system

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)
> _at__at_ -321,7 +321,8 _at__at_ 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):
> 
> 

John, 

The above patch fixes the boot issue for me.  Thanks for the
quick response.

-- 
Steve
http://troutmask.apl.washington.edu/~kargl/
 2. https://www.youtube.com/watch?v=Py6d6o2jbaE
 1. https://www.youtube.com/watch?v=6hwgPfCcpyQ