On Tue, Apr 20, 2021 at 04:12:57PM -0400, Jeff Layton wrote:
> > @@ -210,6 +208,7 @@ void page_cache_ra_unbounded(struct readahead_control *ractl,
> > * not worth getting one just for that.
> > */
> > read_pages(ractl, &page_pool, true);
> > + i = ractl->_index + ractl->_nr_pages - index;
i = ractl->_index + ractl->_nr_pages - index - 1;
> > @@ -223,6 +222,7 @@ void page_cache_ra_unbounded(struct readahead_control *ractl,
> > gfp_mask) < 0) {
> > put_page(page);
> > read_pages(ractl, &page_pool, true);
> > + i = ractl->_index + ractl->_nr_pages - index;
i = ractl->_index + ractl->_nr_pages - index - 1;
> Thanks Willy, but I think this may not be quite right. A kernel with
> this patch failed to boot for me:
Silly off-by-one errors. xfstests running against xfs is up to generic/278
with the off-by-one fixed.
