Hi, Johannes No. I think that I am actually fixing the wrong assignment of the previous patch. if (_chg_mesh_attr(NL80211_MESHCONF_ELEMENT_TTL, mask)) conf->dot11MeshTTL = nconf->element_ttl; to conf->element_ttl = nconf->element_ttl; Regards, Chun-Yeow On Fri, Jun 15, 2012 at 4:20 PM, Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote: > On Fri, 2012-06-15 at 08:32 +0800, Yeoh Chun-Yeow wrote: >> Yes. >> >> On Fri, Jun 15, 2012 at 2:57 AM, Johannes Berg >> <johannes@xxxxxxxxxxxxxxxx> wrote: >> > On Fri, 2012-06-15 at 00:23 +0800, Chun-Yeow Yeoh wrote: >> >> This patch fixes the wrong assignment of mesh element TTL. >> > >> > Since I haven't pushed to John yet, should I rebase and merge this with >> > one of the previous patches? > > Umm, I actually shouldn't ... The patch this fixes has been merged by > John way back: > > commit 45904f21655cf4f0ae7d0fab5906fe51bf56ecf4 > Author: Javier Cardona <javier@xxxxxxxxxxx> > AuthorDate: Fri Dec 3 09:20:40 2010 +0100 > Commit: John W. Linville <linville@xxxxxxxxxxxxx> > CommitDate: Mon Dec 6 16:01:28 2010 -0500 > > nl80211/mac80211: define and allow configuring mesh element TTL > > > WAY WAY back. Maybe this patch should go into 3.5 and even stable? > > johannes > -- 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