Re: [PATCH v2 1/7] test/hash: fix bucket size in hash perf test
From: Wang, Yipeng1 <hidden>
Date: 2018-09-27 03:40:26
From: Wang, Yipeng1 <hidden>
Date: 2018-09-27 03:40:26
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?