* [jpirko-mlxsw:linecards 110/112] drivers/platform/mellanox/mlxreg-lc.c:485:1-7: preceding lock on line 469 (fwd)
@ 2021-08-28 11:07 Julia Lawall
0 siblings, 0 replies; only message in thread
From: Julia Lawall @ 2021-08-28 11:07 UTC (permalink / raw)
To: Vadim Pasternak; +Cc: Jiri Pirko, kbuild-all, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 3038 bytes --]
This looks suspicious. Maybe the fail label shoudl be before line 482 to
include the unlock?
julia
---------- Forwarded message ----------
Date: Thu, 26 Aug 2021 05:55:03 +0800
From: kernel test robot <lkp@intel.com>
To: kbuild@lists.01.org
Cc: lkp@intel.com, Julia Lawall <julia.lawall@lip6.fr>
Subject: [jpirko-mlxsw:linecards 110/112]
drivers/platform/mellanox/mlxreg-lc.c:485:1-7: preceding lock on line 469
CC: kbuild-all@lists.01.org
CC: linux-kernel@vger.kernel.org
TO: Vadim Pasternak <vadimp@nvidia.com>
CC: Jiri Pirko <jiri@nvidia.com>
tree: https://github.com/jpirko/linux_mlxsw linecards
head: a37df1af1f3e6a47a9718298d5e50b842f79b2d2
commit: fbdba1b20b39b5e3b79d6df5b44cc5be3a0ddc0c [110/112] TMP: platform/mellanox
:::::: branch date: 9 hours ago
:::::: commit date: 12 hours ago
config: xtensa-randconfig-c023-20210825 (attached as .config)
compiler: xtensa-linux-gcc (GCC) 11.2.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
Reported-by: Julia Lawall <julia.lawall@lip6.fr>
cocci warnings: (new ones prefixed by >>)
>> drivers/platform/mellanox/mlxreg-lc.c:485:1-7: preceding lock on line 469
vim +485 drivers/platform/mellanox/mlxreg-lc.c
b386a25dac45ae Vadim Pasternak 2021-03-25 463
b386a25dac45ae Vadim Pasternak 2021-03-25 464 static int mlxreg_lc_power_on_off(struct mlxreg_lc *mlxreg_lc, u8 action)
b386a25dac45ae Vadim Pasternak 2021-03-25 465 {
b386a25dac45ae Vadim Pasternak 2021-03-25 466 u32 regval;
b386a25dac45ae Vadim Pasternak 2021-03-25 467 int err;
b386a25dac45ae Vadim Pasternak 2021-03-25 468
fbdba1b20b39b5 Vadim Pasternak 2021-07-01 @469 mutex_lock(&mlxreg_lc->lock);
fbdba1b20b39b5 Vadim Pasternak 2021-07-01 470
b386a25dac45ae Vadim Pasternak 2021-03-25 471 err = regmap_read(mlxreg_lc->par_regmap, mlxreg_lc->data->reg_pwr, ®val);
b386a25dac45ae Vadim Pasternak 2021-03-25 472 if (err)
fbdba1b20b39b5 Vadim Pasternak 2021-07-01 473 goto regmap_read_fail;
b386a25dac45ae Vadim Pasternak 2021-03-25 474
b386a25dac45ae Vadim Pasternak 2021-03-25 475 if (action)
fbdba1b20b39b5 Vadim Pasternak 2021-07-01 476 regval |= BIT(mlxreg_lc->data->slot - 1);
b386a25dac45ae Vadim Pasternak 2021-03-25 477 else
fbdba1b20b39b5 Vadim Pasternak 2021-07-01 478 regval &= ~BIT(mlxreg_lc->data->slot - 1);
fbdba1b20b39b5 Vadim Pasternak 2021-07-01 479
fbdba1b20b39b5 Vadim Pasternak 2021-07-01 480 err = regmap_write(mlxreg_lc->par_regmap, mlxreg_lc->data->reg_pwr, regval);
b386a25dac45ae Vadim Pasternak 2021-03-25 481
fbdba1b20b39b5 Vadim Pasternak 2021-07-01 482 mutex_unlock(&mlxreg_lc->lock);
fbdba1b20b39b5 Vadim Pasternak 2021-07-01 483
fbdba1b20b39b5 Vadim Pasternak 2021-07-01 484 regmap_read_fail:
fbdba1b20b39b5 Vadim Pasternak 2021-07-01 @485 return err;
b386a25dac45ae Vadim Pasternak 2021-03-25 486 }
b386a25dac45ae Vadim Pasternak 2021-03-25 487
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
[-- Attachment #2: Type: application/gzip, Size: 32878 bytes --]
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2021-08-28 11:07 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-28 11:07 [jpirko-mlxsw:linecards 110/112] drivers/platform/mellanox/mlxreg-lc.c:485:1-7: preceding lock on line 469 (fwd) Julia Lawall
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).