[ovs-git] [openvswitch/ovs] ba8ec1: bridge: Fix bug where IDL wakeup causes 100% CPU.

GitHub noreply at github.com
Mon Sep 29 18:53:33 UTC 2014


  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: ba8ec11566e9d1862d88723dff0ab2378677db37
      https://github.com/openvswitch/ovs/commit/ba8ec11566e9d1862d88723dff0ab2378677db37
  Author: Joe Stringer <joestringer at nicira.com>
  Date:   2014-09-30 (Tue, 30 Sep 2014)

  Changed paths:
    M vswitchd/bridge.c

  Log Message:
  -----------
  bridge: Fix bug where IDL wakeup causes 100% CPU.

Commit 9c537baf613a16e (bridge: Refactor the stats and status update.)
inadvertently changed the way that the status transaction is destroyed,
which could cause the main thread to constantly wake up.

The bug occurs when ovsdb_idl_txn_commit() returns TXN_INCOMPLETE and
there are no further changes to connectivity or 'status_txn_try_again'.

- ovsdb_idl_run() receives the transaction reply and updates the
  transaction status to TXN_SUCCESS.
- status_update_wait() detects that the transaction is in progress, and
  the status is TXN_SUCCESS so wakes up the main thread immediately.
- run_status_update() is meant to destroy the transaction now that it is
  finished, however the logic is never run because there were no changes.
- Repeat the wakeup every time the main loop runs.

This patch fixes the behaviour by ensuring that ovsdb_idl_txn_commit()
gets a chance to run whenever there is an ongoing status transaction.

Bug was found by unloading and reloading the kernel module, then
immediately restarting ovs-vswitchd.

Signed-off-by: Joe Stringer <joestringer at nicira.com>
Acked-by: Alex Wang <alexw at nicira.com>




More information about the git mailing list