All of lore.kernel.org
 help / color / mirror / Atom feed
From: Chris Wilson <chris@chris-wilson.co.uk>
To: Mika Kuoppala <mika.kuoppala@linux.intel.com>
Cc: intel-gfx@lists.freedesktop.org
Subject: Re: [PATCH i-g-t] tests: Add gem_spin_batch
Date: Wed, 8 Mar 2017 11:08:02 +0000	[thread overview]
Message-ID: <20170308110802.GI25284@nuc-i3427.alporthouse.com> (raw)
In-Reply-To: <1488969846-13428-1-git-send-email-mika.kuoppala@intel.com>

On Wed, Mar 08, 2017 at 12:44:06PM +0200, Mika Kuoppala wrote:
> Add gem_spin_batch to test that the dummyload infra
> is working properly. Can be also act as tool to force
> a single engine to be busy for controlled period of time.
> 
> v2: plenty of igt-fu improvements (Chris)
> v3: nesting batches for more utilization, epsilon fun (Chris)
> 
> Cc: Abdiel Janulgue <abdiel.janulgue@linux.intel.com>
> Cc: Chris Wilson <chris@chris-wilson.co.uk>
> Signed-off-by: Mika Kuoppala <mika.kuoppala@intel.com>
> ---
>  tests/Makefile.sources |  1 +
>  tests/gem_spin_batch.c | 85 ++++++++++++++++++++++++++++++++++++++++++++++++++
>  2 files changed, 86 insertions(+)
>  create mode 100644 tests/gem_spin_batch.c
> 
> diff --git a/tests/Makefile.sources b/tests/Makefile.sources
> index 6e07d93..1763b67 100644
> --- a/tests/Makefile.sources
> +++ b/tests/Makefile.sources
> @@ -214,6 +214,7 @@ TESTS_progs = \
>  	gem_unfence_active_buffers \
>  	gem_unref_active_buffers \
>  	gem_wait \
> +	gem_spin_batch \
>  	gem_workarounds \
>  	gen3_mixed_blits \
>  	gen3_render_linear_blits \
> diff --git a/tests/gem_spin_batch.c b/tests/gem_spin_batch.c
> new file mode 100644
> index 0000000..a408fe1
> --- /dev/null
> +++ b/tests/gem_spin_batch.c
> @@ -0,0 +1,85 @@
> +/*
> + * Copyright © 2017 Intel Corporation
> + *
> + * Permission is hereby granted, free of charge, to any person obtaining a
> + * copy of this software and associated documentation files (the "Software"),
> + * to deal in the Software without restriction, including without limitation
> + * the rights to use, copy, modify, merge, publish, distribute, sublicense,
> + * and/or sell copies of the Software, and to permit persons to whom the
> + * Software is furnished to do so, subject to the following conditions:
> + *
> + * The above copyright notice and this permission notice (including the next
> + * paragraph) shall be included in all copies or substantial portions of the
> + * Software.
> + *
> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
> + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
> + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
> + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
> + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
> + * IN THE SOFTWARE.
> + *
> + */
> +
> +#include "igt.h"
> +
> +#define MAX_ERROR 3 /* % */
> +
> +#define assert_within_epsilon(x, ref, tolerance) \
> +	igt_assert_f(100 * x <= (100 + tolerance) * ref && \
> +		     100 * x >= (100 - tolerance) * ref, \
> +		     "'%s' != '%s' (%ld not within %d%% tolerance of %ld)\n", \
> +		     #x, #ref, (int64_t)x, tolerance, (int64_t)ref)
> +
> +static void basic(int fd, unsigned int engine, unsigned int timeout_sec)
> +{
> +	const uint64_t timeout_100ms = 100000000LL;
> +	unsigned long loops = 0;
> +	igt_spin_t *spin;
> +	struct timespec tv = { };
> +	uint64_t elapsed;
> +
> +	spin = igt_spin_batch_new(fd, engine, 0);
> +	while ((elapsed = igt_nsec_elapsed(&tv)) >> 30 < timeout_sec) {
> +		igt_spin_t *next = igt_spin_batch_new(fd, engine, 0);
> +
> +		igt_spin_batch_set_timeout(spin, timeout_100ms);
> +		gem_sync(fd, spin->handle);
> +		loops++;
> +
> +		igt_spin_batch_free(fd, spin);
> +		spin = next;
> +	}
> +	igt_spin_batch_free(fd, spin);
> +
> +	assert_within_epsilon(timeout_100ms * loops, elapsed, MAX_ERROR);
> +	igt_assert_eq(intel_detect_and_clear_missed_interrupts(fd), 0);
> +}
> +
> +igt_main
> +{
> +	const struct intel_execution_engine *e;
> +	int fd = -1;
> +
> +	igt_skip_on_simulation();
> +
> +	igt_fixture {
> +		fd = drm_open_driver(DRIVER_INTEL);
> +		igt_require_gem(fd);
> +		igt_fork_hang_detector(fd);

intel_detect_and_clear_missed_interrupts(fd);

And I'm done! :)
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

  reply	other threads:[~2017-03-08 11:08 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-03-07 16:02 [PATCH i-g-t] tests: Add gem_spin_batch Mika Kuoppala
2017-03-07 16:12 ` Chris Wilson
2017-03-08 10:44   ` Mika Kuoppala
2017-03-08 11:08     ` Chris Wilson [this message]
2017-03-08 11:28       ` Mika Kuoppala

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20170308110802.GI25284@nuc-i3427.alporthouse.com \
    --to=chris@chris-wilson.co.uk \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=mika.kuoppala@linux.intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.