On Sun, 4 Nov 2012 09:21:27 -0800, Ben Widawsky <ben at bwidawsk.net> wrote: > void i915_gem_gtt_bind_object(struct drm_i915_gem_object *obj, > enum i915_cache_level cache_level) > { > struct drm_device *dev = obj->base.dev; > - unsigned int agp_type = cache_level_to_agp_type(dev, cache_level); > + if (INTEL_INFO(dev)->gen < 6) { > + unsigned int flags = (cache_level == I915_CACHE_LLC) ? > + AGP_USER_CACHED_MEMORY : AGP_USER_MEMORY; And we still failed to get this right. :( cache_level == I915_CACHE_NONE ? AGP_USER_MEMORY : AGP_USER_CACHED_MEMORY; -Chris -- Chris Wilson, Intel Open Source Technology Centre