On Sun, Jan 02, 2022 at 05:32:28PM -0800, Hugh Dickins wrote: > > + same_folio = (lstart >> PAGE_SHIFT) == (lend >> PAGE_SHIFT); Should this move to the else branch? Same for the other copy of this code. Otherwise this looks sane.
On Sun, Jan 02, 2022 at 05:32:28PM -0800, Hugh Dickins wrote: > > + same_folio = (lstart >> PAGE_SHIFT) == (lend >> PAGE_SHIFT); Should this move to the else branch? Same for the other copy of this code. Otherwise this looks sane.