From: kernel test robot <lkp@intel.com>
To: Pankaj Raghav <p.raghav@samsung.com>,
hch@lst.de, snitzer@redhat.com, damien.lemoal@opensource.wdc.com,
axboe@kernel.dk
Cc: kbuild-all@lists.01.org, bvanassche@acm.org,
linux-kernel@vger.kernel.org, jiangbo.365@bytedance.com,
hare@suse.de, pankydev8@gmail.com, dm-devel@redhat.com,
jonathan.derrick@linux.dev, gost.dev@samsung.com,
dsterba@suse.com, jaegeuk@kernel.org,
linux-nvme@lists.infradead.org, Johannes.Thumshirn@wdc.com,
linux-block@vger.kernel.org, Pankaj Raghav <p.raghav@samsung.com>,
Damien Le Moal <damien.lemoal@wdc.com>
Subject: Re: [PATCH v7 13/13] dm: add non power of 2 zoned target
Date: Wed, 15 Jun 2022 22:19:48 +0800 [thread overview]
Message-ID: <202206152257.pnoPyl7X-lkp@intel.com> (raw)
In-Reply-To: <20220615101920.329421-14-p.raghav@samsung.com>
Hi Pankaj,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on axboe-block/for-next]
[also build test WARNING on device-mapper-dm/for-next linus/master v5.19-rc2 next-20220615]
[cannot apply to song-md/md-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/intel-lab-lkp/linux/commits/Pankaj-Raghav/block-make-blkdev_nr_zones-and-blk_queue_zone_no-generic-for-npo2-zsze/20220615-191927
base: https://git.kernel.org/pub/scm/linux/kernel/git/axboe/linux-block.git for-next
config: s390-allmodconfig (https://download.01.org/0day-ci/archive/20220615/202206152257.pnoPyl7X-lkp@intel.com/config)
compiler: s390-linux-gcc (GCC) 11.3.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/intel-lab-lkp/linux/commit/add4ab54d5b34d4a2f91f241007f23a56c164fb3
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review Pankaj-Raghav/block-make-blkdev_nr_zones-and-blk_queue_zone_no-generic-for-npo2-zsze/20220615-191927
git checkout add4ab54d5b34d4a2f91f241007f23a56c164fb3
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.3.0 make.cross W=1 O=build_dir ARCH=s390 SHELL=/bin/bash
If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
drivers/md/dm-zoned-npo2-target.c: In function 'dmz_npo2_ctr':
>> drivers/md/dm-zoned-npo2-target.c:62:13: warning: variable 'ret' set but not used [-Wunused-but-set-variable]
62 | int ret = 0;
| ^~~
vim +/ret +62 drivers/md/dm-zoned-npo2-target.c
53
54 /*
55 * <dev-path>
56 * This target works on the complete zoned device. Partial mapping is not
57 * supported
58 */
59 static int dmz_npo2_ctr(struct dm_target *ti, unsigned int argc, char **argv)
60 {
61 struct dmz_npo2_target *dmh = NULL;
> 62 int ret = 0;
63 sector_t zsze;
64 sector_t disk_size;
65
66 if (argc < 1)
67 return -EINVAL;
68
69 dmh = kmalloc(sizeof(*dmh), GFP_KERNEL);
70 if (!dmh)
71 return -ENOMEM;
72
73 ret = dm_get_device(ti, argv[0], dm_table_get_mode(ti->table),
74 &dmh->dev);
75
76 zsze = blk_queue_zone_sectors(bdev_get_queue(dmh->dev->bdev));
77
78 disk_size = get_capacity(dmh->dev->bdev->bd_disk);
79
80 if (ti->len != disk_size || ti->begin) {
81 DMERR("%pg Partial mapping of the target not supported",
82 dmh->dev->bdev);
83 return -EINVAL;
84 }
85
86 if (is_power_of_2(zsze)) {
87 DMERR("%pg zone size is power of 2", dmh->dev->bdev);
88 return -EINVAL;
89 }
90
91 dmh->zsze = zsze;
92 dmh->zsze_po2 = 1 << get_count_order_long(zsze);
93 dmh->zsze_diff = dmh->zsze_po2 - dmh->zsze;
94
95 ti->private = dmh;
96 ti->num_flush_bios = 1;
97 ti->num_discard_bios = 1;
98 ti->num_secure_erase_bios = 1;
99 ti->num_write_zeroes_bios = 1;
100
101 dmh->nr_zones = npo2_zone_no(dmh, ti->len);
102 ti->len = dmh->zsze_po2 * dmh->nr_zones;
103
104 return 0;
105 }
106
--
0-DAY CI Kernel Test Service
https://01.org/lkp
WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com>
To: Pankaj Raghav <p.raghav@samsung.com>,
hch@lst.de, snitzer@redhat.com, damien.lemoal@opensource.wdc.com,
axboe@kernel.dk
Cc: Pankaj Raghav <p.raghav@samsung.com>,
Damien Le Moal <damien.lemoal@wdc.com>,
kbuild-all@lists.01.org, bvanassche@acm.org, pankydev8@gmail.com,
gost.dev@samsung.com, jiangbo.365@bytedance.com,
linux-nvme@lists.infradead.org, linux-kernel@vger.kernel.org,
linux-block@vger.kernel.org, dm-devel@redhat.com,
jonathan.derrick@linux.dev, Johannes.Thumshirn@wdc.com,
dsterba@suse.com, jaegeuk@kernel.org
Subject: Re: [dm-devel] [PATCH v7 13/13] dm: add non power of 2 zoned target
Date: Wed, 15 Jun 2022 22:19:48 +0800 [thread overview]
Message-ID: <202206152257.pnoPyl7X-lkp@intel.com> (raw)
In-Reply-To: <20220615101920.329421-14-p.raghav@samsung.com>
Hi Pankaj,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on axboe-block/for-next]
[also build test WARNING on device-mapper-dm/for-next linus/master v5.19-rc2 next-20220615]
[cannot apply to song-md/md-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/intel-lab-lkp/linux/commits/Pankaj-Raghav/block-make-blkdev_nr_zones-and-blk_queue_zone_no-generic-for-npo2-zsze/20220615-191927
base: https://git.kernel.org/pub/scm/linux/kernel/git/axboe/linux-block.git for-next
config: s390-allmodconfig (https://download.01.org/0day-ci/archive/20220615/202206152257.pnoPyl7X-lkp@intel.com/config)
compiler: s390-linux-gcc (GCC) 11.3.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/intel-lab-lkp/linux/commit/add4ab54d5b34d4a2f91f241007f23a56c164fb3
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review Pankaj-Raghav/block-make-blkdev_nr_zones-and-blk_queue_zone_no-generic-for-npo2-zsze/20220615-191927
git checkout add4ab54d5b34d4a2f91f241007f23a56c164fb3
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.3.0 make.cross W=1 O=build_dir ARCH=s390 SHELL=/bin/bash
If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
drivers/md/dm-zoned-npo2-target.c: In function 'dmz_npo2_ctr':
>> drivers/md/dm-zoned-npo2-target.c:62:13: warning: variable 'ret' set but not used [-Wunused-but-set-variable]
62 | int ret = 0;
| ^~~
vim +/ret +62 drivers/md/dm-zoned-npo2-target.c
53
54 /*
55 * <dev-path>
56 * This target works on the complete zoned device. Partial mapping is not
57 * supported
58 */
59 static int dmz_npo2_ctr(struct dm_target *ti, unsigned int argc, char **argv)
60 {
61 struct dmz_npo2_target *dmh = NULL;
> 62 int ret = 0;
63 sector_t zsze;
64 sector_t disk_size;
65
66 if (argc < 1)
67 return -EINVAL;
68
69 dmh = kmalloc(sizeof(*dmh), GFP_KERNEL);
70 if (!dmh)
71 return -ENOMEM;
72
73 ret = dm_get_device(ti, argv[0], dm_table_get_mode(ti->table),
74 &dmh->dev);
75
76 zsze = blk_queue_zone_sectors(bdev_get_queue(dmh->dev->bdev));
77
78 disk_size = get_capacity(dmh->dev->bdev->bd_disk);
79
80 if (ti->len != disk_size || ti->begin) {
81 DMERR("%pg Partial mapping of the target not supported",
82 dmh->dev->bdev);
83 return -EINVAL;
84 }
85
86 if (is_power_of_2(zsze)) {
87 DMERR("%pg zone size is power of 2", dmh->dev->bdev);
88 return -EINVAL;
89 }
90
91 dmh->zsze = zsze;
92 dmh->zsze_po2 = 1 << get_count_order_long(zsze);
93 dmh->zsze_diff = dmh->zsze_po2 - dmh->zsze;
94
95 ti->private = dmh;
96 ti->num_flush_bios = 1;
97 ti->num_discard_bios = 1;
98 ti->num_secure_erase_bios = 1;
99 ti->num_write_zeroes_bios = 1;
100
101 dmh->nr_zones = npo2_zone_no(dmh, ti->len);
102 ti->len = dmh->zsze_po2 * dmh->nr_zones;
103
104 return 0;
105 }
106
--
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
next prev parent reply other threads:[~2022-06-15 14:20 UTC|newest]
Thread overview: 74+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <CGME20220615101924eucas1p27fbce623c0e1b3097169bf23dd6266d8@eucas1p2.samsung.com>
2022-06-15 10:19 ` [PATCH v7 00/13] support non power of 2 zoned device Pankaj Raghav
2022-06-15 10:19 ` [dm-devel] " Pankaj Raghav
[not found] ` <CGME20220615101927eucas1p17220c7a36f69f59ff8ddd560b42967ec@eucas1p1.samsung.com>
2022-06-15 10:19 ` [PATCH v7 01/13] block: make blkdev_nr_zones and blk_queue_zone_no generic for npo2 zsze Pankaj Raghav
2022-06-15 10:19 ` [dm-devel] " Pankaj Raghav
2022-06-15 20:18 ` Bart Van Assche
2022-06-15 20:18 ` Bart Van Assche
[not found] ` <CGME20220615101931eucas1p15ed09ae433a2c378b599e9086130d8eb@eucas1p1.samsung.com>
2022-06-15 10:19 ` [PATCH v7 02/13] block: allow blk-zoned devices to have non-power-of-2 zone size Pankaj Raghav
2022-06-15 10:19 ` [dm-devel] " Pankaj Raghav
2022-06-15 20:28 ` Bart Van Assche
2022-06-15 20:28 ` Bart Van Assche
2022-06-16 10:09 ` Pankaj Raghav
2022-06-16 10:09 ` Pankaj Raghav
2022-06-16 16:04 ` Luis Chamberlain
2022-06-16 16:04 ` Luis Chamberlain
2022-06-16 23:30 ` Damien Le Moal
2022-06-16 23:30 ` Damien Le Moal
[not found] ` <CGME20220615101935eucas1p26a7bc245d88a89312158d7a265f64aef@eucas1p2.samsung.com>
2022-06-15 10:19 ` [PATCH v7 03/13] nvme: zns: Allow ZNS drives that have non-power_of_2 " Pankaj Raghav
2022-06-15 10:19 ` [dm-devel] " Pankaj Raghav
[not found] ` <CGME20220615101938eucas1p26ab159a1ffd0fa5a16d7f202ba7206e7@eucas1p2.samsung.com>
2022-06-15 10:19 ` [PATCH v7 04/13] nvmet: Allow ZNS target to support non-power_of_2 zone sizes Pankaj Raghav
2022-06-15 10:19 ` [dm-devel] " Pankaj Raghav
[not found] ` <CGME20220615101941eucas1p25e1c27b363e6b288b848521298e31705@eucas1p2.samsung.com>
2022-06-15 10:19 ` [PATCH v7 05/13] null_blk: allow non power of 2 zoned devices Pankaj Raghav
2022-06-15 10:19 ` [dm-devel] " Pankaj Raghav
[not found] ` <CGME20220615101945eucas1p16fa264e81d9b6027ff131dd311ed91e2@eucas1p1.samsung.com>
2022-06-15 10:19 ` [PATCH v7 06/13] null_blk: use zone_size_sects_shift for " Pankaj Raghav
2022-06-15 10:19 ` [dm-devel] " Pankaj Raghav
2022-06-15 11:56 ` Damien Le Moal
2022-06-15 11:56 ` [dm-devel] " Damien Le Moal
[not found] ` <CGME20220615101948eucas1p2d8d801735c39b25256a019134adb0c6f@eucas1p2.samsung.com>
2022-06-15 10:19 ` [PATCH v7 07/13] zonefs: allow non " Pankaj Raghav
2022-06-15 10:19 ` [dm-devel] " Pankaj Raghav
[not found] ` <CGME20220615101951eucas1p238eb45e563bd9645af81bf16c56d98ec@eucas1p2.samsung.com>
2022-06-15 10:19 ` [PATCH v7 08/13] dm-zoned: ensure only power of 2 zone sizes are allowed Pankaj Raghav
2022-06-15 10:19 ` [dm-devel] " Pankaj Raghav
[not found] ` <CGME20220615101955eucas1p19b9d42ead7331f69f7dad1ec100312c2@eucas1p1.samsung.com>
2022-06-15 10:19 ` [dm-devel] [PATCH v7 09/13] dm-zone: use generic helpers to calculate offset from zone start Pankaj Raghav
2022-06-15 10:19 ` Pankaj Raghav
[not found] ` <CGME20220615102000eucas1p27720aaa3c309327b2b9a33c5f840f498@eucas1p2.samsung.com>
2022-06-15 10:19 ` [dm-devel] [PATCH v7 10/13] dm-table: use bdev_is_zone_start helper in device_area_is_invalid() Pankaj Raghav
2022-06-15 10:19 ` Pankaj Raghav
2022-06-15 11:53 ` Damien Le Moal
2022-06-15 11:53 ` [dm-devel] " Damien Le Moal
2022-06-16 9:55 ` Pankaj Raghav
2022-06-16 9:55 ` [dm-devel] " Pankaj Raghav
2022-06-16 23:29 ` Damien Le Moal
2022-06-16 23:29 ` [dm-devel] " Damien Le Moal
[not found] ` <CGME20220615102004eucas1p1e458ea097d381058b16fc6daa3eec998@eucas1p1.samsung.com>
2022-06-15 10:19 ` [dm-devel] [PATCH v7 11/13] dm-table: allow non po2 zoned devices Pankaj Raghav
2022-06-15 10:19 ` Pankaj Raghav
[not found] ` <CGME20220615102007eucas1p1106f9520e2a86beb3792107dffd8071b@eucas1p1.samsung.com>
2022-06-15 10:19 ` [dm-devel] [PATCH v7 12/13] dm: call dm_zone_endio after the target endio callback for " Pankaj Raghav
2022-06-15 10:19 ` Pankaj Raghav
2022-06-15 11:01 ` [dm-devel] " Damien Le Moal
2022-06-15 11:01 ` Damien Le Moal
2022-06-16 12:24 ` Pankaj Raghav
2022-06-16 12:24 ` Pankaj Raghav
2022-06-16 23:33 ` Damien Le Moal
2022-06-16 23:33 ` Damien Le Moal
[not found] ` <CGME20220615102011eucas1p220368db4a186181b1927dea50a79e5d4@eucas1p2.samsung.com>
2022-06-15 10:19 ` [PATCH v7 13/13] dm: add non power of 2 zoned target Pankaj Raghav
2022-06-15 10:19 ` [dm-devel] " Pankaj Raghav
2022-06-15 11:49 ` Damien Le Moal
2022-06-15 11:49 ` [dm-devel] " Damien Le Moal
2022-06-16 16:12 ` Pankaj Raghav
2022-06-16 16:12 ` [dm-devel] " Pankaj Raghav
2022-06-16 23:49 ` Damien Le Moal
2022-06-16 23:49 ` [dm-devel] " Damien Le Moal
2022-06-17 5:45 ` Pankaj Raghav
2022-06-17 5:45 ` [dm-devel] " Pankaj Raghav
2022-06-17 6:12 ` Damien Le Moal
2022-06-17 6:12 ` [dm-devel] " Damien Le Moal
2022-06-17 6:40 ` Pankaj Raghav
2022-06-17 6:40 ` [dm-devel] " Pankaj Raghav
2022-06-17 6:56 ` Damien Le Moal
2022-06-17 6:56 ` [dm-devel] " Damien Le Moal
2022-06-17 8:03 ` Pankaj Raghav
2022-06-17 8:03 ` [dm-devel] " Pankaj Raghav
2022-06-15 14:19 ` kernel test robot [this message]
2022-06-15 14:19 ` kernel test robot
2022-06-15 19:54 ` Randy Dunlap
2022-06-15 19:54 ` [dm-devel] " Randy Dunlap
2022-06-16 10:28 ` Pankaj Raghav
2022-06-16 10:28 ` [dm-devel] " Pankaj Raghav
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=202206152257.pnoPyl7X-lkp@intel.com \
--to=lkp@intel.com \
--cc=Johannes.Thumshirn@wdc.com \
--cc=axboe@kernel.dk \
--cc=bvanassche@acm.org \
--cc=damien.lemoal@opensource.wdc.com \
--cc=damien.lemoal@wdc.com \
--cc=dm-devel@redhat.com \
--cc=dsterba@suse.com \
--cc=gost.dev@samsung.com \
--cc=hare@suse.de \
--cc=hch@lst.de \
--cc=jaegeuk@kernel.org \
--cc=jiangbo.365@bytedance.com \
--cc=jonathan.derrick@linux.dev \
--cc=kbuild-all@lists.01.org \
--cc=linux-block@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-nvme@lists.infradead.org \
--cc=p.raghav@samsung.com \
--cc=pankydev8@gmail.com \
--cc=snitzer@redhat.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.