On Sat, 2013-06-29 at 15:58 -0700, greearb@xxxxxxxxxxxxxxx wrote: > From: Ben Greear <greearb@xxxxxxxxxxxxxxx> > > Add sdata hash (based on sdata->vif.addr) to local > structure. > > Add sta_vhash (based on sta->sta.addr) to sdata struct. > > Make STA_HASH give a better hash spread more often. > > Use new hashes where we can. Might be able to completely > get rid of the local->sta_hash, but didn't want to try that > quite yet. > > This significantly improves performance when using lots > of station VIFs connected to the same AP. It will likely > help other cases where the old hash logic failed to create > a decent spread. I think this is too much code for a corner case unlikely to happen outside of your specific scenario, so I'm not taking this either. I also don't like maintaining two separate hash tables and all that. I'd reconsider if you actually remove the hash entirely, but that'll be tricky to walk the station list and will quite possibly make the RX path there more expensive? johannes -- 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
