[PATCH i-g-t v2 8/8] tests/core_hotunplug: Print info between steps
Kamil Konieczny
kamil.konieczny at linux.intel.com
Mon Aug 5 14:37:11 UTC 2024
There are some info printed directly into kernel dmesg before
subtest steps. Turn comment before steps into direct prints
to stdout to help differentiate which steps where a success
and where it starts failing.
Fixed also one misspelling s/realoading/reloading/
Signed-off-by: Kamil Konieczny <kamil.konieczny at linux.intel.com>
---
tests/core_hotunplug.c | 26 ++++++++++++++++++++------
1 file changed, 20 insertions(+), 6 deletions(-)
diff --git a/tests/core_hotunplug.c b/tests/core_hotunplug.c
index 51edee985..415c74381 100644
--- a/tests/core_hotunplug.c
+++ b/tests/core_hotunplug.c
@@ -184,12 +184,13 @@ static void prepare(struct hotunplug *priv)
igt_assert_fd(priv->fd.sysfs_bus);
priv->fd.sysfs_dev = close_sysfs(priv->fd.sysfs_dev);
+ igt_info("Preparations done\n");
}
-/* Unbind the driver from the device */
static void driver_unbind(struct hotunplug *priv, const char *prefix,
int timeout)
{
+ igt_info("Unbind the driver from the device (bus: %s)\n", priv->dev_bus_addr);
/*
* FIXME: on some devices, the audio driver (snd_hda_intel)
* binds into the i915 driver. On such hardware, kernel warnings
@@ -216,11 +217,13 @@ static void driver_unbind(struct hotunplug *priv, const char *prefix,
igt_assert_f(faccessat(priv->fd.sysfs_drv, priv->dev_bus_addr, F_OK, 0),
"Unbound device still present (%s)\n", priv->dev_bus_addr);
+
+ igt_info("Unbind done\n");
}
-/* Re-bind the driver to the device */
static void driver_bind(struct hotunplug *priv, int timeout)
{
+ igt_info("Re-bind the driver to the device (bus: %s)\n", priv->dev_bus_addr);
local_debug("%s\n", "rebinding the driver to the device");
priv->failure = "Driver re-bind failure!";
@@ -235,17 +238,19 @@ static void driver_bind(struct hotunplug *priv, int timeout)
"Rebound device not present (%s)!\n", priv->dev_bus_addr);
if (priv->snd_driver) {
- igt_info("Realoading %s\n", priv->snd_driver);
+ igt_info("Reloading %s\n", priv->snd_driver);
igt_kmod_load(priv->snd_driver, NULL);
free(priv->snd_driver);
priv->snd_driver = NULL;
}
+
+ igt_info("Re-bind done\n");
}
-/* Remove (virtually unplug) the device from its bus */
static void device_unplug(struct hotunplug *priv, const char *prefix,
int timeout)
{
+ igt_info("Remove (virtually unplug) the device from its bus (%s)\n", priv->dev_bus_addr);
igt_require(priv->fd.sysfs_dev == -1);
/*
@@ -267,7 +272,8 @@ static void device_unplug(struct hotunplug *priv, const char *prefix,
O_DIRECTORY);
igt_assert_fd(priv->fd.sysfs_dev);
- local_debug("%sunplugging the device\n", prefix);
+ igt_info("%sunplugging the device %s\n", prefix, priv->dev_bus_addr);
+ local_debug("%sunplugging the device %s\n", prefix, priv->dev_bus_addr);
priv->failure = "Device unplug failure!";
igt_set_timeout(timeout, "Device unplug timeout!");
@@ -280,11 +286,13 @@ static void device_unplug(struct hotunplug *priv, const char *prefix,
igt_assert_f(faccessat(priv->fd.sysfs_bus, priv->dev_bus_addr, F_OK, 0),
"Unplugged device still present (%s)\n", priv->dev_bus_addr);
+
+ igt_info("Unplug done\n");
}
-/* Re-discover the device by rescanning its bus */
static void bus_rescan(struct hotunplug *priv, int timeout)
{
+ igt_info("Re-discover the device by rescanning its bus (%s)\n", priv->dev_bus_addr);
local_debug("%s\n", "rediscovering the device");
priv->failure = "Bus rescan failure!";
@@ -303,6 +311,8 @@ static void bus_rescan(struct hotunplug *priv, int timeout)
free(priv->snd_driver);
priv->snd_driver = NULL;
}
+
+ igt_info("Bus rescan done\n");
}
static void cleanup(struct hotunplug *priv)
@@ -317,6 +327,7 @@ static void cleanup(struct hotunplug *priv)
}
priv->fd.sysfs_dev = close_sysfs(priv->fd.sysfs_dev);
+ igt_info("Cleanup done\n");
}
static bool local_i915_is_wedged(int i915)
@@ -498,6 +509,8 @@ static bool healthcheck(struct hotunplug *priv, bool recover)
node_healthcheck(priv,
FLAG_RENDER | (recover ? FLAG_RECOVER : 0));
+ igt_info("Healthcheck: %s\n", priv->failure ? "failed" : "ok");
+
return !priv->failure;
}
@@ -532,6 +545,7 @@ static void recover(struct hotunplug *priv)
else if (faccessat(priv->fd.sysfs_drv, priv->dev_bus_addr, F_OK, 0))
driver_bind(priv, 60);
+ igt_info("Recovery: %s\n", priv->failure ? "failed" : "ok");
if (priv->failure)
igt_assert_f(healthcheck(priv, true), "%s\n", priv->failure);
}
--
2.43.0
More information about the igt-dev
mailing list