[PATCH intel-gpu-tools 1/1] drmtest: exit() rather than abort() for simple usage errors.
Bryce Harrington
bryce at canonical.com
Tue Jan 24 19:44:44 PST 2012
When the benchmarks are run as non-root, they terminate since they can't
read the drm files. However, by terminating with abort(), this raises
SIGSEGV which has the side effect of triggering crash reporting
utilities (e.g. apport). As a result we've been accumulating bug
reports about it.
As the code is displaying a unique error message prior to termination,
it should be discoverable enough where in the code the failure occurs,
so an exit(1) should be sufficient for termination.
Signed-off-by: Bryce Harrington <bryce at canonical.com>
---
lib/drmtest.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/lib/drmtest.c b/lib/drmtest.c
index fc40ad1..5ebd6d6 100644
--- a/lib/drmtest.c
+++ b/lib/drmtest.c
@@ -65,7 +65,7 @@ int drm_open_any(void)
close(fd);
}
fprintf(stderr, "failed to open any drm device. retry as root?\n");
- abort();
+ exit(1);
}
@@ -108,5 +108,5 @@ int drm_open_any_master(void)
return fd;
}
fprintf(stderr, "Couldn't find an un-controlled DRM device\n");
- abort();
+ exit(1);
}
--
1.7.4.1
More information about the xorg-devel
mailing list