From: kernel test robot <lkp@intel.com>
To: "Andreas Hindborg (Samsung)" <nmi@metaspace.dk>,
Ming Lei <ming.lei@redhat.com>
Cc: oe-kbuild-all@lists.linux.dev,
Christoph Hellwig <hch@infradead.org>,
gost.dev@samsung.com, linux-block@vger.kernel.org,
Hans Holmberg <Hans.Holmberg@wdc.com>,
Matias Bjorling <Matias.Bjorling@wdc.com>,
Andreas Hindborg <a.hindborg@samsung.com>,
Minwoo Im <minwoo.im.dev@gmail.com>, Jens Axboe <axboe@kernel.dk>,
Johannes Thumshirn <jth@kernel.org>,
Aravind Ramesh <Aravind.Ramesh@wdc.com>,
Damien Le Moal <dlemoal@kernel.org>,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v10 3/3] ublk: enable zoned storage support
Date: Fri, 4 Aug 2023 02:58:14 +0800 [thread overview]
Message-ID: <202308040229.pFTUh2cG-lkp@intel.com> (raw)
In-Reply-To: <20230803140701.18515-4-nmi@metaspace.dk>
Hi Andreas,
kernel test robot noticed the following build errors:
[auto build test ERROR on 5d0c230f1de8c7515b6567d9afba1f196fb4e2f4]
url: https://github.com/intel-lab-lkp/linux/commits/Andreas-Hindborg-Samsung/ublk-add-helper-to-check-if-device-supports-user-copy/20230803-224003
base: 5d0c230f1de8c7515b6567d9afba1f196fb4e2f4
patch link: https://lore.kernel.org/r/20230803140701.18515-4-nmi%40metaspace.dk
patch subject: [PATCH v10 3/3] ublk: enable zoned storage support
config: sh-randconfig-r024-20230731 (https://download.01.org/0day-ci/archive/20230804/202308040229.pFTUh2cG-lkp@intel.com/config)
compiler: sh4-linux-gcc (GCC) 12.3.0
reproduce: (https://download.01.org/0day-ci/archive/20230804/202308040229.pFTUh2cG-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/202308040229.pFTUh2cG-lkp@intel.com/
All error/warnings (new ones prefixed by >>):
drivers/block/ublk_drv.c: In function 'ublk_alloc_report_buffer':
>> drivers/block/ublk_drv.c:281:23: error: implicit declaration of function '__vmalloc'; did you mean '__kmalloc'? [-Werror=implicit-function-declaration]
281 | buf = __vmalloc(bufsize, GFP_KERNEL | __GFP_NORETRY);
| ^~~~~~~~~
| __kmalloc
>> drivers/block/ublk_drv.c:281:21: warning: assignment to 'void *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
281 | buf = __vmalloc(bufsize, GFP_KERNEL | __GFP_NORETRY);
| ^
cc1: some warnings being treated as errors
vim +281 drivers/block/ublk_drv.c
264
265 /* Based on virtblk_alloc_report_buffer */
266 static void *ublk_alloc_report_buffer(struct ublk_device *ublk,
267 unsigned int nr_zones, size_t *buflen)
268 {
269 struct request_queue *q = ublk->ub_disk->queue;
270 size_t bufsize;
271 void *buf;
272
273 nr_zones = min_t(unsigned int, nr_zones,
274 ublk->ub_disk->nr_zones);
275
276 bufsize = nr_zones * sizeof(struct blk_zone);
277 bufsize =
278 min_t(size_t, bufsize, queue_max_hw_sectors(q) << SECTOR_SHIFT);
279
280 while (bufsize >= sizeof(struct blk_zone)) {
> 281 buf = __vmalloc(bufsize, GFP_KERNEL | __GFP_NORETRY);
282 if (buf) {
283 *buflen = bufsize;
284 return buf;
285 }
286 bufsize >>= 1;
287 }
288
289 *buflen = 0;
290 return NULL;
291 }
292
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
next prev parent reply other threads:[~2023-08-03 18:59 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-08-03 14:06 [PATCH v10 0/3] ublk: enable zoned storage support Andreas Hindborg (Samsung)
2023-08-03 14:06 ` [PATCH v10 1/3] ublk: add helper to check if device supports user copy Andreas Hindborg (Samsung)
2023-08-03 19:02 ` Johannes Thumshirn
2023-08-03 14:07 ` [PATCH v10 2/3] ublk: move check for empty address field on command submission Andreas Hindborg (Samsung)
2023-08-04 9:01 ` Ming Lei
2023-08-04 10:08 ` Andreas Hindborg (Samsung)
2023-08-03 14:07 ` [PATCH v10 3/3] ublk: enable zoned storage support Andreas Hindborg (Samsung)
2023-08-03 18:58 ` kernel test robot [this message]
2023-08-03 19:07 ` Johannes Thumshirn
2023-08-04 8:59 ` Andreas Hindborg (Samsung)
2023-08-04 10:25 ` Johannes Thumshirn
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=202308040229.pFTUh2cG-lkp@intel.com \
--to=lkp@intel.com \
--cc=Aravind.Ramesh@wdc.com \
--cc=Hans.Holmberg@wdc.com \
--cc=Matias.Bjorling@wdc.com \
--cc=a.hindborg@samsung.com \
--cc=axboe@kernel.dk \
--cc=dlemoal@kernel.org \
--cc=gost.dev@samsung.com \
--cc=hch@infradead.org \
--cc=jth@kernel.org \
--cc=linux-block@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=ming.lei@redhat.com \
--cc=minwoo.im.dev@gmail.com \
--cc=nmi@metaspace.dk \
--cc=oe-kbuild-all@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox