RE: [PATCH v3 3/5] ui: Create sync objects and fences only for blobs

2021-06-24 Thread Kasireddy, Vivek
Hi Gerd, > > > > > > > dmabuf->buf.fd = res->dmabuf_fd; > > > > +dmabuf->buf.blob = true; > > > > > > Do you actually need the 'blob' field? > > > I think checking 'fd' instead should work too. > > [Kasireddy, Vivek] I want these changes to be limited to blob resources > > only as I do n

Re: [PATCH v3 3/5] ui: Create sync objects and fences only for blobs

2021-06-24 Thread Gerd Hoffmann
> > Hi, > > > > > dmabuf->buf.fd = res->dmabuf_fd; > > > +dmabuf->buf.blob = true; > > > > Do you actually need the 'blob' field? > > I think checking 'fd' instead should work too. > [Kasireddy, Vivek] I want these changes to be limited to blob resources only > as I do not > know how

RE: [PATCH v3 3/5] ui: Create sync objects and fences only for blobs

2021-06-23 Thread Kasireddy, Vivek
Hi Gerd, > Hi, > > > dmabuf->buf.fd = res->dmabuf_fd; > > +dmabuf->buf.blob = true; > > Do you actually need the 'blob' field? > I think checking 'fd' instead should work too. [Kasireddy, Vivek] I want these changes to be limited to blob resources only as I do not know how they might

Re: [PATCH v3 3/5] ui: Create sync objects and fences only for blobs

2021-06-23 Thread Gerd Hoffmann
Hi, > dmabuf->buf.fd = res->dmabuf_fd; > +dmabuf->buf.blob = true; Do you actually need the 'blob' field? I think checking 'fd' instead should work too. take care, Gerd