lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [day] [month] [year] [list]
Date:   Sun, 5 Jul 2020 11:13:59 +0800
From:   kernel test robot <lkp@...el.com>
To:     Luc Van Oostenryck <luc.vanoostenryck@...il.com>
Cc:     kbuild-all@...ts.01.org, linux-kernel@...r.kernel.org
Subject: drivers/soc/qcom/smd-rpm.c:177:47: sparse: sparse: incorrect type in
 argument 2 (different address spaces)

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   35e884f89df4c48566d745dc5a97a0d058d04263
commit: 670d0a4b10704667765f7d18f7592993d02783aa sparse: use identifiers to define address spaces
date:   2 weeks ago
config: ia64-randconfig-s031-20200705 (attached as .config)
compiler: ia64-linux-gcc (GCC) 9.3.0
reproduce:
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # apt-get install sparse
        # sparse version: v0.6.2-3-gfa153962-dirty
        git checkout 670d0a4b10704667765f7d18f7592993d02783aa
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' ARCH=ia64 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@...el.com>


sparse warnings: (new ones prefixed by >>)

>> drivers/soc/qcom/smd-rpm.c:177:47: sparse: sparse: incorrect type in argument 2 (different address spaces) @@     expected void const volatile [noderef] __iomem *src @@     got unsigned char const * @@
>> drivers/soc/qcom/smd-rpm.c:177:47: sparse:     expected void const volatile [noderef] __iomem *src
   drivers/soc/qcom/smd-rpm.c:177:47: sparse:     got unsigned char const *

vim +177 drivers/soc/qcom/smd-rpm.c

936f14cf4e6716 Bjorn Andersson 2015-07-27  146  
5052de8deff561 Bjorn Andersson 2017-03-27  147  static int qcom_smd_rpm_callback(struct rpmsg_device *rpdev,
5052de8deff561 Bjorn Andersson 2017-03-27  148  				 void *data,
5052de8deff561 Bjorn Andersson 2017-03-27  149  				 int count,
5052de8deff561 Bjorn Andersson 2017-03-27  150  				 void *priv,
5052de8deff561 Bjorn Andersson 2017-03-27  151  				 u32 addr)
936f14cf4e6716 Bjorn Andersson 2015-07-27  152  {
936f14cf4e6716 Bjorn Andersson 2015-07-27  153  	const struct qcom_rpm_header *hdr = data;
30b7ea5eda0afb Stephen Boyd    2015-09-02  154  	size_t hdr_length = le32_to_cpu(hdr->length);
936f14cf4e6716 Bjorn Andersson 2015-07-27  155  	const struct qcom_rpm_message *msg;
5052de8deff561 Bjorn Andersson 2017-03-27  156  	struct qcom_smd_rpm *rpm = dev_get_drvdata(&rpdev->dev);
936f14cf4e6716 Bjorn Andersson 2015-07-27  157  	const u8 *buf = data + sizeof(struct qcom_rpm_header);
30b7ea5eda0afb Stephen Boyd    2015-09-02  158  	const u8 *end = buf + hdr_length;
936f14cf4e6716 Bjorn Andersson 2015-07-27  159  	char msgbuf[32];
936f14cf4e6716 Bjorn Andersson 2015-07-27  160  	int status = 0;
30b7ea5eda0afb Stephen Boyd    2015-09-02  161  	u32 len, msg_length;
936f14cf4e6716 Bjorn Andersson 2015-07-27  162  
30b7ea5eda0afb Stephen Boyd    2015-09-02  163  	if (le32_to_cpu(hdr->service_type) != RPM_SERVICE_TYPE_REQUEST ||
30b7ea5eda0afb Stephen Boyd    2015-09-02  164  	    hdr_length < sizeof(struct qcom_rpm_message)) {
b853cb9628bfbc Bjorn Andersson 2016-03-28  165  		dev_err(rpm->dev, "invalid request\n");
936f14cf4e6716 Bjorn Andersson 2015-07-27  166  		return 0;
936f14cf4e6716 Bjorn Andersson 2015-07-27  167  	}
936f14cf4e6716 Bjorn Andersson 2015-07-27  168  
936f14cf4e6716 Bjorn Andersson 2015-07-27  169  	while (buf < end) {
936f14cf4e6716 Bjorn Andersson 2015-07-27  170  		msg = (struct qcom_rpm_message *)buf;
30b7ea5eda0afb Stephen Boyd    2015-09-02  171  		msg_length = le32_to_cpu(msg->length);
30b7ea5eda0afb Stephen Boyd    2015-09-02  172  		switch (le32_to_cpu(msg->msg_type)) {
936f14cf4e6716 Bjorn Andersson 2015-07-27  173  		case RPM_MSG_TYPE_MSG_ID:
936f14cf4e6716 Bjorn Andersson 2015-07-27  174  			break;
936f14cf4e6716 Bjorn Andersson 2015-07-27  175  		case RPM_MSG_TYPE_ERR:
30b7ea5eda0afb Stephen Boyd    2015-09-02  176  			len = min_t(u32, ALIGN(msg_length, 4), sizeof(msgbuf));
936f14cf4e6716 Bjorn Andersson 2015-07-27 @177  			memcpy_fromio(msgbuf, msg->message, len);
936f14cf4e6716 Bjorn Andersson 2015-07-27  178  			msgbuf[len - 1] = 0;
936f14cf4e6716 Bjorn Andersson 2015-07-27  179  
936f14cf4e6716 Bjorn Andersson 2015-07-27  180  			if (!strcmp(msgbuf, "resource does not exist"))
936f14cf4e6716 Bjorn Andersson 2015-07-27  181  				status = -ENXIO;
936f14cf4e6716 Bjorn Andersson 2015-07-27  182  			else
936f14cf4e6716 Bjorn Andersson 2015-07-27  183  				status = -EINVAL;
936f14cf4e6716 Bjorn Andersson 2015-07-27  184  			break;
936f14cf4e6716 Bjorn Andersson 2015-07-27  185  		}
936f14cf4e6716 Bjorn Andersson 2015-07-27  186  
30b7ea5eda0afb Stephen Boyd    2015-09-02  187  		buf = PTR_ALIGN(buf + 2 * sizeof(u32) + msg_length, 4);
936f14cf4e6716 Bjorn Andersson 2015-07-27  188  	}
936f14cf4e6716 Bjorn Andersson 2015-07-27  189  
936f14cf4e6716 Bjorn Andersson 2015-07-27  190  	rpm->ack_status = status;
936f14cf4e6716 Bjorn Andersson 2015-07-27  191  	complete(&rpm->ack);
936f14cf4e6716 Bjorn Andersson 2015-07-27  192  	return 0;
936f14cf4e6716 Bjorn Andersson 2015-07-27  193  }
936f14cf4e6716 Bjorn Andersson 2015-07-27  194  

:::::: The code at line 177 was first introduced by commit
:::::: 936f14cf4e67168fcd37f10cebf5a475f490fb6e soc: qcom: Driver for the Qualcomm RPM over SMD

:::::: TO: Bjorn Andersson <bjorn.andersson@...ymobile.com>
:::::: CC: Andy Gross <agross@...eaurora.org>

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

Download attachment ".config.gz" of type "application/gzip" (31283 bytes)

Powered by blists - more mailing lists