Re: [PATCH net-next v5 07/22] ethtool: netlink bitset handling

From: Jiri Pirko
Date: Tue Mar 26 2019 - 11:59:16 EST

Mon, Mar 25, 2019 at 06:08:15PM CET, mkubecek@xxxxxxx wrote:
>Declare attribute type constants and add helper functions to generate and
>parse arbitrary length bit sets.

Hmm, this looks like a lot of work. Two things:
1) This is generic. Not really related to ethtool in any way. Could this
be done in netlink common code?
2) Did you think about leveraging NLA_BITFIELD32? What I mean is this:
NLA_BITFIELD32 index 0 (bit 0-31)
NLA_BITFIELD32 index 1 (bit 32-63)
NLA_BITFIELD32 index 2 (bit 64-95)
NLA_BITFIELD32 index 3 (bit 96-127)
It is basically an array of NLA_BITFIELD32.