Bug fix for kdbus_do_iteration() causing busy loop 17/67817/1 submit/tizen/20160428.103823
authorHyotaek Shim <hyotaek.shim@samsung.com>
Thu, 28 Apr 2016 10:35:50 +0000 (19:35 +0900)
committerHyotaek Shim <hyotaek.shim@samsung.com>
Thu, 28 Apr 2016 10:35:50 +0000 (19:35 +0900)
Change-Id: I75a6267471b7fdff2d147514210726a3cec6c5dd
Signed-off-by: Hyotaek Shim <hyotaek.shim@samsung.com>
dbus/dbus-transport-kdbus.c

index e9aca6f..03de72e 100644 (file)
@@ -3726,11 +3726,16 @@ kdbus_do_iteration (DBusTransport *transport,
     * When timeout is set to -1 in client application,
     * error messages are inserted directly to incoming queue and
     * application hangs on dbus_poll.
-    */
+    *
+    * This causes a busy loop in _dbus_connection_block_pending_call() 
+    * There is no case of waiting for the locally-generated error reply
+
    if (_dbus_connection_get_n_incoming (transport->connection) > 0)
    {
      timeout_milliseconds = 0;
    }
+    */
+
    /* This is kind of a hack; if we have stuff to write, then try
     * to avoid the poll. This is probably about a 5% speedup on an
     * echo client/server.