On Tue, Nov 24, 2015 at 09:55:35AM +0100, Daniel Vetter wrote:
> On Mon, Nov 23, 2015 at 03:08:53PM -0800, Alex Goins wrote:
> > In intel_prepare_plane_fb, if fb is backed by dma-buf, wait for exclusive
> > fence
> >
> > v2: First commit
> > v3: Remove object_name_lock acquire
> > Move wait from intel_atomic_commit() to intel_prepare_plane_fb()
> > v4: Wait only on exclusive fences, interruptible with no timeout
> > v5: Style tweaks to more closely match rest of file
> > v6: Properly handle interrupted waits
> >
> > Signed-off-by: Alex Goins <agoins at nvidia.com>
> > ---
> > drivers/gpu/drm/i915/intel_display.c | 11 +++++++++++
> > 1 file changed, 11 insertions(+)
> >
> > diff --git a/drivers/gpu/drm/i915/intel_display.c
> > b/drivers/gpu/drm/i915/intel_display.c
> > index bacf336..604186b 100644
> > --- a/drivers/gpu/drm/i915/intel_display.c
> > +++ b/drivers/gpu/drm/i915/intel_display.c
> > @@ -13347,6 +13347,17 @@ intel_prepare_plane_fb(struct drm_plane *plane,
> > if (!obj)
> > return 0;
> >
> > + /* For framebuffer backed by dmabuf, wait for fence */
> > + if (obj->base.dma_buf) {
> > + ret =
> > reservation_object_wait_timeout_rcu(obj->base.dma_buf->resv,
> > + false, true,
> > + MAX_SCHEDULE_TIMEOUT);
> > + if (ret == -ERESTARTSYS)
> > + return ret;
> > +
> > + WARN_ON(ret < 0);
> > + }
> > +
> > mutex_lock(&dev->struct_mutex);
> >
> > if (plane->type == DRM_PLANE_TYPE_CURSOR &&
> > --
> > 1.9.1
> >
> >
> > -----------------------------------------------------------------------------------
> > This email message is for the sole use of the intended recipient(s) and may
> > contain
> > confidential information. Any unauthorized review, use, disclosure or
> > distribution
> > is prohibited. If you are not the intended recipient, please contact the
> > sender by
> > reply email and destroy all copies of the original message.
> > -----------------------------------------------------------------------------------
>
> This disclaimer here pretty much tells me this isn't for public
> consumption and I can't merge this patch ... Would be good if you can make
> the final submission without this. Easiest way is usually to send the
> patches out over your private mail account (but with git author and sob
> still @nvidia.com).
Alternatively to the private account there's a way to prevent the
corporate email system from attaching the confidentiality statement.
I've sent instructions to Alex internally.
Thierry
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20151124/46b1477d/attachment.sig>
