<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Apr 11, 2016 at 8:25 AM, Ilia Mirkin <span dir="ltr"><<a href="mailto:imirkin@alum.mit.edu" target="_blank">imirkin@alum.mit.edu</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">On Mon, Apr 11, 2016 at 11:18 AM, Jakob Sinclair<br>
<<a href="mailto:sinclair.jakob@openmailbox.org">sinclair.jakob@openmailbox.org</a>> wrote:<br>
> On 2016-04-11 14:07, Marek Olšák wrote:<br>
>><br>
>> Hi Jakob,<br>
>><br>
>> Getting familiar with the code should be the first priority. I suggest<br>
>> you start with a simple feature like adding an easy GL or GL ES<br>
>> extension into your favourite driver.<br></span></blockquote><div><br></div><div>Agreed.  One easy extension for i965 that I've been trying to pawn off on someone for a while is ARB_shader_atomic_counter_ops:<br><br><a href="https://www.opengl.org/registry/specs/ARB/shader_atomic_counter_ops.txt">https://www.opengl.org/registry/specs/ARB/shader_atomic_counter_ops.txt</a><br><br></div><div>Ilia already got it plumbed through the core GLSL compiler and into gallium.  Getting it hooked up in i965 should be mostly just plumbing work.  It would provide a good compiler intro.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">
>> You should also use a test suite for testing driver correctness. The<br>
>> main one everybody should use is called "piglit". Some people also use<br>
>> dEQP.<br></span></blockquote><div><br></div><div>Writing tests can also be a decent place to start.  There are many places where piglit testing can be improved.  That said, it helps if you have a particular bug or some new piece of functionality to test.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">
>> Marek<br>
><br>
><br>
> Thanks for the tips! I have been looking at the ARB_shader_image_load_store<br>
> and ARB_shader_image size extensions for GL ES 3.1 and noticed that the<br>
> radeonsi drivers does not support them even though the driver supports the<br>
> extensions in the core profile. From what I can tell all supported functions<br>
> are already in the driver. What needs to be added to the driver to gain<br>
> support for the extensions in a GL ES 3.1 context?<br>
<br>
</span>Nothing. They aren't extensions in GL ES 3.1, but rather core<br>
features. Have you tested them and found that something was<br>
unsupported through GL ES 3.1 entrypoints?<br></blockquote><div><br></div><div>Another pretty easy task is to find GL ES extensions where the desktop variant is already implemented.  Then 95% of the code is there, you just have to see what (if anything) is different with the ES variant and wire it up.  If you're going to do that, might be good to coordinate with Ilia.  He's been knocking them out like crazy lately.<br><br></div><div>--Jason <br></div></div></div></div>