Re: [PATCH] i915: Add support for drm syncobjs

2017-08-03 Thread Jason Ekstrand
On Thu, Aug 3, 2017 at 11:15 AM, Chris Wilson wrote: > Quoting Jason Ekstrand (2017-08-03 19:06:02) > > I'm not concerned about what happens to racy clients. They get what > they get. > > What concerns me is what happens if somehow the fence is replaced and > deleted > > before i915_gem_request_

Re: [PATCH] i915: Add support for drm syncobjs

2017-08-03 Thread Chris Wilson
Quoting Jason Ekstrand (2017-08-03 19:06:02) > I'm not concerned about what happens to racy clients.  They get what they > get.  > What concerns me is what happens if somehow the fence is replaced and deleted > before i915_gem_request_await_dma_fence takes it's reference.  Can this cause > the ker

Re: [PATCH] i915: Add support for drm syncobjs

2017-08-03 Thread Jason Ekstrand
On Thu, Aug 3, 2017 at 10:00 AM, Chris Wilson wrote: > Quoting Jason Ekstrand (2017-07-05 22:15:09) > > On Wed, Jul 5, 2017 at 2:13 PM, Jason Ekstrand > wrote: > > > > This commit adds support for waiting on or signaling DRM syncobjs as > > part of execbuf. It does so by hijacking the c

Re: [PATCH] i915: Add support for drm syncobjs

2017-08-03 Thread Chris Wilson
Quoting Jason Ekstrand (2017-07-05 22:15:09) > On Wed, Jul 5, 2017 at 2:13 PM, Jason Ekstrand wrote: > > This commit adds support for waiting on or signaling DRM syncobjs as > part of execbuf.  It does so by hijacking the currently unused cliprects > pointer to instead point to an arr

Re: [PATCH] i915: Add support for drm syncobjs

2017-07-05 Thread Jason Ekstrand
On Wed, Jul 5, 2017 at 2:13 PM, Jason Ekstrand wrote: > This commit adds support for waiting on or signaling DRM syncobjs as > part of execbuf. It does so by hijacking the currently unused cliprects > pointer to instead point to an array of i915_gem_exec_fence structs > which containe a DRM sync

[PATCH] i915: Add support for drm syncobjs

2017-07-05 Thread Jason Ekstrand
This commit adds support for waiting on or signaling DRM syncobjs as part of execbuf. It does so by hijacking the currently unused cliprects pointer to instead point to an array of i915_gem_exec_fence structs which containe a DRM syncobj and a flags parameter which specifies whether to wait on it

Re: [PATCH] i915: Add support for drm syncobjs

2017-07-05 Thread Jason Ekstrand
On Wed, Jul 5, 2017 at 11:42 AM, Chris Wilson wrote: > Quoting Jason Ekstrand (2017-07-05 19:32:21) > > On Wed, Jul 5, 2017 at 10:37 AM, Chris Wilson > wrote: > > > > Quoting Jason Ekstrand (2017-07-05 18:21:22) > > > This commit adds support for waiting on or signaling DRM syncobjs > as

Re: [PATCH] i915: Add support for drm syncobjs

2017-07-05 Thread Chris Wilson
Quoting Jason Ekstrand (2017-07-05 19:32:21) > On Wed, Jul 5, 2017 at 10:37 AM, Chris Wilson > wrote: > > Quoting Jason Ekstrand (2017-07-05 18:21:22) > > This commit adds support for waiting on or signaling DRM syncobjs as > > part of execbuf.  It does so by hijacking the currently

Re: [PATCH] i915: Add support for drm syncobjs

2017-07-05 Thread Jason Ekstrand
On Wed, Jul 5, 2017 at 10:37 AM, Chris Wilson wrote: > Quoting Jason Ekstrand (2017-07-05 18:21:22) > > This commit adds support for waiting on or signaling DRM syncobjs as > > part of execbuf. It does so by hijacking the currently unused cliprects > > pointer to instead point to an array of i91

Re: [PATCH] i915: Add support for drm syncobjs

2017-07-05 Thread Chris Wilson
Quoting Jason Ekstrand (2017-07-05 18:21:22) > This commit adds support for waiting on or signaling DRM syncobjs as > part of execbuf. It does so by hijacking the currently unused cliprects > pointer to instead point to an array of i915_gem_exec_fence structs > which containe a DRM syncobj and a f

[PATCH] i915: Add support for drm syncobjs

2017-07-05 Thread Jason Ekstrand
This commit adds support for waiting on or signaling DRM syncobjs as part of execbuf. It does so by hijacking the currently unused cliprects pointer to instead point to an array of i915_gem_exec_fence structs which containe a DRM syncobj and a flags parameter which specifies whether to wait on it