> > + /* snap to EOF */ > > + size += min(size, ra->ra_pages / 4); > I'd probably choose: > size += min(size / 2, ra->ra_pages / 4); > to increase current window only to 3/2 and not twice but I don't have a OK it looks good on large ra_pages. I'll use this form. > strong opinion. Otherwise I think the code is fine now so you can add: > Acked-by: Jan Kara <jack@xxxxxxx> Thanks! Fengguang -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html