<html xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" 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=Windows-1252">
<meta name="Generator" content="Microsoft Word 15 (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:Aptos;
panose-1:2 11 0 4 2 2 2 2 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0cm;
font-size:10.0pt;
font-family:"Aptos",sans-serif;}
span.EmailStyle19
{mso-style-type:personal-reply;
font-family:"Aptos",sans-serif;
color:windowtext;}
.MsoChpDefault
{mso-style-type:export-only;
font-size:10.0pt;
mso-ligatures:none;}
@page WordSection1
{size:612.0pt 792.0pt;
margin:72.0pt 72.0pt 72.0pt 72.0pt;}
div.WordSection1
{page:WordSection1;}
--></style>
</head>
<body lang="en-PS" link="blue" vlink="purple" style="word-wrap:break-word">
<div class="WordSection1">
<div id="mail-editor-reference-message-container">
<div>
<div>
<div>
<p class="MsoNormal" style="margin-bottom:12.0pt"><span lang="EN-US" style="font-size:11.0pt">></span><span style="font-size:11.0pt">> - eth_hdr->ether_type == _htons(RTE_ETHER_TYPE_QINQ)) {<br>
</span><span lang="EN-US" style="font-size:11.0pt">></span><span style="font-size:11.0pt">> - vlan_hdr = (struct rte_vlan_hdr *)<br>
</span><span lang="EN-US" style="font-size:11.0pt">></span><span style="font-size:11.0pt">> - ((char *)eth_hdr + sizeof(*eth_hdr));<br>
</span><span lang="EN-US" style="font-size:11.0pt">></span><span style="font-size:11.0pt">> + vlan_hdr = (struct rte_vlan_hdr *) RTE_PTR_ADD(eth_hdr,<br>
</span><span lang="EN-US" style="font-size:11.0pt">></span><span style="font-size:11.0pt">> + offsetof(struct rte_ether_hdr, ether_type));<br>
</span><span lang="EN-US" style="font-size:11.0pt">></span><span style="font-size:11.0pt">> + while ((ethertype == _htons(RTE_ETHER_TYPE_VLAN) ||<br>
</span><span lang="EN-US" style="font-size:11.0pt">></span><span style="font-size:11.0pt">> + ethertype == _htons(RTE_ETHER_TYPE_QINQ))) {<br>
<br>
</span><span lang="EN-US" style="font-size:11.0pt">></span><span style="font-size:11.0pt">Two details:<br>
</span><span lang="EN-US" style="font-size:11.0pt">></span><span style="font-size:11.0pt">1. Lines continuing on the next line must be indented by two TABs.<br>
</span><span lang="EN-US" style="font-size:11.0pt">></span><span style="font-size:11.0pt">2. Please add the boundary check, as requested by Stephen. (I realize this mail may have crossed my response arguing why it is required.)<br>
<br>
</span><span lang="EN-US" style="font-size:11.0pt"><o:p></o:p></span></p>
<p class="MsoNormal" style="margin-bottom:12.0pt"><span lang="EN-US" style="font-size:11.0pt">Sure, I’ve sent a V3 having these fixed thanks for your review and comments.<o:p></o:p></span></p>
<p class="MsoNormal" style="margin-bottom:12.0pt"><span style="font-size:11.0pt"><br>
</span><span lang="EN-US" style="font-size:11.0pt">></span><span style="font-size:11.0pt">> + vlan_hdr++;<br>
</span><span lang="EN-US" style="font-size:11.0pt">></span><span style="font-size:11.0pt">> ethertype = vlan_hdr->eth_proto;<br>
</span><span lang="EN-US" style="font-size:11.0pt">></span><span style="font-size:11.0pt">> }<br>
</span><span lang="EN-US" style="font-size:11.0pt">></span><span style="font-size:11.0pt">> return ethertype;<br>
</span><span lang="EN-US" style="font-size:11.0pt">></span><span style="font-size:11.0pt">> --<br>
</span><span lang="EN-US" style="font-size:11.0pt">></span><span style="font-size:11.0pt">> 2.39.5 (Apple Git-154)<o:p></o:p></span></p>
</div>
</div>
</div>
</div>
</div>
</body>
</html>