Re: svn commit: r247068 - head/sys/x86/isa

2013-02-21 Thread Bruce Evans
On Thu, 21 Feb 2013, Warner Losh wrote: Log: Fix broken usage of splhigh() by removing it. This is more broken than before. The splhigh() served to indicate missing locking. Modified: head/sys/x86/isa/atrtc.c ==

Re: svn commit: r247068 - head/sys/x86/isa

2013-02-21 Thread Warner Losh
On Feb 21, 2013, at 5:31 AM, Bruce Evans wrote: On Thu, 21 Feb 2013, Warner Losh wrote: Log: Fix broken usage of splhigh() by removing it. This is more broken than before. The splhigh() served to indicate missing locking. Depends on what you mean by more :) It is less depessimized

Re: svn commit: r247068 - head/sys/x86/isa

2013-02-21 Thread Bruce Evans
On Thu, 21 Feb 2013, Warner Losh wrote: On Feb 21, 2013, at 5:31 AM, Bruce Evans wrote: On Thu, 21 Feb 2013, Warner Losh wrote: Log: Fix broken usage of splhigh() by removing it. This is more broken than before. The splhigh() served to indicate missing locking. Depends on what you mean

svn commit: r247068 - head/sys/x86/isa

2013-02-20 Thread Warner Losh
Author: imp Date: Thu Feb 21 00:40:08 2013 New Revision: 247068 URL: http://svnweb.freebsd.org/changeset/base/247068 Log: Fix broken usage of splhigh() by removing it. Modified: head/sys/x86/isa/atrtc.c Modified: head/sys/x86/isa/atrtc.c