<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:p="urn:schemas-microsoft-com:office:powerpoint" xmlns:a="urn:schemas-microsoft-com:office:access" xmlns:dt="uuid:C2F41010-65B3-11d1-A29F-00AA00C14882" xmlns:s="uuid:BDC6E3F0-6DA3-11d1-A2A3-00AA00C14882" xmlns:rs="urn:schemas-microsoft-com:rowset" xmlns:z="#RowsetSchema" xmlns:b="urn:schemas-microsoft-com:office:publisher" xmlns:ss="urn:schemas-microsoft-com:office:spreadsheet" xmlns:c="urn:schemas-microsoft-com:office:component:spreadsheet" xmlns:odc="urn:schemas-microsoft-com:office:odc" xmlns:oa="urn:schemas-microsoft-com:office:activation" xmlns:html="http://www.w3.org/TR/REC-html40" xmlns:q="http://schemas.xmlsoap.org/soap/envelope/" xmlns:rtc="http://microsoft.com/officenet/conferencing" xmlns:D="DAV:" xmlns:Repl="http://schemas.microsoft.com/repl/" xmlns:mt="http://schemas.microsoft.com/sharepoint/soap/meetings/" xmlns:x2="http://schemas.microsoft.com/office/excel/2003/xml" xmlns:ppda="http://www.passport.com/NameSpace.xsd" xmlns:ois="http://schemas.microsoft.com/sharepoint/soap/ois/" xmlns:dir="http://schemas.microsoft.com/sharepoint/soap/directory/" xmlns:ds="http://www.w3.org/2000/09/xmldsig#" xmlns:dsp="http://schemas.microsoft.com/sharepoint/dsp" xmlns:udc="http://schemas.microsoft.com/data/udc" xmlns:xsd="http://www.w3.org/2001/XMLSchema" xmlns:sub="http://schemas.microsoft.com/sharepoint/soap/2002/1/alerts/" xmlns:ec="http://www.w3.org/2001/04/xmlenc#" xmlns:sp="http://schemas.microsoft.com/sharepoint/" xmlns:sps="http://schemas.microsoft.com/sharepoint/soap/" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:udcs="http://schemas.microsoft.com/data/udc/soap" xmlns:udcxf="http://schemas.microsoft.com/data/udc/xmlfile" xmlns:udcp2p="http://schemas.microsoft.com/data/udc/parttopart" xmlns:wf="http://schemas.microsoft.com/sharepoint/soap/workflow/" xmlns:dsss="http://schemas.microsoft.com/office/2006/digsig-setup" xmlns:dssi="http://schemas.microsoft.com/office/2006/digsig" xmlns:mdssi="http://schemas.openxmlformats.org/package/2006/digital-signature" xmlns:mver="http://schemas.openxmlformats.org/markup-compatibility/2006" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns:mrels="http://schemas.openxmlformats.org/package/2006/relationships" xmlns:spwp="http://microsoft.com/sharepoint/webpartpages" xmlns:ex12t="http://schemas.microsoft.com/exchange/services/2006/types" xmlns:ex12m="http://schemas.microsoft.com/exchange/services/2006/messages" xmlns:pptsl="http://schemas.microsoft.com/sharepoint/soap/SlideLibrary/" xmlns:spsl="http://microsoft.com/webservices/SharePointPortalServer/PublishedLinksService" xmlns:Z="urn:schemas-microsoft-com:" xmlns:st="" 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:Calibri;
        panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
        {font-family:Tahoma;
        panose-1:2 11 6 4 3 5 4 4 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;
        font-family:"Calibri","sans-serif";
        color:windowtext;}
