[PATCH i-g-t v3 15/18] tests/core_hotunplug: Assert expected device presence/absence
Janusz Krzysztofik
janusz.krzysztofik at linux.intel.com
Mon Aug 17 09:22:26 UTC 2020
Don't rely on successful write to sysfs control files, assert existence
/ non-existence of a respective device sysfs node as well.
v2: Rebase.
v3: Rebase.
Signed-off-by: Janusz Krzysztofik <janusz.krzysztofik at linux.intel.com>
Reviewed-by: Michał Winiarski <michal.winiarski at intel.com>
---
tests/core_hotunplug.c | 14 ++++++++++++++
1 file changed, 14 insertions(+)
diff --git a/tests/core_hotunplug.c b/tests/core_hotunplug.c
index 76e8abd80..32ef0f43c 100644
--- a/tests/core_hotunplug.c
+++ b/tests/core_hotunplug.c
@@ -131,6 +131,9 @@ static void driver_unbind(struct hotunplug *priv, const char *prefix,
priv->dev_bus_addr),
"Driver unbind failure!\n");
igt_reset_timeout();
+
+ igt_assert_f(faccessat(priv->fd.sysfs_drv, priv->dev_bus_addr, F_OK, 0),
+ "Unbound device still present\n");
}
/* Re-bind the driver to the device */
@@ -144,6 +147,10 @@ static void driver_bind(struct hotunplug *priv, int timeout)
priv->dev_bus_addr),
"Driver re-bind failure\n!");
igt_reset_timeout();
+
+ igt_fail_on_f(faccessat(priv->fd.sysfs_drv, priv->dev_bus_addr,
+ F_OK, 0),
+ "Rebound device not present!\n");
}
/* Remove (virtually unplug) the device from its bus */
@@ -168,6 +175,9 @@ static void device_unplug(struct hotunplug *priv, const char *prefix,
priv->fd.sysfs_dev = local_close(priv->fd.sysfs_dev);
igt_assert_f(priv->fd.sysfs_dev == -1,
"Device sysfs node close failed\n");
+
+ igt_assert_f(faccessat(priv->fd.sysfs_bus, priv->dev_bus_addr, F_OK, 0),
+ "Unplugged device still present\n");
}
/* Re-discover the device by rescanning its bus */
@@ -180,6 +190,10 @@ static void bus_rescan(struct hotunplug *priv, int timeout)
igt_assert_f(igt_sysfs_set(priv->fd.sysfs_bus, "../rescan", "1"),
"Bus rescan failure!\n");
igt_reset_timeout();
+
+ igt_fail_on_f(faccessat(priv->fd.sysfs_bus, priv->dev_bus_addr,
+ F_OK, 0),
+ "Fakely unplugged device not rediscovered!\n");
}
static void cleanup(struct hotunplug *priv)
--
2.21.1
More information about the Intel-gfx-trybot
mailing list