Re: svn commit: r225821 - head/sys/dev/ath

2011-09-28 Thread Adrian Chadd
On 28 September 2011 18:31, Kostik Belousov wrote: > Does the unlock order matter ? Unlock cannot block. > > I do not object against the change, rather, I am nit-picking against the > commit message. Nope it doesn't. It's just a cosmetic change. Adrian _

Re: svn commit: r225821 - head/sys/dev/ath

2011-09-28 Thread Kostik Belousov
On Wed, Sep 28, 2011 at 03:07:52AM +, Adrian Chadd wrote: > Author: adrian > Date: Wed Sep 28 03:07:51 2011 > New Revision: 225821 > URL: http://svn.freebsd.org/changeset/base/225821 > > Log: > Fix lock order to be correcter. > > Nothing else locks these two queues (cabq, avp mcastq), b

svn commit: r225821 - head/sys/dev/ath

2011-09-27 Thread Adrian Chadd
Author: adrian Date: Wed Sep 28 03:07:51 2011 New Revision: 225821 URL: http://svn.freebsd.org/changeset/base/225821 Log: Fix lock order to be correcter. Nothing else locks these two queues (cabq, avp mcastq), but it should be consistent and correct. Modified: head/sys/dev/ath/if_ath.c