Re: [PATCH v11 21/33] mm/filemap: Add __folio_lock_async()
From: David Howells <dhowells@redhat.com>
Date: 2021-06-16 10:22:46
Also in:
linux-fsdevel, lkml
From: David Howells <dhowells@redhat.com>
Date: 2021-06-16 10:22:46
Also in:
linux-fsdevel, lkml
Matthew Wilcox (Oracle) [off-list ref] wrote:
There aren't any actual callers of lock_page_async(), so remove it. Convert filemap_update_page() to call __folio_lock_async(). __folio_lock_async() is 21 bytes smaller than __lock_page_async(), but the real savings come from using a folio in filemap_update_page(), shrinking it from 515 bytes to 404 bytes, saving 110 bytes. The text shrinks by 132 bytes in total. 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 <redacted> Acked-by: Vlastimil Babka <redacted> Reviewed-by: William Kucharski <redacted>
Reviewed-by: David Howells <dhowells@redhat.com>