On July 26, 2017 8:48:18 AM Samuel Pitoiset <samuel.pitoi...@gmail.com> wrote:
Hi guys,
I didn't review the radeonsi and radv patches, but I have sent comments
on other parts.
More generally:
- I wonder if some intermediate patches can break Mesa, would be nice to
avoid that (especially for bisecting) Can you double check?
- I don't see any piglit tests... or did I miss them? :)
+1 on the need for tests. I'll go even further and say that the obvious
minimal testing to exercise the extension is far from sufficient. There
are a lot of complex ways in which two APIs can interact and we need to
ensure we get it right. In particular, if we're going to claim that the
extension actually works properly we need:
1. Tests which use more than just GL. In particular, we need to test
sharing between GL and Vulcan.
2. Test which exercise "complex" textures, i.e. textures with multiple
mip-levels, multiple array slices, 1D, 2D, 3D, etc.
3. Tests which test different texture formats.
4. Tests which test several different combinations of rendering, clearing,
glTexSubImage, etc. in one API and combinations of texturing,
glGetTexSubImage, etc. in the other API.
Preferably, a good set of combinations of the above.
Also, the Vulcan bits need to pass validation. I shouldn't have to say
this but it's important and an easy thing to forget so I thought it worth
mentioning.
--Jason
Thanks,
Samuel.
On 07/26/2017 01:46 PM, Timothy Arceri wrote:
Hi all,
Andres is not around at the moment so as well are reviewing the
remaining patches I've rebased and added all Marek's suggestions.
I've also made a few minor changes (see commit messages) and
reworked some of the patches to reduce code churn.
I thought I'd send it out one last time to see if there was any
more feedback otherwise I'll probably push later in the week.
Thanks,
Tim
Series available at:
https://github.com/tarceri/Mesa.git (memobj2)
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev