Re: [PATCH 16/16] cifs: Use find_get_pages_range_tag()

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

To: Jan Kara <jack@xxxxxxx>
Subject: Re: [PATCH 16/16] cifs: Use find_get_pages_range_tag()
From: Daniel Jordan <daniel.m.jordan@xxxxxxxxxx>
Date: Tue, 10 Oct 2017 13:48:24 -0400
Cc: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>, linux-mm@xxxxxxxxx, linux-fsdevel@xxxxxxxxxxxxxxx, linux-cifs@xxxxxxxxxxxxxxx, Steve French <sfrench@xxxxxxxxx>
In-reply-to: <20171009151359.31984-17-jack@suse.cz>
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0

Hi Jan,

On 10/09/2017 11:13 AM, Jan Kara wrote:

wdata_alloc_and_fillpages() needlessly iterates calls to
find_get_pages_tag(). Also it wants only pages from given range. Make it
use find_get_pages_range_tag().


Looks good, so

Reviewed-by: Daniel Jordan <daniel.m.jordan@xxxxxxxxxx>

The rest of the v3 updates seem fine, too.

Daniel

References:
- [PATCH 0/16 v3] Ranged pagevec tagged lookup
  - From: Jan Kara
- [PATCH 16/16] cifs: Use find_get_pages_range_tag()
  - From: Jan Kara

Prev by Date: Re: [PATCH v7 07/12] dma-mapping: introduce dma_has_iommu()
Next by Date: Re: [PATCH v3] mm, sysctl: make NUMA stats configurable
Previous by thread: [PATCH 16/16] cifs: Use find_get_pages_range_tag()
Next by thread: [PATCH 06/16] f2fs: Simplify page iteration loops
Index(es):
- Date
- Thread

[Index of Archives] [Linux Ext4 Filesystem] [Union Filesystem] [Filesystem Testing] [Ceph Users] [Ecryptfs] [AutoFS] [Kernel Newbies] [Share Photos] [Security] [Netfilter] [Bugtraq] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Linux Cachefs] [Reiser Filesystem] [Linux RAID] [Samba] [Device Mapper] [CEPH Development]

Powered by Linux