On 6/16/19 6:28 PM, Chaitanya Kulkarni wrote:
Changes from V1:-

1. Drop the target_pscsi patch. (Bart)
2. Remove rcu locking which is not needed. (Bart)

Chaitanya Kulkarni (7):
   block: add a helper function to read nr_setcs
   blk-zoned: update blkdev_nr_zones() with helper
   blk-zoned: update blkdev_report_zone() with helper
   blk-zoned: update blkdev_reset_zones() with helper
   bcache: update cached_dev_init() with helper
   f2fs: use helper in init_blkz_info()
   blktrace: use helper in blk_trace_setup_lba()

  block/blk-zoned.c         | 12 ++++++------
  drivers/md/bcache/super.c |  2 +-
  fs/f2fs/super.c           |  2 +-
  include/linux/blkdev.h    | 10 ++++++++++
  kernel/trace/blktrace.c   |  2 +-
  5 files changed, 19 insertions(+), 9 deletions(-)

My feedback about the pscsi_get_blocks() was misleading: what I meant is that it is not necessary to introduce RCU locking in that function. I think that using bdev_nr_sects() or part_nr_sects_read() to read nr_sects in that function is useful.

Is there any reason that the following Xen macro has not been converted?

#define vbd_sz(_v)      ((_v)->bdev->bd_part ? \
                         (_v)->bdev->bd_part->nr_sects : \
                          get_capacity((_v)->bdev->bd_disk))

Thanks,

Bart.

Reply via email to