Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 07 Apr 2010 18:42:24 +0100
From:      Gavin Atkinson <gavin@FreeBSD.org>
To:        Rui Paulo <rpaulo@FreeBSD.org>
Cc:        svn-src-head@FreeBSD.org, svn-src-all@FreeBSD.org, src-committers@FreeBSD.org
Subject:   Re: svn commit: r206358 - in head/sys: conf dev/bwi dev/bwn dev/iwn dev/ral dev/usb/wlan dev/wpi modules/wlan net80211
Message-ID:  <1270662144.74915.108.camel@buffy.york.ac.uk>
In-Reply-To: <201004071529.o37FTDi0035619@svn.freebsd.org>
References:  <201004071529.o37FTDi0035619@svn.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, 2010-04-07 at 15:29 +0000, Rui Paulo wrote:
> Author: rpaulo
> Date: Wed Apr  7 15:29:13 2010
> New Revision: 206358
> URL: http://svn.freebsd.org/changeset/base/206358
>=20
> Log:
>   net80211 rate control framework (net80211 ratectl).

This looks to be great work!  Thanks for doing this.

[...]

>   MFC after:    1 months

The changes change KBI - I don't think it can be MFC'd as-is?
(especially the sys/net80211/ieee80211_var.h changes)

Also...

> @@ -3393,8 +3368,8 @@ _bwi_txeof(struct bwi_softc *sc, uint16_
>  	bus_dmamap_unload(sc->sc_buf_dtag, tb->tb_dmap);
> =20
>  	ni =3D tb->tb_ni;
> +	vap =3D ni->ni_vap;
>  	if (tb->tb_ni !=3D NULL) {
> -		struct bwi_node *bn =3D (struct bwi_node *) tb->tb_ni;

If (tb->tb_ni) can indeed be NULL here, we'll now panic. If not, there's
no need for the conditional.

> @@ -921,13 +900,12 @@ rt2661_tx_intr(struct rt2661_softc *sc)
>  		data->m =3D NULL;
>  		ni =3D data->ni;
>  		data->ni =3D NULL;
> +		vap =3D ni->ni_vap;
> =20
>  		/* if no frame has been sent, ignore */
>  		if (ni =3D=3D NULL)
>  			continue;
> =20
> -		rn =3D RT2661_NODE(ni);
> -

Same here.  Either we'll panic, or the test for (ni =3D=3D NULL) isn't
needed.

> @@ -1022,6 +1008,7 @@ rt2560_tx_intr(struct rt2560_softc *sc)
>                 data->m =3D NULL;
>                 ieee80211_free_node(data->ni);
>                 data->ni =3D NULL;
> +               ni =3D NULL;
>=20
>                 /* descriptor is no longer valid */
>                 desc->flags &=3D ~htole32(RT2560_TX_VALID);

This seems unnecessary - we never read "ni" again.  Either we fall out
of the loop and exit the subroutine, or we go round the loop again and
overwrite it.

> Index: head/sys/dev/usb/wlan/if_urtw.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
> --- head/sys/dev/usb/wlan/if_urtw.c     (revision 206357)
> +++ head/sys/dev/usb/wlan/if_urtw.c     (revision 206358)
> @@ -62,6 +62,7 @@ __FBSDID("$FreeBSD$");
>  #include <dev/usb/wlan/if_urtwvar.h>
>=20
>  SYSCTL_NODE(_hw_usb, OID_AUTO, urtw, CTLFLAG_RW, 0, "USB Realtek 8187L")=
;
> +#define URTW_DEBUG
>  #ifdef URTW_DEBUG
>  int urtw_debug =3D 0;
> SYSCTL_INT(_hw_usb_urtw, OID_AUTO, debug, CTLFLAG_RW, &urtw_debug, 0,

Is this change intentional?

Thanks,

Gavin



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1270662144.74915.108.camel>