From: kernel test robot <lkp@intel.com>
To: Lucas De Marchi <lucas.demarchi@intel.com>
Cc: llvm@lists.linux.dev, oe-kbuild-all@lists.linux.dev,
intel-xe@lists.freedesktop.org,
Jonathan Cavitt <jonathan.cavitt@intel.com>
Subject: [drm-xe:drm-xe-next 9/11] drivers/gpu/drm/xe/xe_heci_gsc.c:190:9: warning: variable 'def' is used uninitialized whenever 'if' condition is false
Date: Wed, 26 Feb 2025 15:05:59 +0800 [thread overview]
Message-ID: <202502261421.aNNUqfV9-lkp@intel.com> (raw)
tree: https://gitlab.freedesktop.org/drm/xe/kernel.git drm-xe-next
head: 6b68c4542ffecc36087a9e14db8fc990c88bb01b
commit: 292b1a8a50545b47d4fafc54452147abd2d1d86c [9/11] drm/xe: Stop ignoring errors from xe_heci_gsc_init()
config: powerpc64-randconfig-001-20250226 (https://download.01.org/0day-ci/archive/20250226/202502261421.aNNUqfV9-lkp@intel.com/config)
compiler: clang version 18.1.8 (https://github.com/llvm/llvm-project 3b5b5c1ec4a3095ab096dd780e84d7ab81f3d7ff)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250226/202502261421.aNNUqfV9-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/202502261421.aNNUqfV9-lkp@intel.com/
All warnings (new ones prefixed by >>):
>> drivers/gpu/drm/xe/xe_heci_gsc.c:190:9: warning: variable 'def' is used uninitialized whenever 'if' condition is false [-Wsometimes-uninitialized]
190 | } else if (xe->info.platform == XE_DG1) {
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler.h:55:28: note: expanded from macro 'if'
55 | #define if(cond, ...) if ( __trace_if_var( !!(cond , ## __VA_ARGS__) ) )
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler.h:57:30: note: expanded from macro '__trace_if_var'
57 | #define __trace_if_var(cond) (__builtin_constant_p(cond) ? (cond) : __trace_if_value(cond))
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/xe/xe_heci_gsc.c:194:7: note: uninitialized use occurs here
194 | if (!def || !def->name) {
| ^~~
include/linux/compiler.h:55:47: note: expanded from macro 'if'
55 | #define if(cond, ...) if ( __trace_if_var( !!(cond , ## __VA_ARGS__) ) )
| ^~~~
include/linux/compiler.h:57:52: note: expanded from macro '__trace_if_var'
57 | #define __trace_if_var(cond) (__builtin_constant_p(cond) ? (cond) : __trace_if_value(cond))
| ^~~~
drivers/gpu/drm/xe/xe_heci_gsc.c:190:9: note: remove the 'if' if its condition is always true
190 | } else if (xe->info.platform == XE_DG1) {
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler.h:55:23: note: expanded from macro 'if'
55 | #define if(cond, ...) if ( __trace_if_var( !!(cond , ## __VA_ARGS__) ) )
| ^
drivers/gpu/drm/xe/xe_heci_gsc.c:176:32: note: initialize the variable 'def' to silence this warning
176 | const struct heci_gsc_def *def;
| ^
| = NULL
1 warning generated.
vim +190 drivers/gpu/drm/xe/xe_heci_gsc.c
87a4c85d3a3ed5 Vitaly Lubart 2023-08-28 172
292b1a8a50545b Lucas De Marchi 2025-02-21 173 int xe_heci_gsc_init(struct xe_device *xe)
87a4c85d3a3ed5 Vitaly Lubart 2023-08-28 174 {
87a4c85d3a3ed5 Vitaly Lubart 2023-08-28 175 struct xe_heci_gsc *heci_gsc = &xe->heci_gsc;
87a4c85d3a3ed5 Vitaly Lubart 2023-08-28 176 const struct heci_gsc_def *def;
87a4c85d3a3ed5 Vitaly Lubart 2023-08-28 177 int ret;
87a4c85d3a3ed5 Vitaly Lubart 2023-08-28 178
caf4ee1a44511f Lucas De Marchi 2024-11-14 179 if (!xe->info.has_heci_gscfi && !xe->info.has_heci_cscfi)
292b1a8a50545b Lucas De Marchi 2025-02-21 180 return 0;
87a4c85d3a3ed5 Vitaly Lubart 2023-08-28 181
87a4c85d3a3ed5 Vitaly Lubart 2023-08-28 182 heci_gsc->irq = -1;
87a4c85d3a3ed5 Vitaly Lubart 2023-08-28 183
e02cea83d32d3a Alexander Usyskin 2024-07-08 184 if (xe->info.platform == XE_BATTLEMAGE) {
e02cea83d32d3a Alexander Usyskin 2024-07-08 185 def = &heci_gsc_def_dg2;
e02cea83d32d3a Alexander Usyskin 2024-07-08 186 } else if (xe->info.platform == XE_PVC) {
86017f3898d4ac Alexander Usyskin 2023-11-07 187 def = &heci_gsc_def_pvc;
86017f3898d4ac Alexander Usyskin 2023-11-07 188 } else if (xe->info.platform == XE_DG2) {
87a4c85d3a3ed5 Vitaly Lubart 2023-08-28 189 def = &heci_gsc_def_dg2;
87a4c85d3a3ed5 Vitaly Lubart 2023-08-28 @190 } else if (xe->info.platform == XE_DG1) {
87a4c85d3a3ed5 Vitaly Lubart 2023-08-28 191 def = &heci_gsc_def_dg1;
87a4c85d3a3ed5 Vitaly Lubart 2023-08-28 192 }
87a4c85d3a3ed5 Vitaly Lubart 2023-08-28 193
292b1a8a50545b Lucas De Marchi 2025-02-21 194 if (!def || !def->name) {
292b1a8a50545b Lucas De Marchi 2025-02-21 195 drm_warn(&xe->drm, "HECI is not implemented!\n");
292b1a8a50545b Lucas De Marchi 2025-02-21 196 return 0;
87a4c85d3a3ed5 Vitaly Lubart 2023-08-28 197 }
87a4c85d3a3ed5 Vitaly Lubart 2023-08-28 198
292b1a8a50545b Lucas De Marchi 2025-02-21 199 ret = devm_add_action_or_reset(xe->drm.dev, xe_heci_gsc_fini, heci_gsc);
292b1a8a50545b Lucas De Marchi 2025-02-21 200 if (ret)
292b1a8a50545b Lucas De Marchi 2025-02-21 201 return ret;
292b1a8a50545b Lucas De Marchi 2025-02-21 202
d40f275d96e890 Lucas De Marchi 2025-02-21 203 if (!def->use_polling && !xe_survivability_mode_is_enabled(xe)) {
87a4c85d3a3ed5 Vitaly Lubart 2023-08-28 204 ret = heci_gsc_irq_setup(xe);
87a4c85d3a3ed5 Vitaly Lubart 2023-08-28 205 if (ret)
292b1a8a50545b Lucas De Marchi 2025-02-21 206 return ret;
87a4c85d3a3ed5 Vitaly Lubart 2023-08-28 207 }
87a4c85d3a3ed5 Vitaly Lubart 2023-08-28 208
292b1a8a50545b Lucas De Marchi 2025-02-21 209 return heci_gsc_add_device(xe, def);
87a4c85d3a3ed5 Vitaly Lubart 2023-08-28 210 }
87a4c85d3a3ed5 Vitaly Lubart 2023-08-28 211
:::::: The code at line 190 was first introduced by commit
:::::: 87a4c85d3a3ed579c86fd2612715ccb94c4001ff drm/xe/gsc: add gsc device support
:::::: TO: Vitaly Lubart <vitaly.lubart@intel.com>
:::::: CC: Rodrigo Vivi <rodrigo.vivi@intel.com>
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
reply other threads:[~2025-02-26 7:09 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
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=202502261421.aNNUqfV9-lkp@intel.com \
--to=lkp@intel.com \
--cc=intel-xe@lists.freedesktop.org \
--cc=jonathan.cavitt@intel.com \
--cc=llvm@lists.linux.dev \
--cc=lucas.demarchi@intel.com \
--cc=oe-kbuild-all@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox