On Tue, May 9, 2017 at 4:51 PM, Eric Dumazet <eric.duma...@gmail.com> wrote: > On Tue, 2017-05-09 at 16:35 -0700, Cong Wang wrote: > >> This statement is only used to ensure we pass the "dead == fi->fib_nhs" >> check right below the inner loop, it is fine to keep it without break since >> fi is not changed in the inner loop. >> > > So the dead++ above wont end up with (dead > fi->fib_nhs) ?
Good point, it could happen, we probably need another boolean to address this.