diff options
| author | Maarten Lankhorst <dev@lankhorst.se> | 2026-04-09 16:26:59 +0200 |
|---|---|---|
| committer | Dave Airlie <airlied@redhat.com> | 2026-04-10 06:54:49 +1000 |
| commit | 9fb0106249ca3e01d60c15d4f5592cd58a9164b0 (patch) | |
| tree | 63fd1054ffbf0e01027fcdca17f0ce671b9fb405 /drivers | |
| parent | 3b053cd71598f7769f41b4f01f4540aab2e77b93 (diff) | |
drm/ttm/tests: Remove checks from ttm_pool_free_no_dma_alloc
On !x86, the pool type is never initialised, and the pages are freed
back to the system.
The test broke on the list_lru rewrite, but I'm not sure how that it was
supposed to work previously. In the meantime CI is broken so reverting
for now.
Fixes: 444e2a19d7fd ("ttm/pool: port to list_lru. (v2)")
Cc: Christian Koenig <christian.koenig@amd.com>
Cc: Johannes Weiner <hannes@cmpxchg.org>
Cc: Dave Chinner <david@fromorbit.com>
Cc: Christian König <christian.koenig@amd.com>
Cc: Dave Airlie <airlied@redhat.com>
Signed-off-by: Maarten Lankhorst <dev@lankhorst.se>
Signed-off-by: Dave Airlie <airlied@redhat.com>
Link: https://patch.msgid.link/20260409142658.1511941-2-dev@lankhorst.se
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/gpu/drm/ttm/tests/ttm_pool_test.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/drivers/gpu/drm/ttm/tests/ttm_pool_test.c b/drivers/gpu/drm/ttm/tests/ttm_pool_test.c index 01197014b83f..be75c8abf388 100644 --- a/drivers/gpu/drm/ttm/tests/ttm_pool_test.c +++ b/drivers/gpu/drm/ttm/tests/ttm_pool_test.c @@ -368,7 +368,6 @@ static void ttm_pool_free_no_dma_alloc(struct kunit *test) struct ttm_test_devices *devs = priv->devs; struct ttm_tt *tt; struct ttm_pool *pool; - struct ttm_pool_type *pt; enum ttm_caching caching = ttm_uncached; unsigned int order = 2; size_t size = (1 << order) * PAGE_SIZE; @@ -382,14 +381,9 @@ static void ttm_pool_free_no_dma_alloc(struct kunit *test) ttm_pool_init(pool, devs->dev, NUMA_NO_NODE, 0); ttm_pool_alloc(pool, tt, &simple_ctx); - pt = &pool->caching[caching].orders[order]; - KUNIT_ASSERT_TRUE(test, list_lru_count(&pt->pages) == 1); - ttm_pool_free(pool, tt); ttm_tt_fini(tt); - KUNIT_ASSERT_TRUE(test, list_lru_count(&pt->pages) == 1); - ttm_pool_fini(pool); } |