[igt-dev] [PATCH i-g-t 3/3] lib/i915/gem_create: Use hash helper for gem bo pool

Mauro Carvalho Chehab mauro.chehab at linux.intel.com
Thu Feb 23 11:26:55 UTC 2023


On Thu, 23 Feb 2023 11:53:21 +0100
Zbigniew Kempczyński <zbigniew.kempczynski at intel.com> wrote:

> Remove local 64-bit hash and use common one defined in igt_map.
> 
> Signed-off-by: Zbigniew Kempczyński <zbigniew.kempczynski at intel.com>

Reviewed-by: Mauro Carvalho Chehab <mchehab at kernel.org>

> ---
>  lib/i915/gem_create.c | 11 +----------
>  1 file changed, 1 insertion(+), 10 deletions(-)
> 
> diff --git a/lib/i915/gem_create.c b/lib/i915/gem_create.c
> index 1f9b7fcc01..99c839d7db 100644
> --- a/lib/i915/gem_create.c
> +++ b/lib/i915/gem_create.c
> @@ -252,15 +252,6 @@ void gem_pool_dump(void)
>  	pthread_mutex_unlock(&pool_mutex);
>  }
>  
> -#define GOLDEN_RATIO_PRIME_64 0x9e37fffffffc0001ULL
> -static inline uint32_t hash_pool(const void *val)
> -{
> -	uint64_t hash = *(uint64_t *) val;
> -
> -	hash = hash * GOLDEN_RATIO_PRIME_64;
> -	return hash >> 32;
> -}
> -
>  static int equal_pool(const void *a, const void *b)
>  {
>  	struct pool_list *p1 = (struct pool_list *) a;
> @@ -356,7 +347,7 @@ void gem_pool_init(void)
>  {
>  	pthread_mutex_init(&pool_mutex, NULL);
>  	__destroy_pool(pool, &pool_mutex);
> -	pool = igt_map_create(hash_pool, equal_pool);
> +	pool = igt_map_create(igt_map_hash_64, equal_pool);
>  }
>  
>  igt_constructor {


More information about the igt-dev mailing list