Hi, Javier Will do.
Chun-Yeow On Fri, May 4, 2012 at 2:17 PM, Javier Cardona <[email protected]> wrote: > Hi Yeoh, > > > On Friday, May 4, 2012, Chun-Yeow Yeoh wrote: >> >> Forwarded PREQ is either unicast or multicast. The appropriate counters >> should be incremented accordingly. >> >> Signed-off-by: Chun-Yeow Yeoh <[email protected]> >> --- >> net/mac80211/mesh_hwmp.c | 5 ++++- >> 1 files changed, 4 insertions(+), 1 deletions(-) >> >> diff --git a/net/mac80211/mesh_hwmp.c b/net/mac80211/mesh_hwmp.c >> index 503016f..f0695e5 100644 >> --- a/net/mac80211/mesh_hwmp.c >> +++ b/net/mac80211/mesh_hwmp.c >> @@ -603,7 +603,10 @@ static void hwmp_preq_frame_process(struct >> ieee80211_sub_if_data *sdata, >> hopcount, ttl, cpu_to_le32(lifetime), >> cpu_to_le32(metric), cpu_to_le32(preq_id), >> sdata); >> - ifmsh->mshstats.fwded_mcast++; >> + if (da != broadcast_addr) >> + ifmsh->mshstats.fwded_unicast++; >> + else >> + ifmsh->mshstats.fwded_mcast++; >> ifmsh->mshstats.fwded_frames++; >> } >> } >> -- >> 1.7.0.4 > > > I think you should check for multicast, not just bcast. And use the > existing helper function to do so. > > Javier > > > -- > Javier Cardona > cozybit Inc. > http://www.cozybit.com _______________________________________________ Devel mailing list [email protected] http://lists.open80211s.org/cgi-bin/mailman/listinfo/devel
