<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:x="urn:schemas-microsoft-com:office:excel" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
<meta name="Generator" content="Microsoft Word 12 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
        {font-family:"Cambria Math";
        panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
        {font-family:Calibri;
        panose-1:2 15 5 2 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
        {margin:0in;
        margin-bottom:.0001pt;
        font-size:11.0pt;
        font-family:"Calibri","sans-serif";}
a:link, span.MsoHyperlink
        {mso-style-priority:99;
        color:blue;
        text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
        {mso-style-priority:99;
        color:purple;
        text-decoration:underline;}
span.EmailStyle17
        {mso-style-type:personal-compose;
        font-family:"Calibri","sans-serif";
        color:windowtext;}
.MsoChpDefault
        {mso-style-type:export-only;}
@page WordSection1
        {size:8.5in 11.0in;
        margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
        {page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang="EN-US" link="blue" vlink="purple">
<div class="WordSection1">
<p class="MsoNormal">On an ip header there are 3 bits flags and 13 bits of offset.<o:p></o:p></p>
<p class="MsoNormal">We used to look at them as one word, which cause us not to use flags well.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">The following code fixes this.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Thanks<o:p></o:p></p>
<p class="MsoNormal">Tzachi<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Index: inc/kernel/ip_packet.h<o:p></o:p></p>
<p class="MsoNormal">===================================================================<o:p></o:p></p>
<p class="MsoNormal">--- inc/kernel/ip_packet.h           (revision 7290)<o:p></o:p></p>
<p class="MsoNormal">+++ inc/kernel/ip_packet.h        (working copy)<o:p></o:p></p>
<p class="MsoNormal">@@ -221,7 +221,7 @@<o:p></o:p></p>
<p class="MsoNormal">               uint8_t                 svc_type;<o:p></o:p></p>
<p class="MsoNormal">               net16_t                                length;<o:p></o:p></p>
<p class="MsoNormal">               net16_t                                id;<o:p></o:p></p>
<p class="MsoNormal">-              net16_t                                offset;<o:p></o:p></p>
<p class="MsoNormal">+             net16_t                                offset_flags;<o:p></o:p></p>
<p class="MsoNormal">               uint8_t                 ttl;<o:p></o:p></p>
<p class="MsoNormal">               uint8_t                 prot;<o:p></o:p></p>
<p class="MsoNormal">               net16_t                                chksum;<o:p></o:p></p>
<p class="MsoNormal">@@ -264,6 +264,10 @@<o:p></o:p></p>
<p class="MsoNormal">* SEE ALSO<o:p></o:p></p>
<p class="MsoNormal">*            IB Network Drivers, eth_hdr_t, arp_pkt_t, tcp_hdr_t, udp_hdr_t<o:p></o:p></p>
<p class="MsoNormal">*********/<o:p></o:p></p>
<p class="MsoNormal">+<o:p></o:p></p>
<p class="MsoNormal">+#define OFFSET_MASK  0x1fff<o:p></o:p></p>
<p class="MsoNormal">+<o:p></o:p></p>
<p class="MsoNormal">+<o:p></o:p></p>
<p class="MsoNormal">#include <complib/cl_packoff.h><o:p></o:p></p>
<p class="MsoNormal"><o:p></o:p></p>
<p class="MsoNormal"> #include <complib/cl_packon.h><o:p></o:p></p>
<p class="MsoNormal">Index: ulp/ipoib_NDIS6_CM/kernel/ipoib_port.cpp<o:p></o:p></p>
<p class="MsoNormal">===================================================================<o:p></o:p></p>
<p class="MsoNormal">--- ulp/ipoib_NDIS6_CM/kernel/ipoib_port.cpp                (revision 7290)<o:p></o:p></p>
<p class="MsoNormal">+++ ulp/ipoib_NDIS6_CM/kernel/ipoib_port.cpp             (working copy)<o:p></o:p></p>
<p class="MsoNormal">@@ -441,7 +441,7 @@<o:p></o:p></p>
<p class="MsoNormal">                                                               p_ip_hdr->svc_type,<o:p></o:p></p>
<p class="MsoNormal">                                                               cl_ntoh16( p_ip_hdr->length ),<o:p></o:p></p>
<p class="MsoNormal">                                                               cl_ntoh16( p_ip_hdr->id ),<o:p></o:p></p>
<p class="MsoNormal">-                                                              cl_ntoh16( p_ip_hdr->offset ),<o:p></o:p></p>
<p class="MsoNormal">+                                                             cl_ntoh16( p_ip_hdr->offset_flags ),<o:p></o:p></p>
<p class="MsoNormal">                                                               p_ip_hdr->ttl,<o:p></o:p></p>
<p class="MsoNormal">                                                               p_ip_hdr->prot,<o:p></o:p></p>
<p class="MsoNormal">                                                               cl_ntoh16( p_ip_hdr->chksum ),<o:p></o:p></p>
<p class="MsoNormal">@@ -2821,7 +2821,7 @@<o:p></o:p></p>
<p class="MsoNormal">                                                               break;<o:p></o:p></p>
<p class="MsoNormal">                                               }<o:p></o:p></p>
<p class="MsoNormal"><o:p></o:p></p>
<p class="MsoNormal">-                                              if( p_ipoib->type.ip.hdr.offset ||<o:p></o:p></p>
<p class="MsoNormal">+                                             if( (cl_ntoh16(p_ipoib->type.ip.hdr.offset_flags) & OFFSET_MASK) ||
<o:p></o:p></p>
<p class="MsoNormal">                                                                p_ipoib->type.ip.hdr.prot != IP_PROT_UDP )<o:p></o:p></p>
<p class="MsoNormal">                                               {<o:p></o:p></p>
<p class="MsoNormal">                                                               /* Unfiltered.    Setup the ethernet header and report. */<o:p></o:p></p>
<p class="MsoNormal">@@ -5102,7 +5102,7 @@<o:p></o:p></p>
<p class="MsoNormal">               PERF_DECLARE( FilterDhcp );<o:p></o:p></p>
<p class="MsoNormal">               IPOIB_ENTER( IPOIB_DBG_SEND );<o:p></o:p></p>
<p class="MsoNormal"><o:p></o:p></p>
<p class="MsoNormal">-              if ( (ethertype == ETH_PROT_TYPE_IP) && ((ip_hdr_t*)p_ip_hdr)->offset > 0 )
<o:p></o:p></p>
<p class="MsoNormal">+             if ( (ethertype == ETH_PROT_TYPE_IP) && (cl_ntoh16(((ip_hdr_t*)p_ip_hdr)->offset_flags) & OFFSET_MASK) > 0 )<o:p></o:p></p>
<p class="MsoNormal">               {<o:p></o:p></p>
<p class="MsoNormal">                               /* This is a fragmented part of UDP packet<o:p></o:p></p>
<p class="MsoNormal">                                * Only first packet will contain UDP header in such case<o:p></o:p></p>
</div>
</body>
</html>