Chad Versace <[email protected]> writes:

> Add two new functions: intel_miptree_{map,unmap}_multisample, to which
> intel_miptree_{map,unmap} dispatch. Only mapping flat, renderbuffer-like
> miptrees are supported.

>  void
> @@ -1513,5 +1606,8 @@ intel_miptree_unmap(struct intel_context *intel,
>                   unsigned int level,
>                   unsigned int slice)
>  {
> -   intel_miptree_unmap_singlesample(intel, mt, level, slice);
> +   if (mt->num_samples == 0)

Isn't mt->num_samples sometimes 1 for a singlesample mt now?

> +      intel_miptree_unmap_singlesample(intel, mt, level, slice);
> +   else
> +      intel_miptree_unmap_multisample(intel, mt, level, slice);
>  }

Other than that, downsamples get done in some cases where needs
needs_downsample was false, which is suboptimal.  But I think the code
is correct, and this is a fallback path so I don't care too much.

Attachment: pgpUQXmPYgCuQ.pgp
Description: PGP signature

_______________________________________________
mesa-dev mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to