linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
* [linux-next:master 4907/6030] drivers/ptp/ptp_ocp.c:830:2: warning: Undefined or garbage value returned to caller [clang-analyzer-core.uninitialized.UndefReturn]
       [not found] <202108081526.htnwreV6-lkp@intel.com>
@ 2021-08-09 10:46 ` kernel test robot
  0 siblings, 0 replies; only message in thread
From: kernel test robot @ 2021-08-09 10:46 UTC (permalink / raw)
  To: Jonathan Lemon
  Cc: clang-built-linux, kbuild-all, Linux Memory Management List

[-- Attachment #1: Type: text/plain, Size: 17210 bytes --]


tree: 
https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
head:   7999516e20bd9bb5d1f7351cbd05ca529a3a8d60
commit: 773bda96492153e11d21eb63ac814669b51fc701 [4907/6030] ptp: ocp: 
Expose various resources on the timecard.
:::::: branch date: 2 days ago
:::::: commit date: 4 days ago
config: x86_64-randconfig-c001-20210807 (attached as .config)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project 
41a6b50c25961addc04438b567ee1f4ef9e40f98)
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 x86_64 cross compiling tool for clang build
         # apt-get install binutils-x86-64-linux-gnu
         # 
https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?id=773bda96492153e11d21eb63ac814669b51fc701
         git remote add linux-next 
https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git
         git fetch --no-tags linux-next master
         git checkout 773bda96492153e11d21eb63ac814669b51fc701
         # save the attached .config to linux build tree
         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross 
ARCH=x86_64 clang-analyzer
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>


clang-analyzer warnings: (new ones prefixed by >>)
    2 warnings generated.
    Suppressed 2 warnings (2 in non-user code).
    Use -header-filter=.* to display errors from all non-system headers. 
Use -system-headers to display errors from system headers as well.
    2 warnings generated.
    Suppressed 2 warnings (2 in non-user code).
    Use -header-filter=.* to display errors from all non-system headers. 
Use -system-headers to display errors from system headers as well.
    3 warnings generated.
    drivers/thermal/devfreq_cooling.c:87:17: warning: Value stored to 
'dev' during its initialization is never read 
[clang-analyzer-deadcode.DeadStores]
            struct device *dev = df->dev.parent;
                           ^~~   ~~~~~~~~~~~~~~
    drivers/thermal/devfreq_cooling.c:87:17: note: Value stored to 'dev' 
during its initialization is never read
            struct device *dev = df->dev.parent;
                           ^~~   ~~~~~~~~~~~~~~
    Suppressed 2 warnings (2 in non-user code).
    Use -header-filter=.* to display errors from all non-system headers. 
Use -system-headers to display errors from system headers as well.
    3 warnings generated.
    Suppressed 3 warnings (3 in non-user code).
    Use -header-filter=.* to display errors from all non-system headers. 
Use -system-headers to display errors from system headers as well.
    3 warnings generated.
    Suppressed 3 warnings (3 in non-user code).
    Use -header-filter=.* to display errors from all non-system headers. 
Use -system-headers to display errors from system headers as well.
    3 warnings generated.
    Suppressed 3 warnings (3 in non-user code).
    Use -header-filter=.* to display errors from all non-system headers. 
Use -system-headers to display errors from system headers as well.
    3 warnings generated.
    Suppressed 3 warnings (3 in non-user code).
    Use -header-filter=.* to display errors from all non-system headers. 
Use -system-headers to display errors from system headers as well.
    4 warnings generated.
    Suppressed 4 warnings (4 in non-user code).
    Use -header-filter=.* to display errors from all non-system headers. 
Use -system-headers to display errors from system headers as well.
    4 warnings generated.
    Suppressed 4 warnings (4 in non-user code).
    Use -header-filter=.* to display errors from all non-system headers. 
Use -system-headers to display errors from system headers as well.
    4 warnings generated.
    Suppressed 4 warnings (4 in non-user code).
    Use -header-filter=.* to display errors from all non-system headers. 
Use -system-headers to display errors from system headers as well.
    4 warnings generated.
    Suppressed 4 warnings (4 in non-user code).
    Use -header-filter=.* to display errors from all non-system headers. 
Use -system-headers to display errors from system headers as well.
    4 warnings generated.
    Suppressed 4 warnings (4 in non-user code).
    Use -header-filter=.* to display errors from all non-system headers. 
Use -system-headers to display errors from system headers as well.
    4 warnings generated.
    Suppressed 4 warnings (4 in non-user code).
    Use -header-filter=.* to display errors from all non-system headers. 
Use -system-headers to display errors from system headers as well.
    4 warnings generated.
    Suppressed 4 warnings (4 in non-user code).
    Use -header-filter=.* to display errors from all non-system headers. 
Use -system-headers to display errors from system headers as well.
    4 warnings generated.
    Suppressed 4 warnings (4 in non-user code).
    Use -header-filter=.* to display errors from all non-system headers. 
Use -system-headers to display errors from system headers as well.
    4 warnings generated.
    Suppressed 4 warnings (4 in non-user code).
    Use -header-filter=.* to display errors from all non-system headers. 
Use -system-headers to display errors from system headers as well.
    4 warnings generated.
    Suppressed 4 warnings (4 in non-user code).
    Use -header-filter=.* to display errors from all non-system headers. 
Use -system-headers to display errors from system headers as well.
    5 warnings generated.
    Suppressed 5 warnings (5 in non-user code).
    Use -header-filter=.* to display errors from all non-system headers. 
Use -system-headers to display errors from system headers as well.
    5 warnings generated.
    drivers/media/i2c/ov5640.c:1005:2: warning: Value stored to 'ret' is 
never read [clang-analyzer-deadcode.DeadStores]
            ret = ov5640_mod_reg(sensor, OV5640_REG_SC_PLL_CTRL0,
            ^     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    drivers/media/i2c/ov5640.c:1005:2: note: Value stored to 'ret' is 
never read
            ret = ov5640_mod_reg(sensor, OV5640_REG_SC_PLL_CTRL0,
            ^     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    Suppressed 4 warnings (4 in non-user code).
    Use -header-filter=.* to display errors from all non-system headers. 
Use -system-headers to display errors from system headers as well.
    4 warnings generated.
    Suppressed 4 warnings (4 in non-user code).
    Use -header-filter=.* to display errors from all non-system headers. 
Use -system-headers to display errors from system headers as well.
    3 warnings generated.
    Suppressed 3 warnings (3 in non-user code).
    Use -header-filter=.* to display errors from all non-system headers. 
Use -system-headers to display errors from system headers as well.
    4 warnings generated.
    Suppressed 4 warnings (4 in non-user code).
    Use -header-filter=.* to display errors from all non-system headers. 
Use -system-headers to display errors from system headers as well.
    4 warnings generated.
    Suppressed 4 warnings (4 in non-user code).
    Use -header-filter=.* to display errors from all non-system headers. 
Use -system-headers to display errors from system headers as well.
    3 warnings generated.
    Suppressed 3 warnings (3 in non-user code).
    Use -header-filter=.* to display errors from all non-system headers. 
Use -system-headers to display errors from system headers as well.
    3 warnings generated.
    Suppressed 3 warnings (3 in non-user code).
    Use -header-filter=.* to display errors from all non-system headers. 
Use -system-headers to display errors from system headers as well.
    2 warnings generated.
    Suppressed 2 warnings (2 in non-user code).
    Use -header-filter=.* to display errors from all non-system headers. 
Use -system-headers to display errors from system headers as well.
    3 warnings generated.
    Suppressed 3 warnings (3 in non-user code).
    Use -header-filter=.* to display errors from all non-system headers. 
Use -system-headers to display errors from system headers as well.
    3 warnings generated.
    Suppressed 3 warnings (3 in non-user code).
    Use -header-filter=.* to display errors from all non-system headers. 
Use -system-headers to display errors from system headers as well.
    3 warnings generated.
    Suppressed 3 warnings (3 in non-user code).
    Use -header-filter=.* to display errors from all non-system headers. 
Use -system-headers to display errors from system headers as well.
    6 warnings generated.
>> drivers/ptp/ptp_ocp.c:830:2: warning: Undefined or garbage value returned to caller [clang-analyzer-core.uninitialized.UndefReturn]
            return err;
            ^
    drivers/ptp/ptp_ocp.c:844:7: note: 'dev' is non-null
            if (!dev) {
                 ^~~
    drivers/ptp/ptp_ocp.c:844:2: note: Taking false branch
            if (!dev) {
            ^
    drivers/ptp/ptp_ocp.c:852:8: note: Calling 'ptp_ocp_devlink_flash'
            err = ptp_ocp_devlink_flash(devlink, dev, params->fw);
                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    drivers/ptp/ptp_ocp.c:803:2: note: 'err' declared without an initial 
value
            int err;
            ^~~~~~~
    drivers/ptp/ptp_ocp.c:810:2: note: Loop condition is false. 
Execution continues on line 830
            while (resid) {
            ^
    drivers/ptp/ptp_ocp.c:830:2: note: Undefined or garbage value 
returned to caller
            return err;
            ^      ~~~
    Suppressed 5 warnings (5 in non-user code).
    Use -header-filter=.* to display errors from all non-system headers. 
Use -system-headers to display errors from system headers as well.
    6 warnings generated.
    include/linux/list.h:135:13: warning: Use of memory after it is 
freed [clang-analyzer-unix.Malloc]
            __list_del(entry->prev, entry->next);
                       ^
    drivers/media/cec/core/cec-api.c:637:29: note: Left side of '&&' is 
false
            struct cec_adapter *adap = to_cec_adapter(devnode);
                                       ^
    drivers/media/cec/core/cec-priv.h:21:30: note: expanded from macro 
'to_cec_adapter'
    #define to_cec_adapter(node) container_of(node, struct cec_adapter, 
devnode)
                                 ^
    include/linux/kernel.h:495:61: note: expanded from macro 'container_of'
            BUILD_BUG_ON_MSG(!__same_type(*(ptr), ((type *)0)->member) 
&&   \
                                                                       ^
    drivers/media/cec/core/cec-api.c:637:29: note: Taking false branch
            struct cec_adapter *adap = to_cec_adapter(devnode);
                                       ^
    drivers/media/cec/core/cec-priv.h:21:30: note: expanded from macro 
'to_cec_adapter'
    #define to_cec_adapter(node) container_of(node, struct cec_adapter, 
devnode)
                                 ^
    include/linux/kernel.h:495:2: note: expanded from macro 'container_of'
            BUILD_BUG_ON_MSG(!__same_type(*(ptr), ((type *)0)->member) 
&&   \
            ^
    include/linux/build_bug.h:39:37: note: expanded from macro 
'BUILD_BUG_ON_MSG'
    #define BUILD_BUG_ON_MSG(cond, msg) compiletime_assert(!(cond), msg)
                                        ^
    include/linux/compiler_types.h:328:2: note: expanded from macro 
'compiletime_assert'
            _compiletime_assert(condition, msg, __compiletime_assert_, 
__COUNTER__)
            ^
    include/linux/compiler_types.h:316:2: note: expanded from macro 
'_compiletime_assert'
            __compiletime_assert(condition, msg, prefix, suffix)
            ^
    include/linux/compiler_types.h:308:3: note: expanded from macro 
'__compiletime_assert'
                    if (!(condition)) 
     \
                    ^
    drivers/media/cec/core/cec-api.c:637:29: note: Loop condition is 
false.  Exiting loop
            struct cec_adapter *adap = to_cec_adapter(devnode);
                                       ^
    drivers/media/cec/core/cec-priv.h:21:30: note: expanded from macro 
'to_cec_adapter'
    #define to_cec_adapter(node) container_of(node, struct cec_adapter, 
devnode)
                                 ^
    include/linux/kernel.h:495:2: note: expanded from macro 'container_of'
            BUILD_BUG_ON_MSG(!__same_type(*(ptr), ((type *)0)->member) 
&&   \
            ^
    include/linux/build_bug.h:39:37: note: expanded from macro 
'BUILD_BUG_ON_MSG'
    #define BUILD_BUG_ON_MSG(cond, msg) compiletime_assert(!(cond), msg)
                                        ^
    include/linux/compiler_types.h:328:2: note: expanded from macro 
'compiletime_assert'
            _compiletime_assert(condition, msg, __compiletime_assert_, 
__COUNTER__)
            ^
    include/linux/compiler_types.h:316:2: note: expanded from macro 
'_compiletime_assert'
            __compiletime_assert(condition, msg, prefix, suffix)
            ^
    include/linux/compiler_types.h:306:2: note: expanded from macro 
'__compiletime_assert'
            do { 
     \
            ^
    drivers/media/cec/core/cec-api.c:642:6: note: Assuming 'fh' is equal 
to field 'cec_initiator'
            if (adap->cec_initiator == fh)
                ^~~~~~~~~~~~~~~~~~~~~~~~~
    drivers/media/cec/core/cec-api.c:642:2: note: Taking true branch
            if (adap->cec_initiator == fh)
            ^
    drivers/media/cec/core/cec-api.c:644:6: note: Assuming 'fh' is equal 
to field 'cec_follower'
            if (adap->cec_follower == fh) {
                ^~~~~~~~~~~~~~~~~~~~~~~~
    drivers/media/cec/core/cec-api.c:644:2: note: Taking true branch
            if (adap->cec_follower == fh) {
            ^
    drivers/media/cec/core/cec-api.c:648:6: note: Assuming the condition 
is false
            if (fh->mode_follower == CEC_MODE_FOLLOWER)
                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    drivers/media/cec/core/cec-api.c:648:2: note: Taking false branch
            if (fh->mode_follower == CEC_MODE_FOLLOWER)
            ^
    drivers/media/cec/core/cec-api.c:650:6: note: Assuming the condition 
is true
            if (fh->mode_follower == CEC_MODE_MONITOR_PIN)
                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    drivers/media/cec/core/cec-api.c:650:2: note: Taking true branch
            if (fh->mode_follower == CEC_MODE_MONITOR_PIN)

vim +830 drivers/ptp/ptp_ocp.c

773bda96492153 Jonathan Lemon 2021-08-03  793  773bda96492153 Jonathan 
Lemon 2021-08-03  794  static int
773bda96492153 Jonathan Lemon 2021-08-03  795 
ptp_ocp_devlink_flash(struct devlink *devlink, struct device *dev,
773bda96492153 Jonathan Lemon 2021-08-03  796  		      const struct 
firmware *fw)
773bda96492153 Jonathan Lemon 2021-08-03  797  {
773bda96492153 Jonathan Lemon 2021-08-03  798  	struct mtd_info *mtd = 
dev_get_drvdata(dev);
773bda96492153 Jonathan Lemon 2021-08-03  799  	struct ptp_ocp *bp = 
devlink_priv(devlink);
773bda96492153 Jonathan Lemon 2021-08-03  800  	size_t off, len, resid, 
wrote;
773bda96492153 Jonathan Lemon 2021-08-03  801  	struct erase_info erase;
773bda96492153 Jonathan Lemon 2021-08-03  802  	size_t base, blksz;
773bda96492153 Jonathan Lemon 2021-08-03  803  	int err;
773bda96492153 Jonathan Lemon 2021-08-03  804  773bda96492153 Jonathan 
Lemon 2021-08-03  805  	off = 0;
773bda96492153 Jonathan Lemon 2021-08-03  806  	base = bp->flash_start;
773bda96492153 Jonathan Lemon 2021-08-03  807  	blksz = 4096;
773bda96492153 Jonathan Lemon 2021-08-03  808  	resid = fw->size;
773bda96492153 Jonathan Lemon 2021-08-03  809  773bda96492153 Jonathan 
Lemon 2021-08-03  810  	while (resid) {
773bda96492153 Jonathan Lemon 2021-08-03  811  	 
devlink_flash_update_status_notify(devlink, "Flashing",
773bda96492153 Jonathan Lemon 2021-08-03  812  						   NULL, off, 
fw->size);
773bda96492153 Jonathan Lemon 2021-08-03  813  773bda96492153 Jonathan 
Lemon 2021-08-03  814  		len = min_t(size_t, resid, blksz);
773bda96492153 Jonathan Lemon 2021-08-03  815  		erase.addr = base + off;
773bda96492153 Jonathan Lemon 2021-08-03  816  		erase.len = blksz;
a7e1abad13f3f0 Jonathan Lemon 2020-12-03  817  773bda96492153 Jonathan 
Lemon 2021-08-03  818  		err = mtd_erase(mtd, &erase);
a7e1abad13f3f0 Jonathan Lemon 2020-12-03  819  		if (err)
a7e1abad13f3f0 Jonathan Lemon 2020-12-03  820  			goto out;
a7e1abad13f3f0 Jonathan Lemon 2020-12-03  821  773bda96492153 Jonathan 
Lemon 2021-08-03  822  		err = mtd_write(mtd, base + off, len, &wrote, 
&fw->data[off]);
773bda96492153 Jonathan Lemon 2021-08-03  823  		if (err)
a7e1abad13f3f0 Jonathan Lemon 2020-12-03  824  			goto out;
773bda96492153 Jonathan Lemon 2021-08-03  825  773bda96492153 Jonathan 
Lemon 2021-08-03  826  		off += blksz;
773bda96492153 Jonathan Lemon 2021-08-03  827  		resid -= len;
773bda96492153 Jonathan Lemon 2021-08-03  828  	}
773bda96492153 Jonathan Lemon 2021-08-03  829  out:
773bda96492153 Jonathan Lemon 2021-08-03 @830  	return err;
a7e1abad13f3f0 Jonathan Lemon 2020-12-03  831  }
a7e1abad13f3f0 Jonathan Lemon 2020-12-03  832
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org


[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 36847 bytes --]

[-- Attachment #3: Attached Message Part --]
[-- Type: text/plain, Size: 150 bytes --]

_______________________________________________
kbuild mailing list -- kbuild@lists.01.org
To unsubscribe send an email to kbuild-leave@lists.01.org


^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2021-08-09 10:46 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <202108081526.htnwreV6-lkp@intel.com>
2021-08-09 10:46 ` [linux-next:master 4907/6030] drivers/ptp/ptp_ocp.c:830:2: warning: Undefined or garbage value returned to caller [clang-analyzer-core.uninitialized.UndefReturn] kernel test robot

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox