On Fri, Oct 19, 2012 at 6:50 AM, Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote: > On Tue, 2012-10-09 at 13:27 -0700, Ashok Nagarajan wrote: > >> /** >> + * struct mesh_stats - 802.11s mesh statistics >> + * >> + * Used to provide statistical information about mesh >> + * >> + * @fwded_mcast: Mesh forwarded multicast frames >> + * @fwded_unicast: Mesh forwarded unicast frames >> + * @fwded_frames: Mesh total forwarded frames >> + * @dropped_frames_ttl: Not transmitted since mesh_ttl == 0 >> + * @dropped_frames_no_route: Not transmitted, no route found >> + * @dropped_frames_congestion: Not forwarded due to congestion >> + */ >> +struct mesh_stats { >> + u32 fwded_mcast; > > Would it make sense to use u64 right away? A lot of netdev stats were at > some point extended from u32 to u64 and that caused problems, maybe > using u64 right away here would be good? Definitely. Javier -- Javier Cardona cozybit Inc. http://www.cozybit.com -- 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