Re: [dm-devel] [PATCH v12 13/13] dm: add power-of-2 target for zoned devices with non power-of-2 zone sizes

2022-09-05 Thread Pankaj Raghav
>> >> 1) why is a partial mapping of the underlying device disallowed? >> 2) why is it assumed all IO is read-only? (talk to me and others like >>we don't know the inherent limitations of this class of zoned hw) >> >> On a code level: >> 1) are you certain you're properly failing all writes?

Re: [dm-devel] [PATCH v12 13/13] dm: add power-of-2 target for zoned devices with non power-of-2 zone sizes

2022-09-05 Thread Pankaj Raghav
Hi Mike, >> Note that the target does not support partial mapping of the underlying >> device. >> >> Signed-off-by: Pankaj Raghav >> Suggested-by: Johannes Thumshirn >> Suggested-by: Damien Le Moal >> Suggested-by: Hannes Reinecke > > > This target needs more review from those who

Re: [dm-devel] [PATCH v12 13/13] dm: add power-of-2 target for zoned devices with non power-of-2 zone sizes

2022-09-02 Thread Mike Snitzer
On Fri, Sep 02 2022 at 4:55P -0400, Mike Snitzer wrote: > On Tue, Aug 23 2022 at 8:18P -0400, > Pankaj Raghav wrote: > > > Only zoned devices with power-of-2(po2) number of sectors per zone(zone > > size) were supported in linux but now non power-of-2(npo2) zone sizes > > support has been

Re: [dm-devel] [PATCH v12 13/13] dm: add power-of-2 target for zoned devices with non power-of-2 zone sizes

2022-09-02 Thread Mike Snitzer
On Tue, Aug 23 2022 at 8:18P -0400, Pankaj Raghav wrote: > Only zoned devices with power-of-2(po2) number of sectors per zone(zone > size) were supported in linux but now non power-of-2(npo2) zone sizes > support has been added to the block layer. > > Filesystems such as F2FS and btrfs have

Re: [dm-devel] [PATCH v12 13/13] dm: add power-of-2 target for zoned devices with non power-of-2 zone sizes

2022-09-02 Thread Pankaj Raghav
On 2022-08-23 14:18, Pankaj Raghav wrote: > +static int dm_po2z_iterate_devices(struct dm_target *ti, > +iterate_devices_callout_fn fn, void *data) > +{ > + struct dm_po2z_target *dmh = ti->private; > + sector_t len = dmh->nr_zones * dmh->zone_size; > + > +

Re: [dm-devel] [PATCH v12 13/13] dm: add power-of-2 target for zoned devices with non power-of-2 zone sizes

2022-08-30 Thread Pankaj Raghav
On 2022-08-30 04:52, Shinichiro Kawasaki wrote: > On Aug 23, 2022 / 14:18, Pankaj Raghav wrote: >> Only zoned devices with power-of-2(po2) number of sectors per zone(zone >> size) were supported in linux but now non power-of-2(npo2) zone sizes >> support has been added to the block layer. >> >>

Re: [dm-devel] [PATCH v12 13/13] dm: add power-of-2 target for zoned devices with non power-of-2 zone sizes

2022-08-30 Thread Shinichiro Kawasaki
On Aug 23, 2022 / 14:18, Pankaj Raghav wrote: > Only zoned devices with power-of-2(po2) number of sectors per zone(zone > size) were supported in linux but now non power-of-2(npo2) zone sizes > support has been added to the block layer. > > Filesystems such as F2FS and btrfs have support for

[dm-devel] [PATCH v12 13/13] dm: add power-of-2 target for zoned devices with non power-of-2 zone sizes

2022-08-23 Thread Pankaj Raghav
Only zoned devices with power-of-2(po2) number of sectors per zone(zone size) were supported in linux but now non power-of-2(npo2) zone sizes support has been added to the block layer. Filesystems such as F2FS and btrfs have support for zoned devices with po2 zone size assumption. Before adding