| Date | Tue, 20 Jul 2021 13:45:21 +0300 | From | Mike Rapoport <> | Subject | Re: [PATCH v14 022/138] mm/filemap: Add __folio_lock_or_retry() |
| |
On Thu, Jul 15, 2021 at 04:35:08AM +0100, Matthew Wilcox (Oracle) wrote: > Convert __lock_page_or_retry() to __folio_lock_or_retry(). This actually > saves 4 bytes in the only caller of lock_page_or_retry() (due to better > register allocation) and saves the 14 byte cost of calling page_folio() > in __folio_lock_or_retry() for a total saving of 18 bytes. Also use > a bool for the return type. > > Signed-off-by: Matthew Wilcox (Oracle) <willy@infradead.org> > Reviewed-by: Christoph Hellwig <hch@lst.de> > Acked-by: Jeff Layton <jlayton@kernel.org> > Acked-by: Kirill A. Shutemov <kirill.shutemov@linux.intel.com> > Reviewed-by: William Kucharski <william.kucharski@oracle.com> > --- > include/linux/pagemap.h | 11 +++++++---- > mm/filemap.c | 20 +++++++++----------- > mm/memory.c | 8 ++++---- > 3 files changed, 20 insertions(+), 19 deletions(-)
Acked-by: Mike Rapoport <rppt@linux.ibm.com>
|