On 4/1/2017 8:28 AM, Lu, Wenzhuo wrote: > Hi, > >> -----Original Message----- >> From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Zhang, Helin <...> >>> -----Original Message----- >>> From: Thomas Monjalon [mailto:thomas.monja...@6wind.com] <...> >>> 2017-03-20 16:04, Yongseok Koh: >>>> In rte_eth_check_reta_mask(), it is required to align the size of >>>> the RETA table to RTE_RETA_GROUP_SIZE but as the size can be less >>>> than the limit, this should be removed. The change is also applied >>>> to a command of >>> testpmd. >>>> >>>> Signed-off-by: Yongseok Koh <ys...@mellanox.com>
<...> >>> There is no comment for this constraint neither in the code nor in the >>> commit: http://dpdk.org/commit/66c594904 So, I guess it can be removed. >>> If a check is needed, it could be added in the relevant drivers. >>> >>> Helin, Konstantin, please check for Intel drivers. >> Hi Thomas >> >> Thank you very much for the reminder! >> We will check that and see if there is any impacts to Intel drivers. > I don't think it has any impact to the drivers. > To my opinion, it's a good fix as it makes the name ' reta_size' more > reasonable. Series applied to dpdk-next-net/master, thanks.
