Hi Tvrtko,

kernel test robot noticed the following build errors:

[auto build test ERROR on drm/drm-next]
[also build test ERROR on drm-i915/for-linux-next drm-i915/for-linux-next-fixes 
drm-xe/drm-xe-next drm-exynos/exynos-drm-next drm-intel/for-linux-next 
drm-intel/for-linux-next-fixes drm-misc/drm-misc-next drm-tip/drm-tip 
linus/master v6.17 next-20251009]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]

url:    
https://github.com/intel-lab-lkp/linux/commits/Tvrtko-Ursulin/drm-ttm-Add-getter-for-some-pool-properties/20251010-052711
base:   git://anongit.freedesktop.org/drm/drm drm-next
patch link:    
https://lore.kernel.org/r/20251008115314.55438-3-tvrtko.ursulin%40igalia.com
patch subject: [PATCH v3 2/5] drm/ttm: Replace multiple booleans with flags in 
pool init
config: sh-randconfig-002-20251010 
(https://download.01.org/0day-ci/archive/20251010/[email protected]/config)
compiler: sh4-linux-gcc (GCC) 15.1.0
reproduce (this is a W=1 build): 
(https://download.01.org/0day-ci/archive/20251010/[email protected]/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <[email protected]>
| Closes: 
https://lore.kernel.org/oe-kbuild-all/[email protected]/

All errors (new ones prefixed by >>):

   drivers/gpu/drm/ttm/tests/ttm_pool_test.c: In function 
'ttm_pool_alloc_basic':
>> drivers/gpu/drm/ttm/tests/ttm_pool_test.c:167:21: error: implicit 
>> declaration of function 'ttm_pool_uses_dma_alloc'; did you mean 
>> 'ttm_pool_restore_and_alloc'? [-Wimplicit-function-declaration]
     167 |                 if (ttm_pool_uses_dma_alloc(pool)) {
         |                     ^~~~~~~~~~~~~~~~~~~~~~~
         |                     ttm_pool_restore_and_alloc
--
   drivers/gpu/drm/ttm/tests/ttm_device_test.c: In function 
'ttm_device_init_pools':
>> drivers/gpu/drm/ttm/tests/ttm_device_test.c:177:37: error: implicit 
>> declaration of function 'ttm_pool_uses_dma_alloc'; did you mean 
>> 'ttm_pool_restore_and_alloc'? [-Wimplicit-function-declaration]
     177 |                                 if (ttm_pool_uses_dma_alloc(pool))
         |                                     ^~~~~~~~~~~~~~~~~~~~~~~
         |                                     ttm_pool_restore_and_alloc


vim +167 drivers/gpu/drm/ttm/tests/ttm_pool_test.c

   130  
   131  KUNIT_ARRAY_PARAM(ttm_pool_alloc_basic, ttm_pool_basic_cases,
   132                    ttm_pool_alloc_case_desc);
   133  
   134  static void ttm_pool_alloc_basic(struct kunit *test)
   135  {
   136          struct ttm_pool_test_priv *priv = test->priv;
   137          struct ttm_test_devices *devs = priv->devs;
   138          const struct ttm_pool_test_case *params = test->param_value;
   139          struct ttm_tt *tt;
   140          struct ttm_pool *pool;
   141          struct page *fst_page, *last_page;
   142          enum ttm_caching caching = ttm_uncached;
   143          unsigned int expected_num_pages = 1 << params->order;
   144          size_t size = expected_num_pages * PAGE_SIZE;
   145          int err;
   146  
   147          tt = ttm_tt_kunit_init(test, 0, caching, size);
   148          KUNIT_ASSERT_NOT_NULL(test, tt);
   149  
   150          pool = kunit_kzalloc(test, sizeof(*pool), GFP_KERNEL);
   151          KUNIT_ASSERT_NOT_NULL(test, pool);
   152  
   153          ttm_pool_init(pool, devs->dev, NUMA_NO_NODE, params->flags);
   154  
   155          KUNIT_ASSERT_PTR_EQ(test, pool->dev, devs->dev);
   156          KUNIT_ASSERT_EQ(test, pool->nid, NUMA_NO_NODE);
   157          KUNIT_ASSERT_EQ(test, pool->flags, params->flags);
   158  
   159          err = ttm_pool_alloc(pool, tt, &simple_ctx);
   160          KUNIT_ASSERT_EQ(test, err, 0);
   161          KUNIT_ASSERT_EQ(test, tt->num_pages, expected_num_pages);
   162  
   163          fst_page = tt->pages[0];
   164          last_page = tt->pages[tt->num_pages - 1];
   165  
   166          if (params->order <= MAX_PAGE_ORDER) {
 > 167                  if (ttm_pool_uses_dma_alloc(pool)) {
   168                          KUNIT_ASSERT_NOT_NULL(test, (void 
*)fst_page->private);
   169                          KUNIT_ASSERT_NOT_NULL(test, (void 
*)last_page->private);
   170                  } else {
   171                          KUNIT_ASSERT_EQ(test, fst_page->private, 
params->order);
   172                  }
   173          } else {
   174                  if (ttm_pool_uses_dma_alloc(pool)) {
   175                          KUNIT_ASSERT_NOT_NULL(test, (void 
*)fst_page->private);
   176                          KUNIT_ASSERT_NULL(test, (void 
*)last_page->private);
   177                  } else {
   178                          /*
   179                           * We expect to alloc one big block, followed by
   180                           * order 0 blocks
   181                           */
   182                          KUNIT_ASSERT_EQ(test, fst_page->private,
   183                                          min_t(unsigned int, 
MAX_PAGE_ORDER,
   184                                                params->order));
   185                          KUNIT_ASSERT_EQ(test, last_page->private, 0);
   186                  }
   187          }
   188  
   189          ttm_pool_free(pool, tt);
   190          ttm_tt_fini(tt);
   191          ttm_pool_fini(pool);
   192  }
   193  

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

Reply via email to