<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:"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.25in 1.0in 1.25in;}
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>[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>