<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On 21 May 2014 02:52, Ryan Wilson <span dir="ltr">&lt;<a href="mailto:wryan@nicira.com" target="_blank">wryan@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">


When adding support for Openflow group and bucket stats, a group entry is added<br>
to the xlate cache. If the group is removed from ofproto configuration, we need<br>
to guarantee the group exists in the case where stats are attributed to the<br>
group by a handler thread and the xlate cache has not yet been cleared by a<br>
revalidator thread.</blockquote><div><br></div><div>A bit more accurate would be:</div><div><br></div><div>&quot;When adding support for OpenFlow group and bucket stats, a group entry is added</div><div>to the xlate_cache. If the main thread removes the group from an ofproto, we need to</div>


<div>guarantee that remains accessible to users of the xlate_cache, as the xlate_cache</div><div>will not be cleaned up until the corresponding datapath flows are revalidated.&quot;</div><div><br></div><div>(Reiterating, on current master the handler threads don&#39;t handle xlate_cache at all, although I&#39;m working on changing this)</div>


</div></div></div>