On Sat 29 Apr 2017, Xu, Randy wrote:
> > -----Original Message-----
> > From: Emil Velikov [mailto:[email protected]]
> > Sent: Saturday, April 29, 2017 8:36 PM
> > To: Chad Versace <[email protected]>; Emil Velikov
> > <[email protected]>; Xu, Randy <[email protected]>; mesa-
> > [email protected]
> > Subject: Re: [Mesa-dev] [PATCH] i965: Solve Android native fence fd double
> > close issue
> > 
> > On 28 April 2017 at 17:18, Chad Versace <[email protected]>
> > wrote:


> > > 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,  if create_fence_fd just transfer fd ownership to the driver, 
> not dup, we also need change dri2_dup_native_fence_fd, which should not
> keep the fd after calls get_fence_fd. It should be workable, but I need do
> more test.

I suspect that it may be possible to eliminate _EGLSync::SyncFd, and
store the fd only in the driver. That would eliminate any confusion
about the semantics of DRI's create_fence_fd.

> Chad, please help to polish my comments, or give me some suggestions.

Randy, the patch is pushed to master.
_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to