On Tue, Jun 18, 2013 at 05:09:58PM +0000, Rajahalme, Jarno (NSN - FI/Espoo)
wrote:
> All looks good and tests pass. However, I'd probably add a comment
> on the hindex.mask (in hindex.h) stating that it has a number of
> lowest order bits set, others cleared. After realizing that reading
> the code became a bit easier..
That is a good idea. I wrote it like this:
size_t mask; /* 0 or more lowest-order bits set, others cleared. */
> I also changed the cookie indexing to use this and it also passes
> all tests and seems to work as expected in my performance tests.
Thank you. I will apply this soon.
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev