<div dir="ltr">You are correct that we haven't keep compatible with latest mesa on the cl-gl<div>sharing extension. The current implementation is such a hacky way that it highly</div><div>depends on the mesa/dri driver's internal implementation. Just as you mentioned,</div>
<div>the mesa/dri driver have changed a lot recently, and the cl-gl extension in beignet</div><div>is broken with the upstream git master version.</div><div><br></div><div>You can see that the major focus of our team is not cl-gl sharing extension</div>
<div>currently.  Please feel free to do it at your own pace. And don't need to keep</div><div>backward compatible with the old mesa repository. IMO, to keep it with latest</div><div>upstream mesa is always a good thing so far.</div>
<div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, May 3, 2014 at 2:16 AM, Steven Newbury <span dir="ltr"><<a href="mailto:steve@snewbury.org.uk" target="_blank">steve@snewbury.org.uk</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Beignet has got a little left behind upstream Mesa changes so I've been<br>
patching to compile against mesa/master for my gentoo-opencl overlay.<br>
Eric Anholt has just landed a significant patch series dropping support<br>
for regions in the i965 driver.  Unfortunately, the Beignet DRI resource<br>
sharing code depended on that functionality, so I've been working on<br>
converting it to use buffer objects like the other previous users of the<br>
region code.<br>
<br>
First an old issue that I've just worked around for now, I can't figure<br>
out how to fix, but needs addressing:<br>
<br>
i965/intel_context.h has been merged with i965/brw_context.h which in<br>
turn needs to be compiled with -std=c99/gnu99, which then breaks<br>
beignet/src/cl_extensions.c, where this happens:<br>
<br>
<br>
> cl_extensions.c:19:12: error: initializer element is not constant<br>
>    DECL_ALL_EXTENSIONS<br>
>             ^<br>
<br>
cl_utils.h seriously clashes with headers pulled in by brw_context.h.<br>
<br>
There are a small number of easy fixes, for which I'll see if I can make<br>
conditional on Mesa version and send to the list.<br>
<br>
As for the big one, the intel_region struct is gone, I've had a shot at<br>
converting the code, I'll keep plugging away when I have time in a<br>
couple of days, but if somebody else wants to give it a go don't let me<br>
hold you back! :)  A question though, should the code need to also<br>
compile against older release versions of Mesa, or is it sufficient to<br>
just target current upstream and have it maybe land in a branch?<br>
<br>
<br>
_______________________________________________<br>
Beignet mailing list<br>
<a href="mailto:Beignet@lists.freedesktop.org">Beignet@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/beignet" target="_blank">http://lists.freedesktop.org/mailman/listinfo/beignet</a><br>
</blockquote></div><br></div></div></div>