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