On Sun, Jan 31, 2016 at 12:56 AM, Bob Copeland <me@xxxxxxxxxxxxxxx> 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 majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
