[PATCH RESEND v9 1/8] drm/ttm/tests: Fix argument in ttm_tt_kunit_init()

Karolina Stolarek karolina.stolarek at intel.com
Thu Jan 11 15:54:52 UTC 2024


On 11.01.2024 15:59, Christian König wrote:
> I've gone ahead and pushed this one to drm-misc-next.
> 
> Going to take a look at the rest as well.

Awesome, thanks a lot!

All the best,
Karolina

> 
> @Amar could you run another test round with those? And maybe ping Kenny 
> if we can integrate them into the CI.
> 
> Thanks,
> Christian.
> 
> Am 11.01.24 um 11:28 schrieb Karolina Stolarek:
>> Remove a leftover definition of page order and pass an empty flag value
>> in ttm_pool_pre_populated().
>>
>> Signed-off-by: Karolina Stolarek <karolina.stolarek at intel.com>
>> Tested-by: Amaranath Somalapuram <Amaranath.Somalapuram at amd.com>
>> Reviewed-by: Dominik Karol Piątkowski 
>> <dominik.karol.piatkowski at intel.com>
>> Acked-by: Christian König <christian.koenig at amd.com>
>> ---
>>   drivers/gpu/drm/ttm/tests/ttm_pool_test.c | 3 +--
>>   1 file changed, 1 insertion(+), 2 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/ttm/tests/ttm_pool_test.c 
>> b/drivers/gpu/drm/ttm/tests/ttm_pool_test.c
>> index 2d9cae8cd984..b97f7b6daf5b 100644
>> --- a/drivers/gpu/drm/ttm/tests/ttm_pool_test.c
>> +++ b/drivers/gpu/drm/ttm/tests/ttm_pool_test.c
>> @@ -78,10 +78,9 @@ static struct ttm_pool 
>> *ttm_pool_pre_populated(struct kunit *test,
>>       struct ttm_test_devices *devs = priv->devs;
>>       struct ttm_pool *pool;
>>       struct ttm_tt *tt;
>> -    unsigned long order = __fls(size / PAGE_SIZE);
>>       int err;
>> -    tt = ttm_tt_kunit_init(test, order, caching, size);
>> +    tt = ttm_tt_kunit_init(test, 0, caching, size);
>>       KUNIT_ASSERT_NOT_NULL(test, tt);
>>       pool = kunit_kzalloc(test, sizeof(*pool), GFP_KERNEL);
> 


More information about the dri-devel mailing list