From: kernel test robot <lkp@intel.com>
To: Nitesh Shetty <nj.shetty@samsung.com>
Cc: Vincent Fu <vincent.fu@samsung.com>,
djwong@kernel.org, llvm@lists.linux.dev,
linux-nvme@lists.infradead.org, clm@fb.com, dm-devel@redhat.com,
osandov@fb.com, Alasdair Kergon <agk@redhat.com>,
msnitzer@redhat.com, bvanassche@acm.org,
linux-scsi@vger.kernel.org, gost.dev@samsung.com,
nitheshshetty@gmail.com, hch@lst.de,
Nitesh Shetty <nj.shetty@samsung.com>,
chaitanyak@nvidia.com, Mike Snitzer <snitzer@kernel.org>,
josef@toxicpanda.com, linux-block@vger.kernel.org,
dsterba@suse.com, kbusch@kernel.org, tytso@mit.edu,
Frederick.Knight@netapp.com, Sagi Grimberg <sagi@grimberg.me>,
axboe@kernel.dk, kbuild-all@lists.01.org,
martin.petersen@oracle.com, Arnav Dawn <arnav.dawn@samsung.com>,
jack@suse.com, linux-fsdevel@vger.kernel.org,
lsf-pc@lists.linux-foundation.org
Subject: Re: [dm-devel] [PATCH v4 04/10] block: add emulation for copy
Date: Wed, 27 Apr 2022 09:33:04 +0800 [thread overview]
Message-ID: <202204270913.Ecb3uQx1-lkp@intel.com> (raw)
In-Reply-To: <20220426101241.30100-5-nj.shetty@samsung.com>
Hi Nitesh,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on next-20220422]
[cannot apply to axboe-block/for-next device-mapper-dm/for-next linus/master v5.18-rc4 v5.18-rc3 v5.18-rc2 v5.18-rc4]
[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]
url: https://github.com/intel-lab-lkp/linux/commits/Nitesh-Shetty/block-Introduce-queue-limits-for-copy-offload-support/20220426-201825
base: e7d6987e09a328d4a949701db40ef63fbb970670
config: hexagon-randconfig-r041-20220425 (https://download.01.org/0day-ci/archive/20220427/202204270913.Ecb3uQx1-lkp@intel.com/config)
compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project 1cddcfdc3c683b393df1a5c9063252eb60e52818)
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
# https://github.com/intel-lab-lkp/linux/commit/c406c5145dc7d628d4197f6726c23a3f1179b88e
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review Nitesh-Shetty/block-Introduce-queue-limits-for-copy-offload-support/20220426-201825
git checkout c406c5145dc7d628d4197f6726c23a3f1179b88e
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=hexagon SHELL=/bin/bash
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
block/blk-lib.c:178:5: warning: no previous prototype for function 'blk_copy_offload' [-Wmissing-prototypes]
int blk_copy_offload(struct block_device *src_bdev, int nr_srcs,
^
block/blk-lib.c:178:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
int blk_copy_offload(struct block_device *src_bdev, int nr_srcs,
^
static
>> block/blk-lib.c:276:5: warning: no previous prototype for function 'blk_submit_rw_buf' [-Wmissing-prototypes]
int blk_submit_rw_buf(struct block_device *bdev, void *buf, sector_t buf_len,
^
block/blk-lib.c:276:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
int blk_submit_rw_buf(struct block_device *bdev, void *buf, sector_t buf_len,
^
static
2 warnings generated.
vim +/blk_submit_rw_buf +276 block/blk-lib.c
275
> 276 int blk_submit_rw_buf(struct block_device *bdev, void *buf, sector_t buf_len,
277 sector_t sector, unsigned int op, gfp_t gfp_mask)
278 {
279 struct request_queue *q = bdev_get_queue(bdev);
280 struct bio *bio, *parent = NULL;
281 sector_t max_hw_len = min_t(unsigned int, queue_max_hw_sectors(q),
282 queue_max_segments(q) << (PAGE_SHIFT - SECTOR_SHIFT)) << SECTOR_SHIFT;
283 sector_t len, remaining;
284 int ret;
285
286 for (remaining = buf_len; remaining > 0; remaining -= len) {
287 len = min_t(int, max_hw_len, remaining);
288 retry:
289 bio = bio_map_kern(q, buf, len, gfp_mask);
290 if (IS_ERR(bio)) {
291 len >>= 1;
292 if (len)
293 goto retry;
294 return PTR_ERR(bio);
295 }
296
297 bio->bi_iter.bi_sector = sector >> SECTOR_SHIFT;
298 bio->bi_opf = op;
299 bio_set_dev(bio, bdev);
300 bio->bi_end_io = NULL;
301 bio->bi_private = NULL;
302
303 if (parent) {
304 bio_chain(parent, bio);
305 submit_bio(parent);
306 }
307 parent = bio;
308 sector += len;
309 buf = (char *) buf + len;
310 }
311 ret = submit_bio_wait(bio);
312 bio_put(bio);
313
314 return ret;
315 }
316
--
0-DAY CI Kernel Test Service
https://01.org/lkp
--
dm-devel mailing list
dm-devel@redhat.com
https://listman.redhat.com/mailman/listinfo/dm-devel
WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com>
To: Nitesh Shetty <nj.shetty@samsung.com>
Cc: llvm@lists.linux.dev, kbuild-all@lists.01.org,
chaitanyak@nvidia.com, linux-block@vger.kernel.org,
linux-scsi@vger.kernel.org, dm-devel@redhat.com,
linux-nvme@lists.infradead.org, linux-fsdevel@vger.kernel.org,
axboe@kernel.dk, msnitzer@redhat.com, bvanassche@acm.org,
martin.petersen@oracle.com, hare@suse.de, kbusch@kernel.org,
hch@lst.de, Frederick.Knight@netapp.com, osandov@fb.com,
lsf-pc@lists.linux-foundation.org, djwong@kernel.org,
josef@toxicpanda.com, clm@fb.com, dsterba@suse.com,
tytso@mit.edu, jack@suse.com, nitheshshetty@gmail.com,
gost.dev@samsung.com, Nitesh Shetty <nj.shetty@samsung.com>,
Vincent Fu <vincent.fu@samsung.com>,
Arnav Dawn <arnav.dawn@samsung.com>,
Alasdair Kergon <agk@redhat.com>,
Mike Snitzer <snitzer@kernel.org>,
Sagi Grimberg <sagi@grimberg.me>
Subject: Re: [PATCH v4 04/10] block: add emulation for copy
Date: Wed, 27 Apr 2022 09:33:04 +0800 [thread overview]
Message-ID: <202204270913.Ecb3uQx1-lkp@intel.com> (raw)
In-Reply-To: <20220426101241.30100-5-nj.shetty@samsung.com>
Hi Nitesh,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on next-20220422]
[cannot apply to axboe-block/for-next device-mapper-dm/for-next linus/master v5.18-rc4 v5.18-rc3 v5.18-rc2 v5.18-rc4]
[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]
url: https://github.com/intel-lab-lkp/linux/commits/Nitesh-Shetty/block-Introduce-queue-limits-for-copy-offload-support/20220426-201825
base: e7d6987e09a328d4a949701db40ef63fbb970670
config: hexagon-randconfig-r041-20220425 (https://download.01.org/0day-ci/archive/20220427/202204270913.Ecb3uQx1-lkp@intel.com/config)
compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project 1cddcfdc3c683b393df1a5c9063252eb60e52818)
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
# https://github.com/intel-lab-lkp/linux/commit/c406c5145dc7d628d4197f6726c23a3f1179b88e
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review Nitesh-Shetty/block-Introduce-queue-limits-for-copy-offload-support/20220426-201825
git checkout c406c5145dc7d628d4197f6726c23a3f1179b88e
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=hexagon SHELL=/bin/bash
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
block/blk-lib.c:178:5: warning: no previous prototype for function 'blk_copy_offload' [-Wmissing-prototypes]
int blk_copy_offload(struct block_device *src_bdev, int nr_srcs,
^
block/blk-lib.c:178:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
int blk_copy_offload(struct block_device *src_bdev, int nr_srcs,
^
static
>> block/blk-lib.c:276:5: warning: no previous prototype for function 'blk_submit_rw_buf' [-Wmissing-prototypes]
int blk_submit_rw_buf(struct block_device *bdev, void *buf, sector_t buf_len,
^
block/blk-lib.c:276:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
int blk_submit_rw_buf(struct block_device *bdev, void *buf, sector_t buf_len,
^
static
2 warnings generated.
vim +/blk_submit_rw_buf +276 block/blk-lib.c
275
> 276 int blk_submit_rw_buf(struct block_device *bdev, void *buf, sector_t buf_len,
277 sector_t sector, unsigned int op, gfp_t gfp_mask)
278 {
279 struct request_queue *q = bdev_get_queue(bdev);
280 struct bio *bio, *parent = NULL;
281 sector_t max_hw_len = min_t(unsigned int, queue_max_hw_sectors(q),
282 queue_max_segments(q) << (PAGE_SHIFT - SECTOR_SHIFT)) << SECTOR_SHIFT;
283 sector_t len, remaining;
284 int ret;
285
286 for (remaining = buf_len; remaining > 0; remaining -= len) {
287 len = min_t(int, max_hw_len, remaining);
288 retry:
289 bio = bio_map_kern(q, buf, len, gfp_mask);
290 if (IS_ERR(bio)) {
291 len >>= 1;
292 if (len)
293 goto retry;
294 return PTR_ERR(bio);
295 }
296
297 bio->bi_iter.bi_sector = sector >> SECTOR_SHIFT;
298 bio->bi_opf = op;
299 bio_set_dev(bio, bdev);
300 bio->bi_end_io = NULL;
301 bio->bi_private = NULL;
302
303 if (parent) {
304 bio_chain(parent, bio);
305 submit_bio(parent);
306 }
307 parent = bio;
308 sector += len;
309 buf = (char *) buf + len;
310 }
311 ret = submit_bio_wait(bio);
312 bio_put(bio);
313
314 return ret;
315 }
316
--
0-DAY CI Kernel Test Service
https://01.org/lkp
next prev parent reply other threads:[~2022-04-27 1:33 UTC|newest]
Thread overview: 101+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <CGME20220426101804epcas5p4a0a325d3ce89e868e4924bbdeeba6d15@epcas5p4.samsung.com>
2022-04-26 10:12 ` [dm-devel] [PATCH v4 00/10] Add Copy offload support Nitesh Shetty
2022-04-26 10:12 ` Nitesh Shetty
2022-04-26 10:12 ` Nitesh Shetty
2022-04-26 10:12 ` [dm-devel] [PATCH v4 01/10] block: Introduce queue limits for copy-offload support Nitesh Shetty
2022-04-26 10:12 ` Nitesh Shetty
2022-04-26 10:12 ` Nitesh Shetty
2022-04-27 1:59 ` [dm-devel] " Damien Le Moal
2022-04-27 1:59 ` Damien Le Moal
2022-04-27 15:30 ` [dm-devel] " Nitesh Shetty
2022-04-27 15:30 ` Nitesh Shetty
2022-04-27 21:57 ` [dm-devel] " Damien Le Moal
2022-04-27 21:57 ` Damien Le Moal
2022-04-27 10:30 ` [dm-devel] " Hannes Reinecke
2022-04-27 10:30 ` Hannes Reinecke
2022-04-26 10:12 ` [dm-devel] [PATCH v4 02/10] block: Add copy offload support infrastructure Nitesh Shetty
2022-04-26 10:12 ` Nitesh Shetty
2022-04-26 10:12 ` Nitesh Shetty
2022-04-27 0:11 ` [dm-devel] " kernel test robot
2022-04-27 0:11 ` kernel test robot
2022-04-27 2:45 ` [dm-devel] " Damien Le Moal
2022-04-27 2:45 ` Damien Le Moal
2022-04-27 15:15 ` [dm-devel] " Nitesh Shetty
2022-04-27 15:15 ` Nitesh Shetty
2022-04-27 22:04 ` [dm-devel] " Damien Le Moal
2022-04-27 22:04 ` Damien Le Moal
2022-04-28 8:01 ` [dm-devel] " Nitesh Shetty
2022-04-28 8:01 ` Nitesh Shetty
2022-04-27 10:29 ` [dm-devel] " Hannes Reinecke
2022-04-27 10:29 ` Hannes Reinecke
2022-04-27 15:48 ` [dm-devel] " Nitesh Shetty
2022-04-27 15:48 ` Nitesh Shetty
2022-04-26 10:12 ` [dm-devel] [PATCH v4 03/10] block: Introduce a new ioctl for copy Nitesh Shetty
2022-04-26 10:12 ` Nitesh Shetty
2022-04-26 10:12 ` Nitesh Shetty
2022-04-27 2:48 ` [dm-devel] " Damien Le Moal
2022-04-27 2:48 ` Damien Le Moal
2022-04-27 13:03 ` [dm-devel] " Nitesh Shetty
2022-04-27 13:03 ` Nitesh Shetty
2022-04-27 10:37 ` [dm-devel] " Hannes Reinecke
2022-04-27 10:37 ` Hannes Reinecke
2022-04-26 10:12 ` [dm-devel] [PATCH v4 04/10] block: add emulation " Nitesh Shetty
2022-04-26 10:12 ` Nitesh Shetty
2022-04-26 10:12 ` Nitesh Shetty
2022-04-27 1:33 ` kernel test robot [this message]
2022-04-27 1:33 ` kernel test robot
2022-04-26 10:12 ` [dm-devel] [PATCH v4 05/10] nvme: add copy offload support Nitesh Shetty
2022-04-26 10:12 ` Nitesh Shetty
2022-04-26 10:12 ` Nitesh Shetty
2022-04-28 14:02 ` [dm-devel] " kernel test robot
2022-04-28 14:02 ` kernel test robot
2022-04-26 10:12 ` [dm-devel] [PATCH v4 06/10] nvmet: add copy command support for bdev and file ns Nitesh Shetty
2022-04-26 10:12 ` Nitesh Shetty
2022-04-26 10:12 ` Nitesh Shetty
2022-04-28 14:53 ` [dm-devel] " kernel test robot
2022-04-28 14:53 ` kernel test robot
2022-04-26 10:12 ` [dm-devel] [PATCH v4 07/10] dm: Add support for copy offload Nitesh Shetty
2022-04-26 10:12 ` Nitesh Shetty
2022-04-26 10:12 ` Nitesh Shetty
2022-04-28 15:54 ` [dm-devel] " kernel test robot
2022-04-28 15:54 ` kernel test robot
2022-04-26 10:12 ` [dm-devel] [PATCH v4 08/10] dm: Enable copy offload for dm-linear target Nitesh Shetty
2022-04-26 10:12 ` Nitesh Shetty
2022-04-26 10:12 ` Nitesh Shetty
2022-04-26 10:12 ` [dm-devel] [PATCH v4 09/10] dm kcopyd: use copy offload support Nitesh Shetty
2022-04-26 10:12 ` Nitesh Shetty
2022-04-26 10:12 ` Nitesh Shetty
2022-04-26 10:12 ` [dm-devel] [PATCH v4 10/10] fs: add support for copy file range in zonefs Nitesh Shetty
2022-04-26 10:12 ` Nitesh Shetty
2022-04-26 10:12 ` Nitesh Shetty
2022-04-27 1:42 ` [dm-devel] " Damien Le Moal
2022-04-27 1:42 ` Damien Le Moal
2022-04-27 1:46 ` [dm-devel] [PATCH v4 00/10] Add Copy offload support Damien Le Moal
2022-04-27 1:46 ` Damien Le Moal
2022-04-27 15:38 ` [dm-devel] " Nitesh Shetty
2022-04-27 15:38 ` Nitesh Shetty
2022-04-27 21:56 ` [dm-devel] " Damien Le Moal
2022-04-27 21:56 ` Damien Le Moal
2022-04-27 2:00 ` [dm-devel] " Damien Le Moal
2022-04-27 2:00 ` Damien Le Moal
2022-04-27 2:19 ` [dm-devel] " Damien Le Moal
2022-04-27 2:19 ` Damien Le Moal
2022-04-27 12:49 ` [dm-devel] " Nitesh Shetty
2022-04-27 12:49 ` Nitesh Shetty
2022-04-27 22:05 ` [dm-devel] " Damien Le Moal
2022-04-27 22:05 ` Damien Le Moal
2022-04-28 7:49 ` [dm-devel] " Nitesh Shetty
2022-04-28 7:49 ` Nitesh Shetty
2022-04-28 21:37 ` [dm-devel] " Damien Le Moal
2022-04-28 21:37 ` Damien Le Moal
2022-04-29 3:39 ` [dm-devel] " Bart Van Assche
2022-04-29 3:39 ` Bart Van Assche
2022-05-02 4:09 ` Dave Chinner
2022-05-02 4:09 ` Dave Chinner
2022-05-02 12:54 ` [dm-devel] " Damien Le Moal
2022-05-02 12:54 ` Damien Le Moal
2022-05-02 23:20 ` [dm-devel] " Dave Chinner
2022-05-02 23:20 ` Dave Chinner
2022-05-02 12:14 ` [dm-devel] " Damien Le Moal
2022-05-02 12:14 ` Damien Le Moal
2022-05-02 12:16 ` Damien Le Moal
2022-05-02 12:16 ` Damien Le Moal
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=202204270913.Ecb3uQx1-lkp@intel.com \
--to=lkp@intel.com \
--cc=Frederick.Knight@netapp.com \
--cc=agk@redhat.com \
--cc=arnav.dawn@samsung.com \
--cc=axboe@kernel.dk \
--cc=bvanassche@acm.org \
--cc=chaitanyak@nvidia.com \
--cc=clm@fb.com \
--cc=djwong@kernel.org \
--cc=dm-devel@redhat.com \
--cc=dsterba@suse.com \
--cc=gost.dev@samsung.com \
--cc=hch@lst.de \
--cc=jack@suse.com \
--cc=josef@toxicpanda.com \
--cc=kbuild-all@lists.01.org \
--cc=kbusch@kernel.org \
--cc=linux-block@vger.kernel.org \
--cc=linux-fsdevel@vger.kernel.org \
--cc=linux-nvme@lists.infradead.org \
--cc=linux-scsi@vger.kernel.org \
--cc=llvm@lists.linux.dev \
--cc=lsf-pc@lists.linux-foundation.org \
--cc=martin.petersen@oracle.com \
--cc=msnitzer@redhat.com \
--cc=nitheshshetty@gmail.com \
--cc=nj.shetty@samsung.com \
--cc=osandov@fb.com \
--cc=sagi@grimberg.me \
--cc=snitzer@kernel.org \
--cc=tytso@mit.edu \
--cc=vincent.fu@samsung.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.