svn commit: r216173 - head/sys/dev/e1000

Jack F Vogel jfv at FreeBSD.org
Sat Dec 4 02:04:03 UTC 2010


Author: jfv
Date: Sat Dec  4 02:04:02 2010
New Revision: 216173
URL: http://svn.freebsd.org/changeset/base/216173

Log:
  Remove the bogus test in the TX context setup for IPV6,
  the size can be smaller than the constant when you are
  doing HW TAGGING, and you still need to process this
  packet in a normal way. I'm not sure where the notion
  to just return came from, but its wrong.
  
  MFC after: 3 days

Modified:
  head/sys/dev/e1000/if_igb.c

Modified: head/sys/dev/e1000/if_igb.c
==============================================================================
--- head/sys/dev/e1000/if_igb.c	Sat Dec  4 01:59:58 2010	(r216172)
+++ head/sys/dev/e1000/if_igb.c	Sat Dec  4 02:04:02 2010	(r216173)
@@ -3429,8 +3429,6 @@ igb_tx_ctx_setup(struct tx_ring *txr, st
 		case ETHERTYPE_IPV6:
 			ip6 = (struct ip6_hdr *)(mp->m_data + ehdrlen);
 			ip_hlen = sizeof(struct ip6_hdr);
-			if (mp->m_len < ehdrlen + ip_hlen)
-				return (FALSE);
 			ipproto = ip6->ip6_nxt;
 			type_tucmd_mlhl |= E1000_ADVTXD_TUCMD_IPV6;
 			break;


More information about the svn-src-head mailing list