On Thu, 2008-02-07 at 01:53 +0100, Johannes Berg wrote:
> > + if (!is_estab_plink(hdr->addr2, rx->dev)) {
>
> This would get to be something like
>
> if (sta->flags & STA_FLAG_MESH_LINK_ESTABLISHED)
>
> right?
I probably can use just the AUTHORIZED flag.
> I suppose you should be adding sta_info structs for each mesh peer and
> set the AUTHORIZED flag on them. Is this what you were referring to in
> the patch description?
Right, and adding a pointer to a reduced mesh peer link structure with
variables only needed for mesh (it would only be allocated for mesh
interfaces).
> type == ETH_P_PAE &&
> > + if (sdata->vif.type != IEEE80211_IF_TYPE_MESH_POINT &&
> > + (!is_multicast_ether_addr(hdr.addr1) &&
> > + !(sta_flags & WLAN_STA_AUTHORIZED) &&
> > + !(ethertype == ETH_P_PAE &&
> > compare_ether_addr(dev->dev_addr,
> > skb->data + ETH_ALEN) == 0))) {
>
> That's also an artifact of not adding sta info structs, right?
Indeed, will be gone in the next round.
Btw, I will also fix all the issues I'm not commenting on in this patch
and the others. Thanks for you patience with me!
--
Luis Carlos Cobo Rus GnuPG ID: 44019B60
cozybit Inc.
-
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
