From: Dan Carpenter <dan.carpenter@oracle.com>
To: kbuild@lists.01.org, Alvin Lee <Alvin.Lee2@amd.com>
Cc: lkp@intel.com, kbuild-all@lists.01.org,
linux-kernel@vger.kernel.org,
Alex Deucher <alexander.deucher@amd.com>,
Samson Tam <Samson.Tam@amd.com>
Subject: drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_stream.c:395 dc_stream_set_cursor_position() warn: variable dereferenced before check 'stream' (see line 392)
Date: Thu, 18 Aug 2022 14:05:57 +0300 [thread overview]
Message-ID: <202208171701.gFan3WQE-lkp@intel.com> (raw)
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 3cc40a443a04d52b0c95255dce264068b01e9bfe
commit: a141d2083b462505727e14d98db5fc3cd43d59c6 drm/amd/display: Add debug option for exiting idle optimizations on cursor updates
config: ia64-randconfig-m031-20220814 (https://download.01.org/0day-ci/archive/20220817/202208171701.gFan3WQE-lkp@intel.com/config)
compiler: ia64-linux-gcc (GCC) 12.1.0
If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <lkp@intel.com>
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
smatch warnings:
drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_stream.c:395 dc_stream_set_cursor_position() warn: variable dereferenced before check 'stream' (see line 392)
vim +/stream +395 drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_stream.c
ab2541b6739508 Aric Cyr 2016-12-29 388 bool dc_stream_set_cursor_position(
0971c40e180696 Harry Wentland 2017-07-27 389 struct dc_stream_state *stream,
beb16b6a0f921f Dmytro Laktyushkin 2017-04-21 390 const struct dc_cursor_position *position)
ab2541b6739508 Aric Cyr 2016-12-29 391 {
a141d2083b4625 Alvin Lee 2022-06-03 @392 struct dc *dc = stream->ctx->dc;
^^^^^^^^^^^
Dereference
4d55b0dd1cdd85 Bhawanpreet Lakha 2020-05-21 393 bool reset_idle_optimizations = false;
ab2541b6739508 Aric Cyr 2016-12-29 394
4fa086b9b66408 Leo (Sunpeng Li 2017-07-25 @395) if (NULL == stream) {
^^^^^^^^^^^^^^^
NULL check too late. Presumably it can be deleted.
ab2541b6739508 Aric Cyr 2016-12-29 396 dm_error("DC: dc_stream is NULL!\n");
ab2541b6739508 Aric Cyr 2016-12-29 397 return false;
ab2541b6739508 Aric Cyr 2016-12-29 398 }
ab2541b6739508 Aric Cyr 2016-12-29 399
ab2541b6739508 Aric Cyr 2016-12-29 400 if (NULL == position) {
ab2541b6739508 Aric Cyr 2016-12-29 401 dm_error("DC: cursor position is NULL!\n");
ab2541b6739508 Aric Cyr 2016-12-29 402 return false;
ab2541b6739508 Aric Cyr 2016-12-29 403 }
ab2541b6739508 Aric Cyr 2016-12-29 404
2b77dcc5e5aa38 Anthony Koo 2019-11-05 405 dc = stream->ctx->dc;
bae1f0b8a5b189 Nicholas Kazlauskas 2021-05-19 406 dc_z10_restore(dc);
4d55b0dd1cdd85 Bhawanpreet Lakha 2020-05-21 407
4d55b0dd1cdd85 Bhawanpreet Lakha 2020-05-21 408 /* disable idle optimizations if enabling cursor */
a141d2083b4625 Alvin Lee 2022-06-03 409 if (dc->idle_optimizations_allowed && (!stream->cursor_position.enable || dc->debug.exit_idle_opt_for_cursor_updates)
a141d2083b4625 Alvin Lee 2022-06-03 410 && position->enable) {
4b675aad969507 Joshua Aberback 2020-08-31 411 dc_allow_idle_optimizations(dc, false);
4d55b0dd1cdd85 Bhawanpreet Lakha 2020-05-21 412 reset_idle_optimizations = true;
4d55b0dd1cdd85 Bhawanpreet Lakha 2020-05-21 413 }
4d55b0dd1cdd85 Bhawanpreet Lakha 2020-05-21 414
33fd17d9125c00 Eric Yang 2018-01-18 415 stream->cursor_position = *position;
ab2541b6739508 Aric Cyr 2016-12-29 416
4fd771ea441ed9 Roy Chan 2021-07-19 417 program_cursor_position(dc, stream, position);
4d55b0dd1cdd85 Bhawanpreet Lakha 2020-05-21 418 /* re-enable idle optimizations if necessary */
4d55b0dd1cdd85 Bhawanpreet Lakha 2020-05-21 419 if (reset_idle_optimizations)
4b675aad969507 Joshua Aberback 2020-08-31 420 dc_allow_idle_optimizations(dc, true);
4d55b0dd1cdd85 Bhawanpreet Lakha 2020-05-21 421
beb16b6a0f921f Dmytro Laktyushkin 2017-04-21 422 return true;
ab2541b6739508 Aric Cyr 2016-12-29 423 }
--
0-DAY CI Kernel Test Service
https://01.org/lkp
WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com>
To: kbuild@lists.01.org
Subject: drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_stream.c:395 dc_stream_set_cursor_position() warn: variable dereferenced before check 'stream' (see line 392)
Date: Wed, 17 Aug 2022 17:15:08 +0800 [thread overview]
Message-ID: <202208171701.gFan3WQE-lkp@intel.com> (raw)
[-- Attachment #1: Type: text/plain, Size: 4419 bytes --]
BCC: lkp(a)intel.com
CC: kbuild-all(a)lists.01.org
CC: linux-kernel(a)vger.kernel.org
TO: Alvin Lee <Alvin.Lee2@amd.com>
CC: Alex Deucher <alexander.deucher@amd.com>
CC: Samson Tam <Samson.Tam@amd.com>
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 3cc40a443a04d52b0c95255dce264068b01e9bfe
commit: a141d2083b462505727e14d98db5fc3cd43d59c6 drm/amd/display: Add debug option for exiting idle optimizations on cursor updates
date: 9 weeks ago
:::::: branch date: 14 hours ago
:::::: commit date: 9 weeks ago
config: ia64-randconfig-m031-20220814 (https://download.01.org/0day-ci/archive/20220817/202208171701.gFan3WQE-lkp(a)intel.com/config)
compiler: ia64-linux-gcc (GCC) 12.1.0
If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <lkp@intel.com>
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
smatch warnings:
drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_stream.c:395 dc_stream_set_cursor_position() warn: variable dereferenced before check 'stream' (see line 392)
vim +/stream +395 drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_stream.c
ab2541b6739508 Aric Cyr 2016-12-29 387
ab2541b6739508 Aric Cyr 2016-12-29 388 bool dc_stream_set_cursor_position(
0971c40e180696 Harry Wentland 2017-07-27 389 struct dc_stream_state *stream,
beb16b6a0f921f Dmytro Laktyushkin 2017-04-21 390 const struct dc_cursor_position *position)
ab2541b6739508 Aric Cyr 2016-12-29 391 {
a141d2083b4625 Alvin Lee 2022-06-03 @392 struct dc *dc = stream->ctx->dc;
4d55b0dd1cdd85 Bhawanpreet Lakha 2020-05-21 393 bool reset_idle_optimizations = false;
ab2541b6739508 Aric Cyr 2016-12-29 394
4fa086b9b66408 Leo (Sunpeng Li 2017-07-25 @395) if (NULL == stream) {
ab2541b6739508 Aric Cyr 2016-12-29 396 dm_error("DC: dc_stream is NULL!\n");
ab2541b6739508 Aric Cyr 2016-12-29 397 return false;
ab2541b6739508 Aric Cyr 2016-12-29 398 }
ab2541b6739508 Aric Cyr 2016-12-29 399
ab2541b6739508 Aric Cyr 2016-12-29 400 if (NULL == position) {
ab2541b6739508 Aric Cyr 2016-12-29 401 dm_error("DC: cursor position is NULL!\n");
ab2541b6739508 Aric Cyr 2016-12-29 402 return false;
ab2541b6739508 Aric Cyr 2016-12-29 403 }
ab2541b6739508 Aric Cyr 2016-12-29 404
2b77dcc5e5aa38 Anthony Koo 2019-11-05 405 dc = stream->ctx->dc;
bae1f0b8a5b189 Nicholas Kazlauskas 2021-05-19 406 dc_z10_restore(dc);
4d55b0dd1cdd85 Bhawanpreet Lakha 2020-05-21 407
4d55b0dd1cdd85 Bhawanpreet Lakha 2020-05-21 408 /* disable idle optimizations if enabling cursor */
a141d2083b4625 Alvin Lee 2022-06-03 409 if (dc->idle_optimizations_allowed && (!stream->cursor_position.enable || dc->debug.exit_idle_opt_for_cursor_updates)
a141d2083b4625 Alvin Lee 2022-06-03 410 && position->enable) {
4b675aad969507 Joshua Aberback 2020-08-31 411 dc_allow_idle_optimizations(dc, false);
4d55b0dd1cdd85 Bhawanpreet Lakha 2020-05-21 412 reset_idle_optimizations = true;
4d55b0dd1cdd85 Bhawanpreet Lakha 2020-05-21 413 }
4d55b0dd1cdd85 Bhawanpreet Lakha 2020-05-21 414
33fd17d9125c00 Eric Yang 2018-01-18 415 stream->cursor_position = *position;
ab2541b6739508 Aric Cyr 2016-12-29 416
4fd771ea441ed9 Roy Chan 2021-07-19 417 program_cursor_position(dc, stream, position);
4d55b0dd1cdd85 Bhawanpreet Lakha 2020-05-21 418 /* re-enable idle optimizations if necessary */
4d55b0dd1cdd85 Bhawanpreet Lakha 2020-05-21 419 if (reset_idle_optimizations)
4b675aad969507 Joshua Aberback 2020-08-31 420 dc_allow_idle_optimizations(dc, true);
4d55b0dd1cdd85 Bhawanpreet Lakha 2020-05-21 421
beb16b6a0f921f Dmytro Laktyushkin 2017-04-21 422 return true;
ab2541b6739508 Aric Cyr 2016-12-29 423 }
ab2541b6739508 Aric Cyr 2016-12-29 424
:::::: The code at line 395 was first introduced by commit
:::::: 4fa086b9b6640818c053c79d4d7104790ba76cb7 drm/amd/display: Roll core_stream into dc_stream
:::::: TO: Leo (Sunpeng) Li <sunpeng.li@amd.com>
:::::: CC: Alex Deucher <alexander.deucher@amd.com>
--
0-DAY CI Kernel Test Service
https://01.org/lkp
WARNING: multiple messages have this Message-ID (diff)
From: Dan Carpenter <dan.carpenter@oracle.com>
To: kbuild-all@lists.01.org
Subject: drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_stream.c:395 dc_stream_set_cursor_position() warn: variable dereferenced before check 'stream' (see line 392)
Date: Thu, 18 Aug 2022 14:05:57 +0300 [thread overview]
Message-ID: <202208171701.gFan3WQE-lkp@intel.com> (raw)
[-- Attachment #1: Type: text/plain, Size: 3983 bytes --]
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 3cc40a443a04d52b0c95255dce264068b01e9bfe
commit: a141d2083b462505727e14d98db5fc3cd43d59c6 drm/amd/display: Add debug option for exiting idle optimizations on cursor updates
config: ia64-randconfig-m031-20220814 (https://download.01.org/0day-ci/archive/20220817/202208171701.gFan3WQE-lkp(a)intel.com/config)
compiler: ia64-linux-gcc (GCC) 12.1.0
If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <lkp@intel.com>
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
smatch warnings:
drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_stream.c:395 dc_stream_set_cursor_position() warn: variable dereferenced before check 'stream' (see line 392)
vim +/stream +395 drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_stream.c
ab2541b6739508 Aric Cyr 2016-12-29 388 bool dc_stream_set_cursor_position(
0971c40e180696 Harry Wentland 2017-07-27 389 struct dc_stream_state *stream,
beb16b6a0f921f Dmytro Laktyushkin 2017-04-21 390 const struct dc_cursor_position *position)
ab2541b6739508 Aric Cyr 2016-12-29 391 {
a141d2083b4625 Alvin Lee 2022-06-03 @392 struct dc *dc = stream->ctx->dc;
^^^^^^^^^^^
Dereference
4d55b0dd1cdd85 Bhawanpreet Lakha 2020-05-21 393 bool reset_idle_optimizations = false;
ab2541b6739508 Aric Cyr 2016-12-29 394
4fa086b9b66408 Leo (Sunpeng Li 2017-07-25 @395) if (NULL == stream) {
^^^^^^^^^^^^^^^
NULL check too late. Presumably it can be deleted.
ab2541b6739508 Aric Cyr 2016-12-29 396 dm_error("DC: dc_stream is NULL!\n");
ab2541b6739508 Aric Cyr 2016-12-29 397 return false;
ab2541b6739508 Aric Cyr 2016-12-29 398 }
ab2541b6739508 Aric Cyr 2016-12-29 399
ab2541b6739508 Aric Cyr 2016-12-29 400 if (NULL == position) {
ab2541b6739508 Aric Cyr 2016-12-29 401 dm_error("DC: cursor position is NULL!\n");
ab2541b6739508 Aric Cyr 2016-12-29 402 return false;
ab2541b6739508 Aric Cyr 2016-12-29 403 }
ab2541b6739508 Aric Cyr 2016-12-29 404
2b77dcc5e5aa38 Anthony Koo 2019-11-05 405 dc = stream->ctx->dc;
bae1f0b8a5b189 Nicholas Kazlauskas 2021-05-19 406 dc_z10_restore(dc);
4d55b0dd1cdd85 Bhawanpreet Lakha 2020-05-21 407
4d55b0dd1cdd85 Bhawanpreet Lakha 2020-05-21 408 /* disable idle optimizations if enabling cursor */
a141d2083b4625 Alvin Lee 2022-06-03 409 if (dc->idle_optimizations_allowed && (!stream->cursor_position.enable || dc->debug.exit_idle_opt_for_cursor_updates)
a141d2083b4625 Alvin Lee 2022-06-03 410 && position->enable) {
4b675aad969507 Joshua Aberback 2020-08-31 411 dc_allow_idle_optimizations(dc, false);
4d55b0dd1cdd85 Bhawanpreet Lakha 2020-05-21 412 reset_idle_optimizations = true;
4d55b0dd1cdd85 Bhawanpreet Lakha 2020-05-21 413 }
4d55b0dd1cdd85 Bhawanpreet Lakha 2020-05-21 414
33fd17d9125c00 Eric Yang 2018-01-18 415 stream->cursor_position = *position;
ab2541b6739508 Aric Cyr 2016-12-29 416
4fd771ea441ed9 Roy Chan 2021-07-19 417 program_cursor_position(dc, stream, position);
4d55b0dd1cdd85 Bhawanpreet Lakha 2020-05-21 418 /* re-enable idle optimizations if necessary */
4d55b0dd1cdd85 Bhawanpreet Lakha 2020-05-21 419 if (reset_idle_optimizations)
4b675aad969507 Joshua Aberback 2020-08-31 420 dc_allow_idle_optimizations(dc, true);
4d55b0dd1cdd85 Bhawanpreet Lakha 2020-05-21 421
beb16b6a0f921f Dmytro Laktyushkin 2017-04-21 422 return true;
ab2541b6739508 Aric Cyr 2016-12-29 423 }
--
0-DAY CI Kernel Test Service
https://01.org/lkp
next reply other threads:[~2022-08-18 11:07 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-08-17 9:15 kernel test robot [this message]
2022-08-18 11:05 ` drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_stream.c:395 dc_stream_set_cursor_position() warn: variable dereferenced before check 'stream' (see line 392) Dan Carpenter
2022-08-18 11:05 ` Dan Carpenter
2022-08-19 3:30 kernel test robot
2022-08-25 7:12 kernel test robot
2022-10-27 7:36 Dan Carpenter
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=202208171701.gFan3WQE-lkp@intel.com \
--to=dan.carpenter@oracle.com \
--cc=Alvin.Lee2@amd.com \
--cc=Samson.Tam@amd.com \
--cc=alexander.deucher@amd.com \
--cc=kbuild-all@lists.01.org \
--cc=kbuild@lists.01.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lkp@intel.com \
/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.