From: kernel test robot <lkp@intel.com>
To: oe-kbuild@lists.linux.dev
Cc: lkp@intel.com, Dan Carpenter <error27@gmail.com>
Subject: drivers/gpu/drm/i915/gem/i915_gem_stolen.c:173 request_smem_stolen() warn: consider using resource_size() here
Date: Thu, 16 Feb 2023 05:29:13 +0800 [thread overview]
Message-ID: <202302160504.G6Rsomen-lkp@intel.com> (raw)
BCC: lkp@intel.com
CC: oe-kbuild-all@lists.linux.dev
CC: linux-kernel@vger.kernel.org
TO: Lucas De Marchi <lucas.demarchi@intel.com>
CC: Wayne Boyer <wayne.boyer@intel.com>
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: e1c04510f521e853019afeca2a5991a5ef8d6a5b
commit: 3d99597c6496b9319f7522e0d073afab314d518a drm/i915: Split i915_gem_init_stolen()
date: 5 months ago
:::::: branch date: 28 hours ago
:::::: commit date: 5 months ago
config: x86_64-randconfig-m001-20230213 (https://download.01.org/0day-ci/archive/20230216/202302160504.G6Rsomen-lkp@intel.com/config)
compiler: gcc-11 (Debian 11.3.0-8) 11.3.0
If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot <lkp@intel.com>
| Reported-by: Dan Carpenter <error27@gmail.com>
| Link: https://lore.kernel.org/r/202302160504.G6Rsomen-lkp@intel.com/
smatch warnings:
drivers/gpu/drm/i915/gem/i915_gem_stolen.c:173 request_smem_stolen() warn: consider using resource_size() here
drivers/gpu/drm/i915/gem/i915_gem_stolen.c:455 init_reserved_stolen() warn: missing error code 'ret'
vim +173 drivers/gpu/drm/i915/gem/i915_gem_stolen.c
3d99597c6496b9 drivers/gpu/drm/i915/gem/i915_gem_stolen.c Lucas De Marchi 2022-09-16 141
3d99597c6496b9 drivers/gpu/drm/i915/gem/i915_gem_stolen.c Lucas De Marchi 2022-09-16 142 static int request_smem_stolen(struct drm_i915_private *i915,
3d99597c6496b9 drivers/gpu/drm/i915/gem/i915_gem_stolen.c Lucas De Marchi 2022-09-16 143 struct resource *dsm)
3d99597c6496b9 drivers/gpu/drm/i915/gem/i915_gem_stolen.c Lucas De Marchi 2022-09-16 144 {
3d99597c6496b9 drivers/gpu/drm/i915/gem/i915_gem_stolen.c Lucas De Marchi 2022-09-16 145 struct resource *r;
3d99597c6496b9 drivers/gpu/drm/i915/gem/i915_gem_stolen.c Lucas De Marchi 2022-09-16 146
d57d4a1daf5e83 drivers/gpu/drm/i915/gem/i915_gem_stolen.c CQ Tang 2021-04-21 147 /*
3d99597c6496b9 drivers/gpu/drm/i915/gem/i915_gem_stolen.c Lucas De Marchi 2022-09-16 148 * With stolen lmem, we don't need to request system memory for the
3d99597c6496b9 drivers/gpu/drm/i915/gem/i915_gem_stolen.c Lucas De Marchi 2022-09-16 149 * address range since it's local to the gpu.
d57d4a1daf5e83 drivers/gpu/drm/i915/gem/i915_gem_stolen.c CQ Tang 2021-04-21 150 */
d57d4a1daf5e83 drivers/gpu/drm/i915/gem/i915_gem_stolen.c CQ Tang 2021-04-21 151 if (HAS_LMEM(i915))
d57d4a1daf5e83 drivers/gpu/drm/i915/gem/i915_gem_stolen.c CQ Tang 2021-04-21 152 return 0;
d57d4a1daf5e83 drivers/gpu/drm/i915/gem/i915_gem_stolen.c CQ Tang 2021-04-21 153
7789422665f599 drivers/gpu/drm/i915/i915_gem_stolen.c Matthew Auld 2017-12-11 154 /*
7789422665f599 drivers/gpu/drm/i915/i915_gem_stolen.c Matthew Auld 2017-12-11 155 * Verify that nothing else uses this physical address. Stolen
eaba1b8f3379b5 drivers/gpu/drm/i915/i915_gem_stolen.c Chris Wilson 2013-07-04 156 * memory should be reserved by the BIOS and hidden from the
eaba1b8f3379b5 drivers/gpu/drm/i915/i915_gem_stolen.c Chris Wilson 2013-07-04 157 * kernel. So if the region is already marked as busy, something
eaba1b8f3379b5 drivers/gpu/drm/i915/i915_gem_stolen.c Chris Wilson 2013-07-04 158 * is seriously wrong.
eaba1b8f3379b5 drivers/gpu/drm/i915/i915_gem_stolen.c Chris Wilson 2013-07-04 159 */
bdce2bea0f5e6b drivers/gpu/drm/i915/gem/i915_gem_stolen.c Chris Wilson 2019-11-11 160 r = devm_request_mem_region(i915->drm.dev, dsm->start,
7789422665f599 drivers/gpu/drm/i915/i915_gem_stolen.c Matthew Auld 2017-12-11 161 resource_size(dsm),
eaba1b8f3379b5 drivers/gpu/drm/i915/i915_gem_stolen.c Chris Wilson 2013-07-04 162 "Graphics Stolen Memory");
3617dc9675f018 drivers/gpu/drm/i915/i915_gem_stolen.c Akash Goel 2014-01-13 163 if (r == NULL) {
3617dc9675f018 drivers/gpu/drm/i915/i915_gem_stolen.c Akash Goel 2014-01-13 164 /*
3617dc9675f018 drivers/gpu/drm/i915/i915_gem_stolen.c Akash Goel 2014-01-13 165 * One more attempt but this time requesting region from
7789422665f599 drivers/gpu/drm/i915/i915_gem_stolen.c Matthew Auld 2017-12-11 166 * start + 1, as we have seen that this resolves the region
3617dc9675f018 drivers/gpu/drm/i915/i915_gem_stolen.c Akash Goel 2014-01-13 167 * conflict with the PCI Bus.
3617dc9675f018 drivers/gpu/drm/i915/i915_gem_stolen.c Akash Goel 2014-01-13 168 * This is a BIOS w/a: Some BIOS wrap stolen in the root
3617dc9675f018 drivers/gpu/drm/i915/i915_gem_stolen.c Akash Goel 2014-01-13 169 * PCI bus, but have an off-by-one error. Hence retry the
3617dc9675f018 drivers/gpu/drm/i915/i915_gem_stolen.c Akash Goel 2014-01-13 170 * reservation starting from 1 instead of 0.
023f807989f4d6 drivers/gpu/drm/i915/i915_gem_stolen.c Daniel Vetter 2017-07-19 171 * There's also BIOS with off-by-one on the other end.
3617dc9675f018 drivers/gpu/drm/i915/i915_gem_stolen.c Akash Goel 2014-01-13 172 */
bdce2bea0f5e6b drivers/gpu/drm/i915/gem/i915_gem_stolen.c Chris Wilson 2019-11-11 @173 r = devm_request_mem_region(i915->drm.dev, dsm->start + 1,
7789422665f599 drivers/gpu/drm/i915/i915_gem_stolen.c Matthew Auld 2017-12-11 174 resource_size(dsm) - 2,
3617dc9675f018 drivers/gpu/drm/i915/i915_gem_stolen.c Akash Goel 2014-01-13 175 "Graphics Stolen Memory");
0b6d24c01932db drivers/gpu/drm/i915/i915_gem_stolen.c Daniel Vetter 2014-04-11 176 /*
0b6d24c01932db drivers/gpu/drm/i915/i915_gem_stolen.c Daniel Vetter 2014-04-11 177 * GEN3 firmware likes to smash pci bridges into the stolen
0b6d24c01932db drivers/gpu/drm/i915/i915_gem_stolen.c Daniel Vetter 2014-04-11 178 * range. Apparently this works.
0b6d24c01932db drivers/gpu/drm/i915/i915_gem_stolen.c Daniel Vetter 2014-04-11 179 */
40e1956ec505a1 drivers/gpu/drm/i915/gem/i915_gem_stolen.c Lucas De Marchi 2021-06-05 180 if (!r && GRAPHICS_VER(i915) != 3) {
baa89ba3f1feda drivers/gpu/drm/i915/gem/i915_gem_stolen.c Wambui Karuga 2020-01-22 181 drm_err(&i915->drm,
baa89ba3f1feda drivers/gpu/drm/i915/gem/i915_gem_stolen.c Wambui Karuga 2020-01-22 182 "conflict detected with stolen region: %pR\n",
7789422665f599 drivers/gpu/drm/i915/i915_gem_stolen.c Matthew Auld 2017-12-11 183 dsm);
920bcd1820a696 drivers/gpu/drm/i915/i915_gem_stolen.c Paulo Zanoni 2017-01-26 184
7789422665f599 drivers/gpu/drm/i915/i915_gem_stolen.c Matthew Auld 2017-12-11 185 return -EBUSY;
eaba1b8f3379b5 drivers/gpu/drm/i915/i915_gem_stolen.c Chris Wilson 2013-07-04 186 }
3617dc9675f018 drivers/gpu/drm/i915/i915_gem_stolen.c Akash Goel 2014-01-13 187 }
eaba1b8f3379b5 drivers/gpu/drm/i915/i915_gem_stolen.c Chris Wilson 2013-07-04 188
7789422665f599 drivers/gpu/drm/i915/i915_gem_stolen.c Matthew Auld 2017-12-11 189 return 0;
9797fbfbcfe251 drivers/gpu/drm/i915/i915_gem_stolen.c Chris Wilson 2012-04-24 190 }
9797fbfbcfe251 drivers/gpu/drm/i915/i915_gem_stolen.c Chris Wilson 2012-04-24 191
:::::: The code at line 173 was first introduced by commit
:::::: bdce2bea0f5e6bd3e3fa85278cc2fb8632f71572 drm/i915/gem: Replace implicit dev_priv->uncore for stolen init
:::::: TO: Chris Wilson <chris@chris-wilson.co.uk>
:::::: CC: Chris Wilson <chris@chris-wilson.co.uk>
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests
next reply other threads:[~2023-02-15 21:30 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-02-15 21:29 kernel test robot [this message]
-- strict thread matches above, loose matches on Subject: below --
2023-02-18 14:23 drivers/gpu/drm/i915/gem/i915_gem_stolen.c:173 request_smem_stolen() warn: consider using resource_size() here kernel test robot
2023-02-20 8:05 kernel test robot
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=202302160504.G6Rsomen-lkp@intel.com \
--to=lkp@intel.com \
--cc=error27@gmail.com \
--cc=oe-kbuild@lists.linux.dev \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.