On Thu 01-06-17 04:36:04, Christoph Hellwig wrote: > On Thu, Jun 01, 2017 at 11:32:10AM +0200, Jan Kara wrote: > > * Implement ranged variants for pagevec_lookup and find_get_ functions. Lot > > of callers actually want a ranged lookup and we unnecessarily opencode this > > in lot of them. > > How does this compare to Kents page cache iterators: > > http://www.spinics.net/lists/linux-mm/msg104737.html Interesting. I didn't know about that work. I guess the tradeoff is pretty obvious - my patches are more conservative (changing less) and as a result the API is not as neat as Kent's one. That being said I was also thinking about something similar to what Kent did but what I didn't like about such iterator is that you still need to specially handle the cases where you break out of the loop (you need to do that with pagevecs too but there it is kind of obvious from the API). Honza -- Jan Kara <jack@xxxxxxxx> SUSE Labs, CR -- To unsubscribe from this list: send the line "unsubscribe linux-xfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
