From: Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>
To: Chris Wilson <chris@chris-wilson.co.uk>, intel-gfx@lists.freedesktop.org
Subject: Re: [PATCH RFC] drm/i915: Pad GTT views of exec objects up to user specified size
Date: Wed, 25 Mar 2015 15:16:58 +0000 [thread overview]
Message-ID: <5512D16A.5040404@linux.intel.com> (raw)
In-Reply-To: <1427290150-21144-1-git-send-email-chris@chris-wilson.co.uk>
On 03/25/2015 01:29 PM, Chris Wilson wrote:
> Our GPUs impose certain requirements upon buffers that depend upon how
> exactly they are used. Typically this is expressed as that they require
> a larger surface than would be naively computed by pitch * height.
> Normally such requirements are hidden away in the userspace driver, but
> when we accept pointers from strangers and later impose extra conditions
> on them, the original client allocator has no idea about the
> monstrosities in the GPU and we require the userspace driver to inform
> the kernel how many padding pages are required beyond the client
> allocation.
>
> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
> Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
> ---
> Tvrtko is working on an igt to exercise this feature. The original
> request basically stems from trying to mix client allocations (userptr)
> and tiling. But I see it as a nice extension of our lazy_fencing code to
> reduce buffer sizes for tiled allocations (as we then do not need to
> allocate the entire last tilerow) and for relaxing restrictions on
> foriegn buffers like DRI3.
> ---
(Original request was actually about sampler overfetch in various
texture modes.)
We talked about per-process (or so) scratch pages before. I looked now
but I see at the moment even PPGTT shares the same scratch page so
presumably that means someone has thought about security and decided
that is fine?
Otherwise looks good to me.
Regards,
Tvrtko
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
next prev parent reply other threads:[~2015-03-25 15:17 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-03-25 13:29 [PATCH RFC] drm/i915: Pad GTT views of exec objects up to user specified size Chris Wilson
2015-03-25 15:16 ` Tvrtko Ursulin [this message]
2015-03-25 20:01 ` shuang.he
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=5512D16A.5040404@linux.intel.com \
--to=tvrtko.ursulin@linux.intel.com \
--cc=chris@chris-wilson.co.uk \
--cc=intel-gfx@lists.freedesktop.org \
/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