From: Tony Nguyen <anthony.l.nguyen@intel.com>
To: Mateusz Palczewski <mateusz.palczewski@intel.com>,
<intel-wired-lan@lists.osuosl.org>
Cc: Michal Jaron <michalx.jaron@intel.com>
Subject: Re: [Intel-wired-lan] [PATCH net-next v2 0/2] i40e: Fix ethtool rx-flow-hash setting for X722
Date: Tue, 12 Jul 2022 15:37:37 -0700 [thread overview]
Message-ID: <841bf24b-51dc-2ef1-895b-14466a4724b2@intel.com> (raw)
In-Reply-To: <20220712141653.2590868-1-mateusz.palczewski@intel.com>
On 7/12/2022 7:16 AM, Mateusz Palczewski wrote:
> From: Michal Jaron <michalx.jaron@intel.com>
>
> When enabling flow type for RSS hash via ethtool:
>
> ethtool -N $pf rx-flow-hash tcp4|tcp6|udp4|udp6 s|d
>
> the driver would fail to setup this setting on X722
> device since it was using the mask on the register
> dedicated for X710 devices.
>
> Implement a bitmap to collect the flow pc types that
> shall be applied on the inset and hena registers.
> Apply a different mask on the register when setting the
> RSS hash for the X722 device.
This should be split. Patch one should go to net and patch two to -next
as one is a bug fix and one is a refactor.
Also, both patches have the same issues:
ERROR: Unrecognized email address: 'Slawomir Laba slawomirx.laba@intel.com'
Based on sign-off ordering, I would expect Slawomir to be the author,
however, it's currently listed as Michal. Please either fix ordering to
have Michal first if he is the author or change the author to be Slawomir.
> Michal Jaron (2):
> i40e: Fix ethtool rx-flow-hash setting for X722
> i40e: Refactor i40e_set_rss_hash_opt
>
> .../net/ethernet/intel/i40e/i40e_ethtool.c | 105 +++++++++++-------
> drivers/net/ethernet/intel/i40e/i40e_type.h | 4 +
> 2 files changed, 69 insertions(+), 40 deletions(-)
>
_______________________________________________
Intel-wired-lan mailing list
Intel-wired-lan@osuosl.org
https://lists.osuosl.org/mailman/listinfo/intel-wired-lan
prev parent reply other threads:[~2022-07-12 22:37 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-07-12 14:16 [Intel-wired-lan] [PATCH net-next v2 0/2] i40e: Fix ethtool rx-flow-hash setting for X722 Mateusz Palczewski
2022-07-12 14:16 ` [Intel-wired-lan] [PATCH net-next v2 1/2] " Mateusz Palczewski
2022-07-12 14:16 ` [Intel-wired-lan] [PATCH net-next v2 2/2] i40e: Refactor i40e_set_rss_hash_opt Mateusz Palczewski
2022-07-12 22:37 ` Tony Nguyen [this message]
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=841bf24b-51dc-2ef1-895b-14466a4724b2@intel.com \
--to=anthony.l.nguyen@intel.com \
--cc=intel-wired-lan@lists.osuosl.org \
--cc=mateusz.palczewski@intel.com \
--cc=michalx.jaron@intel.com \
/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