On Wed, 2013-05-29 at 12:21 -0700, Jacob Minshall wrote:
> Max peerings capped at 63 in accordance with IEEE-2012 8.4.2.100.7,
> and trigger a beacon regeneration every time the number of peerings
> changes. 

Can you write full sentences? Maybe also in the subject ("set mesh ...
properly"?) :-)


> -     neighbors = (neighbors > 15) ? 15 : neighbors;
> +     neighbors = (neighbors > IEEE80211_MAX_MESH_PEERINGS) ?
> +                     IEEE80211_MAX_MESH_PEERINGS : neighbors;

min() or min_t()?

>       atomic_inc(&sdata->u.mesh.estab_plinks);
> -     return mesh_accept_plinks_update(sdata);
> +     mesh_accept_plinks_update(sdata);
> +     return BSS_CHANGED_BEACON;

These changes seem odd to me, why doesn't mesh_accept_plinks_update()
just return the change value then?

johannes

_______________________________________________
Devel mailing list
[email protected]
http://lists.open80211s.org/cgi-bin/mailman/listinfo/devel

Reply via email to