Re: [PATCH v2 5/5] zram: use generic PAGE_SECTORS and PAGE_SECTORS_SHIFT

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Subject: Re: [PATCH v2 5/5] zram: use generic PAGE_SECTORS and PAGE_SECTORS_SHIFT
From: Johannes Thumshirn <Johannes.Thumshirn@xxxxxxx>
Date: Fri, 26 May 2023 09:24:14 +0000
Accept-language: en-US
Cc: "p.raghav@xxxxxxxxxxx" <p.raghav@xxxxxxxxxxx>, "linux-xfs@xxxxxxxxxxxxxxx" <linux-xfs@xxxxxxxxxxxxxxx>, "rohan.puri@xxxxxxxxxxx" <rohan.puri@xxxxxxxxxxx>, "da.gomez@xxxxxxxxxxx" <da.gomez@xxxxxxxxxxx>, "patches@xxxxxxxxxxxxxxx" <patches@xxxxxxxxxxxxxxx>, "willy@xxxxxxxxxxxxx" <willy@xxxxxxxxxxxxx>, "linux-kernel@xxxxxxxxxxxxxxx" <linux-kernel@xxxxxxxxxxxxxxx>, "linux-block@xxxxxxxxxxxxxxx" <linux-block@xxxxxxxxxxxxxxx>, "linux-mm@xxxxxxxxx" <linux-mm@xxxxxxxxx>, "dm-devel@xxxxxxxxxx" <dm-devel@xxxxxxxxxx>, "linux-fsdevel@xxxxxxxxxxxxxxx" <linux-fsdevel@xxxxxxxxxxxxxxx>, "rpuri.linux@xxxxxxxxx" <rpuri.linux@xxxxxxxxx>, "kbusch@xxxxxxxxxx" <kbusch@xxxxxxxxxx>, "drbd-dev@xxxxxxxxxxxxxxxx" <drbd-dev@xxxxxxxxxxxxxxxx>
In-reply-to: <20230526073336.344543-6-mcgrof@kernel.org>
References: <20230526073336.344543-1-mcgrof@kernel.org> <20230526073336.344543-6-mcgrof@kernel.org>
User-agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:102.0) Gecko/20100101 Thunderbird/102.11.0
Wdcipoutbound: EOP-TRUE

On 26.05.23 09:34, Luis Chamberlain wrote:
> +	u32 index = bio->bi_iter.bi_sector >> PAGE_SECTORS_SHIFT;
> +	u32 offset = (bio->bi_iter.bi_sector & (PAGE_SECTORS - 1)) <<
>  			SECTOR_SHIFT;

(PAGE_SECTORS - 1) is SECTOR_MASK, please use this.

Thanks,
	Johannes
--
dm-devel mailing list
dm-devel@xxxxxxxxxx
https://listman.redhat.com/mailman/listinfo/dm-devel

References:
- [PATCH v2 0/5] block: simplify with PAGE_SECTORS_SHIFT
  - From: Luis Chamberlain
- [PATCH v2 5/5] zram: use generic PAGE_SECTORS and PAGE_SECTORS_SHIFT
  - From: Luis Chamberlain

Prev by Date: Re: [PATCH v2 4/5] dm bufio: simplify by using PAGE_SECTORS_SHIFT
Next by Date: Re: [PATCH v2 0/5] block: simplify with PAGE_SECTORS_SHIFT
Previous by thread: [PATCH v2 5/5] zram: use generic PAGE_SECTORS and PAGE_SECTORS_SHIFT
Next by thread: [PATCH v2 4/5] dm bufio: simplify by using PAGE_SECTORS_SHIFT
Index(es):
- Date
- Thread

[Index of Archives] [DM Crypt] [Fedora Desktop] [ATA RAID] [Fedora Marketing] [Fedora Packaging] [Fedora SELinux] [Yosemite Discussion] [KDE Users] [Fedora Docs]