[Intel-gfx] [PATCH 18/28] lib: unnecessary header removal for drmtest.h, part 2

Daniel Vetter daniel.vetter at ffwll.ch
Sun Mar 23 15:18:52 CET 2014


I've left unistd.h in it - it's not strictly required but most users
of drmtest.h want it for the open helpers, and then you kinda need to
close that file descriptor again ...

Signed-off-by: Daniel Vetter <daniel.vetter at ffwll.ch>
---
 benchmarks/intel_upload_blit_large.c | 6 ++++--
 lib/drmtest.h                        | 9 +++------
 lib/igt_core.c                       | 2 +-
 lib/igt_core.h                       | 2 ++
 lib/igt_debugfs.c                    | 1 +
 lib/igt_kms.c                        | 6 +++++-
 lib/igt_kms.h                        | 2 ++
 lib/ioctl_wrappers.c                 | 1 +
 tests/core_getclient.c               | 3 +++
 tests/core_getstats.c                | 5 +++++
 tests/gem_ctx_bad_destroy.c          | 2 ++
 tests/gem_ctx_create.c               | 2 ++
 tests/gem_fenced_exec_thrash.c       | 4 +++-
 tests/gem_flink_race.c               | 1 +
 tests/gem_reg_read.c                 | 2 ++
 tests/gem_seqno_wrap.c               | 1 +
 tests/gem_vmap_blits.c               | 1 +
 tests/igt_simulation.c               | 1 +
 tests/pm_psr.c                       | 2 ++
 tests/pm_rc6_residency.c             | 2 ++
 tests/pm_rps.c                       | 1 +
 tests/prime_udl.c                    | 3 ++-
 22 files changed, 47 insertions(+), 12 deletions(-)

diff --git a/benchmarks/intel_upload_blit_large.c b/benchmarks/intel_upload_blit_large.c
index b44433f9a30d..40bdbcfb8fe3 100644
--- a/benchmarks/intel_upload_blit_large.c
+++ b/benchmarks/intel_upload_blit_large.c
@@ -53,8 +53,10 @@
 #include <errno.h>
 #include <sys/stat.h>
 #include <sys/time.h>
-#include "drm.h"
-#include "i915_drm.h"
+
+#include <drm.h>
+#include <i915_drm.h>
+
 #include "drmtest.h"
 #include "intel_bufmgr.h"
 #include "intel_batchbuffer.h"
diff --git a/lib/drmtest.h b/lib/drmtest.h
index 4c55068ee54c..97d60d0c29dd 100644
--- a/lib/drmtest.h
+++ b/lib/drmtest.h
@@ -28,16 +28,13 @@
 #ifndef DRMTEST_H
 #define DRMTEST_H
 
-#include <stdio.h>
-#include <stdlib.h>
 #include <unistd.h>
-#include <errno.h>
 #include <stdbool.h>
+#include <stdint.h>
 #include <sys/mman.h>
 
-#include "xf86drm.h"
-#include "xf86drmMode.h"
-#include "intel_batchbuffer.h"
+#include <xf86drm.h>
+#include <intel_batchbuffer.h>
 
 #ifdef ANDROID
 #ifndef HAVE_MMAP64
diff --git a/lib/igt_core.c b/lib/igt_core.c
index efcc4a11fcc5..81e37313fb45 100644
--- a/lib/igt_core.c
+++ b/lib/igt_core.c
@@ -48,9 +48,9 @@
 #include <sys/syscall.h>
 #include <sys/utsname.h>
 #include <termios.h>
+#include <errno.h>
 
 #include "drmtest.h"
-#include "i915_drm.h"
 #include "intel_chipset.h"
 #include "intel_gpu_tools.h"
 #include "igt_debugfs.h"
diff --git a/lib/igt_core.h b/lib/igt_core.h
index 47576da652a0..7ede0d37a540 100644
--- a/lib/igt_core.h
+++ b/lib/igt_core.h
@@ -32,8 +32,10 @@
 
 #include <setjmp.h>
 #include <stdbool.h>
+#include <stdlib.h>
 #include <string.h>
 #include <sys/types.h>
+#include <stdarg.h>
 
 bool __igt_fixture(void);
 void __igt_fixture_complete(void);
