All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>
To: Chris Wilson <chris@chris-wilson.co.uk>,
	Intel-gfx@lists.freedesktop.org,
	Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Subject: Re: [RFC i-g-t] tests/gem_exec_pad_to_size: Test object padding at execbuf
Date: Wed, 25 Mar 2015 14:57:32 +0000	[thread overview]
Message-ID: <5512CCDC.9050200@linux.intel.com> (raw)
In-Reply-To: <20150325144110.GB13857@nuc-i3427.alporthouse.com>


On 03/25/2015 02:41 PM, Chris Wilson wrote:
> On Wed, Mar 25, 2015 at 02:32:05PM +0000, Tvrtko Ursulin wrote:
>>
>> On 03/25/2015 02:28 PM, Chris Wilson wrote:
>>> What's important to make this trick work is to allocate new handles
>>> every time. That way we fill up the GTT and thereby hope to skip over
>>> the fragmented part.
>>
>> It does do that!
>
> I was just looking for some filler for the email!

Try Reviewed-by? :D

>> Only perhaps I need to make sure they are kept bound so maybe keep
>> them mapped or something while trying new ones?
>
> What I was expecting to see were gem_create()s every loop. I only feel
> confident in say that a pair of freshly allocated and bound objects are
> likely to line up against each other (so long as space is not
> fragmented).
>
> I would allocate a new device fd for the test, leak the individual
> handles on every try, then close the fd at the end of the test to avoid
> having to track all the allocations. That's the pattern I was looking
> for when I skimmed over your test.

Apart from reusing the fd, tracking the handles and allocating on all 
loops but the first one when it re-uses the passed in ones, it is 
exactly like that! :)

So far it always manages to get neighbouring BOs on my system on first 
try. Which is probably not unexpected since is is PPGTT.

One thing I wasn't sure about is max_tries = 1024. That will be 8MB of 
filling to avoid fragmentation. Will that be enough on GGTT systems?

Regards,

Tvrtko
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

  reply	other threads:[~2015-03-25 14:57 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-03-25 14:21 [RFC i-g-t] tests/gem_exec_pad_to_size: Test object padding at execbuf Tvrtko Ursulin
2015-03-25 14:28 ` Chris Wilson
2015-03-25 14:32   ` Tvrtko Ursulin
2015-03-25 14:41     ` Chris Wilson
2015-03-25 14:57       ` Tvrtko Ursulin [this message]
2015-04-01 11:21 ` [RFC i-g-t v2] " Tvrtko Ursulin
2015-04-01 13:06   ` Chris Wilson
2015-04-01 13:17     ` Chris Wilson
2015-04-01 13:36     ` Tvrtko Ursulin
2015-04-01 13:56       ` Chris Wilson
2015-04-01 14:14         ` Tvrtko Ursulin
2015-04-01 14:27           ` Chris Wilson
2015-04-01 15:14 ` [RFC i-g-t v3] " Tvrtko Ursulin
2015-04-01 15:42   ` Chris Wilson
2015-04-01 16:07     ` Tvrtko Ursulin
2015-04-01 16:31       ` Chris Wilson
2015-04-01 16:39         ` Chris Wilson
2015-04-02 10:47           ` Tvrtko Ursulin
2015-04-02 10:45 ` [RFC i-g-t v4] " Tvrtko Ursulin
2015-04-02 11:02   ` Chris Wilson
2015-04-02 12:54 ` [RFC i-g-t v5] " Tvrtko Ursulin
2015-04-02 16:09   ` Thomas Wood

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=5512CCDC.9050200@linux.intel.com \
    --to=tvrtko.ursulin@linux.intel.com \
    --cc=Intel-gfx@lists.freedesktop.org \
    --cc=chris@chris-wilson.co.uk \
    --cc=tvrtko.ursulin@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.