From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Tue, 5 Dec 2017 16:01:23 -0700 From: Ross Zwisler To: Matthew Wilcox Cc: linux-mm@kvack.org, "zhangyi (F)" , linux-fsdevel@vger.kernel.org Subject: Re: [PATCH v2] mm: Add unmap_mapping_pages Message-ID: <20171205230123.GB20978@linux.intel.com> References: <20171205154453.GD28760@bombadil.infradead.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20171205154453.GD28760@bombadil.infradead.org> Sender: owner-linux-mm@kvack.org List-ID: On Tue, Dec 05, 2017 at 07:44:53AM -0800, Matthew Wilcox wrote: > v2: > - Fix inverted mask in dax.c > - Pass 'false' instead of '0' for 'only_cows' > - nommu definition > > >From ceee2e58548a5264b61000c02371956a1da3bee4 Mon Sep 17 00:00:00 2001 > From: Matthew Wilcox > Date: Tue, 5 Dec 2017 00:15:54 -0500 > Subject: [PATCH] mm: Add unmap_mapping_pages > > Several users of unmap_mapping_range() would much prefer to express > their range in pages rather than bytes. Unfortuately, on a 32-bit > kernel, you have to remember to cast your page number to a 64-bit type > before shifting it, and four places in the current tree didn't remember > to do that. That's a sign of a bad interface. > > Conveniently, unmap_mapping_range() actually converts from bytes into > pages, so hoist the guts of unmap_mapping_range() into the new function > unmap_mapping_pages() and convert the callers which want to use pages. > > Signed-off-by: Matthew Wilcox > Reported-by: "zhangyi (F)" Looks good. You can add: Reviewed-by: Ross Zwisler -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org