From: Adam Jackson <ajax@redhat.com>
To: Ben Widawsky <ben@bwidawsk.net>
Cc: intel-gfx@lists.freedesktop.org
Subject: Re: [RFC PATCH 00/11] i915 HW context support
Date: Tue, 14 Feb 2012 18:17:57 -0500 [thread overview]
Message-ID: <4F3AEBA5.6060504@redhat.com> (raw)
In-Reply-To: <1329253758-24943-1-git-send-email-ben@bwidawsk.net>
On 2/14/12 4:09 PM, Ben Widawsky wrote:
> The IOCTLs defined here may also be used with ppgtt (real ppgtt) but the
> interface may be insufficient for that. The contexts do provide a
>
A... what?
Series looks reasonable to me. The one thing that makes me nervous is
4/11 mentioning the BIOS, which honestly seems like an insane thing to
let your BIOS have _any_ say in, but then Intel sure does like to give
the BIOS a chance to make the OS less functional. The code in 7/11
effectively limits things to between 4k and 1M; does that actually make
sense? I can't imagine wanting to allocate only 4k if it turns out we'd
always need at least (say) 32k.
I have admittedly not read any of the docs on how this is meant to work,
apologies if that'd answer this.
- ajax
next prev parent reply other threads:[~2012-02-14 23:18 UTC|newest]
Thread overview: 45+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-02-14 21:09 [RFC PATCH 00/11] i915 HW context support Ben Widawsky
2012-02-14 21:09 ` [PATCH 01/11] drm/i915: MI_ARB_ON_OFF Ben Widawsky
2012-02-14 21:09 ` [PATCH 02/11] drm/i915: track tlb invalidate GFX_MODE state Ben Widawsky
2012-02-14 22:48 ` Eugeni Dodonov
2012-03-05 18:18 ` Ben Widawsky
2012-02-15 19:36 ` Eric Anholt
2012-03-05 18:19 ` Ben Widawsky
2012-02-14 21:09 ` [PATCH 03/11] drm/i915: PIPE_CONTROL_TLB_INVALIDATE Ben Widawsky
2012-02-14 21:09 ` [PATCH 04/11] drm/i915/context: CXT_SIZE register offsets added Ben Widawsky
2012-02-15 19:39 ` Eric Anholt
2012-02-15 20:05 ` Ben Widawsky
2012-02-14 21:09 ` [PATCH 05/11] drm/i915/context: Preliminary context support Ben Widawsky
2012-02-14 22:49 ` Eugeni Dodonov
2012-03-05 18:19 ` Ben Widawsky
2012-02-15 20:23 ` Eric Anholt
2012-03-05 18:20 ` Ben Widawsky
2012-02-14 21:09 ` [PATCH 06/11] drm/i915/context: ringbuffer context switch code Ben Widawsky
2012-02-15 20:11 ` Eric Anholt
2012-02-16 10:30 ` Ben Widawsky
2012-02-14 21:09 ` [PATCH 07/11] drm/i915/context: implementation details Ben Widawsky
2012-02-14 21:09 ` [PATCH 08/11] drm/i915/context: extend contexts to execbuffer2 Ben Widawsky
2012-02-14 22:55 ` Eugeni Dodonov
2012-02-15 19:55 ` Eric Anholt
2012-02-15 20:00 ` Ben Widawsky
2012-02-17 18:03 ` Jesse Barnes
2012-02-15 20:16 ` Eric Anholt
2012-03-05 18:18 ` Ben Widawsky
2012-02-14 21:09 ` [PATCH 09/11] drm/i915/context: add params Ben Widawsky
2012-02-14 22:56 ` Eugeni Dodonov
2012-02-15 19:41 ` Chris Wilson
2012-02-15 20:02 ` Ben Widawsky
2012-02-14 21:09 ` [PATCH 10/11] drm/i915/context: track contexts per device Ben Widawsky
2012-02-14 22:56 ` Eugeni Dodonov
2012-02-14 21:09 ` [PATCH 11/11] drm/i915: show contexts in debugfs Ben Widawsky
2012-02-14 22:57 ` Eugeni Dodonov
2012-02-14 23:17 ` Adam Jackson [this message]
2012-02-14 23:41 ` [RFC PATCH 00/11] i915 HW context support Eugeni Dodonov
2012-02-15 8:43 ` Ben Widawsky
2012-02-15 20:33 ` Eric Anholt
2012-02-16 12:04 ` Ben Widawsky
2012-02-16 12:21 ` Ben Widawsky
2012-02-16 13:58 ` Ben Widawsky
2012-02-16 18:57 ` Eric Anholt
2012-02-15 22:12 ` Daniel Vetter
2012-03-05 18:20 ` Ben Widawsky
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=4F3AEBA5.6060504@redhat.com \
--to=ajax@redhat.com \
--cc=ben@bwidawsk.net \
--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 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.