From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from smtp1.osuosl.org (smtp1.osuosl.org [140.211.166.138]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 4E5DEC433EF for ; Fri, 10 Dec 2021 23:30:31 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by smtp1.osuosl.org (Postfix) with ESMTP id DFB5D8568E; Fri, 10 Dec 2021 23:30:30 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Received: from smtp1.osuosl.org ([127.0.0.1]) by localhost (smtp1.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id I5xziG4BT_xT; Fri, 10 Dec 2021 23:30:29 +0000 (UTC) Received: from lists.linuxfoundation.org (lf-lists.osuosl.org [140.211.9.56]) by smtp1.osuosl.org (Postfix) with ESMTPS id 5010A856F9; Fri, 10 Dec 2021 23:30:29 +0000 (UTC) Received: from lf-lists.osuosl.org (localhost [127.0.0.1]) by lists.linuxfoundation.org (Postfix) with ESMTP id 25275C002F; Fri, 10 Dec 2021 23:30:29 +0000 (UTC) Received: from smtp3.osuosl.org (smtp3.osuosl.org [140.211.166.136]) by lists.linuxfoundation.org (Postfix) with ESMTP id 31666C0012 for ; Fri, 10 Dec 2021 23:30:28 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by smtp3.osuosl.org (Postfix) with ESMTP id 0A5A861B42 for ; Fri, 10 Dec 2021 23:30:28 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Authentication-Results: smtp3.osuosl.org (amavisd-new); dkim=pass (2048-bit key) header.d=intel.com Received: from smtp3.osuosl.org ([127.0.0.1]) by localhost (smtp3.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id aM0IN7Lt0NBQ for ; Fri, 10 Dec 2021 23:30:26 +0000 (UTC) X-Greylist: domain auto-whitelisted by SQLgrey-1.8.0 Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by smtp3.osuosl.org (Postfix) with ESMTPS id 1FB1F61527 for ; Fri, 10 Dec 2021 23:30:25 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1639179026; x=1670715026; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=m2GhuwJ4UnyKyqy83lVmEBmiMbGKiHLXbCCHqnR/ajM=; b=Lzg4HB+yXdbnhHvGmgzE/nfIaCijjppyQBRkZh6CHJoSKZduwFKcIYjx xMgepYpy7xG//P7ZOG2c76TZO1ObO/99OsGcLEADFz5vTY3INiKpyvOmD BoUin3MNEDTiiXqdl2JRlJo+FWR3GD7E71pPM+v9poyno4N7koIInjN9l AJYnmVkfndO7kOfzGP91ceo/BC0A3N/tqM0cx5k9YZxbaKVg4Ut9t95BO RMbd7zhMKNnVj3skNIKmzJTxqlbYTkKTKowlebMZYV93keB4ydCgtwkpU 7fcBQk/q7ZbVrwL+jeESawIq8ll0KfyQ9BjpYDhU9OrpbganzgA7PS5QE w==; X-IronPort-AV: E=McAfee;i="6200,9189,10194"; a="225752890" X-IronPort-AV: E=Sophos;i="5.88,196,1635231600"; d="scan'208";a="225752890" Received: from fmsmga008.fm.intel.com ([10.253.24.58]) by orsmga101.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 10 Dec 2021 15:30:24 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.88,196,1635231600"; d="scan'208";a="565479477" Received: from lkp-server02.sh.intel.com (HELO 9e1e9f9b3bcb) ([10.239.97.151]) by fmsmga008.fm.intel.com with ESMTP; 10 Dec 2021 15:30:21 -0800 Received: from kbuild by 9e1e9f9b3bcb with local (Exim 4.92) (envelope-from ) id 1mvpLI-0003nt-Si; Fri, 10 Dec 2021 23:30:20 +0000 Date: Sat, 11 Dec 2021 07:30:01 +0800 From: kernel test robot To: Robin Murphy , joro@8bytes.org, will@kernel.org Subject: Re: [PATCH v2 11/11] iommu: Move flush queue data into iommu_dma_cookie Message-ID: <202112110753.vYbSlMnq-lkp@intel.com> References: MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.10.1 (2018-07-13) Cc: kbuild-all@lists.01.org, linux-kernel@vger.kernel.org, willy@infradead.org, linux-mm@kvack.org, iommu@lists.linux-foundation.org X-BeenThere: iommu@lists.linux-foundation.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: Development issues for Linux IOMMU support List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: iommu-bounces@lists.linux-foundation.org Sender: "iommu" Hi Robin, I love your patch! Yet something to improve: [auto build test ERROR on joro-iommu/next] [also build test ERROR on tegra/for-next v5.16-rc4] [cannot apply to tegra-drm/drm/tegra/for-next] [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] url: https://github.com/0day-ci/linux/commits/Robin-Murphy/iommu-refactor-flush-queues-into-iommu-dma/20211211-015635 base: https://git.kernel.org/pub/scm/linux/kernel/git/joro/iommu.git next config: arm-randconfig-r013-20211210 (https://download.01.org/0day-ci/archive/20211211/202112110753.vYbSlMnq-lkp@intel.com/config) compiler: arm-linux-gnueabi-gcc (GCC) 11.2.0 reproduce (this is a W=1 build): wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # https://github.com/0day-ci/linux/commit/3b6adb4a8ec42d7b5c1b3b1af2c857a2375fd7e1 git remote add linux-review https://github.com/0day-ci/linux git fetch --no-tags linux-review Robin-Murphy/iommu-refactor-flush-queues-into-iommu-dma/20211211-015635 git checkout 3b6adb4a8ec42d7b5c1b3b1af2c857a2375fd7e1 # save the config file to linux build tree mkdir build_dir COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross O=build_dir ARCH=arm SHELL=/bin/bash drivers/gpu/drm/tegra/ drivers/iommu/ If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot All errors (new ones prefixed by >>): drivers/gpu/drm/tegra/hub.c: In function 'tegra_display_hub_probe': >> drivers/gpu/drm/tegra/hub.c:1043:24: error: implicit declaration of function 'dma_get_mask'; did you mean 'xa_get_mark'? [-Werror=implicit-function-declaration] 1043 | u64 dma_mask = dma_get_mask(pdev->dev.parent); | ^~~~~~~~~~~~ | xa_get_mark >> drivers/gpu/drm/tegra/hub.c:1050:15: error: implicit declaration of function 'dma_coerce_mask_and_coherent' [-Werror=implicit-function-declaration] 1050 | err = dma_coerce_mask_and_coherent(&pdev->dev, dma_mask); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~ cc1: some warnings being treated as errors -- drivers/gpu/drm/tegra/plane.c: In function 'tegra_plane_reset': >> drivers/gpu/drm/tegra/plane.c:46:42: error: 'DMA_MAPPING_ERROR' undeclared (first use in this function) 46 | state->iova[i] = DMA_MAPPING_ERROR; | ^~~~~~~~~~~~~~~~~ drivers/gpu/drm/tegra/plane.c:46:42: note: each undeclared identifier is reported only once for each function it appears in drivers/gpu/drm/tegra/plane.c: In function 'tegra_plane_atomic_duplicate_state': drivers/gpu/drm/tegra/plane.c:76:33: error: 'DMA_MAPPING_ERROR' undeclared (first use in this function) 76 | copy->iova[i] = DMA_MAPPING_ERROR; | ^~~~~~~~~~~~~~~~~ drivers/gpu/drm/tegra/plane.c: In function 'tegra_dc_pin': >> drivers/gpu/drm/tegra/plane.c:170:31: error: implicit declaration of function 'dma_map_sgtable'; did you mean 'iommu_map_sgtable'? [-Werror=implicit-function-declaration] 170 | err = dma_map_sgtable(dc->dev, sgt, DMA_TO_DEVICE, 0); | ^~~~~~~~~~~~~~~ | iommu_map_sgtable >> drivers/gpu/drm/tegra/plane.c:170:61: error: 'DMA_TO_DEVICE' undeclared (first use in this function); did you mean 'MT_DEVICE'? 170 | err = dma_map_sgtable(dc->dev, sgt, DMA_TO_DEVICE, 0); | ^~~~~~~~~~~~~ | MT_DEVICE >> drivers/gpu/drm/tegra/plane.c:202:25: error: implicit declaration of function 'dma_unmap_sgtable'; did you mean 'iommu_map_sgtable'? [-Werror=implicit-function-declaration] 202 | dma_unmap_sgtable(dc->dev, sgt, DMA_TO_DEVICE, 0); | ^~~~~~~~~~~~~~~~~ | iommu_map_sgtable drivers/gpu/drm/tegra/plane.c:205:34: error: 'DMA_MAPPING_ERROR' undeclared (first use in this function) 205 | state->iova[i] = DMA_MAPPING_ERROR; | ^~~~~~~~~~~~~~~~~ drivers/gpu/drm/tegra/plane.c: In function 'tegra_dc_unpin': drivers/gpu/drm/tegra/plane.c:221:57: error: 'DMA_TO_DEVICE' undeclared (first use in this function); did you mean 'MT_DEVICE'? 221 | dma_unmap_sgtable(dc->dev, sgt, DMA_TO_DEVICE, 0); | ^~~~~~~~~~~~~ | MT_DEVICE drivers/gpu/drm/tegra/plane.c:224:34: error: 'DMA_MAPPING_ERROR' undeclared (first use in this function) 224 | state->iova[i] = DMA_MAPPING_ERROR; | ^~~~~~~~~~~~~~~~~ cc1: some warnings being treated as errors -- drivers/gpu/drm/tegra/dc.c: In function 'tegra_crtc_calculate_memory_bandwidth': drivers/gpu/drm/tegra/dc.c:2225:38: warning: variable 'old_state' set but not used [-Wunused-but-set-variable] 2225 | const struct drm_crtc_state *old_state; | ^~~~~~~~~ drivers/gpu/drm/tegra/dc.c: In function 'tegra_dc_probe': >> drivers/gpu/drm/tegra/dc.c:2978:24: error: implicit declaration of function 'dma_get_mask'; did you mean 'xa_get_mark'? [-Werror=implicit-function-declaration] 2978 | u64 dma_mask = dma_get_mask(pdev->dev.parent); | ^~~~~~~~~~~~ | xa_get_mark >> drivers/gpu/drm/tegra/dc.c:2982:15: error: implicit declaration of function 'dma_coerce_mask_and_coherent' [-Werror=implicit-function-declaration] 2982 | err = dma_coerce_mask_and_coherent(&pdev->dev, dma_mask); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~ cc1: some warnings being treated as errors vim +1043 drivers/gpu/drm/tegra/hub.c c4755fb9064f64 Thierry Reding 2017-11-13 1040 c4755fb9064f64 Thierry Reding 2017-11-13 1041 static int tegra_display_hub_probe(struct platform_device *pdev) c4755fb9064f64 Thierry Reding 2017-11-13 1042 { 86044e749be77a Thierry Reding 2021-03-26 @1043 u64 dma_mask = dma_get_mask(pdev->dev.parent); 0cffbde2e318cc Thierry Reding 2018-11-29 1044 struct device_node *child = NULL; c4755fb9064f64 Thierry Reding 2017-11-13 1045 struct tegra_display_hub *hub; 0cffbde2e318cc Thierry Reding 2018-11-29 1046 struct clk *clk; c4755fb9064f64 Thierry Reding 2017-11-13 1047 unsigned int i; c4755fb9064f64 Thierry Reding 2017-11-13 1048 int err; c4755fb9064f64 Thierry Reding 2017-11-13 1049 86044e749be77a Thierry Reding 2021-03-26 @1050 err = dma_coerce_mask_and_coherent(&pdev->dev, dma_mask); 86044e749be77a Thierry Reding 2021-03-26 1051 if (err < 0) { 86044e749be77a Thierry Reding 2021-03-26 1052 dev_err(&pdev->dev, "failed to set DMA mask: %d\n", err); 86044e749be77a Thierry Reding 2021-03-26 1053 return err; 86044e749be77a Thierry Reding 2021-03-26 1054 } 86044e749be77a Thierry Reding 2021-03-26 1055 c4755fb9064f64 Thierry Reding 2017-11-13 1056 hub = devm_kzalloc(&pdev->dev, sizeof(*hub), GFP_KERNEL); c4755fb9064f64 Thierry Reding 2017-11-13 1057 if (!hub) c4755fb9064f64 Thierry Reding 2017-11-13 1058 return -ENOMEM; c4755fb9064f64 Thierry Reding 2017-11-13 1059 c4755fb9064f64 Thierry Reding 2017-11-13 1060 hub->soc = of_device_get_match_data(&pdev->dev); c4755fb9064f64 Thierry Reding 2017-11-13 1061 c4755fb9064f64 Thierry Reding 2017-11-13 1062 hub->clk_disp = devm_clk_get(&pdev->dev, "disp"); c4755fb9064f64 Thierry Reding 2017-11-13 1063 if (IS_ERR(hub->clk_disp)) { c4755fb9064f64 Thierry Reding 2017-11-13 1064 err = PTR_ERR(hub->clk_disp); c4755fb9064f64 Thierry Reding 2017-11-13 1065 return err; c4755fb9064f64 Thierry Reding 2017-11-13 1066 } c4755fb9064f64 Thierry Reding 2017-11-13 1067 5725daaab55ca0 Thierry Reding 2018-09-21 1068 if (hub->soc->supports_dsc) { c4755fb9064f64 Thierry Reding 2017-11-13 1069 hub->clk_dsc = devm_clk_get(&pdev->dev, "dsc"); c4755fb9064f64 Thierry Reding 2017-11-13 1070 if (IS_ERR(hub->clk_dsc)) { c4755fb9064f64 Thierry Reding 2017-11-13 1071 err = PTR_ERR(hub->clk_dsc); c4755fb9064f64 Thierry Reding 2017-11-13 1072 return err; c4755fb9064f64 Thierry Reding 2017-11-13 1073 } 5725daaab55ca0 Thierry Reding 2018-09-21 1074 } c4755fb9064f64 Thierry Reding 2017-11-13 1075 c4755fb9064f64 Thierry Reding 2017-11-13 1076 hub->clk_hub = devm_clk_get(&pdev->dev, "hub"); c4755fb9064f64 Thierry Reding 2017-11-13 1077 if (IS_ERR(hub->clk_hub)) { c4755fb9064f64 Thierry Reding 2017-11-13 1078 err = PTR_ERR(hub->clk_hub); c4755fb9064f64 Thierry Reding 2017-11-13 1079 return err; c4755fb9064f64 Thierry Reding 2017-11-13 1080 } c4755fb9064f64 Thierry Reding 2017-11-13 1081 c4755fb9064f64 Thierry Reding 2017-11-13 1082 hub->rst = devm_reset_control_get(&pdev->dev, "misc"); c4755fb9064f64 Thierry Reding 2017-11-13 1083 if (IS_ERR(hub->rst)) { c4755fb9064f64 Thierry Reding 2017-11-13 1084 err = PTR_ERR(hub->rst); c4755fb9064f64 Thierry Reding 2017-11-13 1085 return err; c4755fb9064f64 Thierry Reding 2017-11-13 1086 } c4755fb9064f64 Thierry Reding 2017-11-13 1087 c4755fb9064f64 Thierry Reding 2017-11-13 1088 hub->wgrps = devm_kcalloc(&pdev->dev, hub->soc->num_wgrps, c4755fb9064f64 Thierry Reding 2017-11-13 1089 sizeof(*hub->wgrps), GFP_KERNEL); c4755fb9064f64 Thierry Reding 2017-11-13 1090 if (!hub->wgrps) c4755fb9064f64 Thierry Reding 2017-11-13 1091 return -ENOMEM; c4755fb9064f64 Thierry Reding 2017-11-13 1092 c4755fb9064f64 Thierry Reding 2017-11-13 1093 for (i = 0; i < hub->soc->num_wgrps; i++) { c4755fb9064f64 Thierry Reding 2017-11-13 1094 struct tegra_windowgroup *wgrp = &hub->wgrps[i]; c4755fb9064f64 Thierry Reding 2017-11-13 1095 char id[8]; c4755fb9064f64 Thierry Reding 2017-11-13 1096 c4755fb9064f64 Thierry Reding 2017-11-13 1097 snprintf(id, sizeof(id), "wgrp%u", i); c4755fb9064f64 Thierry Reding 2017-11-13 1098 mutex_init(&wgrp->lock); c4755fb9064f64 Thierry Reding 2017-11-13 1099 wgrp->usecount = 0; c4755fb9064f64 Thierry Reding 2017-11-13 1100 wgrp->index = i; c4755fb9064f64 Thierry Reding 2017-11-13 1101 c4755fb9064f64 Thierry Reding 2017-11-13 1102 wgrp->rst = devm_reset_control_get(&pdev->dev, id); c4755fb9064f64 Thierry Reding 2017-11-13 1103 if (IS_ERR(wgrp->rst)) c4755fb9064f64 Thierry Reding 2017-11-13 1104 return PTR_ERR(wgrp->rst); c4755fb9064f64 Thierry Reding 2017-11-13 1105 c4755fb9064f64 Thierry Reding 2017-11-13 1106 err = reset_control_assert(wgrp->rst); c4755fb9064f64 Thierry Reding 2017-11-13 1107 if (err < 0) c4755fb9064f64 Thierry Reding 2017-11-13 1108 return err; c4755fb9064f64 Thierry Reding 2017-11-13 1109 } c4755fb9064f64 Thierry Reding 2017-11-13 1110 0cffbde2e318cc Thierry Reding 2018-11-29 1111 hub->num_heads = of_get_child_count(pdev->dev.of_node); 0cffbde2e318cc Thierry Reding 2018-11-29 1112 0cffbde2e318cc Thierry Reding 2018-11-29 1113 hub->clk_heads = devm_kcalloc(&pdev->dev, hub->num_heads, sizeof(clk), 0cffbde2e318cc Thierry Reding 2018-11-29 1114 GFP_KERNEL); 0cffbde2e318cc Thierry Reding 2018-11-29 1115 if (!hub->clk_heads) 0cffbde2e318cc Thierry Reding 2018-11-29 1116 return -ENOMEM; 0cffbde2e318cc Thierry Reding 2018-11-29 1117 0cffbde2e318cc Thierry Reding 2018-11-29 1118 for (i = 0; i < hub->num_heads; i++) { 0cffbde2e318cc Thierry Reding 2018-11-29 1119 child = of_get_next_child(pdev->dev.of_node, child); 0cffbde2e318cc Thierry Reding 2018-11-29 1120 if (!child) { 0cffbde2e318cc Thierry Reding 2018-11-29 1121 dev_err(&pdev->dev, "failed to find node for head %u\n", 0cffbde2e318cc Thierry Reding 2018-11-29 1122 i); 0cffbde2e318cc Thierry Reding 2018-11-29 1123 return -ENODEV; 0cffbde2e318cc Thierry Reding 2018-11-29 1124 } 0cffbde2e318cc Thierry Reding 2018-11-29 1125 0cffbde2e318cc Thierry Reding 2018-11-29 1126 clk = devm_get_clk_from_child(&pdev->dev, child, "dc"); 0cffbde2e318cc Thierry Reding 2018-11-29 1127 if (IS_ERR(clk)) { 0cffbde2e318cc Thierry Reding 2018-11-29 1128 dev_err(&pdev->dev, "failed to get clock for head %u\n", 0cffbde2e318cc Thierry Reding 2018-11-29 1129 i); 0cffbde2e318cc Thierry Reding 2018-11-29 1130 of_node_put(child); 0cffbde2e318cc Thierry Reding 2018-11-29 1131 return PTR_ERR(clk); 0cffbde2e318cc Thierry Reding 2018-11-29 1132 } 0cffbde2e318cc Thierry Reding 2018-11-29 1133 0cffbde2e318cc Thierry Reding 2018-11-29 1134 hub->clk_heads[i] = clk; 0cffbde2e318cc Thierry Reding 2018-11-29 1135 } 0cffbde2e318cc Thierry Reding 2018-11-29 1136 0cffbde2e318cc Thierry Reding 2018-11-29 1137 of_node_put(child); 0cffbde2e318cc Thierry Reding 2018-11-29 1138 c4755fb9064f64 Thierry Reding 2017-11-13 1139 /* XXX: enable clock across reset? */ c4755fb9064f64 Thierry Reding 2017-11-13 1140 err = reset_control_assert(hub->rst); c4755fb9064f64 Thierry Reding 2017-11-13 1141 if (err < 0) c4755fb9064f64 Thierry Reding 2017-11-13 1142 return err; c4755fb9064f64 Thierry Reding 2017-11-13 1143 c4755fb9064f64 Thierry Reding 2017-11-13 1144 platform_set_drvdata(pdev, hub); c4755fb9064f64 Thierry Reding 2017-11-13 1145 pm_runtime_enable(&pdev->dev); c4755fb9064f64 Thierry Reding 2017-11-13 1146 c4755fb9064f64 Thierry Reding 2017-11-13 1147 INIT_LIST_HEAD(&hub->client.list); c4755fb9064f64 Thierry Reding 2017-11-13 1148 hub->client.ops = &tegra_display_hub_ops; c4755fb9064f64 Thierry Reding 2017-11-13 1149 hub->client.dev = &pdev->dev; c4755fb9064f64 Thierry Reding 2017-11-13 1150 c4755fb9064f64 Thierry Reding 2017-11-13 1151 err = host1x_client_register(&hub->client); c4755fb9064f64 Thierry Reding 2017-11-13 1152 if (err < 0) c4755fb9064f64 Thierry Reding 2017-11-13 1153 dev_err(&pdev->dev, "failed to register host1x client: %d\n", c4755fb9064f64 Thierry Reding 2017-11-13 1154 err); c4755fb9064f64 Thierry Reding 2017-11-13 1155 a101e3dad8a90a Thierry Reding 2020-06-12 1156 err = devm_of_platform_populate(&pdev->dev); a101e3dad8a90a Thierry Reding 2020-06-12 1157 if (err < 0) a101e3dad8a90a Thierry Reding 2020-06-12 1158 goto unregister; a101e3dad8a90a Thierry Reding 2020-06-12 1159 a101e3dad8a90a Thierry Reding 2020-06-12 1160 return err; a101e3dad8a90a Thierry Reding 2020-06-12 1161 a101e3dad8a90a Thierry Reding 2020-06-12 1162 unregister: a101e3dad8a90a Thierry Reding 2020-06-12 1163 host1x_client_unregister(&hub->client); a101e3dad8a90a Thierry Reding 2020-06-12 1164 pm_runtime_disable(&pdev->dev); c4755fb9064f64 Thierry Reding 2017-11-13 1165 return err; c4755fb9064f64 Thierry Reding 2017-11-13 1166 } c4755fb9064f64 Thierry Reding 2017-11-13 1167 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org _______________________________________________ iommu mailing list iommu@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/iommu From mboxrd@z Thu Jan 1 00:00:00 1970 Content-Type: multipart/mixed; boundary="===============4110714428119910004==" MIME-Version: 1.0 From: kernel test robot To: kbuild-all@lists.01.org Subject: Re: [PATCH v2 11/11] iommu: Move flush queue data into iommu_dma_cookie Date: Sat, 11 Dec 2021 07:30:01 +0800 Message-ID: <202112110753.vYbSlMnq-lkp@intel.com> In-Reply-To: List-Id: --===============4110714428119910004== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Hi Robin, I love your patch! Yet something to improve: [auto build test ERROR on joro-iommu/next] [also build test ERROR on tegra/for-next v5.16-rc4] [cannot apply to tegra-drm/drm/tegra/for-next] [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] url: https://github.com/0day-ci/linux/commits/Robin-Murphy/iommu-refacto= r-flush-queues-into-iommu-dma/20211211-015635 base: https://git.kernel.org/pub/scm/linux/kernel/git/joro/iommu.git next config: arm-randconfig-r013-20211210 (https://download.01.org/0day-ci/archi= ve/20211211/202112110753.vYbSlMnq-lkp(a)intel.com/config) compiler: arm-linux-gnueabi-gcc (GCC) 11.2.0 reproduce (this is a W=3D1 build): wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/= make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # https://github.com/0day-ci/linux/commit/3b6adb4a8ec42d7b5c1b3b1af= 2c857a2375fd7e1 git remote add linux-review https://github.com/0day-ci/linux git fetch --no-tags linux-review Robin-Murphy/iommu-refactor-flush-= queues-into-iommu-dma/20211211-015635 git checkout 3b6adb4a8ec42d7b5c1b3b1af2c857a2375fd7e1 # save the config file to linux build tree mkdir build_dir COMPILER_INSTALL_PATH=3D$HOME/0day COMPILER=3Dgcc-11.2.0 make.cross= O=3Dbuild_dir ARCH=3Darm SHELL=3D/bin/bash drivers/gpu/drm/tegra/ drivers/= iommu/ If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot All errors (new ones prefixed by >>): drivers/gpu/drm/tegra/hub.c: In function 'tegra_display_hub_probe': >> drivers/gpu/drm/tegra/hub.c:1043:24: error: implicit declaration of func= tion 'dma_get_mask'; did you mean 'xa_get_mark'? [-Werror=3Dimplicit-functi= on-declaration] 1043 | u64 dma_mask =3D dma_get_mask(pdev->dev.parent); | ^~~~~~~~~~~~ | xa_get_mark >> drivers/gpu/drm/tegra/hub.c:1050:15: error: implicit declaration of func= tion 'dma_coerce_mask_and_coherent' [-Werror=3Dimplicit-function-declaratio= n] 1050 | err =3D dma_coerce_mask_and_coherent(&pdev->dev, dma_mas= k); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~ cc1: some warnings being treated as errors -- drivers/gpu/drm/tegra/plane.c: In function 'tegra_plane_reset': >> drivers/gpu/drm/tegra/plane.c:46:42: error: 'DMA_MAPPING_ERROR' undeclar= ed (first use in this function) 46 | state->iova[i] =3D DMA_MAPPING_ERROR; | ^~~~~~~~~~~~~~~~~ drivers/gpu/drm/tegra/plane.c:46:42: note: each undeclared identifier is= reported only once for each function it appears in drivers/gpu/drm/tegra/plane.c: In function 'tegra_plane_atomic_duplicate= _state': drivers/gpu/drm/tegra/plane.c:76:33: error: 'DMA_MAPPING_ERROR' undeclar= ed (first use in this function) 76 | copy->iova[i] =3D DMA_MAPPING_ERROR; | ^~~~~~~~~~~~~~~~~ drivers/gpu/drm/tegra/plane.c: In function 'tegra_dc_pin': >> drivers/gpu/drm/tegra/plane.c:170:31: error: implicit declaration of fun= ction 'dma_map_sgtable'; did you mean 'iommu_map_sgtable'? [-Werror=3Dimpli= cit-function-declaration] 170 | err =3D dma_map_sgtable(dc->dev, sgt, DM= A_TO_DEVICE, 0); | ^~~~~~~~~~~~~~~ | iommu_map_sgtable >> drivers/gpu/drm/tegra/plane.c:170:61: error: 'DMA_TO_DEVICE' undeclared = (first use in this function); did you mean 'MT_DEVICE'? 170 | err =3D dma_map_sgtable(dc->dev, sgt, DM= A_TO_DEVICE, 0); | ^~~~= ~~~~~~~~~ | MT_D= EVICE >> drivers/gpu/drm/tegra/plane.c:202:25: error: implicit declaration of fun= ction 'dma_unmap_sgtable'; did you mean 'iommu_map_sgtable'? [-Werror=3Dimp= licit-function-declaration] 202 | dma_unmap_sgtable(dc->dev, sgt, DMA_TO_D= EVICE, 0); | ^~~~~~~~~~~~~~~~~ | iommu_map_sgtable drivers/gpu/drm/tegra/plane.c:205:34: error: 'DMA_MAPPING_ERROR' undecla= red (first use in this function) 205 | state->iova[i] =3D DMA_MAPPING_ERROR; | ^~~~~~~~~~~~~~~~~ drivers/gpu/drm/tegra/plane.c: In function 'tegra_dc_unpin': drivers/gpu/drm/tegra/plane.c:221:57: error: 'DMA_TO_DEVICE' undeclared = (first use in this function); did you mean 'MT_DEVICE'? 221 | dma_unmap_sgtable(dc->dev, sgt, DMA_TO_D= EVICE, 0); | ^~~~~~~~= ~~~~~ | MT_DEVICE drivers/gpu/drm/tegra/plane.c:224:34: error: 'DMA_MAPPING_ERROR' undecla= red (first use in this function) 224 | state->iova[i] =3D DMA_MAPPING_ERROR; | ^~~~~~~~~~~~~~~~~ cc1: some warnings being treated as errors -- drivers/gpu/drm/tegra/dc.c: In function 'tegra_crtc_calculate_memory_ban= dwidth': drivers/gpu/drm/tegra/dc.c:2225:38: warning: variable 'old_state' set bu= t not used [-Wunused-but-set-variable] 2225 | const struct drm_crtc_state *old_state; | ^~~~~~~~~ drivers/gpu/drm/tegra/dc.c: In function 'tegra_dc_probe': >> drivers/gpu/drm/tegra/dc.c:2978:24: error: implicit declaration of funct= ion 'dma_get_mask'; did you mean 'xa_get_mark'? [-Werror=3Dimplicit-functio= n-declaration] 2978 | u64 dma_mask =3D dma_get_mask(pdev->dev.parent); | ^~~~~~~~~~~~ | xa_get_mark >> drivers/gpu/drm/tegra/dc.c:2982:15: error: implicit declaration of funct= ion 'dma_coerce_mask_and_coherent' [-Werror=3Dimplicit-function-declaration] 2982 | err =3D dma_coerce_mask_and_coherent(&pdev->dev, dma_mas= k); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~ cc1: some warnings being treated as errors vim +1043 drivers/gpu/drm/tegra/hub.c c4755fb9064f64 Thierry Reding 2017-11-13 1040 = c4755fb9064f64 Thierry Reding 2017-11-13 1041 static int tegra_display_hu= b_probe(struct platform_device *pdev) c4755fb9064f64 Thierry Reding 2017-11-13 1042 { 86044e749be77a Thierry Reding 2021-03-26 @1043 u64 dma_mask =3D dma_get_m= ask(pdev->dev.parent); 0cffbde2e318cc Thierry Reding 2018-11-29 1044 struct device_node *child = =3D NULL; c4755fb9064f64 Thierry Reding 2017-11-13 1045 struct tegra_display_hub *= hub; 0cffbde2e318cc Thierry Reding 2018-11-29 1046 struct clk *clk; c4755fb9064f64 Thierry Reding 2017-11-13 1047 unsigned int i; c4755fb9064f64 Thierry Reding 2017-11-13 1048 int err; c4755fb9064f64 Thierry Reding 2017-11-13 1049 = 86044e749be77a Thierry Reding 2021-03-26 @1050 err =3D dma_coerce_mask_an= d_coherent(&pdev->dev, dma_mask); 86044e749be77a Thierry Reding 2021-03-26 1051 if (err < 0) { 86044e749be77a Thierry Reding 2021-03-26 1052 dev_err(&pdev->dev, "fail= ed to set DMA mask: %d\n", err); 86044e749be77a Thierry Reding 2021-03-26 1053 return err; 86044e749be77a Thierry Reding 2021-03-26 1054 } 86044e749be77a Thierry Reding 2021-03-26 1055 = c4755fb9064f64 Thierry Reding 2017-11-13 1056 hub =3D devm_kzalloc(&pdev= ->dev, sizeof(*hub), GFP_KERNEL); c4755fb9064f64 Thierry Reding 2017-11-13 1057 if (!hub) c4755fb9064f64 Thierry Reding 2017-11-13 1058 return -ENOMEM; c4755fb9064f64 Thierry Reding 2017-11-13 1059 = c4755fb9064f64 Thierry Reding 2017-11-13 1060 hub->soc =3D of_device_get= _match_data(&pdev->dev); c4755fb9064f64 Thierry Reding 2017-11-13 1061 = c4755fb9064f64 Thierry Reding 2017-11-13 1062 hub->clk_disp =3D devm_clk= _get(&pdev->dev, "disp"); c4755fb9064f64 Thierry Reding 2017-11-13 1063 if (IS_ERR(hub->clk_disp))= { c4755fb9064f64 Thierry Reding 2017-11-13 1064 err =3D PTR_ERR(hub->clk_= disp); c4755fb9064f64 Thierry Reding 2017-11-13 1065 return err; c4755fb9064f64 Thierry Reding 2017-11-13 1066 } c4755fb9064f64 Thierry Reding 2017-11-13 1067 = 5725daaab55ca0 Thierry Reding 2018-09-21 1068 if (hub->soc->supports_dsc= ) { c4755fb9064f64 Thierry Reding 2017-11-13 1069 hub->clk_dsc =3D devm_clk= _get(&pdev->dev, "dsc"); c4755fb9064f64 Thierry Reding 2017-11-13 1070 if (IS_ERR(hub->clk_dsc))= { c4755fb9064f64 Thierry Reding 2017-11-13 1071 err =3D PTR_ERR(hub->clk= _dsc); c4755fb9064f64 Thierry Reding 2017-11-13 1072 return err; c4755fb9064f64 Thierry Reding 2017-11-13 1073 } 5725daaab55ca0 Thierry Reding 2018-09-21 1074 } c4755fb9064f64 Thierry Reding 2017-11-13 1075 = c4755fb9064f64 Thierry Reding 2017-11-13 1076 hub->clk_hub =3D devm_clk_= get(&pdev->dev, "hub"); c4755fb9064f64 Thierry Reding 2017-11-13 1077 if (IS_ERR(hub->clk_hub)) { c4755fb9064f64 Thierry Reding 2017-11-13 1078 err =3D PTR_ERR(hub->clk_= hub); c4755fb9064f64 Thierry Reding 2017-11-13 1079 return err; c4755fb9064f64 Thierry Reding 2017-11-13 1080 } c4755fb9064f64 Thierry Reding 2017-11-13 1081 = c4755fb9064f64 Thierry Reding 2017-11-13 1082 hub->rst =3D devm_reset_co= ntrol_get(&pdev->dev, "misc"); c4755fb9064f64 Thierry Reding 2017-11-13 1083 if (IS_ERR(hub->rst)) { c4755fb9064f64 Thierry Reding 2017-11-13 1084 err =3D PTR_ERR(hub->rst); c4755fb9064f64 Thierry Reding 2017-11-13 1085 return err; c4755fb9064f64 Thierry Reding 2017-11-13 1086 } c4755fb9064f64 Thierry Reding 2017-11-13 1087 = c4755fb9064f64 Thierry Reding 2017-11-13 1088 hub->wgrps =3D devm_kcallo= c(&pdev->dev, hub->soc->num_wgrps, c4755fb9064f64 Thierry Reding 2017-11-13 1089 sizeof(*hub->wgrps), = GFP_KERNEL); c4755fb9064f64 Thierry Reding 2017-11-13 1090 if (!hub->wgrps) c4755fb9064f64 Thierry Reding 2017-11-13 1091 return -ENOMEM; c4755fb9064f64 Thierry Reding 2017-11-13 1092 = c4755fb9064f64 Thierry Reding 2017-11-13 1093 for (i =3D 0; i < hub->soc= ->num_wgrps; i++) { c4755fb9064f64 Thierry Reding 2017-11-13 1094 struct tegra_windowgroup = *wgrp =3D &hub->wgrps[i]; c4755fb9064f64 Thierry Reding 2017-11-13 1095 char id[8]; c4755fb9064f64 Thierry Reding 2017-11-13 1096 = c4755fb9064f64 Thierry Reding 2017-11-13 1097 snprintf(id, sizeof(id), = "wgrp%u", i); c4755fb9064f64 Thierry Reding 2017-11-13 1098 mutex_init(&wgrp->lock); c4755fb9064f64 Thierry Reding 2017-11-13 1099 wgrp->usecount =3D 0; c4755fb9064f64 Thierry Reding 2017-11-13 1100 wgrp->index =3D i; c4755fb9064f64 Thierry Reding 2017-11-13 1101 = c4755fb9064f64 Thierry Reding 2017-11-13 1102 wgrp->rst =3D devm_reset_= control_get(&pdev->dev, id); c4755fb9064f64 Thierry Reding 2017-11-13 1103 if (IS_ERR(wgrp->rst)) c4755fb9064f64 Thierry Reding 2017-11-13 1104 return PTR_ERR(wgrp->rst= ); c4755fb9064f64 Thierry Reding 2017-11-13 1105 = c4755fb9064f64 Thierry Reding 2017-11-13 1106 err =3D reset_control_ass= ert(wgrp->rst); c4755fb9064f64 Thierry Reding 2017-11-13 1107 if (err < 0) c4755fb9064f64 Thierry Reding 2017-11-13 1108 return err; c4755fb9064f64 Thierry Reding 2017-11-13 1109 } c4755fb9064f64 Thierry Reding 2017-11-13 1110 = 0cffbde2e318cc Thierry Reding 2018-11-29 1111 hub->num_heads =3D of_get_= child_count(pdev->dev.of_node); 0cffbde2e318cc Thierry Reding 2018-11-29 1112 = 0cffbde2e318cc Thierry Reding 2018-11-29 1113 hub->clk_heads =3D devm_kc= alloc(&pdev->dev, hub->num_heads, sizeof(clk), 0cffbde2e318cc Thierry Reding 2018-11-29 1114 GFP_KERNEL); 0cffbde2e318cc Thierry Reding 2018-11-29 1115 if (!hub->clk_heads) 0cffbde2e318cc Thierry Reding 2018-11-29 1116 return -ENOMEM; 0cffbde2e318cc Thierry Reding 2018-11-29 1117 = 0cffbde2e318cc Thierry Reding 2018-11-29 1118 for (i =3D 0; i < hub->num= _heads; i++) { 0cffbde2e318cc Thierry Reding 2018-11-29 1119 child =3D of_get_next_chi= ld(pdev->dev.of_node, child); 0cffbde2e318cc Thierry Reding 2018-11-29 1120 if (!child) { 0cffbde2e318cc Thierry Reding 2018-11-29 1121 dev_err(&pdev->dev, "fai= led to find node for head %u\n", 0cffbde2e318cc Thierry Reding 2018-11-29 1122 i); 0cffbde2e318cc Thierry Reding 2018-11-29 1123 return -ENODEV; 0cffbde2e318cc Thierry Reding 2018-11-29 1124 } 0cffbde2e318cc Thierry Reding 2018-11-29 1125 = 0cffbde2e318cc Thierry Reding 2018-11-29 1126 clk =3D devm_get_clk_from= _child(&pdev->dev, child, "dc"); 0cffbde2e318cc Thierry Reding 2018-11-29 1127 if (IS_ERR(clk)) { 0cffbde2e318cc Thierry Reding 2018-11-29 1128 dev_err(&pdev->dev, "fai= led to get clock for head %u\n", 0cffbde2e318cc Thierry Reding 2018-11-29 1129 i); 0cffbde2e318cc Thierry Reding 2018-11-29 1130 of_node_put(child); 0cffbde2e318cc Thierry Reding 2018-11-29 1131 return PTR_ERR(clk); 0cffbde2e318cc Thierry Reding 2018-11-29 1132 } 0cffbde2e318cc Thierry Reding 2018-11-29 1133 = 0cffbde2e318cc Thierry Reding 2018-11-29 1134 hub->clk_heads[i] =3D clk; 0cffbde2e318cc Thierry Reding 2018-11-29 1135 } 0cffbde2e318cc Thierry Reding 2018-11-29 1136 = 0cffbde2e318cc Thierry Reding 2018-11-29 1137 of_node_put(child); 0cffbde2e318cc Thierry Reding 2018-11-29 1138 = c4755fb9064f64 Thierry Reding 2017-11-13 1139 /* XXX: enable clock acros= s reset? */ c4755fb9064f64 Thierry Reding 2017-11-13 1140 err =3D reset_control_asse= rt(hub->rst); c4755fb9064f64 Thierry Reding 2017-11-13 1141 if (err < 0) c4755fb9064f64 Thierry Reding 2017-11-13 1142 return err; c4755fb9064f64 Thierry Reding 2017-11-13 1143 = c4755fb9064f64 Thierry Reding 2017-11-13 1144 platform_set_drvdata(pdev,= hub); c4755fb9064f64 Thierry Reding 2017-11-13 1145 pm_runtime_enable(&pdev->d= ev); c4755fb9064f64 Thierry Reding 2017-11-13 1146 = c4755fb9064f64 Thierry Reding 2017-11-13 1147 INIT_LIST_HEAD(&hub->clien= t.list); c4755fb9064f64 Thierry Reding 2017-11-13 1148 hub->client.ops =3D &tegra= _display_hub_ops; c4755fb9064f64 Thierry Reding 2017-11-13 1149 hub->client.dev =3D &pdev-= >dev; c4755fb9064f64 Thierry Reding 2017-11-13 1150 = c4755fb9064f64 Thierry Reding 2017-11-13 1151 err =3D host1x_client_regi= ster(&hub->client); c4755fb9064f64 Thierry Reding 2017-11-13 1152 if (err < 0) c4755fb9064f64 Thierry Reding 2017-11-13 1153 dev_err(&pdev->dev, "fail= ed to register host1x client: %d\n", c4755fb9064f64 Thierry Reding 2017-11-13 1154 err); c4755fb9064f64 Thierry Reding 2017-11-13 1155 = a101e3dad8a90a Thierry Reding 2020-06-12 1156 err =3D devm_of_platform_p= opulate(&pdev->dev); a101e3dad8a90a Thierry Reding 2020-06-12 1157 if (err < 0) a101e3dad8a90a Thierry Reding 2020-06-12 1158 goto unregister; a101e3dad8a90a Thierry Reding 2020-06-12 1159 = a101e3dad8a90a Thierry Reding 2020-06-12 1160 return err; a101e3dad8a90a Thierry Reding 2020-06-12 1161 = a101e3dad8a90a Thierry Reding 2020-06-12 1162 unregister: a101e3dad8a90a Thierry Reding 2020-06-12 1163 host1x_client_unregister(&= hub->client); a101e3dad8a90a Thierry Reding 2020-06-12 1164 pm_runtime_disable(&pdev->= dev); c4755fb9064f64 Thierry Reding 2017-11-13 1165 return err; c4755fb9064f64 Thierry Reding 2017-11-13 1166 } c4755fb9064f64 Thierry Reding 2017-11-13 1167 = --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org --===============4110714428119910004==-- From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 00C4BC433F5 for ; Fri, 10 Dec 2021 23:30:48 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 3D7A96B0072; Fri, 10 Dec 2021 18:30:38 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 386C86B0073; Fri, 10 Dec 2021 18:30:38 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 2006C6B0074; Fri, 10 Dec 2021 18:30:38 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0150.hostedemail.com [216.40.44.150]) by kanga.kvack.org (Postfix) with ESMTP id 091A66B0072 for ; Fri, 10 Dec 2021 18:30:38 -0500 (EST) Received: from smtpin06.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay04.hostedemail.com (Postfix) with ESMTP id BC9ED88485 for ; Fri, 10 Dec 2021 23:30:27 +0000 (UTC) X-FDA: 78903480894.06.5EECF02 Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by imf23.hostedemail.com (Postfix) with ESMTP id 114BF140008 for ; Fri, 10 Dec 2021 23:30:24 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1639179026; x=1670715026; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=m2GhuwJ4UnyKyqy83lVmEBmiMbGKiHLXbCCHqnR/ajM=; b=Lzg4HB+yXdbnhHvGmgzE/nfIaCijjppyQBRkZh6CHJoSKZduwFKcIYjx xMgepYpy7xG//P7ZOG2c76TZO1ObO/99OsGcLEADFz5vTY3INiKpyvOmD BoUin3MNEDTiiXqdl2JRlJo+FWR3GD7E71pPM+v9poyno4N7koIInjN9l AJYnmVkfndO7kOfzGP91ceo/BC0A3N/tqM0cx5k9YZxbaKVg4Ut9t95BO RMbd7zhMKNnVj3skNIKmzJTxqlbYTkKTKowlebMZYV93keB4ydCgtwkpU 7fcBQk/q7ZbVrwL+jeESawIq8ll0KfyQ9BjpYDhU9OrpbganzgA7PS5QE w==; X-IronPort-AV: E=McAfee;i="6200,9189,10194"; a="225752894" X-IronPort-AV: E=Sophos;i="5.88,196,1635231600"; d="scan'208";a="225752894" Received: from fmsmga008.fm.intel.com ([10.253.24.58]) by orsmga101.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 10 Dec 2021 15:30:25 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.88,196,1635231600"; d="scan'208";a="565479477" Received: from lkp-server02.sh.intel.com (HELO 9e1e9f9b3bcb) ([10.239.97.151]) by fmsmga008.fm.intel.com with ESMTP; 10 Dec 2021 15:30:21 -0800 Received: from kbuild by 9e1e9f9b3bcb with local (Exim 4.92) (envelope-from ) id 1mvpLI-0003nt-Si; Fri, 10 Dec 2021 23:30:20 +0000 Date: Sat, 11 Dec 2021 07:30:01 +0800 From: kernel test robot To: Robin Murphy , joro@8bytes.org, will@kernel.org Cc: kbuild-all@lists.01.org, iommu@lists.linux-foundation.org, suravee.suthikulpanit@amd.com, baolu.lu@linux.intel.com, willy@infradead.org, linux-kernel@vger.kernel.org, john.garry@huawei.com, linux-mm@kvack.org Subject: Re: [PATCH v2 11/11] iommu: Move flush queue data into iommu_dma_cookie Message-ID: <202112110753.vYbSlMnq-lkp@intel.com> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.10.1 (2018-07-13) X-Rspamd-Server: rspam12 X-Rspamd-Queue-Id: 114BF140008 X-Stat-Signature: gs9e9usapdtorw9fm3wy5d8ag88hftcc Authentication-Results: imf23.hostedemail.com; dkim=pass header.d=intel.com header.s=Intel header.b=Lzg4HB+y; spf=none (imf23.hostedemail.com: domain of lkp@intel.com has no SPF policy when checking 134.134.136.20) smtp.mailfrom=lkp@intel.com; dmarc=pass (policy=none) header.from=intel.com X-HE-Tag: 1639179024-189276 X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: Hi Robin, I love your patch! Yet something to improve: [auto build test ERROR on joro-iommu/next] [also build test ERROR on tegra/for-next v5.16-rc4] [cannot apply to tegra-drm/drm/tegra/for-next] [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] url: https://github.com/0day-ci/linux/commits/Robin-Murphy/iommu-refactor-flush-queues-into-iommu-dma/20211211-015635 base: https://git.kernel.org/pub/scm/linux/kernel/git/joro/iommu.git next config: arm-randconfig-r013-20211210 (https://download.01.org/0day-ci/archive/20211211/202112110753.vYbSlMnq-lkp@intel.com/config) compiler: arm-linux-gnueabi-gcc (GCC) 11.2.0 reproduce (this is a W=1 build): wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # https://github.com/0day-ci/linux/commit/3b6adb4a8ec42d7b5c1b3b1af2c857a2375fd7e1 git remote add linux-review https://github.com/0day-ci/linux git fetch --no-tags linux-review Robin-Murphy/iommu-refactor-flush-queues-into-iommu-dma/20211211-015635 git checkout 3b6adb4a8ec42d7b5c1b3b1af2c857a2375fd7e1 # save the config file to linux build tree mkdir build_dir COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross O=build_dir ARCH=arm SHELL=/bin/bash drivers/gpu/drm/tegra/ drivers/iommu/ If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot All errors (new ones prefixed by >>): drivers/gpu/drm/tegra/hub.c: In function 'tegra_display_hub_probe': >> drivers/gpu/drm/tegra/hub.c:1043:24: error: implicit declaration of function 'dma_get_mask'; did you mean 'xa_get_mark'? [-Werror=implicit-function-declaration] 1043 | u64 dma_mask = dma_get_mask(pdev->dev.parent); | ^~~~~~~~~~~~ | xa_get_mark >> drivers/gpu/drm/tegra/hub.c:1050:15: error: implicit declaration of function 'dma_coerce_mask_and_coherent' [-Werror=implicit-function-declaration] 1050 | err = dma_coerce_mask_and_coherent(&pdev->dev, dma_mask); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~ cc1: some warnings being treated as errors -- drivers/gpu/drm/tegra/plane.c: In function 'tegra_plane_reset': >> drivers/gpu/drm/tegra/plane.c:46:42: error: 'DMA_MAPPING_ERROR' undeclared (first use in this function) 46 | state->iova[i] = DMA_MAPPING_ERROR; | ^~~~~~~~~~~~~~~~~ drivers/gpu/drm/tegra/plane.c:46:42: note: each undeclared identifier is reported only once for each function it appears in drivers/gpu/drm/tegra/plane.c: In function 'tegra_plane_atomic_duplicate_state': drivers/gpu/drm/tegra/plane.c:76:33: error: 'DMA_MAPPING_ERROR' undeclared (first use in this function) 76 | copy->iova[i] = DMA_MAPPING_ERROR; | ^~~~~~~~~~~~~~~~~ drivers/gpu/drm/tegra/plane.c: In function 'tegra_dc_pin': >> drivers/gpu/drm/tegra/plane.c:170:31: error: implicit declaration of function 'dma_map_sgtable'; did you mean 'iommu_map_sgtable'? [-Werror=implicit-function-declaration] 170 | err = dma_map_sgtable(dc->dev, sgt, DMA_TO_DEVICE, 0); | ^~~~~~~~~~~~~~~ | iommu_map_sgtable >> drivers/gpu/drm/tegra/plane.c:170:61: error: 'DMA_TO_DEVICE' undeclared (first use in this function); did you mean 'MT_DEVICE'? 170 | err = dma_map_sgtable(dc->dev, sgt, DMA_TO_DEVICE, 0); | ^~~~~~~~~~~~~ | MT_DEVICE >> drivers/gpu/drm/tegra/plane.c:202:25: error: implicit declaration of function 'dma_unmap_sgtable'; did you mean 'iommu_map_sgtable'? [-Werror=implicit-function-declaration] 202 | dma_unmap_sgtable(dc->dev, sgt, DMA_TO_DEVICE, 0); | ^~~~~~~~~~~~~~~~~ | iommu_map_sgtable drivers/gpu/drm/tegra/plane.c:205:34: error: 'DMA_MAPPING_ERROR' undeclared (first use in this function) 205 | state->iova[i] = DMA_MAPPING_ERROR; | ^~~~~~~~~~~~~~~~~ drivers/gpu/drm/tegra/plane.c: In function 'tegra_dc_unpin': drivers/gpu/drm/tegra/plane.c:221:57: error: 'DMA_TO_DEVICE' undeclared (first use in this function); did you mean 'MT_DEVICE'? 221 | dma_unmap_sgtable(dc->dev, sgt, DMA_TO_DEVICE, 0); | ^~~~~~~~~~~~~ | MT_DEVICE drivers/gpu/drm/tegra/plane.c:224:34: error: 'DMA_MAPPING_ERROR' undeclared (first use in this function) 224 | state->iova[i] = DMA_MAPPING_ERROR; | ^~~~~~~~~~~~~~~~~ cc1: some warnings being treated as errors -- drivers/gpu/drm/tegra/dc.c: In function 'tegra_crtc_calculate_memory_bandwidth': drivers/gpu/drm/tegra/dc.c:2225:38: warning: variable 'old_state' set but not used [-Wunused-but-set-variable] 2225 | const struct drm_crtc_state *old_state; | ^~~~~~~~~ drivers/gpu/drm/tegra/dc.c: In function 'tegra_dc_probe': >> drivers/gpu/drm/tegra/dc.c:2978:24: error: implicit declaration of function 'dma_get_mask'; did you mean 'xa_get_mark'? [-Werror=implicit-function-declaration] 2978 | u64 dma_mask = dma_get_mask(pdev->dev.parent); | ^~~~~~~~~~~~ | xa_get_mark >> drivers/gpu/drm/tegra/dc.c:2982:15: error: implicit declaration of function 'dma_coerce_mask_and_coherent' [-Werror=implicit-function-declaration] 2982 | err = dma_coerce_mask_and_coherent(&pdev->dev, dma_mask); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~ cc1: some warnings being treated as errors vim +1043 drivers/gpu/drm/tegra/hub.c c4755fb9064f64 Thierry Reding 2017-11-13 1040 c4755fb9064f64 Thierry Reding 2017-11-13 1041 static int tegra_display_hub_probe(struct platform_device *pdev) c4755fb9064f64 Thierry Reding 2017-11-13 1042 { 86044e749be77a Thierry Reding 2021-03-26 @1043 u64 dma_mask = dma_get_mask(pdev->dev.parent); 0cffbde2e318cc Thierry Reding 2018-11-29 1044 struct device_node *child = NULL; c4755fb9064f64 Thierry Reding 2017-11-13 1045 struct tegra_display_hub *hub; 0cffbde2e318cc Thierry Reding 2018-11-29 1046 struct clk *clk; c4755fb9064f64 Thierry Reding 2017-11-13 1047 unsigned int i; c4755fb9064f64 Thierry Reding 2017-11-13 1048 int err; c4755fb9064f64 Thierry Reding 2017-11-13 1049 86044e749be77a Thierry Reding 2021-03-26 @1050 err = dma_coerce_mask_and_coherent(&pdev->dev, dma_mask); 86044e749be77a Thierry Reding 2021-03-26 1051 if (err < 0) { 86044e749be77a Thierry Reding 2021-03-26 1052 dev_err(&pdev->dev, "failed to set DMA mask: %d\n", err); 86044e749be77a Thierry Reding 2021-03-26 1053 return err; 86044e749be77a Thierry Reding 2021-03-26 1054 } 86044e749be77a Thierry Reding 2021-03-26 1055 c4755fb9064f64 Thierry Reding 2017-11-13 1056 hub = devm_kzalloc(&pdev->dev, sizeof(*hub), GFP_KERNEL); c4755fb9064f64 Thierry Reding 2017-11-13 1057 if (!hub) c4755fb9064f64 Thierry Reding 2017-11-13 1058 return -ENOMEM; c4755fb9064f64 Thierry Reding 2017-11-13 1059 c4755fb9064f64 Thierry Reding 2017-11-13 1060 hub->soc = of_device_get_match_data(&pdev->dev); c4755fb9064f64 Thierry Reding 2017-11-13 1061 c4755fb9064f64 Thierry Reding 2017-11-13 1062 hub->clk_disp = devm_clk_get(&pdev->dev, "disp"); c4755fb9064f64 Thierry Reding 2017-11-13 1063 if (IS_ERR(hub->clk_disp)) { c4755fb9064f64 Thierry Reding 2017-11-13 1064 err = PTR_ERR(hub->clk_disp); c4755fb9064f64 Thierry Reding 2017-11-13 1065 return err; c4755fb9064f64 Thierry Reding 2017-11-13 1066 } c4755fb9064f64 Thierry Reding 2017-11-13 1067 5725daaab55ca0 Thierry Reding 2018-09-21 1068 if (hub->soc->supports_dsc) { c4755fb9064f64 Thierry Reding 2017-11-13 1069 hub->clk_dsc = devm_clk_get(&pdev->dev, "dsc"); c4755fb9064f64 Thierry Reding 2017-11-13 1070 if (IS_ERR(hub->clk_dsc)) { c4755fb9064f64 Thierry Reding 2017-11-13 1071 err = PTR_ERR(hub->clk_dsc); c4755fb9064f64 Thierry Reding 2017-11-13 1072 return err; c4755fb9064f64 Thierry Reding 2017-11-13 1073 } 5725daaab55ca0 Thierry Reding 2018-09-21 1074 } c4755fb9064f64 Thierry Reding 2017-11-13 1075 c4755fb9064f64 Thierry Reding 2017-11-13 1076 hub->clk_hub = devm_clk_get(&pdev->dev, "hub"); c4755fb9064f64 Thierry Reding 2017-11-13 1077 if (IS_ERR(hub->clk_hub)) { c4755fb9064f64 Thierry Reding 2017-11-13 1078 err = PTR_ERR(hub->clk_hub); c4755fb9064f64 Thierry Reding 2017-11-13 1079 return err; c4755fb9064f64 Thierry Reding 2017-11-13 1080 } c4755fb9064f64 Thierry Reding 2017-11-13 1081 c4755fb9064f64 Thierry Reding 2017-11-13 1082 hub->rst = devm_reset_control_get(&pdev->dev, "misc"); c4755fb9064f64 Thierry Reding 2017-11-13 1083 if (IS_ERR(hub->rst)) { c4755fb9064f64 Thierry Reding 2017-11-13 1084 err = PTR_ERR(hub->rst); c4755fb9064f64 Thierry Reding 2017-11-13 1085 return err; c4755fb9064f64 Thierry Reding 2017-11-13 1086 } c4755fb9064f64 Thierry Reding 2017-11-13 1087 c4755fb9064f64 Thierry Reding 2017-11-13 1088 hub->wgrps = devm_kcalloc(&pdev->dev, hub->soc->num_wgrps, c4755fb9064f64 Thierry Reding 2017-11-13 1089 sizeof(*hub->wgrps), GFP_KERNEL); c4755fb9064f64 Thierry Reding 2017-11-13 1090 if (!hub->wgrps) c4755fb9064f64 Thierry Reding 2017-11-13 1091 return -ENOMEM; c4755fb9064f64 Thierry Reding 2017-11-13 1092 c4755fb9064f64 Thierry Reding 2017-11-13 1093 for (i = 0; i < hub->soc->num_wgrps; i++) { c4755fb9064f64 Thierry Reding 2017-11-13 1094 struct tegra_windowgroup *wgrp = &hub->wgrps[i]; c4755fb9064f64 Thierry Reding 2017-11-13 1095 char id[8]; c4755fb9064f64 Thierry Reding 2017-11-13 1096 c4755fb9064f64 Thierry Reding 2017-11-13 1097 snprintf(id, sizeof(id), "wgrp%u", i); c4755fb9064f64 Thierry Reding 2017-11-13 1098 mutex_init(&wgrp->lock); c4755fb9064f64 Thierry Reding 2017-11-13 1099 wgrp->usecount = 0; c4755fb9064f64 Thierry Reding 2017-11-13 1100 wgrp->index = i; c4755fb9064f64 Thierry Reding 2017-11-13 1101 c4755fb9064f64 Thierry Reding 2017-11-13 1102 wgrp->rst = devm_reset_control_get(&pdev->dev, id); c4755fb9064f64 Thierry Reding 2017-11-13 1103 if (IS_ERR(wgrp->rst)) c4755fb9064f64 Thierry Reding 2017-11-13 1104 return PTR_ERR(wgrp->rst); c4755fb9064f64 Thierry Reding 2017-11-13 1105 c4755fb9064f64 Thierry Reding 2017-11-13 1106 err = reset_control_assert(wgrp->rst); c4755fb9064f64 Thierry Reding 2017-11-13 1107 if (err < 0) c4755fb9064f64 Thierry Reding 2017-11-13 1108 return err; c4755fb9064f64 Thierry Reding 2017-11-13 1109 } c4755fb9064f64 Thierry Reding 2017-11-13 1110 0cffbde2e318cc Thierry Reding 2018-11-29 1111 hub->num_heads = of_get_child_count(pdev->dev.of_node); 0cffbde2e318cc Thierry Reding 2018-11-29 1112 0cffbde2e318cc Thierry Reding 2018-11-29 1113 hub->clk_heads = devm_kcalloc(&pdev->dev, hub->num_heads, sizeof(clk), 0cffbde2e318cc Thierry Reding 2018-11-29 1114 GFP_KERNEL); 0cffbde2e318cc Thierry Reding 2018-11-29 1115 if (!hub->clk_heads) 0cffbde2e318cc Thierry Reding 2018-11-29 1116 return -ENOMEM; 0cffbde2e318cc Thierry Reding 2018-11-29 1117 0cffbde2e318cc Thierry Reding 2018-11-29 1118 for (i = 0; i < hub->num_heads; i++) { 0cffbde2e318cc Thierry Reding 2018-11-29 1119 child = of_get_next_child(pdev->dev.of_node, child); 0cffbde2e318cc Thierry Reding 2018-11-29 1120 if (!child) { 0cffbde2e318cc Thierry Reding 2018-11-29 1121 dev_err(&pdev->dev, "failed to find node for head %u\n", 0cffbde2e318cc Thierry Reding 2018-11-29 1122 i); 0cffbde2e318cc Thierry Reding 2018-11-29 1123 return -ENODEV; 0cffbde2e318cc Thierry Reding 2018-11-29 1124 } 0cffbde2e318cc Thierry Reding 2018-11-29 1125 0cffbde2e318cc Thierry Reding 2018-11-29 1126 clk = devm_get_clk_from_child(&pdev->dev, child, "dc"); 0cffbde2e318cc Thierry Reding 2018-11-29 1127 if (IS_ERR(clk)) { 0cffbde2e318cc Thierry Reding 2018-11-29 1128 dev_err(&pdev->dev, "failed to get clock for head %u\n", 0cffbde2e318cc Thierry Reding 2018-11-29 1129 i); 0cffbde2e318cc Thierry Reding 2018-11-29 1130 of_node_put(child); 0cffbde2e318cc Thierry Reding 2018-11-29 1131 return PTR_ERR(clk); 0cffbde2e318cc Thierry Reding 2018-11-29 1132 } 0cffbde2e318cc Thierry Reding 2018-11-29 1133 0cffbde2e318cc Thierry Reding 2018-11-29 1134 hub->clk_heads[i] = clk; 0cffbde2e318cc Thierry Reding 2018-11-29 1135 } 0cffbde2e318cc Thierry Reding 2018-11-29 1136 0cffbde2e318cc Thierry Reding 2018-11-29 1137 of_node_put(child); 0cffbde2e318cc Thierry Reding 2018-11-29 1138 c4755fb9064f64 Thierry Reding 2017-11-13 1139 /* XXX: enable clock across reset? */ c4755fb9064f64 Thierry Reding 2017-11-13 1140 err = reset_control_assert(hub->rst); c4755fb9064f64 Thierry Reding 2017-11-13 1141 if (err < 0) c4755fb9064f64 Thierry Reding 2017-11-13 1142 return err; c4755fb9064f64 Thierry Reding 2017-11-13 1143 c4755fb9064f64 Thierry Reding 2017-11-13 1144 platform_set_drvdata(pdev, hub); c4755fb9064f64 Thierry Reding 2017-11-13 1145 pm_runtime_enable(&pdev->dev); c4755fb9064f64 Thierry Reding 2017-11-13 1146 c4755fb9064f64 Thierry Reding 2017-11-13 1147 INIT_LIST_HEAD(&hub->client.list); c4755fb9064f64 Thierry Reding 2017-11-13 1148 hub->client.ops = &tegra_display_hub_ops; c4755fb9064f64 Thierry Reding 2017-11-13 1149 hub->client.dev = &pdev->dev; c4755fb9064f64 Thierry Reding 2017-11-13 1150 c4755fb9064f64 Thierry Reding 2017-11-13 1151 err = host1x_client_register(&hub->client); c4755fb9064f64 Thierry Reding 2017-11-13 1152 if (err < 0) c4755fb9064f64 Thierry Reding 2017-11-13 1153 dev_err(&pdev->dev, "failed to register host1x client: %d\n", c4755fb9064f64 Thierry Reding 2017-11-13 1154 err); c4755fb9064f64 Thierry Reding 2017-11-13 1155 a101e3dad8a90a Thierry Reding 2020-06-12 1156 err = devm_of_platform_populate(&pdev->dev); a101e3dad8a90a Thierry Reding 2020-06-12 1157 if (err < 0) a101e3dad8a90a Thierry Reding 2020-06-12 1158 goto unregister; a101e3dad8a90a Thierry Reding 2020-06-12 1159 a101e3dad8a90a Thierry Reding 2020-06-12 1160 return err; a101e3dad8a90a Thierry Reding 2020-06-12 1161 a101e3dad8a90a Thierry Reding 2020-06-12 1162 unregister: a101e3dad8a90a Thierry Reding 2020-06-12 1163 host1x_client_unregister(&hub->client); a101e3dad8a90a Thierry Reding 2020-06-12 1164 pm_runtime_disable(&pdev->dev); c4755fb9064f64 Thierry Reding 2017-11-13 1165 return err; c4755fb9064f64 Thierry Reding 2017-11-13 1166 } c4755fb9064f64 Thierry Reding 2017-11-13 1167 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org