On 10/1/22 04:38, Bart Van Assche wrote: > On 9/30/22 08:13, Jens Axboe wrote: >> On 9/29/22 12:31 AM, Pankaj Raghav wrote: >>>> Hi Jens, >>>> Please consider this patch series for the 6.1 release. >>>> >>> >>> Hi Jens, Christoph, and Keith, >>> All the patches have a Reviewed-by tag at this point. Can we queue this up >>> for 6.1? >> >> It's getting pretty late for 6.1 and I'd really like to have both Christoph >> and Martin sign off on these changes. > > Hi Jens, > > Agreed that it's getting late for 6.1. > > Since this has not been mentioned in the cover letter, I want to add > that in the near future we will need these patches for Android devices. > JEDEC is working on supporting zoned storage for UFS devices, the > storage devices used in all modern Android phones. Although it would be > possible to make the offset between zone starts a power of two by > inserting gap zones between data zones, UFS vendors asked not to do this > and hence need support for zone sizes that are not a power of two. An > advantage of not having to deal with gap zones is better filesystem > performance since filesystem extents cannot span gap zones. Having to > split filesystem extents because of gap zones reduces filesystem > performance.
As mentioned many times, my opinion is that a good implementation should *not* have any extent span zone boundaries. So personally, I do not consider such argument as a valid justification for the non-power-of-2 zone size support. > > Thanks, > > Bart. > > -- Damien Le Moal Western Digital Research -- dm-devel mailing list dm-devel@redhat.com https://listman.redhat.com/mailman/listinfo/dm-devel
