On 4/16/21 8:55 AM, Ido Schimmel wrote: > From: Ido Schimmel <[email protected]> > > Currently, a multi-part nexthop dump is restarted based on the number of > nexthops that have been dumped so far. This can result in a lot of > nexthops not being dumped when nexthops are simultaneously deleted: > > # ip nexthop | wc -l > 65536 > # ip nexthop flush > Dump was interrupted and may be inconsistent. > Flushed 36040 nexthops > # ip nexthop | wc -l > 29496 > > Instead, restart the dump based on the nexthop identifier (fixed number) > of the last successfully dumped nexthop: > > # ip nexthop | wc -l > 65536 > # ip nexthop flush > Dump was interrupted and may be inconsistent. > Flushed 65536 nexthops > # ip nexthop | wc -l > 0 > > Reported-by: Maksym Yaremchuk <[email protected]> > Tested-by: Maksym Yaremchuk <[email protected]> > Signed-off-by: Ido Schimmel <[email protected]> > Reviewed-by: Petr Machata <[email protected]> > --- > net/ipv4/nexthop.c | 14 ++++++-------- > 1 file changed, 6 insertions(+), 8 deletions(-) >
Reviewed-by: David Ahern <[email protected]> Any reason not to put this in -net with a Fixes tag?