diff --git a/lib/igt_debugfs.c b/lib/igt_debugfs.c
index f467ce0f3992..96adf396887b 100644
--- a/lib/igt_debugfs.c
+++ b/lib/igt_debugfs.c
@@ -27,6 +27,7 @@
 #include <sys/mount.h>
 #include <errno.h>
 #include <stdio.h>
+#include <stdlib.h>
 #include <string.h>
 #include <fcntl.h>
 #include <unistd.h>
diff --git a/lib/igt_kms.c b/lib/igt_kms.c
index 88ee83be2f91..f74f7070d040 100644
--- a/lib/igt_kms.c
+++ b/lib/igt_kms.c
@@ -23,14 +23,18 @@
  */
 
 #define _GNU_SOURCE
+#include <unistd.h>
 #include <stdio.h>
 #include <stdarg.h>
 #include <fcntl.h>
 #include <sys/stat.h>
 #include <string.h>
+#include <stdlib.h>
 #include <math.h>
 #include <linux/kd.h>
-#include "drm_fourcc.h"
+#include <errno.h>
+
+#include <drm_fourcc.h>
 
 #include "drmtest.h"
 #include "igt_kms.h"
diff --git a/lib/igt_kms.h b/lib/igt_kms.h
index d141cfce8504..95735e8ffdd7 100644
--- a/lib/igt_kms.h
+++ b/lib/igt_kms.h
@@ -29,6 +29,8 @@
 #include <stdint.h>
 #include <cairo.h>
 
