[bvanassche:block-bitwise-opf 38/64] drivers/nvme/target/zns.c:559:35: error: use of undeclared identifier 'op'
From: kernel test robot
Date: Wed Jul 13 2022 - 22:42:12 EST
tree: https://github.com/bvanassche/linux block-bitwise-opf
head: a18eedf7d10c719e2da2a802964f8aaade09e72c
commit: 5d612f7990f6438fe5d09c30560c0e25a1d292a2 [38/64] nvme/target: Use the new blk_opf_t type
config: hexagon-randconfig-r041-20220713 (https://download.01.org/0day-ci/archive/20220714/202207141043.jpp94dQ8-lkp@xxxxxxxxx/config)
compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project 5e61b9c556267086ef9b743a0b57df302eef831b)
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/bvanassche/linux/commit/5d612f7990f6438fe5d09c30560c0e25a1d292a2
git remote add bvanassche https://github.com/bvanassche/linux
git fetch --no-tags bvanassche block-bitwise-opf
git checkout 5d612f7990f6438fe5d09c30560c0e25a1d292a2
# 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 drivers/nvme/target/
If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <lkp@xxxxxxxxx>
All errors (new ones prefixed by >>):
>> drivers/nvme/target/zns.c:559:35: error: use of undeclared identifier 'op'
ARRAY_SIZE(req->inline_bvec), op);
^
drivers/nvme/target/zns.c:561:47: error: use of undeclared identifier 'op'
bio = bio_alloc(req->ns->bdev, req->sg_cnt, op, GFP_KERNEL);
^
2 errors generated.
vim +/op +559 drivers/nvme/target/zns.c
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 524
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 525 void nvmet_bdev_execute_zone_append(struct nvmet_req *req)
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 526 {
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 527 sector_t sect = nvmet_lba_to_sect(req->ns, req->cmd->rw.slba);
5d612f7990f643 Bart Van Assche 2022-06-17 528 const blk_opf_t opf = REQ_OP_ZONE_APPEND | REQ_SYNC | REQ_IDLE;
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 529 u16 status = NVME_SC_SUCCESS;
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 530 unsigned int total_len = 0;
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 531 struct scatterlist *sg;
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 532 struct bio *bio;
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 533 int sg_cnt;
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 534
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 535 /* Request is completed on len mismatch in nvmet_check_transter_len() */
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 536 if (!nvmet_check_transfer_len(req, nvmet_rw_data_len(req)))
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 537 return;
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 538
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 539 if (!req->sg_cnt) {
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 540 nvmet_req_complete(req, 0);
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 541 return;
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 542 }
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 543
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 544 if (sect >= get_capacity(req->ns->bdev->bd_disk)) {
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 545 req->error_loc = offsetof(struct nvme_rw_command, slba);
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 546 status = NVME_SC_LBA_RANGE | NVME_SC_DNR;
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 547 goto out;
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 548 }
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 549
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 550 if (sect & (bdev_zone_sectors(req->ns->bdev) - 1)) {
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 551 req->error_loc = offsetof(struct nvme_rw_command, slba);
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 552 status = NVME_SC_INVALID_FIELD | NVME_SC_DNR;
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 553 goto out;
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 554 }
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 555
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 556 if (nvmet_use_inline_bvec(req)) {
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 557 bio = &req->z.inline_bio;
49add4966d7924 Christoph Hellwig 2022-01-24 558 bio_init(bio, req->ns->bdev, req->inline_bvec,
49add4966d7924 Christoph Hellwig 2022-01-24 @559 ARRAY_SIZE(req->inline_bvec), op);
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 560 } else {
07888c665b405b Christoph Hellwig 2022-01-24 561 bio = bio_alloc(req->ns->bdev, req->sg_cnt, op, GFP_KERNEL);
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 562 }
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 563
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 564 bio->bi_end_io = nvmet_bdev_zone_append_bio_done;
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 565 bio->bi_iter.bi_sector = sect;
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 566 bio->bi_private = req;
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 567 if (req->cmd->rw.control & cpu_to_le16(NVME_RW_FUA))
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 568 bio->bi_opf |= REQ_FUA;
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 569
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 570 for_each_sg(req->sg, sg, req->sg_cnt, sg_cnt) {
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 571 struct page *p = sg_page(sg);
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 572 unsigned int l = sg->length;
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 573 unsigned int o = sg->offset;
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 574 unsigned int ret;
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 575
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 576 ret = bio_add_zone_append_page(bio, p, l, o);
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 577 if (ret != sg->length) {
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 578 status = NVME_SC_INTERNAL;
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 579 goto out_put_bio;
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 580 }
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 581 total_len += sg->length;
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 582 }
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 583
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 584 if (total_len != nvmet_rw_data_len(req)) {
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 585 status = NVME_SC_INTERNAL | NVME_SC_DNR;
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 586 goto out_put_bio;
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 587 }
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 588
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 589 submit_bio(bio);
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 590 return;
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 591
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 592 out_put_bio:
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 593 nvmet_req_bio_put(req, bio);
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 594 out:
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 595 nvmet_req_complete(req, status);
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 596 }
aaf2e048af2704 Chaitanya Kulkarni 2021-06-09 597
:::::: The code at line 559 was first introduced by commit
:::::: 49add4966d79244013fce35f95c6833fae82b8b1 block: pass a block_device and opf to bio_init
:::::: TO: Christoph Hellwig <hch@xxxxxx>
:::::: CC: Jens Axboe <axboe@xxxxxxxxx>
--
0-DAY CI Kernel Test Service
https://01.org/lkp