From: kernel test robot <lkp@intel.com>
To: Nas Chung <nas.chung@chipsnmedia.com>
Cc: llvm@lists.linux.dev, kbuild-all@lists.01.org,
linux-kernel@vger.kernel.org, Hans Verkuil <hverkuil@xs4all.nl>,
Robert Beckett <bob.beckett@collabora.com>,
Dafna Hirschfeld <dafna.hirschfeld@collabora.com>
Subject: [hverkuil-media-tree:for-v5.20g 3/6] drivers/staging/media/wave5/wave5-vpu-dec.c:1338:6: warning: variable 'ret' is used uninitialized whenever 'if' condition is true
Date: Fri, 1 Jul 2022 06:16:51 +0800 [thread overview]
Message-ID: <202207010647.LBFPssdW-lkp@intel.com> (raw)
tree: git://linuxtv.org/hverkuil/media_tree.git for-v5.20g
head: 4a01406b7d1d14a1789d2fde39e8803622a76a5f
commit: 06872f144660558e5bf17a3c8979d0fe39f8414a [3/6] staging: media: wave5: Add the v4l2 layer
config: i386-allmodconfig (https://download.01.org/0day-ci/archive/20220701/202207010647.LBFPssdW-lkp@intel.com/config)
compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project a774ba7f60d1fef403b5507b1b1a7475d3684d71)
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
git remote add hverkuil-media-tree git://linuxtv.org/hverkuil/media_tree.git
git fetch --no-tags hverkuil-media-tree for-v5.20g
git checkout 06872f144660558e5bf17a3c8979d0fe39f8414a
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=i386 SHELL=/bin/bash drivers/staging/
If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
drivers/staging/media/wave5/wave5-vpu-dec.c:1143:7: warning: variable 'ret' set but not used [-Wunused-but-set-variable]
int ret;
^
>> drivers/staging/media/wave5/wave5-vpu-dec.c:1338:6: warning: variable 'ret' is used uninitialized whenever 'if' condition is true [-Wsometimes-uninitialized]
if (kfifo_alloc(&inst->irq_status, 16 * sizeof(int), GFP_KERNEL)) {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/kfifo.h:350:43: note: expanded from macro 'kfifo_alloc'
#define kfifo_alloc(fifo, size, gfp_mask) \
^
drivers/staging/media/wave5/wave5-vpu-dec.c:1358:9: note: uninitialized use occurs here
return ret;
^~~
drivers/staging/media/wave5/wave5-vpu-dec.c:1338:2: note: remove the 'if' if its condition is always false
if (kfifo_alloc(&inst->irq_status, 16 * sizeof(int), GFP_KERNEL)) {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/staging/media/wave5/wave5-vpu-dec.c:1286:9: note: initialize the variable 'ret' to silence this warning
int ret;
^
= 0
2 warnings generated.
--
>> drivers/staging/media/wave5/wave5-vpu.c:233:3: warning: format specifies type 'unsigned long long' but the argument has type 'dma_addr_t' (aka 'unsigned int') [-Wformat]
dev->sram_buf.daddr, dev->sram_buf.size);
^~~~~~~~~~~~~~~~~~~
include/linux/dev_printk.h:144:65: note: expanded from macro 'dev_err'
dev_printk_index_wrap(_dev_err, KERN_ERR, dev, dev_fmt(fmt), ##__VA_ARGS__)
~~~ ^~~~~~~~~~~
include/linux/dev_printk.h:110:23: note: expanded from macro 'dev_printk_index_wrap'
_p_func(dev, fmt, ##__VA_ARGS__); \
~~~ ^~~~~~~~~~~
>> drivers/staging/media/wave5/wave5-vpu.c:233:24: warning: format specifies type 'unsigned long' but the argument has type 'size_t' (aka 'unsigned int') [-Wformat]
dev->sram_buf.daddr, dev->sram_buf.size);
^~~~~~~~~~~~~~~~~~
include/linux/dev_printk.h:144:65: note: expanded from macro 'dev_err'
dev_printk_index_wrap(_dev_err, KERN_ERR, dev, dev_fmt(fmt), ##__VA_ARGS__)
~~~ ^~~~~~~~~~~
include/linux/dev_printk.h:110:23: note: expanded from macro 'dev_printk_index_wrap'
_p_func(dev, fmt, ##__VA_ARGS__); \
~~~ ^~~~~~~~~~~
2 warnings generated.
--
drivers/staging/media/wave5/wave5-vpu-enc.c:1247:7: warning: variable 'ret' set but not used [-Wunused-but-set-variable]
int ret;
^
>> drivers/staging/media/wave5/wave5-vpu-enc.c:1459:6: warning: variable 'ret' is used uninitialized whenever 'if' condition is true [-Wsometimes-uninitialized]
if (kfifo_alloc(&inst->irq_status, 16 * sizeof(int), GFP_KERNEL)) {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/kfifo.h:350:43: note: expanded from macro 'kfifo_alloc'
#define kfifo_alloc(fifo, size, gfp_mask) \
^
drivers/staging/media/wave5/wave5-vpu-enc.c:1479:9: note: uninitialized use occurs here
return ret;
^~~
drivers/staging/media/wave5/wave5-vpu-enc.c:1459:2: note: remove the 'if' if its condition is always false
if (kfifo_alloc(&inst->irq_status, 16 * sizeof(int), GFP_KERNEL)) {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/staging/media/wave5/wave5-vpu-enc.c:1379:9: note: initialize the variable 'ret' to silence this warning
int ret;
^
= 0
2 warnings generated.
vim +1338 drivers/staging/media/wave5/wave5-vpu-dec.c
1280
1281 static int wave5_vpu_open_dec(struct file *filp)
1282 {
1283 struct video_device *vdev = video_devdata(filp);
1284 struct vpu_device *dev = video_drvdata(filp);
1285 struct vpu_instance *inst = NULL;
1286 int ret;
1287
1288 inst = kzalloc(sizeof(*inst), GFP_KERNEL);
1289 if (!inst)
1290 return -ENOMEM;
1291
1292 inst->dev = dev;
1293 inst->type = VPU_INST_TYPE_DEC;
1294 inst->ops = &wave5_vpu_dec_inst_ops;
1295
1296 v4l2_fh_init(&inst->v4l2_fh, vdev);
1297 filp->private_data = &inst->v4l2_fh;
1298 v4l2_fh_add(&inst->v4l2_fh);
1299
1300 INIT_LIST_HEAD(&inst->list);
1301 list_add_tail(&inst->list, &dev->instances);
1302
1303 inst->v4l2_m2m_dev = v4l2_m2m_init(&wave5_vpu_dec_m2m_ops);
1304 if (IS_ERR(inst->v4l2_m2m_dev)) {
1305 ret = PTR_ERR(inst->v4l2_m2m_dev);
1306 dev_err(inst->dev->dev, "v4l2_m2m_init fail: %d\n", ret);
1307 goto free_inst;
1308 }
1309
1310 inst->v4l2_fh.m2m_ctx =
1311 v4l2_m2m_ctx_init(inst->v4l2_m2m_dev, inst, wave5_vpu_dec_queue_init);
1312 if (IS_ERR(inst->v4l2_fh.m2m_ctx)) {
1313 ret = PTR_ERR(inst->v4l2_fh.m2m_ctx);
1314 goto err_m2m_release;
1315 }
1316
1317 v4l2_ctrl_handler_init(&inst->v4l2_ctrl_hdl, 10);
1318 v4l2_ctrl_new_custom(&inst->v4l2_ctrl_hdl, &wave5_vpu_thumbnail_mode, NULL);
1319 v4l2_ctrl_new_std(&inst->v4l2_ctrl_hdl, &wave5_vpu_dec_ctrl_ops,
1320 V4L2_CID_MIN_BUFFERS_FOR_CAPTURE, 1, 32, 1, 1);
1321
1322 if (inst->v4l2_ctrl_hdl.error) {
1323 ret = -ENODEV;
1324 goto err_m2m_release;
1325 }
1326
1327 inst->v4l2_fh.ctrl_handler = &inst->v4l2_ctrl_hdl;
1328 v4l2_ctrl_handler_setup(&inst->v4l2_ctrl_hdl);
1329
1330 wave5_set_default_format(&inst->src_fmt, &inst->dst_fmt);
1331 inst->colorspace = V4L2_COLORSPACE_REC709;
1332 inst->ycbcr_enc = V4L2_YCBCR_ENC_DEFAULT;
1333 inst->hsv_enc = 0;
1334 inst->quantization = V4L2_QUANTIZATION_DEFAULT;
1335 inst->xfer_func = V4L2_XFER_FUNC_DEFAULT;
1336
1337 init_completion(&inst->irq_done);
> 1338 if (kfifo_alloc(&inst->irq_status, 16 * sizeof(int), GFP_KERNEL)) {
1339 dev_err(inst->dev->dev, "failed to allocate fifo\n");
1340 goto err_m2m_release;
1341 }
1342
1343 inst->id = ida_alloc(&inst->dev->inst_ida, GFP_KERNEL);
1344 if (inst->id < 0) {
1345 dev_warn(inst->dev->dev, "unable to allocate instance ID: %d\n", inst->id);
1346 ret = inst->id;
1347 goto err_kfifo_free;
1348 }
1349
1350 return 0;
1351
1352 err_kfifo_free:
1353 kfifo_free(&inst->irq_status);
1354 err_m2m_release:
1355 v4l2_m2m_release(inst->v4l2_m2m_dev);
1356 free_inst:
1357 kfree(inst);
1358 return ret;
1359 }
1360
--
0-DAY CI Kernel Test Service
https://01.org/lkp
reply other threads:[~2022-06-30 22:17 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=202207010647.LBFPssdW-lkp@intel.com \
--to=lkp@intel.com \
--cc=bob.beckett@collabora.com \
--cc=dafna.hirschfeld@collabora.com \
--cc=hverkuil@xs4all.nl \
--cc=kbuild-all@lists.01.org \
--cc=linux-kernel@vger.kernel.org \
--cc=llvm@lists.linux.dev \
--cc=nas.chung@chipsnmedia.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.