Re: [PATCH 3/5] iomap: simplify iomap_init() with PAGE_SECTORS

From: Matthew Wilcox
Date: Fri Apr 21 2023 - 16:14:25 EST


On Fri, Apr 21, 2023 at 12:58:05PM -0700, Luis Chamberlain wrote:
> Just use the PAGE_SECTORS generic define. This produces no functional
> changes. While at it use left shift to simplify this even further.

How is FOO << 2 simpler than FOO * 4?

> - return bioset_init(&iomap_ioend_bioset, 4 * (PAGE_SIZE / SECTOR_SIZE),
> + return bioset_init(&iomap_ioend_bioset, PAGE_SECTORS << 2,