span.EmailStyle18
        {mso-style-type:personal-reply;
        font-family:"Calibri","sans-serif";
        color:#1F497D;}
.MsoChpDefault
        {mso-style-type:export-only;
        font-size:10.0pt;}
@page Section1
        {size:8.5in 11.0in;
        margin:1.0in 1.25in 1.0in 1.25in;}
div.Section1
        {page:Section1;}
-->
</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=Section1>

<p class=MsoNormal><span style='color:#1F497D'>Applied at 3005<o:p></o:p></span></p>

<p class=MsoNormal><span style='color:#1F497D'><o:p> </o:p></span></p>

<div>

<div style='border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0in 0in 0in'>

<p class=MsoNormal><b><span style='font-size:10.0pt;font-family:"Tahoma","sans-serif"'>From:</span></b><span
style='font-size:10.0pt;font-family:"Tahoma","sans-serif"'> Alex Naslednikov <br>
<b>Sent:</b> Thursday, November 25, 2010 4:10 PM<br>
<b>To:</b> ofw@lists.openfabrics.org<br>
<b>Subject:</b> [ofw][Patch][IPoIB_NDIS6_CM] LSO revisited<o:p></o:p></span></p>

</div>

</div>

<p class=MsoNormal><o:p> </o:p></p>

<p class=MsoNormal>[IPoIB_NDIS6_CM]<o:p></o:p></p>

<p class=MsoNormal>This patch rearrange LSO flow and makes functions more
robust and reusable. <o:p></o:p></p>

<p class=MsoNormal>This patch affects regular flow as well  (reuse same
code for LSO and non-LSO flows)<o:p></o:p></p>

<p class=MsoNormal>Signed-off by: Alexander Naslednikov (xalex at
mellanox.co.il)<o:p></o:p></p>

<p class=MsoNormal>Index: ipoib_port.cpp<o:p></o:p></p>

<p class=MsoNormal>===================================================================<o:p></o:p></p>

<p class=MsoNormal>---
ipoib_port.cpp           
(revision 2999)<o:p></o:p></p>

<p class=MsoNormal>+++
ipoib_port.cpp          (working
copy)<o:p></o:p></p>

<p class=MsoNormal>@@ -329,7 +329,6 @@<o:p></o:p></p>

<p class=MsoNormal>static NDIS_STATUS<o:p></o:p></p>

<p class=MsoNormal>__send_gen(<o:p></o:p></p>

<p class=MsoNormal>              
IN                                                          
ipoib_send_NB_SG
*                                     
s_buf,<o:p></o:p></p>

<p class=MsoNormal>-             
IN                                                          
INT                                                                                        
lso_data_index,<o:p></o:p></p>

<p class=MsoNormal>              
IN                                                          
UINT                                                                                     
lso_header_size OPTIONAL);<o:p></o:p></p>

<p class=MsoNormal><o:p> </o:p></p>

<p class=MsoNormal> static NDIS_STATUS<o:p></o:p></p>

<p class=MsoNormal>@@ -3591,7 +3590,7 @@<o:p></o:p></p>

<p class=MsoNormal><o:p> </o:p></p>

<p class=MsoNormal>                               
p_desc->send_dir = SEND_UD_QP;<o:p></o:p></p>

<p class=MsoNormal>                              
cl_perf_start( SendGen );<o:p></o:p></p>

<p class=MsoNormal>-                             
status = __send_gen( s_buf, 0, 0 );<o:p></o:p></p>

<p class=MsoNormal>+                             
status = __send_gen( s_buf, 0 );<o:p></o:p></p>

<p class=MsoNormal>                              
cl_perf_stop( &p_port->p_adapter->perf, SendGen );<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>@@ -3699,13 +3698,13 @@<o:p></o:p></p>

<p class=MsoNormal>__send_copy(<o:p></o:p></p>

<p class=MsoNormal>              
IN                                                          
ipoib_port_t*
const                                       
p_port,<o:p></o:p></p>

<p class=MsoNormal>              
IN                                                          
ipoib_send_NB_SG *                                     
s_buf,<o:p></o:p></p>

<p class=MsoNormal>-             
IN                                                          
UINT                                                                                     
lso_header_size)<o:p></o:p></p>

<p class=MsoNormal>+             
IN                                                          
UINT                                                                                     
total_offset)<o:p></o:p></p>

<p class=MsoNormal>{<o:p></o:p></p>

<p class=MsoNormal>              
ULONG                                
tot_len = 0;<o:p></o:p></p>

<p class=MsoNormal>-             
uint32_t                              
offset = <o:p></o:p></p>

<p class=MsoNormal>-                                                                                             
(lso_header_size ? EthIPoIBHeaderOffset : EthHeaderOffset );<o:p></o:p></p>

<p class=MsoNormal>-             
int seg_index = lso_header_size ? 0 : 1;<o:p></o:p></p>

<p class=MsoNormal><o:p> </o:p></p>

<p class=MsoNormal>+             
// first DS does not contain IPoIB header in the case of LSO, so we set it back
to 0<o:p></o:p></p>

<p class=MsoNormal>+             
int seg_index = ( total_offset == EthHeaderOffset ? 1 : 0 );<o:p></o:p></p>

<p class=MsoNormal>+<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>               
ipoib_send_desc_t *p_desc = p_port->p_desc;<o:p></o:p></p>

<p class=MsoNormal>@@ -3720,15 +3719,15 @@<o:p></o:p></p>

<p class=MsoNormal>                              
return NDIS_STATUS_RESOURCES;<o:p></o:p></p>

<p class=MsoNormal>              
}<o:p></o:p></p>

<p class=MsoNormal>              
tot_len = CopyNetBuffer(s_buf->p_curr_nb, (PUCHAR) s_buf->p_send_buf);<o:p></o:p></p>

<p class=MsoNormal>-             
if (tot_len <= lso_header_size) {<o:p></o:p></p>

<p class=MsoNormal>-                             
ASSERT(tot_len > lso_header_size);<o:p></o:p></p>

<p class=MsoNormal>+             
if ( !tot_len ) {<o:p></o:p></p>

<p class=MsoNormal>+                             
ASSERT( tot_len );<o:p></o:p></p>

<p class=MsoNormal>                              
return NDIS_STATUS_FAILURE;<o:p></o:p></p>

<p class=MsoNormal>              
}<o:p></o:p></p>

<p class=MsoNormal>              
<o:p></o:p></p>

<p class=MsoNormal>               
/* Setup the work request. */<o:p></o:p></p>

<p class=MsoNormal>              
p_desc->send_wr[0].local_ds[seg_index].vaddr = cl_get_physaddr(<o:p></o:p></p>

<p class=MsoNormal>-                             
((uint8_t*)s_buf->p_send_buf) + lso_header_size + offset );<o:p></o:p></p>

<p class=MsoNormal>-             
p_desc->send_wr[0].local_ds[seg_index].length = tot_len - lso_header_size -
offset;<o:p></o:p></p>

<p class=MsoNormal>+                             
((uint8_t*)s_buf->p_send_buf) + total_offset );<o:p></o:p></p>

<p class=MsoNormal>+             
p_desc->send_wr[0].local_ds[seg_index].length = tot_len - total_offset;<o:p></o:p></p>

<p class=MsoNormal>              
p_desc->send_wr[0].local_ds[seg_index].lkey = p_port->ib_mgr.lkey;<o:p></o:p></p>

<p class=MsoNormal>              
p_desc->send_wr[0].wr.num_ds = seg_index+1;<o:p></o:p></p>

