On Wed, Aug 4, 2010 at 10:27 AM, Ben Pfaff <span dir="ltr">&lt;<a href="mailto:blp@nicira.com">blp@nicira.com</a>&gt;</span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
There is no need to add &quot;tag&quot; values with only a single bit set, but until<br>
now tag_set_add() would do so anyhow.  (That is not a bug, it is only a<br>
possible performance loss.)  It is likely that no existing caller ever<br>
tries to do this, but an upcoming commit will start creating 1-bit tags<br>
and thus introduce the possibility, so it seems like a good idea to avoid<br>
the problem before it starts.<br></blockquote><div><br>I don&#39;t like the concept of 1 bit tags (see the response to the next patch in this series) so rather than try to handle them I would prefer that we just ensure that they never occur.<br>
 </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
It is also not necessary to add a &quot;tag&quot; if all of the bits in it are<br>
already present in some member of the set, so add that optimization too.<br></blockquote><div><br>This part looks fine. <br></div></div>