drivers/scsi/hisi_sas/hisi_sas_v3_hw.c:2174:43: sparse: sparse: restricted __le32 degrades to integer

From: kernel test robot
Date: Sat Aug 20 2022 - 20:42:32 EST

Next message: Joe Perches: "Re: [PATCH] checkpatch: add ALLOC_UNNECESSARY_ARRAY test"
Previous message: Al Viro: "Re: [RFC][PATCHES] termios.h cleanups"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

Hi Huacai,

First bad commit (maybe != root cause):

tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 15b3f48a4339e3c16acf18624e2b7f60bc5e9a2c
commit: 57fc7323a8e7c2e7c1d5795ab63cb3ffea3cfdfb LoongArch: Add PCI controller support
date: 9 days ago
config: loongarch-randconfig-s041-20220821 (https://download.01.org/0day-ci/archive/20220821/202208210816.46BZqtAB-lkp@xxxxxxxxx/config)
compiler: loongarch64-linux-gcc (GCC) 12.1.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.4-39-gce1a6720-dirty
# https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=57fc7323a8e7c2e7c1d5795ab63cb3ffea3cfdfb
git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus master
git checkout 57fc7323a8e7c2e7c1d5795ab63cb3ffea3cfdfb
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' O=build_dir ARCH=loongarch SHELL=/bin/bash drivers/scsi/hisi_sas/ net/core/

If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <lkp@xxxxxxxxx>

sparse warnings: (new ones prefixed by >>)
>> drivers/scsi/hisi_sas/hisi_sas_v3_hw.c:2174:43: sparse: sparse: restricted __le32 degrades to integer
drivers/scsi/hisi_sas/hisi_sas_v3_hw.c:2175:46: sparse: sparse: restricted __le32 degrades to integer
>> drivers/scsi/hisi_sas/hisi_sas_v3_hw.c:4440:35: sparse: sparse: incorrect type in argument 4 (different base types) @@ expected restricted __le32 [usertype] *[assigned] ptr @@ got unsigned int * @@
drivers/scsi/hisi_sas/hisi_sas_v3_hw.c:4440:35: sparse: expected restricted __le32 [usertype] *[assigned] ptr
drivers/scsi/hisi_sas/hisi_sas_v3_hw.c:4440:35: sparse: got unsigned int *

vim +2174 drivers/scsi/hisi_sas/hisi_sas_v3_hw.c

fa2314081571f5 Xiaofei Tan 2017-10-24 2150
62413199cd6d29 Xingui Yang 2022-02-24 2151 static bool
60b4a5ee90349a Xiang Chen 2017-06-14 2152 slot_err_v3_hw(struct hisi_hba *hisi_hba, struct sas_task *task,
60b4a5ee90349a Xiang Chen 2017-06-14 2153 struct hisi_sas_slot *slot)
60b4a5ee90349a Xiang Chen 2017-06-14 2154 {
60b4a5ee90349a Xiang Chen 2017-06-14 2155 struct task_status_struct *ts = &task->task_status;
60b4a5ee90349a Xiang Chen 2017-06-14 2156 struct hisi_sas_complete_v3_hdr *complete_queue =
60b4a5ee90349a Xiang Chen 2017-06-14 2157 hisi_hba->complete_hdr[slot->cmplt_queue];
60b4a5ee90349a Xiang Chen 2017-06-14 2158 struct hisi_sas_complete_v3_hdr *complete_hdr =
60b4a5ee90349a Xiang Chen 2017-06-14 2159 &complete_queue[slot->cmplt_queue_slot];
f557e32c0023ea Xiaofei Tan 2017-06-29 2160 struct hisi_sas_err_record_v3 *record =
f557e32c0023ea Xiaofei Tan 2017-06-29 2161 hisi_sas_status_buf_addr_mem(slot);
735bcc77e6ba83 John Garry 2018-12-06 2162 u32 dma_rx_err_type = le32_to_cpu(record->dma_rx_err_type);
735bcc77e6ba83 John Garry 2018-12-06 2163 u32 trans_tx_fail_type = le32_to_cpu(record->trans_tx_fail_type);
735bcc77e6ba83 John Garry 2018-12-06 2164 u32 dw3 = le32_to_cpu(complete_hdr->dw3);
60b4a5ee90349a Xiang Chen 2017-06-14 2165
60b4a5ee90349a Xiang Chen 2017-06-14 2166 switch (task->task_proto) {
60b4a5ee90349a Xiang Chen 2017-06-14 2167 case SAS_PROTOCOL_SSP:
60b4a5ee90349a Xiang Chen 2017-06-14 2168 if (dma_rx_err_type & RX_DATA_LEN_UNDERFLOW_MSK) {
62413199cd6d29 Xingui Yang 2022-02-24 2169 /*
62413199cd6d29 Xingui Yang 2022-02-24 2170 * If returned response frame is incorrect because of data underflow,
62413199cd6d29 Xingui Yang 2022-02-24 2171 * but I/O information has been written to the host memory, we examine
62413199cd6d29 Xingui Yang 2022-02-24 2172 * response IU.
62413199cd6d29 Xingui Yang 2022-02-24 2173 */
62413199cd6d29 Xingui Yang 2022-02-24 @2174 if (!(complete_hdr->dw0 & CMPLT_HDR_RSPNS_GOOD_MSK) &&
62413199cd6d29 Xingui Yang 2022-02-24 2175 (complete_hdr->dw0 & CMPLT_HDR_RSPNS_XFRD_MSK))
62413199cd6d29 Xingui Yang 2022-02-24 2176 return false;
62413199cd6d29 Xingui Yang 2022-02-24 2177
60b4a5ee90349a Xiang Chen 2017-06-14 2178 ts->residual = trans_tx_fail_type;
60b4a5ee90349a Xiang Chen 2017-06-14 2179 ts->stat = SAS_DATA_UNDERRUN;
735bcc77e6ba83 John Garry 2018-12-06 2180 } else if (dw3 & CMPLT_HDR_IO_IN_TARGET_MSK) {
60b4a5ee90349a Xiang Chen 2017-06-14 2181 ts->stat = SAS_QUEUE_FULL;
60b4a5ee90349a Xiang Chen 2017-06-14 2182 slot->abort = 1;
60b4a5ee90349a Xiang Chen 2017-06-14 2183 } else {
60b4a5ee90349a Xiang Chen 2017-06-14 2184 ts->stat = SAS_OPEN_REJECT;
60b4a5ee90349a Xiang Chen 2017-06-14 2185 ts->open_rej_reason = SAS_OREJ_RSVD_RETRY;
60b4a5ee90349a Xiang Chen 2017-06-14 2186 }
60b4a5ee90349a Xiang Chen 2017-06-14 2187 break;
60b4a5ee90349a Xiang Chen 2017-06-14 2188 case SAS_PROTOCOL_SATA:
60b4a5ee90349a Xiang Chen 2017-06-14 2189 case SAS_PROTOCOL_STP:
60b4a5ee90349a Xiang Chen 2017-06-14 2190 case SAS_PROTOCOL_SATA | SAS_PROTOCOL_STP:
60b4a5ee90349a Xiang Chen 2017-06-14 2191 if (dma_rx_err_type & RX_DATA_LEN_UNDERFLOW_MSK) {
60b4a5ee90349a Xiang Chen 2017-06-14 2192 ts->residual = trans_tx_fail_type;
60b4a5ee90349a Xiang Chen 2017-06-14 2193 ts->stat = SAS_DATA_UNDERRUN;
735bcc77e6ba83 John Garry 2018-12-06 2194 } else if (dw3 & CMPLT_HDR_IO_IN_TARGET_MSK) {
60b4a5ee90349a Xiang Chen 2017-06-14 2195 ts->stat = SAS_PHY_DOWN;
60b4a5ee90349a Xiang Chen 2017-06-14 2196 slot->abort = 1;
60b4a5ee90349a Xiang Chen 2017-06-14 2197 } else {
60b4a5ee90349a Xiang Chen 2017-06-14 2198 ts->stat = SAS_OPEN_REJECT;
60b4a5ee90349a Xiang Chen 2017-06-14 2199 ts->open_rej_reason = SAS_OREJ_RSVD_RETRY;
60b4a5ee90349a Xiang Chen 2017-06-14 2200 }
60b4a5ee90349a Xiang Chen 2017-06-14 2201 hisi_sas_sata_done(task, slot);
60b4a5ee90349a Xiang Chen 2017-06-14 2202 break;
60b4a5ee90349a Xiang Chen 2017-06-14 2203 case SAS_PROTOCOL_SMP:
d377f415dddc18 Bart Van Assche 2021-05-23 2204 ts->stat = SAS_SAM_STAT_CHECK_CONDITION;
60b4a5ee90349a Xiang Chen 2017-06-14 2205 break;
60b4a5ee90349a Xiang Chen 2017-06-14 2206 default:
60b4a5ee90349a Xiang Chen 2017-06-14 2207 break;
60b4a5ee90349a Xiang Chen 2017-06-14 2208 }
62413199cd6d29 Xingui Yang 2022-02-24 2209 return true;
60b4a5ee90349a Xiang Chen 2017-06-14 2210 }
60b4a5ee90349a Xiang Chen 2017-06-14 2211

:::::: The code at line 2174 was first introduced by commit
:::::: 62413199cd6d2906c121c2dfa3d7b82fd05f08db scsi: hisi_sas: Modify v3 HW SSP underflow error processing

:::::: TO: Xingui Yang <yangxingui@xxxxxxxxxx>
:::::: CC: Martin K. Petersen <martin.petersen@xxxxxxxxxx>

--
0-DAY CI Kernel Test Service
https://01.org/lkp

Next message: Joe Perches: "Re: [PATCH] checkpatch: add ALLOC_UNNECESSARY_ARRAY test"
Previous message: Al Viro: "Re: [RFC][PATCHES] termios.h cleanups"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]