On Sun, Jan 31, 2016 at 12:56 AM, Bob Copeland <[email protected]> wrote: > On Fri, Jan 29, 2016 at 11:08:58AM +0100, Henning Rogge wrote: >> @@ -951,37 +974,14 @@ enddel: >> */ >> static int mpp_path_del(struct ieee80211_sub_if_data *sdata, const u8 *addr) >> { >> - struct mesh_table *tbl; >> - struct mesh_path *mpath; >> - struct mpath_node *node; >> - struct hlist_head *bucket; >> - int hash_idx; >> - int err = 0; >> - >> - /* flush relevant mpp entries first */ >> - mpp_flush_by_proxy(sdata, addr); >> - > > Is it intentional that mpp_path_del no longer calls mpp_flush_by_proxy() > while mesh_path_del does?
You just found a bug in the original patch... the "mpp_flush_by_proxy" is the function that erase all MPP entries going through a certain MPath entry... so it is wrong in the mpp_path_del() function. I will respin the series tomorrow. Henning -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
