On 28 April 2017 at 17:18, Chad Versace <chadvers...@chromium.org> wrote: > On Thu 27 Apr 2017, Emil Velikov wrote: >> On 27 April 2017 at 12:14, Xu, Randy <randy...@intel.com> wrote: >> > Hi, Chad >> > >> > Please review this patch, we need it to solve some instability issues > > Randy and Tapani, could you provide a few dEQP test names that this > patch fixes? I'd like to mention at least one EGL and one Vulkan test in > the commit message. > >> The patch is correct, although the commit message can be improved upon. >> Read through the following example and consider the alternative >> solution mentioned within. > > Yes, this patch is correct. It makes brw_dri_create_fence_fd() behave > like all the other drivers' create_fence_fd funcs, which call dup(). > Since this is an easy one-liner that can backport to stable, let's take > it. > > However, I believe the fully correct solution is Emil's plan B: > __DRI2fenceExtensionRec::create_fence_fd should transfer fd ownership to > the driver, and therefore no dup is needed. But that's a slightly more > invasive change that's not as easily backported to stable. > > Reviewed-by: Chad Versace <chadvers...@chromium.org> > Cc: mesa-sta...@lists.freedesktop.org > > Emil, how about one of us appends your extended commit message to > Randy's, and then pushes? > Feel free to polish and push Chad.
Randy please don't forget to send a patch for dri_interface.h. Thanks Emil _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev