Date: Tue, 12 Jun 2012 13:31:32 +0000 (UTC) From: Randall Stewart <rrs@FreeBSD.org> To: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: svn commit: r236957 - head/sys/net Message-ID: <201206121331.q5CDVWLk056000@svn.freebsd.org>
next in thread | raw e-mail | index | archive | help
Author: rrs Date: Tue Jun 12 13:31:32 2012 New Revision: 236957 URL: http://svn.freebsd.org/changeset/base/236957 Log: Fix comment to better reflect how we are cheating and using the csum_data. Also fix style issues with the comments. Modified: head/sys/net/if_gif.c Modified: head/sys/net/if_gif.c ============================================================================== --- head/sys/net/if_gif.c Tue Jun 12 13:15:27 2012 (r236956) +++ head/sys/net/if_gif.c Tue Jun 12 13:31:32 2012 (r236957) @@ -385,8 +385,9 @@ keep_going: m->m_pkthdr.len -= GIF_HDR_LEN6; #endif #endif - /* Now pull back the af in packet that - * was saved in the address location. + /* + * Now pull back the af that we + * stashed in the csum_data. */ af = m->m_pkthdr.csum_data; @@ -504,9 +505,12 @@ gif_output(ifp, m, dst, ro) dst->sa_family = af; } af = dst->sa_family; - /* Now save the af in the inbound pkt csum - * data, this is a cheat since really - * gif tunnels don't do offload. + /* + * Now save the af in the inbound pkt csum + * data, this is a cheat since we are using + * the inbound csum_data field to carry the + * af over to the gif_start() routine, avoiding + * using yet another mtag. */ m->m_pkthdr.csum_data = af; if (!(ifp->if_flags & IFF_UP) || @@ -516,7 +520,8 @@ gif_output(ifp, m, dst, ro) goto end; } #ifdef ALTQ - /* Make altq aware of the bytes we will add + /* + * Make altq aware of the bytes we will add * when we actually send it. */ #ifdef INET
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201206121331.q5CDVWLk056000>