+#include <xf86drmMode.h>
+
 #include "igt_display.h"
 
 struct kmstest_connector_config {
diff --git a/lib/ioctl_wrappers.c b/lib/ioctl_wrappers.c
index 147334f3286b..5806f2a13bd8 100644
--- a/lib/ioctl_wrappers.c
+++ b/lib/ioctl_wrappers.c
@@ -47,6 +47,7 @@
 #include <sys/syscall.h>
 #include <sys/utsname.h>
 #include <termios.h>
+#include <errno.h>
 
 #include "drmtest.h"
 #include "i915_drm.h"
diff --git a/tests/core_getclient.c b/tests/core_getclient.c
index 22f0ab2e2b69..ed46e51924ea 100644
--- a/tests/core_getclient.c
+++ b/tests/core_getclient.c
@@ -27,6 +27,9 @@
 
 #include <limits.h>
 #include <sys/ioctl.h>
+#include <stdlib.h>
+#include <errno.h>
+
 #include "drmtest.h"
 
 /**
diff --git a/tests/core_getstats.c b/tests/core_getstats.c
index cce8271b561d..1e2c5651d5eb 100644
--- a/tests/core_getstats.c
+++ b/tests/core_getstats.c
@@ -27,6 +27,11 @@
 
 #include <limits.h>
 #include <sys/ioctl.h>
+#include <sys/types.h>
+
+#include <drm.h>
+
+#include "igt_core.h"
 #include "drmtest.h"
 
 /**
diff --git a/tests/gem_ctx_bad_destroy.c b/tests/gem_ctx_bad_destroy.c
index 092f34221061..97a706660dc1 100644
--- a/tests/gem_ctx_bad_destroy.c
+++ b/tests/gem_ctx_bad_destroy.c
@@ -31,6 +31,8 @@
 
 #include <stdio.h>
 #include <string.h>
+#include <errno.h>
+
 #include "ioctl_wrappers.h"
 #include "drmtest.h"
 
diff --git a/tests/gem_ctx_create.c b/tests/gem_ctx_create.c
index f32c2287f8d9..57494fc44638 100644
--- a/tests/gem_ctx_create.c
+++ b/tests/gem_ctx_create.c
@@ -27,6 +27,8 @@
 
 #include <stdio.h>
 #include <string.h>
+#include <errno.h>
+
 #include "ioctl_wrappers.h"
 #include "drmtest.h"
 
diff --git a/tests/gem_fenced_exec_thrash.c b/tests/gem_fenced_exec_thrash.c
index cd53cafc5fdf..2ba85f2bcb5a 100644
--- a/tests/gem_fenced_exec_thrash.c
+++ b/tests/gem_fenced_exec_thrash.c
@@ -33,9 +33,11 @@
 #include <string.h>
 #include <fcntl.h>
 #include <inttypes.h>
+#include <errno.h>
+
 #include <drm.h>
-#include "ioctl_wrappers.h"
 
+#include "ioctl_wrappers.h"
 #include "drmtest.h"
 #include "intel_chipset.h"
 #include "intel_gpu_tools.h"
diff --git a/tests/gem_flink_race.c b/tests/gem_flink_race.c
index 26c71cecce07..3353b83e6acd 100644
--- a/tests/gem_flink_race.c
+++ b/tests/gem_flink_race.c
@@ -31,6 +31,7 @@
 #include <fcntl.h>
 #include <unistd.h>
 #include <pthread.h>
+#include <errno.h>
 
 #include "drmtest.h"
 #include "ioctl_wrappers.h"
diff --git a/tests/gem_reg_read.c b/tests/gem_reg_read.c
index 19f1351b3fd1..b4a6650a64d4 100644
--- a/tests/gem_reg_read.c
+++ b/tests/gem_reg_read.c
@@ -27,6 +27,8 @@
 
 #include <stdio.h>
 #include <string.h>
+#include <errno.h>
+
 #include "ioctl_wrappers.h"
 #include "drmtest.h"
 
diff --git a/tests/gem_seqno_wrap.c b/tests/gem_seqno_wrap.c
index 19c0201e21ba..30d1a8687090 100644
--- a/tests/gem_seqno_wrap.c
+++ b/tests/gem_seqno_wrap.c
@@ -41,6 +41,7 @@
 #include <wordexp.h>
 #include <getopt.h>
 #include <signal.h>
+#include <errno.h>
 
 #include "ioctl_wrappers.h"
 #include "drmtest.h"
diff --git a/tests/gem_vmap_blits.c b/tests/gem_vmap_blits.c
index 4fb2a215e3d5..157d67460e6c 100644
--- a/tests/gem_vmap_blits.c
+++ b/tests/gem_vmap_blits.c
@@ -43,6 +43,7 @@
 #include <errno.h>
 #include <sys/stat.h>
 #include <sys/time.h>
+
 #include "drm.h"
 #include "ioctl_wrappers.h"
 #include "drmtest.h"
diff --git a/tests/igt_simulation.c b/tests/igt_simulation.c
index f4007b80929e..d3331176cf2c 100644
--- a/tests/igt_simulation.c
+++ b/tests/igt_simulation.c
@@ -29,6 +29,7 @@
 #include <sys/wait.h>
 #include <sys/types.h>
 #include <assert.h>
+#include <errno.h>
 
 #include "drmtest.h"
 #include "igt_core.h"
diff --git a/tests/pm_psr.c b/tests/pm_psr.c
index 5d28c7a59a3b..4ba121f10a38 100644
--- a/tests/pm_psr.c
+++ b/tests/pm_psr.c
@@ -29,6 +29,8 @@
 #include <stdlib.h>
 #include <string.h>
 #include <unistd.h>
+#include <errno.h>
+
 #include "drmtest.h"
 
 #define SLEEP_DURATION 5000 // in milliseconds
diff --git a/tests/pm_rc6_residency.c b/tests/pm_rc6_residency.c
index 3fd9d47e9db3..84eb8a909019 100644
--- a/tests/pm_rc6_residency.c
+++ b/tests/pm_rc6_residency.c
@@ -30,6 +30,8 @@
 #include <stdlib.h>
 #include <string.h>
 #include <unistd.h>
+#include <errno.h>
+
 #include "drmtest.h"
 
 #define SLEEP_DURATION 3000 // in milliseconds
diff --git a/tests/pm_rps.c b/tests/pm_rps.c
index e6ab37907e8e..1f4b284cd42a 100644
--- a/tests/pm_rps.c
+++ b/tests/pm_rps.c
@@ -34,6 +34,7 @@
 #include <getopt.h>
 #include <fcntl.h>
 #include <signal.h>
+#include <errno.h>
 
 #include "drmtest.h"
 #include "intel_gpu_tools.h"
diff --git a/tests/prime_udl.c b/tests/prime_udl.c
index 892a952795a7..96661fcb501f 100644
--- a/tests/prime_udl.c
+++ b/tests/prime_udl.c
@@ -24,7 +24,8 @@
 #include <errno.h>
 
 #include "xf86drm.h"
-#include "xf86drmMode.h"
+#include <xf86drmMode.h>
+
 #include "ioctl_wrappers.h"
 #include "intel_bufmgr.h"
 #include "intel_gpu_tools.h"
-- 
1.8.5.2




More information about the Intel-gfx mailing list