All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: kbuild@lists.01.org
Subject: Re: [RFC PATCH v5 4/4] dm kcopyd: add simple copy offload support
Date: Sat, 20 Feb 2021 13:52:53 +0800	[thread overview]
Message-ID: <202102201309.E8VcBvvn-lkp@intel.com> (raw)

[-- Attachment #1: Type: text/plain, Size: 4249 bytes --]

CC: kbuild-all(a)lists.01.org
In-Reply-To: <20210219124517.79359-5-selvakuma.s1@samsung.com>
References: <20210219124517.79359-5-selvakuma.s1@samsung.com>
TO: SelvaKumar S <selvakuma.s1@samsung.com>

Hi SelvaKumar,

[FYI, it's a private test report for your RFC patch.]
[auto build test WARNING on linus/master]
[also build test WARNING on v5.11]
[cannot apply to block/for-next dm/for-next linux-nvme/for-next next-20210219]
[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/20210220-100547
base:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git f40ddce88593482919761f74910f42f4b84c004b
:::::: branch date: 4 hours ago
:::::: commit date: 4 hours ago
config: i386-randconfig-m021-20210219 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-15) 9.3.0

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>

smatch warnings:
drivers/md/dm-kcopyd.c:613 run_copy_job() error: uninitialized symbol 'r'.

vim +/r +613 drivers/md/dm-kcopyd.c

^1da177e4c3f41 drivers/md/kcopyd.c    Linus Torvalds 2005-04-16  585  
f79939bb63c402 drivers/md/dm-kcopyd.c SelvaKumar S   2021-02-19  586  static int run_copy_job(struct kcopyd_job *job)
f79939bb63c402 drivers/md/dm-kcopyd.c SelvaKumar S   2021-02-19  587  {
f79939bb63c402 drivers/md/dm-kcopyd.c SelvaKumar S   2021-02-19  588  	int r, i, count = 0;
f79939bb63c402 drivers/md/dm-kcopyd.c SelvaKumar S   2021-02-19  589  	unsigned long flags = 0;
f79939bb63c402 drivers/md/dm-kcopyd.c SelvaKumar S   2021-02-19  590  	struct range_entry srange;
f79939bb63c402 drivers/md/dm-kcopyd.c SelvaKumar S   2021-02-19  591  
f79939bb63c402 drivers/md/dm-kcopyd.c SelvaKumar S   2021-02-19  592  	flags |= BLKDEV_COPY_NOEMULATION;
f79939bb63c402 drivers/md/dm-kcopyd.c SelvaKumar S   2021-02-19  593  	for (i = 0; i < job->num_dests; i++) {
f79939bb63c402 drivers/md/dm-kcopyd.c SelvaKumar S   2021-02-19  594  		srange.src = job->source.sector;
f79939bb63c402 drivers/md/dm-kcopyd.c SelvaKumar S   2021-02-19  595  		srange.len = job->source.count;
f79939bb63c402 drivers/md/dm-kcopyd.c SelvaKumar S   2021-02-19  596  
f79939bb63c402 drivers/md/dm-kcopyd.c SelvaKumar S   2021-02-19  597  		r = blkdev_issue_copy(job->source.bdev, 1, &srange,
f79939bb63c402 drivers/md/dm-kcopyd.c SelvaKumar S   2021-02-19  598  			job->dests[i].bdev, job->dests[i].sector, GFP_KERNEL, flags);
f79939bb63c402 drivers/md/dm-kcopyd.c SelvaKumar S   2021-02-19  599  		if (r)
f79939bb63c402 drivers/md/dm-kcopyd.c SelvaKumar S   2021-02-19  600  			break;
f79939bb63c402 drivers/md/dm-kcopyd.c SelvaKumar S   2021-02-19  601  
f79939bb63c402 drivers/md/dm-kcopyd.c SelvaKumar S   2021-02-19  602  		job->dests[i].count = 0;
f79939bb63c402 drivers/md/dm-kcopyd.c SelvaKumar S   2021-02-19  603  		count++;
f79939bb63c402 drivers/md/dm-kcopyd.c SelvaKumar S   2021-02-19  604  	}
f79939bb63c402 drivers/md/dm-kcopyd.c SelvaKumar S   2021-02-19  605  
f79939bb63c402 drivers/md/dm-kcopyd.c SelvaKumar S   2021-02-19  606  	if (count == job->num_dests) {
f79939bb63c402 drivers/md/dm-kcopyd.c SelvaKumar S   2021-02-19  607  		push(&job->kc->complete_jobs, job);
f79939bb63c402 drivers/md/dm-kcopyd.c SelvaKumar S   2021-02-19  608  	} else {
f79939bb63c402 drivers/md/dm-kcopyd.c SelvaKumar S   2021-02-19  609  		push(&job->kc->pages_jobs, job);
f79939bb63c402 drivers/md/dm-kcopyd.c SelvaKumar S   2021-02-19  610  		r = 0;
f79939bb63c402 drivers/md/dm-kcopyd.c SelvaKumar S   2021-02-19  611  	}
f79939bb63c402 drivers/md/dm-kcopyd.c SelvaKumar S   2021-02-19  612  
f79939bb63c402 drivers/md/dm-kcopyd.c SelvaKumar S   2021-02-19 @613  	return r;
f79939bb63c402 drivers/md/dm-kcopyd.c SelvaKumar S   2021-02-19  614  }
f79939bb63c402 drivers/md/dm-kcopyd.c SelvaKumar S   2021-02-19  615  

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 35671 bytes --]

             reply	other threads:[~2021-02-20  5:52 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-02-20  5:52 kernel test robot [this message]
     [not found] <CGME20210219124611epcas5p1c775b63b537e75da161556e375fcf05e@epcas5p1.samsung.com>
2021-02-19 12:45 ` [dm-devel] [RFC PATCH v5 0/4] add simple copy support SelvaKumar S
2021-02-19 12:45   ` [RFC PATCH v5 4/4] dm kcopyd: add simple copy offload support SelvaKumar S
2021-02-19 12:45     ` SelvaKumar S
2021-02-22  5:28     ` Dan Carpenter
2021-02-22  5:28       ` Dan Carpenter

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=202102201309.E8VcBvvn-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=kbuild@lists.01.org \
    /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.