diff options
| author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2016-05-11 17:06:28 +0200 |
|---|---|---|
| committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2016-05-12 11:37:21 +0200 |
| commit | d7050f9f79fee8fb7c790f355c984d9e5141e1c5 (patch) | |
| tree | ea1ff5b7b729b0fc22f29ee5af8df8860ae6c7e8 /tests/gem_concurrent_all.c | |
| parent | 701d8fdb363443453c4b613e9180f75ad36a7321 (diff) | |
lib/igt_aux: Polish docs for igt_interruptible
- Give __ prefix to internal funcstion and structs, only
igt_interruptible is used by tests.
- Move docs to igt_interruptible and adjust.
- Explain more clearly how the timeout is getting doubled each
iteration until no more interruptions happen. Also rename the
argument to give it a more meaningful name in the docs.
- Link from other functions to this one for cross-referencing.
- Rename to igt_do_interruptible to make it clearer it's a loop,
inspired by do {} while () loops.
v2: Rename instead to igt_while_interruptible and fix typos (Chris).
And add gtk-doc for igt_ioctl, too.
Acked-by: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
Diffstat (limited to 'tests/gem_concurrent_all.c')
| -rw-r--r-- | tests/gem_concurrent_all.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tests/gem_concurrent_all.c b/tests/gem_concurrent_all.c index c0af60d4..019628ed 100644 --- a/tests/gem_concurrent_all.c +++ b/tests/gem_concurrent_all.c @@ -1294,7 +1294,7 @@ static void run_interruptible(struct buffers *buffers, do_hang do_hang_func) { pass = 0; - igt_interruptible(true) + igt_while_interruptible(true) do_test_func(buffers, do_copy_func, do_hang_func); igt_assert_eq(intel_detect_and_clear_missed_interrupts(fd), 0); } @@ -1340,7 +1340,7 @@ static void __run_forked(struct buffers *buffers, buffers_reset(buffers, true); buffers_create(buffers); - igt_interruptible(interrupt) { + igt_while_interruptible(interrupt) { for (pass = 0; pass < loops; pass++) do_test_func(buffers, do_copy_func, |