Commit f995f95a authored by Haiyue Wang's avatar Haiyue Wang Committed by Tony Nguyen

iavf: change the flex-byte support number to macro definition

The maximum number (2) of flex-byte support is derived from ethtool
use-def data size (8 byte).

Change the magic number 2 to macro definition, and add the comment to
track the design thinking, so the code is clear and easily maintained.
Signed-off-by: default avatarHaiyue Wang <haiyue.wang@intel.com>
Signed-off-by: default avatarTony Nguyen <anthony.l.nguyen@intel.com>
parent 1a0e880b
...@@ -939,7 +939,7 @@ iavf_parse_rx_flow_user_data(struct ethtool_rx_flow_spec *fsp, ...@@ -939,7 +939,7 @@ iavf_parse_rx_flow_user_data(struct ethtool_rx_flow_spec *fsp,
if (!(fsp->flow_type & FLOW_EXT)) if (!(fsp->flow_type & FLOW_EXT))
return 0; return 0;
for (i = 0; i < 2; i++) { for (i = 0; i < IAVF_FLEX_WORD_NUM; i++) {
#define IAVF_USERDEF_FLEX_WORD_M GENMASK(15, 0) #define IAVF_USERDEF_FLEX_WORD_M GENMASK(15, 0)
#define IAVF_USERDEF_FLEX_OFFS_S 16 #define IAVF_USERDEF_FLEX_OFFS_S 16
#define IAVF_USERDEF_FLEX_OFFS_M GENMASK(31, IAVF_USERDEF_FLEX_OFFS_S) #define IAVF_USERDEF_FLEX_OFFS_M GENMASK(31, IAVF_USERDEF_FLEX_OFFS_S)
......
...@@ -35,6 +35,11 @@ enum iavf_fdir_flow_type { ...@@ -35,6 +35,11 @@ enum iavf_fdir_flow_type {
IAVF_FDIR_FLOW_PTYPE_MAX, IAVF_FDIR_FLOW_PTYPE_MAX,
}; };
/* Must not exceed the array element number of '__be32 data[2]' in the ethtool
* 'struct ethtool_rx_flow_spec.m_ext.data[2]' to express the flex-byte (word).
*/
#define IAVF_FLEX_WORD_NUM 2
struct iavf_flex_word { struct iavf_flex_word {
u16 offset; u16 offset;
u16 word; u16 word;
...@@ -71,7 +76,7 @@ struct iavf_fdir_ip { ...@@ -71,7 +76,7 @@ struct iavf_fdir_ip {
}; };
struct iavf_fdir_extra { struct iavf_fdir_extra {
u32 usr_def[2]; u32 usr_def[IAVF_FLEX_WORD_NUM];
}; };
/* bookkeeping of Flow Director filters */ /* bookkeeping of Flow Director filters */
...@@ -95,7 +100,7 @@ struct iavf_fdir_fltr { ...@@ -95,7 +100,7 @@ struct iavf_fdir_fltr {
/* flex byte filter data */ /* flex byte filter data */
u8 ip_ver; /* used to adjust the flex offset, 4 : IPv4, 6 : IPv6 */ u8 ip_ver; /* used to adjust the flex offset, 4 : IPv4, 6 : IPv6 */
u8 flex_cnt; u8 flex_cnt;
struct iavf_flex_word flex_words[2]; struct iavf_flex_word flex_words[IAVF_FLEX_WORD_NUM];
u32 flow_id; u32 flow_id;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment