netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Vinicius Costa Gomes <vinicius.gomes@intel.com>
To: "Brown\, Aaron F" <aaron.f.brown@intel.com>,
	"intel-wired-lan\@lists.osuosl.org"
	<intel-wired-lan@lists.osuosl.org>
Cc: "netdev\@vger.kernel.org" <netdev@vger.kernel.org>,
	"Sanchez-Palencia\, Jesus" <jesus.sanchez-palencia@intel.com>
Subject: RE: [Intel-wired-lan] [next-queue PATCH v5 7/9] igb: Add MAC address support for ethtool nftuple filters
Date: Tue, 27 Mar 2018 17:12:49 -0700	[thread overview]
Message-ID: <87h8p1awhq.fsf@intel.com> (raw)
In-Reply-To: <309B89C4C689E141A5FF6A0C5FB2118B8C81CD24@ORSMSX101.amr.corp.intel.com>

Hi Aaron,

"Brown, Aaron F" <aaron.f.brown@intel.com> writes:

[...]

> And watching the rx_queue counters continues to be spread across the different queues.  This is with Jeff Kirsher's  next queue, kernel 4.16.0-rc4_next-queue_dev-queue_e31d20a, which has the series of 8 igb patches applied.
>
> When I go back and run the an older build (with an earlier version of
> the series) of the same tree, 4.16.0-rc4_next-queue_dev-queue_84a3942,
> with the same procedure and same systems all the rx traffic is
> relegated to queue 0 (or whichever queue I assign it to) for either
> the src or dst filter.  Here is a sample of my counters after it had
> been running netperf_stress over the weekend:

The difference in behaviour between v4 and v5 is that v4 is configuring
(wrongly) the controller to send all the traffic directed to the
local MAC address to queue 0, v5 allows that filter to be added, but it
does nothing in reality.

I am working on a new version of this series that should work for adding
filters that involve the local MAC address. The initial use cases that I
had in mind all used MAC addresses different from the local one, but I
see that this indeed useful (and less surprising).


Thank you,

  reply	other threads:[~2018-03-28  0:12 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-03-22  0:33 [next-queue PATCH v5 0/9] igb: offloading of receive filters Vinicius Costa Gomes
2018-03-22  0:33 ` [next-queue PATCH v5 1/9] igb: Fix not adding filter elements to the list Vinicius Costa Gomes
2018-03-22  0:33 ` [next-queue PATCH v5 2/9] igb: Fix queue selection on MAC filters on i210 Vinicius Costa Gomes
2018-03-22  0:33 ` [next-queue PATCH v5 3/9] igb: Enable the hardware traffic class feature bit for igb models Vinicius Costa Gomes
2018-03-22  0:33 ` [next-queue PATCH v5 4/9] igb: Add support for MAC address filters specifying source addresses Vinicius Costa Gomes
2018-03-22  0:33 ` [next-queue PATCH v5 5/9] igb: Add support for enabling queue steering in filters Vinicius Costa Gomes
2018-03-22  0:33 ` [next-queue PATCH v5 6/9] igb: Enable nfc filters to specify MAC addresses Vinicius Costa Gomes
2018-03-22  0:33 ` [next-queue PATCH v5 7/9] igb: Add MAC address support for ethtool nftuple filters Vinicius Costa Gomes
2018-03-24  3:40   ` [Intel-wired-lan] " Brown, Aaron F
2018-03-26 23:55     ` Vinicius Costa Gomes
2018-03-27  1:40       ` Brown, Aaron F
2018-03-28  0:12         ` Vinicius Costa Gomes [this message]
2018-03-22  0:33 ` [next-queue PATCH v5 8/9] igb: Add the skeletons for tc-flower offloading Vinicius Costa Gomes
2018-03-22  0:33 ` [next-queue PATCH v5 9/9] igb: Add support for adding offloaded clsflower filters Vinicius Costa Gomes

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87h8p1awhq.fsf@intel.com \
    --to=vinicius.gomes@intel.com \
    --cc=aaron.f.brown@intel.com \
    --cc=intel-wired-lan@lists.osuosl.org \
    --cc=jesus.sanchez-palencia@intel.com \
    --cc=netdev@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).