Hey Ethan,<br><br>Thanks for the review. <br>I will make these changes before pushing the patches. <br><br>thanx!<br>mehak<br><br><div class="gmail_quote">On Sat, Oct 20, 2012 at 10:49 AM, Ethan Jackson <span dir="ltr">&lt;<a href="mailto:ethan@nicira.com" target="_blank">ethan@nicira.com</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">&gt; -    if (f-&gt;nw_proto == IPPROTO_ICMP) {<br>
&gt; +    if (f-&gt;dl_type == htons(ETH_TYPE_IP) &amp;&amp;<br>
&gt; +        f-&gt;nw_proto == IPPROTO_ICMP) {<br>
&gt;          format_be16_masked(s, &quot;icmp_type&quot;, f-&gt;tp_src, wc-&gt;masks.tp_src);<br>
&gt;          format_be16_masked(s, &quot;icmp_code&quot;, f-&gt;tp_dst, wc-&gt;masks.tp_dst);<br>
&gt; -    } else if (f-&gt;nw_proto == IPPROTO_ICMPV6) {<br>
&gt; +    } else if (f-&gt;dl_type == htons(ETH_TYPE_IPV6) &amp;&amp;<br>
&gt; +               f-&gt;nw_proto == IPPROTO_ICMPV6) {<br>
<br>
</div>Traditionally we&#39;ve put the &amp;&amp; on the next line so you end up with:<br>
<div class="im"><br>
&gt; +    if (f-&gt;dl_type == htons(ETH_TYPE_IP)<br>
</div>&gt; +        &amp;&amp; f-&gt;nw_proto == IPPROTO_ICMP) {<br>
<br>
Otherwise looks good, thanks.<br>
<br>
Acked-by: Ethan Jackson &lt;<a href="mailto:ethan@nicira.com">ethan@nicira.com</a>&gt;<br>
</blockquote></div><br>