Comments

From: Alexander Duyck <alexander.h.duyck@intel.com>
This change fixes several issues found in ntuple filtering while I was
doing the ATR refactor.
Specifically I updated the masks to work correctly with the latest version
of ethtool, I cleaned up the exception handling and added detailed error
output when a filter is rejected, and corrected several bits that were set
incorrectly in ixgbe_type.h.
The previous version of this patch included a printk that was left over from
me fixing the filter setup. This patch does not include that printk.
Signed-off-by: Alexander Duyck <alexander.h.duyck@intel.com>Tested-by: Stephen Ko <stephen.s.ko@intel.com>Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
---
drivers/net/ixgbe/ixgbe.h | 14 --
drivers/net/ixgbe/ixgbe_82599.c | 436 ++++++++++++-------------------------
drivers/net/ixgbe/ixgbe_ethtool.c | 134 ++++++++----
drivers/net/ixgbe/ixgbe_main.c | 21 +-
drivers/net/ixgbe/ixgbe_type.h | 16 +-
5 files changed, 250 insertions(+), 371 deletions(-)

On Thu, 2011-01-06 at 16:29 -0800, jeffrey.t.kirsher@intel.com wrote:
> From: Alexander Duyck <alexander.h.duyck@intel.com>> > This change fixes several issues found in ntuple filtering while I was> doing the ATR refactor.> > Specifically I updated the masks to work correctly with the latest version> of ethtool,
[...]
Did the previous code not correctly handle a zero value with a non-zero
mask for some fields? If so, I can revert that change to ethtool.
Ben.

On Thu, Jan 6, 2011 at 5:02 PM, Ben Hutchings <bhutchings@solarflare.com> wrote:
> On Thu, 2011-01-06 at 16:29 -0800, jeffrey.t.kirsher@intel.com wrote:>> From: Alexander Duyck <alexander.h.duyck@intel.com>>>>> This change fixes several issues found in ntuple filtering while I was>> doing the ATR refactor.>>>> Specifically I updated the masks to work correctly with the latest version>> of ethtool,> [...]>> Did the previous code not correctly handle a zero value with a non-zero> mask for some fields? If so, I can revert that change to ethtool.>> Ben.
Actually I think the ethtool mention doesn't really apply to the
in-kernel driver. I think I just carried that over from the check-in
for our out of tree driver which hadn't been updated when you added
the code to cleanup the rx ntuple filters. Also for the driver I'm
not too worried about what the status of it was before since there are
blatant errors in bit/byte ordering and mask bit values in the code
that from what I can tell would have significantly diminished the
usability of the filters.
Thanks,
Alex
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html