On Wed, 2013-03-27 at 11:35 -0700, Ben Greear wrote: > On 03/26/2013 12:42 PM, Johannes Berg wrote: > > On Mon, 2013-03-25 at 17:57 -0700, greearb@xxxxxxxxxxxxxxx wrote: > > > >> v3: Use rcu_dereference_protected as suggested. > > > > Heh. > > > >> + some_sta = rcu_dereference_protected(sta->sdata->some_sta, 1); > > > > No no no, you're supposed to give it a proper argument like > > "lockdep_is_held(&local->sta_mtx)" > > Before I screw this up again..do you prefer that I line up > method arguments and go over 80 character line length, or > force the argument leftward to meet the 80 char limit? > > I see no good way to make it fit in 80 chars and also > line up properly. Huh, no idea. I guess I'd rather have it less than 80 cols. 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
