[Mesa-dev] [PATCH 2/3] egl: return corresponding offset of EGLImage instead of 0.

[Weng, Chuanbo]

From: mesa-dev [mailto:mesa-dev-bounces at lists.freedesktop.org] On Behalf Of Axel Davy
Sent: Friday, September 9, 2016 2:12 PM
To: Weng, Chuanbo <chuanbo.weng at intel.com>; mesa-dev at lists.freedesktop.org; emil.l.velikov at gmail.com
Subject: Re: [Mesa-dev] [PATCH 2/3] egl: return corresponding offset of EGLImage instead of 0.






That doesn't seem good to me.



With that patch, that means that since no one is implementing



__DRI_IMAGE_ATTRIB_OFFSET



(yes I know in a later patch you implement it for i965),



then what used to work will stop working (as the queryImage will return false).



You need to introduce some interface version implementation check.

[****Chuanbo****] Maybe I can add more comment to git log (such as "This patch just implements egl loader side, the driver side

implementation is also needed for corresponding platform"), so user can be aware of this.

Introduce interface version implementation check will make mesa code more complex, because we should also add related check to

other dri2 functions(dri2_****).

Another solution is combining the three patches into one patch, as I did before:

https://lists.freedesktop.org/archives/mesa-dev/2016-August/126945.html

This is not as easy as this version for reviewers, but more clearer for users.

Emil, what do you think?



No, that's not ok.

First i965 isn't the only one to implement the dri image interface (see the gallium one), second a new implementer doesn't have to start from the most recent version, and can choose to implement older version, which wouldn't implement your new functionnality.

[****Chuanbo****] OK. I’ll send out a new version based on your comments. Thanks.



The code has to be something like:

if (offsets) {

   offsets[0] = 0;

   if (dri2_dpy->image->base.version >= 13)

dri2_dpy->image->queryImage(dri2_img->dri_image, __DRI_IMAGE_ATTRIB_OFFSET, offsets);

}

Axel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20160909/b26ff12f/attachment-0001.html>