[igt-dev] [PATCH i-g-t] tests/kms_chamelium: Wait for chamelium being reachable after a suspend

Arkadiusz Hiler
2019-08-16T08:35:41Z

Getting network back after waking up from suspend takes some time.
If it's not quick enough, it may cause any following chamelium_rpc
to fail.

This includes the chamelium port reset in exit handler:

igt_chamelium-CRITICAL: Test assertion failure function chamelium_rpc,
file ../lib/igt_chamelium.c:349:
igt_chamelium-CRITICAL: Failed assertion: !chamelium->env.fault_occurred
igt_chamelium-CRITICAL: Last errno: 113, No route to host
igt_chamelium-CRITICAL: Chamelium RPC call failed: libcurl failed to execute
the HTTP POST transaction, explaining:
Failed to connect to 192.168.1.224 port 9992: No route to host

Stack trace:
#0 ../lib/igt_core.c:1674 __igt_fail_assert()
#1 [chamelium_rpc+0x1ea]
#2 ../lib/igt_chamelium.c:2274 chamelium_reset()
#3 ../lib/igt_chamelium.c:2361 chamelium_deinit()
#4 ../lib/igt_core.c:2124 call_exit_handlers()

So let's wait for Chamelium to be reachable after each suspend_autoresume.

Cc: Simon Ser <simon.ser@intel.com>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=109380
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
---
lib/igt_chamelium.c | 21 +++++++++++++++++++++
lib/igt_chamelium.h | 1 +
tests/kms_chamelium.c | 4 +++-
3 files changed, 25 insertions(+), 1 deletion(-)

diff --git a/lib/igt_chamelium.c b/lib/igt_chamelium.c
index 1c0e1b39..bb1420c3 100644
--- a/lib/igt_chamelium.c
+++ b/lib/igt_chamelium.c
@@ -351,6 +351,27 @@ static xmlrpc_value *chamelium_rpc(struct chamelium *chamelium,
return res;
}

+static bool __chamelium_is_reachable(struct chamelium *chamelium)
+{
+ /* GetSupportedInputs does not require a port and is harmless */
+ xmlrpc_DECREF(__chamelium_rpc(chamelium, NULL, "GetSupportedInputs", "()"));
+
+ if (chamelium->env.fault_occurred)
+ igt_debug("Chamelium RPC call failed: %s\n",
+ chamelium->env.fault_string);
+
+ return !chamelium->env.fault_occurred;
+}
+
+void chamelium_wait_reachable(struct chamelium *chamelium, int timeout)
+{
+ bool chamelium_online = igt_wait(__chamelium_is_reachable(chamelium),
+ timeout * 1000, 100);
+
+ igt_assert_f(chamelium_online,
+ "Couldn't connect to Chamelium for %ds", timeout);
+}
+
/**
* chamelium_plug:
* @chamelium: The Chamelium instance to use
diff --git a/lib/igt_chamelium.h b/lib/igt_chamelium.h
index 1cc5e2d8..08705a9d 100644
--- a/lib/igt_chamelium.h
+++ b/lib/igt_chamelium.h
@@ -112,6 +112,7 @@ drmModeConnector *chamelium_port_get_connector(struct chamelium *chamelium,
bool reprobe);
const char *chamelium_port_get_name(struct chamelium_port *port);

+void chamelium_wait_reachable(struct chamelium *chamelium, int timeout);
void chamelium_plug(struct chamelium *chamelium, struct chamelium_port *port);
void chamelium_unplug(struct chamelium *chamelium, struct chamelium_port *port);
bool chamelium_is_plugged(struct chamelium *chamelium,
diff --git a/tests/kms_chamelium.c b/tests/kms_chamelium.c
index 0cf1bfbd..357d9f35 100644
--- a/tests/kms_chamelium.c
+++ b/tests/kms_chamelium.c
@@ -57,6 +57,7 @@ typedef struct {
} data_t;

#define HOTPLUG_TIMEOUT 20 /* seconds */
+#define ONLINE_TIMEOUT 10 /* seconds */

#define HPD_STORM_PULSE_INTERVAL_DP 100 /* ms */
#define HPD_STORM_PULSE_INTERVAL_HDMI 200 /* ms */
@@ -368,6 +369,7 @@ try_suspend_resume_hpd(data_t *data, struct chamelium_port *port,

igt_system_suspend_autoresume(state, test);
igt_assert(wait_for_hotplug(mon, &timeout));
+ chamelium_wait_reachable(data->chamelium, ONLINE_TIMEOUT);

if (port) {
igt_assert_eq(reprobe_connector(data, port), target_state);
@@ -473,8 +475,8 @@ test_suspend_resume_edid_change(data_t *data, struct chamelium_port *port,
igt_flush_hotplugs(mon);

igt_system_suspend_autoresume(state, test);
-
igt_assert(igt_hotplug_detected(mon, HOTPLUG_TIMEOUT));
+ chamelium_wait_reachable(data->chamelium, ONLINE_TIMEOUT);

get_connectors_link_status_failed(data, link_status_failed[1]);

--
2.21.0