On 02/28/2013 03:52 AM, Kristian Høgsberg wrote:
--- include/GL/internal/dri_interface.h | 14 +++++++- src/mesa/drivers/dri/intel/intel_regions.c | 33 +++++++++++++++++++ src/mesa/drivers/dri/intel/intel_regions.h | 6 ++++ src/mesa/drivers/dri/intel/intel_screen.c | 53 ++++++++++++++++++++++++++++-- 4 files changed, 102 insertions(+), 4 deletions(-)
[...]
diff --git a/src/mesa/drivers/dri/intel/intel_screen.c b/src/mesa/drivers/dri/intel/intel_screen.c index 277b133..3ee0d69 100644 --- a/src/mesa/drivers/dri/intel/intel_screen.c +++ b/src/mesa/drivers/dri/intel/intel_screen.c @@ -546,6 +546,9 @@ intel_query_image(__DRIimage *image, int attrib, int *value) return false; *value = image->planar_format->components; return true; + case __DRI_IMAGE_ATTRIB_FD: + drm_intel_bo_gem_export_to_prime(image->region->bo, value); + return true;
Need to check the return value of drm_intel_bo_gem_export_to_prime(). [...]
.createImageFromRenderbuffer = intel_create_image_from_renderbuffer, @@ -705,6 +751,7 @@ static struct __DRIimageExtensionRec intelImageExtension = { .createImageFromNames = intel_create_image_from_names, .fromPlanar = intel_from_planar, .createImageFromTexture = intel_create_image_from_texture + .createImageFromFd = intel_create_image_from_fd
There's missing comma above, as you pointed out in IRC. Other than this, the patch seems fine to me. Cheers, Ander _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev