All of lore.kernel.org
 help / color / mirror / Atom feed
From: Matthew Auld <matthew.william.auld@gmail.com>
To: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Intel Graphics Development <intel-gfx@lists.freedesktop.org>,
	Matthew Auld <matthew.auld@intel.com>
Subject: Re: [PATCH] drm/i915/selftests: Tighten the timeout testing for partial mmaps
Date: Tue, 10 Sep 2019 13:29:11 +0100	[thread overview]
Message-ID: <CAM0jSHPKhTbj-ohG+L2VrekYS111fWqZEcOdyT5vV+YQJT3msQ@mail.gmail.com> (raw)
In-Reply-To: <20190910121009.13431-1-chris@chris-wilson.co.uk>

On Tue, 10 Sep 2019 at 13:10, Chris Wilson <chris@chris-wilson.co.uk> wrote:
>
> Currently, if there is time remaining before the start of the loop, we
> do one full iteration over many possible different chunks within the
> object. A full loop may take 50+s (depending on speed of indirect GTT
> mmapings) and we try separately with LINEAR, X and Y -- at which point
> igt times out. If we check more frequently, we will interrupt the loop
> upon our timeout -- it is hard to argue for as this significantly reduces
> the test coverage as we dramatically reduce the runtime. In practical
> terms, the coverage we should prioritise is in using different fence
> setups, forcing verification of the tile row computations over the
> current preference of checking extracting chunks. Though the exhaustive
> search is great given an infinite timeout, to improve our current
> coverage, we also add a randomised smoketest of partial mmaps. So let's
> do both, add a randomised smoketest of partial tiling chunks and the
> exhaustive (though time limited) search for failures.
>
> Even in adding another subtest, we should shave 100s off BAT! (With,
> hopefully, no loss in coverage, at least over multiple runs.)
>
> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
> Cc: Matthew Auld <matthew.auld@intel.com>
Reviewed-by: Matthew Auld <matthew.auld@intel.com>
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

  reply	other threads:[~2019-09-10 12:29 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-09-10 12:10 [PATCH] drm/i915/selftests: Tighten the timeout testing for partial mmaps Chris Wilson
2019-09-10 12:29 ` Matthew Auld [this message]
2019-09-10 13:10 ` ✗ Fi.CI.CHECKPATCH: warning for " Patchwork
2019-09-10 13:33 ` ✓ Fi.CI.BAT: success " Patchwork
2019-09-10 19:44 ` ✓ Fi.CI.IGT: " Patchwork

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=CAM0jSHPKhTbj-ohG+L2VrekYS111fWqZEcOdyT5vV+YQJT3msQ@mail.gmail.com \
    --to=matthew.william.auld@gmail.com \
    --cc=chris@chris-wilson.co.uk \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=matthew.auld@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.