[gabbayo:amdkfd-next 6/28] drivers/gpu//drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c:632:3: error: implicit declaration of function 'release_pages'; did you mean 'release_task'?
tree: git://people.freedesktop.org/~gabbayo/linux amdkfd-next head: 8feaccf71dd61f2201493068055e0d1d699014df commit: 5ae0283e831a94c714fce61063e4724baf364ef3 [6/28] drm/amdgpu: Add userptr support for KFD config: sparc64-allmodconfig (attached as .config) compiler: sparc64-linux-gnu-gcc (Debian 7.2.0-11) 7.2.0 reproduce: wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross git checkout 5ae0283e831a94c714fce61063e4724baf364ef3 # save the attached .config to linux build tree make.cross ARCH=sparc64
All errors (new ones prefixed by >>):
drivers/gpu//drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c: In function 'init_user_pages':
drivers/gpu//drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c:632:3: error: implicit declaration of function 'release_pages'; did you mean 'release_task'? [-Werror=implicit-function-declaration]
release_pages(mem->user_pages, bo->tbo.ttm->num_pages); ^~~~~~~~~~~~~ release_task cc1: some warnings being treated as errors
vim +632 drivers/gpu//drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c
561 562 /* Initializes user pages. It registers the MMU notifier and validates 563 * the userptr BO in the GTT domain. 564 * 565 * The BO must already be on the userptr_valid_list. Otherwise an 566 * eviction and restore may happen that leaves the new BO unmapped 567 * with the user mode queues running. 568 * 569 * Takes the process_info->lock to protect against concurrent restore 570 * workers. 571 * 572 * Returns 0 for success, negative errno for errors. 573 */ 574 static int init_user_pages(struct kgd_mem *mem, struct mm_struct *mm, 575 uint64_t user_addr) 576 { 577 struct amdkfd_process_info *process_info = mem->process_info; 578 struct amdgpu_bo *bo = mem->bo; 579 struct ttm_operation_ctx ctx = { true, false }; 580 int ret = 0; 581 582 mutex_lock(&process_info->lock); 583 584 ret = amdgpu_ttm_tt_set_userptr(bo->tbo.ttm, user_addr, 0); 585 if (ret) { 586 pr_err("%s: Failed to set userptr: %d\n", __func__, ret); 587 goto out; 588 } 589 590 ret = amdgpu_mn_register(bo, user_addr); 591 if (ret) { 592 pr_err("%s: Failed to register MMU notifier: %d\n", 593 __func__, ret); 594 goto out; 595 } 596 597 /* If no restore worker is running concurrently, user_pages 598 * should not be allocated 599 */ 600 WARN(mem->user_pages, "Leaking user_pages array"); 601 602 mem->user_pages = kvmalloc_array(bo->tbo.ttm->num_pages, 603 sizeof(struct page *), 604 GFP_KERNEL | __GFP_ZERO); 605 if (!mem->user_pages) { 606 pr_err("%s: Failed to allocate pages array\n", __func__); 607 ret = -ENOMEM; 608 goto unregister_out; 609 } 610 611 ret = amdgpu_ttm_tt_get_user_pages(bo->tbo.ttm, mem->user_pages); 612 if (ret) { 613 pr_err("%s: Failed to get user pages: %d\n", __func__, ret); 614 goto free_out; 615 } 616 617 amdgpu_ttm_tt_set_user_pages(bo->tbo.ttm, mem->user_pages); 618 619 ret = amdgpu_bo_reserve(bo, true); 620 if (ret) { 621 pr_err("%s: Failed to reserve BO\n", __func__); 622 goto release_out; 623 } 624 amdgpu_ttm_placement_from_domain(bo, mem->domain); 625 ret = ttm_bo_validate(&bo->tbo, &bo->placement, &ctx); 626 if (ret) 627 pr_err("%s: failed to validate BO\n", __func__); 628 amdgpu_bo_unreserve(bo); 629 630 release_out: 631 if (ret)
632 release_pages(mem->user_pages, bo->tbo.ttm->num_pages);
633 free_out: 634 kvfree(mem->user_pages); 635 mem->user_pages = NULL; 636 unregister_out: 637 if (ret) 638 amdgpu_mn_unregister(bo); 639 out: 640 mutex_unlock(&process_info->lock); 641 return ret; 642 } 643
--- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation
We need to #include <linux/pagemap.h>.
On 2018-05-15 09:44 AM, kbuild test robot wrote:
tree: git://people.freedesktop.org/~gabbayo/linux amdkfd-next head: 8feaccf71dd61f2201493068055e0d1d699014df commit: 5ae0283e831a94c714fce61063e4724baf364ef3 [6/28] drm/amdgpu: Add userptr support for KFD config: sparc64-allmodconfig (attached as .config) compiler: sparc64-linux-gnu-gcc (Debian 7.2.0-11) 7.2.0 reproduce: wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross git checkout 5ae0283e831a94c714fce61063e4724baf364ef3 # save the attached .config to linux build tree make.cross ARCH=sparc64
All errors (new ones prefixed by >>):
drivers/gpu//drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c: In function 'init_user_pages':
drivers/gpu//drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c:632:3: error: implicit declaration of function 'release_pages'; did you mean 'release_task'? [-Werror=implicit-function-declaration]
release_pages(mem->user_pages, bo->tbo.ttm->num_pages); ^~~~~~~~~~~~~ release_taskcc1: some warnings being treated as errors
vim +632 drivers/gpu//drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c
561 562 /* Initializes user pages. It registers the MMU notifier and validates 563 * the userptr BO in the GTT domain. 564 * 565 * The BO must already be on the userptr_valid_list. Otherwise an 566 * eviction and restore may happen that leaves the new BO unmapped 567 * with the user mode queues running. 568 * 569 * Takes the process_info->lock to protect against concurrent restore 570 * workers. 571 * 572 * Returns 0 for success, negative errno for errors. 573 */ 574 static int init_user_pages(struct kgd_mem *mem, struct mm_struct *mm, 575 uint64_t user_addr) 576 { 577 struct amdkfd_process_info *process_info = mem->process_info; 578 struct amdgpu_bo *bo = mem->bo; 579 struct ttm_operation_ctx ctx = { true, false }; 580 int ret = 0; 581 582 mutex_lock(&process_info->lock); 583 584 ret = amdgpu_ttm_tt_set_userptr(bo->tbo.ttm, user_addr, 0); 585 if (ret) { 586 pr_err("%s: Failed to set userptr: %d\n", __func__, ret); 587 goto out; 588 } 589 590 ret = amdgpu_mn_register(bo, user_addr); 591 if (ret) { 592 pr_err("%s: Failed to register MMU notifier: %d\n", 593 __func__, ret); 594 goto out; 595 } 596 597 /* If no restore worker is running concurrently, user_pages 598 * should not be allocated 599 */ 600 WARN(mem->user_pages, "Leaking user_pages array"); 601 602 mem->user_pages = kvmalloc_array(bo->tbo.ttm->num_pages, 603 sizeof(struct page *), 604 GFP_KERNEL | __GFP_ZERO); 605 if (!mem->user_pages) { 606 pr_err("%s: Failed to allocate pages array\n", __func__); 607 ret = -ENOMEM; 608 goto unregister_out; 609 } 610 611 ret = amdgpu_ttm_tt_get_user_pages(bo->tbo.ttm, mem->user_pages); 612 if (ret) { 613 pr_err("%s: Failed to get user pages: %d\n", __func__, ret); 614 goto free_out; 615 } 616 617 amdgpu_ttm_tt_set_user_pages(bo->tbo.ttm, mem->user_pages); 618 619 ret = amdgpu_bo_reserve(bo, true); 620 if (ret) { 621 pr_err("%s: Failed to reserve BO\n", __func__); 622 goto release_out; 623 } 624 amdgpu_ttm_placement_from_domain(bo, mem->domain); 625 ret = ttm_bo_validate(&bo->tbo, &bo->placement, &ctx); 626 if (ret) 627 pr_err("%s: failed to validate BO\n", __func__); 628 amdgpu_bo_unreserve(bo); 629 630 release_out: 631 if (ret)
632 release_pages(mem->user_pages, bo->tbo.ttm->num_pages);
633 free_out: 634 kvfree(mem->user_pages); 635 mem->user_pages = NULL; 636 unregister_out: 637 if (ret) 638 amdgpu_mn_unregister(bo); 639 out: 640 mutex_unlock(&process_info->lock); 641 return ret; 642 } 643
0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation
dri-devel@lists.freedesktop.org
-
Felix Kuehling -
kbuild test robot