From: kernel test robot <lkp@intel.com>
To: Cristian Marussi <cristian.marussi@arm.com>
Cc: llvm@lists.linux.dev, kbuild-all@lists.01.org
Subject: Re: [RFC PATCH 2/2] hwmon: (scmi) Filter out results wrongly interpreted as negatives
Date: Tue, 21 Dec 2021 11:54:53 +0800 [thread overview]
Message-ID: <202112211141.DiW591Y0-lkp@intel.com> (raw)
In-Reply-To: <20211220174155.40239-3-cristian.marussi@arm.com>
Hi Cristian,
[FYI, it's a private test report for your RFC patch.]
[auto build test WARNING on linux/master]
[also build test WARNING on groeck-staging/hwmon-next soc/for-next linus/master v5.16-rc6 next-20211220]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Cristian-Marussi/Sensor-readings-fixes/20211221-014235
base: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 136057256686de39cc3a07c2e39ef6bc43003ff6
config: arm-randconfig-r031-20211220 (https://download.01.org/0day-ci/archive/20211221/202112211141.DiW591Y0-lkp@intel.com/config)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project 555eacf75f21cd1dfc6363d73ad187b730349543)
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
# install arm cross compiling tool for clang build
# apt-get install binutils-arm-linux-gnueabi
# https://github.com/0day-ci/linux/commit/6a3ccc22734d65dfe13f598a7469c4ca3094eeea
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Cristian-Marussi/Sensor-readings-fixes/20211221-014235
git checkout 6a3ccc22734d65dfe13f598a7469c4ca3094eeea
# save the config file to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=arm SHELL=/bin/bash drivers/hwmon/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
>> drivers/hwmon/scmi-hwmon.c:88:14: warning: shift count >= width of type [-Wshift-count-overflow]
if (value & BIT(63)) {
^~~~~~~
include/vdso/bits.h:7:26: note: expanded from macro 'BIT'
#define BIT(nr) (UL(1) << (nr))
^ ~~~~
1 warning generated.
vim +88 drivers/hwmon/scmi-hwmon.c
66
67 static int scmi_hwmon_read(struct device *dev, enum hwmon_sensor_types type,
68 u32 attr, int channel, long *val)
69 {
70 int ret;
71 u64 value;
72 const struct scmi_sensor_info *sensor;
73 struct scmi_sensors *scmi_sensors = dev_get_drvdata(dev);
74
75 sensor = *(scmi_sensors->info[type] + channel);
76 /*
77 * Can fail with EIO if the backing SCMI Sensor FW version tried to
78 * report a negative value.
79 */
80 ret = sensor_ops->reading_get(scmi_sensors->ph, sensor->id, &value);
81 if (ret)
82 return ret;
83
84 /*
85 * Cannot accept either valid positive values so big that would be
86 * interpreted as negative by HWMON signed long *val return value.
87 */
> 88 if (value & BIT(63)) {
89 dev_warn_once(dev,
90 "Reported unsigned value too big.\n");
91 return -EIO;
92 }
93
94 ret = scmi_hwmon_scale(sensor, &value);
95 if (!ret)
96 *val = value;
97
98 return ret;
99 }
100
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: Re: [RFC PATCH 2/2] hwmon: (scmi) Filter out results wrongly interpreted as negatives
Date: Tue, 21 Dec 2021 11:54:53 +0800 [thread overview]
Message-ID: <202112211141.DiW591Y0-lkp@intel.com> (raw)
In-Reply-To: <20211220174155.40239-3-cristian.marussi@arm.com>
[-- Attachment #1: Type: text/plain, Size: 3483 bytes --]
Hi Cristian,
[FYI, it's a private test report for your RFC patch.]
[auto build test WARNING on linux/master]
[also build test WARNING on groeck-staging/hwmon-next soc/for-next linus/master v5.16-rc6 next-20211220]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Cristian-Marussi/Sensor-readings-fixes/20211221-014235
base: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 136057256686de39cc3a07c2e39ef6bc43003ff6
config: arm-randconfig-r031-20211220 (https://download.01.org/0day-ci/archive/20211221/202112211141.DiW591Y0-lkp(a)intel.com/config)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project 555eacf75f21cd1dfc6363d73ad187b730349543)
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
# install arm cross compiling tool for clang build
# apt-get install binutils-arm-linux-gnueabi
# https://github.com/0day-ci/linux/commit/6a3ccc22734d65dfe13f598a7469c4ca3094eeea
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Cristian-Marussi/Sensor-readings-fixes/20211221-014235
git checkout 6a3ccc22734d65dfe13f598a7469c4ca3094eeea
# save the config file to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=arm SHELL=/bin/bash drivers/hwmon/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
>> drivers/hwmon/scmi-hwmon.c:88:14: warning: shift count >= width of type [-Wshift-count-overflow]
if (value & BIT(63)) {
^~~~~~~
include/vdso/bits.h:7:26: note: expanded from macro 'BIT'
#define BIT(nr) (UL(1) << (nr))
^ ~~~~
1 warning generated.
vim +88 drivers/hwmon/scmi-hwmon.c
66
67 static int scmi_hwmon_read(struct device *dev, enum hwmon_sensor_types type,
68 u32 attr, int channel, long *val)
69 {
70 int ret;
71 u64 value;
72 const struct scmi_sensor_info *sensor;
73 struct scmi_sensors *scmi_sensors = dev_get_drvdata(dev);
74
75 sensor = *(scmi_sensors->info[type] + channel);
76 /*
77 * Can fail with EIO if the backing SCMI Sensor FW version tried to
78 * report a negative value.
79 */
80 ret = sensor_ops->reading_get(scmi_sensors->ph, sensor->id, &value);
81 if (ret)
82 return ret;
83
84 /*
85 * Cannot accept either valid positive values so big that would be
86 * interpreted as negative by HWMON signed long *val return value.
87 */
> 88 if (value & BIT(63)) {
89 dev_warn_once(dev,
90 "Reported unsigned value too big.\n");
91 return -EIO;
92 }
93
94 ret = scmi_hwmon_scale(sensor, &value);
95 if (!ret)
96 *val = value;
97
98 return ret;
99 }
100
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org
next prev parent reply other threads:[~2021-12-21 3:55 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-12-20 17:41 [RFC PATCH 0/2] Sensor readings fixes Cristian Marussi
2021-12-20 17:41 ` Cristian Marussi
2021-12-20 17:41 ` [RFC PATCH 1/2] firmware: arm_scmi: Filter out negative results on scmi_reading_get Cristian Marussi
2021-12-20 17:41 ` Cristian Marussi
2021-12-20 22:28 ` kernel test robot
2021-12-20 22:28 ` kernel test robot
2021-12-20 17:41 ` [RFC PATCH 2/2] hwmon: (scmi) Filter out results wrongly interpreted as negatives Cristian Marussi
2021-12-20 17:41 ` Cristian Marussi
2021-12-21 3:54 ` kernel test robot [this message]
2021-12-21 3:54 ` kernel test robot
2022-03-25 11:41 ` [RFC PATCH 0/2] Sensor readings fixes Cristian Marussi
2022-03-25 11:41 ` Cristian Marussi
2022-03-25 21:38 ` Florian Fainelli
2022-03-25 21:38 ` Florian Fainelli
2022-03-30 14:43 ` Cristian Marussi
2022-03-30 14:43 ` Cristian Marussi
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=202112211141.DiW591Y0-lkp@intel.com \
--to=lkp@intel.com \
--cc=cristian.marussi@arm.com \
--cc=kbuild-all@lists.01.org \
--cc=llvm@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.