[ovs-dev] [PATCH] bond: Use correct type for slave's change_seq.

Jarno Rajahalme jarno at ovn.org
Fri Dec 4 19:11:46 UTC 2015


> On Dec 3, 2015, at 10:57 PM, Ben Pfaff <blp at ovn.org> wrote:
> 
> On Thu, Dec 03, 2015 at 10:23:19PM -0800, Ben Pfaff wrote:
>> On Thu, Dec 03, 2015 at 05:59:12PM -0800, Jarno Rajahalme wrote:
>>> seq values are 64-bit, and storing them to a 32-bit variable causes
>>> the stored value never to match actual seq value after the seq value
>>> gets big enough.
>>> 
>>> This is a likely cause of OVS main thread using 100% CPU in a system
>>> using bonds after some runtime.
>>> 
>>> VMware-BZ: #1564993
>>> Reported-by: Hiram Bayless <hbayless at vmware.com>
>>> Signed-off-by: Jarno Rajahalme <jarno at ovn.org>
>> 
>> Great work!  Thank you.
> 
> Also,
> Acked-by: Ben Pfaff <blp at ovn.org>

Thanks for the reviews, pushed to master and branches 2.1-2.5. This bug is not present in branch-2.0 or earlier.

  Jarno




More information about the dev mailing list