Date: Wed, 28 Sep 2011 13:31:45 +0300 From: Kostik Belousov <kostikbel@gmail.com> To: Adrian Chadd <adrian@freebsd.org> Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r225821 - head/sys/dev/ath Message-ID: <20110928103145.GU1511@deviant.kiev.zoral.com.ua> In-Reply-To: <201109280307.p8S37qKB052603@svn.freebsd.org> References: <201109280307.p8S37qKB052603@svn.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--hze9XmyoPo8YE0yg Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, Sep 28, 2011 at 03:07:52AM +0000, Adrian Chadd wrote: > Author: adrian > Date: Wed Sep 28 03:07:51 2011 > New Revision: 225821 > URL: http://svn.freebsd.org/changeset/base/225821 >=20 > Log: > Fix lock order to be correcter. > =20 > Nothing else locks these two queues (cabq, avp mcastq), but it should > be consistent and correct. >=20 > Modified: > head/sys/dev/ath/if_ath.c >=20 > Modified: head/sys/dev/ath/if_ath.c > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D > --- head/sys/dev/ath/if_ath.c Wed Sep 28 03:05:04 2011 (r225820) > +++ head/sys/dev/ath/if_ath.c Wed Sep 28 03:07:51 2011 (r225821) > @@ -2620,8 +2620,8 @@ ath_beacon_generate(struct ath_softc *sc > } > /* NB: gated by beacon so safe to start here */ > ath_hal_txstart(ah, cabq->axq_qnum); > - ATH_TXQ_UNLOCK(cabq); > ATH_TXQ_UNLOCK(&avp->av_mcastq); > + ATH_TXQ_UNLOCK(cabq); > } > return bf; > } Does the unlock order matter ? Unlock cannot block. I do not object against the change, rather, I am nit-picking against the commit message. --hze9XmyoPo8YE0yg Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.18 (FreeBSD) iEYEARECAAYFAk6C95EACgkQC3+MBN1Mb4gEXQCg9sTuILdWjTClNXiFH+70Pdk/ 3ToAnR/cHzUQtWvaY89CKJ9DftwVnJCZ =XjQa -----END PGP SIGNATURE----- --hze9XmyoPo8YE0yg--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20110928103145.GU1511>