[Intel-gfx] [PATCH] drm/agp: agp-intel/i915: trim stolen space to 16M

[Jesse Barnes]

On Wed, 24 Mar 2010 21:25:07 +0100
Daniel Vetter <daniel at ffwll.ch> wrote:

> On Wed, Mar 24, 2010 at 01:06:05PM -0700, Eric Anholt wrote:
> > I love the idea!  The interactions between the two modules are sure
> > ugly, though -- like the duplication of i915_probe_agp().  That's part
> > of why I've wished we would just pull GTT setup into the DRM and leave
> > intel-agp to rot for non-KMS support that actually uses the agpgart
> > interfaces.
> 
> Fixing up the interaction between the gtt driver in intel-agp and the i915
> drm modules is my current pet project. We can't remove the gtt support
> without breaking old non-gem userspace code, though. So my idea is to
> create a private, direct interface to the gtt for gem and implement the
> agp stuff on top of it. First step in this direction is my patch-pile to
> seperate the gtt support code from the agp driver I've sent a few days
> ago.
> 
> Does this sound like a reasonable plan to go forward? Obviously my patches
> conflict with all recent patches against intel-agp. I've you'd ack the
> general direction I could rebase these patches at a convenient time and
> then start with clean-ups and killing duplicate code when you've merged
> them.

That sounds like a good approach.  Basically the current AGP interfaces
would be deprecated unless someone was using the old GART ioctls?
Sounds great to me.

-- 
Jesse Barnes, Intel Open Source Technology Center