Re: [PATCH 2/2] drm/fence: allow fence waiting to be interrupted by userspace

From: Gustavo Padovan
Date: Mon Aug 15 2016 - 10:45:55 EST


2016-08-15 Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx>:

> Op 11-08-16 om 20:39 schreef Gustavo Padovan:
> > From: Gustavo Padovan <gustavo.padovan@xxxxxxxxxxxxxxx>
> >
> > If userspace is running an synchronously atomic commit and interrupts the
> > atomic operation during fence_wait() it will hang until the timer expires,
> > so here we change the wait to be interruptible so it stop immediately when
> > userspace wants to quit.
> >
> > Also adds the necessary error checking for fence_wait().
> >
> > v2: Comment by Daniel Vetter
> > - Add error checking for fence_wait()
> >
> > v3: Rebase on top of new atomic noblocking support
> Meh, I don't like the swapped parameter much, couldn't we infer it from intr? or rename intr to swapped?

Definitely, I didn't realized it myself that both were saying the same
thing. Thanks for the suggesttion.

Gustavo