From: Vlad Zolotarov <vladz@cloudius-systems.com>
To: netdev@vger.kernel.org
Cc: jeffrey.t.kirsher@intel.com, avi@cloudius-systems.com,
gleb@cloudius-systems.com,
Vlad Zolotarov <vladz@cloudius-systems.com>
Subject: [PATCH net-next v7 0/7]: ixgbevf: Allow querying VFs RSS indirection table and key
Date: Tue, 24 Mar 2015 15:05:39 +0200 [thread overview]
Message-ID: <1427202346-12502-1-git-send-email-vladz@cloudius-systems.com> (raw)
Add the ethtool ops to VF driver to allow querying the RSS indirection table
and RSS Random Key.
On some devices VFs share the RSS Redirection Table and Hash Key with a PF and letting
the VF query this information may introduce some security risks. Therefore we disable this
feature by default for such devices (e.g. 82599) and allow it for those where there isn't any
possible risk (e.g. on x550). The new netdev op is going to allow a system administrator to
change the default behaviour with "ip link set" command.
- netdev: Add a new netdev op to allow/block VF from querying RSS Indirection Table and
RSS Hash Key.
- PF driver: Add new VF-PF channel commands.
- VF driver: Utilize these new commands and add the corresponding
ethtool callbacks.
New in v7:
- Add ixgbe_mbox_api_12 case in ixgbevf_set_num_queues().
- Properly expand HW RETA into the ethtool buffer.
New in v6:
- Add a proper return code when an VF query operations are blocked by PF.
- Added a required get_rxnfc callback to ixgbevf_ethtool_ops.
- Changed a description of PATCH7: set the correct ethtool options names.
New in v5:
- Added a new netdev op to allow/block VF from querying RSS Indirection Table and
RSS Hash Key.
- Let VF query the RSS info only if VF is allowed to.
New in v4:
- Forgot to run checkpatch on v3 and there were a few styling things to fix. ;)
New in v3:
- Added a missing support for x550 devices.
- Mask the indirection table values according to PSRTYPE[n].RQPL.
- Minimized the number of added VF-PF commands.
New in v2:
- Added a detailed description to patches 4 and 5.
New in v1 (compared to RFC):
- Use "if-else" statement instead of a "switch-case" for a single option case.
More specifically: in cases where the newly added API version is the only one
allowed. We may consider using a "switch-case" back again when the list of
allowed API versions in these specific places grows up.
Vlad Zolotarov (7):
if_link: Add an additional parameter to ifla_vf_info for RSS querying
ixgbe: Add a new netdev op to allow/prevent a VF from querying an RSS
info
ixgbe: Add a RETA query command to VF-PF channel API
ixgbevf: Add a RETA query code
ixgbe: Add GET_RSS_KEY command to VF-PF channel commands set
ixgbevf: Add RSS Key query code
ixgbevf: Add the appropriate ethtool ops to query RSS indirection
table and key
drivers/net/ethernet/intel/ixgbe/ixgbe.h | 1 +
drivers/net/ethernet/intel/ixgbe/ixgbe_main.c | 7 ++
drivers/net/ethernet/intel/ixgbe/ixgbe_mbx.h | 10 ++
drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c | 119 ++++++++++++++++++
drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.h | 2 +
drivers/net/ethernet/intel/ixgbevf/ethtool.c | 58 +++++++++
drivers/net/ethernet/intel/ixgbevf/ixgbevf_main.c | 5 +-
drivers/net/ethernet/intel/ixgbevf/mbx.h | 10 ++
drivers/net/ethernet/intel/ixgbevf/vf.c | 145 ++++++++++++++++++++++
drivers/net/ethernet/intel/ixgbevf/vf.h | 2 +
include/linux/if_link.h | 1 +
include/linux/netdevice.h | 8 ++
include/uapi/linux/if_link.h | 8 ++
net/core/rtnetlink.c | 32 ++++-
14 files changed, 401 insertions(+), 7 deletions(-)
--
2.1.0
next reply other threads:[~2015-03-24 13:05 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-03-24 13:05 Vlad Zolotarov [this message]
2015-03-24 13:05 ` [PATCH net-next v7 1/7] if_link: Add an additional parameter to ifla_vf_info for RSS querying Vlad Zolotarov
2015-03-24 13:05 ` [PATCH net-next v7 2/7] ixgbe: Add a new netdev op to allow/prevent a VF from querying an RSS info Vlad Zolotarov
2015-03-24 13:05 ` [PATCH net-next v7 3/7] ixgbe: Add a RETA query command to VF-PF channel API Vlad Zolotarov
2015-03-25 18:03 ` Tantilov, Emil S
2015-03-25 19:29 ` Vlad Zolotarov
[not found] ` <55130AD2.4010509@cloudius-systems.com>
2015-03-25 19:33 ` Tantilov, Emil S
2015-03-25 19:51 ` Vlad Zolotarov
2015-03-24 13:05 ` [PATCH net-next v7 4/7] ixgbevf: Add a RETA query code Vlad Zolotarov
2015-03-24 13:05 ` [PATCH net-next v7 5/7] ixgbe: Add GET_RSS_KEY command to VF-PF channel commands set Vlad Zolotarov
2015-03-24 13:05 ` [PATCH net-next v7 6/7] ixgbevf: Add RSS Key query code Vlad Zolotarov
2015-03-24 13:05 ` [PATCH net-next v7 7/7] ixgbevf: Add the appropriate ethtool ops to query RSS indirection table and key Vlad Zolotarov
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=1427202346-12502-1-git-send-email-vladz@cloudius-systems.com \
--to=vladz@cloudius-systems.com \
--cc=avi@cloudius-systems.com \
--cc=gleb@cloudius-systems.com \
--cc=jeffrey.t.kirsher@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).