On Thu, Jan 09, 2014 at 11:32:10AM -0800, Jose Fonseca wrote: > This series is pretty much what I proposed several months ago, rebased on top > of master, with minor if any tweaks. > > Still to do is remove the deprecated abstractions (which now are mere > wrappers of the c11 ones). > > That said, this series is already useful as is, not only because of > the code de-duplication, but also because the Windows conditional var > implemention in master is poll-based so performs quite badly, in > particular llvmpipe with many threads. Which is fixed here. > > IMHO the rest can be done as time permits... > > Jose
This series looks good to me. But I'd like to see patch 7 squashed into patch 2 to prevent possible build failures during bisects. I like how this series opens up opportunities for additional cleanups. In particular, we can now begin to remove the threading wrappers in src/egl/main. I'm not familiar with C11's thread headers, so patch 2 is Acked-by: Chad Versace <chad.vers...@linux.intel.com> Patches 3-7 are: Reviewed-by: Chad Versace <chad.vers...@linux.intel.com> _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
