lkml.org 
[lkml]   [2022]   [Jul]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    SubjectRe: [PATCH v8 09/11] dm-table: allow non po2 zoned devices
    From
    On 7/28/22 01:22, Pankaj Raghav wrote:
    > As the block layer now supports non po2 zoned devices, allow dm to
    > support non po2 zoned device.

    Please rephrase "non po2 zoned devices" here and in the title to correctly
    refer to the zone size of zoned devices. Because "non po2 zoned devices"
    means absolutely nothing. Let's be clear please.

    >
    > Signed-off-by: Pankaj Raghav <p.raghav@samsung.com>
    > ---
    > drivers/md/dm-table.c | 8 ++++----
    > 1 file changed, 4 insertions(+), 4 deletions(-)
    >
    > diff --git a/drivers/md/dm-table.c b/drivers/md/dm-table.c
    > index 332f96b58252..534fddfc2b42 100644
    > --- a/drivers/md/dm-table.c
    > +++ b/drivers/md/dm-table.c
    > @@ -250,7 +250,7 @@ static int device_area_is_invalid(struct dm_target *ti, struct dm_dev *dev,
    > if (bdev_is_zoned(bdev)) {
    > unsigned int zone_sectors = bdev_zone_sectors(bdev);
    >
    > - if (start & (zone_sectors - 1)) {
    > + if (!bdev_is_zone_aligned(bdev, start)) {
    > DMWARN("%s: start=%llu not aligned to h/w zone size %u of %pg",
    > dm_device_name(ti->table->md),
    > (unsigned long long)start,
    > @@ -267,7 +267,7 @@ static int device_area_is_invalid(struct dm_target *ti, struct dm_dev *dev,
    > * devices do not end up with a smaller zone in the middle of
    > * the sector range.
    > */
    > - if (len & (zone_sectors - 1)) {
    > + if (!bdev_is_zone_aligned(bdev, len)) {
    > DMWARN("%s: len=%llu not aligned to h/w zone size %u of %pg",
    > dm_device_name(ti->table->md),
    > (unsigned long long)len,
    > @@ -1642,8 +1642,8 @@ static int validate_hardware_zoned_model(struct dm_table *t,
    > return -EINVAL;
    > }
    >
    > - /* Check zone size validity and compatibility */
    > - if (!zone_sectors || !is_power_of_2(zone_sectors))
    > + /* Check zone size validity */

    The comment is not super useful now given the trivial test.

    > + if (!zone_sectors)
    > return -EINVAL;
    >
    > if (dm_table_any_dev_attr(t, device_not_matches_zone_sectors, &zone_sectors)) {


    --
    Damien Le Moal
    Western Digital Research

    \
     
     \ /
      Last update: 2022-07-28 05:24    [W:3.620 / U:0.032 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site