From: kernel test robot <lkp@intel.com>
To: Bartosz Golaszewski <brgl@bgdev.pl>,
Andy Gross <agross@kernel.org>,
Bjorn Andersson <andersson@kernel.org>,
Konrad Dybcio <konrad.dybcio@linaro.org>,
Elliot Berman <quic_eberman@quicinc.com>,
Krzysztof Kozlowski <krzk@kernel.org>,
Guru Das Srinagesh <quic_gurus@quicinc.com>,
Andrew Halaney <ahalaney@redhat.com>,
Maximilian Luz <luzmaximilian@gmail.com>,
Alex Elder <elder@linaro.org>,
Srini Kandagatla <srinivas.kandagatla@linaro.org>
Cc: oe-kbuild-all@lists.linux.dev, linux-arm-msm@vger.kernel.org,
linux-kernel@vger.kernel.org,
linux-arm-kernel@lists.infradead.org, kernel@quicinc.com,
Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
Subject: Re: [RESEND PATCH v5 02/12] firmware: qcom: scm: enable the TZ mem allocator
Date: Fri, 24 Nov 2023 23:23:31 +0800 [thread overview]
Message-ID: <202311241607.v3tvaLvH-lkp@intel.com> (raw)
In-Reply-To: <20231120132118.30473-3-brgl@bgdev.pl>
Hi Bartosz,
kernel test robot noticed the following build errors:
[auto build test ERROR on linus/master]
[also build test ERROR on v6.7-rc2 next-20231124]
[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/Bartosz-Golaszewski/firmware-qcom-add-a-dedicated-TrustZone-buffer-allocator/20231120-213154
base: linus/master
patch link: https://lore.kernel.org/r/20231120132118.30473-3-brgl%40bgdev.pl
patch subject: [RESEND PATCH v5 02/12] firmware: qcom: scm: enable the TZ mem allocator
config: arm-randconfig-r071-20231123 (https://download.01.org/0day-ci/archive/20231124/202311241607.v3tvaLvH-lkp@intel.com/config)
compiler: arm-linux-gnueabi-gcc (GCC) 13.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20231124/202311241607.v3tvaLvH-lkp@intel.com/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 <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202311241607.v3tvaLvH-lkp@intel.com/
All errors (new ones prefixed by >>):
drivers/firmware/qcom/qcom_tzmem.c: In function 'qcom_tzmem_pool_new':
>> drivers/firmware/qcom/qcom_tzmem.c:87:64: error: passing argument 3 of 'dma_alloc_coherent' from incompatible pointer type [-Werror=incompatible-pointer-types]
87 | pool->vbase = dma_alloc_coherent(qcom_tzmem_dev, size, &pool->pbase,
| ^~~~~~~~~~~~
| |
| phys_addr_t * {aka unsigned int *}
In file included from drivers/firmware/qcom/qcom_tzmem.c:10:
include/linux/dma-mapping.h:429:29: note: expected 'dma_addr_t *' {aka 'long long unsigned int *'} but argument is of type 'phys_addr_t *' {aka 'unsigned int *'}
429 | dma_addr_t *dma_handle, gfp_t gfp)
| ~~~~~~~~~~~~^~~~~~~~~~
cc1: some warnings being treated as errors
vim +/dma_alloc_coherent +87 drivers/firmware/qcom/qcom_tzmem.c
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 59
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 60 /**
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 61 * qcom_tzmem_pool_new() - Create a new TZ memory pool.
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 62 * @size: Size of the new pool in bytes.
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 63 *
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 64 * Create a new pool of memory suitable for sharing with the TrustZone.
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 65 *
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 66 * Must not be used in atomic context.
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 67 *
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 68 * Returns:
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 69 * New memory pool address or ERR_PTR() on error.
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 70 */
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 71 struct qcom_tzmem_pool *qcom_tzmem_pool_new(size_t size)
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 72 {
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 73 struct qcom_tzmem_pool *pool;
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 74 int ret = -ENOMEM;
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 75
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 76 if (!size)
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 77 return ERR_PTR(-EINVAL);
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 78
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 79 size = PAGE_ALIGN(size);
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 80
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 81 pool = kzalloc(sizeof(*pool), GFP_KERNEL);
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 82 if (!pool)
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 83 return ERR_PTR(-ENOMEM);
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 84
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 85 pool->size = size;
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 86
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 @87 pool->vbase = dma_alloc_coherent(qcom_tzmem_dev, size, &pool->pbase,
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 88 GFP_KERNEL);
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 89 if (!pool->vbase)
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 90 goto err_kfree_pool;
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 91
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 92 pool->pool = gen_pool_create(PAGE_SHIFT, -1);
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 93 if (!pool)
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 94 goto err_dma_free;
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 95
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 96 gen_pool_set_algo(pool->pool, gen_pool_best_fit, NULL);
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 97
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 98 ret = gen_pool_add_virt(pool->pool, (unsigned long)pool->vbase,
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 99 pool->pbase, size, -1);
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 100 if (ret)
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 101 goto err_destroy_genpool;
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 102
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 103 ret = qcom_tzmem_init_pool(pool);
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 104 if (ret)
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 105 goto err_destroy_genpool;
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 106
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 107 return pool;
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 108
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 109 err_destroy_genpool:
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 110 gen_pool_destroy(pool->pool);
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 111 err_dma_free:
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 112 dma_free_coherent(qcom_tzmem_dev, size, pool->vbase, pool->pbase);
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 113 err_kfree_pool:
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 114 kfree(pool);
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 115 return ERR_PTR(ret);
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 116 }
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 117 EXPORT_SYMBOL_GPL(qcom_tzmem_pool_new);
757fc9ecf8f244 Bartosz Golaszewski 2023-11-20 118
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
next prev parent reply other threads:[~2023-11-24 15:24 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-11-20 13:21 [RESEND PATCH v5 00/12] arm64: qcom: add and enable SHM Bridge support Bartosz Golaszewski
2023-11-20 13:21 ` [RESEND PATCH v5 01/12] firmware: qcom: add a dedicated TrustZone buffer allocator Bartosz Golaszewski
2023-11-20 13:21 ` [RESEND PATCH v5 02/12] firmware: qcom: scm: enable the TZ mem allocator Bartosz Golaszewski
2023-11-21 6:35 ` Prasad Sodagudi
2023-11-21 9:25 ` Bartosz Golaszewski
2023-11-24 15:23 ` kernel test robot [this message]
2023-11-20 13:21 ` [RESEND PATCH v5 03/12] firmware: qcom: scm: smc: switch to using the SCM allocator Bartosz Golaszewski
2023-11-21 6:38 ` Prasad Sodagudi
2023-11-21 9:47 ` Bartosz Golaszewski
2023-11-20 13:21 ` [RESEND PATCH v5 04/12] firmware: qcom: scm: make qcom_scm_assign_mem() use the TZ allocator Bartosz Golaszewski
2023-11-20 13:21 ` [RESEND PATCH v5 05/12] firmware: qcom: scm: make qcom_scm_ice_set_key() " Bartosz Golaszewski
2023-11-20 13:21 ` [RESEND PATCH v5 06/12] firmware: qcom: scm: make qcom_scm_lmh_dcvsh() " Bartosz Golaszewski
2023-11-20 13:21 ` [RESEND PATCH v5 07/12] firmware: qcom: scm: make qcom_scm_qseecom_app_get_id() " Bartosz Golaszewski
2023-11-20 13:21 ` [RESEND PATCH v5 08/12] firmware: qcom: qseecom: convert to using " Bartosz Golaszewski
2023-11-20 13:21 ` [RESEND PATCH v5 09/12] firmware: qcom: scm: add support for SHM bridge operations Bartosz Golaszewski
2023-11-20 13:21 ` [RESEND PATCH v5 10/12] firmware: qcom: tzmem: enable SHM Bridge support Bartosz Golaszewski
2023-11-20 13:21 ` [RESEND PATCH v5 11/12] firmware: qcom: scm: clarify the comment in qcom_scm_pas_init_image() Bartosz Golaszewski
2023-11-20 13:21 ` [RESEND PATCH v5 12/12] arm64: defconfig: enable SHM Bridge support for the TZ memory allocator Bartosz Golaszewski
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202311241607.v3tvaLvH-lkp@intel.com \
--to=lkp@intel.com \
--cc=agross@kernel.org \
--cc=ahalaney@redhat.com \
--cc=andersson@kernel.org \
--cc=bartosz.golaszewski@linaro.org \
--cc=brgl@bgdev.pl \
--cc=elder@linaro.org \
--cc=kernel@quicinc.com \
--cc=konrad.dybcio@linaro.org \
--cc=krzk@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=luzmaximilian@gmail.com \
--cc=oe-kbuild-all@lists.linux.dev \
--cc=quic_eberman@quicinc.com \
--cc=quic_gurus@quicinc.com \
--cc=srinivas.kandagatla@linaro.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox