From: "Xia, Chenbo" <chenbo.xia@intel.com>
To: Maxime Coquelin <maxime.coquelin@redhat.com>,
"dev@dpdk.org" <dev@dpdk.org>,
"jasowang@redhat.com" <jasowang@redhat.com>,
"david.marchand@redhat.com" <david.marchand@redhat.com>,
"Matz, Olivier" <olivier.matz@6wind.com>,
"Ma, WenwuX" <wenwux.ma@intel.com>,
"Zhang, Yuying" <yuying.zhang@intel.com>,
"Singh, Aman Deep" <aman.deep.singh@intel.com>
Subject: RE: [PATCH v2 6/6] net/vhost: perform SW checksum in Tx path
Date: Thu, 9 Jun 2022 02:26:19 +0000 [thread overview]
Message-ID: <SN6PR11MB35040E403C71B29C2374B81C9CA79@SN6PR11MB3504.namprd11.prod.outlook.com> (raw)
In-Reply-To: <20220608124946.102623-7-maxime.coquelin@redhat.com>
Hi Maxime,
> -----Original Message-----
> From: Maxime Coquelin <maxime.coquelin@redhat.com>
> Sent: Wednesday, June 8, 2022 8:50 PM
> To: dev@dpdk.org; jasowang@redhat.com; Xia, Chenbo <chenbo.xia@intel.com>;
> david.marchand@redhat.com; Matz, Olivier <olivier.matz@6wind.com>; Ma,
> WenwuX <wenwux.ma@intel.com>; Zhang, Yuying <yuying.zhang@intel.com>;
> Singh, Aman Deep <aman.deep.singh@intel.com>
> Cc: Maxime Coquelin <maxime.coquelin@redhat.com>
> Subject: [PATCH v2 6/6] net/vhost: perform SW checksum in Tx path
>
> Virtio specification supports guest checksum offloading
> for L4, which is enabled with VIRTIO_NET_F_GUEST_CSUM
> feature negotiation. However, the Vhost PMD does not
> advertise Tx checksum offload capabilities.
>
> Advertising these offload capabilities at the ethdev level
> is not enough, because we could still end-up with the
> application enabling these offloads while the guest not
> negotiating it.
>
> This patch advertises the Tx checksum offload capabilities,
> and introduces a compatibility layer to cover the case
> VIRTIO_NET_F_GUEST_CSUM has not been negotiated but the
> application does configure the Tx checksum offloads. This
> function performs the L4 Tx checksum in SW for UDP and TCP.
> Compared to Rx SW checksum, the Tx SW checksum function
> needs to compute the pseudo-header checksum, as we cannot
> know whether it was done before.
>
> This patch does not advertise SCTP checksum offloading
> capability for now, but it could be handled later if the
> need arises.
>
> Reported-by: Jason Wang <jasowang@redhat.com>
> Signed-off-by: Maxime Coquelin <maxime.coquelin@redhat.com>
> ---
> drivers/net/vhost/rte_eth_vhost.c | 62 +++++++++++++++++++++++++++++++
> 1 file changed, 62 insertions(+)
>
> diff --git a/drivers/net/vhost/rte_eth_vhost.c
> b/drivers/net/vhost/rte_eth_vhost.c
> index 42f0d52ebc..d75d256040 100644
> --- a/drivers/net/vhost/rte_eth_vhost.c
> +++ b/drivers/net/vhost/rte_eth_vhost.c
> @@ -92,6 +92,7 @@ struct pmd_internal {
> rte_atomic32_t started;
> bool vlan_strip;
> bool rx_sw_csum;
> + bool tx_sw_csum;
> };
>
> struct internal_list {
> @@ -283,8 +284,10 @@ vhost_dev_csum_configure(struct rte_eth_dev *eth_dev)
> {
> struct pmd_internal *internal = eth_dev->data->dev_private;
> const struct rte_eth_rxmode *rxmode = ð_dev->data-
> >dev_conf.rxmode;
> + const struct rte_eth_txmode *txmode = ð_dev->data-
> >dev_conf.txmode;
>
> internal->rx_sw_csum = false;
> + internal->tx_sw_csum = false;
>
> /* SW checksum is not compatible with legacy mode */
> if (!(internal->flags & RTE_VHOST_USER_NET_COMPLIANT_OL_FLAGS))
> @@ -297,6 +300,56 @@ vhost_dev_csum_configure(struct rte_eth_dev *eth_dev)
> internal->rx_sw_csum = true;
> }
> }
> +
> + if (!(internal->features & (1ULL << VIRTIO_NET_F_GUEST_CSUM))) {
> + if (txmode->offloads &
> + (RTE_ETH_TX_OFFLOAD_UDP_CKSUM |
> RTE_ETH_TX_OFFLOAD_TCP_CKSUM)) {
> + VHOST_LOG(NOTICE, "Tx csum will be done in SW, may
> impact performance.");
Missing \n
With above fixed:
Reviewed-by: Chenbo Xia <chenbo.xia@intel.com>
> + internal->tx_sw_csum = true;
> + }
> + }
> +}
> +
> +static void
> +vhost_dev_tx_sw_csum(struct rte_mbuf *mbuf)
> +{
> + uint32_t hdr_len;
> + uint16_t csum = 0, csum_offset;
> +
> + switch (mbuf->ol_flags & RTE_MBUF_F_TX_L4_MASK) {
> + case RTE_MBUF_F_TX_L4_NO_CKSUM:
> + return;
> + case RTE_MBUF_F_TX_TCP_CKSUM:
> + csum_offset = offsetof(struct rte_tcp_hdr, cksum);
> + break;
> + case RTE_MBUF_F_TX_UDP_CKSUM:
> + csum_offset = offsetof(struct rte_udp_hdr, dgram_cksum);
> + break;
> + default:
> + /* Unsupported packet type. */
> + return;
> + }
> +
> + hdr_len = mbuf->l2_len + mbuf->l3_len;
> + csum_offset += hdr_len;
> +
> + /* Prepare the pseudo-header checksum */
> + if (rte_net_intel_cksum_prepare(mbuf) < 0)
> + return;
> +
> + if (rte_raw_cksum_mbuf(mbuf, hdr_len, rte_pktmbuf_pkt_len(mbuf) -
> hdr_len, &csum) < 0)
> + return;
> +
> + csum = ~csum;
> + /* See RFC768 */
> + if (unlikely((mbuf->packet_type & RTE_PTYPE_L4_UDP) && csum == 0))
> + csum = 0xffff;
> +
> + if (rte_pktmbuf_data_len(mbuf) >= csum_offset + 1)
> + *rte_pktmbuf_mtod_offset(mbuf, uint16_t *, csum_offset) = csum;
> +
> + mbuf->ol_flags &= ~RTE_MBUF_F_TX_L4_MASK;
> + mbuf->ol_flags |= RTE_MBUF_F_TX_L4_NO_CKSUM;
> }
>
> static void
> @@ -423,6 +476,10 @@ eth_vhost_tx(void *q, struct rte_mbuf **bufs,
> uint16_t nb_bufs)
> }
> }
>
> + if (r->internal->tx_sw_csum)
> + vhost_dev_tx_sw_csum(m);
> +
> +
> bufs[nb_send] = m;
> ++nb_send;
> }
> @@ -1267,6 +1324,11 @@ eth_dev_info(struct rte_eth_dev *dev,
>
> dev_info->tx_offload_capa = RTE_ETH_TX_OFFLOAD_MULTI_SEGS |
> RTE_ETH_TX_OFFLOAD_VLAN_INSERT;
> + if (internal->flags & RTE_VHOST_USER_NET_COMPLIANT_OL_FLAGS) {
> + dev_info->tx_offload_capa |= RTE_ETH_TX_OFFLOAD_UDP_CKSUM |
> + RTE_ETH_TX_OFFLOAD_TCP_CKSUM;
> + }
> +
> dev_info->rx_offload_capa = RTE_ETH_RX_OFFLOAD_VLAN_STRIP;
> if (internal->flags & RTE_VHOST_USER_NET_COMPLIANT_OL_FLAGS) {
> dev_info->rx_offload_capa |= RTE_ETH_RX_OFFLOAD_UDP_CKSUM |
> --
> 2.35.3
next prev parent reply other threads:[~2022-06-09 2:27 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-06-08 12:49 [PATCH v2 0/6] Vhost checksum offload improvements Maxime Coquelin
2022-06-08 12:49 ` [PATCH v2 1/6] Revert "app/testpmd: modify mac in csum forwarding" Maxime Coquelin
2022-06-13 12:24 ` Singh, Aman Deep
2022-06-17 12:42 ` Maxime Coquelin
2022-06-08 12:49 ` [PATCH v2 2/6] vhost: fix missing enqueue pseudo-header calculation Maxime Coquelin
2022-06-08 12:49 ` [PATCH v2 3/6] net/vhost: enable compliant offloading mode Maxime Coquelin
2022-06-08 12:49 ` [PATCH v2 4/6] net/vhost: make VLAN stripping flag a boolean Maxime Coquelin
2022-06-08 12:49 ` [PATCH v2 5/6] net/vhost: perform SW checksum in Rx path Maxime Coquelin
2022-06-09 1:59 ` Xia, Chenbo
2022-06-10 3:49 ` Xia, Chenbo
2022-06-10 7:19 ` Jiang, Cheng1
2022-06-08 12:49 ` [PATCH v2 6/6] net/vhost: perform SW checksum in Tx path Maxime Coquelin
2022-06-09 2:26 ` Xia, Chenbo [this message]
2022-06-10 3:49 ` Xia, Chenbo
2022-06-10 7:31 ` Jiang, Cheng1
2022-06-10 3:50 ` Xia, Chenbo
2022-06-17 14:06 ` [PATCH v2 0/6] Vhost checksum offload improvements Maxime Coquelin
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=SN6PR11MB35040E403C71B29C2374B81C9CA79@SN6PR11MB3504.namprd11.prod.outlook.com \
--to=chenbo.xia@intel.com \
--cc=aman.deep.singh@intel.com \
--cc=david.marchand@redhat.com \
--cc=dev@dpdk.org \
--cc=jasowang@redhat.com \
--cc=maxime.coquelin@redhat.com \
--cc=olivier.matz@6wind.com \
--cc=wenwux.ma@intel.com \
--cc=yuying.zhang@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;
as well as URLs for NNTP newsgroup(s).