Hi Bruce, In the test, the bucket size and number of buckets are used to map to the underneath rte_hash structure. They are used to test performance of two scenarios: keys in primary buckets only and keys in both primary and secondary buckets.
Although there is no functional issue with bucket size set to 4, it mismatches the underneath rte_hash structure (i.e. 8), which may affect code readability and future extension. I added this description into the commit message. Thanks Yipeng >-----Original Message----- > >Can you perhaps give a little detail on what actual problems this caused. >Did it just mean that we used up too much memory in the test because we >thought there were more buckets than there were, or something else? >