vppinfra: move hash bitmap out of vec header

Type: refactor
Change-Id: Ibd29a717eaf12d795b3bceb31835d6fc655268b1
Signed-off-by: Damjan Marion <damarion@cisco.com>
This commit is contained in:
Damjan Marion 2022-03-17 17:54:48 +01:00 committed by Florin Coras
parent d591b82cc8
commit 2b702da86c
2 changed files with 11 additions and 8 deletions

View File

@ -753,14 +753,19 @@ _hash_create (uword elts, hash_t * h_user)
/* data bytes: */
(elts << log2_pair_size) * sizeof (hash_pair_t),
/* header bytes: */
sizeof (h[0]) +
(elts / BITS (h->is_user[0])) * sizeof (h->is_user[0]),
sizeof (h[0]),
/* alignment */ sizeof (hash_pair_t));
h = hash_header (v);
if (h_user)
h[0] = h_user[0];
{
h[0] = h_user[0];
h->is_user = 0;
}
vec_validate_aligned (
h->is_user, ((elts / BITS (h->is_user[0])) * sizeof (h->is_user[0])) - 1,
CLIB_CACHE_LINE_BYTES);
h->log2_pair_size = log2_pair_size;
h->elts = 0;
@ -800,6 +805,7 @@ _hash_free (void *v)
clib_mem_free (p->indirect.pairs);
}
vec_free (h->is_user);
vec_free_header (h);
return 0;

View File

@ -93,17 +93,14 @@ typedef struct hash_header
/* Bit i is set if pair i is a user object (as opposed to being
either zero or an indirect array of pairs). */
uword is_user[0];
uword *is_user;
} hash_t;
/* Hash header size in bytes */
always_inline uword
hash_header_bytes (void *v)
{
hash_t *h;
uword is_user_bytes =
(sizeof (h->is_user[0]) * vec_len (v)) / BITS (h->is_user[0]);
return sizeof (h[0]) + is_user_bytes;
return sizeof (hash_t);
}
/* Returns a pointer to the hash header given the vector pointer */