Re: svn commit: r325734 - head/sys/amd64/amd64

2017-11-11 Thread Bruce Evans
On Sun, 12 Nov 2017, Bruce Evans wrote: On Sun, 12 Nov 2017, Mateusz Guzik wrote: Log: amd64: stop nesting preemption counter in spinlock_enter Discussed with:jhb This seems to break it. i386 still has the old code. ... The main broken case is: - both levels initially 0 -

Re: svn commit: r325734 - head/sys/amd64/amd64

2017-11-11 Thread Bruce Evans
On Sun, 12 Nov 2017, Mateusz Guzik wrote: Log: amd64: stop nesting preemption counter in spinlock_enter Discussed with:jhb This seems to break it. i386 still has the old code. It also moves the critical section a little, so that it is inconsistent for enter/exit. I think the

svn commit: r325734 - head/sys/amd64/amd64

2017-11-11 Thread Mateusz Guzik
Author: mjg Date: Sun Nov 12 03:13:01 2017 New Revision: 325734 URL: https://svnweb.freebsd.org/changeset/base/325734 Log: amd64: stop nesting preemption counter in spinlock_enter Discussed with: jhb Modified: head/sys/amd64/amd64/machdep.c Modified: