Marek Olšák <mar...@gmail.com> writes:

> From: Nicolai Hähnle <nicolai.haeh...@amd.com>
>
> Allow the caller to specify the row stride (in bytes) with which an image
> should be mapped. Note that completely ignoring USER_STRIDE is a valid
> implementation of mapImage.
>
> This is horrible API design. Unfortunately, cros_gralloc does indeed have
> a horrible API design -- in that arbitrary images should be allowed to be
> mapped with the stride that a linear image of the same width would have.
>
> There is no separate capability bit because it's unclear how stricter
> requirements should be defined.

I think that this is a bad interface, and you should just push an extra
copy into the caller so that they're encouraged to fix their bad API
instead of pushing their mistake down into Mesa ABI that we get stuck
with forever.

Attachment: signature.asc
Description: PGP signature

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to