From: Daniel Vetter <daniel@ffwll.ch>
To: Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>
Cc: Intel-gfx@lists.freedesktop.org
Subject: Re: [PATCH 3/4] tests/gem_userptr_blits: Expanded userptr test cases
Date: Tue, 4 Feb 2014 15:09:35 +0100 [thread overview]
Message-ID: <20140204140935.GU17001@phenom.ffwll.local> (raw)
In-Reply-To: <1391425183-1980-4-git-send-email-tvrtko.ursulin@linux.intel.com>
On Mon, Feb 03, 2014 at 10:59:42AM +0000, Tvrtko Ursulin wrote:
> From: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
>
> A set of userptr test cases to support the new feature.
>
> For the eviction and swapping stress testing I have extracted
> some common behaviour from gem_evict_everything and made both
> test cases use it to avoid duplicating the code.
>
> Both unsynchronized and synchronized userptr objects are
> tested but the latter set of tests will be skipped if kernel
> is compiled without MMU_NOTIFIERS.
>
> Also, with 32-bit userspace swapping tests are skipped if
> the system has a lot more RAM than process address space.
> Forking swapping tests are not skipped since they can still
> trigger swapping by cumulative effect.
>
> Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Nitpick on process: Please keep in per-patch changelog in the commit
message and update it every time you resend the patches (only if something
changed in that patch ofc). Makes it easier to keep track of things.
-Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
next prev parent reply other threads:[~2014-02-04 15:06 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-02-03 10:59 [PATCH 0/4] tests: New userptr test case Tvrtko Ursulin
2014-02-03 10:59 ` [PATCH 1/4] tests/gem_evict_everything: Factor out eviction logic Tvrtko Ursulin
2014-02-03 10:59 ` [PATCH 2/4] tests/eviction_common: Avoid submitting duplicate objects Tvrtko Ursulin
2014-02-04 12:25 ` Daniel Vetter
2014-02-03 10:59 ` [PATCH 3/4] tests/gem_userptr_blits: Expanded userptr test cases Tvrtko Ursulin
2014-02-04 14:09 ` Daniel Vetter [this message]
2014-02-03 10:59 ` [PATCH 4/4] tests/gem_vmap_blits: Remove obsolete test case Tvrtko Ursulin
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=20140204140935.GU17001@phenom.ffwll.local \
--to=daniel@ffwll.ch \
--cc=Intel-gfx@lists.freedesktop.org \
--cc=tvrtko.ursulin@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox