[Mesa-dev] radv initial submission
nhaehnle at gmail.com
Tue Oct 4 11:03:56 UTC 2016
On 04.10.2016 03:48, Dave Airlie wrote:
> Hi all,
> I fully expect this won't make it to the list in one piece due to size.
> I've pushed the same patchset to
Even Github refuses to render the diff of patch #3 for me, but I skimmed
over it using proper tools ;-)
Some things I noticed:
- radv_device_info.h has no copyright header
- radv_timestamp.h shouldn't be in Git
- Makefile.am still refers to GEN7_FILES
- lots of odd-looking alignment and whitespace at end-of-lines; please
at least fix the whitespace at EOL before pushing, since that tends to
create annoying churn
I've also sent comments on the other patches. I'd say this is fairly
close to mergeable as far as I'm concerned. Congratulations on your
> This is an initial submission of the open source radv vulkan driver
> for AMD GPUs supported by the amdgpu driver. It is a project Bas and I
> started some months ago as an interesting learning experience and it's
> been great to see it get this far in such a short time.
> This driver is in no way to be considered conformant or called such.
> It has NOT passed the vulkan conformance suite or ever been submitted
> to Khronos. It prints a big warning on startup to this effect.
> In saying that, it runs quite well with Dota 2, Talos Principle and
> vkQuake in our testing so far.
> Future plans are to add support for various hw speedups (fast clears),
> missing things like spilling, and multiple queues along with finishing
> out things like geom/tess shaders. Also trying to reuse code between
> this and the radeonsi driver where possible. radv already reuses
> the LLVM compiler backend and the addrlib code.
> I've split things out into 4 patches, 3 just add code files to various
> places and number 4 intergrates the build system changes so they
> are easier to review.
> Dave (and on behalf of Bas).
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
More information about the mesa-dev