[Intel-wired-lan] [next PATCH S84-V2 13/13] i40e: fix reported mask for ntuple filters
Bowers, AndrewX
andrewx.bowers at intel.com
Thu Jan 4 22:33:24 UTC 2018
> -----Original Message-----
> From: Intel-wired-lan [mailto:intel-wired-lan-bounces at osuosl.org] On
> Behalf Of Alice Michael
> Sent: Wednesday, December 27, 2017 5:27 AM
> To: Michael, Alice <alice.michael at intel.com>; intel-wired-
> lan at lists.osuosl.org
> Subject: [Intel-wired-lan] [next PATCH S84-V2 13/13] i40e: fix reported mask
> for ntuple filters
>
> From: Jacob Keller <jacob.e.keller at intel.com>
>
> In commit 36777d9fa24c ("i40e: check current configured input set when
> adding ntuple filters") some code was added to report the input set mask for
> a given filter when reporting it to the user.
>
> This code is necessary so that the reported filter correctly displays that it is or
> is not masking certain fields.
>
> Unfortunately the code was incorrect. Development error accidentally
> swapped the mask values for the IPv4 addresses with the L4 port numbers.
> The port numbers are only 16bits wide while IPv4 addresses are 32 bits.
> Unfortunately we assigned only 16 bits to the IPv4 address masks.
> Additionally we assigned 32bit value 0xFFFFFFF to the TCP port numbers.
> This second part does not matter as the value would be truncated to 16bits
> regardless, but it is unnecessary.
>
> Fix the reported masks to properly report that the entire field is masked.
>
> Testing-hints:
> Upstream commit message will need to reword to target the equivalent
> upstream commit for the "Fixes" line.
>
> Signed-off-by: Jacob Keller <jacob.e.keller at intel.com>
> ---
> drivers/net/ethernet/intel/i40e/i40e_ethtool.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
Tested-by: Andrew Bowers <andrewx.bowers at intel.com>
More information about the Intel-wired-lan
mailing list