From: kernel test robot <lkp@intel.com>
To: Bart Van Assche <bvanassche@acm.org>,
"Martin K . Petersen" <martin.petersen@oracle.com>
Cc: llvm@lists.linux.dev, oe-kbuild-all@lists.linux.dev,
Jaegeuk Kim <jaegeuk@kernel.org>,
linux-scsi@vger.kernel.org, Avri Altman <avri.altman@wdc.com>,
Adrian Hunter <adrian.hunter@intel.com>,
Bart Van Assche <bvanassche@acm.org>,
Bean Huo <beanhuo@micron.com>,
ChanWoo Lee <cw9316.lee@samsung.com>,
Daejun Park <daejun7.park@samsung.com>,
Keoseong Park <keosung.park@samsung.com>,
"James E.J. Bottomley" <jejb@linux.ibm.com>,
Lu Hongfei <luhongfei@vivo.com>,
Daniil Lunev <dlunev@chromium.org>,
Jinyoung Choi <j-young.choi@samsung.com>,
Manivannan Sadhasivam <mani@kernel.org>,
Asutosh Das <quic_asutoshd@quicinc.com>,
Can Guo <quic_cang@quicinc.com>,
Peter Wang <peter.wang@mediatek.com>,
Stanley Chu <stanley.chu@mediatek.com>,
"Bao D. Nguyen" <quic_nguyenb@quicinc.com>,
Arthur Simchaev <arthur.simchaev@wdc.com>,
Jens Axboe <axboe@kernel.dk>, Markus Fuchs <mklntf@gmail.com>,
Po-Wen Kao <powen.kao@mediatek.com>,
Eric Biggers <ebiggers@google.com>
Subject: Re: [PATCH] scsi: ufs: Remove HPB support
Date: Wed, 19 Jul 2023 10:46:38 +0800 [thread overview]
Message-ID: <202307191053.lTuPwUD6-lkp@intel.com> (raw)
In-Reply-To: <20230717193827.2001174-1-bvanassche@acm.org>
Hi Bart,
kernel test robot noticed the following build warnings:
[auto build test WARNING on mkp-scsi/for-next]
[also build test WARNING on next-20230718]
[cannot apply to jejb-scsi/for-next linus/master v6.5-rc2]
[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#_base_tree_information]
url: https://github.com/intel-lab-lkp/linux/commits/Bart-Van-Assche/scsi-ufs-Remove-HPB-support/20230718-195403
base: https://git.kernel.org/pub/scm/linux/kernel/git/mkp/scsi.git for-next
patch link: https://lore.kernel.org/r/20230717193827.2001174-1-bvanassche%40acm.org
patch subject: [PATCH] scsi: ufs: Remove HPB support
config: hexagon-randconfig-r045-20230718 (https://download.01.org/0day-ci/archive/20230719/202307191053.lTuPwUD6-lkp@intel.com/config)
compiler: clang version 17.0.0 (https://github.com/llvm/llvm-project.git 4a5ac14ee968ff0ad5d2cc1ffa0299048db4c88a)
reproduce: (https://download.01.org/0day-ci/archive/20230719/202307191053.lTuPwUD6-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202307191053.lTuPwUD6-lkp@intel.com/
All warnings (new ones prefixed by >>):
In file included from drivers/ufs/core/ufshcd.c:18:
In file included from include/linux/blkdev.h:9:
In file included from include/linux/blk_types.h:10:
In file included from include/linux/bvec.h:10:
In file included from include/linux/highmem.h:12:
In file included from include/linux/hardirq.h:11:
In file included from ./arch/hexagon/include/generated/asm/hardirq.h:1:
In file included from include/asm-generic/hardirq.h:17:
In file included from include/linux/irq.h:20:
In file included from include/linux/io.h:13:
In file included from arch/hexagon/include/asm/io.h:334:
include/asm-generic/io.h:547:31: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
547 | val = __raw_readb(PCI_IOBASE + addr);
| ~~~~~~~~~~ ^
include/asm-generic/io.h:560:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
560 | val = __le16_to_cpu((__le16 __force)__raw_readw(PCI_IOBASE + addr));
| ~~~~~~~~~~ ^
include/uapi/linux/byteorder/little_endian.h:37:51: note: expanded from macro '__le16_to_cpu'
37 | #define __le16_to_cpu(x) ((__force __u16)(__le16)(x))
| ^
In file included from drivers/ufs/core/ufshcd.c:18:
In file included from include/linux/blkdev.h:9:
In file included from include/linux/blk_types.h:10:
In file included from include/linux/bvec.h:10:
In file included from include/linux/highmem.h:12:
In file included from include/linux/hardirq.h:11:
In file included from ./arch/hexagon/include/generated/asm/hardirq.h:1:
In file included from include/asm-generic/hardirq.h:17:
In file included from include/linux/irq.h:20:
In file included from include/linux/io.h:13:
In file included from arch/hexagon/include/asm/io.h:334:
include/asm-generic/io.h:573:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
573 | val = __le32_to_cpu((__le32 __force)__raw_readl(PCI_IOBASE + addr));
| ~~~~~~~~~~ ^
include/uapi/linux/byteorder/little_endian.h:35:51: note: expanded from macro '__le32_to_cpu'
35 | #define __le32_to_cpu(x) ((__force __u32)(__le32)(x))
| ^
In file included from drivers/ufs/core/ufshcd.c:18:
In file included from include/linux/blkdev.h:9:
In file included from include/linux/blk_types.h:10:
In file included from include/linux/bvec.h:10:
In file included from include/linux/highmem.h:12:
In file included from include/linux/hardirq.h:11:
In file included from ./arch/hexagon/include/generated/asm/hardirq.h:1:
In file included from include/asm-generic/hardirq.h:17:
In file included from include/linux/irq.h:20:
In file included from include/linux/io.h:13:
In file included from arch/hexagon/include/asm/io.h:334:
include/asm-generic/io.h:584:33: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
584 | __raw_writeb(value, PCI_IOBASE + addr);
| ~~~~~~~~~~ ^
include/asm-generic/io.h:594:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
594 | __raw_writew((u16 __force)cpu_to_le16(value), PCI_IOBASE + addr);
| ~~~~~~~~~~ ^
include/asm-generic/io.h:604:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
604 | __raw_writel((u32 __force)cpu_to_le32(value), PCI_IOBASE + addr);
| ~~~~~~~~~~ ^
>> drivers/ufs/core/ufshcd.c:8092:5: warning: variable 'b_ufs_feature_sup' set but not used [-Wunused-but-set-variable]
8092 | u8 b_ufs_feature_sup;
| ^
7 warnings generated.
vim +/b_ufs_feature_sup +8092 drivers/ufs/core/ufshcd.c
c28c00ba4f0609 drivers/scsi/ufs/ufshcd.c Stanley Chu 2020-05-08 8087
097500666ec991 drivers/scsi/ufs/ufshcd.c Bean Huo 2020-01-20 8088 static int ufs_get_device_desc(struct ufs_hba *hba)
c58ab7aab71e2c drivers/scsi/ufs/ufshcd.c Yaniv Gardi 2016-03-10 8089 {
c58ab7aab71e2c drivers/scsi/ufs/ufshcd.c Yaniv Gardi 2016-03-10 8090 int err;
c58ab7aab71e2c drivers/scsi/ufs/ufshcd.c Yaniv Gardi 2016-03-10 8091 u8 model_index;
f02bc9754a6887 drivers/scsi/ufs/ufshcd.c Daejun Park 2021-07-12 @8092 u8 b_ufs_feature_sup;
bbe21d7a979245 drivers/scsi/ufs/ufshcd.c Kees Cook 2018-05-02 8093 u8 *desc_buf;
097500666ec991 drivers/scsi/ufs/ufshcd.c Bean Huo 2020-01-20 8094 struct ufs_dev_info *dev_info = &hba->dev_info;
4b828fe156a662 drivers/scsi/ufs/ufshcd.c Tomas Winkler 2019-07-30 8095
f2a89b071b26b7 drivers/ufs/core/ufshcd.c Arthur Simchaev 2022-12-11 8096 desc_buf = kzalloc(QUERY_DESC_MAX_SIZE, GFP_KERNEL);
bbe21d7a979245 drivers/scsi/ufs/ufshcd.c Kees Cook 2018-05-02 8097 if (!desc_buf) {
bbe21d7a979245 drivers/scsi/ufs/ufshcd.c Kees Cook 2018-05-02 8098 err = -ENOMEM;
bbe21d7a979245 drivers/scsi/ufs/ufshcd.c Kees Cook 2018-05-02 8099 goto out;
bbe21d7a979245 drivers/scsi/ufs/ufshcd.c Kees Cook 2018-05-02 8100 }
c58ab7aab71e2c drivers/scsi/ufs/ufshcd.c Yaniv Gardi 2016-03-10 8101
c4607a09450d67 drivers/scsi/ufs/ufshcd.c Bean Huo 2020-06-03 8102 err = ufshcd_read_desc_param(hba, QUERY_DESC_IDN_DEVICE, 0, 0, desc_buf,
f2a89b071b26b7 drivers/ufs/core/ufshcd.c Arthur Simchaev 2022-12-11 8103 QUERY_DESC_MAX_SIZE);
c58ab7aab71e2c drivers/scsi/ufs/ufshcd.c Yaniv Gardi 2016-03-10 8104 if (err) {
c58ab7aab71e2c drivers/scsi/ufs/ufshcd.c Yaniv Gardi 2016-03-10 8105 dev_err(hba->dev, "%s: Failed reading Device Desc. err = %d\n",
c58ab7aab71e2c drivers/scsi/ufs/ufshcd.c Yaniv Gardi 2016-03-10 8106 __func__, err);
c58ab7aab71e2c drivers/scsi/ufs/ufshcd.c Yaniv Gardi 2016-03-10 8107 goto out;
c58ab7aab71e2c drivers/scsi/ufs/ufshcd.c Yaniv Gardi 2016-03-10 8108 }
c58ab7aab71e2c drivers/scsi/ufs/ufshcd.c Yaniv Gardi 2016-03-10 8109
c58ab7aab71e2c drivers/scsi/ufs/ufshcd.c Yaniv Gardi 2016-03-10 8110 /*
c58ab7aab71e2c drivers/scsi/ufs/ufshcd.c Yaniv Gardi 2016-03-10 8111 * getting vendor (manufacturerID) and Bank Index in big endian
c58ab7aab71e2c drivers/scsi/ufs/ufshcd.c Yaniv Gardi 2016-03-10 8112 * format
c58ab7aab71e2c drivers/scsi/ufs/ufshcd.c Yaniv Gardi 2016-03-10 8113 */
097500666ec991 drivers/scsi/ufs/ufshcd.c Bean Huo 2020-01-20 8114 dev_info->wmanufacturerid = desc_buf[DEVICE_DESC_PARAM_MANF_ID] << 8 |
c58ab7aab71e2c drivers/scsi/ufs/ufshcd.c Yaniv Gardi 2016-03-10 8115 desc_buf[DEVICE_DESC_PARAM_MANF_ID + 1];
c58ab7aab71e2c drivers/scsi/ufs/ufshcd.c Yaniv Gardi 2016-03-10 8116
09f17791e640dc drivers/scsi/ufs/ufshcd.c Can Guo 2020-02-10 8117 /* getting Specification Version in big endian format */
09f17791e640dc drivers/scsi/ufs/ufshcd.c Can Guo 2020-02-10 8118 dev_info->wspecversion = desc_buf[DEVICE_DESC_PARAM_SPEC_VER] << 8 |
09f17791e640dc drivers/scsi/ufs/ufshcd.c Can Guo 2020-02-10 8119 desc_buf[DEVICE_DESC_PARAM_SPEC_VER + 1];
7224c806876e46 drivers/ufs/core/ufshcd.c Asutosh Das 2023-01-13 8120 dev_info->bqueuedepth = desc_buf[DEVICE_DESC_PARAM_Q_DPTH];
f02bc9754a6887 drivers/scsi/ufs/ufshcd.c Daejun Park 2021-07-12 8121 b_ufs_feature_sup = desc_buf[DEVICE_DESC_PARAM_UFS_FEAT];
09f17791e640dc drivers/scsi/ufs/ufshcd.c Can Guo 2020-02-10 8122
c58ab7aab71e2c drivers/scsi/ufs/ufshcd.c Yaniv Gardi 2016-03-10 8123 model_index = desc_buf[DEVICE_DESC_PARAM_PRDCT_NAME];
3d17b9b5ab1155 drivers/scsi/ufs/ufshcd.c Asutosh Das 2020-04-22 8124
4b828fe156a662 drivers/scsi/ufs/ufshcd.c Tomas Winkler 2019-07-30 8125 err = ufshcd_read_string_desc(hba, model_index,
097500666ec991 drivers/scsi/ufs/ufshcd.c Bean Huo 2020-01-20 8126 &dev_info->model, SD_ASCII_STD);
4b828fe156a662 drivers/scsi/ufs/ufshcd.c Tomas Winkler 2019-07-30 8127 if (err < 0) {
c58ab7aab71e2c drivers/scsi/ufs/ufshcd.c Yaniv Gardi 2016-03-10 8128 dev_err(hba->dev, "%s: Failed reading Product Name. err = %d\n",
c58ab7aab71e2c drivers/scsi/ufs/ufshcd.c Yaniv Gardi 2016-03-10 8129 __func__, err);
c58ab7aab71e2c drivers/scsi/ufs/ufshcd.c Yaniv Gardi 2016-03-10 8130 goto out;
c58ab7aab71e2c drivers/scsi/ufs/ufshcd.c Yaniv Gardi 2016-03-10 8131 }
c58ab7aab71e2c drivers/scsi/ufs/ufshcd.c Yaniv Gardi 2016-03-10 8132
b294ff3e34490f drivers/scsi/ufs/ufshcd.c Asutosh Das 2021-04-23 8133 hba->luns_avail = desc_buf[DEVICE_DESC_PARAM_NUM_LU] +
b294ff3e34490f drivers/scsi/ufs/ufshcd.c Asutosh Das 2021-04-23 8134 desc_buf[DEVICE_DESC_PARAM_NUM_WLU];
b294ff3e34490f drivers/scsi/ufs/ufshcd.c Asutosh Das 2021-04-23 8135
817d7e140283f4 drivers/scsi/ufs/ufshcd.c Stanley Chu 2020-05-08 8136 ufs_fixup_device_setup(hba);
817d7e140283f4 drivers/scsi/ufs/ufshcd.c Stanley Chu 2020-05-08 8137
817d7e140283f4 drivers/scsi/ufs/ufshcd.c Stanley Chu 2020-05-08 8138 ufshcd_wb_probe(hba, desc_buf);
817d7e140283f4 drivers/scsi/ufs/ufshcd.c Stanley Chu 2020-05-08 8139
e88e2d32200a17 drivers/scsi/ufs/ufshcd.c Avri Altman 2021-09-15 8140 ufshcd_temp_notif_probe(hba, desc_buf);
e88e2d32200a17 drivers/scsi/ufs/ufshcd.c Avri Altman 2021-09-15 8141
6e1d850acff947 drivers/ufs/core/ufshcd.c Asutosh Das 2023-01-13 8142 if (hba->ext_iid_sup)
6e1d850acff947 drivers/ufs/core/ufshcd.c Asutosh Das 2023-01-13 8143 ufshcd_ext_iid_probe(hba, desc_buf);
6e1d850acff947 drivers/ufs/core/ufshcd.c Asutosh Das 2023-01-13 8144
4b828fe156a662 drivers/scsi/ufs/ufshcd.c Tomas Winkler 2019-07-30 8145 /*
4b828fe156a662 drivers/scsi/ufs/ufshcd.c Tomas Winkler 2019-07-30 8146 * ufshcd_read_string_desc returns size of the string
4b828fe156a662 drivers/scsi/ufs/ufshcd.c Tomas Winkler 2019-07-30 8147 * reset the error value
4b828fe156a662 drivers/scsi/ufs/ufshcd.c Tomas Winkler 2019-07-30 8148 */
4b828fe156a662 drivers/scsi/ufs/ufshcd.c Tomas Winkler 2019-07-30 8149 err = 0;
c58ab7aab71e2c drivers/scsi/ufs/ufshcd.c Yaniv Gardi 2016-03-10 8150
c58ab7aab71e2c drivers/scsi/ufs/ufshcd.c Yaniv Gardi 2016-03-10 8151 out:
bbe21d7a979245 drivers/scsi/ufs/ufshcd.c Kees Cook 2018-05-02 8152 kfree(desc_buf);
c58ab7aab71e2c drivers/scsi/ufs/ufshcd.c Yaniv Gardi 2016-03-10 8153 return err;
c58ab7aab71e2c drivers/scsi/ufs/ufshcd.c Yaniv Gardi 2016-03-10 8154 }
c58ab7aab71e2c drivers/scsi/ufs/ufshcd.c Yaniv Gardi 2016-03-10 8155
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
prev parent reply other threads:[~2023-07-19 2:47 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-07-17 19:36 [PATCH] scsi: ufs: Remove HPB support Bart Van Assche
2023-07-17 19:50 ` Avri Altman
2023-07-17 21:01 ` Bean Huo
2023-07-18 6:43 ` Adrian Hunter
2023-07-18 13:18 ` Bart Van Assche
2023-07-18 17:52 ` Adrian Hunter
2023-07-18 17:02 ` kernel test robot
2023-07-18 17:41 ` Bart Van Assche
2023-07-19 2:46 ` kernel test robot [this message]
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=202307191053.lTuPwUD6-lkp@intel.com \
--to=lkp@intel.com \
--cc=adrian.hunter@intel.com \
--cc=arthur.simchaev@wdc.com \
--cc=avri.altman@wdc.com \
--cc=axboe@kernel.dk \
--cc=beanhuo@micron.com \
--cc=bvanassche@acm.org \
--cc=cw9316.lee@samsung.com \
--cc=daejun7.park@samsung.com \
--cc=dlunev@chromium.org \
--cc=ebiggers@google.com \
--cc=j-young.choi@samsung.com \
--cc=jaegeuk@kernel.org \
--cc=jejb@linux.ibm.com \
--cc=keosung.park@samsung.com \
--cc=linux-scsi@vger.kernel.org \
--cc=llvm@lists.linux.dev \
--cc=luhongfei@vivo.com \
--cc=mani@kernel.org \
--cc=martin.petersen@oracle.com \
--cc=mklntf@gmail.com \
--cc=oe-kbuild-all@lists.linux.dev \
--cc=peter.wang@mediatek.com \
--cc=powen.kao@mediatek.com \
--cc=quic_asutoshd@quicinc.com \
--cc=quic_cang@quicinc.com \
--cc=quic_nguyenb@quicinc.com \
--cc=stanley.chu@mediatek.com \
/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.