[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <ad5c229c-4c7f-e9ff-3326-477aad226388@wdc.com>
Date: Fri, 26 May 2023 09:21:58 +0000
From: Johannes Thumshirn <Johannes.Thumshirn@....com>
To: Luis Chamberlain <mcgrof@...nel.org>,
"axboe@...nel.dk" <axboe@...nel.dk>,
"agk@...hat.com" <agk@...hat.com>,
"snitzer@...nel.org" <snitzer@...nel.org>,
"philipp.reisner@...bit.com" <philipp.reisner@...bit.com>,
"lars.ellenberg@...bit.com" <lars.ellenberg@...bit.com>,
"christoph.boehmwalder@...bit.com" <christoph.boehmwalder@...bit.com>,
"hch@...radead.org" <hch@...radead.org>,
"djwong@...nel.org" <djwong@...nel.org>,
"minchan@...nel.org" <minchan@...nel.org>,
"senozhatsky@...omium.org" <senozhatsky@...omium.org>
CC: "patches@...ts.linux.dev" <patches@...ts.linux.dev>,
"linux-block@...r.kernel.org" <linux-block@...r.kernel.org>,
"linux-mm@...ck.org" <linux-mm@...ck.org>,
"linux-xfs@...r.kernel.org" <linux-xfs@...r.kernel.org>,
"linux-fsdevel@...r.kernel.org" <linux-fsdevel@...r.kernel.org>,
"dm-devel@...hat.com" <dm-devel@...hat.com>,
"drbd-dev@...ts.linbit.com" <drbd-dev@...ts.linbit.com>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"willy@...radead.org" <willy@...radead.org>,
"hare@...e.de" <hare@...e.de>,
"p.raghav@...sung.com" <p.raghav@...sung.com>,
"da.gomez@...sung.com" <da.gomez@...sung.com>,
"rohan.puri@...sung.com" <rohan.puri@...sung.com>,
"rpuri.linux@...il.com" <rpuri.linux@...il.com>,
"kbusch@...nel.org" <kbusch@...nel.org>
Subject: Re: [PATCH v2 4/5] dm bufio: simplify by using PAGE_SECTORS_SHIFT
On 26.05.23 09:33, Luis Chamberlain wrote:
> *data_mode = DATA_MODE_GET_FREE_PAGES;
> return (void *)__get_free_pages(gfp_mask,
> - c->sectors_per_block_bits - (PAGE_SHIFT - SECTOR_SHIFT));
> + c->sectors_per_block_bits - (PAGE_SECTORS_SHIFT));
> case DATA_MODE_GET_FREE_PAGES:
> free_pages((unsigned long)data,
> - c->sectors_per_block_bits - (PAGE_SHIFT - SECTOR_SHIFT));
> + c->sectors_per_block_bits - (PAGE_SECTORS_SHIFT));
The parenthesis are completely unneeded in the new version,
please remove them.
Powered by blists - more mailing lists