<div dir="ltr"><div dir="auto"><div><br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="m_-8193314265774096139quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="m_-8193314265774096139elided-text">
><br>
> This is something I've been on the fence about for a while, mostly<br>
> because the vulkan and gl driver for mesa come out of the same source<br>
> base.<br>
<br>
</div>In addition to the issues raised by Jason, radv isn't the only Vulkan<br>
driver for AMD GPUs.<br></blockquote></div></div></div><br></div><div>I don't think Andres patch should be made jump through hoops for a driver<br></div><div>that isn't open source. AMD are free to provide patches to support this use<br></div><div>case but asking anyone outside AMD to care for a piece of software that isn't<br></div><div>open source, isn't a very nice thing to do.<br><br></div><div>From reading Jason's requests it does look like just leave it to the drivers to<br></div><div>sort out is the best option, and for radeonsi/radv we could go with a static<br></div><div>string as we at least share addrlib in this world. Beyond that I think AMD would<br></div><div>need to propose later patch for interop with the pro driver.<br><br></div><div>Dave.<br><br></div></div>