On 31/01/2014, Michel Dänzer wrote :
On Don, 2014-01-30 at 16:10 +0100, Axel Davy wrote:
dri2_dup_image was not copying the dri_format field.

This was causing some bugs, for example:
. we create an gbm_bo.
. we get an EGLImage from the gbm_bo.
. Bug: impossible to get again the gbm_bo from the EGLImage by importing. (gbm 
dri2 backend)

Signed-off-by: Axel Davy <[email protected]>
---
  src/gallium/state_trackers/dri/drm/dri2.c | 1 +
  1 file changed, 1 insertion(+)

diff --git a/src/gallium/state_trackers/dri/drm/dri2.c 
b/src/gallium/state_trackers/dri/drm/dri2.c
index 2a5b7b4..0589644 100644
--- a/src/gallium/state_trackers/dri/drm/dri2.c
+++ b/src/gallium/state_trackers/dri/drm/dri2.c
@@ -777,6 +777,7 @@ dri2_dup_image(__DRIimage *image, void *loaderPrivate)
     pipe_resource_reference(&img->texture, image->texture);
     img->level = image->level;
     img->layer = image->layer;
+   img->dri_format = image->dri_format;
     /* This should be 0 for sub images, but dup is also used for base images. 
*/
     img->dri_components = image->dri_components;
     img->loader_private = loaderPrivate;
Reviewed-by: Michel Dänzer <[email protected]>

Do you have Git write access?


I don't have Git access, could you push the patch?

Thanks,

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

Reply via email to