On Wed, 2016-10-05 at 09:06 -0400, Alex Sidorenko wrote: > Roundrobin runner of team driver uses 'unsigned int' variable to count the > number of sent_packets. > Later it is passed to a subroutine team_num_to_port_index(struct team *team, > int num) as > 'num' and when we reach MAXINT (2**31-1), 'num' becomes negative. > > This leads to using incorrect hash-bucket for port lookup and as a result, > packets are dropped. The fix > consists of changing 'int num' to 'unsigned int num'. Testing of a fixed > kernel shows that there > is no packet drop anymore. > > > Signed-off-by: Alex Sidorenko <alexandre.sidore...@hpe.com>
Note that lines in your changelog are longer than the norm ( Documentation/SubmittingPatches around line 619 ) - The body of the explanation, line wrapped at 75 columns, which will be copied to the permanent changelog to describe this patch. Otherwise, patch looks, welcome to the club Alex ! Acked-by: Eric Dumazet <eduma...@google.com>
