Re: [net-next v2 2/5] devlink: introduce flash update overwrite mask
From: Jacob Keller <jacob.e.keller@intel.com>
Date: 2020-08-03 23:08:08
On 8/3/2020 8:38 AM, Jiri Pirko wrote:
Sat, Aug 01, 2020 at 02:21:56AM CEST, jacob.e.keller@intel.com wrote: [...]quoted
+ nla_mask = info->attrs[DEVLINK_ATTR_FLASH_UPDATE_OVERWRITE_MASK]; + if (nla_mask) { + if (!(supported_params & DEVLINK_SUPPORT_FLASH_UPDATE_OVERWRITE_MASK)) { + NL_SET_ERR_MSG_ATTR(info->extack, nla_mask, + "overwrite is not supported"); + return -EOPNOTSUPP; + } + params.overwrite_mask = nla_get_u32(nla_mask);It's a bitfield, should be NL_ATTR_TYPE_BITFIELD32.
v3 will use a bitfield, despite I don't believe it needs this. The overwrite mask provided to the driver will be the bitwise AND of the selector and the value (i.e. userspace will have to set the bit in both the selector and value to get it enabled). Thanks, Jake