<p class=MsoNormal><o:p> </o:p></p>

<p class=MsoNormal>@@ -4259,13 +4258,34 @@<o:p></o:p></p>

<p class=MsoNormal>static NDIS_STATUS<o:p></o:p></p>

<p class=MsoNormal>__send_gen(<o:p></o:p></p>

<p class=MsoNormal>              
IN                                                          
ipoib_send_NB_SG *                                     
s_buf,<o:p></o:p></p>

<p class=MsoNormal>-             
IN                                                          
INT                                                                                                        
lso_data_index,<o:p></o:p></p>

<p class=MsoNormal>              
IN                                                          
UINT                                                                                     
lso_header_size OPTIONAL)<o:p></o:p></p>

<p class=MsoNormal>{<o:p></o:p></p>

<p class=MsoNormal>              
NDIS_STATUS                   
status;<o:p></o:p></p>

<p class=MsoNormal>-             
uint32_t                              
i, j
                         
= 1;<o:p></o:p></p>

<p class=MsoNormal>+             
uint32_t                              
i = 0;       //Index of SG element<o:p></o:p></p>

<p class=MsoNormal>+             
uint32_t                              
j;             //Index
of DS elements;<o:p></o:p></p>

<p class=MsoNormal>              
ULONG
                                               
DataOffset          = 0;
        <o:p></o:p></p>

<p class=MsoNormal>-             
uint32_t                              
offset
                  
= EthHeaderOffset;<o:p></o:p></p>

<p class=MsoNormal>+             
UINT                                     
total_offset;<o:p></o:p></p>

<p class=MsoNormal>+             
<o:p></o:p></p>

<p class=MsoNormal>+             
/* We calculate the amount of bytes to skip over ETH header in a case of normal
send or<o:p></o:p></p>

<p class=MsoNormal>+             
  * LSO header in a case of LSO.<o:p></o:p></p>

<p class=MsoNormal>+             
  * But in the case of LSO we replace last 4 bytes of ETH header by IPoIB
header<o:p></o:p></p>

<p class=MsoNormal>+             
  * <o:p></o:p></p>

<p class=MsoNormal>+             
  * Thus, the calulation should be:<o:p></o:p></p>

<p class=MsoNormal>+             
  * Normal send: offset = sizeof ETH header<o:p></o:p></p>

<p class=MsoNormal>+             
  *
LSO                  
  : offset = sizeof ETH header+sizeof IP header+ sizeof TCP header <o:p></o:p></p>

<p class=MsoNormal>+             
 
                                                                             
== sizeof LSO header + (sizeof ETH header-sizeof IPoIB header)<o:p></o:p></p>

<p class=MsoNormal>+             
  */<o:p></o:p></p>

<p class=MsoNormal>+             
if ( lso_header_size ) <o:p></o:p></p>

<p class=MsoNormal>+             
{<o:p></o:p></p>

<p class=MsoNormal>+                             
total_offset = lso_header_size + EthIPoIBHeaderOffset;<o:p></o:p></p>

<p class=MsoNormal>+                             
j = 0;<o:p></o:p></p>

<p class=MsoNormal>+             
}<o:p></o:p></p>

<p class=MsoNormal>+             
else<o:p></o:p></p>

<p class=MsoNormal>+             
{<o:p></o:p></p>

<p class=MsoNormal>+                             
total_offset =  EthHeaderOffset;<o:p></o:p></p>

<p class=MsoNormal>+                             
j = 1; //Skip on the first DS, because it alredy contain IPoIB header<o:p></o:p></p>

<p class=MsoNormal>+             
}<o:p></o:p></p>

<p class=MsoNormal>+             
<o:p></o:p></p>

<p class=MsoNormal>              
PERF_DECLARE( SendCopy );<o:p></o:p></p>

<p class=MsoNormal><o:p> </o:p></p>

<p class=MsoNormal>               
IPOIB_ENTER( IPOIB_DBG_SEND );<o:p></o:p></p>

<p class=MsoNormal>@@ -4298,7 +4318,7 @@<o:p></o:p></p>

<p class=MsoNormal>                              
if( !s_buf->p_port->p_adapter->params.cm_enabled )<o:p></o:p></p>

<p class=MsoNormal>                              
{<o:p></o:p></p>

<p class=MsoNormal>                                              
cl_perf_start( SendCopy );<o:p></o:p></p>

<p class=MsoNormal>-                                             
status = __send_copy( s_buf->p_port, s_buf, lso_header_size );<o:p></o:p></p>

<p class=MsoNormal>+                                             
status = __send_copy( s_buf->p_port, s_buf, total_offset );<o:p></o:p></p>

<p class=MsoNormal>                                              
cl_perf_stop( &s_buf->p_port->p_adapter->perf, SendCopy );<o:p></o:p></p>

<p class=MsoNormal>                              
}<o:p></o:p></p>

<p class=MsoNormal>                              
else <o:p></o:p></p>

<p class=MsoNormal>@@ -4309,55 +4329,33 @@<o:p></o:p></p>

<p class=MsoNormal>                              
return status;<o:p></o:p></p>

<p class=MsoNormal>              
}<o:p></o:p></p>

<p class=MsoNormal><o:p> </o:p></p>

<p class=MsoNormal>-             
/*<o:p></o:p></p>

<p class=MsoNormal>-             
* Skip the ethernet header.  It is either the first element,<o:p></o:p></p>

<p class=MsoNormal>-             
* or part of it.<o:p></o:p></p>

<p class=MsoNormal>-             
*/<o:p></o:p></p>

