<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Oct 23, 2017 at 6:27 AM, Nicolai Hähnle <span dir="ltr"><<a href="mailto:nhaehnle@gmail.com" target="_blank">nhaehnle@gmail.com</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="gmail-">On 21.10.2017 03:00, Dylan Baker wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
This very short series creates a new src/wsi folder, and moves wayland-drm into<br>
it. Basically wsi stuff is scattered about, and is needed by multiple components<br>
within mesa, wayland-drm, for example, is used by EGL, GBM, and vulkan<br>
wayland-wsi.<br>
<br>
I think there's more that could be moved into wsi, we could move EGL, GBM, and<br>
GLX, and vulkan/wsi, for example.<br>
</blockquote>
<br></span>
The general thrust sounds good to me.<br>
<br>
Is there a clean model for what should go into src/wsi and what shouldn't? Where's the boundary?<br>
<br>
For an example of the type of headaches, does DRI driver code (stuff that ends up in xxx_dri.so, for example) count as part of src/wsi? If so, then what about gallium/state_trackers/dri? What about mesa/drivers/dri/common?<br></blockquote><div><br></div><div>Maybe?  I guess it depends on what it does and how much sense it makes to share it higher than the gallium level.<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">
I don't have a full picture of all this code so it's hard for me to say, but I really hope your changes will lead to a clearer picture overall :)<br></blockquote></div></div><div class="gmail_extra"><br></div><div class="gmail_extra">Dylan and I talked about it quite a bit off-line so I have a few opinions. :)  In particular, here's what I envison:</div><div class="gmail_extra"><br></div><div class="gmail_extra">src/wsi/gbm</div><div class="gmail_extra">src/wsi/egl</div><div class="gmail_extra">src/wsi/wayland-drm</div><div class="gmail_extra">src/wsi/dri3 (currently src/loader)</div><div class="gmail_extra">src/wsi/glx</div><div class="gmail_extra">src/wsi/vulkan (currently src/vulkan/wsi)</div><div class="gmail_extra">src/wsi/hgl</div><div class="gmail_extra"><br></div><div class="gmail_extra">So, basically, it's window system integration layers and the helpers we want to share among them.  I wasn't envisioning it containing anything to handle the DRI layer though I suppose it probably could.</div><div class="gmail_extra"><br></div><div class="gmail_extra">--Jason<br></div></div>