<p dir="ltr">Based on the commit message it isn&#39;t clear to me why we shouldn&#39;t cover ARP here. (I don&#39;t have easy access to the full source code at the moment, so maybe this implies a source code revision and maybe a commit message revision and maybe I just misunderstand.)</p>

<div class="gmail_quote">On Feb 21, 2013 8:50 PM, &quot;Justin Pettit&quot; &lt;<a href="mailto:jpettit@nicira.com">jpettit@nicira.com</a>&gt; wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
When printing a match, we would print &quot;tp_src&quot; and &quot;tp_dst&quot; if the<br>
packet wasn&#39;t ICMPv4 or ICMPv6.  Unfortunately, this doesn&#39;t cover ARP.<br>
This changes the check to only print those keys if the network protocol<br>
is TCP or UDP.<br>
<br>
Signed-off-by: Justin Pettit &lt;<a href="mailto:jpettit@nicira.com">jpettit@nicira.com</a>&gt;<br>
---<br>
 lib/match.c |    3 ++-<br>
 1 files changed, 2 insertions(+), 1 deletions(-)<br>
<br>
diff --git a/lib/match.c b/lib/match.c<br>
index f4b0a6c..2395fb4 100644<br>
--- a/lib/match.c<br>
+++ b/lib/match.c<br>
@@ -1053,7 +1053,8 @@ match_format(const struct match *match, struct ds *s, unsigned int priority)<br>
                             &amp;wc-&gt;masks.nd_target);<br>
         format_eth_masked(s, &quot;nd_sll&quot;, f-&gt;arp_sha, wc-&gt;masks.arp_sha);<br>
         format_eth_masked(s, &quot;nd_tll&quot;, f-&gt;arp_tha, wc-&gt;masks.arp_tha);<br>
-    } else {<br>
+    } else if (f-&gt;nw_proto == IPPROTO_TCP ||<br>
+               f-&gt;nw_proto == IPPROTO_UDP) {<br>
         format_be16_masked(s, &quot;tp_src&quot;, f-&gt;tp_src, wc-&gt;masks.tp_src);<br>
         format_be16_masked(s, &quot;tp_dst&quot;, f-&gt;tp_dst, wc-&gt;masks.tp_dst);<br>
     }<br>
--<br>
1.7.5.4<br>
<br>
_______________________________________________<br>
dev mailing list<br>
<a href="mailto:dev@openvswitch.org">dev@openvswitch.org</a><br>
<a href="http://openvswitch.org/mailman/listinfo/dev" target="_blank">http://openvswitch.org/mailman/listinfo/dev</a><br>
</blockquote></div>