<p class=MsoNormal>-             
i = 0;<o:p></o:p></p>

<p class=MsoNormal>              
DataOffset= (ULONG)(NET_BUFFER_CURRENT_MDL_OFFSET(s_buf->p_curr_nb)); <o:p></o:p></p>

<p class=MsoNormal>               
<o:p></o:p></p>

<p class=MsoNormal>-             
if( lso_data_index )<o:p></o:p></p>

<p class=MsoNormal>-             
{ /* we have an LSO packet */<o:p></o:p></p>

<p class=MsoNormal>-                             
i = lso_data_index;<o:p></o:p></p>

<p class=MsoNormal>-                             
j = 0;<o:p></o:p></p>

<p class=MsoNormal>-                             
ASSERT( i <= p_sgl->NumberOfElements);<o:p></o:p></p>

<p class=MsoNormal>-                             
if (i == p_sgl->NumberOfElements) {<o:p></o:p></p>

<p class=MsoNormal>-<o:p></o:p></p>

<p class=MsoNormal>-                                             
/****************************<o:p></o:p></p>

<p class=MsoNormal>-                                             
* Handle the case when there is only one SG element !<o:p></o:p></p>

<p class=MsoNormal>-                                             
****************************/<o:p></o:p></p>

<p class=MsoNormal>-                                             
<o:p></o:p></p>

<p class=MsoNormal>-                                             
p_desc->send_wr[0].local_ds[j].vaddr = <o:p></o:p></p>

<p class=MsoNormal>-                                                             
p_sgl->Elements[0].Address.QuadPart + lso_header_size + DataOffset +
EthIPoIBHeaderOffset;<o:p></o:p></p>

<p class=MsoNormal>-                                             
p_desc->send_wr[0].local_ds[j].length = <o:p></o:p></p>

<p class=MsoNormal>-                                                             
p_sgl->Elements[0].Length - lso_header_size - DataOffset -
EthIPoIBHeaderOffset;<o:p></o:p></p>

<p class=MsoNormal>-                                             
p_desc->send_wr[0].local_ds[j].lkey = s_buf->p_port->ib_mgr.lkey;<o:p></o:p></p>

<p class=MsoNormal>-                                             
/* Set the number of data segments. */<o:p></o:p></p>

<p class=MsoNormal>-                                             
p_desc->send_wr[0].wr.num_ds = 1;<o:p></o:p></p>

<p class=MsoNormal>-                                             
return NDIS_STATUS_SUCCESS;<o:p></o:p></p>

<p class=MsoNormal>-                             
}                             
<o:p></o:p></p>

<p class=MsoNormal>+             
<o:p></o:p></p>

<p class=MsoNormal>+             
/*<o:p></o:p></p>

<p class=MsoNormal>+             
* Skip the Ethernet or LSO header.  It is contained at N+1 first elements
(N>=0),<o:p></o:p></p>

<p class=MsoNormal>+             
* while (N+1) element may contain only part of it<o:p></o:p></p>

<p class=MsoNormal>+             
*/<o:p></o:p></p>

<p class=MsoNormal>+             
              
<o:p></o:p></p>

<p class=MsoNormal>+             
while( total_offset >= p_sgl->Elements[i].Length  )<o:p></o:p></p>

<p class=MsoNormal>+             
{<o:p></o:p></p>

<p class=MsoNormal>+                             
// skip the current element and increment the index<o:p></o:p></p>

<p class=MsoNormal>+                             
total_offset -= p_sgl->Elements[i++].Length;<o:p></o:p></p>

<p class=MsoNormal>              
}<o:p></o:p></p>

<p class=MsoNormal>-             
else while( offset )<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 (total_offset > 0 )<o:p></o:p></p>

<p class=MsoNormal>              
{<o:p></o:p></p>

<p class=MsoNormal>-                             
if( p_sgl->Elements[i].Length <= offset )<o:p></o:p></p>

<p class=MsoNormal>-                             
{<o:p></o:p></p>

<p class=MsoNormal>-                                             
offset -= p_sgl->Elements[i++].Length;<o:p></o:p></p>

<p class=MsoNormal>-                             
}<o:p></o:p></p>

<p class=MsoNormal>-                             
else<o:p></o:p></p>

<p class=MsoNormal>-                             
{<o:p></o:p></p>

<p class=MsoNormal>-                                             
p_desc->send_wr[0].local_ds[j].vaddr =<o:p></o:p></p>

<p class=MsoNormal>-                                                             
p_sgl->Elements[i].Address.QuadPart + offset + DataOffset;<o:p></o:p></p>

<p class=MsoNormal>-                                             
p_desc->send_wr[0].local_ds[j].length =<o:p></o:p></p>

<p class=MsoNormal>-                                                             
p_sgl->Elements[i].Length - offset - DataOffset;<o:p></o:p></p>

<p class=MsoNormal>-                                             
p_desc->send_wr[0].local_ds[j].lkey = s_buf->p_port->ib_mgr.lkey;<o:p></o:p></p>

<p class=MsoNormal>-                                             
if( p_desc->send_wr[0].local_ds[j].length > 0 )<o:p></o:p></p>

<p class=MsoNormal>-                                             
{<o:p></o:p></p>

<p class=MsoNormal>-                                                             
j++;<o:p></o:p></p>

<p class=MsoNormal>-                                             
}<o:p></o:p></p>

<p class=MsoNormal>-                                             
i++;<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>+                             
//Handle the (N+1) element that can probably contain both Header and Data<o:p></o:p></p>

<p class=MsoNormal>+                             
p_desc->send_wr[0].local_ds[j].vaddr =<o:p></o:p></p>

<p class=MsoNormal>+                                             
p_sgl->Elements[i].Address.QuadPart + total_offset + DataOffset;<o:p></o:p></p>

<p class=MsoNormal>+                             
p_desc->send_wr[0].local_ds[j].length =<o:p></o:p></p>

<p class=MsoNormal>+                                             
p_sgl->Elements[i].Length - total_offset - DataOffset;<o:p></o:p></p>

<p class=MsoNormal>+                             
p_desc->send_wr[0].local_ds[j].lkey = s_buf->p_port->ib_mgr.lkey;<o:p></o:p></p>

<p class=MsoNormal>+                             
j++;          <o:p></o:p></p>

<p class=MsoNormal>+                             
i++;<o:p></o:p></p>

<p class=MsoNormal>              
}<o:p></o:p></p>

<p class=MsoNormal>+                                             
<o:p></o:p></p>

<p class=MsoNormal>              
/* Now fill in the rest of the local data segments. */<o:p></o:p></p>

<p class=MsoNormal>              
while( i < p_sgl->NumberOfElements )<o:p></o:p></p>

<p class=MsoNormal>              
{<o:p></o:p></p>

<p class=MsoNormal>@@ -4527,7 +4525,7 @@<o:p></o:p></p>

<p class=MsoNormal><o:p> </o:p></p>

<p class=MsoNormal> send_gen:<o:p></o:p></p>

<p class=MsoNormal>              
cl_perf_start( SendTcp );<o:p></o:p></p>

<p class=MsoNormal>-             
status = __send_gen(  s_buf, 0, 0 );<o:p></o:p></p>

<p class=MsoNormal>+             
status = __send_gen(  s_buf, 0 );<o:p></o:p></p>

<p class=MsoNormal>              
cl_perf_stop( &p_port->p_adapter->perf, SendTcp );<o:p></o:p></p>

<p class=MsoNormal><o:p> </o:p></p>

<p class=MsoNormal>               
IPOIB_EXIT( IPOIB_DBG_SEND );<o:p></o:p></p>

<p class=MsoNormal>@@ -5273,27 +5271,20 @@<o:p></o:p></p>

<p class=MsoNormal>              
        ASSERT(p_lso_info->LsoV1Transmit.TcpHeaderOffset
== p_lso_info->LsoV2Transmit.TcpHeaderOffset);<o:p></o:p></p>

<p class=MsoNormal>              
    }<o:p></o:p></p>

<p class=MsoNormal>              
}<o:p></o:p></p>

<p class=MsoNormal>-                                             
<o:p></o:p></p>

<p class=MsoNormal>+<o:p></o:p></p>

<p class=MsoNormal>              
/* Format the send descriptor. */<o:p></o:p></p>

<p class=MsoNormal>              
hdr_idx = cl_atomic_inc( &s_buf->p_port->hdr_idx );<o:p></o:p></p>

<p class=MsoNormal>              
hdr_idx &= (s_buf->p_port->p_adapter->params.sq_depth - 1);<o:p></o:p></p>

<p class=MsoNormal>              
ASSERT( hdr_idx < s_buf->p_port->p_adapter->params.sq_depth );<o:p></o:p></p>

<p class=MsoNormal><o:p> </o:p></p>

<p class=MsoNormal>+             
/* Set up IPoIB Header */<o:p></o:p></p>

<p class=MsoNormal>              
s_buf->p_port->hdr[hdr_idx].type = p_eth_hdr->type;<o:p></o:p></p>

<p class=MsoNormal>              
s_buf->p_port->hdr[hdr_idx].resv = 0;<o:p></o:p></p>

<p class=MsoNormal>+                                             
<o:p></o:p></p>

<p class=MsoNormal><o:p> </o:p></p>

<p class=MsoNormal>-             
//TODO why enter this block for LSO ???<o:p></o:p></p>

<p class=MsoNormal>-             
p_desc->send_wr[0].local_ds[0].vaddr =<o:p></o:p></p>

<p class=MsoNormal>-                                                                                                                             
cl_get_physaddr( &s_buf->p_port->hdr[hdr_idx] );<o:p></o:p></p>

<p class=MsoNormal>-             
p_desc->send_wr[0].local_ds[0].length = sizeof(ipoib_hdr_t);<o:p></o:p></p>

<p class=MsoNormal>-             
p_desc->send_wr[0].local_ds[0].lkey = s_buf->p_port->ib_mgr.lkey;<o:p></o:p></p>

<p class=MsoNormal>-             
p_desc->send_wr[0].wr.send_opt = 0;<o:p></o:p></p>

<p class=MsoNormal>-<o:p></o:p></p>

<p class=MsoNormal>              
//Init send buffer to 0<o:p></o:p></p>

<p class=MsoNormal>              
s_buf->p_send_buf = NULL;<o:p></o:p></p>

<p class=MsoNormal><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 (mss && (p_lso_info->LsoV1Transmit.TcpHeaderOffset != 0))<o:p></o:p></p>

<p class=MsoNormal>              
{ //We have LSO packet<o:p></o:p></p>

<p class=MsoNormal>                              
ASSERT( mss == (p_lso_info->LsoV1Transmit.MSS & p_lso_info->LsoV2Transmit.MSS));<o:p></o:p></p>

<p class=MsoNormal>@@ -5316,6 +5307,13 @@<o:p></o:p></p>

<p class=MsoNormal>                              
uint32_t              
i;<o:p></o:p></p>

<p class=MsoNormal><o:p> </o:p></p>

<p class=MsoNormal>                               
cl_perf_start( SendMgrFilter );<o:p></o:p></p>

<p class=MsoNormal>+<o:p></o:p></p>

<p class=MsoNormal>+                             
/* Put first DS to be IPoIB Header */<o:p></o:p></p>

<p class=MsoNormal>+                             
p_desc->send_wr[0].local_ds[0].vaddr =<o:p></o:p></p>

<p class=MsoNormal>+                                                                                                                             
cl_get_physaddr( &s_buf->p_port->hdr[hdr_idx] );<o:p></o:p></p>

<p class=MsoNormal>+                             
p_desc->send_wr[0].local_ds[0].length = sizeof(ipoib_hdr_t);<o:p></o:p></p>

<p class=MsoNormal>+                             
p_desc->send_wr[0].local_ds[0].lkey = s_buf->p_port->ib_mgr.lkey;<o:p></o:p></p>

<p class=MsoNormal>+                             
<o:p></o:p></p>

<p class=MsoNormal>                              
status = __send_mgr_filter( p_eth_hdr, p_mdl, mdl_len, s_buf );<o:p></o:p></p>

<p class=MsoNormal>                              
cl_perf_stop( &p_port->p_adapter->perf, SendMgrFilter );<o:p></o:p></p>

<p class=MsoNormal>                              
if( status != NDIS_STATUS_SUCCESS )<o:p></o:p></p>

<p class=MsoNormal>@@ -5327,6 +5325,7 @@<o:p></o:p></p>

<p class=MsoNormal><o:p> </o:p></p>

<p class=MsoNormal>                               
if( p_desc->send_dir == SEND_UD_QP )<o:p></o:p></p>

<p class=MsoNormal>                              
{<o:p></o:p></p>

<p class=MsoNormal>+                                             
ASSERT ( p_desc->num_wrs == 1 );<o:p></o:p></p>

<p class=MsoNormal>                                              
p_desc->send_qp = s_buf->p_port->ib_mgr.h_qp; // UD QP<o:p></o:p></p>

<p class=MsoNormal>                                              
for( i = 0; i < p_desc->num_wrs; i++ )<o:p></o:p></p>

<p class=MsoNormal>                                              
{<o:p></o:p></p>

<p class=MsoNormal>@@ -5423,14 +5422,10 @@<o:p></o:p></p>

<p class=MsoNormal>                              
return status;<o:p></o:p></p>

<p class=MsoNormal>              
}<o:p></o:p></p>

<p class=MsoNormal>              
ASSERT(TheLsoData.LsoHeaderSize > 0);<o:p></o:p></p>

<p class=MsoNormal>+                             
<o:p></o:p></p>

<p class=MsoNormal>              
// Tell NDIS how much we will send.<o:p></o:p></p>

<p class=MsoNormal>-             
//PktExt->NdisPacketInfo[TcpLargeSendPacketInfo] = UlongToPtr(PacketLength);<o:p></o:p></p>

<p class=MsoNormal>-             
//p_lso_info->LsoV1TransmitComplete.TcpPayload = PacketLength;<o:p></o:p></p>

<p class=MsoNormal>-             
<o:p></o:p></p>

<p class=MsoNormal>-             
// Tell NDIS how much we will send.<o:p></o:p></p>

<p class=MsoNormal>              
if(p_lso_info->LsoV1Transmit.Type == NDIS_TCP_LARGE_SEND_OFFLOAD_V1_TYPE)<o:p></o:p></p>

<p class=MsoNormal>     {<o:p></o:p></p>

<p class=MsoNormal>-   
                         
//IPOIB_TCP_PAYLOAD_FROM_NB(p_netbuf) += PacketLength-TheLsoData.LsoHeaderSize;<o:p></o:p></p>

<p class=MsoNormal>    
                          
s_buf->tcp_payload = PacketLength-TheLsoData.LsoHeaderSize;<o:p></o:p></p>

<p class=MsoNormal>    
                          
<o:p></o:p></p>

<p class=MsoNormal>               
}<o:p></o:p></p>

<p class=MsoNormal>@@ -5453,7 +5448,7 @@<o:p></o:p></p>

<p class=MsoNormal>              
p_desc->send_wr[0].wr.p_next = NULL;<o:p></o:p></p>

<p class=MsoNormal>              
p_desc->send_qp = p_port->ib_mgr.h_qp;<o:p></o:p></p>

<p class=MsoNormal>              
p_desc->send_dir = SEND_UD_QP;<o:p></o:p></p>

<p class=MsoNormal>-             
status = __send_gen( s_buf, IndexOfData, TheLsoData.LsoHeaderSize );<o:p></o:p></p>

<p class=MsoNormal>+             
status = __send_gen( s_buf, TheLsoData.LsoHeaderSize );<o:p></o:p></p>

<p class=MsoNormal><o:p> </o:p></p>

<p class=MsoNormal>               
IPOIB_EXIT( IPOIB_DBG_SEND );<o:p></o:p></p>

<p class=MsoNormal>              
return status;<o:p></o:p></p>

<p class=MsoNormal>@@ -8218,7 +8213,7 @@<o:p></o:p></p>

<p class=MsoNormal>--*/<o:p></o:p></p>

<p class=MsoNormal><o:p> </o:p></p>

<p class=MsoNormal> NDIS_STATUS GetLsoHeaderSize(<o:p></o:p></p>

<p class=MsoNormal>-             
IN                          
PNET_BUFFER                   
pNetBuffer,<o:p></o:p></p>

<p class=MsoNormal>+             
IN OUT 
PNET_BUFFER                   
pNetBuffer,<o:p></o:p></p>

<p class=MsoNormal>              
IN                          
LsoData                               
*pLsoData,<o:p></o:p></p>

<p class=MsoNormal>              
OUT      
UINT                                     
*IndexOfData,<o:p></o:p></p>

<p class=MsoNormal>              
IN                          
ipoib_hdr_t *ipoib_hdr<o:p></o:p></p>

<p class=MsoNormal>@@ -8255,7 +8250,7 @@<o:p></o:p></p>

<p class=MsoNormal>              
NdisQueryMdl(pMDL, &pSrc, &CurrLength, NormalPagePriority);<o:p></o:p></p>

<p class=MsoNormal>              
<o:p></o:p></p>

<p class=MsoNormal>               
if (pSrc == NULL) {<o:p></o:p></p>

<p class=MsoNormal>-                             
IPOIB_PRINT(TRACE_LEVEL_ERROR, IPOIB_DBG_ERROR, ("Error processing packets\n"));<o:p></o:p></p>

<p class=MsoNormal>+                             
IPOIB_PRINT(TRACE_LEVEL_ERROR, IPOIB_DBG_ERROR, ("NdisQueryMdl
failed\n"));<o:p></o:p></p>

<p class=MsoNormal>                              
return status;<o:p></o:p></p>

<p class=MsoNormal>              
}<o:p></o:p></p>

<p class=MsoNormal>              
<o:p></o:p></p>

<p class=MsoNormal>@@ -8265,7 +8260,8 @@<o:p></o:p></p>

<p class=MsoNormal><o:p> </o:p></p>

<p class=MsoNormal>               
// We start by looking for the ethernet and the IP<o:p></o:p></p>

<p class=MsoNormal>              
if (CurrLength < ETH_OFFSET) {<o:p></o:p></p>

<p class=MsoNormal>-                             
IPOIB_PRINT(TRACE_LEVEL_ERROR, IPOIB_DBG_ERROR, ("Error porcessing
packets\n"));<o:p></o:p></p>

<p class=MsoNormal>+                             
ASSERT ( CurrLength >= ETH_OFFSET );<o:p></o:p></p>

<p class=MsoNormal>+                             
IPOIB_PRINT(TRACE_LEVEL_ERROR, IPOIB_DBG_ERROR, ("CurrLength < ETH_OFFSET\n"));<o:p></o:p></p>

<p class=MsoNormal>                              
return status;<o:p></o:p></p>

<p class=MsoNormal>              
}<o:p></o:p></p>

<p class=MsoNormal>              
<o:p></o:p></p>

<p class=MsoNormal>@@ -8280,7 +8276,7 @@<o:p></o:p></p>

<p class=MsoNormal>                              
pNetBuffer = NET_BUFFER_NEXT_NB(pNetBuffer);<o:p></o:p></p>

<p class=MsoNormal>        
NdisQueryMdl(NET_BUFFER_CURRENT_MDL(pNetBuffer), &pSrc, &CurrLength,
NormalPagePriority);<o:p></o:p></p>

<p class=MsoNormal>                              
if (pSrc == NULL) {<o:p></o:p></p>

<p class=MsoNormal>-                                             
IPOIB_PRINT(TRACE_LEVEL_VERBOSE, IPOIB_DBG_ERROR, ("Error porcessing
packets\n"));<o:p></o:p></p>

<p class=MsoNormal>+                                             
IPOIB_PRINT(TRACE_LEVEL_ERROR, IPOIB_DBG_ERROR, ("NdisQueryMdl
failed\n"));<o:p></o:p></p>

<p class=MsoNormal>                                              
return status;<o:p></o:p></p>

<p class=MsoNormal>              
    }<o:p></o:p></p>

<p class=MsoNormal>              
} else {<o:p></o:p></p>

<p class=MsoNormal>@@ -8288,6 +8284,7 @@<o:p></o:p></p>

<p class=MsoNormal>                              
pLsoData->LsoBuffers[0].pData = pSrc + (ETH_OFFSET - sizeof (ipoib_hdr_t));<o:p></o:p></p>

<p class=MsoNormal>                              
<o:p></o:p></p>

<p class=MsoNormal>                               
//IMPORTANT: we de-facto replace ETH header by IPoIB header here<o:p></o:p></p>

<p class=MsoNormal>+                             
//TODO: This is not good practice to change data we got from NDIS<o:p></o:p></p>

<p class=MsoNormal>                              
memcpy (pLsoData->LsoBuffers[0].pData, ipoib_hdr, sizeof (ipoib_hdr_t));<o:p></o:p></p>

<p class=MsoNormal>                              
<o:p></o:p></p>

<p class=MsoNormal>                               
CurrLength -= ETH_OFFSET;<o:p></o:p></p>

<p class=MsoNormal>@@ -8296,14 +8293,14 @@<o:p></o:p></p>

<p class=MsoNormal>              
}<o:p></o:p></p>

<p class=MsoNormal>              
// we should now be having at least the size of ethernet data<o:p></o:p></p>

<p class=MsoNormal>              
if (CurrLength < sizeof (ip_hdr_t)) {<o:p></o:p></p>

<p class=MsoNormal>-                             
IPOIB_PRINT(TRACE_LEVEL_VERBOSE, IPOIB_DBG_ERROR, ("Error porcessing packets\n"));<o:p></o:p></p>

<p class=MsoNormal>+                             
IPOIB_PRINT(TRACE_LEVEL_ERROR, IPOIB_DBG_ERROR, ("CurrLength < sizeof
(ip_hdr_t)\n"));<o:p></o:p></p>

<p class=MsoNormal>                              
return status;<o:p></o:p></p>

<p class=MsoNormal>              
}<o:p></o:p></p>

<p class=MsoNormal>              
IpHdr = (ip_hdr_t UNALIGNED*)pSrc;<o:p></o:p></p>

<p class=MsoNormal>              
IpHeaderLen = (uint16_t)IP_HEADER_LENGTH(IpHdr);<o:p></o:p></p>

<p class=MsoNormal>              
ASSERT(IpHdr->prot == IP_PROT_TCP);<o:p></o:p></p>

<p class=MsoNormal>              
if (CurrLength < IpHeaderLen) {<o:p></o:p></p>

<p class=MsoNormal>-                             
IPOIB_PRINT(TRACE_LEVEL_ERROR, IPOIB_DBG_ERROR, ("Error processing packets\n"));<o:p></o:p></p>

<p class=MsoNormal>+                             
IPOIB_PRINT(TRACE_LEVEL_ERROR, IPOIB_DBG_ERROR, ("CurrLength <
IpHeaderLe\n"));<o:p></o:p></p>

<p class=MsoNormal>                              
return status;<o:p></o:p></p>

<p class=MsoNormal>              
}<o:p></o:p></p>

<p class=MsoNormal>              
pLsoData->LsoHeaderSize = pLsoData->LsoHeaderSize + IpHeaderLen;<o:p></o:p></p>

<p class=MsoNormal>@@ -8324,12 +8321,10 @@<o:p></o:p></p>

<p class=MsoNormal><o:p> </o:p></p>

<p class=MsoNormal>                               
FullBuffers++;<o:p></o:p></p>

<p class=MsoNormal>                              
IsRegularFlow = FALSE;<o:p></o:p></p>

<p class=MsoNormal>-                             
//NdisGetNextBuffer( CurrBuffer, &CurrBuffer);<o:p></o:p></p>

<p class=MsoNormal>-                             
//NdisQueryBufferSafe( CurrBuffer, &pSrc, &CurrLength,
NormalPagePriority );<o:p></o:p></p>

<p class=MsoNormal>                              
pNetBuffer = NET_BUFFER_NEXT_NB(pNetBuffer);<o:p></o:p></p>

<p class=MsoNormal>                              
NdisQueryMdl(NET_BUFFER_CURRENT_MDL(pNetBuffer), &pSrc, &CurrLength,
NormalPagePriority);<o:p></o:p></p>

<p class=MsoNormal>                              
if (pSrc == NULL) {<o:p></o:p></p>

<p class=MsoNormal>-                                             
IPOIB_PRINT(TRACE_LEVEL_VERBOSE, IPOIB_DBG_ERROR, ("Error porcessing
packets\n"));<o:p></o:p></p>

<p class=MsoNormal>+                                             
IPOIB_PRINT(TRACE_LEVEL_VERBOSE, IPOIB_DBG_ERROR, ("NdisQueryMdl
failed\n"));<o:p></o:p></p>

<p class=MsoNormal>                                              
return status;<o:p></o:p></p>

<p class=MsoNormal>                              
}<o:p></o:p></p>

<p class=MsoNormal>              
} else {<o:p></o:p></p>

<p class=MsoNormal>@@ -8346,17 +8341,15 @@<o:p></o:p></p>

<p class=MsoNormal>                              
pSrc = pSrc + IpHeaderLen;<o:p></o:p></p>

<p class=MsoNormal>              
}<o:p></o:p></p>

<p class=MsoNormal>              
if (CurrLength < sizeof (tcp_hdr_t)) {<o:p></o:p></p>

<p class=MsoNormal>-                             
IPOIB_PRINT(TRACE_LEVEL_VERBOSE, IPOIB_DBG_ERROR, ("Error porcessing packets\n"));<o:p></o:p></p>

<p class=MsoNormal>+                             
IPOIB_PRINT(TRACE_LEVEL_VERBOSE, IPOIB_DBG_ERROR, ("Error processing
packets\n"));<o:p></o:p></p>

<p class=MsoNormal>                              
return status;<o:p></o:p></p>

<p class=MsoNormal>              
}<o:p></o:p></p>

<p class=MsoNormal>              
// We have finaly found the TCP header<o:p></o:p></p>

<p class=MsoNormal>              
TcpHdr = (tcp_hdr_t UNALIGNED *)pSrc;<o:p></o:p></p>

<p class=MsoNormal>              
TcpHeaderLen = TCP_HEADER_LENGTH(TcpHdr);<o:p></o:p></p>

<p class=MsoNormal>-<o:p></o:p></p>

<p class=MsoNormal>-             
//ASSERT(TcpHeaderLen == 20);<o:p></o:p></p>

<p class=MsoNormal>              
<o:p></o:p></p>

<p class=MsoNormal>               
if (CurrLength < TcpHeaderLen) {<o:p></o:p></p>

<p class=MsoNormal>-                             
//IPOIB_PRINT(TRACE_LEVEL_VERBOSE, ETH, ("Error porcessing packets\n"));<o:p></o:p></p>

<p class=MsoNormal>+                             
IPOIB_PRINT(TRACE_LEVEL_ERROR, IPOIB_DBG_ERROR, ("Error processing
packets\n"));<o:p></o:p></p>

<p class=MsoNormal>                              
return status;<o:p></o:p></p>

<p class=MsoNormal>              
}<o:p></o:p></p>

<p class=MsoNormal>              
pLsoData->LsoHeaderSize =  pLsoData->LsoHeaderSize + TcpHeaderLen;<o:p></o:p></p>

</div>

</body>

</html>