diff options
| author | Janusz Krzysztofik <janusz.krzysztofik@linux.intel.com> | 2021-03-17 09:21:53 +0100 |
|---|---|---|
| committer | Janusz Krzysztofik <janusz.krzysztofik@linux.intel.com> | 2021-03-23 10:58:26 +0100 |
| commit | 24c95fb6f5e1d18609391dffe33f247b5a39a895 (patch) | |
| tree | 0df888466b9b66c4a116208be7337f4ba36e5c93 /tests/core_hotunplug.c | |
| parent | 69b578b6ab0a36750f0d23c223a3487fc88fb6a7 (diff) | |
tests/core_hotunplug: Be more specific on sysfs vs. debugfs issues
Messages displayed on sysfs health check failures don't provide
information which subtree of sysfs actually failed - device sysfs
itself or device debugfs. That information could make debugging more
easy if available.
Be more specific when reporting sysfs health check failures.
Signed-off-by: Janusz Krzysztofik <janusz.krzysztofik@linux.intel.com>
Reviewed-by: Adam Miszczak <adam.miszczak@linux.intel.com>
Diffstat (limited to 'tests/core_hotunplug.c')
| -rw-r--r-- | tests/core_hotunplug.c | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/tests/core_hotunplug.c b/tests/core_hotunplug.c index e7d2a447..56a88fef 100644 --- a/tests/core_hotunplug.c +++ b/tests/core_hotunplug.c @@ -351,11 +351,13 @@ static void node_healthcheck(struct hotunplug *priv, unsigned flags) if (!priv->failure) { char path[200]; - priv->failure = "Device sysfs healthckeck failure!"; local_debug("%s\n", "running device sysfs healthcheck"); - if (igt_sysfs_path(fd_drm, path, sizeof(path)) && - igt_debugfs_path(fd_drm, path, sizeof(path))) - priv->failure = NULL; + priv->failure = "Device sysfs healthckeck failure!"; + if (igt_sysfs_path(fd_drm, path, sizeof(path))) { + priv->failure = "Device debugfs healthckeck failure!"; + if (igt_debugfs_path(fd_drm, path, sizeof(path))) + priv->failure = NULL; + } } fd_drm = close_device(fd_drm, "", "health checked "); |