On 06/10/2016 02:08 PM, Johannes Berg wrote:
On Fri, 2016-06-10 at 12:47 -0700, Ben Greear wrote:
I see this was added sometime recently: NL80211_ATTR_PAD
If another enum member is added, should it replace the PAD enum?
No.
At the least, I think we need some comments about how this is to be
dealt with.
You simply ignore it :)
Please add a comment...it is normal behaviour to fill in 'pads' when
possible, and even when I looked up the commit message it wasn't exactly
clear what this member was for.
Even reading nla_put_u64_64bit offers no useful clues...I didn't follow
the call chain longer, probably somewhere it would start making sense,
but then again, this is netlink code, so who knows!
Thanks,
Ben
--
Ben Greear <greearb@xxxxxxxxxxxxxxx>
Candela Technologies Inc http://www.candelatech.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
