[ovs-dev] [PATCH] ovs-vswitchd: Fix high cpu utilization when acquire idl lock fails.

Alex Wang alexw at nicira.com
Thu Oct 9 00:13:59 UTC 2014


On Wed, Oct 8, 2014 at 4:13 PM, Ben Pfaff <blp at nicira.com> wrote:

> On Wed, Oct 08, 2014 at 03:19:29PM -0700, Alex Wang wrote:
> > When ovs-vswitchd fails to acquire the ovsdb idl lock (either due to
> > contention or due to invalid database path), ovs-vswitchd will spin
> > on the global connectivity sequence number and consume 100% cpu.
> > This is in that the local copy is different to the global sequence
> > number and never updated when ovsdb idl is not locked.
> >
> > To fix this issue, this commit makes ovs-vswitchd not checking the
> > global connectivity sequence number in that situation.
> >
> > Signed-off-by: Alex Wang <alexw at nicira.com>
>
> Do we need the new variable?  Can we just call ovsdb_idl_has_lock()
> from status_update_wait?
>

Thanks for pointing it out,  that is really clean!



More information about the dev mailing list