* [dpdk-dev] [PATCH v2 1/2] net/iavf: fix default RSS field for IP fragment packets
@ 2021-08-17 6:47 Wenjun Wu
0 siblings, 0 replies; 2+ messages in thread
From: Wenjun Wu @ 2021-08-17 6:47 UTC (permalink / raw)
To: dev, jingjing.wu, beilei.xing; +Cc: Wenjun Wu
Previously IP ID field is supported in default RSS for IP fragment
packets. Actually it is not expected.
This patch removes redundant default RSS field for IP fragment packets.
The default RSS only needs to support the src and dst IP fields.
Fixes: 9e29a278bc0c ("net/iavf: support default RSS for IP fragment")
Signed-off-by: Wenjun Wu <wenjun1.wu@intel.com>
---
drivers/net/iavf/iavf_hash.c | 26 ++------------------------
1 file changed, 2 insertions(+), 24 deletions(-)
diff --git a/drivers/net/iavf/iavf_hash.c b/drivers/net/iavf/iavf_hash.c
index 2b03dad858..eba55ecea5 100644
--- a/drivers/net/iavf/iavf_hash.c
+++ b/drivers/net/iavf/iavf_hash.c
@@ -624,34 +624,12 @@ iavf_rss_hash_set(struct iavf_adapter *ad, uint64_t rss_hf, bool add)
}
if (rss_hf & ETH_RSS_FRAG_IPV4) {
- struct virtchnl_proto_hdrs hdr = {
- .tunnel_level = TUNNEL_LEVEL_OUTER,
- .count = 3,
- .proto_hdr = {
- proto_hdr_eth,
- proto_hdr_ipv4,
- {
- VIRTCHNL_PROTO_HDR_IPV4_FRAG,
- FIELD_SELECTOR(VIRTCHNL_PROTO_HDR_IPV4_FRAG_PKID),
- {BUFF_NOUSED},
- },
- },
- };
- rss_cfg.proto_hdrs = hdr;
+ rss_cfg.proto_hdrs = outer_ipv4_tmplt;
iavf_add_del_rss_cfg(ad, &rss_cfg, add);
}
if (rss_hf & ETH_RSS_FRAG_IPV6) {
- struct virtchnl_proto_hdrs hdr = {
- .tunnel_level = TUNNEL_LEVEL_OUTER,
- .count = 3,
- .proto_hdr = {
- proto_hdr_eth,
- proto_hdr_ipv6,
- proto_hdr_ipv6_frag,
- },
- };
- rss_cfg.proto_hdrs = hdr;
+ rss_cfg.proto_hdrs = outer_ipv6_tmplt;
iavf_add_del_rss_cfg(ad, &rss_cfg, add);
}
--
2.25.1
^ permalink raw reply [flat|nested] 2+ messages in thread* [dpdk-dev] [PATCH v1 1/2] net/iavf: remove redundant default RSS field for IP fragment packets
@ 2021-08-02 8:36 Wenjun Wu
2021-08-17 6:53 ` [dpdk-dev] [PATCH v2 1/2] net/iavf: fix " Wenjun Wu
0 siblings, 1 reply; 2+ messages in thread
From: Wenjun Wu @ 2021-08-02 8:36 UTC (permalink / raw)
To: dev, jingjing.wu, beilei.xing, qi.z.zhang; +Cc: Wenjun Wu
This patch removes redundant default RSS field for IP fragment packets,
only src MAC address and dst MAC address are needed.
Signed-off-by: Wenjun Wu <wenjun1.wu@intel.com>
---
drivers/net/iavf/iavf_hash.c | 26 ++------------------------
1 file changed, 2 insertions(+), 24 deletions(-)
diff --git a/drivers/net/iavf/iavf_hash.c b/drivers/net/iavf/iavf_hash.c
index 2b03dad858..eba55ecea5 100644
--- a/drivers/net/iavf/iavf_hash.c
+++ b/drivers/net/iavf/iavf_hash.c
@@ -624,34 +624,12 @@ iavf_rss_hash_set(struct iavf_adapter *ad, uint64_t rss_hf, bool add)
}
if (rss_hf & ETH_RSS_FRAG_IPV4) {
- struct virtchnl_proto_hdrs hdr = {
- .tunnel_level = TUNNEL_LEVEL_OUTER,
- .count = 3,
- .proto_hdr = {
- proto_hdr_eth,
- proto_hdr_ipv4,
- {
- VIRTCHNL_PROTO_HDR_IPV4_FRAG,
- FIELD_SELECTOR(VIRTCHNL_PROTO_HDR_IPV4_FRAG_PKID),
- {BUFF_NOUSED},
- },
- },
- };
- rss_cfg.proto_hdrs = hdr;
+ rss_cfg.proto_hdrs = outer_ipv4_tmplt;
iavf_add_del_rss_cfg(ad, &rss_cfg, add);
}
if (rss_hf & ETH_RSS_FRAG_IPV6) {
- struct virtchnl_proto_hdrs hdr = {
- .tunnel_level = TUNNEL_LEVEL_OUTER,
- .count = 3,
- .proto_hdr = {
- proto_hdr_eth,
- proto_hdr_ipv6,
- proto_hdr_ipv6_frag,
- },
- };
- rss_cfg.proto_hdrs = hdr;
+ rss_cfg.proto_hdrs = outer_ipv6_tmplt;
iavf_add_del_rss_cfg(ad, &rss_cfg, add);
}
--
2.25.1
^ permalink raw reply [flat|nested] 2+ messages in thread* [dpdk-dev] [PATCH v2 1/2] net/iavf: fix default RSS field for IP fragment packets
2021-08-02 8:36 [dpdk-dev] [PATCH v1 1/2] net/iavf: remove redundant " Wenjun Wu
@ 2021-08-17 6:53 ` Wenjun Wu
0 siblings, 0 replies; 2+ messages in thread
From: Wenjun Wu @ 2021-08-17 6:53 UTC (permalink / raw)
To: dev, jingjing.wu, beilei.xing; +Cc: Wenjun Wu
Previously IP ID field is supported in default RSS for IP fragment
packets. Actually it is not expected.
This patch removes redundant default RSS field for IP fragment packets.
The default RSS only needs to support the src and dst IP fields.
Fixes: 9e29a278bc0c ("net/iavf: support default RSS for IP fragment")
Signed-off-by: Wenjun Wu <wenjun1.wu@intel.com>
---
drivers/net/iavf/iavf_hash.c | 26 ++------------------------
1 file changed, 2 insertions(+), 24 deletions(-)
diff --git a/drivers/net/iavf/iavf_hash.c b/drivers/net/iavf/iavf_hash.c
index 2b03dad858..eba55ecea5 100644
--- a/drivers/net/iavf/iavf_hash.c
+++ b/drivers/net/iavf/iavf_hash.c
@@ -624,34 +624,12 @@ iavf_rss_hash_set(struct iavf_adapter *ad, uint64_t rss_hf, bool add)
}
if (rss_hf & ETH_RSS_FRAG_IPV4) {
- struct virtchnl_proto_hdrs hdr = {
- .tunnel_level = TUNNEL_LEVEL_OUTER,
- .count = 3,
- .proto_hdr = {
- proto_hdr_eth,
- proto_hdr_ipv4,
- {
- VIRTCHNL_PROTO_HDR_IPV4_FRAG,
- FIELD_SELECTOR(VIRTCHNL_PROTO_HDR_IPV4_FRAG_PKID),
- {BUFF_NOUSED},
- },
- },
- };
- rss_cfg.proto_hdrs = hdr;
+ rss_cfg.proto_hdrs = outer_ipv4_tmplt;
iavf_add_del_rss_cfg(ad, &rss_cfg, add);
}
if (rss_hf & ETH_RSS_FRAG_IPV6) {
- struct virtchnl_proto_hdrs hdr = {
- .tunnel_level = TUNNEL_LEVEL_OUTER,
- .count = 3,
- .proto_hdr = {
- proto_hdr_eth,
- proto_hdr_ipv6,
- proto_hdr_ipv6_frag,
- },
- };
- rss_cfg.proto_hdrs = hdr;
+ rss_cfg.proto_hdrs = outer_ipv6_tmplt;
iavf_add_del_rss_cfg(ad, &rss_cfg, add);
}
--
2.25.1
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-08-17 7:11 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-17 6:47 [dpdk-dev] [PATCH v2 1/2] net/iavf: fix default RSS field for IP fragment packets Wenjun Wu
-- strict thread matches above, loose matches on Subject: below --
2021-08-02 8:36 [dpdk-dev] [PATCH v1 1/2] net/iavf: remove redundant " Wenjun Wu
2021-08-17 6:53 ` [dpdk-dev] [PATCH v2 1/2] net/iavf: fix " Wenjun Wu
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).