From: kernel test robot <lkp@intel.com>
To: SelvaKumar S <selvakuma.s1@samsung.com>,
linux-nvme@lists.infradead.org, linux-block@vger.kernel.org
Cc: axboe@kernel.dk, damien.lemoal@wdc.com, kbuild-all@lists.01.org,
linux-scsi@vger.kernel.org, linux-api@vger.kernel.org,
kbusch@kernel.org, clang-built-linux@googlegroups.com,
dm-devel@redhat.com, linux-fsdevel@vger.kernel.org,
asml.silence@gmail.com
Subject: Re: [dm-devel] [PATCH 3/7] block: copy offload support infrastructure
Date: Wed, 18 Aug 2021 04:35:43 +0800 [thread overview]
Message-ID: <202108180431.JPZU44Bn-lkp@intel.com> (raw)
In-Reply-To: <20210817101423.12367-4-selvakuma.s1@samsung.com>
[-- Attachment #1: Type: text/plain, Size: 7342 bytes --]
Hi SelvaKumar,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on block/for-next]
[also build test WARNING on dm/for-next linus/master v5.14-rc6 next-20210817]
[cannot apply to linux-nvme/for-next]
[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/0day-ci/linux/commits/SelvaKumar-S/block-make-bio_map_kern-non-static/20210817-193111
base: https://git.kernel.org/pub/scm/linux/kernel/git/axboe/linux-block.git for-next
config: hexagon-randconfig-r013-20210816 (attached as .config)
compiler: clang version 12.0.0
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/0day-ci/linux/commit/35fc502a7f20a7cd42432cee2777a621c40a3bd3
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review SelvaKumar-S/block-make-bio_map_kern-non-static/20210817-193111
git checkout 35fc502a7f20a7cd42432cee2777a621c40a3bd3
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=hexagon
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:197:5: warning: no previous prototype for function 'blk_copy_offload_submit_bio' [-Wmissing-prototypes]
int blk_copy_offload_submit_bio(struct block_device *bdev,
^
block/blk-lib.c:197:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
int blk_copy_offload_submit_bio(struct block_device *bdev,
^
static
>> block/blk-lib.c:250:5: warning: no previous prototype for function 'blk_copy_offload_scc' [-Wmissing-prototypes]
int blk_copy_offload_scc(struct block_device *src_bdev, int nr_srcs,
^
block/blk-lib.c:250:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
int blk_copy_offload_scc(struct block_device *src_bdev, int nr_srcs,
^
static
2 warnings generated.
vim +/blk_copy_offload_submit_bio +197 block/blk-lib.c
196
> 197 int blk_copy_offload_submit_bio(struct block_device *bdev,
198 struct blk_copy_payload *payload, int payload_size,
199 struct cio *cio, gfp_t gfp_mask)
200 {
201 struct request_queue *q = bdev_get_queue(bdev);
202 struct bio *bio;
203
204 bio = bio_map_kern(q, payload, payload_size, gfp_mask);
205 if (IS_ERR(bio))
206 return PTR_ERR(bio);
207
208 bio_set_dev(bio, bdev);
209 bio->bi_opf = REQ_OP_COPY | REQ_NOMERGE;
210 bio->bi_iter.bi_sector = payload->dest;
211 bio->bi_end_io = cio_bio_end_io;
212 bio->bi_private = cio;
213 atomic_inc(&cio->refcount);
214 submit_bio(bio);
215
216 return 0;
217 }
218
219 /* Go through all the enrties inside user provided payload, and determine the
220 * maximum number of entries in a payload, based on device's scc-limits.
221 */
222 static inline int blk_max_payload_entries(int nr_srcs, struct range_entry *rlist,
223 int max_nr_srcs, sector_t max_copy_range_sectors, sector_t max_copy_len)
224 {
225 sector_t range_len, copy_len = 0, remaining = 0;
226 int ri = 0, pi = 1, max_pi = 0;
227
228 for (ri = 0; ri < nr_srcs; ri++) {
229 for (remaining = rlist[ri].len; remaining > 0; remaining -= range_len) {
230 range_len = min3(remaining, max_copy_range_sectors,
231 max_copy_len - copy_len);
232 pi++;
233 copy_len += range_len;
234
235 if ((pi == max_nr_srcs) || (copy_len == max_copy_len)) {
236 max_pi = max(max_pi, pi);
237 pi = 1;
238 copy_len = 0;
239 }
240 }
241 }
242
243 return max(max_pi, pi);
244 }
245
246 /*
247 * blk_copy_offload_scc - Use device's native copy offload feature
248 * Go through user provide payload, prepare new payload based on device's copy offload limits.
249 */
> 250 int blk_copy_offload_scc(struct block_device *src_bdev, int nr_srcs,
251 struct range_entry *rlist, struct block_device *dest_bdev,
252 sector_t dest, gfp_t gfp_mask)
253 {
254 struct request_queue *q = bdev_get_queue(dest_bdev);
255 struct cio *cio = NULL;
256 struct blk_copy_payload *payload;
257 sector_t range_len, copy_len = 0, remaining = 0;
258 sector_t src_blk, cdest = dest;
259 sector_t max_copy_range_sectors, max_copy_len;
260 int ri = 0, pi = 0, ret = 0, payload_size, max_pi, max_nr_srcs;
261
262 cio = kzalloc(sizeof(struct cio), GFP_KERNEL);
263 if (!cio)
264 return -ENOMEM;
265 atomic_set(&cio->refcount, 0);
266
267 max_nr_srcs = q->limits.max_copy_nr_ranges;
268 max_copy_range_sectors = q->limits.max_copy_range_sectors;
269 max_copy_len = q->limits.max_copy_sectors;
270
271 max_pi = blk_max_payload_entries(nr_srcs, rlist, max_nr_srcs,
272 max_copy_range_sectors, max_copy_len);
273 payload_size = struct_size(payload, range, max_pi);
274
275 payload = kvmalloc(payload_size, gfp_mask);
276 if (!payload) {
277 ret = -ENOMEM;
278 goto free_cio;
279 }
280 payload->src_bdev = src_bdev;
281
282 for (ri = 0; ri < nr_srcs; ri++) {
283 for (remaining = rlist[ri].len, src_blk = rlist[ri].src; remaining > 0;
284 remaining -= range_len, src_blk += range_len) {
285
286 range_len = min3(remaining, max_copy_range_sectors,
287 max_copy_len - copy_len);
288 payload->range[pi].len = range_len;
289 payload->range[pi].src = src_blk;
290 pi++;
291 copy_len += range_len;
292
293 /* Submit current payload, if crossing device copy limits */
294 if ((pi == max_nr_srcs) || (copy_len == max_copy_len)) {
295 payload->dest = cdest;
296 payload->copy_nr_ranges = pi;
297 ret = blk_copy_offload_submit_bio(dest_bdev, payload,
298 payload_size, cio, gfp_mask);
299 if (ret)
300 goto free_payload;
301
302 /* reset index, length and allocate new payload */
303 pi = 0;
304 cdest += copy_len;
305 copy_len = 0;
306 payload = kvmalloc(payload_size, gfp_mask);
307 if (!payload) {
308 ret = -ENOMEM;
309 goto free_cio;
310 }
311 payload->src_bdev = src_bdev;
312 }
313 }
314 }
315
316 if (pi) {
317 payload->dest = cdest;
318 payload->copy_nr_ranges = pi;
319 ret = blk_copy_offload_submit_bio(dest_bdev, payload, payload_size, cio, gfp_mask);
320 if (ret)
321 goto free_payload;
322 }
323
324 /* Wait for completion of all IO's*/
325 ret = cio_await_completion(cio);
326
327 return ret;
328
329 free_payload:
330 kvfree(payload);
331 free_cio:
332 cio_await_completion(cio);
333 return ret;
334 }
335
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 22159 bytes --]
[-- Attachment #3: Type: text/plain, Size: 97 bytes --]
--
dm-devel mailing list
dm-devel@redhat.com
https://listman.redhat.com/mailman/listinfo/dm-devel
next prev parent reply other threads:[~2021-08-17 20:40 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <CGME20210817101741epcas5p174ca0a539587da6a67b9f58cd13f2bad@epcas5p1.samsung.com>
2021-08-17 10:14 ` [dm-devel] [PATCH 0/7] add simple copy support SelvaKumar S
[not found] ` <CGME20210817101747epcas5p1242e63ec29b127b03b6f9f5f1b57f86e@epcas5p1.samsung.com>
2021-08-17 10:14 ` [dm-devel] [PATCH 1/7] block: make bio_map_kern() non static SelvaKumar S
[not found] ` <CGME20210817101753epcas5p4f4257f8edda27e184ecbb273b700ccbc@epcas5p4.samsung.com>
2021-08-17 10:14 ` [dm-devel] [PATCH 2/7] block: Introduce queue limits for copy-offload support SelvaKumar S
2021-08-17 13:08 ` Greg KH
2021-08-17 14:42 ` Nitesh Shetty
[not found] ` <CGME20210817101758epcas5p1ec353b3838d64654e69488229256d9eb@epcas5p1.samsung.com>
2021-08-17 10:14 ` [dm-devel] [PATCH 3/7] block: copy offload support infrastructure SelvaKumar S
2021-08-17 17:14 ` Bart Van Assche
2021-08-17 20:41 ` Mikulas Patocka
2021-08-17 21:53 ` Douglas Gilbert
2021-08-17 22:06 ` Bart Van Assche
2021-08-20 10:39 ` Kanchan Joshi
2021-08-20 21:18 ` Bart Van Assche
2021-08-26 7:46 ` Nitesh Shetty
2021-08-17 20:35 ` kernel test robot [this message]
2021-08-18 18:35 ` Martin K. Petersen
2021-08-20 11:11 ` Kanchan Joshi
[not found] ` <CGME20210817101803epcas5p10cda1d52f8a8f1172e34b1f9cf8eef3b@epcas5p1.samsung.com>
2021-08-17 10:14 ` [dm-devel] [PATCH 4/7] block: Introduce a new ioctl for simple copy SelvaKumar S
2021-08-17 13:09 ` Greg KH
2021-08-17 13:10 ` Greg KH
2021-08-17 14:48 ` Nitesh Shetty
2021-08-17 23:36 ` Darrick J. Wong
2021-08-18 15:37 ` Nitesh Shetty
2021-08-18 16:17 ` Darrick J. Wong
[not found] ` <CGME20210817101809epcas5p39eed3531ed82f5f08127eb3dba1fc50f@epcas5p3.samsung.com>
2021-08-17 10:14 ` [dm-devel] [PATCH 5/7] block: add emulation " SelvaKumar S
2021-08-17 22:10 ` kernel test robot
[not found] ` <CGME20210817101814epcas5p41db3d7269f5139efcaf2ca685cd04a16@epcas5p4.samsung.com>
2021-08-17 10:14 ` [dm-devel] [PATCH 6/7] nvme: add simple copy support SelvaKumar S
[not found] ` <CGME20210817101822epcas5p470644cf681d5e8db5367dc7998305c65@epcas5p4.samsung.com>
2021-08-17 10:14 ` [dm-devel] [PATCH 7/7] dm kcopyd: add simple copy offload support SelvaKumar S
2021-08-17 20:29 ` Mikulas Patocka
2021-08-17 23:37 ` [dm-devel] [PATCH 0/7] add simple copy support Darrick J. Wong
2021-08-18 15:40 ` Nitesh Shetty
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=202108180431.JPZU44Bn-lkp@intel.com \
--to=lkp@intel.com \
--cc=asml.silence@gmail.com \
--cc=axboe@kernel.dk \
--cc=clang-built-linux@googlegroups.com \
--cc=damien.lemoal@wdc.com \
--cc=dm-devel@redhat.com \
--cc=kbuild-all@lists.01.org \
--cc=kbusch@kernel.org \
--cc=linux-api@vger.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=selvakuma.s1@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).