[Differential] [Closed] D5102: hyperv/hn: Enable UDP RXCSUM

[Phabricator]

This revision was automatically updated to reflect the committed changes.
Closed by commit rS295299: hyperv/hn: Enable UDP RXCSUM (authored by sephe).

CHANGED PRIOR TO COMMIT
  https://reviews.freebsd.org/D5102?vs=12780&id=13033#toc

REPOSITORY
  rS FreeBSD src repository

CHANGES SINCE LAST UPDATE
  https://reviews.freebsd.org/D5102?vs=12780&id=13033

REVISION DETAIL
  https://reviews.freebsd.org/D5102

AFFECTED FILES
  head/sys/dev/hyperv/netvsc/hv_net_vsc.h
  head/sys/dev/hyperv/netvsc/hv_netvsc_drv_freebsd.c

CHANGE DETAILS
  diff --git a/head/sys/dev/hyperv/netvsc/hv_netvsc_drv_freebsd.c b/head/sys/dev/hyperv/netvsc/hv_netvsc_drv_freebsd.c
  --- a/head/sys/dev/hyperv/netvsc/hv_netvsc_drv_freebsd.c
  +++ b/head/sys/dev/hyperv/netvsc/hv_netvsc_drv_freebsd.c
  @@ -456,6 +456,8 @@
   	    CTLFLAG_RW, &sc->hn_csum_ip, "RXCSUM IP");
   	SYSCTL_ADD_ULONG(ctx, child, OID_AUTO, "csum_tcp",
   	    CTLFLAG_RW, &sc->hn_csum_tcp, "RXCSUM TCP");
  +	SYSCTL_ADD_ULONG(ctx, child, OID_AUTO, "csum_udp",
  +	    CTLFLAG_RW, &sc->hn_csum_udp, "RXCSUM UDP");
   	SYSCTL_ADD_ULONG(ctx, child, OID_AUTO, "csum_trusted",
   	    CTLFLAG_RW, &sc->hn_csum_trusted,
   	    "# of TCP segements that we trust host's csum verification");
  @@ -1156,20 +1158,24 @@
   	m_new->m_pkthdr.rcvif = ifp;
   
   	/* receive side checksum offload */
  -	if (NULL != csum_info) {
  +	if (csum_info != NULL) {
   		/* IP csum offload */
   		if (csum_info->receive.ip_csum_succeeded) {
   			m_new->m_pkthdr.csum_flags |=
   			    (CSUM_IP_CHECKED | CSUM_IP_VALID);
   			sc->hn_csum_ip++;
   		}
   
  -		/* TCP csum offload */
  -		if (csum_info->receive.tcp_csum_succeeded) {
  +		/* TCP/UDP csum offload */
  +		if (csum_info->receive.tcp_csum_succeeded ||
  +		    csum_info->receive.udp_csum_succeeded) {
   			m_new->m_pkthdr.csum_flags |=
   			    (CSUM_DATA_VALID | CSUM_PSEUDO_HDR);
   			m_new->m_pkthdr.csum_data = 0xffff;
  -			sc->hn_csum_tcp++;
  +			if (csum_info->receive.tcp_csum_succeeded)
  +				sc->hn_csum_tcp++;
  +			else
  +				sc->hn_csum_udp++;
   		}
   
   		if (csum_info->receive.ip_csum_succeeded &&
  diff --git a/head/sys/dev/hyperv/netvsc/hv_net_vsc.h b/head/sys/dev/hyperv/netvsc/hv_net_vsc.h
  --- a/head/sys/dev/hyperv/netvsc/hv_net_vsc.h
  +++ b/head/sys/dev/hyperv/netvsc/hv_net_vsc.h
  @@ -1036,6 +1036,7 @@
   
   	u_long		hn_csum_ip;
   	u_long		hn_csum_tcp;
  +	u_long		hn_csum_udp;
   	u_long		hn_csum_trusted;
   	u_long		hn_lro_tried;
   	u_long		hn_small_pkts;

EMAIL PREFERENCES
  https://reviews.freebsd.org/settings/panel/emailpreferences/

To: sepherosa_gmail.com, delphij, royger, decui_microsoft.com, honzhan_microsoft.com, howard0su_gmail.com, adrian, network
Cc: freebsd-net-list
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D5102.13033.patch
Type: text/x-patch
Size: 1856 bytes
Desc: not available
URL: <http://lists.freebsd.org/pipermail/freebsd-net/attachments/20160205/da4084dc/attachment.bin>