On Mon, 6 Jun 2011 15:18:44 +0100, Chris Wilson <[email protected]> wrote:
> case I915_PARAM_HAS_RELAXED_FENCING: > - value = 1; > + value = 2; This looks like a change in ABI to me. I think this means you want a new ioctl so that applications using the existing HAS_RELAXED_FENCING ioctl and expecting a boolean will continue to work. > uint32_t > -i915_gem_get_unfenced_gtt_alignment(struct drm_i915_gem_object *obj); > +i915_gem_get_unfenced_gtt_alignment(struct drm_i915_gem_object *obj, > + int tiling_mode); ... > - i915_gem_get_unfenced_gtt_alignment(obj); > + i915_gem_get_unfenced_gtt_alignment(obj, > + > args->tiling_mode); This combination looks like a simple bug fix -- not using the new tiling mode when computing the required alignment. Can you separate out this From the power-of-two change? -- [email protected]
pgpUe4rfzexIL.pgp
Description: PGP signature
_______________________________________________ stable mailing list [email protected] http://linux.kernel.org/mailman/listinfo/stable
