diff mbox series

[BlueZ] test-runner: Fix not waiting for system_bus_socket

Message ID 20220601211230.1355268-1-luiz.dentz@gmail.com (mailing list archive)
State Accepted
Commit 7bcd32e67e9388c49212e5dd5dd431c9ec079eb0
Headers show
Series [BlueZ] test-runner: Fix not waiting for system_bus_socket | expand

Checks

Context Check Description
tedd_an/pre-ci_am success Success
tedd_an/checkpatch success Checkpatch PASS
tedd_an/gitlint success Gitlint PASS
tedd_an/setupell success Setup ELL PASS
tedd_an/buildprep success Build Prep PASS
tedd_an/build success Build Configuration PASS
tedd_an/makecheck success Make Check PASS
tedd_an/makecheckvalgrind success Make Check PASS
tedd_an/makedistcheck success Make Distcheck PASS
tedd_an/build_extell success Build External ELL PASS
tedd_an/build_extell_make success Build Make with External ELL PASS

Commit Message

Luiz Augusto von Dentz June 1, 2022, 9:12 p.m. UTC
From: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>

This makes test-runner wait for system_bus_socket to be available before
continuing otherwise the likes of bluetoothd would likely fail to start.
---
 tools/test-runner.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

Comments

bluez.test.bot@gmail.com June 1, 2022, 10:26 p.m. UTC | #1
This is automated email and please do not reply to this email!

Dear submitter,

Thank you for submitting the patches to the linux bluetooth mailing list.
This is a CI test results with your patch series:
PW Link:https://patchwork.kernel.org/project/bluetooth/list/?series=646680

---Test result---

Test Summary:
CheckPatch                    PASS      1.06 seconds
GitLint                       PASS      0.72 seconds
Prep - Setup ELL              PASS      44.61 seconds
Build - Prep                  PASS      0.57 seconds
Build - Configure             PASS      8.79 seconds
Build - Make                  PASS      1393.52 seconds
Make Check                    PASS      12.16 seconds
Make Check w/Valgrind         PASS      447.51 seconds
Make Distcheck                PASS      233.13 seconds
Build w/ext ELL - Configure   PASS      8.55 seconds
Build w/ext ELL - Make        PASS      1382.88 seconds
Incremental Build with patchesPASS      0.00 seconds



---
Regards,
Linux Bluetooth
patchwork-bot+bluetooth@kernel.org June 2, 2022, 7:10 p.m. UTC | #2
Hello:

This patch was applied to bluetooth/bluez.git (master)
by Luiz Augusto von Dentz <luiz.von.dentz@intel.com>:

On Wed,  1 Jun 2022 14:12:30 -0700 you wrote:
> From: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>
> 
> This makes test-runner wait for system_bus_socket to be available before
> continuing otherwise the likes of bluetoothd would likely fail to start.
> ---
>  tools/test-runner.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)

Here is the summary with links:
  - [BlueZ] test-runner: Fix not waiting for system_bus_socket
    https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=7bcd32e67e93

You are awesome, thank you!
diff mbox series

Patch

diff --git a/tools/test-runner.c b/tools/test-runner.c
index 1f1a8c36f..945a16a77 100644
--- a/tools/test-runner.c
+++ b/tools/test-runner.c
@@ -452,13 +452,13 @@  static pid_t start_dbus_daemon(void)
 
 		if (!stat("/run/dbus/system_bus_socket", &st)) {
 			printf("Found D-Bus daemon socket\n");
-			break;
+			return pid;
 		}
 
-		usleep(25 * 1000);
+		sleep(1);
 	}
 
-	return pid;
+	return -1;
 }
 
 static const char *daemon_table[] = {