[Mesa-dev] [PATCH] egl/dri2: Allow modifiers to add FDs to imports
Tapani Pälli
tapani.palli at intel.com
Tue Aug 8 04:29:53 UTC 2017
On 08/07/2017 03:05 PM, Philipp Zabel wrote:
> On Mon, 2017-07-31 at 18:35 +0100, Daniel Stone wrote:
>> When using dmabuf import, make sure that the modifier is actually
>> allowed to add planes to the base format, as implied by the comment.
>>
>> Signed-off-by: Daniel Stone <daniels at collabora.com>
>> ---
>> src/egl/drivers/dri2/egl_dri2.c | 4 +++-
>> 1 file changed, 3 insertions(+), 1 deletion(-)
>>
>> diff --git a/src/egl/drivers/dri2/egl_dri2.c b/src/egl/drivers/dri2/egl_dri2.c
>> index b73dcd72b6..76294897a5 100644
>> --- a/src/egl/drivers/dri2/egl_dri2.c
>> +++ b/src/egl/drivers/dri2/egl_dri2.c
>> @@ -2166,8 +2166,10 @@ dri2_check_dma_buf_format(const _EGLImageAttribs *attrs)
>> * this extension."
>> */
>> if (attrs->DMABufPlaneModifiersLo[i].IsPresent &&
>> - attrs->DMABufPlaneModifiersHi[i].IsPresent)
>> + attrs->DMABufPlaneModifiersHi[i].IsPresent) {
>> + plane_n = i + 1;
>
> Since this increments plane_n, Should a check be added that the
> corresponding DMABufPlanFds[i] is present?
Check for the fd is right above this check.
> What if there are holes in DMABufPlaneModifiersLo/Hi?
>
>> continue;
>> + }
>>
>> _eglError(EGL_BAD_ATTRIBUTE, "too many plane attributes");
>> return 0;
>
> Reviewed-by: Philipp Zabel <p.zabel at pengutronix.de>
>
> regards
> Philipp
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
More information about the mesa-dev
mailing list