Max Kellermann <max.kellermann@xxxxxxxxx> wrote: > prev = list_prev_entry(subreq, rreq_link); > ... > + if (subreq->start == prev->start + prev->len) { > + prev = list_prev_entry(subreq, rreq_link); Actually, that doubles the setting of prev redundantly. It shouldn't hurt, but we might want to remove the inner one. David